Re: [PATCH v2 3/3] staging/fbtft: Fix braces coding style

Hi,

On Sat, Jul 24, 2021 at 08:01:53PM +0200, Geert Uytterhoeven wrote:

Hi Len,

On Sat, Jul 24, 2021 at 7:44 PM Len Baker len.baker@gmx.com wrote:

...

Add braces to the "for" loop and remove braces from the "if" statement. This way the kernel coding style is followed.

Signed-off-by: Len Baker len.baker@gmx.com

Thanks for your patch!

...

--- a/drivers/staging/fbtft/fbtft-core.c +++ b/drivers/staging/fbtft/fbtft-core.c

...

@@ -1016,10 +1018,9 @@ int fbtft_init_display(struct fbtft_par *par)

    i = 0;
    while (i < FBTFT_MAX_INIT_SEQUENCE) {

•           if (par->init_sequence[i] == -3) {
  

•                   /* done */
  

•                   return 0;
  

•           }
  

These braces should not be removed, due to the presence of the comment.

Ok, I leave it as is.

Thanks, Len