Re: [PATCH v2 3/3] arm64: dts: qcom: eliza: Add IMEM node

[Alexander Koskovich]

On Thursday, April 16th, 2026 at 6:09 AM, Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote:

> On 16/04/2026 11:40, Alexander Koskovich wrote:
> > Add a node for the IMEM found on Eliza, which contains pil-reloc-info
> > and the modem tables for IPA, among others.
> >
> > Signed-off-by: Alexander Koskovich <akoskovich@xxxxx>
> > ---
> >  arch/arm64/boot/dts/qcom/eliza.dtsi | 20 ++++++++++++++++++++
> >  1 file changed, 20 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/eliza.dtsi b/arch/arm64/boot/dts/qcom/eliza.dtsi
> > index 6fa5679c1a62..551df07e44c6 100644
> > --- a/arch/arm64/boot/dts/qcom/eliza.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/eliza.dtsi
> > @@ -1029,6 +1029,26 @@ qup_uart14_default: qup-uart14-default-state {
> >  			};
> >  		};
> >
> > +		sram@14680000 {
> > +			compatible = "qcom,eliza-imem", "mmio-sram";
> > +			reg = <0x0 0x14680000 0x0 0x2c000>;
> > +			ranges = <0x0 0x0 0x14680000 0x2c000>;
> > +
> > +			no-memory-wc;
> > +
> > +			#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.
> 
> Either this is part of board or common DTSI for SMxxxx SoC. Whatever is
> chosen here, should be consistent with other platforms.

Would you want the IPA node, MPSS remoteproc, etc to follow same pattern? Can
just throw them in a sm7550.dtsi.

Since sm7550.dtsi wouldn't have any consumers until I push my board dts, I guess should hold off on this until then?

The node sort part of this patchset can be applied separately.

> 
> Best regards,
> Krzysztof
> 

Thanks,
Alex