• Jacopo Mondi's avatar
    media: imx258: Remove mandatory 180 degrees rotation · 23c3b904
    Jacopo Mondi authored
    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.
    
    Now that the driver registers V4L2_CID_CAMERA_SENSOR_ROTATION
    and V4L2_CID_HFLIP/VFLIP correctly, userspace has all the required
    information to handle the rotation correctly, hence it is not necessary
    to require the 'rotation' property to be fixed to 180 degrees in DTS.
    Signed-off-by: default avatarJacopo Mondi <jacopo.mondi@ideasonboard.com>
    Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
    Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
    23c3b904
imx258.c 31.8 KB