| Message ID | 20221130200950.144618-2-a39.skl@gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | SM6115 DTS changes | expand |
On 30/11/2022 21:09, Adam Skladowski wrote: > Follow other YAMLs and replace mdss name into display-subystem. > > Signed-off-by: Adam Skladowski <a39.skl@gmail.com> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On 30/11/2022 22:09, Adam Skladowski wrote: > Follow other YAMLs and replace mdss name into display-subystem. > > Signed-off-by: Adam Skladowski <a39.skl@gmail.com> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> We will pick this into msm-fixes during the next cycle. > --- > .../devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml | 2 +- > .../devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml > index d6f043a4b08d..4795e13c7b59 100644 > --- a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml > +++ b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml > @@ -72,7 +72,7 @@ examples: > #include <dt-bindings/interconnect/qcom,qcm2290.h> > #include <dt-bindings/power/qcom-rpmpd.h> > > - mdss@5e00000 { > + display-subsystem@5e00000 { > #address-cells = <1>; > #size-cells = <1>; > compatible = "qcom,qcm2290-mdss"; > diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml > index a86d7f53fa84..886858ef6700 100644 > --- a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml > +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml > @@ -62,7 +62,7 @@ examples: > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/power/qcom-rpmpd.h> > > - mdss@5e00000 { > + display-subsystem@5e00000 { > #address-cells = <1>; > #size-cells = <1>; > compatible = "qcom,sm6115-mdss";
On 12/1/2022 11:54 AM, Dmitry Baryshkov wrote: > On 30/11/2022 22:09, Adam Skladowski wrote: >> Follow other YAMLs and replace mdss name into display-subystem. >> >> Signed-off-by: Adam Skladowski <a39.skl@gmail.com> > > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Going to add two fixes tags here as we are touching two chipsets: Fixes: b93bdff44a85 ("dt-bindings: display/msm: add support for SM6115") Fixes: 06097b13ef97 ("dt-bindings: display/msm: split dpu-qcm2290 into DPU and MDSS parts") Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com> > > We will pick this into msm-fixes during the next cycle. Yes, we can with the above fixes tags but first, can you please send a MR from msm-next-lumag to msm-next? So that I can send a MR for fixes to msm-next. ATM, they are out of sync. > >> --- >> .../devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml | 2 +- >> .../devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git >> a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >> b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >> index d6f043a4b08d..4795e13c7b59 100644 >> --- >> a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >> +++ >> b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >> @@ -72,7 +72,7 @@ examples: >> #include <dt-bindings/interconnect/qcom,qcm2290.h> >> #include <dt-bindings/power/qcom-rpmpd.h> >> - mdss@5e00000 { >> + display-subsystem@5e00000 { >> #address-cells = <1>; >> #size-cells = <1>; >> compatible = "qcom,qcm2290-mdss"; >> diff --git >> a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >> b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >> index a86d7f53fa84..886858ef6700 100644 >> --- a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >> +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >> @@ -62,7 +62,7 @@ examples: >> #include <dt-bindings/interrupt-controller/arm-gic.h> >> #include <dt-bindings/power/qcom-rpmpd.h> >> - mdss@5e00000 { >> + display-subsystem@5e00000 { >> #address-cells = <1>; >> #size-cells = <1>; >> compatible = "qcom,sm6115-mdss"; >
On 13 December 2022 23:53:48 EET, Abhinav Kumar <quic_abhinavk@quicinc.com> wrote: > > >On 12/1/2022 11:54 AM, Dmitry Baryshkov wrote: >> On 30/11/2022 22:09, Adam Skladowski wrote: >>> Follow other YAMLs and replace mdss name into display-subystem. >>> >>> Signed-off-by: Adam Skladowski <a39.skl@gmail.com> >> >> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > >Going to add two fixes tags here as we are touching two chipsets: > >Fixes: b93bdff44a85 ("dt-bindings: display/msm: add support for SM6115") >Fixes: 06097b13ef97 ("dt-bindings: display/msm: split dpu-qcm2290 into DPU and MDSS parts") Note, your tag gif wrapped, so patchwork will pick it up incorrectly. > >Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com> > >> >> We will pick this into msm-fixes during the next cycle. > >Yes, we can with the above fixes tags but first, can you please send a MR from msm-next-lumag to msm-next? So that I can send a MR for fixes to msm-next. This would create an additional merge commit in msm-next for no particular reason. You can branch -fixes from rc1, or from the msm-next-lumag and then send MR to msm-next. > >ATM, they are out of sync. > > >> >>> --- >>> .../devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml | 2 +- >>> .../devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml | 2 +- >>> 2 files changed, 2 insertions(+), 2 deletions(-) >>> >>> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >>> index d6f043a4b08d..4795e13c7b59 100644 >>> --- a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >>> +++ b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >>> @@ -72,7 +72,7 @@ examples: >>> #include <dt-bindings/interconnect/qcom,qcm2290.h> >>> #include <dt-bindings/power/qcom-rpmpd.h> >>> - mdss@5e00000 { >>> + display-subsystem@5e00000 { >>> #address-cells = <1>; >>> #size-cells = <1>; >>> compatible = "qcom,qcm2290-mdss"; >>> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >>> index a86d7f53fa84..886858ef6700 100644 >>> --- a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >>> +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >>> @@ -62,7 +62,7 @@ examples: >>> #include <dt-bindings/interrupt-controller/arm-gic.h> >>> #include <dt-bindings/power/qcom-rpmpd.h> >>> - mdss@5e00000 { >>> + display-subsystem@5e00000 { >>> #address-cells = <1>; >>> #size-cells = <1>; >>> compatible = "qcom,sm6115-mdss"; >>
On 12/13/2022 2:11 PM, Dmitry Baryshkov wrote: > > > On 13 December 2022 23:53:48 EET, Abhinav Kumar <quic_abhinavk@quicinc.com> wrote: >> >> >> On 12/1/2022 11:54 AM, Dmitry Baryshkov wrote: >>> On 30/11/2022 22:09, Adam Skladowski wrote: >>>> Follow other YAMLs and replace mdss name into display-subystem. >>>> >>>> Signed-off-by: Adam Skladowski <a39.skl@gmail.com> >>> >>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> >> >> Going to add two fixes tags here as we are touching two chipsets: >> >> Fixes: b93bdff44a85 ("dt-bindings: display/msm: add support for SM6115") >> Fixes: 06097b13ef97 ("dt-bindings: display/msm: split dpu-qcm2290 into DPU and MDSS parts") > > Note, your tag gif wrapped, so patchwork will pick it up incorrectly. Yes, this is a mail client issue. Am aware of it , will fix it up when i apply it. > >> >> Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com> >> >>> >>> We will pick this into msm-fixes during the next cycle. >> >> Yes, we can with the above fixes tags but first, can you please send a MR from msm-next-lumag to msm-next? So that I can send a MR for fixes to msm-next. > > This would create an additional merge commit in msm-next for no particular reason. You can branch -fixes from rc1, or from the msm-next-lumag and then send MR to msm-next. So msm-next would need to be updated then to rc1 to accept that MR. Rob? > >> >> ATM, they are out of sync. >> >> >>> >>>> --- >>>> .../devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml | 2 +- >>>> .../devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml | 2 +- >>>> 2 files changed, 2 insertions(+), 2 deletions(-) >>>> >>>> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >>>> index d6f043a4b08d..4795e13c7b59 100644 >>>> --- a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >>>> +++ b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml >>>> @@ -72,7 +72,7 @@ examples: >>>> #include <dt-bindings/interconnect/qcom,qcm2290.h> >>>> #include <dt-bindings/power/qcom-rpmpd.h> >>>> - mdss@5e00000 { >>>> + display-subsystem@5e00000 { >>>> #address-cells = <1>; >>>> #size-cells = <1>; >>>> compatible = "qcom,qcm2290-mdss"; >>>> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >>>> index a86d7f53fa84..886858ef6700 100644 >>>> --- a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >>>> +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml >>>> @@ -62,7 +62,7 @@ examples: >>>> #include <dt-bindings/interrupt-controller/arm-gic.h> >>>> #include <dt-bindings/power/qcom-rpmpd.h> >>>> - mdss@5e00000 { >>>> + display-subsystem@5e00000 { >>>> #address-cells = <1>; >>>> #size-cells = <1>; >>>> compatible = "qcom,sm6115-mdss"; >>> >
14 декабря 2022 г. 00:11:58 GMT+02:00, Dmitry Baryshkov <dmitry.baryshkov@linaro.org> пишет: > > >On 13 December 2022 23:53:48 EET, Abhinav Kumar <quic_abhinavk@quicinc.com> wrote: >> >> >>On 12/1/2022 11:54 AM, Dmitry Baryshkov wrote: >>> On 30/11/2022 22:09, Adam Skladowski wrote: >> >>> >>> We will pick this into msm-fixes during the next cycle. >> >>Yes, we can with the above fixes tags but first, can you please send a MR from msm-next-lumag to msm-next? So that I can send a MR for fixes to msm-next. > >This would create an additional merge commit in msm-next for no particular reason. You can branch -fixes from rc1, or from the msm-next-lumag and then send MR to msm-next. Another option would be to base msm-fixes on drm-next directly. > >> >>ATM, they are out of sync. >> >>
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml index d6f043a4b08d..4795e13c7b59 100644 --- a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml +++ b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml @@ -72,7 +72,7 @@ examples: #include <dt-bindings/interconnect/qcom,qcm2290.h> #include <dt-bindings/power/qcom-rpmpd.h> - mdss@5e00000 { + display-subsystem@5e00000 { #address-cells = <1>; #size-cells = <1>; compatible = "qcom,qcm2290-mdss"; diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml index a86d7f53fa84..886858ef6700 100644 --- a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml @@ -62,7 +62,7 @@ examples: #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/power/qcom-rpmpd.h> - mdss@5e00000 { + display-subsystem@5e00000 { #address-cells = <1>; #size-cells = <1>; compatible = "qcom,sm6115-mdss";
Follow other YAMLs and replace mdss name into display-subystem. Signed-off-by: Adam Skladowski <a39.skl@gmail.com> --- .../devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml | 2 +- .../devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)