linux-kernel - Re: [PATCH] arm64: dts: qcom: qcom6490-idp: Add IPA nodes

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <ab08fdfb-9268-4d32-bf3a-552b0f5720a4@quicinc.com>
Date: Wed, 5 Mar 2025 21:24:59 +0530
From: Kaustubh Pandey <quic_kapandey@...cinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
CC: <andersson@...nel.org>, <konradybcio@...nel.org>, <robh@...nel.org>,
        <krzk+dt@...nel.org>, <conor+dt@...nel.org>,
        <linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <quic_rpavan@...cinc.com>,
        <quic_sharathv@...cinc.com>, <quic_sarata@...cinc.com>
Subject: Re: [PATCH] arm64: dts: qcom: qcom6490-idp: Add IPA nodes

On 3/5/2025 2:28 AM, Dmitry Baryshkov wrote:
> On Tue, Mar 04, 2025 at 08:51:33PM +0530, Kaustubh Pandey wrote:
>> Add IPA nodes for Qualcomm qcm6490 board.
>>
>> Signed-off-by: Kaustubh Pandey <quic_kapandey@...cinc.com>
>> ---
>>  arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 8 +++++++-
>>  1 file changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
>> index 9209efcc49b5..ba47786d6474 100644
>> --- a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
>> +++ b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
>> @@ -19,7 +19,6 @@
>>  #include "pm8350c.dtsi"
>>  #include "pmk8350.dtsi"
>>
>> -/delete-node/ &ipa_fw_mem;
>>  /delete-node/ &rmtfs_mem;
>>  /delete-node/ &adsp_mem;
>>  /delete-node/ &cdsp_mem;
>> @@ -515,6 +514,13 @@ &gpu_zap_shader {
>>  	firmware-name = "qcom/qcm6490/a660_zap.mbn";
>>  };
>>
>> +&ipa {
>> +	qcom,gsi-loader = "self";
>> +	memory-region = <&ipa_fw_mem>;
>> +	firmware-name = "qcom/qcm6490/ipa_fws.mdt";
> 
> "qcom/qcm6490/ipa_fws.mbn". There is no .mdt in linux-firmware.
> 
>> +	status = "okay";
>> +};
>> +
>>  &mdss {
>>  	status = "okay";
>>  };
> 

Sure, thanks for review.
Will post another patch with correct format.

Thanks,
Kaustubh