Hi Manivannan, On Fri, Nov 29, 2019 at 4:07 PM Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> wrote: } > + > + imx290->bpp = 10; > + > + break; > + case MEDIA_BUS_FMT_SRGGB12_1X12: > + ret = imx290_set_register_array(imx290, imx290_12bit_settings, > + ARRAY_SIZE( > + imx290_12bit_settings)); Could you please write the ARRAY_SIZE and its parameter in the same line? It would improve readability. Thanks