Re: [PATCH 1/1] arm64: dts: qcom: qrb5165-rb5: Enable the IMX577 on cam1

From: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
To: Bryan O'Donoghue <bryan.odonoghue@linaro.org>,
	robert.foss@linaro.org, todor.too@gmail.com, agross@kernel.org,
	bjorn.andersson@linaro.org
Cc: mchehab@kernel.org, robh+dt@kernel.org, krzk+dt@kernel.org,
	linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	devicetree@vger.kernel.org, mmitkov@quicinc.com,
	jgrahsl@snap.com, hfink@snap.com
Subject: Re: [PATCH 1/1] arm64: dts: qcom: qrb5165-rb5: Enable the IMX577 on cam1
Date: Mon, 23 May 2022 16:50:51 +0300	[thread overview]
Message-ID: <e7bbe076-a1d1-a005-8110-5d35bc0d80de@linaro.org> (raw)
In-Reply-To: <19c92f9d-fa1c-fbe8-50ef-324da3e00695@linaro.org>

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