From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v6 07/15] dt-bindings: memory: tegra30: Convert to Tegra124 YAML Date: Wed, 3 Jul 2019 07:22:34 -0600 Message-ID: References: <20190630210019.26914-1-digetx@gmail.com> <20190630210019.26914-8-digetx@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Dmitry Osipenko Cc: Michael Turquette , Joseph Lo , Thierry Reding , Jonathan Hunter , Peter De Schrijver , Prashant Gaikwad , Stephen Boyd , devicetree@vger.kernel.org, linux-clk , linux-tegra@vger.kernel.org, "linux-kernel@vger.kernel.org" List-Id: devicetree@vger.kernel.org On Tue, Jul 2, 2019 at 6:48 PM Dmitry Osipenko wrote: > > 01.07.2019 22:30, Dmitry Osipenko =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > > 01.07.2019 22:11, Rob Herring =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > >> On Sun, Jun 30, 2019 at 3:04 PM Dmitry Osipenko wro= te: > >>> > >> > >> "Convert" implies you delete the old binding doc. > > > > Yes, unfortunately the deletion got lost by accident after rebase and i= t was already > > too late when I noticed that. Will be fixed in the next revision. > > > >>> The Tegra30 binding will actually differ from the Tegra124 a tad, in > >>> particular the EMEM configuration description. Hence rename the bindi= ng > >>> to Tegra124 during of the conversion to YAML. > >>> > >>> Signed-off-by: Dmitry Osipenko > >>> --- > >>> .../nvidia,tegra124-mc.yaml | 149 ++++++++++++++++= ++ > >>> 1 file changed, 149 insertions(+) > >>> create mode 100644 Documentation/devicetree/bindings/memory-controll= ers/nvidia,tegra124-mc.yaml > >>> > >>> diff --git a/Documentation/devicetree/bindings/memory-controllers/nvi= dia,tegra124-mc.yaml b/Documentation/devicetree/bindings/memory-controllers= /nvidia,tegra124-mc.yaml > >>> new file mode 100644 > >>> index 000000000000..d18242510295 > >>> --- /dev/null > >>> +++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,teg= ra124-mc.yaml > >>> @@ -0,0 +1,149 @@ > >>> +# SPDX-License-Identifier: (GPL-2.0) > >>> +%YAML 1.2 > >>> +--- > >>> +$id: http://devicetree.org/schemas/memory-controllers/nvidia,tegra12= 4-mc.yaml# > >>> +$schema: http://devicetree.org/meta-schemas/core.yaml# > >>> + > >>> +title: > >>> + NVIDIA Tegra124 SoC Memory Controller > >>> + > >>> +maintainers: > >>> + - Jon Hunter > >>> + - Thierry Reding > >>> + > >>> +description: | > >>> + Tegra124 SoC features a hybrid 2x32-bit / 1x64-bit memory controll= er. > >>> + These are interleaved to provide high performance with the load sh= ared across > >>> + two memory channels. The Tegra124 Memory Controller handles memory= requests > >>> + from internal clients and arbitrates among them to allocate memory= bandwidth > >>> + for DDR3L and LPDDR3 SDRAMs. > >>> + > >>> +properties: > >>> + compatible: > >>> + const: nvidia,tegra124-mc > >>> + > >>> + reg: > >>> + maxItems: 1 > >>> + description: > >>> + Physical base address. > >>> + > >>> + clocks: > >>> + maxItems: 1 > >>> + description: > >>> + Memory Controller clock. > >>> + > >>> + clock-names: > >>> + items: > >>> + - const: mc > >>> + > >>> + interrupts: > >>> + maxItems: 1 > >>> + description: > >>> + Memory Controller interrupt. > >>> + > >>> + "#reset-cells": > >>> + const: 1 > >>> + > >>> + "#iommu-cells": > >>> + const: 1 > >>> + > >>> +patternProperties: > >>> + ".*": > >> > >> Please define a node name or pattern for node names. > > > > There was no pattern specified in the original binding. But I guess the= existing > > upstream device-trees could be used as the source for the pattern. > > Actually it looks like the use of explicit pattern is not really a good i= dea because > device-tree could have node named in a way that it doesn't match the patt= ern and hence > dtbs_check silently skips the non-matching nodes. Is there any way to exp= ress that > non-matching nodes shall be rejected? additionalProperties: false It's not ideal because you have to list all properties and can't combine multiple schema, but that's getting addressed in json-schema draft8. That shouldn't matter for you in this case though. Rob