Re: [PATCH] ARM: dts: msm8974: Add definitions for QCE & cryptobam

From: Iaroslav Gridin
Date: Wed Sep 07 2016 - 12:20:41 EST



> > + clocks = <&gcc GCC_CE2_AHB_CLK>,
> > + <&gcc GCC_CE2_AXI_CLK>,
> > + <&gcc GCC_CE2_CLK>;
> > + clock-names = "bam_clk", "axi_clk", "core_clk";
> > + #dma-cells = <1>;
> > + qcom,ee = <1>;
> > + qcom,controlled-remotely;
> > + };
>
> indentation please.

Similar indentation (tabs then spaces to align) is used all over dts, am I doing it wrong or is
this bad practice?

Example @ line 334:

> interrupts = <0 8 0x4>,
> <0 7 0x4>;