On Thu, 13 Jul 2023 at 21:13, Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> wrote: > > On Thu, 13 Jul 2023 at 18:34, Amit Pundir <amit.pundir@xxxxxxxxxx> wrote: > > > > On Wed, 12 Jul 2023 at 18:45, Dmitry Baryshkov > > <dmitry.baryshkov@xxxxxxxxxx> wrote: > > > > > > On 12/07/2023 16:02, Amit Pundir wrote: > > > > Add and document the reserved memory region property > > > > in the qcom,sdm845-mdss schema. > > > > > > > > Signed-off-by: Amit Pundir <amit.pundir@xxxxxxxxxx> > > > > --- > > > > .../devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml | 5 +++++ > > > > 1 file changed, 5 insertions(+) > > > > > > > > diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml > > > > index 6ecb00920d7f..3ea1dbd7e317 100644 > > > > --- a/Documentation/devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml > > > > +++ b/Documentation/devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml > > > > @@ -39,6 +39,11 @@ properties: > > > > interconnect-names: > > > > maxItems: 2 > > > > > > > > + memory-region: > > > > + maxItems: 1 > > > > + description: > > > > + Phandle to a node describing a reserved memory region. > > > > + > > > > > > Please add it to mdss-common.yaml instead > > > > mdss-common.yaml didn't like this property at all and > > I ran into a lot of new dtbs_check warnings: > > https://www.irccloud.com/pastebin/raw/pEYAeaB1 > > > > I need some help in decoding these please. > > I'm not sure what happened there (and it's hard to understand without > seeing your patch). Yup.. It was my broken patch. I used "For example:" in the property description and it tripped off the checks. Didn't realise that casually used ":" can break yaml parsing until now. Sorry for all the noise. Regards, Amit Pundir > But after applying your patch to mdss-common.yaml, > `make dt_binding_check' passes: > > diff --git a/Documentation/devicetree/bindings/display/msm/mdss-common.yaml > b/Documentation/devicetree/bindings/display/msm/mdss-common.yaml > index ccd7d6417523..924fe383e4a1 100644 > --- a/Documentation/devicetree/bindings/display/msm/mdss-common.yaml > +++ b/Documentation/devicetree/bindings/display/msm/mdss-common.yaml > @@ -77,6 +77,11 @@ properties: > items: > - description: MDSS_CORE reset > > + memory-region: > + maxItems: 1 > + description: > + Phandle to a node describing a reserved memory region. > + > required: > - reg > - reg-names > > > > > > Regards, > > Amit Pundir > > > > > > > > > patternProperties: > > > > "^display-controller@[0-9a-f]+$": > > > > type: object > > > > > > -- > > > With best wishes > > > Dmitry > > > > > > > -- > With best wishes > Dmitry
