Re: [PATCH v5 2/6] w1: ds2438: fixed if brackets coding style issue

2021-04-16 Thread Luiz Sampaio
On Fri, Apr 09, 2021 at 07:40:57AM -0700, Joe Perches wrote: > On Fri, 2021-04-09 at 00:09 -0300, Luiz Sampaio wrote: > > Since there is only one statement inside the if clause, no brackets are > > required. > [] > > diff --git a/drivers/w1/slaves/w1_ds2438.c b/drivers/w1/slaves/w1_ds2438.c > [] >

Re: [PATCH v5 2/6] w1: ds2438: fixed if brackets coding style issue

2021-04-09 Thread Joe Perches
On Fri, 2021-04-09 at 00:09 -0300, Luiz Sampaio wrote: > Since there is only one statement inside the if clause, no brackets are > required. [] > diff --git a/drivers/w1/slaves/w1_ds2438.c b/drivers/w1/slaves/w1_ds2438.c [] > @@ -287,9 +287,9 @@ static ssize_t iad_read(struct file *filp, struct

Re: [PATCH v5 2/6] w1: ds2438: fixed if brackets coding style issue

2021-04-08 Thread Joe Perches
On Fri, 2021-04-09 at 00:09 -0300, Luiz Sampaio wrote: > Since there is only one statement inside the if clause, no brackets are > required. > > Signed-off-by: Luiz Sampaio > --- >  drivers/w1/slaves/w1_ds2438.c | 16 >  1 file changed, 8 insertions(+), 8 deletions(-) > > diff

[PATCH v5 2/6] w1: ds2438: fixed if brackets coding style issue

2021-04-08 Thread Luiz Sampaio
Since there is only one statement inside the if clause, no brackets are required. Signed-off-by: Luiz Sampaio --- drivers/w1/slaves/w1_ds2438.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/w1/slaves/w1_ds2438.c b/drivers/w1/slaves/w1_ds2438.c