Re: [PATCH v4 1/3] dt-bindings: edac: xlnx,versal-ddrmc: Add xlnx,versal-xilsem compatible

[Borislav Petkov]

On Thu, Oct 29, 2026 at 05:12:44PM +0100, Michal Simek wrote:
> With Versal Gen 2 which is still under development I can imagine that we can
> design it in a way that we say that we won't allow any MMIO interface and
> only firmware interface will be used and firmware itself will integrate all
> edacs in the system and export it via the same API. Then we will have only
> one single driver for the whole platform and it depends only on firmware
> which functionality is going to be exported to Linux. Which is IMHO model
> which you would like to see.

I appreciate the intention to make it saner but ARM is insane by definition in
that regard. You have IP blocks which can be built in in all kinds of
platforms and sw needs to mimick that. For some EDAC drivers we even have .c
compilation units which get linked into each platform which supports them.

We would usually do that when you have too many EDAC wrapper drivers loading
on the system and them needing to talk to each other. Which the subsystem
hasn't really planned on supporting.

But if the wrappers can be relatively independent, we can do the simple
thing and do a separate wrapper for each. No biggie.

Thx.

-- 
Regards/Gruss,
    Boris.

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