Quoting Rajendra Nayak (2021-03-03 04:17:56) > diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi > index fe4fdb9..aa6f847 100644 > --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi > +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi > @@ -239,6 +239,25 @@ > interrupt-controller; > }; > > + spmi_bus: spmi@c440000 { > + compatible = "qcom,spmi-pmic-arb"; > + reg = <0 0x0c440000 0 0x1100>, > + <0 0x0c600000 0 0x2000000>, > + <0 0x0e600000 0 0x100000>, > + <0 0x0e700000 0 0xa0000>, > + <0 0x0c40a000 0 0x26000>; > + reg-names = "core", "chnls", "obsrvr", "intr", "cnfg"; > + interrupt-names = "periph_irq"; > + interrupts-extended = <&pdc 1 IRQ_TYPE_LEVEL_HIGH>; > + qcom,ee = <0>; > + qcom,channel = <0>; > + #address-cells = <1>; > + #size-cells = <1>; I see the binding says these should be 2 instead of 1 but I suspect that is incorrect. > + interrupt-controller; > + #interrupt-cells = <4>; > + cell-index = <0>; Is cell-index used? Please remove as I don't see it used anywhere and not in the binding. > + }; > + > tlmm: pinctrl@f100000 { > compatible = "qcom,sc7280-pinctrl"; > reg = <0 0x0f100000 0 0x1000000>;