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

[Kaustubh Pandey]

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@xxxxxxxxxxx>
>> ---
>>  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