Re: [PATCH RESEND v3 14/17] EDAC/synopsys: Detach Zynq DDRC controller support
From: Borislav Petkov
Date: Wed Oct 12 2022 - 15:44:19 EST
On Wed, Oct 12, 2022 at 10:27:43PM +0300, Serge Semin wrote:
> ... inter-device parts of the core. The registration procedure is
> protected by the mutex and RCU. So it seems as the EDAC core developer
Seems, schmeems. As I said already, EDAC has always had a single
chipset-specific driver. Period. So if one needs to run more than one
chipset-specific driver concurrently, then the whole code needs to be
audited because this hasn't been done before.
> If it has never needed to, then please explain why did you let the
> Synopsys EDAC driver being accepted like that then?
I think I already did.
> In my case it's a single EDAC driver per-chip. There can be several
> DDR-controllers installed on the same SoC, but all of them of the same
> type (Synopsys DW uMCTL2 v2.61a).
Good.
I'll look at your patches as time allows.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette