For some drivers, during initialization error reporting is enabled,
but not properly handled during teardown or init fail, and this leaves 
it enabled in the register on exit.

This inconsistency was first identified for i7300_edac.c and a patch 
addressing it was reviewed: 

Link: https://lore.kernel.org/linux-edac/20260429094806.25097-1-tushar.tibude1000@gmail.com/

It was then discovered across multiple i5xxx drivers:

i5000_edac.c
i5100_edac.c
i5400_edac.c

This patch series aims to implement a fix for each of them.

Hi Tushar,

> From: Tushar Tibude <tushar.tibude1000@gmail.com>
> Sent: Thursday, April 30, 2026 4:42 PM
> To: mchehab@kernel.org; bp@alien8.de; Luck, Tony <tony.luck@intel.com>;
> linux-edac@vger.kernel.org; linux-kernel@vger.kernel.org
> Cc: Zhuo, Qiuxu <qiuxu.zhuo@intel.com>; Tushar Tibude
> <tushar.tibude1000@gmail.com>
> Subject: [PATCH 0/3] EDAC: fix error reporting handling during init for
> 
> For some drivers, during initialization error reporting is enabled, but not
> properly handled during teardown or init fail, and this leaves it enabled in the
> register on exit.
> 
> This inconsistency was first identified for i7300_edac.c and a patch addressing
> it was reviewed:
> 
> Link: https://lore.kernel.org/linux-edac/20260429094806.25097-1-
> tushar.tibude1000@gmail.com/
> 
> It was then discovered across multiple i5xxx drivers:
> 
> i5000_edac.c
> i5100_edac.c
> i5400_edac.c
> 
> This patch series aims to implement a fix for each of them.

Thanks for the patch.

Since this addresses the same issue with the same simple fix in the same subsystem, 
IMHO it's better to combine this series and the missing fix for driver [1] with your 
previous fix [2] into a single patch, rather than having 5 separate patches.

But please wait for comments from @Luck, Tony and @Borislav Petkov (AMD) on whether 
to use a single patch or multiple patches.

[1] https://web.git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/edac/i7300_edac.c#n1092
[2] https://lore.kernel.org/all/20260429094806.25097-1-tushar.tibude1000@gmail.com/

Thanks!
-Qiuxu

> Since this addresses the same issue with the same simple fix in the same subsystem,
> IMHO it's better to combine this series and the missing fix for driver [1] with your
> previous fix [2] into a single patch, rather than having 5 separate patches.
>
> But please wait for comments from @Luck, Tony and @Borislav Petkov (AMD) on whether
> to use a single patch or multiple patches.
>
> [1] https://web.git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/edac/i7300_edac.c#n1092
> [2] https://lore.kernel.org/all/20260429094806.25097-1-tushar.tibude1000@gmail.com/

If the patch was only 5 lines of diff, bundling them all in one patch would be good. But each
is around 30+ lines:

   drivers/edac/i7300_edac.c | 33 ++++++++++++++++++++++++---------

which would make the combined patch a bit bigger than I like.

I'll look at the individual patches and if they are OK I'll pick them up separately.

-Tony

> From: Luck, Tony <tony.luck@intel.com>
> Sent: Friday, May 1, 2026 6:40 AM
> To: Zhuo, Qiuxu <qiuxu.zhuo@intel.com>; Tushar Tibude
> <tushar.tibude1000@gmail.com>; mchehab@kernel.org; bp@alien8.de;
> linux-edac@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: RE: [PATCH 0/3] EDAC: fix error reporting handling during init for
> 
> > Since this addresses the same issue with the same simple fix in the
> > same subsystem, IMHO it's better to combine this series and the
> > missing fix for driver [1] with your previous fix [2] into a single patch, rather
> than having 5 separate patches.
> >
> > But please wait for comments from @Luck, Tony and @Borislav Petkov
> > (AMD) on whether to use a single patch or multiple patches.
> >
> > [1]
> > https://web.git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> > /tree/drivers/edac/i7300_edac.c#n1092
> > [2]
> > https://lore.kernel.org/all/20260429094806.25097-1-tushar.tibude1000@g
> > mail.com/
> 
> If the patch was only 5 lines of diff, bundling them all in one patch would be
> good. But each is around 30+ lines:
> 
>    drivers/edac/i7300_edac.c | 33 ++++++++++++++++++++++++---------
> 
> which would make the combined patch a bit bigger than I like.
> 
> I'll look at the individual patches and if they are OK I'll pick them up
> separately.
>

Thanks @Luck, Tony for the comments.

For the patch series, LGTM.

   Reviewed-by: Qiuxu Zhuo <qiuxu.zhuo@intel.com>