Messages in this thread | | | Date | Sun, 17 Mar 2024 14:52:44 +0000 | From | Conor Dooley <> | Subject | Re: [PATCH 1/3] dt-bindings: remoteproc: add Versal platform support |
| |
On Sun, Mar 17, 2024 at 02:50:27PM +0000, Conor Dooley wrote: > On Fri, Mar 15, 2024 at 02:15:31PM -0700, 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. > > No, don't be lazy. Add a compatible with a fallback please.
> This doesn't apply to linux-next either FYI.
Ordinarily'd I'd not even bother applying a patch like this and I wouldn't notice, but the diff for the versal-net patch is difficult to read without colour-moved enabled and since I can't apply this I'm not going to review that patch. [unhandled content-type:application/pgp-signature] | |