[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ba4b1276-90cc-4cff-b80b-d4d096fc7bc2@quicinc.com>
Date: Thu, 11 Sep 2025 19:11:56 +0530
From: Vikram Sharma <quic_vikramsa@...cinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>,
Nihal Kumar Gupta
<quic_nihalkum@...cinc.com>
CC: <bryan.odonoghue@...aro.org>, <mchehab@...nel.org>, <robh@...nel.org>,
<krzk+dt@...nel.org>, <conor+dt@...nel.org>, <andersson@...nel.org>,
<konradybcio@...nel.org>, <hverkuil-cisco@...all.nl>,
<cros-qcom-dts-watchers@...omium.org>, <catalin.marinas@....com>,
<will@...nel.org>, <linux-arm-kernel@...ts.infradead.org>,
<quic_svankada@...cinc.com>, <linux-media@...r.kernel.org>,
<linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>,
Ravi Shankar <quic_rshankar@...cinc.com>,
Vishal Verma <quic_vishverm@...cinc.com>
Subject: Re: [PATCH 3/4] arm64: dts: qcom: monaco-evk-camera: Add DT overlay
On 9/11/2025 6:33 PM, Dmitry Baryshkov wrote:
> On Thu, Sep 11, 2025 at 06:19:02PM +0530, Nihal Kumar Gupta wrote:
>>
>>
>> On 11-09-2025 15:38, Dmitry Baryshkov wrote:
>>> On Thu, Sep 11, 2025 at 02:49:59PM +0530, Nihal Kumar Gupta wrote:
>>>>
>>>> On 10-09-2025 12:33, Vikram Sharma wrote:
>>>>> On 9/9/2025 7:31 PM, Dmitry Baryshkov wrote:
>>>>>> On Tue, Sep 09, 2025 at 05:12:40PM +0530, Vikram Sharma wrote:
>>>>>>> From: Nihal Kumar Gupta<quic_nihalkum@...cinc.com>
>>>>>>>
>>>>>>> Enable IMX577 sensor using the CCI1 interface on Monaco EVK.
>>>>>>> Camera reset is controlled through an I2C expander,
>>>>>>> and power supply is managed via tlmm GPIO74.
>>>>>> Why is done as an overlay? Is it a seprate mezzanine?
>>>>> It’s not a separate mezzanine; monaco-evk does not come with an attached camera
>>>>> sensor by default.
>>>>> The overlay simply reflects an optional camera sensor attachment (e.g., IMX577
>>>>> on CSIPHY1)
>>>>> That is why camera sensor is configured in monaco-evk-camera.dtso instead of
>>>>> modifying the base monaco-evk.dts.
>>>>> Please suggest and alternative approach if you feel this is not correct
>>>> Dmitry, Do you agree with this justification or you have a follow-up comment?
>>>> Can we post v2 with the same change addressing other comments.
>>> The previous email didn't land to the mailing list, I'm not sure why.
>>>
>>> Anyway:
>>> - What are other options that we might support? Having a single
>>> monaco-evk-camera overlay means that this is the only configuration
>>> that we plan to support.
>>>
>>> - It all should have been described in the commit message. Please update
>>> it for the next revision.
>>
>> Yes, We have planned to support device tree overlay of the IMX577 sensor via CSIPHY1 only.
>> I'll update the commit message in v2
>
> I'd still suggest monaco-evk-camera-imx577.dtso
ACK, Will address this in next version.
>
Best Regards,
Vikram
Powered by blists - more mailing lists