Messages in this thread | | | Date | Tue, 17 Oct 2017 22:24:07 +0200 | From | Thierry Reding <> | Subject | Re: [PATCH v3 1/2] staging: Introduce NVIDIA Tegra20 video decoder driver |
| |
On Tue, Oct 17, 2017 at 03:13:54PM -0500, Rob Herring wrote: [...] > > diff --git a/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra20-vde.txt b/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra20-vde.txt [...] > > +- resets : Must contain an entry for each entry in reset-names. > > + See ../reset/reset.txt for details. > > +- reset-names : Must include the following entries: > > + - vde > > -names is pointless when there is only one.
I'd prefer to keep it. In the past we occasionally had to add clocks or resets to a device tree node where only one had been present (and hence no -names property) and that caused some awkwardness because verbiage had to be added to the bindings that clarified that one particular entry (the original one) always had to come first.
Thierry [unhandled content-type:application/pgp-signature] | |