On Tue, Nov 5, 2019 at 6:26 PM Stephen Boyd <sboyd@xxxxxxxxxx> wrote: > > Quoting Taniya Das (2019-10-31 05:21:08) > > diff --git a/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml b/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml > > new file mode 100644 > > index 0000000..96aaf36 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml > > @@ -0,0 +1,69 @@ > > +# SPDX-License-Identifier: GPL-2.0-only > > Can it be GPL2 or BSD? I think Rob is asking for that sort of license on > these files. I do, but only on new bindings unless we determine relicensing is okay. Though here it doesn't look like much is copied over. > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/bindings/clock/qcom,gpucc.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Qualcomm Graphics Clock & Reset Controller Binding > > + > > +maintainers: > > + - Taniya Das <tdas@xxxxxxxxxxxxxx> > > + > > +description: | > > + Qualcomm grpahics clock control module which supports the clocks, resets and > > + power domains. > > + > > +properties: > > + compatible: > > + enum: > > + - qcom,sdm845-gpucc > > + - qcom,msm8998-gpucc > > Sort please. When you get tired of telling people to do this we can make the tooling do it. :) Shouldn't be too hard. The majority of the work is probably fixing the existing cases that aren't sorted. Rob