Re: [PATCH v5 15/15] ARM: dts: qcom: Add initial device tree for Kuno SoC

[Hardeep Sharma]

On 9/3/2026 4:27 PM, Konrad Dybcio wrote:
> On 8/27/26 7:51 PM, Hardeep Sharma wrote:
> > Add SoC-level DTSI (qcom-kuno.dtsi) and IDP board DTS
> > (qcom-kuno-idp.dts) for the Qualcomm Kuno SoC.
> >
> > Peripherals included in this initial set:
> >    - Single Cortex-A7 CPU with PSCI idle states
> >    - RPMh RSC with clock controller, power domains, and BCM voter
> >    - GCC (Global Clock Controller)
> >    - TLMM pin controller (110 GPIOs)
> >    - Interconnect (aggre-noc, cnoc-main)
> >    - GENI UART console (QUP SE3)
> >    - SMEM, hardware spinlocks (TCSR mutex)
> >
> > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> > Reviewed-by: Linus Walleij <linusw@xxxxxxxxxx>
> > Reviewed-by: Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Hardeep Sharma <hardeep.sharma@xxxxxxxxxxxxxxxx>
> > ---
> [...]
>
> > +	soc: soc@0 {
> > +		compatible = "simple-bus";
> > +		#address-cells = <1>;
> > +		#size-cells = <1>;
> > +		ranges = <0 0 0xffffffff>;
> that's just "ranges;"
>
> [...]
>
> > +		qup0: geniqup@9c0000 {
> > +			compatible = "qcom,geni-se-qup";
> > +			reg = <0x009c0000 0x2000>;
> > +			clocks = <&gcc GCC_QUPV3_WRAP_0_M_AHB_CLK>,
> > +				 <&gcc GCC_QUPV3_WRAP_0_S_AHB_CLK>;
> > +			clock-names = "m-ahb", "s-ahb";
> > +			#address-cells = <2>;
> > +			#size-cells = <2>;
> > +			ranges = <0x0 0x0 0x0 0x0 0x1000000>;
> likewise here, you don't need to precisely limit this.
>
> Konrad

Thanks Konrad for review.

fixed in v6 : 
https://lore.kernel.org/all/20260929-kuno-soc-support-v6-1-a16afafbcbc7@xxxxxxxxxxxxxxxx/

Best Wishes

Hardeep Sharma