* [PATCH] staging:bcm2048 : Add parentheses around variable x
@ 2016-12-03 9:44 Tabrez khan
2017-02-03 9:55 ` Mauro Carvalho Chehab
2017-02-03 10:12 ` Dan Carpenter
0 siblings, 2 replies; 4+ messages in thread
From: Tabrez khan @ 2016-12-03 9:44 UTC (permalink / raw)
To: mchehab; +Cc: linux-media, devel, linux-kernel
Add parentheses around variable x for the readability purpose.
This warning was found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabrez21@gmail.com>
---
drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c
index 4d9bd02..2f28dd0 100644
--- a/drivers/staging/media/bcm2048/radio-bcm2048.c
+++ b/drivers/staging/media/bcm2048/radio-bcm2048.c
@@ -185,7 +185,7 @@
#define v4l2_to_dev(f) ((f * BCM2048_FREQV4L2_MULTI) / BCM2048_FREQDEV_UNIT)
#define msb(x) ((u8)((u16)x >> 8))
-#define lsb(x) ((u8)((u16)x & 0x00FF))
+#define lsb(x) ((u8)((u16)(x) & 0x00FF))
#define compose_u16(msb, lsb) (((u16)msb << 8) | lsb)
#define BCM2048_DEFAULT_POWERING_DELAY 20
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] staging:bcm2048 : Add parentheses around variable x
2016-12-03 9:44 [PATCH] staging:bcm2048 : Add parentheses around variable x Tabrez khan
@ 2017-02-03 9:55 ` Mauro Carvalho Chehab
2017-02-03 10:12 ` Dan Carpenter
1 sibling, 0 replies; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2017-02-03 9:55 UTC (permalink / raw)
To: Tabrez khan; +Cc: mchehab, linux-media, devel, linux-kernel
Em Sat, 3 Dec 2016 15:14:26 +0530
Tabrez khan <khan.tabrez21@gmail.com> escreveu:
> Add parentheses around variable x for the readability purpose.
>
> This warning was found using checkpatch.pl.
>
> Signed-off-by: Tabrez khan <khan.tabrez21@gmail.com>
> ---
> drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c
> index 4d9bd02..2f28dd0 100644
> --- a/drivers/staging/media/bcm2048/radio-bcm2048.c
> +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c
> @@ -185,7 +185,7 @@
> #define v4l2_to_dev(f) ((f * BCM2048_FREQV4L2_MULTI) / BCM2048_FREQDEV_UNIT)
>
> #define msb(x) ((u8)((u16)x >> 8))
> -#define lsb(x) ((u8)((u16)x & 0x00FF))
> +#define lsb(x) ((u8)((u16)(x) & 0x00FF))
If you're willing to do that, you should do on all macros. Also,
plese remove the extra space before the hexa value.
> #define compose_u16(msb, lsb) (((u16)msb << 8) | lsb)
>
> #define BCM2048_DEFAULT_POWERING_DELAY 20
Thanks,
Mauro
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] staging:bcm2048 : Add parentheses around variable x
2016-12-03 9:44 [PATCH] staging:bcm2048 : Add parentheses around variable x Tabrez khan
2017-02-03 9:55 ` Mauro Carvalho Chehab
@ 2017-02-03 10:12 ` Dan Carpenter
2017-02-15 17:26 ` Tabrez Khan
1 sibling, 1 reply; 4+ messages in thread
From: Dan Carpenter @ 2017-02-03 10:12 UTC (permalink / raw)
To: Tabrez khan; +Cc: mchehab, devel, linux-kernel, linux-media
On Sat, Dec 03, 2016 at 03:14:26PM +0530, Tabrez khan wrote:
> Add parentheses around variable x for the readability purpose.
>
It's not really about readability...
regards,
dan carpenter
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] staging:bcm2048 : Add parentheses around variable x
2017-02-03 10:12 ` Dan Carpenter
@ 2017-02-15 17:26 ` Tabrez Khan
0 siblings, 0 replies; 4+ messages in thread
From: Tabrez Khan @ 2017-02-15 17:26 UTC (permalink / raw)
To: Dan Carpenter; +Cc: mchehab, devel, linux-kernel, linux-media
On Fri, Feb 3, 2017 at 3:42 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> On Sat, Dec 03, 2016 at 03:14:26PM +0530, Tabrez khan wrote:
>> Add parentheses around variable x for the readability purpose.
>>
>
> It's not really about readability...
>
> regards,
> dan carpenter
>
I will resend it again.
--
Regards
Tabrez Khan
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-02-15 17:26 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-03 9:44 [PATCH] staging:bcm2048 : Add parentheses around variable x Tabrez khan
2017-02-03 9:55 ` Mauro Carvalho Chehab
2017-02-03 10:12 ` Dan Carpenter
2017-02-15 17:26 ` Tabrez Khan
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.