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

[Prasanna Kumar T S M]

Hi Rama Devi,
> > XILSEM edac device is present on Versal and Versal NET SOCs.
> > https://docs.amd.com/r/en-US/oslib_rm/Tool-Configuration-for-XilSEM
> > (Actually also on Versal Gen 2 but firmware interface is going to be different).
> >
> > I don't think we should try to integrate EDAC memory controller with EDAC
> > device because their usage is independent of each other.
>
> Are you saying that you will have this XILSEM hw in different platforms?
>
> IOW, you will have a Versal or Versalnet thing and then something else, and
> then XILSEM will be in all of them?
>
> Then perhaps it is better to have it as a separate EDAC "driver". In air
> quotes because this is not really a driver but simply a wrapper so that it can
> be called an EDAC driver.
>
> It all depends on what hw configurations will be built.

The v2 [1] had a separate XilSEM EDAC driver. Are you planning to revive 
this?
> But if you have XILSEM *only* with Versal, then I don't want a separate EDAC,
> hm, driver.
1. 
https://lore.kernel.org/lkml/20250722160315.2979294-1-rama.devi.veggalam@xxxxxxx/

Thanks,
Prasanna