On 07/20/2016 03:24 AM, Rob Herring wrote:

On Tue, Jul 19, 2016 at 01:47:31PM +0200, H. Nikolaus Schaller wrote:

quoted

Signed-off-by: H. Nikolaus Schaller <redacted>
---
  .../devicetree/bindings/leds/is31fl319x.txt        | 59 ++++++++++++++++++++++
  1 file changed, 59 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/leds/is31fl319x.txt

diff --git a/Documentation/devicetree/bindings/leds/is31fl319x.txt b/Documentation/devicetree/bindings/leds/is31fl319x.txt
new file mode 100644
index 0000000..03287c0
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/is31fl319x.txt

@@ -0,0 +1,59 @@
+LEDs connected to is31fl319x LED controller chip
+
+Required properties:
+- compatible : Should be any of
+	"issi,is31fl3190"
+	"issi,is31fl3191"
+	"issi,is31fl3193"
+	"issi,is31fl3196"
+	"issi,is31fl3199"
+	"si-en,sn3199".
+- #address-cells: Must be 1.
+- #size-cells: Must be 0.
+- reg: 0x64, 0x65, 0x66, 0x67.

This is an OR or AND? OR it seems. With that,

Nikolaus, could you please make it explicit?

Acked-by: Rob Herring <robh@kernel.org>

-- 
Best regards,
Jacek Anaszewski