Messages in this thread | | | Date | Sat, 6 Apr 2024 00:36:47 -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. >
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.
| |