SMBUS: Add DeviceIDs for SunrisePoint LP

PATCH Applied Devin Ryles 3 participants 5 messages Nov 7, 2014 – Nov 12, 2014

Subsystems (3)

I2C subsystem host drivers I2C I2C/smbus controller drivers for pc

lore

Share & tools

lore

Devin Ryles

author

2014-11-07 22:55 UTC #0 lore raw

PATCH 1/3 SMBUS: Add DeviceIDs for SunrisePoint LP

Quoted 1 line

From 8386c61977dce52fcdfe7fb279692606f47650bc Mon Sep 17 00:00:00 2001

From: Devin Ryles <[email protected]>
Date: Wed, 5 Nov 2014 16:30:03 -0500
Subject: [PATCH 1/3] SMBUS: Add DeviceIDs for SunrisePoint LP

This patch adds the DeviceIDs for SunrisePoint LP

Signed-off-by Devin Ryles <[email protected]>

---
 Documentation/i2c/busses/i2c-i801 | 1 +
 drivers/i2c/busses/Kconfig        | 1 +
 drivers/i2c/busses/i2c-i801.c     | 3 +++
 3 files changed, 5 insertions(+)

(quoted diff) +0 -0

b/Documentation/i2c/busses/i2c-i801
index 793c83d..82f48f7 100644

Documentation/i2c/busses/i2c-i801 +1 -0

@@ -29,6 +29,7 @@ Supported adapters:
   * Intel Wildcat Point-LP (PCH)
   * Intel BayTrail (SOC)
   * Intel Sunrise Point-H (PCH)
+  * Intel Sunrise Point-LP (PCH)
    Datasheets: Publicly available at the Intel website
 
 On Intel Patsburg and later chipsets, both the normal host SMBus

controller

drivers/i2c/busses/Kconfig +1 -0

index 917c358..06e99eb 100644
@@ -123,6 +123,7 @@ config I2C_I801
 	    Wildcat Point-LP (PCH)
 	    BayTrail (SOC)
 	    Sunrise Point-H (PCH)
+	    Sunrise Point-LP (PCH)
 
 	  This driver can also be built as a module.  If so, the module
 	  will be called i2c-i801.

(quoted diff) +0 -0

b/drivers/i2c/busses/i2c-i801.c
index 7cfc183..e158e18 100644

drivers/i2c/busses/i2c-i801.c +2 -0

@@ -63,6 +63,7 @@
  * Wildcat Point-LP (PCH)	0x9ca2	32	hard	yes	yes	yes
  * BayTrail (SOC)		0x0f12	32	hard	yes	yes	yes
  * Sunrise Point-H (PCH) 	0xa123  32	hard	yes	yes	yes
+ * Sunrise Point-LP (PCH)	0x9d23	32	hard	yes	yes	yes
  *
  * Features supported by this driver:
  * Software PEC				no

@@ -186,6 +187,7 @@
 #define PCI_DEVICE_ID_INTEL_LYNXPOINT_LP_SMBUS		0x9c22
 #define PCI_DEVICE_ID_INTEL_WILDCATPOINT_LP_SMBUS	0x9ca2
 #define PCI_DEVICE_ID_INTEL_SUNRISEPOINT_H_SMBUS	0xa123
+#define PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SMBUS	0x9d23
 
 struct i801_mux_config {
 	char *gpio_chip;

@@ -833,6 +835,7 @@ static const struct pci_device_id i801_ids[] = {
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL,

PCI_DEVICE_ID_INTEL_BAYTRAIL_SMBUS) },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL,
PCI_DEVICE_ID_INTEL_BRASWELL_SMBUS) },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL,
PCI_DEVICE_ID_INTEL_SUNRISEPOINT_H_SMBUS) },
+ { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SMBUS) },
{ 0, }
};

Signature · 1 line

1.9.3

Jean Delvare

2014-11-10 06:45 UTC #1 lore raw

Hi Devin,

Quoted 69 lines · Devin Ryles

From 8386c61977dce52fcdfe7fb279692606f47650bc Mon Sep 17 00:00:00 2001
From: Devin Ryles <[email protected]>
Date: Wed, 5 Nov 2014 16:30:03 -0500
Subject: [PATCH 1/3] SMBUS: Add DeviceIDs for SunrisePoint LP

This patch adds the DeviceIDs for SunrisePoint LP

Signed-off-by Devin Ryles <[email protected]>

---
 Documentation/i2c/busses/i2c-i801 | 1 +
 drivers/i2c/busses/Kconfig        | 1 +
 drivers/i2c/busses/i2c-i801.c     | 3 +++
 3 files changed, 5 insertions(+)

Documentation/i2c/busses/i2c-i801 +1 -0

index 793c83d..82f48f7 100644
@@ -29,6 +29,7 @@ Supported adapters:
   * Intel Wildcat Point-LP (PCH)
   * Intel BayTrail (SOC)
   * Intel Sunrise Point-H (PCH)
