* [PATCH] dt-bindings: backlight: lm3630a: correct schema validation
@ 2019-05-20 8:58 Brian Masney
2019-05-20 13:14 ` Rob Herring
2019-06-28 19:22 ` Rob Herring
0 siblings, 2 replies; 5+ messages in thread
From: Brian Masney @ 2019-05-20 8:58 UTC (permalink / raw)
To: robh+dt, lee.jones
Cc: daniel.thompson, jingoohan1, jacek.anaszewski, pavel,
mark.rutland, b.zolnierkie, dri-devel, linux-leds, devicetree,
linux-kernel, linux-fbdev, dmurphy, jonathan
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: 32fcb75c66a0 ("dt-bindings: backlight: Add lm3630a bindings")
Signed-off-by: Brian Masney <masneyb@onstation.org>
Reported-by: Rob Herring <robh+dt@kernel.org>
---
.../leds/backlight/lm3630a-backlight.yaml | 20 +++++++++++++------
1 file changed, 14 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/leds/backlight/lm3630a-backlight.yaml b/Documentation/devicetree/bindings/leds/backlight/lm3630a-backlight.yaml
index 4d61fe0a98a4..f0855e248ae5 100644
--- a/Documentation/devicetree/bindings/leds/backlight/lm3630a-backlight.yaml
+++ b/Documentation/devicetree/bindings/leds/backlight/lm3630a-backlight.yaml
@@ -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]$":
@@ -73,6 +74,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
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: backlight: lm3630a: correct schema validation
2019-05-20 8:58 [PATCH] dt-bindings: backlight: lm3630a: correct schema validation Brian Masney
@ 2019-05-20 13:14 ` Rob Herring
2019-05-20 13:30 ` Daniel Thompson
2019-06-28 19:22 ` Rob Herring
1 sibling, 1 reply; 5+ messages in thread
From: Rob Herring @ 2019-05-20 13:14 UTC (permalink / raw)
To: Brian Masney
Cc: Mark Rutland, devicetree, Daniel Thompson, Jonathan Marek,
Bartlomiej Zolnierkiewicz, Jingoo Han, linux-kernel, dri-devel,
Linux Fbdev development list, Jacek Anaszewski, Pavel Machek,
Lee Jones, Linux LED Subsystem, Dan Murphy
On Mon, May 20, 2019 at 3:59 AM Brian Masney <masneyb@onstation.org> wrote:
>
> 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: 32fcb75c66a0 ("dt-bindings: backlight: Add lm3630a bindings")
> Signed-off-by: Brian Masney <masneyb@onstation.org>
> Reported-by: Rob Herring <robh+dt@kernel.org>
> ---
> .../leds/backlight/lm3630a-backlight.yaml | 20 +++++++++++++------
> 1 file changed, 14 insertions(+), 6 deletions(-)
Reviewed-by: Rob Herring <robh@kernel.org>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: backlight: lm3630a: correct schema validation
2019-05-20 13:14 ` Rob Herring
@ 2019-05-20 13:30 ` Daniel Thompson
2019-05-22 0:53 ` Dan Murphy
0 siblings, 1 reply; 5+ messages in thread
From: Daniel Thompson @ 2019-05-20 13:30 UTC (permalink / raw)
To: Rob Herring
Cc: Brian Masney, Lee Jones, Jingoo Han, Jacek Anaszewski,
Pavel Machek, Mark Rutland, Bartlomiej Zolnierkiewicz, dri-devel,
Linux LED Subsystem, devicetree, linux-kernel,
Linux Fbdev development list, Dan Murphy, Jonathan Marek
On Mon, May 20, 2019 at 08:14:03AM -0500, Rob Herring wrote:
> On Mon, May 20, 2019 at 3:59 AM Brian Masney <masneyb@onstation.org> wrote:
> >
> > 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: 32fcb75c66a0 ("dt-bindings: backlight: Add lm3630a bindings")
> > Signed-off-by: Brian Masney <masneyb@onstation.org>
> > Reported-by: Rob Herring <robh+dt@kernel.org>
> > ---
> > .../leds/backlight/lm3630a-backlight.yaml | 20 +++++++++++++------
> > 1 file changed, 14 insertions(+), 6 deletions(-)
>
> Reviewed-by: Rob Herring <robh@kernel.org>
Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: backlight: lm3630a: correct schema validation
2019-05-20 13:30 ` Daniel Thompson
@ 2019-05-22 0:53 ` Dan Murphy
0 siblings, 0 replies; 5+ messages in thread
From: Dan Murphy @ 2019-05-22 0:53 UTC (permalink / raw)
To: Daniel Thompson, Rob Herring
Cc: Brian Masney, Lee Jones, Jingoo Han, Jacek Anaszewski,
Pavel Machek, Mark Rutland, Bartlomiej Zolnierkiewicz, dri-devel,
Linux LED Subsystem, devicetree, linux-kernel,
Linux Fbdev development list, Jonathan Marek
On 5/20/19 8:30 AM, Daniel Thompson wrote:
> On Mon, May 20, 2019 at 08:14:03AM -0500, Rob Herring wrote:
>> On Mon, May 20, 2019 at 3:59 AM Brian Masney <masneyb@onstation.org> wrote:
>>>
>>> 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: 32fcb75c66a0 ("dt-bindings: backlight: Add lm3630a bindings")
>>> Signed-off-by: Brian Masney <masneyb@onstation.org>
>>> Reported-by: Rob Herring <robh+dt@kernel.org>
>>> ---
>>> .../leds/backlight/lm3630a-backlight.yaml | 20 +++++++++++++------
>>> 1 file changed, 14 insertions(+), 6 deletions(-)
>>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>
> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>
Acked-by: Dan Murphy <dmurphy@ti.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: backlight: lm3630a: correct schema validation
2019-05-20 8:58 [PATCH] dt-bindings: backlight: lm3630a: correct schema validation Brian Masney
2019-05-20 13:14 ` Rob Herring
@ 2019-06-28 19:22 ` Rob Herring
1 sibling, 0 replies; 5+ messages in thread
From: Rob Herring @ 2019-06-28 19:22 UTC (permalink / raw)
To: Brian Masney
Cc: mark.rutland, devicetree, daniel.thompson, jonathan,
b.zolnierkie, jingoohan1, linux-kernel, dri-devel, linux-fbdev,
robh+dt, jacek.anaszewski, pavel, lee.jones, linux-leds, dmurphy
On Mon, 20 May 2019 04:58:46 -0400, Brian Masney wrote:
> 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: 32fcb75c66a0 ("dt-bindings: backlight: Add lm3630a bindings")
> Signed-off-by: Brian Masney <masneyb@onstation.org>
> Reported-by: Rob Herring <robh+dt@kernel.org>
> ---
> .../leds/backlight/lm3630a-backlight.yaml | 20 +++++++++++++------
> 1 file changed, 14 insertions(+), 6 deletions(-)
>
Applied, thanks.
Rob
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-06-28 19:22 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-20 8:58 [PATCH] dt-bindings: backlight: lm3630a: correct schema validation Brian Masney
2019-05-20 13:14 ` Rob Herring
2019-05-20 13:30 ` Daniel Thompson
2019-05-22 0:53 ` Dan Murphy
2019-06-28 19:22 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).