On 01-12-21, 16:03, Konrad Dybcio wrote: > > On 01.12.2021 08:29, Vinod Koul wrote: > > This add based DTSI for SM8450 SoC and includes base description of > > CPUs, GCC, RPMHCC, UART, interuupt-controller which helps to boot to > > shell with console on boards with this SoC > > > > Signed-off-by: Vinod Koul <vkoul@xxxxxxxxxx> > > --- > > arch/arm64/boot/dts/qcom/sm8450.dtsi | 476 +++++++++++++++++++++++++++ > > 1 file changed, 476 insertions(+) > > create mode 100644 arch/arm64/boot/dts/qcom/sm8450.dtsi > > > > diff --git a/arch/arm64/boot/dts/qcom/sm8450.dtsi b/arch/arm64/boot/dts/qcom/sm8450.dtsi > > new file mode 100644 > > index 000000000000..d838283bde4b > > --- /dev/null > > +++ b/arch/arm64/boot/dts/qcom/sm8450.dtsi > > @@ -0,0 +1,476 @@ > > +// SPDX-License-Identifier: BSD-3-Clause > > +/* > > + * Copyright (c) 2021, Linaro Limited > > + */ > > + > > +#include <dt-bindings/interrupt-controller/arm-gic.h> > > +#include <dt-bindings/clock/qcom,gcc-sm8450.h> > > +#include <dt-bindings/clock/qcom,rpmh.h> > > +#include <dt-bindings/soc/qcom,rpmh-rsc.h> > > + > > +/ { > > + interrupt-parent = <&intc>; > > + > > + #address-cells = <2>; > > + #size-cells = <2>; > > + > > + chosen { }; > > + > > + clocks { > > + xo_board: xo-board { > > + compatible = "fixed-clock"; > > + #clock-cells = <0>; > > + clock-frequency = <76800000>; > > + clock-output-names = "xo_board"; > > + }; > > + > > + sleep_clk: sleep-clk { > > + compatible = "fixed-clock"; > > + clock-frequency = <32000>; > > + #clock-cells = <0>; > > + }; > > No clock-output-names for this one? This should have one, will fix and update in v2 > > > Other than that: > > Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxx> Thanks for the review! -- ~Vinod