[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20161122034332.GA113841@google.com>
Date: Mon, 21 Nov 2016 19:43:33 -0800
From: Brian Norris <briannorris@...omium.org>
To: Caesar Wang <caesar.upstream@...il.com>
Cc: Caesar Wang <wxt@...k-chips.com>, Zhang Rui <rui.zhang@...el.com>,
Eduardo Valentin <edubezval@...il.com>,
Heiko Stuebner <heiko@...ech.de>, linux-pm@...r.kernel.org,
linux-kernel@...r.kernel.org,
Stephen Barber <smbarber@...omium.org>,
linux-rockchip@...ts.infradead.org
Subject: Re: [PATCH 2/3] thermal: rockchip: improve conversion error messages
On Tue, Nov 22, 2016 at 10:33:27AM +0800, Caesar Wang wrote:
> 在 2016年11月22日 10:15, Brian Norris 写道:
> >On Tue, Nov 22, 2016 at 09:51:23AM +0800, Caesar Wang wrote:
> >>CHECK: Alignment should match open parenthesis
> >>#428: FILE: drivers/thermal/rockchip_thermal.c:428:
> >>+ pr_err("%s: invalid temperature, temp=%d error=%d\n",
> >>+ __func__, temp, error);
> >>
> >>CHECK: Alignment should match open parenthesis
> >>#480: FILE: drivers/thermal/rockchip_thermal.c:480:
> >>+ pr_err("%s: invalid conversion table, mode=%d\n",
> >>+ __func__, table->mode);
> >What patch are you checking? I ran mine through checkpatch, and there
> >are no problems.
>
> That just checkcode on Chromeos kernelv4.4, that trivial things :)
> $chromiumos/src/third_party/kernel/v4.4$ checkcode
I'm not familiar with that tool, and that repository isn't upstream
but...
> drivers/thermal/rockchip_thermal.c
> CHECK: Alignment should match open parenthesis
> #428: FILE: drivers/thermal/rockchip_thermal.c:428:
> + pr_err("%s: invalid temperature, temp=%d error=%d\n",
> + __func__, temp, error);
...on approximately my 10th read of this...I guess maybe this tool has
determined that 2 tabs is 1 character too much, because the second line
lines up with the '%' instead of the '"'. If this is so important to
you, you can of course edit my patch and include it with yours.
Regards,
Brian
Powered by blists - more mailing lists