Re: [PATCH] [media] v4l: omap4iss: Fix using BIT macro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Sat, 1 Oct 2016 16:37:46 -0700
Wayne Porter <wporter82@xxxxxxxxx> escreveu:

> Checks found by checkpatch
> 
> Signed-off-by: Wayne Porter <wporter82@xxxxxxxxx>
> ---
>  drivers/staging/media/omap4iss/iss_regs.h | 76 +++++++++++++++----------------
>  1 file changed, 38 insertions(+), 38 deletions(-)
> 
> diff --git a/drivers/staging/media/omap4iss/iss_regs.h b/drivers/staging/media/omap4iss/iss_regs.h
> index cb415e8..c675212 100644
> --- a/drivers/staging/media/omap4iss/iss_regs.h
> +++ b/drivers/staging/media/omap4iss/iss_regs.h
> @@ -42,7 +42,7 @@
>  #define ISS_CTRL_CLK_DIV_MASK				(3 << 4)
>  #define ISS_CTRL_INPUT_SEL_MASK				(3 << 2)
>  #define ISS_CTRL_INPUT_SEL_CSI2A			(0 << 2)
> -#define ISS_CTRL_INPUT_SEL_CSI2B			(1 << 2)
> +#define ISS_CTRL_INPUT_SEL_CSI2B			BIT(2)

Converting just a few of such macros won't help. Either convert all
or none.

Also, as most of the bit masks here have more than one bit, you should
use GENMASK(), instead of BIT, like:

#define ISS_CTRL_CLK_DIV_MASK		GENMASK(4, 5)
#define ISS_CTRL_INPUT_SEL_MASK		GENMASK(2, 3)
#define   ISS_CTRL_INPUT_SEL_CSI2A	0
#define   ISS_CTRL_INPUT_SEL_CSI2B	BIT(2)

Yet, not sure if I would like such patch, as this kind of change
could easily break the driver if you make any typo at the GENMASK
parameters.


Thanks,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux