Hi Jacopo, On Tue, Jan 17, 2023 at 11:06:00AM +0100, Jacopo Mondi wrote: > Currently the imx258 driver requires to have the 'rotation' device node > property specified in DTS with a fixed value of 180 degrees. > > The "rotation" fwnode device property is intended to allow specify the > sensor's physical mounting rotation, so that it can be exposed through > the read-only V4L2_CID_CAMERA_SENSOR_ROTATION control and applications > can decide how to compensate for that. > > The imx258 driver has read-only VFLIP and HFLIP enabled, resulting in > a 180 degrees image rotation being produced by the sensor. But this > doesn't imply that the physical mounting rotation should match the > driver's implementation. > > I took into the series Robert's patch that register device node properties and > on top of that register flips controls, in order to remove the hard requirement > of the 180 degrees rotation property presence. Reconsidering these patches after the flipping vs. rotation discussion, they seem fine. The only thing I'd like to see, after removing the rotation property check, would be to add support for the actual flipping controls. I'm pretty sure they can be found in the same registers as on CCS --- the rest of the registers look very much like that. Would you like to send a patch? :-) -- Regards, Sakari Ailus
