* [PATCH 2.6.32.3] i2c-i801: SMBus patch for Intel Cougar Point DeviceIDs
@ 2010-01-13 0:58 Seth Heasley
2010-01-15 18:15 ` Jesse Barnes
2010-01-19 11:00 ` Jean Delvare
0 siblings, 2 replies; 4+ messages in thread
From: Seth Heasley @ 2010-01-13 0:58 UTC (permalink / raw)
To: linux-kernel, khali, i2c, linux-pci, jbarnes; +Cc: seth.heasley
This patch adds the Intel Cougar Point (PCH) SMBus Controller DeviceIDs.
Signed-off-by: Seth Heasley <seth.heasley@intel.com>
--- linux-2.6.32.3/drivers/i2c/busses/i2c-i801.c.orig 2010-01-06 15:07:45.000000000 -0800
+++ linux-2.6.32.3/drivers/i2c/busses/i2c-i801.c 2010-01-07 13:10:28.000000000 -0800
@@ -41,7 +41,8 @@
Tolapai 0x5032 32 hard yes yes yes
ICH10 0x3a30 32 hard yes yes yes
ICH10 0x3a60 32 hard yes yes yes
- PCH 0x3b30 32 hard yes yes yes
+ 3400/5 Series (PCH) 0x3b30 32 hard yes yes yes
+ CPT 0x1c22 32 hard yes yes yes
Features supported by this driver:
Software PEC no
@@ -578,6 +579,7 @@
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH10_4) },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH10_5) },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_PCH_SMBUS) },
+ { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_CPT_SMBUS) },
{ 0, }
};
@@ -707,6 +709,7 @@
case PCI_DEVICE_ID_INTEL_ICH10_4:
case PCI_DEVICE_ID_INTEL_ICH10_5:
case PCI_DEVICE_ID_INTEL_PCH_SMBUS:
+ case PCI_DEVICE_ID_INTEL_CPT_SMBUS:
i801_features |= FEATURE_I2C_BLOCK_READ;
/* fall through */
case PCI_DEVICE_ID_INTEL_82801DB_3:
--- linux-2.6.32.3/drivers/i2c/busses/Kconfig.orig 2010-01-06 15:07:45.000000000 -0800
+++ linux-2.6.32.3/drivers/i2c/busses/Kconfig 2010-01-07 13:10:28.000000000 -0800
@@ -98,6 +98,7 @@
Tolapai
ICH10
PCH
+ CPT
This driver can also be built as a module. If so, the module
will be called i2c-i801.
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH 2.6.32.3] i2c-i801: SMBus patch for Intel Cougar Point DeviceIDs
2010-01-13 0:58 [PATCH 2.6.32.3] i2c-i801: SMBus patch for Intel Cougar Point DeviceIDs Seth Heasley
@ 2010-01-15 18:15 ` Jesse Barnes
2010-01-19 10:19 ` Jean Delvare
2010-01-19 11:00 ` Jean Delvare
1 sibling, 1 reply; 4+ messages in thread
From: Jesse Barnes @ 2010-01-15 18:15 UTC (permalink / raw)
To: Seth Heasley
Cc: linux-kernel, khali, i2c, linux-pci, seth.heasley,
Jean Delvare (PC drivers, core), Ben Dooks (embedded platforms)
Seems like Jean or Ben should take this.
Jesse
On Tue, 12 Jan 2010 16:58:53 -0800
Seth Heasley <seth.heasley@intel.com> wrote:
> This patch adds the Intel Cougar Point (PCH) SMBus Controller
> DeviceIDs.
>
> Signed-off-by: Seth Heasley <seth.heasley@intel.com>
>
> --- linux-2.6.32.3/drivers/i2c/busses/i2c-i801.c.orig
> 2010-01-06 15:07:45.000000000 -0800 +++
> linux-2.6.32.3/drivers/i2c/busses/i2c-i801.c 2010-01-07
> 13:10:28.000000000 -0800 @@ -41,7 +41,8 @@ Tolapai
> 0x5032 32 hard yes yes yes ICH10
> 0x3a30 32 hard yes yes yes ICH10
> 0x3a60 32 hard yes yes yes
> - PCH 0x3b30 32 hard yes yes
> yes
> + 3400/5 Series (PCH) 0x3b30 32 hard yes yes
> yes
> + CPT 0x1c22 32 hard yes yes
> yes
> Features supported by this driver:
> Software PEC no
> @@ -578,6 +579,7 @@
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
> PCI_DEVICE_ID_INTEL_ICH10_4) }, { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
> PCI_DEVICE_ID_INTEL_ICH10_5) }, { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
> PCI_DEVICE_ID_INTEL_PCH_SMBUS) },
> + { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
> PCI_DEVICE_ID_INTEL_CPT_SMBUS) }, { 0, }
> };
>
> @@ -707,6 +709,7 @@
> case PCI_DEVICE_ID_INTEL_ICH10_4:
> case PCI_DEVICE_ID_INTEL_ICH10_5:
> case PCI_DEVICE_ID_INTEL_PCH_SMBUS:
> + case PCI_DEVICE_ID_INTEL_CPT_SMBUS:
> i801_features |= FEATURE_I2C_BLOCK_READ;
> /* fall through */
> case PCI_DEVICE_ID_INTEL_82801DB_3:
> --- linux-2.6.32.3/drivers/i2c/busses/Kconfig.orig 2010-01-06
> 15:07:45.000000000 -0800 +++
> linux-2.6.32.3/drivers/i2c/busses/Kconfig 2010-01-07
> 13:10:28.000000000 -0800 @@ -98,6 +98,7 @@ Tolapai
> ICH10
> PCH
> + CPT
>
> This driver can also be built as a module. If so, the
> module will be called i2c-i801.
>
--
Jesse Barnes, Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH 2.6.32.3] i2c-i801: SMBus patch for Intel Cougar Point DeviceIDs
2010-01-13 0:58 [PATCH 2.6.32.3] i2c-i801: SMBus patch for Intel Cougar Point DeviceIDs Seth Heasley
2010-01-15 18:15 ` Jesse Barnes
@ 2010-01-19 11:00 ` Jean Delvare
1 sibling, 0 replies; 4+ messages in thread
From: Jean Delvare @ 2010-01-19 11:00 UTC (permalink / raw)
To: Seth Heasley; +Cc: linux-kernel, Linux I2C, linux-pci, jbarnes
Hi Seth,
Please note that the i2c mailing list address has changed.
On Tue, 12 Jan 2010 16:58:53 -0800, Seth Heasley wrote:
> This patch adds the Intel Cougar Point (PCH) SMBus Controller DeviceIDs.
This comment suggests that Cougar Point == PCH, while the comments in
i2c-i801 suggests otherwise. Who is right?
>
> Signed-off-by: Seth Heasley <seth.heasley@intel.com>
>
> --- linux-2.6.32.3/drivers/i2c/busses/i2c-i801.c.orig 2010-01-06 15:07:45.000000000 -0800
> +++ linux-2.6.32.3/drivers/i2c/busses/i2c-i801.c 2010-01-07 13:10:28.000000000 -0800
> @@ -41,7 +41,8 @@
> Tolapai 0x5032 32 hard yes yes yes
> ICH10 0x3a30 32 hard yes yes yes
> ICH10 0x3a60 32 hard yes yes yes
> - PCH 0x3b30 32 hard yes yes yes
> + 3400/5 Series (PCH) 0x3b30 32 hard yes yes yes
> + CPT 0x1c22 32 hard yes yes yes
>
> Features supported by this driver:
> Software PEC no
> @@ -578,6 +579,7 @@
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH10_4) },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH10_5) },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_PCH_SMBUS) },
> + { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_CPT_SMBUS) },
> { 0, }
> };
>
> @@ -707,6 +709,7 @@
> case PCI_DEVICE_ID_INTEL_ICH10_4:
> case PCI_DEVICE_ID_INTEL_ICH10_5:
> case PCI_DEVICE_ID_INTEL_PCH_SMBUS:
> + case PCI_DEVICE_ID_INTEL_CPT_SMBUS:
> i801_features |= FEATURE_I2C_BLOCK_READ;
> /* fall through */
> case PCI_DEVICE_ID_INTEL_82801DB_3:
> --- linux-2.6.32.3/drivers/i2c/busses/Kconfig.orig 2010-01-06 15:07:45.000000000 -0800
> +++ linux-2.6.32.3/drivers/i2c/busses/Kconfig 2010-01-07 13:10:28.000000000 -0800
> @@ -98,6 +98,7 @@
> Tolapai
> ICH10
> PCH
> + CPT
It might be clear for you that "CPT" means "Cougar Point", but for the
reader it may not be. I'd rather list the full chip name. Same in
i2c-i801.c's header comment, BTW.
>
> This driver can also be built as a module. If so, the module
> will be called i2c-i801.
Documentation/i2c/busses/i2c-i801 needs to be updated as well.
--
Jean Delvare
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-01-19 11:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-13 0:58 [PATCH 2.6.32.3] i2c-i801: SMBus patch for Intel Cougar Point DeviceIDs Seth Heasley
2010-01-15 18:15 ` Jesse Barnes
2010-01-19 10:19 ` Jean Delvare
2010-01-19 11:00 ` Jean Delvare
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox