From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: Re: [PATCH] i2c: i801: Add support for Intel Gemini Lake Date: Wed, 1 Feb 2017 11:38:32 +0100 Message-ID: <20170201113832.647a1f3e@endymion> References: <20170131101733.117705-1-mika.westerberg@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de ([195.135.220.15]:45598 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751202AbdBAKig (ORCPT ); Wed, 1 Feb 2017 05:38:36 -0500 In-Reply-To: <20170131101733.117705-1-mika.westerberg@linux.intel.com> Sender: linux-i2c-owner@vger.kernel.org List-Id: linux-i2c@vger.kernel.org To: Mika Westerberg Cc: Jean Delvare , Wolfram Sang , Jarkko Nikula , linux-i2c@vger.kernel.org Hi Mika, On Tue, 31 Jan 2017 13:17:33 +0300, Mika Westerberg wrote: > Intel Gemini Lake has the same SMBus host controller than Intel Broxton. > > Signed-off-by: Mika Westerberg > --- > drivers/i2c/busses/i2c-i801.c | 3 +++ > 1 file changed, 3 insertions(+) Please also update drivers/i2c/busses/Kconfig and Documentation/i2c/busses/i2c-i801. Also please submit the new device to http://pci-ids.ucw.cz/read/PC/8086. > > diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c > index e242db43774b..6484fa6dbb84 100644 > --- a/drivers/i2c/busses/i2c-i801.c > +++ b/drivers/i2c/busses/i2c-i801.c > @@ -65,6 +65,7 @@ > * Lewisburg (PCH) 0xa1a3 32 hard yes yes yes > * Lewisburg Supersku (PCH) 0xa223 32 hard yes yes yes > * Kaby Lake PCH-H (PCH) 0xa2a3 32 hard yes yes yes > + * Gemini Lake (SOC) 0x31d4 32 hard yes yes yes > * > * Features supported by this driver: > * Software PEC no > @@ -213,6 +214,7 @@ > #define PCI_DEVICE_ID_INTEL_BRASWELL_SMBUS 0x2292 > #define PCI_DEVICE_ID_INTEL_DH89XXCC_SMBUS 0x2330 > #define PCI_DEVICE_ID_INTEL_COLETOCREEK_SMBUS 0x23b0 > +#define PCI_DEVICE_ID_INTEL_GEMINILAKE_SMBUS 0x31d4 > #define PCI_DEVICE_ID_INTEL_5_3400_SERIES_SMBUS 0x3b30 > #define PCI_DEVICE_ID_INTEL_BROXTON_SMBUS 0x5ad4 > #define PCI_DEVICE_ID_INTEL_LYNXPOINT_SMBUS 0x8c22 > @@ -1012,6 +1014,7 @@ static const struct pci_device_id i801_ids[] = { > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_WELLSBURG_SMBUS_MS1) }, > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_WELLSBURG_SMBUS_MS2) }, > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_COLETOCREEK_SMBUS) }, > + { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_GEMINILAKE_SMBUS) }, > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_WILDCATPOINT_SMBUS) }, > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_WILDCATPOINT_LP_SMBUS) }, > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_BAYTRAIL_SMBUS) }, Signed-off-by: Jean Delvare Thanks, -- Jean Delvare SUSE L3 Support