diff options
| author |   Jean Delvare <jdelvare@suse.de> | 2018-04-11 18:05:34 +0200 |
|---|---|---|
| committer |   Wolfram Sang <wsa@the-dreams.de> | 2018-04-11 23:08:13 +0200 |
| commit | f7f6d915a10f7f2bce17e3b1b7d3376562395a28 (patch) | |
| tree | 16fe45f9a16a4827353f515a50bbbceac1657d17 /drivers/i2c | |
| parent | i2c: i801: Save register SMBSLVCMD value only once (diff) | |
| download | linux-dev-f7f6d915a10f7f2bce17e3b1b7d3376562395a28.tar.xz linux-dev-f7f6d915a10f7f2bce17e3b1b7d3376562395a28.zip | |
i2c: i801: Restore configuration at shutdown
On some systems, the BIOS expects certain SMBus register values to
match the hardware defaults. Restore these configuration registers at
shutdown time to avoid confusing the BIOS. This avoids hard-locking
such systems upon reboot.
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Tested-by: Jason Andryuk <jandryuk@gmail.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Cc: stable@vger.kernel.org
Diffstat (limited to 'drivers/i2c')
| -rw-r--r-- | drivers/i2c/busses/i2c-i801.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c index 17aa20a55642..e0d59e9ff3c6 100644 --- a/drivers/i2c/busses/i2c-i801.c +++ b/drivers/i2c/busses/i2c-i801.c @@ -1701,6 +1701,15 @@ static void i801_remove(struct pci_dev *dev) */ } +static void i801_shutdown(struct pci_dev *dev) +{ + struct i801_priv *priv = pci_get_drvdata(dev); + + /* Restore config registers to avoid hard hang on some systems */ + i801_disable_host_notify(priv); + pci_write_config_byte(dev, SMBHSTCFG, priv->original_hstcfg); +} + #ifdef CONFIG_PM static int i801_suspend(struct device *dev) { @@ -1730,6 +1739,7 @@ static struct pci_driver i801_driver = { .id_table = i801_ids, .probe = i801_probe, .remove = i801_remove, + .shutdown = i801_shutdown, .driver = { .pm = &i801_pm_ops, }, |