RE: [PATCH] EDAC/ie31200: Disable PCI device when probing fails

From: Luck, Tony

Date: Mon Sep 14 2026 - 13:26:05 EST


> ie31200_init_one() enables the PCI device before calling ie31200_probe1().
> If probing fails, it returns without balancing the successful enable,
> leaving the PCI enable count elevated.
>
> Call pci_disable_device() on that failure path. This also covers the
> direct ie31200_init_one() call from the module initialization fallback,
> while preserving the existing return values and successful probe path.
>
> This issue was identified during our ongoing static-analysis research while
> reviewing kernel code.
>
> Assisted-by: OpenAI:GPT-5.6

I added "LLM" to this tag based on current guidance in Documentation/process/coding-assistants.rst

> Co-developed-by: Ijae Kim <ae878000@xxxxxxxxx>
> Signed-off-by: Ijae Kim <ae878000@xxxxxxxxx>
> Signed-off-by: Myeonghun Pak <mhun512@xxxxxxxxx>

Applied. Thanks.

-Tony