Hi Robert, On Wed, Jan 04, 2023 at 03:14:50PM +0100, Robert Mader wrote: > On 04.01.23 15:11, Jacopo Mondi wrote: > > Hi Robert > > > > On Wed, Jan 04, 2023 at 01:23:37PM +0100, Robert Mader wrote: > > > Analogous to e.g. the imx219. This enables propagating > > > V4L2_CID_CAMERA_ORIENTATION and V4L2_CID_CAMERA_SENSOR_ROTATION > > > values. > > > The motivation is to allow libcamera detect these values from the > > > device tree and propagate them further to e.g. Pipewire. > > > > > > While at it, reserve space for 3 additional controls even if > > > v4l2_ctrl_new_fwnode_properties() can only register 2 of > > > them, to fix the existing implementation which reserve space for 8 > > > controls but actually registers 9. > > > > > > Changes in v2: > > > - Reserve 11 instead of 10 controls > > > - Change order of variable declaration > > > - Slightly extend description > > This part should go below, after ---, as it shouldn't be part of the commit > > message > > > > The patch looks good > > > > Reviewed-by: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> > Thanks! And ops, right, make sense. Should I spin a v3 for that or is it > simple enough to fix when pulling? I can fix it while applying the patch. -- Regards, Sakari Ailus
