Re: [PATCH v7 3/5] ARM: dts: add RK3xxx CPU Thermal data

From: Doug Anderson
Date: Tue Sep 30 2014 - 14:24:35 EST


On Sun, Sep 28, 2014 at 3:38 AM, Caesar Wang <> wrote:
> This patch changes a dtsi file to contain the thermal data
> on RK3288 and later SoCs. This data will
> enable a thermal shutdown over 125C.
> Signed-off-by: Caesar Wang <>
> ---
> arch/arm/boot/dts/rk3xxx-cpu-thermal.dtsi | 32 +++++++++++++++++++++++++++++++
> 1 file changed, 32 insertions(+)
> create mode 100644 arch/arm/boot/dts/rk3xxx-cpu-thermal.dtsi

I haven't done lots of review of this series, but I did notice that
this file has bad spacing. It's fairly obvious in your gerrit patch
because gerrit (by default) shows you "tab" characters...

In general you should use tab for indentation, not spaces. You seem
to use a mix of the two.

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at