Re: [PATCH v3] ARM: dts: exynos5422-odroidxu3: Hook up PWM and use it for LEDs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



>>>>> "Krzysztof" == Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> writes:

Krzysztof> W dniu 13.05.2015 o 13:01, Peter Chubb pisze:

Krzysztof> Please run checkpatch before sending a patch. Here you have
Krzysztof> trailing whitespace error.

Arrgh --- thanks for spotting that.

Krzysztof> Additionally the indentation is broken in most of the lines
Krzysztof> - mixed tabs with spaces. Just stick to the kernel coding
Krzysztof> convention - tabs (8 characters wide).

And that.  I'm working on another project simultaneously that uses
4-space indent, and must have forgotten to reset Emacs's indent rules.

Krzysztof> Best regards, Krzysztof

V4 coming soon.
--
Dr Peter Chubb				        peter.chubb AT nicta.com.au
http://www.ssrg.nicta.com.au          Software Systems Research Group/NICTA
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux