linux-kernel - Re: [PATCHv4 2/6] dt-bindings: arm: ti: Add bindings for AM62D2 SoC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <9ac27b3b-5bb3-4b32-8c18-f66b3b72af89@ti.com>
Date: Tue, 24 Jun 2025 16:51:51 +0530
From: Paresh Bhagat <p-bhagat@...com>
To: Krzysztof Kozlowski <krzk@...nel.org>, <nm@...com>, <vigneshr@...com>,
        <praneeth@...com>
CC: <kristo@...nel.org>, <robh@...nel.org>, <krzk+dt@...nel.org>,
        <conor+dt@...nel.org>, <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <khasim@...com>, <v-singh1@...com>, <afd@...com>, <bb@...com>,
        <devarsht@...com>
Subject: Re: [PATCHv4 2/6] dt-bindings: arm: ti: Add bindings for AM62D2 SoC

Hi Krzysztof,


On 24/06/25 16:25, Krzysztof Kozlowski wrote:
> On 24/06/2025 12:39, Paresh Bhagat wrote:
>>>> This SoC is of part K3 AM62x family, which includes the AM62A and AM62P
>>>> variants. While the AM62A and AM62D are largely similar, the AM62D is
>>>> specifically targeted for general-purpose DSP applications, whereas the
>>>> AM62A focuses on edge AI workloads. A key distinction is that the AM62D
>>>> does not include multimedia components such as the video encoder/decoder,
>>>> MJPEG encoder, Vision Processing Accelerator (VPAC) for image signal
>>>> processing, or the display subsystem. Additionally, the AM62D has a
>>>> different pin configuration compared to the AM62A, which impacts
>>>> embedded software development.
>>
>> This section is important as it clarifies the difference between AM62a
>> and AM62d, as we are reusing AM62a dtsi files for AM62d. Let me know if
>> you need a shorter version.
> This is not important:
>
> "While the AM62A and AM62D are largely similar, the AM62D is
> specifically targeted for general-purpose DSP applications, whereas the
> AM62A focuses on edge AI workloads."
>
> It is irrelevant. Rest is fine.
Fine will do.
>
>>
>>>> This adds dt bindings for TI's AM62D2 family of devices.
>>>>
>>>> More details about the SoCs can be found in the Technical Reference Manual:
>>>> https://www.ti.com/lit/pdf/sprujd4
>>>>
>>>> Signed-off-by: Paresh Bhagat <p-bhagat@...com>
>>> And what happened with the previous comments?
>>
>> Yep there were some indentation problems earlier, which is fixed in this
>> version. There was also an ack from Conor Dooley in v3. I will include that.
> Why you did not include it before?
>
> Can you start using b4 for your process of submitting patches?


I missed it. I got to know about the b4 tool internally, will use it. 
Thanks.
>
>
> Best regards,
> Krzysztof