Re: [PATCH 1/5] staging: fbtft: fix Macros with complex values should be enclosed in parentheses

From: Greg KH
Date: Wed Jan 28 2015 - 15:47:30 EST


On Wed, Jan 28, 2015 at 01:19:25PM +0200, Heba Aamer wrote:
> This patch fixes the following checkpatch.pl error:
> fix Macros with complex values should be enclosed in parentheses
>
> Signed-off-by: Heba Aamer <heba93aamer@xxxxxxxxx>
> ---
> drivers/staging/fbtft/fb_pcd8544.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/fbtft/fb_pcd8544.c b/drivers/staging/fbtft/fb_pcd8544.c
> index 678ab8e..87ca96e 100644
> --- a/drivers/staging/fbtft/fb_pcd8544.c
> +++ b/drivers/staging/fbtft/fb_pcd8544.c
> @@ -33,7 +33,7 @@
> #define DRVNAME "fb_pcd8544"
> #define WIDTH 84
> #define HEIGHT 48
> -#define TXBUFLEN 84*6
> +#define TXBUFLEN (84*6)
> #define DEFAULT_GAMMA "40" /* gamma is used to control contrast in this driver */
>
> static unsigned tc = 0;

This is already in my tree, did you redo this series and not say a 'v2'
in them?

confused,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/