[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <bfab7272-5a41-4072-82a8-0f7401b2affc@kernel.org>
Date: Thu, 10 Apr 2025 08:08:17 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
Cc: Ayushi Makhija <quic_amakhija@...cinc.com>,
linux-arm-msm@...r.kernel.org, dri-devel@...ts.freedesktop.org,
freedreno@...ts.freedesktop.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, robdclark@...il.com,
dmitry.baryshkov@...aro.org, sean@...rly.run, marijn.suijten@...ainline.org,
andersson@...nel.org, robh@...nel.org, robh+dt@...nel.org,
krzk+dt@...nel.org, konradybcio@...nel.org, conor+dt@...nel.org,
andrzej.hajda@...el.com, neil.armstrong@...aro.org, rfoss@...nel.org,
Laurent.pinchart@...asonboard.com, jonas@...boo.se,
jernej.skrabec@...il.com, quic_abhinavk@...cinc.com,
quic_rajeevny@...cinc.com, quic_vproddut@...cinc.com,
quic_jesszhan@...cinc.com
Subject: Re: [PATCH v2 03/10] dt-bindings: display: msm: document DSI
controller and phy on SA8775P
On 09/04/2025 17:24, Dmitry Baryshkov wrote:
> On 09/04/2025 09:07, Krzysztof Kozlowski wrote:
>> On 08/04/2025 22:26, Dmitry Baryshkov wrote:
>>>>>>>>> + - const: qcom,sa8775p-dsi-ctrl
>>>>>>>>> + - const: qcom,mdss-dsi-ctrl
>>>>>>>>
>>>>>>>> Drop fallback
>>>>>>>>
>>>>>>>
>>>>>>> Hi Krzysztof,
>>>>>>>
>>>>>>> I couldn't understand the meaning of "Drop fallback", could please elaborate it ?
>>>>>> Look at SM8750 example on the lists. Keep only front compatible.
>>>>>
>>>>> Why?
>>>>
>>>> To make things simpler and shorter.
>>>
>>> I'd prefer consistency. Previous platforms use qcom,mdss-dsi-ctrl.
>> Then you should have objected month(s) ago when Rob asked for dropping
>> fallback and since then we consistently drop it.
>
> Well... It's still not merged. is it?
> For SM8750 it kinda makes sense, because the clock handling is different
> from all other current platforms. For the existing devices... I'm not
> that sure.
How does it differ? The clock handling does not matter - this is just
select of schema for the child node.
Best regards,
Krzysztof
Powered by blists - more mailing lists