Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1502320

Re: [PATCH v8 1/2] ARM: dts: Add TOPEET itop core board SCP package version

From ayaka <ayaka@soulik.info>
Newsgroups linux.kernel
Subject Re: [PATCH v8 1/2] ARM: dts: Add TOPEET itop core board SCP package version
Date 2016-10-17 20:00 +0200
Message-ID <stkmC-6DJ-17@gated-at.bofh.it> (permalink)
References <sj8Zr-7eA-19@gated-at.bofh.it> <sj8Zr-7eA-17@gated-at.bofh.it> <stiXw-5Qr-25@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw



On 10/18/2016 12:27 AM, Krzysztof Kozlowski wrote:
> On Mon, Sep 19, 2016 at 11:48:22PM +0800, Randy Li wrote:
>> The TOPEET itop is a samsung exnynos 4412 core board, which have
>> two package versions. This patch add the support for SCP version.
>>
>> Currently supported are USB3503A HSIC, USB OTG, eMMC, rtc and
>> PMIC. The future features are in the based board. Also MFC and
>> watchdog have been enabled.
>>
>> Signed-off-by: Randy Li <ayaka@soulik.info>
>> ---
>>   arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi | 501 ++++++++++++++++++++++++
>>   1 file changed, 501 insertions(+)
>>   create mode 100644 arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi
> I wanted to apply it... but then I saw a bunch of checkpatch trivial issues.
> Really, after v8? The code must compile (v6 did not compile...), there
> should be no warnings from smatch, sparse and checkpatch (only the last
> one is applicable for DTS). Unless of course checkpatch would be
> wrong... but in this case it is correct. You did not follow coding
> style:
>
> WARNING: please, no spaces at the start of a line
> #134: FILE: arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi:109:
> +        devfreq = <&bus_leftbus>;$
>
> ERROR: code indent should use tabs where possible
> #135: FILE: arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi:110:
> +        status = "okay";$
Oh, it is a copy mistake, the copy operation makes the original tab 
becomes spaces.
Should I send a new version to correct them?
>
>
> Best regards,
> Krzysztof

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH v8 1/2] ARM: dts: Add TOPEET itop core board SCP package  version Krzysztof Kozlowski <krzk@kernel.org> - 2016-10-17 18:30 +0200
  Re: [PATCH v8 1/2] ARM: dts: Add TOPEET itop core board SCP package  version ayaka <ayaka@soulik.info> - 2016-10-17 20:00 +0200
    Re: [PATCH v8 1/2] ARM: dts: Add TOPEET itop core board SCP package  version Krzysztof Kozlowski <krzk@kernel.org> - 2016-10-17 21:50 +0200

csiph-web