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

[Nihal Kumar Gupta <quic_nihalkum@quicinc.com>]

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

Regards,
Nihal Kumar Gupta