Re: [Outreachy kernel] [PATCH v2] Staging: fbtft: Remove unnecessary parenthesis and split lines to increase readability

[Greg KH <gregkh@linuxfoundation.org>]

On Sun, Mar 15, 2015 at 05:27:05PM +0530, Vatika Harlalka wrote:
> Remove unnecessary parenthesis in condition statement and
> split lines to increase readability.
> 
> Signed-off-by: Vatika Harlalka <vatikaharlalka@gmail.com>
> ---
>  Changes in v2: Changed subject line to make it more
> 		descriptive.
>  drivers/staging/fbtft/fb_ra8875.c | 16 ++++++++++------
>  1 file changed, 10 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/staging/fbtft/fb_ra8875.c b/drivers/staging/fbtft/fb_ra8875.c
> index e21af6c..bc8ca1e 100644
> --- a/drivers/staging/fbtft/fb_ra8875.c
> +++ b/drivers/staging/fbtft/fb_ra8875.c
> @@ -72,12 +72,13 @@ static int init_display(struct fbtft_par *par)
>  
>  	fbtft_par_dbg(DEBUG_INIT_DISPLAY, par,
>  		"%s()\n", __func__);
> -	fbtft_par_dbg(DEBUG_INIT_DISPLAY, par,
> -		"display size %dx%d\n", par->info->var.xres, par->info->var.yres);
> +	fbtft_par_dbg(DEBUG_INIT_DISPLAY, par, "display size %dx%d\n",
> +			par->info->var.xres, par->info->var.yres);
>  
>  	par->fbtftops.reset(par);
>  
> -	if ((par->info->var.xres == 320) && (par->info->var.yres == 240)) {
> +	if (par->info->var.xres == 320 &&
> +		par->info->var.yres == 240) {

While yes, these () are not needed, it's good to keep them there, just
to be "obvious" as to what is going on.  So they are fine to leave
alone.

thanks,

greg k-h