Re: [PATCH v3 11/25] media: i2c: imx258: Add get_selection for pixel array information
On 4/3/24 12:46, Pavel Machek wrote: > Hi! > >> Libcamera requires the cropping information for each mode, so >> add this information to the driver. > >> @@ -116,6 +124,9 @@ struct imx258_mode { >> u32 link_freq_index; >> /* Default register values */ >> struct imx258_reg_list reg_list; >> + >> +/* Analog crop rectangle. */ > > No need for "." at the end, as it is not above. > >> +struct v4l2_rect crop; >> }; > > If the crop is same in all modes, should we have it in common place? > > Best regards, > Pavel I gave this a try similar to what was done on imx219 but its having issues and makes a bunch of changes to the following patches so we might need to hold off on that change for now.
Re: [PATCH v3 11/25] media: i2c: imx258: Add get_selection for pixel array information
On 4/3/24 12:46, Pavel Machek wrote: > Hi! > >> Libcamera requires the cropping information for each mode, so >> add this information to the driver. > >> @@ -116,6 +124,9 @@ struct imx258_mode { >> u32 link_freq_index; >> /* Default register values */ >> struct imx258_reg_list reg_list; >> + >> +/* Analog crop rectangle. */ > > No need for "." at the end, as it is not above. > Done >> +struct v4l2_rect crop; >> }; > > If the crop is same in all modes, should we have it in common place? > > Best regards, > Pavel I see the imx219 made a similar change where it was changed from using that .crop format to just setting it in the set_pad_format function so we can do something similar here.