+  * Intel Sunrise Point-LP (PCH)
    Datasheets: Publicly available at the Intel website
 
 On Intel Patsburg and later chipsets, both the normal host SMBus

controller

drivers/i2c/busses/Kconfig +1 -0

index 917c358..06e99eb 100644
@@ -123,6 +123,7 @@ config I2C_I801
 	    Wildcat Point-LP (PCH)
 	    BayTrail (SOC)
 	    Sunrise Point-H (PCH)
+	    Sunrise Point-LP (PCH)
 
 	  This driver can also be built as a module.  If so, the module
 	  will be called i2c-i801.

drivers/i2c/busses/i2c-i801.c +3 -0

index 7cfc183..e158e18 100644
@@ -63,6 +63,7 @@
  * Wildcat Point-LP (PCH)	0x9ca2	32	hard	yes	yes	yes
  * BayTrail (SOC)		0x0f12	32	hard	yes	yes	yes
  * Sunrise Point-H (PCH) 	0xa123  32	hard	yes	yes	yes
+ * Sunrise Point-LP (PCH)	0x9d23	32	hard	yes	yes	yes
  *
  * Features supported by this driver:
  * Software PEC				no

@@ -186,6 +187,7 @@
 #define PCI_DEVICE_ID_INTEL_LYNXPOINT_LP_SMBUS		0x9c22
 #define PCI_DEVICE_ID_INTEL_WILDCATPOINT_LP_SMBUS	0x9ca2
 #define PCI_DEVICE_ID_INTEL_SUNRISEPOINT_H_SMBUS	0xa123
+#define PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SMBUS	0x9d23
 
 struct i801_mux_config {
 	char *gpio_chip;

@@ -833,6 +835,7 @@ static const struct pci_device_id i801_ids[] = {
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL,
 PCI_DEVICE_ID_INTEL_BAYTRAIL_SMBUS) },
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL,
 PCI_DEVICE_ID_INTEL_BRASWELL_SMBUS) },
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL,
 PCI_DEVICE_ID_INTEL_SUNRISEPOINT_H_SMBUS) },
+	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL,

Quoted 4 lines

PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SMBUS) },
{ 0, }
};

This looks good, however your e-mail client wrapped long lines so the patch can't be applied. Please reconfigure your client to not do that and send the patches again. In Evolution, you can select the entire contents of the patch and set the formatting to "Pre-formatted", that should work. See Documentation/email-clients.txt for more details. If you're not sure, try sending the message to yourself once first to make sure the formatting is good.

Thanks,

Signature · 2 lines

Jean Delvare
SUSE L3 Support

Devin Ryles

author

2014-11-10 17:13 UTC #2 lore raw

PATCH 1/1 SMBUS: Add DeviceIDs for SunrisePoint LP

Quoted 1 line

From 8386c61977dce52fcdfe7fb279692606f47650bc Mon Sep 17 00:00:00 2001

From: Devin Ryles <[email protected]>
Date: Wed, 5 Nov 2014 16:30:03 -0500
Subject: [PATCH] SMBUS: Add DeviceIDs for SunrisePoint LP

This patch adds the DeviceIDs for SunrisePoint LP

Signed-off-by Devin Ryles <[email protected]>

---
 Documentation/i2c/busses/i2c-i801 | 1 +
 drivers/i2c/busses/Kconfig        | 1 +
 drivers/i2c/busses/i2c-i801.c     | 3 +++
 3 files changed, 5 insertions(+)

Documentation/i2c/busses/i2c-i801 +1 -0

index 793c83d..82f48f7 100644
@@ -29,6 +29,7 @@ Supported adapters:
   * Intel Wildcat Point-LP (PCH)
   * Intel BayTrail (SOC)
   * Intel Sunrise Point-H (PCH)
+  * Intel Sunrise Point-LP (PCH)
    Datasheets: Publicly available at the Intel website
 
 On Intel Patsburg and later chipsets, both the normal host SMBus controller

drivers/i2c/busses/Kconfig +1 -0

index 917c358..06e99eb 100644
@@ -123,6 +123,7 @@ config I2C_I801
 	    Wildcat Point-LP (PCH)
 	    BayTrail (SOC)
 	    Sunrise Point-H (PCH)
+	    Sunrise Point-LP (PCH)
 
 	  This driver can also be built as a module.  If so, the module
 	  will be called i2c-i801.

drivers/i2c/busses/i2c-i801.c +3 -0

index 7cfc183..e158e18 100644
@@ -63,6 +63,7 @@
  * Wildcat Point-LP (PCH)	0x9ca2	32	hard	yes	yes	yes
  * BayTrail (SOC)		0x0f12	32	hard	yes	yes	yes
  * Sunrise Point-H (PCH) 	0xa123  32	hard	yes	yes	yes
+ * Sunrise Point-LP (PCH)	0x9d23	32	hard	yes	yes	yes
  *
  * Features supported by this driver:
  * Software PEC				no

