All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Andy Gross <agross@kernel.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konrad.dybcio@linaro.org>,
	Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>,
	Abhinav Kumar <quic_abhinavk@quicinc.com>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Stephen Boyd <swboyd@chromium.org>,
	David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
	linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
	dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org
Subject: Re: [PATCH 1/6] dt-bindings: display/msm: rename mdss nodes to display-sybsystem
Date: Mon, 9 Jan 2023 10:00:05 -0600	[thread overview]
Message-ID: <20230109160005.GA721066-robh@kernel.org> (raw)
In-Reply-To: <20230109051402.317577-2-dmitry.baryshkov@linaro.org>

On Mon, Jan 09, 2023 at 07:13:57AM +0200, Dmitry Baryshkov wrote:
> Follow the 'generic names' rule and rename mdss nodes to
> display-subsystem.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  .../devicetree/bindings/display/msm/mdss-common.yaml      | 8 ++++++++
>  .../devicetree/bindings/display/msm/qcom,mdss.yaml        | 5 ++++-
>  2 files changed, 12 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/msm/mdss-common.yaml b/Documentation/devicetree/bindings/display/msm/mdss-common.yaml
> index 59f17ac898aa..ccd7d6417523 100644
> --- a/Documentation/devicetree/bindings/display/msm/mdss-common.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/mdss-common.yaml
> @@ -15,7 +15,15 @@ description:
>    Device tree bindings for MSM Mobile Display Subsystem(MDSS) that encapsulates
>    sub-blocks like DPU display controller, DSI and DP interfaces etc.
>  
> +# Do not select this by default, otherwise it is also selected for qcom,mdss
> +# devices.

for NON qcom,mdss devices?

> +select:
> +  false

select: false

