Hi Krzysztof, I apologize if this came off as my ignoring your emails.
In the v3 you had said: > + - enum: > + - xlnx,versal-bram-rproc > + - xlnx,versal-net-bram-rproc > + - amd,versal2-bram-rproc If there is going to be a new version, this should be sorted by name. Reviewed-by: Krzysztof Kozlowski <[email protected]> I was uncomfortable in adding the Reviewed-by if that is what you meant. In this v4 I believe your review comment to sort by name was addressed. Apologies for my misunderstanding. Respectfully, Ben Levinsky On 6/29/26 11:53 PM, Krzysztof Kozlowski wrote: > On Mon, Jun 29, 2026 at 09:40:02AM -0700, Ben Levinsky wrote: >> Describe an AMD BRAM-based remote processor controlled through the >> remoteproc framework. >> >> The binding models a soft-core processor subsystem instantiated in AMD >> programmable logic and using dual-port BRAM for firmware storage and >> execution. The remoteproc device is represented as a child node whose >> reg property describes the firmware memory window in the processor-local >> address space. The parent bus node provides standard devicetree address >> translation through ranges so Linux can access the same BRAM through the >> system physical address space. >> >> A clock input feeds the soft-core processor subsystem, and an active-low >> reset GPIO holds the processor in reset until firmware loading >> completes. The firmware-name property is optional. >> >> Signed-off-by: Ben Levinsky <[email protected]> > > So you are going to ignore my emails? You know, it can work both ways. > > I drop this from Patchwork. > > Best regards, > >

