[lkml]   [2019]   [Jan]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH RFC lora-next 1/4] net: lora: sx125x sx1301: correct style warnings
Hi Andreas,

On 29/12/2018 09:05, Andreas Färber wrote:
> Hi Ben,
> Am 19.12.18 um 16:56 schrieb Ben Whitten:
>> Checkpatch highlights some style issues which need to be addressed.
>> Signed-off-by: Ben Whitten <>
>> ---
>> drivers/net/lora/sx125x.c | 20 +++++++++------
>> drivers/net/lora/sx1301.c | 52 ++++++++++++++++++++++-----------------
>> drivers/net/lora/sx1301.h | 7 +++---
>> 3 files changed, 45 insertions(+), 34 deletions(-)
> Thanks for splitting this off from the functional changes. This will
> need some more explanations and discussion. In particular the comment
> changes look wrong to me. Also please don't butcher code just because
> may warn about line length at this early stage.

Yeh they seemed odd but checkpatch doesn't like a /* on a line by its
self it seems.

> A good catch in there was the unsigned casts, which are no longer
> necessary since the regmap conversion, so we should just squash that
> into the earlier commits.
> Note that I used to run as git commit hook:
> But since some git update that started to break git rebase -i in case of
> false positives (with rebase stopping and on trying to continue commits
> unintentionally getting squashed), so I deactivated it again and don't
> manually check each commit I stage anymore, in favor of slowly
> completing implementations to a working point.

Good to know, I'll avoid running it for the time and drop this from the v2

Ben Whitten

 \ /
  Last update: 2019-01-05 08:37    [W:0.048 / U:1.192 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site