[PATCH 1/6] dt-bindings: display/msm: rename mdss nodes to display-sybsystem

[Dmitry Baryshkov]

On 09/01/2023 18:00, Rob Herring wrote:
> 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 at 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?

Actually no. This schema is used by qcom,SoC-mdss devices and it should 
not be selected for legacy 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 at 1a00000 {
>> +    display-subsystem at 1a00000 {
>>           compatible = "qcom,mdss";
>>           reg = <0x1a00000 0x1000>,
>>                 <0x1ac8000 0x3000>;
>> -- 
>> 2.39.0
>>
>>

-- 
With best wishes
Dmitry