Using CCI_I2C Driver for OV9282 on Qualcomm RB5 Vision Kit

Using CCI_I2C Driver for OV9282 on Qualcomm RB5 Vision Kit

[halil.karabacak]

Hello everyone,

I have been trying to use CCI_I2C driver instead of I2C on my OV9282 
with video4linux2(I want to see as /dev/videoX). However, I was not able 
find much, but yesterday I found out 
https://patchwork.kernel.org/project/linux-media/list/?series=647614 
this kernel patchwork focusing on camss & camera drivers. I believe that 
this patch also requires some other patches to work as intended on 
kernel. Is there a way that I can use these drivers on my 
5.15.0(5.15.0-qcomlt-arm64) kernel?


Regards,
Halil Karabacak

Re: Using CCI_I2C Driver for OV9282 on Qualcomm RB5 Vision Kit

[Bryan O'Donoghue]

On 23/08/2022 08:25, halil.karabacak wrote:
> Hello everyone,
> 
> I have been trying to use CCI_I2C driver instead of I2C on my OV9282 
> with video4linux2(I want to see as /dev/videoX). However, I was not able 
> find much, but yesterday I found out 
> https://patchwork.kernel.org/project/linux-media/list/?series=647614 
> this kernel patchwork focusing on camss & camera drivers. I believe that 
> this patch also requires some other patches to work as intended on 
> kernel. Is there a way that I can use these drivers on my 
> 5.15.0(5.15.0-qcomlt-arm64) kernel?
> 
> 
> Regards,
> Halil Karabacak

Hi Halil.

Please give this kernel a try.
https://git.linaro.org/people/bryan.odonoghue/kernel.git/log/?h=linux-stable-22-07-18-imx577-camss

I've also included a defconfig there so others may pick up on the work.

To get the ov9282 to power up on the RB5 we will need to add in 
regulator_block_dostuff() support as was done with the imx412/imx477/imx577.

* 3de9dc7f67731 - media: i2c: imx412: Add bulk regulator support
* 9a199694c6a15 - media: i2c: imx412: Fix power_off ordering
* bb25f071fc92d - media: i2c: imx412: Fix reset GPIO polarity

Also the polarity on the GPIO now that I look at the git log.

Basically right now the media driver doesn't support switching the 
regulator on and we should make sure the polarity is correct for the chip.

I'd not expect the device to respond at all given it is not powered and 
might be held in reset as is.

I might find a few cycles to poke at this if you don't get to it first.

---
bod

Re: Using CCI_I2C Driver for OV9282 on Qualcomm RB5 Vision Kit

[Bryan O'Donoghue]

On 24/08/2022 12:08, halil.karabacak wrote:
> After some trials and errors, I achieved something but, are these 
> resolutions in the attached file correct? And I could read nothing from 
> the camera right now.
> Regards,
> Halil

They don't look right.

Are you trying with the imx577 or the ov9282 ?

Re: Using CCI_I2C Driver for OV9282 on Qualcomm RB5 Vision Kit

[Bryan O'Donoghue]

On 24/08/2022 14:34, Bryan O'Donoghue wrote:
> On 24/08/2022 12:08, halil.karabacak wrote:
>> After some trials and errors, I achieved something but, are these 
>> resolutions in the attached file correct? And I could read nothing 
>> from the camera right now.
>> Regards,
>> Halil
> 
> They don't look right.
> 
> Are you trying with the imx577 or the ov9282 ?

So I had a poke at the ov9282 today.

ov9282 is detected on this branch here.

https://git.linaro.org/people/bryan.odonoghue/kernel.git/log/?h=linux-stable-22-08-01-imx577-camss

libcamera can't eat Y10P though
[1]

Ongoing irritant is that i2c addresses are right shifted by 1 bit in 
addressing so imx577@x34 is imx577@0x1a and ov9282@0xC0 is ov9282@0x60, 
we should fix that, its very confusing.

Anyway I thought I'd share

---
bod

[1]

root@linaro-gnome:/home/linaro/libcamera/build/src# ./cam/cam -l
[0:08:23.443968209] [3393]  INFO IPAManager ipa_manager.cpp:138 
libcamera is not installed. Adding 
'/home/linaro/libcamera/build/src/ipa' to the IPA search path
[0:08:23.489820187] [3393]  INFO Camera camera_manager.cpp:293 libcamera 
v0.0.0+3222-1b30992b
[0:08:23.506972394] [3396]  WARN CameraSensor camera_sensor.cpp:197 
'ov9282 21-0060': Recommended V4L2 control 0x009a0922 not supported
[0:08:23.507017863] [3396] ERROR V4L2 v4l2_subdevice.cpp:286 'ov9282 
21-0060': Unable to get rectangle 2 on pad 0: Inappropriate ioctl for device
[0:08:23.507044685] [3396]  WARN CameraSensor camera_sensor.cpp:224 
'ov9282 21-0060': The PixelArraySize property has been defaulted to 1280x720
[0:08:23.507067862] [3396] ERROR V4L2 v4l2_subdevice.cpp:286 'ov9282 
21-0060': Unable to get rectangle 1 on pad 0: Inappropriate ioctl for device
[0:08:23.507087393] [3396]  WARN CameraSensor camera_sensor.cpp:235 
'ov9282 21-0060': The PixelArrayActiveAreas property has been defaulted 
to (0x0)/1280x720
[0:08:23.507114372] [3396] ERROR V4L2 v4l2_subdevice.cpp:286 'ov9282 
21-0060': Unable to get rectangle 0 on pad 0: Inappropriate ioctl for device
[0:08:23.507131664] [3396]  WARN CameraSensor camera_sensor.cpp:243 
'ov9282 21-0060': Failed to retrieve the sensor crop rectangle
[0:08:23.507148382] [3396]  WARN CameraSensor camera_sensor.cpp:249 
'ov9282 21-0060': The sensor kernel driver needs to be fixed
[0:08:23.507170622] [3396]  WARN CameraSensor camera_sensor.cpp:251 
'ov9282 21-0060': See Documentation/sensor_driver_requirements.rst in 
the libcamera sources for more information
[0:08:23.508584465] [3396]  WARN CameraSensorProperties 
camera_sensor_properties.cpp:141 No static properties available for 'ov9282'
[0:08:23.508604829] [3396]  WARN CameraSensorProperties 
camera_sensor_properties.cpp:143 Please consider updating the camera 
sensor properties database
[0:08:23.508616444] [3396]  WARN CameraSensor camera_sensor.cpp:414 
'ov9282 21-0060': Failed to retrieve the camera location
[0:08:23.508987014] [3396]  WARN CameraSensor camera_sensor.cpp:197 
'imx412 22-001a': Recommended V4L2 control 0x009a0922 not supported
[0:08:23.509003941] [3396] ERROR V4L2 v4l2_subdevice.cpp:286 'imx412 
22-001a': Unable to get rectangle 2 on pad 0: Inappropriate ioctl for device
[0:08:23.509015607] [3396]  WARN CameraSensor camera_sensor.cpp:224 
'imx412 22-001a': The PixelArraySize property has been defaulted to 
4056x3040
[0:08:23.509024149] [3396] ERROR V4L2 v4l2_subdevice.cpp:286 'imx412 
22-001a': Unable to get rectangle 1 on pad 0: Inappropriate ioctl for device
[0:08:23.509031649] [3396]  WARN CameraSensor camera_sensor.cpp:235 
'imx412 22-001a': The PixelArrayActiveAreas property has been defaulted 
to (0x0)/4056x3040
[0:08:23.509043107] [3396] ERROR V4L2 v4l2_subdevice.cpp:286 'imx412 
22-001a': Unable to get rectangle 0 on pad 0: Inappropriate ioctl for device
[0:08:23.509050815] [3396]  WARN CameraSensor camera_sensor.cpp:243 
'imx412 22-001a': Failed to retrieve the sensor crop rectangle
[0:08:23.509057482] [3396]  WARN CameraSensor camera_sensor.cpp:249 
'imx412 22-001a': The sensor kernel driver needs to be fixed
[0:08:23.509064148] [3396]  WARN CameraSensor camera_sensor.cpp:251 
'imx412 22-001a': See Documentation/sensor_driver_requirements.rst in 
the libcamera sources for more information
[0:08:23.509659508] [3396]  WARN CameraSensorProperties 
camera_sensor_properties.cpp:141 No static properties available for 'imx412'
[0:08:23.509673102] [3396]  WARN CameraSensorProperties 
camera_sensor_properties.cpp:143 Please consider updating the camera 
sensor properties database
[0:08:23.509682216] [3396]  WARN CameraSensor camera_sensor.cpp:414 
'imx412 22-001a': Failed to retrieve the camera location
[0:08:23.520666868] [3396]  WARN V4L2 v4l2_pixelformat.cpp:287 
Unsupported V4L2 pixel format Y10P
[0:08:23.520719316] [3396] ERROR SimplePipeline simple.cpp:506 No valid 
configuration found

Re: Using CCI_I2C Driver for OV9282 on Qualcomm RB5 Vision Kit

[Bryan O'Donoghue]

On 07/09/2022 22:46, Kieran Bingham wrote:
> Hi Bryan,
> 
> Quoting Bryan O'Donoghue (2022-09-07 17:03:51)
>> On 24/08/2022 14:34, Bryan O'Donoghue wrote:
>>> On 24/08/2022 12:08, halil.karabacak wrote:
>>>> After some trials and errors, I achieved something but, are these
>>>> resolutions in the attached file correct? And I could read nothing
>>>> from the camera right now.
>>>> Regards,
>>>> Halil
>>>
>>> They don't look right.
>>>
>>> Are you trying with the imx577 or the ov9282 ?
>>
>> So I had a poke at the ov9282 today.
>>
>> ov9282 is detected on this branch here.
>>
>> https://git.linaro.org/people/bryan.odonoghue/kernel.git/log/?h=linux-stable-22-08-01-imx577-camss
>>
>> libcamera can't eat Y10P though
>> [1]
> 
> Hrm - it should 'now'.
> 
> It was originally added in [2] in November last year - but the
> corresponding V4L2 entry [3] was missed. It's been added last month, so
> should work now. Please let us know if it still causes you issues.
> 
> [2] https://git.libcamera.org/libcamera/libcamera.git/commit/?id=9e3470ad3e4f03364c8160917117d62fde5a7d88
> [3] https://git.libcamera.org/libcamera/libcamera.git/commit/?id=26c82ce13697e1af5950f4935ecff83c6453f351
> 
> --
> Kieran

Ah NICE

linaro@linaro-gnome:~/libcamera/build/src/cam$ git show
commit 6225d647b47982833fa654bf9cf65eb8c04d2036 (HEAD -> master, 
origin/master, origin/HEAD)
Author: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date:   Sat Sep 3 17:44:00 2022 +0300

Available cameras:
1: 'ov9282' (/base/soc@0/cci@ac4f000/i2c-bus@1/camera@60)
2: 'imx412' (/base/soc@0/cci@ac50000/i2c-bus@0/camera@1a)

---
bod