On Sat, 2022-01-29 at 12:38 +0100, Moses Christopher Bollavarapu wrote:
> There is a BIT(nr) macro available in Linux Kernel,
> which does the same thing.
>
> Example: BIT(7) = (1UL << 7)
[]
> diff --git a/drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c b/drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c
[]
> @@ -548,7 +548,7 @@ static long __ov5693_set_exposure(struct v4l2_subdev *sd, int coarse_itg,
> * The way is to write coarse_itg/2 to the reg, meanwhile write 2*hts
> * to the reg.
> */
> - if (coarse_itg > (1 << 15)) {
> + if (coarse_itg > BIT(15)) {
Not all uses of 1 left shift should be converted to BIT
Especially when used with a non-bit value comparison test.
This is a size and not a bit position so this is likely not appropriate.
It'd probably be better as
if (coarse_itg > 0x8000)
or
if (coarse_itg > 32768)
or
if (coarse_itg > SOME_CONSTANT_DEFINE)
> diff --git a/drivers/staging/media/atomisp/pci/atomisp_compat_css20.c b/drivers/staging/media/atomisp/pci/atomisp_compat_css20.c
[]
> @@ -1913,11 +1913,11 @@ void atomisp_css_input_set_mode(struct atomisp_sub_device *asd,
> &asd->stream_env[ATOMISP_INPUT_STREAM_GENERAL].stream_config;
> s_config->mode = IA_CSS_INPUT_MODE_TPG;
> s_config->source.tpg.mode = IA_CSS_TPG_MODE_CHECKERBOARD;
> - s_config->source.tpg.x_mask = (1 << 4) - 1;
> + s_config->source.tpg.x_mask = BIT(4) - 1;
These should probably use GENMASK
> diff --git a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c
[]
> @@ -626,11 +626,11 @@ static int atomisp_mrfld_pre_power_down(struct atomisp_device *isp)
> * IRQ, if so, waiting for it to be served
> */
> pci_read_config_dword(pdev, PCI_INTERRUPT_CTRL, &irq);
> - irq = irq & 1 << INTR_IIR;
> + irq = irq & BIT(INTR_IIR);
The rest seems sensible.
