Messages in this thread | ![/](/images/icornerl.gif) | | Date | Wed, 4 Jan 2023 15:59:53 +0000 | From | Sakari Ailus <> | Subject | Re: [PATCH v2] media: i2c: imx258: Parse and register properties |
| |
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@ideasonboard.com> > 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
| ![\](/images/icornerr.gif) |