Commit ef4db28c authored May 20, 2019 by Brian Masney Committed by Rob Herring Jun 28, 2019

dt-bindings: backlight: lm3630a: correct schema validation



The '#address-cells' and '#size-cells' properties were not defined in
the lm3630a bindings and would cause the following error when
attempting to validate the examples against the schema:

Documentation/devicetree/bindings/leds/backlight/lm3630a-backlight.example.dt.yaml:
'#address-cells', '#size-cells' do not match any of the regexes:
'^led@[01]$', 'pinctrl-[0-9]+'

Correct this by adding those two properties.

While we're here, move the ti,linear-mapping-mode property to the
led@[01] child nodes to correct the following validation error:

Documentation/devicetree/bindings/leds/backlight/lm3630a-backlight.example.dt.yaml:
led@0: 'ti,linear-mapping-mode' does not match any of the regexes:
'pinctrl-[0-9]+'

Fixes: 32fcb75c ("dt-bindings: backlight: Add lm3630a bindings")
Signed-off-by: Brian Masney <masneyb@onstation.org>
Reported-by: Rob Herring <robh+dt@kernel.org>
Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
Acked-by: Dan Murphy <dmurphy@ti.com>
[robh: also drop maxItems from child reg]
Signed-off-by: Rob Herring <robh@kernel.org>

parent f80b1dfc

Documentation/devicetree/bindings/leds/backlight/lm3630a-backlight.yaml

+14 −7

Original line number	Diff line number	Diff line
		@@ -23,16 +23,17 @@ properties:
		reg:
		maxItems: 1

		ti,linear-mapping-mode:
		description: \|
		Enable linear mapping mode. If disabled, then it will use exponential
		mapping mode in which the ramp up/down appears to have a more uniform
		transition to the human eye.
		type: boolean
		'#address-cells':
		const: 1

		'#size-cells':
		const: 0

		required:
		- compatible
		- reg
		- '#address-cells'
		- '#size-cells'

		patternProperties:
		"^led@[01]$":
		@@ -48,7 +49,6 @@ patternProperties:
		in this property. The two current sinks can be controlled
		independently with both banks, or bank A can be configured to control
		both sinks with the led-sources property.
		maxItems: 1
		minimum: 0
		maximum: 1

		@@ -73,6 +73,13 @@ patternProperties:
		minimum: 0
		maximum: 255

		ti,linear-mapping-mode:
		description: \|
		Enable linear mapping mode. If disabled, then it will use exponential
		mapping mode in which the ramp up/down appears to have a more uniform
		transition to the human eye.
		type: boolean

		required:
		- reg