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,
> 
> 


Reply via email to