Re: [PATCH v2 3/3] arm64: dts: qcom: eliza: Add IMEM node
From: Krzysztof Kozlowski
Date: Thu Apr 23 2026 - 05:47:44 EST
On 16/04/2026 12:19, Konrad Dybcio wrote:
>>> + #address-cells = <1>;
>>> + #size-cells = <1>;
>>> +
>>> + pilreloc-sram@94c {
>>> + compatible = "qcom,pil-reloc-info";
>>> + reg = <0x94c 0xc8>;
>>> + };
>>> +
>>> + ipa_modem_tables: modem-tables-sram@3000 {
>>> + reg = <0x3000 0x2000>;
>>
>> I don't think these two should be in the main SoC DTSI. The non-modem
>> version obviously does not have modem-tables.
>
> That's not quite right, IMEM is partitioned to have it either way, even
> if it stays unused. You'll notice this slice is there even on platforms
> that were designed with no modem in any SKU
>
OK, seems fine to me.
Best regards,
Krzysztof