Re: [PATCH 2/3] arm64: dts: qcom: qcs8300: add Display Serial Interface device nodes
From: Dmitry Baryshkov
Date: Fri Sep 26 2025 - 10:05:52 EST
On Fri, Sep 26, 2025 at 02:55:19PM +0530, Ayushi Makhija wrote:
> On 9/26/2025 3:32 AM, Dmitry Baryshkov wrote:
> > On Thu, Sep 25, 2025 at 11:06:01AM +0530, Ayushi Makhija wrote:
> >> Add device tree nodes for the DSI0 controller with their corresponding
> >> PHY found on Qualcomm QCS8300 SoC.
> >>
> >> Signed-off-by: Ayushi Makhija <quic_amakhija@xxxxxxxxxxx>
> >> ---
> >> arch/arm64/boot/dts/qcom/qcs8300.dtsi | 95 ++++++++++++++++++++++++++-
> >> 1 file changed, 94 insertions(+), 1 deletion(-)
> >>
> >> +
> >> + mdss_dsi0_phy: phy@ae94400 {
> >> + compatible = "qcom,sa8775p-dsi-phy-5nm";
> >
> > Add qcs8300-specific compatible and use sa8775p as a fallback
> >
>
> Hi Dmitry,
>
> I have one doubt, even though the ctrl and phy versions for sa8775p(LeMans) and qcs8300(Monaco) are same.
> Why can't we use the same compatible string that we have used for LeMans ctrl and phy ? what is the need to define a separate
> compatible string for monaco ctrl and phy ?
Konrad responded with the reason. But also, this question should have
been asked when you took a first look at the qcom,qcs8300-mdss.yaml.
It has two compats for DPU, DP and eDP PHY blocks (for exactly the same
reason).
--
With best wishes
Dmitry