dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Andy Gross <agross@kernel.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konrad.dybcio@somainline.org>,
	Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>,
	Abhinav Kumar <quic_abhinavk@quicinc.com>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: devicetree@vger.kernel.org, David Airlie <airlied@linux.ie>,
	linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
	Stephen Boyd <swboyd@chromium.org>,
	freedreno@lists.freedesktop.org
Subject: Re: [PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes
Date: Sat, 24 Sep 2022 15:38:20 +0300	[thread overview]
Message-ID: <64f7c603-173a-93a3-8878-60dd2a8974f9@linaro.org> (raw)
In-Reply-To: <5ea1e309-6ea8-f6c1-dc53-471d858ff60d@linaro.org>

On 24/09/2022 12:29, Krzysztof Kozlowski wrote:
> On 24/09/2022 11:20, Dmitry Baryshkov wrote:
>> On 24/09/2022 12:11, Krzysztof Kozlowski wrote:
>>> On 24/09/2022 11:00, Dmitry Baryshkov wrote:
>>>> Historically DSI PHY device tree nodes used the dsi-phy@ names. Replace
>>>> them with generic phy@ names.
>>>>
>>>> Dmitry Baryshkov (9):
>>>>     ARM: dts: qcom-apq8064: change DSI PHY node name to generic one
>>>
>>> dsi-phy is similar to other generic names (usb-phy, ethernet-phy) [1] so
>>> it could stay. Is there particular need for this, like coming from DT
>>> schema?
>>
>> No, there is no requirement from the DT schema. However:
>> 1) The resent qcom DT files already use just phy@ for most of PHY nodes
>> 2) The recommended list mentions usb-phy/ethernet-phy, but not
>> <anything>-phy, so I'd think that those two are mostly for backwards
>> compatibility.
>> 3) I liked the example of sc7280 which switched all MDSS PHYs to just
>> phy@ (this includes DSI PHY, eDP PHY and, by extension, the HDMI PHY).
> 
> Good explanation. If there is going to be resent/submit, please add it
> to cover letter. :)

I hope, there will be no v2. For now I have included this explanation 
into the HDMI PHY counterpart.

-- 
With best wishes
Dmitry


  reply	other threads:[~2022-09-24 12:38 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-24  9:00 [PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes Dmitry Baryshkov
2022-09-24  9:01 ` [PATCH 1/9] ARM: dts: qcom-apq8064: change DSI PHY node name to generic one Dmitry Baryshkov
2022-09-24  9:30   ` Krzysztof Kozlowski
2022-09-24  9:01 ` [PATCH 2/9] ARM: dts: qcom-msm8974: " Dmitry Baryshkov
2022-09-24  9:30   ` Krzysztof Kozlowski
2022-09-24  9:01 ` [PATCH 3/9] arm64: dts: qcom: msm8916: " Dmitry Baryshkov
2022-09-24  9:30   ` Krzysztof Kozlowski
2022-09-24  9:01 ` [PATCH 4/9] arm64: dts: qcom: msm8996: " Dmitry Baryshkov
2022-09-24  9:31   ` Krzysztof Kozlowski
2022-09-24  9:01 ` [PATCH 5/9] arm64: dts: qcom: sc7180: " Dmitry Baryshkov
2022-09-24  9:31   ` Krzysztof Kozlowski
2022-09-24  9:01 ` [PATCH 6/9] arm64: dts: qcom: sdm630: " Dmitry Baryshkov
2022-09-24  9:31   ` Krzysztof Kozlowski
2022-09-24  9:01 ` [PATCH 7/9] arm64: dts: qcom: sdm660: " Dmitry Baryshkov
2022-09-24  9:31   ` Krzysztof Kozlowski
2022-09-24  9:01 ` [PATCH 8/9] arm64: dts: qcom: sdm845: " Dmitry Baryshkov
2022-09-24  9:32   ` Krzysztof Kozlowski
2022-09-25 18:59   ` Steev Klimaszewski
2022-09-24  9:01 ` [PATCH 9/9] arm64: dts: qcom: sm8250: " Dmitry Baryshkov
2022-09-24  9:32   ` Krzysztof Kozlowski
2022-09-24  9:11 ` [PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes Krzysztof Kozlowski
2022-09-24  9:20   ` Dmitry Baryshkov
2022-09-24  9:29     ` Krzysztof Kozlowski
2022-09-24 12:38       ` Dmitry Baryshkov [this message]
2022-09-24 14:47 ` Konrad Dybcio
2022-10-18  3:05 ` (subset) " Bjorn Andersson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=64f7c603-173a-93a3-8878-60dd2a8974f9@linaro.org \
    --to=dmitry.baryshkov@linaro.org \
    --cc=agross@kernel.org \
    --cc=airlied@linux.ie \
    --cc=andersson@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=freedreno@lists.freedesktop.org \
    --cc=konrad.dybcio@somainline.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=quic_abhinavk@quicinc.com \
    --cc=robdclark@gmail.com \
    --cc=robh+dt@kernel.org \
    --cc=sean@poorly.run \
    --cc=swboyd@chromium.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).