Re: [PATCH] EDAC/i3200: drop the PCI device reference on module exit

[Borislav Petkov]

On Wed, Nov 04, 2026 at 03:38:33AM +0000, Wentao Liang wrote:
> i3200_init_one() takes an extra reference on the memory controller with
> pci_dev_get() and stores it in the global mci_pdev pointer.  When the
> driver probed a device, i3200_registered stays set and i3200_exit()
> calls pci_dev_put() only in the !i3200_registered branch, so the
> reference is never dropped and the PCI device can not be released.
> 
> Move the put out of the conditional.  i3200_remove_one() still has to be
> called by hand only for the i3200_init() fallback that probed the device
> outside the PCI driver core.
> 
> Fixes: dd8ef1db87a4 ("edac: i3200 memory controller driver")
> Cc: stable@xxxxxxxxxxxxxxx
> Assisted-by: DeepSeek-V4.1-Flash and OpenAI Codex

Grep the kernel tree for Assisted-by guidance.

> Signed-off-by: Wentao Liang <vulab@xxxxxxxxxxx>
> ---
>  drivers/edac/i3200_edac.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

$ ./scripts/get_maintainer.pl -f drivers/edac/i3200_edac.c
Borislav Petkov <bp@xxxxxxxxx> (maintainer:EDAC-CORE)
Tony Luck <tony.luck@xxxxxxxxx> (maintainer:EDAC-CORE)
linux-edac@xxxxxxxxxxxxxxx (open list:EDAC-I3200)
linux-kernel@xxxxxxxxxxxxxxx (open list)
EDAC-I3200 status: Orphan
		   ^^^^^^^

All your patches touching orphaned EDAC drivers are going to be ignored - you
don't want to spend any effort or burn AI energy on them.

I already told you that the last time you sent patches. It looks like probably
no human even read my responses.

Quickly grepping through lore.kernel.org it looks like this effort you're
doing is some AI gunk mindlessly sending kernel patches.

So you will be ignored too.

Just remember that if you don't listen to maintainers and ignore their
feedback, all your effort and wasted energy will be for nothing.

Thx.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette