Re: [PATCH v2 2/3] arm64: dts: qcom: Introduce Eliza Soc base dtsi

[Abel Vesa]

On 26-03-17 15:04:24, Harshal Dev wrote:
> Hi Abel,
> 
> On 3/13/2026 6:53 PM, Abel Vesa wrote:
> > Introduce the initial support for the Qualcomm Eliza SoC. It comes in
> > different flavors. There is SM7750 for mobiles and then QC7790S/M for IoT.
> > Describe the common parts under a common dtsi.
> > 
> > The initial submission enables support for:
> > - CPU nodes with cpufreq and cpuidle support
> > - Global Clock Controller (GCC)
> > - Resource State Coordinator (RSC) with clock controller & genpd provider
> > - Interrupt controller
> > - Power Domain Controller (PDC)
> > - Vendor specific SMMU
> > - SPMI bus arbiter
> > - Top Control and Status Register (TCSR)
> > - Top Level Mode Multiplexer (TLMM)
> > - Debug UART
> > - Reserved memory nodes
> > - Interconnect providers
> > - System timer
> > - UFS
> > 
> > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> > Co-developed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>
> > ---
> >  arch/arm64/boot/dts/qcom/eliza.dtsi | 1317 +++++++++++++++++++++++++++++++++++
> >  1 file changed, 1317 insertions(+)
> > 
> > +
> > +		ice: crypto@1d88000 {
> > +			compatible = "qcom,eliza-inline-crypto-engine",
> > +				     "qcom,inline-crypto-engine";
> > +			reg = <0x0 0x01d88000 0x0 0x18000>;
> > +
> > +			clocks = <&gcc GCC_UFS_PHY_ICE_CORE_CLK>;
> 
> As per the updated ABI for eliza and milos which I am proposing with this patch,
> we should have two clocks (core & iface) and a power-domain here:
> 
> https://lore.kernel.org/all/20260317-qcom_ice_power_and_clk_vote-v3-2-53371dbabd6a@xxxxxxxxxxxxxxxx/

I don't want the merging of this Eliza patchset to be blocked until
yours gets merged.

So, for now, I'll respin without and we can fix it after.

Thanks for pointing this out though!