From mboxrd@z Thu Jan 1 00:00:00 1970 From: Neil Horman Subject: Re: [PATCH v2] i2c: ismt: Add Intel DNV PCI ID Date: Fri, 8 Apr 2016 10:17:15 -0400 Message-ID: <20160408141715.GC11476@hmsreliant.think-freely.org> References: <1460119268-23338-1-git-send-email-mika.westerberg@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from charlotte.tuxdriver.com ([70.61.120.58]:56107 "EHLO smtp.tuxdriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751812AbcDHORq (ORCPT ); Fri, 8 Apr 2016 10:17:46 -0400 Content-Disposition: inline In-Reply-To: <1460119268-23338-1-git-send-email-mika.westerberg@linux.intel.com> Sender: linux-i2c-owner@vger.kernel.org List-Id: linux-i2c@vger.kernel.org To: Mika Westerberg Cc: Wolfram Sang , Seth Heasley , scott.lawson@intel.com, alexandra.yates@intel.com, Andy Shevchenko , linux-i2c@vger.kernel.org On Fri, Apr 08, 2016 at 03:41:08PM +0300, Mika Westerberg wrote: > Intel DNV has the same iSMT SMBus host controller than Intel Avoton. Add > DNV PCI ID to the list of supported devices. > > Signed-off-by: Mika Westerberg > --- > Wolfram, > > This is the same patch that conflicted with another patch by Alexandra [1]. > We have now verified that it is the iSMT driver that is compatible with > this particular PCI ID (not i2c-i801.c). Not tested on a real hardware as I > do not have such machine available. > > v2: Order PCI IDs by their value > > [1] https://patchwork.ozlabs.org/patch/584758/ > > drivers/i2c/busses/i2c-ismt.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/i2c/busses/i2c-ismt.c b/drivers/i2c/busses/i2c-ismt.c > index 7ba795b24e75..1c8707710098 100644 > --- a/drivers/i2c/busses/i2c-ismt.c > +++ b/drivers/i2c/busses/i2c-ismt.c > @@ -75,6 +75,7 @@ > /* PCI DIDs for the Intel SMBus Message Transport (SMT) Devices */ > #define PCI_DEVICE_ID_INTEL_S1200_SMT0 0x0c59 > #define PCI_DEVICE_ID_INTEL_S1200_SMT1 0x0c5a > +#define PCI_DEVICE_ID_INTEL_DNV_SMT 0x19ac > #define PCI_DEVICE_ID_INTEL_AVOTON_SMT 0x1f15 > > #define ISMT_DESC_ENTRIES 2 /* number of descriptor entries */ > @@ -180,6 +181,7 @@ struct ismt_priv { > static const struct pci_device_id ismt_ids[] = { > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_S1200_SMT0) }, > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_S1200_SMT1) }, > + { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_DNV_SMT) }, > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_AVOTON_SMT) }, > { 0, } > }; > -- > 2.8.0.rc3 > > Acked-by: Neil Horman