linux-kernel - Re: [PATCH v8 03/11] arm64: dts: ti: k3-am62a-mcu: Add R5F remote proc node

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 for Android: free password hash cracker in your pocket

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

Message-ID: <6c5e786d-7581-492f-92fb-be92ecbecd87@ti.com>
Date: Mon, 5 May 2025 10:05:54 -0500
From: "Mendez, Judith" <jm@...com>
To: Daniel Schultz <d.schultz@...tec.de>, Nishanth Menon <nm@...com>,
        Vignesh
 Raghavendra <vigneshr@...com>
CC: Tero Kristo <kristo@...nel.org>, Rob Herring <robh@...nel.org>,
        Krzysztof
 Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>, Hari
 Nagalla <hnagalla@...com>,
        Beleswar Padhi <b-padhi@...com>,
        Markus
 Schneider-Pargmann <msp@...libre.com>,
        Andrew Davis <afd@...com>, Devarsh
 Thakkar <devarsht@...com>,
        <linux-arm-kernel@...ts.infradead.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v8 03/11] arm64: dts: ti: k3-am62a-mcu: Add R5F remote
 proc node

Hi Daniel,

On 5/5/2025 4:55 AM, Daniel Schultz wrote:
> Hi,
> 
> I'm unable to load the latest TI firmware 
> (98efd20ec71f8c1c8f909d34ab656731) with this patch.
> 
> [    7.012889] remoteproc remoteproc1: 79000000.r5f is available
> [    7.032640] remoteproc remoteproc1: powering up 79000000.r5f
> [    7.038626] remoteproc remoteproc1: Booting fw image am62a-mcu- 
> r5f0_0-fw, size 53140
> [    7.057209] remoteproc remoteproc1: bad phdr da 0x79100000 mem 0x47ea0
> [    7.064716] remoteproc remoteproc1: Failed to load program segments: -22
> 
> I figured out that the mcu sram node disappeared in v5. Apparently 
> adding it back manually doesn't solve this problem. Any idea what's wrong?

For am62ax, there should be several items changed with this v8
series in order for remoteproc to work with the TI default firmware:

1. memory carveouts were reduced to 15MB [0] & edge-ai memory
carveouts are not included here
2. mcu_sram1 node removed [2]

If you want to catch up on the general direction for this series,
please refer to [3]. atm remoteproc can fail with the default FW,
but we are trying to move away from that firmware and this is the
first step in that direction.

[0] 
https://lore.kernel.org/linux-devicetree/0ab5c5ec-cde3-41f1-8adf-2419b31497c1@ti.com/
[1] 
https://lore.kernel.org/linux-devicetree/04e77daf-e775-44fa-82bf-8b6ebf73bcef@ti.com/
[2] 
https://lore.kernel.org/linux-devicetree/32358aa1-0c02-4f4d-9782-2d8376c0d9fc@ti.com/
[3] 
https://lore.kernel.org/linux-devicetree/e131298f-3713-482a-a740-ff89709270b4@ti.com/

~ Judith

> 
> On 5/3/25 00:03, Judith Mendez wrote:
>> From: Hari Nagalla <hnagalla@...com>
>>
>> AM62A SoCs have a single R5F core in the MCU voltage domain.
>> Add the R5FSS node with the child node for core0 in MCU voltage
>> domain .dtsi file.
>>
>> Signed-off-by: Hari Nagalla <hnagalla@...com>
>> Signed-off-by: Judith Mendez <jm@...com>
>> Acked-by: Andrew Davis <afd@...com>
>> ---
>>   arch/arm64/boot/dts/ti/k3-am62a-mcu.dtsi | 25 ++++++++++++++++++++++++
>>   1 file changed, 25 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62a-mcu.dtsi b/arch/arm64/ 
>> boot/dts/ti/k3-am62a-mcu.dtsi
>> index 9ed9d703ff24..ee961ced7208 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62a-mcu.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62a-mcu.dtsi
>> @@ -174,4 +174,29 @@ mcu_mcan1: can@...8000 {
>>           bosch,mram-cfg = <0x0 128 64 64 64 64 32 32>;
>>           status = "disabled";
>>       };
>> +
>> +    mcu_r5fss0: r5fss@...00000 {
>> +        compatible = "ti,am62-r5fss";
>> +        #address-cells = <1>;
>> +        #size-cells = <1>;
>> +        ranges = <0x79000000 0x00 0x79000000 0x8000>,
>> +             <0x79020000 0x00 0x79020000 0x8000>;
>> +        power-domains = <&k3_pds 7 TI_SCI_PD_EXCLUSIVE>;
>> +        status = "disabled";
>> +
>> +        mcu_r5fss0_core0: r5f@...00000 {
>> +            compatible = "ti,am62-r5f";
>> +            reg = <0x79000000 0x00008000>,
>> +                  <0x79020000 0x00008000>;
>> +            reg-names = "atcm", "btcm";
>> +            resets = <&k3_reset 9 1>;
>> +            firmware-name = "am62a-mcu-r5f0_0-fw";
>> +            ti,atcm-enable = <0>;
>> +            ti,btcm-enable = <1>;
>> +            ti,loczrama = <0>;
>> +            ti,sci = <&dmsc>;
>> +            ti,sci-dev-id = <9>;
>> +            ti,sci-proc-ids = <0x03 0xff>;
>> +        };
>> +    };
>>   };