On 5/15/25 10:15 AM, Dmitry Baryshkov wrote: > On Thu, May 15, 2025 at 09:46:50AM +0530, Gokul Sriram P wrote: >> >> On 4/26/2025 1:53 AM, Konrad Dybcio wrote: >>> On 4/17/25 8:12 AM, Gokul Sriram Palanisamy wrote: >>>> From: Manikanta Mylavarapu <[email protected]> >>>> >>>> Enable nodes required for q6 remoteproc bring up. >>>> >>>> Signed-off-by: Manikanta Mylavarapu <[email protected]> >>>> Signed-off-by: Gokul Sriram Palanisamy <[email protected]> >>>> --- >>>> changes since v3: >>>> - added necessary padding for 8digt hex address in dts >>>> - fixed firmware-name to use .mbn format >>>> >>>> arch/arm64/boot/dts/qcom/ipq5332.dtsi | 64 ++++++++++++++++++++++++++- >>>> 1 file changed, 63 insertions(+), 1 deletion(-) >>>> >>>> diff --git a/arch/arm64/boot/dts/qcom/ipq5332.dtsi >>>> b/arch/arm64/boot/dts/qcom/ipq5332.dtsi >>>> index 69dda757925d..fc120fd8b274 100644 >>>> --- a/arch/arm64/boot/dts/qcom/ipq5332.dtsi >>>> +++ b/arch/arm64/boot/dts/qcom/ipq5332.dtsi >>>> @@ -2,7 +2,7 @@ >>>> /* >>>> * IPQ5332 device tree source >>>> * >>>> - * Copyright (c) 2022-2023 Qualcomm Innovation Center, Inc. All rights >>>> reserved. >>>> + * Copyright (c) 2022-2025 Qualcomm Innovation Center, Inc. All rights >>>> reserved. >>>> */ >>>> >>>> #include <dt-bindings/clock/qcom,apss-ipq.h> >>>> @@ -146,6 +146,11 @@ smem@4a800000 { >>>> >>>> hwlocks = <&tcsr_mutex 3>; >>>> }; >>>> + >>>> + q6_region: wcss@4a900000 { >>>> + reg = <0x0 0x4a900000 0x0 0x2b00000>; >>>> + no-map; >>>> + }; >>>> }; >>>> >>>> soc@0 { >>>> @@ -545,6 +550,39 @@ frame@b128000 { >>>> status = "disabled"; >>>> }; >>>> }; >>>> + >>>> + q6v5_wcss: remoteproc@d100000 { >>>> + compatible = "qcom,ipq5332-wcss-sec-pil"; >>>> + reg = <0x0d100000 0x4040>; >>> This is 0x10_000-long >>> >>>> + firmware-name = "ath12k/IPQ5332/hw1.0/q6_fw0.mbn"; >>> Is the firmware OEM signed? >>> >> No. This isn't OEM signed. userPD firmwares will only be OEM signed. > > Indeed, it contains only Qualcomm / QTI signature, OEM signature is not > present.
Good, thanks for confirming with the size changed Reviewed-by: Konrad Dybcio <[email protected]> Konrad
