Re: [PATCH 1/3] dt-bindings: remoteproc: add Versal platform support

Tanmay Shah
2024-03-19T00:40:02Z

On 3/17/24 1:50 PM, Krzysztof Kozlowski wrote:
> On 15/03/2024 22:15, Tanmay Shah wrote:
>> AMD-Xilinx Versal platform is successor of ZynqMP platform. Real-time
>> Processor Unit R5 cluster IP on Versal is same as of ZynqMP Platform.
>> Only difference is power-domains ID needed by power management firmware.
>> Hence, keeping the compatible property same as of zynqmp node.
>>
>> Signed-off-by: Tanmay Shah <tanmay.shah@amd.com>
>
> There is no binding change, so NAK. Platform is not being added to
> examples. You changed nothing in Versal support...

Thanks for reviews.

Okay got it. That means I don't really need to add anything related to Versal.
I will get rid of patch that says "Versal support". Looks like it's not needed
at all.

Thanks.

>
> Best regards,
> Krzysztof
>