Re: [PATCH v2 2/2] arm64: dts: zena: Add support for Zena CSS

[Sudeep Holla <sudeep.holla@kernel.org>]

On Sun, Mar 08, 2026 at 04:41:55PM +0100, Krzysztof Kozlowski wrote:
> On 12/02/2026 12:16, Debbie Horsfall wrote:
> > Introduce the Zena CSS Fixed Virtual Platform (FVP) dts. This is
> > currently the only Zena CSS variant, however the common definitions are
> > included in a common dtsi for extensibility.
> > 
> > Signed-off-by: Debbie Horsfall <debbie.horsfall@arm.com>
> 
> ...
> 
> > +
> > +	timer {
> > +		compatible = "arm,armv8-timer";
> > +		interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_LOW>,
> > +			     <GIC_PPI 14 IRQ_TYPE_LEVEL_LOW>,
> > +			     <GIC_PPI 11 IRQ_TYPE_LEVEL_LOW>,
> > +			     <GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>,
> > +			     <GIC_PPI 12 IRQ_TYPE_LEVEL_LOW>;
> > +	};
> > +
> > +	sram: sram@104000 {
> 
> This belongs in the soc node.
> 
> > +		compatible = "mmio-sram";
> > +		reg = <0x0 0x00104000 0x0 0x00001000>;
> > +		#address-cells = <1>;
> > +		#size-cells = <1>;
> > +		ranges = <0 0x0 0x00104000 0x00001000>;
> > +
> > +		scmi_shmem_tx: scpshmem-sram-section@0 {
> > +			compatible = "arm,scmi-shmem";
> > +			reg = <0x0 0x100>;
> > +		};
> > +
> > +		scmi_shmem_rx: scpshmem-sram-section@100 {
> > +			compatible = "arm,scmi-shmem";
> > +			reg = <0x100 0x100>;
> > +		};
> > +	};
> > +
> > +	memory@80000000 {
> 
> And placing this in DTSI means that zena-css comes with this memory
> mounted and it is not customizable in final FVP board.
> 

Both valid points, please post a patch addressing this based on my
for-next/juno/updates as I just applied these last night.

-- 
Regards,
Sudeep