linux-kernel - Re: [PATCH 2.6.25-rc1] i2c-i801: SMBus patch for Intel ICH10 DeviceID's

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080212100125.2d2b0440@hyperion.delvare>
Date:	Tue, 12 Feb 2008 10:01:25 +0100
From:	Jean Delvare <khali@...ux-fr.org>
To:	"Gaston, Jason D" <jason.d.gaston@...el.com>
Cc:	<linux-kernel@...r.kernel.org>, Greg KH <gregkh@...e.de>,
	<i2c@...sensors.org>
Subject: Re: [PATCH 2.6.25-rc1] i2c-i801: SMBus patch for Intel ICH10
 DeviceID's

Hi Jason,

On Mon, 11 Feb 2008 16:45:17 -0800, Gaston, Jason D wrote:
> This patch adds the Intel ICH10 SMBus Controller DeviceID's and updates
> Tolapai support.
> 
> Signed-off-by:  Jason Gaston <jason.d.gaston@...el.com>

Patch applied. Please note that your e-mail client corrupted the patch
and I had to edit it manually before it would apply properly. Next time
please make sure that the patch is left unchanged so that it applies
directly.

> --- linux-2.6.25-rc1/Documentation/i2c/busses/i2c-i801.orig
> 2008-02-11 15:17:33.000000000 -0800
> +++ linux-2.6.25-rc1/Documentation/i2c/busses/i2c-i801	2008-02-11
> 15:18:13.000000000 -0800
> @@ -12,8 +12,9 @@
>    * Intel 82801G (ICH7)
>    * Intel 631xESB/632xESB (ESB2)
>    * Intel 82801H (ICH8)
> -  * Intel ICH9
> +  * Intel 82801I (ICH9)
>    * Intel Tolapai
> +  * Intel ICH10
>     Datasheets: Publicly available at the Intel website
>  
>  Authors: 
> --- linux-2.6.25-rc1/drivers/i2c/busses/i2c-i801.c.orig	2008-02-11
> 15:10:52.000000000 -0800
> +++ linux-2.6.25-rc1/drivers/i2c/busses/i2c-i801.c	2008-02-11
> 16:30:11.000000000 -0800
> @@ -40,7 +40,9 @@
>    82801G   (ICH7)       0x27da     32     hard     yes     yes     yes
>    82801H   (ICH8)       0x283e     32     hard     yes     yes     yes
>    82801I   (ICH9)       0x2930     32     hard     yes     yes     yes
> -  Tolapai               0x5032     32     hard     yes     ?       ?
> +  Tolapai               0x5032     32     hard     yes     yes     yes
> +  ICH10                 0x3a30     32     hard     yes     yes     yes
> +  ICH10                 0x3a60     32     hard     yes     yes     yes
>  
>    Features supported by this driver:
>    Software PEC                     no
> @@ -588,6 +590,8 @@
>  	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH8_5) },
>  	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH9_6) },
>  	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_TOLAPAI_1)
> },
> +	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH10_4)
> },
> +	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH10_5)
> },
>  	{ 0, }
>  };
>  
> @@ -608,10 +612,12 @@
>  	case PCI_DEVICE_ID_INTEL_ESB2_17:
>  	case PCI_DEVICE_ID_INTEL_ICH8_5:
>  	case PCI_DEVICE_ID_INTEL_ICH9_6:
> +	case PCI_DEVICE_ID_INTEL_TOLAPAI_1:
> +	case PCI_DEVICE_ID_INTEL_ICH10_4:
> +	case PCI_DEVICE_ID_INTEL_ICH10_5:
>  		i801_features |= FEATURE_I2C_BLOCK_READ;
>  		/* fall through */
>  	case PCI_DEVICE_ID_INTEL_82801DB_3:
> -	case PCI_DEVICE_ID_INTEL_TOLAPAI_1:
>  		i801_features |= FEATURE_SMBUS_PEC;
>  		i801_features |= FEATURE_BLOCK_BUFFER;
>  		break;
> --- linux-2.6.25-rc1/drivers/i2c/busses/Kconfig.orig	2008-02-11
> 15:10:18.000000000 -0800
> +++ linux-2.6.25-rc1/drivers/i2c/busses/Kconfig	2008-02-11
> 16:32:10.000000000 -0800
> @@ -177,6 +177,8 @@
>  	    ESB2
>  	    ICH8
>  	    ICH9
> +	    Tolapai
> +	    ICH10
>  
>  	  This driver can also be built as a module.  If so, the module
>  	  will be called i2c-i801.
> --- linux-2.6.25-rc1/include/linux/pci_ids.h.orig	2008-02-11
> 15:18:58.000000000 -0800
> +++ linux-2.6.25-rc1/include/linux/pci_ids.h	2008-02-11
> 15:20:45.000000000 -0800
> @@ -2373,6 +2373,12 @@
>  #define PCI_DEVICE_ID_INTEL_MCH_PC1	0x359a
>  #define PCI_DEVICE_ID_INTEL_E7525_MCH	0x359e
>  #define PCI_DEVICE_ID_INTEL_IOAT_CNB	0x360b
> +#define PCI_DEVICE_ID_INTEL_ICH10_0	0x3a14
> +#define PCI_DEVICE_ID_INTEL_ICH10_1	0x3a16
> +#define PCI_DEVICE_ID_INTEL_ICH10_2	0x3a18
> +#define PCI_DEVICE_ID_INTEL_ICH10_3	0x3a1a
> +#define PCI_DEVICE_ID_INTEL_ICH10_4	0x3a30
> +#define PCI_DEVICE_ID_INTEL_ICH10_5	0x3a60
>  #define PCI_DEVICE_ID_INTEL_IOAT_SNB	0x402f
>  #define PCI_DEVICE_ID_INTEL_IOAT_SCNB	0x65ff
>  #define PCI_DEVICE_ID_INTEL_TOLAPAI_0	0x5031

I left out this last part as it was already sent separately to the PCI
subsystem maintainer. I will wait for Greg to pick it before I send the
i2c-i801 update upstream.

Thanks,
-- 
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/