@@ -186,6 +187,7 @@
 #define PCI_DEVICE_ID_INTEL_LYNXPOINT_LP_SMBUS		0x9c22
 #define PCI_DEVICE_ID_INTEL_WILDCATPOINT_LP_SMBUS	0x9ca2
 #define PCI_DEVICE_ID_INTEL_SUNRISEPOINT_H_SMBUS	0xa123
+#define PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SMBUS	0x9d23
 
 struct i801_mux_config {
 	char *gpio_chip;

@@ -833,6 +835,7 @@ static const struct pci_device_id i801_ids[] = {
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_BAYTRAIL_SMBUS) },
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_BRASWELL_SMBUS) },
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_SUNRISEPOINT_H_SMBUS) },
+	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SMBUS) },
 	{ 0, }
 };

Signature · 1 line

1.9.3

Jean Delvare

Reviewed-by on v1 1/1

2014-11-10 17:54 UTC #3 lore raw

Quoted 59 lines · Devin Ryles

From 8386c61977dce52fcdfe7fb279692606f47650bc Mon Sep 17 00:00:00 2001
From: Devin Ryles <[email protected]>
Date: Wed, 5 Nov 2014 16:30:03 -0500
Subject: [PATCH] SMBUS: Add DeviceIDs for SunrisePoint LP

This patch adds the DeviceIDs for SunrisePoint LP

Signed-off-by Devin Ryles <[email protected]>

---
 Documentation/i2c/busses/i2c-i801 | 1 +
 drivers/i2c/busses/Kconfig        | 1 +
 drivers/i2c/busses/i2c-i801.c     | 3 +++
 3 files changed, 5 insertions(+)

Documentation/i2c/busses/i2c-i801 +1 -0

index 793c83d..82f48f7 100644
@@ -29,6 +29,7 @@ Supported adapters:
   * Intel Wildcat Point-LP (PCH)
   * Intel BayTrail (SOC)
   * Intel Sunrise Point-H (PCH)
+  * Intel Sunrise Point-LP (PCH)
    Datasheets: Publicly available at the Intel website
 
 On Intel Patsburg and later chipsets, both the normal host SMBus controller

drivers/i2c/busses/Kconfig +1 -0

index 917c358..06e99eb 100644
@@ -123,6 +123,7 @@ config I2C_I801
 	    Wildcat Point-LP (PCH)
 	    BayTrail (SOC)
 	    Sunrise Point-H (PCH)
+	    Sunrise Point-LP (PCH)
 
 	  This driver can also be built as a module.  If so, the module
 	  will be called i2c-i801.

drivers/i2c/busses/i2c-i801.c +2 -0

index 7cfc183..e158e18 100644
@@ -63,6 +63,7 @@
  * Wildcat Point-LP (PCH)	0x9ca2	32	hard	yes	yes	yes
  * BayTrail (SOC)		0x0f12	32	hard	yes	yes	yes
  * Sunrise Point-H (PCH) 	0xa123  32	hard	yes	yes	yes
+ * Sunrise Point-LP (PCH)	0x9d23	32	hard	yes	yes	yes
  *
  * Features supported by this driver:
  * Software PEC				no

@@ -186,6 +187,7 @@
 #define PCI_DEVICE_ID_INTEL_LYNXPOINT_LP_SMBUS		0x9c22
 #define PCI_DEVICE_ID_INTEL_WILDCATPOINT_LP_SMBUS	0x9ca2
 #define PCI_DEVICE_ID_INTEL_SUNRISEPOINT_H_SMBUS	0xa123
+#define PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SMBUS	0x9d23
 
 struct i801_mux_config {
 	char *gpio_chip;

drivers/i2c/busses/i2c-i801.c static const struct pci_device_id i801_ids[] = { · line 841

Quoted 7 lines

drivers/i2c/busses/i2c-i801.c +1 -0

@@ -833,6 +835,7 @@ static const struct pci_device_id i801_ids[] = {
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_BAYTRAIL_SMBUS) },
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_BRASWELL_SMBUS) },
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_SUNRISEPOINT_H_SMBUS) },
+	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SMBUS) },
 	{ 0, }
 };

Reviewed-by Jean Delvare <[email protected]>

Thanks,

Signature · 2 lines

Jean Delvare
SUSE L3 Support

Wolfram Sang

Applied

2014-11-12 16:14 UTC #4 lore raw

In reply to Devin Ryles

Quoted 8 lines · Devin Ryles

From 8386c61977dce52fcdfe7fb279692606f47650bc Mon Sep 17 00:00:00 2001
From: Devin Ryles <[email protected]>
Date: Wed, 5 Nov 2014 16:30:03 -0500
Subject: [PATCH] SMBUS: Add DeviceIDs for SunrisePoint LP
This patch adds the DeviceIDs for SunrisePoint LP
Signed-off-by Devin Ryles <[email protected]>

Applied to for-next, thanks!