Re: [PATCH 3/4] arm64: dts: qcom: monaco-evk-camera: Add DT overlay

[Vikram Sharma <quic_vikramsa@quicinc.com>]

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@quicinc.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