Hi Bryan,
On 5/22/22 20:10, Bryan O'Donoghue wrote:
On 18/05/2022 20:09, Vladimir Zapolskiy wrote:
I run on you branch on top of linux-next, but switch build options from
modules to built-in
CONFIG_I2C_QCOM_CCI=y
CONFIG_VIDEO_QCOM_CAMSS=y
I didn't get the sensor initialized and hence there is no /dev/media0 node:
[ 0.620205] i2c-qcom-cci ac50000.cci: Found 19200000 cci clk rate
while 37500000 was expected
[ 0.620551] i2c 20-001a: Fixing up cyclic dependency with ac6a000.camss
[ 0.620754] imx412 20-001a: Looking up dovdd-supply from device tree
[ 0.620797] imx412 20-001a: Looking up avdd-supply from device tree
[ 0.620860] imx412 20-001a: Looking up dvdd-supply from device tree
[ 0.620876] duplicated lane 1 in clock-lanes, using defaults
[ 0.622789] imx412 20-001a: failed to find sensor: -5
[ 0.622880] imx412: probe of 20-001a failed with error -5
I believe the problem could be related to CCI, please remind me, are
there I2C bus pull-ups?
Hmm.
Just trying to replicate this on linux-next
https://git.linaro.org/people/bryan.odonoghue/kernel.git/log/?h=linux-next-22-05-22%2bimx577-rb5
root@linaro-gnome:~# zcat /proc/config.gz | grep -e CONFIG_I2C_QCOM_CCI
-e CONFIG_VIDEO_QCOM_CAMSS
CONFIG_I2C_QCOM_CCI=y
CONFIG_VIDEO_QCOM_CAMSS=y
root@linaro-gnome:~# uname -a
Linux linaro-gnome 5.18.0-rc7-next-20220518-00006-g3beef4d1d353-dirty
#40 SMP PREEMPT Sun May 22 17:53:29 IST 2022 aarch64 GNU/Linux
root@linaro-gnome:~# cam -l
Available cameras:
1: 'imx412' (/base/soc@0/cci@ac50000/i2c-bus@0/camera@1a)
are you compiling everything in ?
it's some kind of a race related to probes of CAMSS, CCI and IMX412 drivers.
Since I'm able to reproduce it, I'll take the analysis on myself, and it does not
interfere with your patch series.
--
Best wishes,
Vladimir