[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <39cb76ef-3f51-438d-b740-827a4c70035a@qualcomm.com>
Date: Fri, 12 Sep 2025 10:23:22 +0800
From: Wenmeng Liu <quic_wenmliu@...lcomm.com>
To: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
CC: Loic Poulain <loic.poulain@....qualcomm.com>,
Robert Foss
<rfoss@...nel.org>, Andi Shyti <andi.shyti@...nel.org>,
Rob Herring
<robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley
<conor+dt@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio
<konradybcio@...nel.org>,
<vladimir.zapolskiy@...aro.org>, <todor.too@...il.com>,
<bryan.odonoghue@...aro.org>, <linux-i2c@...r.kernel.org>,
<linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <linux-media@...r.kernel.org>,
Konrad Dybcio <konrad.dybcio@....qualcomm.com>
Subject: Re: [PATCH v5 3/3] arm64: dts: qcom: lemans-evk-camera: Add DT
overlay
On 9/11/2025 8:15 PM, Dmitry Baryshkov wrote:
> On Thu, Sep 11, 2025 at 07:55:16PM +0800, Wenmeng Liu wrote:
>> Enable IMX577 via CCI1 on LeMans EVK Core Kit.
>>
>> The LeMans EVK board does not include a camera sensor
>> by default, this overlay reflects the possibility of
>> attaching an optional camera sensor.
>> For this reason, the camera sensor configuration is
>> placed in lemans-evk-camera.dtso, rather than
>> modifying the base lemans-evk.dts.
>
> So, is it the only camera configuration that we are going to support?
> If not, what happens when we get the second sensor or second set of
> sensors to support?
You're right, although currently we only plan to support the IMX577 on
CCI1.>
>>
>> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
>> Reviewed-by: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
>> Signed-off-by: Wenmeng Liu <quic_wenmliu@...lcomm.com>
>> ---
>> arch/arm64/boot/dts/qcom/Makefile | 4 +
>> arch/arm64/boot/dts/qcom/lemans-evk-camera.dtso | 97 +++++++++++++++++++++++++
>
> At least the filename should mention that this is imx577.
>
>> 2 files changed, 101 insertions(+)
>
Would lemans-evk-camera-csi1-imx577.dtso be a more appropriate name?
It more precisely describes the content of the dtso.
Thanks,
Wenmeng
Powered by blists - more mailing lists