Re: [PATCH v5 0/2] media: i2c: Add os02g10 camera sensor driver

[Elgin Perumbilly]

Hi Sakari,

>Hi Sakari,
>
>>Hi Sakari,
>>
>>>The following features are supported:
>>>- Manual exposure and gain control support.
>>>- vblank/hblank control support.
>>>- vflip/hflip control support
>>>- Test pattern control support.
>>>- mode support 1920 x 1080 @ 30 fps, SBGGR10
>>>
>>>The driver is tested on mainline branch v7.0-rc2 on IMX8MP Debix Model a.
>>>
>>>v4l2-compliance 1.31.0-5387, 64 bits, 64-bit time_t
>>>v4l2-compliance SHA: 5508bc4301ac 2025-08-25 08:14:22
>>>
>>>Compliance test for device /dev/v4l-subdev3:
>>>
>>>Driver Info:
>>>        Driver version   : 7.0.0
>>>        Capabilities     : 0x00000000
>>>        Client Capabilities: 0x0000000000000002
>>>interval-uses-which
>>>Required ioctls:
>>>        test VIDIOC_SUDBEV_QUERYCAP: OK
>>>        test invalid ioctls: OK
>>>
>>>Allow for multiple opens:
>>>        test second /dev/v4l-subdev3 open: OK
>>>        test VIDIOC_SUBDEV_QUERYCAP: OK
>>>        test for unlimited opens: OK
>>>
>>>Debug ioctls:
>>>        test VIDIOC_LOG_STATUS: OK (Not Supported)
>>>
>>>Input ioctls:
>>>        test VIDIOC_G/S_TUNER/ENUM_FREQ_BANDS: OK (Not Supported)
>>>        test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
>>>        test VIDIOC_S_HW_FREQ_SEEK: OK (Not Supported)
>>>        test VIDIOC_ENUMAUDIO: OK (Not Supported)
>>>        test VIDIOC_G/S/ENUMINPUT: OK (Not Supported)
>>>        test VIDIOC_G/S_AUDIO: OK (Not Supported)
>>>        Inputs: 0 Audio Inputs: 0 Tuners: 0
>>>
>>>Output ioctls:
>>>        test VIDIOC_G/S_MODULATOR: OK (Not Supported)
>>>        test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
>>>        test VIDIOC_ENUMAUDOUT: OK (Not Supported)
>>>        test VIDIOC_G/S/ENUMOUTPUT: OK (Not Supported)
>>>        test VIDIOC_G/S_AUDOUT: OK (Not Supported)
>>>        Outputs: 0 Audio Outputs: 0 Modulators: 0
>>>
>>>Input/Output configuration ioctls:
>>>        test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported)
>>>        test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported)
>>>        test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported)
>>>        test VIDIOC_G/S_EDID: OK (Not Supported)
>>>
>>>Control ioctls:
>>>        test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK
>>>        test VIDIOC_QUERYCTRL: OK
>>>        test VIDIOC_G/S_CTRL: OK
>>>        test VIDIOC_G/S/TRY_EXT_CTRLS: OK
>>>        test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK
>>>        test VIDIOC_G/S_JPEGCOMP: OK (Not Supported)
>>>        Standard Controls: 12 Private Controls: 0
>>>
>>>Format ioctls:
>>>        test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK (Not Supported)
>>>        test VIDIOC_G/S_PARM: OK (Not Supported)
>>>        test VIDIOC_G_FBUF: OK (Not Supported)
>>>        test VIDIOC_G_FMT: OK (Not Supported)
>>>        test VIDIOC_TRY_FMT: OK (Not Supported)
>>>        test VIDIOC_S_FMT: OK (Not Supported)
>>>        test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported)
>>>        test Cropping: OK (Not Supported)
>>>        test Composing: OK (Not Supported)
>>>        test Scaling: OK (Not Supported)
>>>
>>>Codec ioctls:
>>>        test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported)
>>>        test VIDIOC_G_ENC_INDEX: OK (Not Supported)
>>>        test VIDIOC_(TRY_)DECODER_CMD: OK (Not Supported)
>>>
>>>Buffer ioctls:
>>>        test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK (Not Supported)
>>>        test CREATE_BUFS maximum buffers: OK
>>>        test VIDIOC_REMOVE_BUFS: OK
>>>        test VIDIOC_EXPBUF: OK (Not Supported)
>>>        test Requests: OK (Not Supported)
>>>        test blocking wait: OK (Not Supported)
>>>
>>>Total for device /dev/v4l-subdev3: 46, Succeeded: 46, Failed: 0, Warnings: 0
>>
>>Could you please have a look at this patch series and let me know your thoughts ?
>>
>>Best Regards,
>>Elgin
>
>Just following up on the os02g10 patch series.
>Is there any blocker preventing it from being merged, or anything I need to address?
>
>Best Regards,
>Elgin

Just following up once again on the OS02G10 patch series. Could you please let me know if you had a chance to review it?

Best Regards,
Elgin