* [PATCH v2] i2c: ismt: Add Intel DNV PCI ID
@ 2016-04-08 12:41 Mika Westerberg
2016-04-08 14:17 ` Neil Horman
2016-04-12 21:45 ` Wolfram Sang
0 siblings, 2 replies; 3+ messages in thread
From: Mika Westerberg @ 2016-04-08 12:41 UTC (permalink / raw)
To: Wolfram Sang
Cc: Seth Heasley, Neil Horman, scott.lawson, alexandra.yates,
Andy Shevchenko, Mika Westerberg, linux-i2c
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 <mika.westerberg@linux.intel.com>
---
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
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] i2c: ismt: Add Intel DNV PCI ID
2016-04-08 12:41 [PATCH v2] i2c: ismt: Add Intel DNV PCI ID Mika Westerberg
@ 2016-04-08 14:17 ` Neil Horman
2016-04-12 21:45 ` Wolfram Sang
1 sibling, 0 replies; 3+ messages in thread
From: Neil Horman @ 2016-04-08 14:17 UTC (permalink / raw)
To: Mika Westerberg
Cc: Wolfram Sang, Seth Heasley, scott.lawson, alexandra.yates,
Andy Shevchenko, linux-i2c
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 <mika.westerberg@linux.intel.com>
> ---
> 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 <nhorman@tuxdriver.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] i2c: ismt: Add Intel DNV PCI ID
2016-04-08 12:41 [PATCH v2] i2c: ismt: Add Intel DNV PCI ID Mika Westerberg
2016-04-08 14:17 ` Neil Horman
@ 2016-04-12 21:45 ` Wolfram Sang
1 sibling, 0 replies; 3+ messages in thread
From: Wolfram Sang @ 2016-04-12 21:45 UTC (permalink / raw)
To: Mika Westerberg
Cc: Seth Heasley, Neil Horman, scott.lawson, alexandra.yates,
Andy Shevchenko, linux-i2c
[-- Attachment #1: Type: text/plain, Size: 299 bytes --]
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 <mika.westerberg@linux.intel.com>
Applied to for-current, thanks!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-04-12 21:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-08 12:41 [PATCH v2] i2c: ismt: Add Intel DNV PCI ID Mika Westerberg
2016-04-08 14:17 ` Neil Horman
2016-04-12 21:45 ` Wolfram Sang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.