Re: [PATCH 8/9] w1: fix whitespaces of struct declarations

2018-10-28 Thread Joe Perches
On Mon, 2018-10-29 at 03:02 +0100, Steffen Vogel wrote: > Hi Joe, > > v2 of this patch set is almost ready. > I reran checkpatch.pl with '--strict'. So more warnings will be fixed by > the second version. > > See my comments below. > > On Sun, Oct 28, 2018 at 06:52:23PM -0700, Joe Perches

Re: [PATCH 8/9] w1: fix whitespaces of struct declarations

2018-10-28 Thread Joe Perches
On Mon, 2018-10-29 at 03:02 +0100, Steffen Vogel wrote: > Hi Joe, > > v2 of this patch set is almost ready. > I reran checkpatch.pl with '--strict'. So more warnings will be fixed by > the second version. > > See my comments below. > > On Sun, Oct 28, 2018 at 06:52:23PM -0700, Joe Perches

Re: [PATCH 8/9] w1: fix whitespaces of struct declarations

2018-10-28 Thread Steffen Vogel
Hi Joe, v2 of this patch set is almost ready. I reran checkpatch.pl with '--strict'. So more warnings will be fixed by the second version. See my comments below. On Sun, Oct 28, 2018 at 06:52:23PM -0700, Joe Perches wrote: > On Sun, 2018-10-28 at 23:09 +0100, Steffen Vogel wrote: > > This fixes

Re: [PATCH 8/9] w1: fix whitespaces of struct declarations

2018-10-28 Thread Steffen Vogel
Hi Joe, v2 of this patch set is almost ready. I reran checkpatch.pl with '--strict'. So more warnings will be fixed by the second version. See my comments below. On Sun, Oct 28, 2018 at 06:52:23PM -0700, Joe Perches wrote: > On Sun, 2018-10-28 at 23:09 +0100, Steffen Vogel wrote: > > This fixes

Re: [PATCH 8/9] w1: fix whitespaces of struct declarations

2018-10-28 Thread Joe Perches
On Sun, 2018-10-28 at 23:09 +0100, Steffen Vogel wrote: > This fixes a warning raised by the checkpatch tool. checkpatch does not raise any whitespace errors on this file. It does suggest that braces could be moved. $ ./scripts/checkpatch.pl -f --strict drivers/w1/w1_netlink.h WARNING: Missing

Re: [PATCH 8/9] w1: fix whitespaces of struct declarations

2018-10-28 Thread Joe Perches
On Sun, 2018-10-28 at 23:09 +0100, Steffen Vogel wrote: > This fixes a warning raised by the checkpatch tool. checkpatch does not raise any whitespace errors on this file. It does suggest that braces could be moved. $ ./scripts/checkpatch.pl -f --strict drivers/w1/w1_netlink.h WARNING: Missing

[PATCH 8/9] w1: fix whitespaces of struct declarations

2018-10-28 Thread Steffen Vogel
This fixes a warning raised by the checkpatch tool. Signed-off-by: Steffen Vogel --- drivers/w1/w1_netlink.h | 28 +--- 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/drivers/w1/w1_netlink.h b/drivers/w1/w1_netlink.h index 08cbf08f3649..7873eb54352e

[PATCH 8/9] w1: fix whitespaces of struct declarations

2018-10-28 Thread Steffen Vogel
This fixes a warning raised by the checkpatch tool. Signed-off-by: Steffen Vogel --- drivers/w1/w1_netlink.h | 28 +--- 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/drivers/w1/w1_netlink.h b/drivers/w1/w1_netlink.h index 08cbf08f3649..7873eb54352e