> +
>  properties:
> +  $nodename:
> +    pattern: "^display-subsystem@[0-9a-f]+$"
> +
>    reg:
>      maxItems: 1
>  
> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,mdss.yaml
> index c218c9172608..47fde9b6779f 100644
> --- a/Documentation/devicetree/bindings/display/msm/qcom,mdss.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/qcom,mdss.yaml
> @@ -15,6 +15,9 @@ description:
>    encapsulates sub-blocks like MDP5, DSI, HDMI, eDP, etc.
>  
>  properties:
> +  $nodename:
> +    pattern: "^display-subsystem@[0-9a-f]+$"
> +
>    compatible:
>      enum:
>        - qcom,mdss
> @@ -153,7 +156,7 @@ examples:
>    - |
>      #include <dt-bindings/clock/qcom,gcc-msm8916.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> -    mdss@1a00000 {
> +    display-subsystem@1a00000 {
>          compatible = "qcom,mdss";
>          reg = <0x1a00000 0x1000>,
>                <0x1ac8000 0x3000>;
> -- 
> 2.39.0
> 
> 

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: devicetree@vger.kernel.org,
	Bjorn Andersson <andersson@kernel.org>,
	Abhinav Kumar <quic_abhinavk@quicinc.com>,
	dri-devel@lists.freedesktop.org,
	Stephen Boyd <swboyd@chromium.org>,
	Konrad Dybcio <konrad.dybcio@linaro.org>,
	Andy Gross <agross@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org,
	Sean Paul <sean@poorly.run>
Subject: Re: [PATCH 1/6] dt-bindings: display/msm: rename mdss nodes to display-sybsystem
Date: Mon, 9 Jan 2023 10:00:05 -0600	[thread overview]
Message-ID: <20230109160005.GA721066-robh@kernel.org> (raw)
In-Reply-To: <20230109051402.317577-2-dmitry.baryshkov@linaro.org>

On Mon, Jan 09, 2023 at 07:13:57AM +0200, Dmitry Baryshkov wrote:
> Follow the 'generic names' rule and rename mdss nodes to
> display-subsystem.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  .../devicetree/bindings/display/msm/mdss-common.yaml      | 8 ++++++++
>  .../devicetree/bindings/display/msm/qcom,mdss.yaml        | 5 ++++-
>  2 files changed, 12 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/msm/mdss-common.yaml b/Documentation/devicetree/bindings/display/msm/mdss-common.yaml
> index 59f17ac898aa..ccd7d6417523 100644
> --- a/Documentation/devicetree/bindings/display/msm/mdss-common.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/mdss-common.yaml
> @@ -15,7 +15,15 @@ description:
>    Device tree bindings for MSM Mobile Display Subsystem(MDSS) that encapsulates
>    sub-blocks like DPU display controller, DSI and DP interfaces etc.
>  
> +# Do not select this by default, otherwise it is also selected for qcom,mdss
> +# devices.

for NON qcom,mdss devices?

> +select:
> +  false

select: false

> +
>  properties:
> +  $nodename:
> +    pattern: "^display-subsystem@[0-9a-f]+$"
> +
>    reg:
>      maxItems: 1
>  
> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,mdss.yaml
> index c218c9172608..47fde9b6779f 100644
> --- a/Documentation/devicetree/bindings/display/msm/qcom,mdss.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/qcom,mdss.yaml
> @@ -15,6 +15,9 @@ description:
>    encapsulates sub-blocks like MDP5, DSI, HDMI, eDP, etc.
>  
>  properties:
> +  $nodename:
> +    pattern: "^display-subsystem@[0-9a-f]+$"
> +
>    compatible:
>      enum:
>        - qcom,mdss
> @@ -153,7 +156,7 @@ examples:
>    - |
>      #include <dt-bindings/clock/qcom,gcc-msm8916.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> -    mdss@1a00000 {
> +    display-subsystem@1a00000 {
>          compatible = "qcom,mdss";
>          reg = <0x1a00000 0x1000>,
>                <0x1ac8000 0x3000>;
> -- 
> 2.39.0
> 
> 

  parent reply	other threads:[~2023-01-09 16:01 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-09  5:13 [PATCH 0/6] dt-bindings: display/msm: rename mdss and mdp nodes Dmitry Baryshkov
2023-01-09  5:13 ` Dmitry Baryshkov
2023-01-09  5:13 ` [PATCH 1/6] dt-bindings: display/msm: rename mdss nodes to display-sybsystem Dmitry Baryshkov
2023-01-09  5:13   ` Dmitry Baryshkov
2023-01-09 14:30   ` Rob Herring
2023-01-09 14:30     ` Rob Herring
2023-01-09 16:00   ` Rob Herring [this message]
2023-01-09 16:00     ` Rob Herring
2023-01-09 18:27     ` Dmitry Baryshkov
2023-01-09 18:27       ` Dmitry Baryshkov
2023-01-09  5:13 ` [PATCH 2/6] dt-bindings: display/msm: rename mdp nodes to display-controller Dmitry Baryshkov
2023-01-09  5:13   ` Dmitry Baryshkov
2023-01-09  5:13 ` [PATCH 3/6] arm64: dts: qcom: rename mdss nodes to display-subsystem Dmitry Baryshkov
2023-01-09  5:13   ` Dmitry Baryshkov
2023-01-09  5:14 ` [PATCH 4/6] ARM: dts: qcom-msm8974: rename mdss node " Dmitry Baryshkov
2023-01-09  5:14   ` Dmitry Baryshkov
2023-01-09  5:14 ` [PATCH 5/6] arm64: dts: qcom: rename mdp nodes to display-controller Dmitry Baryshkov
2023-01-09  5:14   ` Dmitry Baryshkov
2023-01-09  5:14 ` [PATCH 6/6] ARM: " Dmitry Baryshkov
2023-01-09  5:14   ` Dmitry Baryshkov
2023-01-19  2:16 ` (subset) [PATCH 0/6] dt-bindings: display/msm: rename mdss and mdp nodes Bjorn Andersson
2023-01-19  2:16   ` 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=20230109160005.GA721066-robh@kernel.org \
    --to=robh@kernel.org \
    --cc=agross@kernel.org \
    --cc=airlied@gmail.com \
    --cc=andersson@kernel.org \
    --cc=daniel@ffwll.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.baryshkov@linaro.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=freedreno@lists.freedesktop.org \
    --cc=konrad.dybcio@linaro.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=quic_abhinavk@quicinc.com \
    --cc=robdclark@gmail.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.