Re: [PATCH 1/3] dt-bindings: memory-controllers: Add support for Versal NET EDAC

From: Krzysztof Kozlowski
Date: Tue Nov 26 2024 - 02:07:39 EST


On 26/11/2024 07:57, Datta, Shubhrajyoti wrote:
> [AMD Official Use Only - AMD Internal Distribution Only]

What is this? Why would we bother? How does it affect me?

Please remove all corporate boilerplate.


...

>>> +
>>> +required:
>>> + - compatible
>>
>> Eh, no resources? How do you talk with the hardware? This looks way too Linuxy...
>
> The address space is secure, making it inaccessible to Linux. In this setup, the secure firmware (NMC)
> communicates the necessary information to Linux through RPMsg.


Don't you need to acquire a reference to the rpmsg interface? Don't you
need to encode the device dependency? All this is achieved with phandles
and proper driver support.

Best regards,
Krzysztof