On Thu, Aug 03, 2023 at 11:53:35AM +0200, Julien Stephan wrote:

On Mon, Jul 03, 2023 at 02:21:17PM +0200, AngeloGioacchino Del Regno wrote:
..snip..

quoted

quoted

+
+	mtk_cam_init_cfg(&cam->subdev, NULL);

mtk_cam_init_cfg() returns an integer, and this is probably because you expect
it to get logic to fail: in that case, check the result of that call here,
otherwise, make that function void.

Hi Angelo,

mtk_cam_init_cfg returns an int because it should match v4l2_subdev_pad_ops->init_cfg prototype..

mtk_cam_init does not fail, always return 0, so I though it would be ok
to discard the return value here. What do you think?

Fine with me.

quoted

quoted

+

..snip..

quoted

quoted

+	unsigned int bpp;

Can we get any format that uses a billion bits per pixel? :-P

u8 bpp;

I am preparing the driver for futur upgrades.. in a billion years ;)

-- 
Regards,

Laurent Pinchart

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel