Re: [PATCH 2/2] dt-bindings: arm: cpus: Add edac-enabled property
From: Marc Zyngier
Date: Sun Apr 13 2025 - 06:38:53 EST
On Fri, 11 Apr 2025 21:02:07 +0100,
Borislav Petkov <bp@xxxxxxxxx> wrote:
>
> On Thu, Apr 10, 2025 at 05:23:26PM +0100, Marc Zyngier wrote:
> > We have some other EDAC implementation for arm64 CPUs (XGene,
> > ThunderX), and they are all perfectly useless (I have them in my
> > collection of horrors).
>
> Oh oh, can I remove, can I remove?
>
> My trigger finger is itching to kill some more useless code...
The drivers do report ECC errors being corrected, which indicates that
the HW itself is doing its job. Yes, I buy cheap memory from eBay.
Do we need actual drivers to output crap on the console? Probably not,
but I'm the wrong person to ask -- I only keep these machines alive to
remind me how things can go horribly wrong.
I don't think there is any harm in keeping this crap around. It
compiles, and if it breaks, I'll fix it. I'm not convinced we need any
more of it though, specially for CPUs that are over a decade old.
M.
--
Without deviation from the norm, progress is not possible.