Thread (19 messages) 19 messages, 6 authors, 2017-10-19

Re: [PATCH v3 1/2] staging: Introduce NVIDIA Tegra20 video decoder driver

From: Thierry Reding <hidden>
Date: 2017-10-17 20:24:14
Also in: linux-media, linux-tegra, lkml

On Tue, Oct 17, 2017 at 03:13:54PM -0500, Rob Herring wrote:
[...]
quoted
diff --git a/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra20-vde.txt b/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra20-vde.txt
[...]
quoted
+- 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

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help