Hi Alexandra, On Fri, 5 Feb 2016 15:22:44 -0800, Alexandra Yates wrote: > Changes log v2 to v3: Clarified commit message. No changes in code. > > Starting from Intel Sunrisepoint (Skylake PCH) the iTCO watchdog > resources have been moved to reside under the i801 SMBus host > controller whereas previously they were under the LPC device. > > This patch adds Intel lewisburg SMBus support for iTCO device. > It allows to load watchdog dynamically when the hardware is > present. This patch is an addition > to cdc5a3110e7c3ae793f367285789a6bc39c962dc i2c: i801: "add > Intel Lewisburg device IDs". > > Signed-off-by: Alexandra Yates <[email protected]> > --- > drivers/i2c/busses/i2c-i801.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c > index f62d697..27fa0cb 100644 > --- a/drivers/i2c/busses/i2c-i801.c > +++ b/drivers/i2c/busses/i2c-i801.c > @@ -1271,6 +1271,8 @@ static int i801_probe(struct pci_dev *dev, const struct > pci_device_id *id) > switch (dev->device) { > case PCI_DEVICE_ID_INTEL_SUNRISEPOINT_H_SMBUS: > case PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SMBUS: > + case PCI_DEVICE_ID_INTEL_LEWISBURG_SMBUS: > + case PCI_DEVICE_ID_INTEL_LEWISBURG_SSKU_SMBUS: > case PCI_DEVICE_ID_INTEL_DNV_SMBUS: > priv->features |= FEATURE_I2C_BLOCK_READ; > priv->features |= FEATURE_IRQ;
Reviewed-by: Jean Delvare <[email protected]> Note this checkpatch warning though: ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 0123456789ab ("commit description")' #40: to cdc5a3110e7c3ae793f367285789a6bc39c962dc i2c: i801: "add It is also common to add a "Fixes:" line with that information so that patch backporting can be automated. And as I said before I would tag the patch for stable as well. -- Jean Delvare SUSE L3 Support
