Messages in this thread | | | Date | Mon, 8 Apr 2024 23:23:12 -0600 | Subject | Re: [PATCH v3 11/25] media: i2c: imx258: Add get_selection for pixel array information | From | Luis Garcia <> |
| |
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.
| |