| Message ID | 20210625151758.12180-1-wsa@kernel.org |
|---|---|
| State | Accepted |
| Headers | show |
| Series | i2c: ali1535: mention that the device should not be disabled | expand |
On Fri, 25 Jun 2021 17:17:58 +0200, Wolfram Sang wrote: > The comment from the i801 driver is valid here, too, so copy it. > > Reported-by: Jean Delvare <jdelvare@suse.de> > Signed-off-by: Wolfram Sang <wsa@kernel.org> > --- > drivers/i2c/busses/i2c-ali1535.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/i2c/busses/i2c-ali1535.c b/drivers/i2c/busses/i2c-ali1535.c > index fb93152845f4..ee83c4581bce 100644 > --- a/drivers/i2c/busses/i2c-ali1535.c > +++ b/drivers/i2c/busses/i2c-ali1535.c > @@ -508,6 +508,11 @@ static void ali1535_remove(struct pci_dev *dev) > { > i2c_del_adapter(&ali1535_adapter); > release_region(ali1535_smba, ALI1535_SMB_IOSIZE); > + > + /* > + * do not call pci_disable_device(dev) since it can cause hard hangs on > + * some systems during power-off > + */ > } > > static struct pci_driver ali1535_driver = { Can't hurt. Reviewed-by: Jean Delvare <jdelvare@suse.de>
On Fri, Jun 25, 2021 at 05:17:58PM +0200, Wolfram Sang wrote: > The comment from the i801 driver is valid here, too, so copy it. > > Reported-by: Jean Delvare <jdelvare@suse.de> > Signed-off-by: Wolfram Sang <wsa@kernel.org> Applied to for-current, thanks!
diff --git a/drivers/i2c/busses/i2c-ali1535.c b/drivers/i2c/busses/i2c-ali1535.c index fb93152845f4..ee83c4581bce 100644 --- a/drivers/i2c/busses/i2c-ali1535.c +++ b/drivers/i2c/busses/i2c-ali1535.c @@ -508,6 +508,11 @@ static void ali1535_remove(struct pci_dev *dev) { i2c_del_adapter(&ali1535_adapter); release_region(ali1535_smba, ALI1535_SMB_IOSIZE); + + /* + * do not call pci_disable_device(dev) since it can cause hard hangs on + * some systems during power-off + */ } static struct pci_driver ali1535_driver = {
The comment from the i801 driver is valid here, too, so copy it. Reported-by: Jean Delvare <jdelvare@suse.de> Signed-off-by: Wolfram Sang <wsa@kernel.org> --- drivers/i2c/busses/i2c-ali1535.c | 5 +++++ 1 file changed, 5 insertions(+)