Re: [PATCH] i2c: amd8111: Switch to pcim_enable_device() and pcim_request_region()

From: Andi Shyti

Date: Tue Apr 14 2026 - 19:13:45 EST


Hi Filippo,

On Tue, Feb 24, 2026 at 11:22:16AM +0100, Filippo Muscherà wrote:
> Following the conversion to managed devm_* APIs, update the driver to use
> the PCI-specific managed APIs.
>
> Use pcim_enable_device() to properly enable the PCI device and
> pcim_request_region() to manage the I/O port region.
>
> Switching to pcim_enable_device() also addresses the fact that
> pci_disable_device() was missing in the driver lifecycle, as the
> managed API now automatically handles the disablement when the driver
> unbinds.
>
> Suggested-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
> Signed-off-by: Filippo Muscherà <filippo.muschera@xxxxxxxxx>

Please, don't send a new patch as a in-reply-to to a different
patch. I personally don't like patches sent as reply to a
different thread, it makes my work as reviewer more difficult.

Said that, I would ask Jean to give an ack here.

Thanks,
Andi