Re: [PATCH] staging: fbtft: Removed unnecessary parenthesis around conditions to comply with the checkpatch coding style.

Thanks for the feedback Bagas, "Then checkpatch is wrong, " lol Ill double check my SoB. Angus On Wed, Sep 13, 2023 at 11:32 AM Bagas Sanjaya wrote: > On Wed, Sep 13, 2023 at 11:02:13AM +1000, Angus Gardner wrote: > > --- > > drivers/staging/fbtft/fb_ra8875.c | 11 --- > > 1 file

[PATCH] staging: fbtft: Removed unnecessary parenthesis around conditions to comply with the checkpatch coding style.

--- drivers/staging/fbtft/fb_ra8875.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/staging/fbtft/fb_ra8875.c b/drivers/staging/fbtft/fb_ra8875.c index 398bdbf53c9a..658f915b8528 100644 --- a/drivers/staging/fbtft/fb_ra8875.c +++

Re: [PATCH] staging: fbtft: Removed unnecessary parenthesis around conditions to comply with the checkpatch coding style.

On Fri, Sep 15, 2023 at 11:58:47AM +1000, angus gardner wrote: > Thanks for the feedback Bagas, > > "Then checkpatch is wrong, " lol > > Ill double check my SoB. > tl;dr: > A: http://en.wikipedia.org/wiki/Top_post > Q: Were do I find info about this thing called top-posting? > A: Because it

Re: [PATCH] staging: fbtft: Removed unnecessary parenthesis around conditions to comply with the checkpatch coding style.

On Wed, Sep 13, 2023 at 11:02:13AM +1000, Angus Gardner wrote: > --- > drivers/staging/fbtft/fb_ra8875.c | 11 --- > 1 file changed, 4 insertions(+), 7 deletions(-) No patch description and SoB, so Greg can't take this as-is. > - if ((par->info->var.xres == 320) &&