* [PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level
@ 2023-05-19 20:05 Alexandru Ardelean
2023-05-19 20:05 ` [PATCH 2/2] dt-bindings: backlight: document " Alexandru Ardelean
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Alexandru Ardelean @ 2023-05-19 20:05 UTC (permalink / raw)
To: dri-devel, linux-leds, devicetree, linux-kernel, linux-fbdev
Cc: lee, daniel.thompson, jingoohan1, pavel, robh+dt,
krzysztof.kozlowski+dt, conor+dt, deller, Yannick Fertre,
Philippe CORNU, Alexandru Ardelean
From: Yannick Fertre <yannick.fertre@foss.st.com>
Add new property to set a brightness by default at probe.
Reviewed-by: Philippe CORNU <philippe.cornu@foss.st.com>
Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
Signed-off-by: Alexandru Ardelean <alex@shruggie.ro>
---
Link to original patch:
https://github.com/STMicroelectronics/linux/commit/c4067d7bd883c6fa14ffd49892c4ce663cdafe98
drivers/video/backlight/gpio_backlight.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/video/backlight/gpio_backlight.c b/drivers/video/backlight/gpio_backlight.c
index 6f78d928f054..d3fa3a8bef4d 100644
--- a/drivers/video/backlight/gpio_backlight.c
+++ b/drivers/video/backlight/gpio_backlight.c
@@ -53,6 +53,7 @@ static int gpio_backlight_probe(struct platform_device *pdev)
struct backlight_device *bl;
struct gpio_backlight *gbl;
int ret, init_brightness, def_value;
+ u32 value;
gbl = devm_kzalloc(dev, sizeof(*gbl), GFP_KERNEL);
if (gbl == NULL)
@@ -93,7 +94,11 @@ static int gpio_backlight_probe(struct platform_device *pdev)
else
bl->props.power = FB_BLANK_UNBLANK;
- bl->props.brightness = 1;
+ ret = device_property_read_u32(dev, "default-brightness-level", &value);
+ if (!ret && value <= props.max_brightness)
+ bl->props.brightness = value;
+ else
+ bl->props.brightness = 1;
init_brightness = backlight_get_brightness(bl);
ret = gpiod_direction_output(gbl->gpiod, init_brightness);
--
2.40.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] dt-bindings: backlight: document new property default-brightness-level
2023-05-19 20:05 [PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level Alexandru Ardelean
@ 2023-05-19 20:05 ` Alexandru Ardelean
2023-05-26 10:20 ` Daniel Thompson
2023-05-26 12:05 ` Philippe CORNU
2023-05-26 10:27 ` [PATCH 1/2] backlight: gpio_backlight: add " Daniel Thompson
2023-05-26 12:04 ` Philippe CORNU
2 siblings, 2 replies; 9+ messages in thread
From: Alexandru Ardelean @ 2023-05-19 20:05 UTC (permalink / raw)
To: dri-devel, linux-leds, devicetree, linux-kernel, linux-fbdev
Cc: lee, daniel.thompson, jingoohan1, pavel, robh+dt,
krzysztof.kozlowski+dt, conor+dt, deller, Yannick Fertre,
Philippe CORNU, Alexandru Ardelean
From: Yannick Fertre <yannick.fertre@foss.st.com>
Add documentation for new default-brightness-level property.
Reviewed-by: Philippe CORNU <philippe.cornu@foss.st.com>
Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
Signed-off-by: Alexandru Ardelean <alex@shruggie.ro>
---
Link to original patch:
https://github.com/STMicroelectronics/linux/commit/c4067d7bd883c6fa14ffd49892c4ce663cdafe98
.../bindings/leds/backlight/gpio-backlight.yaml | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
index 584030b6b0b9..b96c08cff0f0 100644
--- a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
+++ b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
@@ -23,6 +23,15 @@ properties:
description: enable the backlight at boot.
type: boolean
+ default-brightness-level:
+ description:
+ The default brightness level (index into the array defined by the
+ "brightness-levels" property).
+ $ref: /schemas/types.yaml#/definitions/uint32
+
+dependencies:
+ default-brightness-level: [ "brightness-levels" ]
+
required:
- compatible
- gpios
--
2.40.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: backlight: document new property default-brightness-level
2023-05-19 20:05 ` [PATCH 2/2] dt-bindings: backlight: document " Alexandru Ardelean
@ 2023-05-26 10:20 ` Daniel Thompson
2023-05-29 7:03 ` Alexandru Ardelean
2023-05-26 12:05 ` Philippe CORNU
1 sibling, 1 reply; 9+ messages in thread
From: Daniel Thompson @ 2023-05-26 10:20 UTC (permalink / raw)
To: Alexandru Ardelean
Cc: dri-devel, linux-leds, devicetree, linux-kernel, linux-fbdev,
lee, jingoohan1, pavel, robh+dt, krzysztof.kozlowski+dt,
conor+dt, deller, Yannick Fertre, Philippe CORNU
On Fri, May 19, 2023 at 11:05:20PM +0300, Alexandru Ardelean wrote:
> From: Yannick Fertre <yannick.fertre@foss.st.com>
>
> Add documentation for new default-brightness-level property.
>
> Reviewed-by: Philippe CORNU <philippe.cornu@foss.st.com>
> Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> Signed-off-by: Alexandru Ardelean <alex@shruggie.ro>
> ---
>
> Link to original patch:
> https://github.com/STMicroelectronics/linux/commit/c4067d7bd883c6fa14ffd49892c4ce663cdafe98
>
> .../bindings/leds/backlight/gpio-backlight.yaml | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> index 584030b6b0b9..b96c08cff0f0 100644
> --- a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> +++ b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> @@ -23,6 +23,15 @@ properties:
> description: enable the backlight at boot.
> type: boolean
>
> + default-brightness-level:
> + description:
> + The default brightness level (index into the array defined by the
> + "brightness-levels" property).
gpio-backlight does not have a brightness-levels array property!
I think it is also necessary to improve the docs of both properties to
distinguish between the meaning of default-on and
default-brightness-level. The result of setting default-on and
default-brightness level to zero is that the GPIO will be off (this is
correct behaviour but hard to figure out from the current text).
default-on is a control that can "enable" the backlight at boot when it
is not linked to a display in the DT (e.g. it is mostly for legacy
cases). When the backlight is linked to a display then the backlight
enable state will be automatically linked to the display enable state
instead.
default-brightness-level is useful for handling displays that
are still readable with the backlight off (e-ink, reflective/
transflexive LCD, etc), otherwise is should be absent or set to 1.
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> +dependencies:
> + default-brightness-level: [ "brightness-levels" ]
As above, depending on brightness-levels doesn't make any sense here.
Daniel.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level
2023-05-19 20:05 [PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level Alexandru Ardelean
2023-05-19 20:05 ` [PATCH 2/2] dt-bindings: backlight: document " Alexandru Ardelean
@ 2023-05-26 10:27 ` Daniel Thompson
2023-05-26 12:04 ` Philippe CORNU
2 siblings, 0 replies; 9+ messages in thread
From: Daniel Thompson @ 2023-05-26 10:27 UTC (permalink / raw)
To: Alexandru Ardelean
Cc: dri-devel, linux-leds, devicetree, linux-kernel, linux-fbdev,
lee, jingoohan1, pavel, robh+dt, krzysztof.kozlowski+dt,
conor+dt, deller, Yannick Fertre, Philippe CORNU
On Fri, May 19, 2023 at 11:05:19PM +0300, Alexandru Ardelean wrote:
> From: Yannick Fertre <yannick.fertre@foss.st.com>
>
> Add new property to set a brightness by default at probe.
>
> Reviewed-by: Philippe CORNU <philippe.cornu@foss.st.com>
> Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> Signed-off-by: Alexandru Ardelean <alex@shruggie.ro>
Patch order should be reversed for v2. Nevertheless:
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
> struct backlight_device *bl;
> struct gpio_backlight *gbl;
> int ret, init_brightness, def_value;
> + u32 value;
>
> gbl = devm_kzalloc(dev, sizeof(*gbl), GFP_KERNEL);
> if (gbl == NULL)
> @@ -93,7 +94,11 @@ static int gpio_backlight_probe(struct platform_device *pdev)
> else
> bl->props.power = FB_BLANK_UNBLANK;
>
> - bl->props.brightness = 1;
> + ret = device_property_read_u32(dev, "default-brightness-level", &value);
> + if (!ret && value <= props.max_brightness)
> + bl->props.brightness = value;
> + else
> + bl->props.brightness = 1;
>
> init_brightness = backlight_get_brightness(bl);
> ret = gpiod_direction_output(gbl->gpiod, init_brightness);
> --
> 2.40.1
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level
2023-05-19 20:05 [PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level Alexandru Ardelean
2023-05-19 20:05 ` [PATCH 2/2] dt-bindings: backlight: document " Alexandru Ardelean
2023-05-26 10:27 ` [PATCH 1/2] backlight: gpio_backlight: add " Daniel Thompson
@ 2023-05-26 12:04 ` Philippe CORNU
2023-05-29 7:02 ` Alexandru Ardelean
2 siblings, 1 reply; 9+ messages in thread
From: Philippe CORNU @ 2023-05-26 12:04 UTC (permalink / raw)
To: Alexandru Ardelean, dri-devel, linux-leds, devicetree,
linux-kernel, linux-fbdev
Cc: lee, daniel.thompson, jingoohan1, pavel, robh+dt,
krzysztof.kozlowski+dt, conor+dt, deller, Yannick Fertre
On 5/19/23 22:05, Alexandru Ardelean wrote:
> From: Yannick Fertre <yannick.fertre@foss.st.com>
>
> Add new property to set a brightness by default at probe.
>
> Reviewed-by: Philippe CORNU <philippe.cornu@foss.st.com>
Hi Alexandru,
Many thanks for your patch.
You have sent a patch originally pushed on the STMicroelectronics github
as mentioned in your commit message (no problem with that :-). But, the
"Reviewed-by" inside this github patch is linked to our gerrit STM
internal server so you can not use it directly for mainlining this patch.
So please, re-send your this patch without my "Reviewed-by".
Many thanks
Philippe :-)
> Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> Signed-off-by: Alexandru Ardelean <alex@shruggie.ro>
> ---
>
> Link to original patch:
> https://github.com/STMicroelectronics/linux/commit/c4067d7bd883c6fa14ffd49892c4ce663cdafe98
>
> drivers/video/backlight/gpio_backlight.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/video/backlight/gpio_backlight.c b/drivers/video/backlight/gpio_backlight.c
> index 6f78d928f054..d3fa3a8bef4d 100644
> --- a/drivers/video/backlight/gpio_backlight.c
> +++ b/drivers/video/backlight/gpio_backlight.c
> @@ -53,6 +53,7 @@ static int gpio_backlight_probe(struct platform_device *pdev)
> struct backlight_device *bl;
> struct gpio_backlight *gbl;
> int ret, init_brightness, def_value;
> + u32 value;
>
> gbl = devm_kzalloc(dev, sizeof(*gbl), GFP_KERNEL);
> if (gbl == NULL)
> @@ -93,7 +94,11 @@ static int gpio_backlight_probe(struct platform_device *pdev)
> else
> bl->props.power = FB_BLANK_UNBLANK;
>
> - bl->props.brightness = 1;
> + ret = device_property_read_u32(dev, "default-brightness-level", &value);
> + if (!ret && value <= props.max_brightness)
> + bl->props.brightness = value;
> + else
> + bl->props.brightness = 1;
>
> init_brightness = backlight_get_brightness(bl);
> ret = gpiod_direction_output(gbl->gpiod, init_brightness);
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: backlight: document new property default-brightness-level
2023-05-19 20:05 ` [PATCH 2/2] dt-bindings: backlight: document " Alexandru Ardelean
2023-05-26 10:20 ` Daniel Thompson
@ 2023-05-26 12:05 ` Philippe CORNU
2023-05-29 7:03 ` Alexandru Ardelean
1 sibling, 1 reply; 9+ messages in thread
From: Philippe CORNU @ 2023-05-26 12:05 UTC (permalink / raw)
To: Alexandru Ardelean, dri-devel, linux-leds, devicetree,
linux-kernel, linux-fbdev
Cc: lee, daniel.thompson, jingoohan1, pavel, robh+dt,
krzysztof.kozlowski+dt, conor+dt, deller, Yannick Fertre
On 5/19/23 22:05, Alexandru Ardelean wrote:
> From: Yannick Fertre <yannick.fertre@foss.st.com>
>
> Add documentation for new default-brightness-level property.
>
> Reviewed-by: Philippe CORNU <philippe.cornu@foss.st.com>
Hi Alexandru,
same comments as for the 1/2 patch.
Many thanks
Philippe :-)
> Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> Signed-off-by: Alexandru Ardelean <alex@shruggie.ro>
> ---
>
> Link to original patch:
> https://github.com/STMicroelectronics/linux/commit/c4067d7bd883c6fa14ffd49892c4ce663cdafe98
>
> .../bindings/leds/backlight/gpio-backlight.yaml | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> index 584030b6b0b9..b96c08cff0f0 100644
> --- a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> +++ b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> @@ -23,6 +23,15 @@ properties:
> description: enable the backlight at boot.
> type: boolean
>
> + default-brightness-level:
> + description:
> + The default brightness level (index into the array defined by the
> + "brightness-levels" property).
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> +dependencies:
> + default-brightness-level: [ "brightness-levels" ]
> +
> required:
> - compatible
> - gpios
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level
2023-05-26 12:04 ` Philippe CORNU
@ 2023-05-29 7:02 ` Alexandru Ardelean
0 siblings, 0 replies; 9+ messages in thread
From: Alexandru Ardelean @ 2023-05-29 7:02 UTC (permalink / raw)
To: Philippe CORNU
Cc: dri-devel, linux-leds, devicetree, linux-kernel, linux-fbdev,
lee, daniel.thompson, jingoohan1, pavel, robh+dt,
krzysztof.kozlowski+dt, conor+dt, deller, Yannick Fertre
On Fri, May 26, 2023 at 3:04 PM Philippe CORNU
<philippe.cornu@foss.st.com> wrote:
>
>
> On 5/19/23 22:05, Alexandru Ardelean wrote:
> > From: Yannick Fertre <yannick.fertre@foss.st.com>
> >
> > Add new property to set a brightness by default at probe.
> >
> > Reviewed-by: Philippe CORNU <philippe.cornu@foss.st.com>
>
> Hi Alexandru,
>
> Many thanks for your patch.
>
> You have sent a patch originally pushed on the STMicroelectronics github
> as mentioned in your commit message (no problem with that :-). But, the
> "Reviewed-by" inside this github patch is linked to our gerrit STM
> internal server so you can not use it directly for mainlining this patch.
>
> So please, re-send your this patch without my "Reviewed-by".
ack
will do
>
> Many thanks
> Philippe :-)
>
>
> > Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> > Signed-off-by: Alexandru Ardelean <alex@shruggie.ro>
> > ---
> >
> > Link to original patch:
> > https://github.com/STMicroelectronics/linux/commit/c4067d7bd883c6fa14ffd49892c4ce663cdafe98
> >
> > drivers/video/backlight/gpio_backlight.c | 7 ++++++-
> > 1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/video/backlight/gpio_backlight.c b/drivers/video/backlight/gpio_backlight.c
> > index 6f78d928f054..d3fa3a8bef4d 100644
> > --- a/drivers/video/backlight/gpio_backlight.c
> > +++ b/drivers/video/backlight/gpio_backlight.c
> > @@ -53,6 +53,7 @@ static int gpio_backlight_probe(struct platform_device *pdev)
> > struct backlight_device *bl;
> > struct gpio_backlight *gbl;
> > int ret, init_brightness, def_value;
> > + u32 value;
> >
> > gbl = devm_kzalloc(dev, sizeof(*gbl), GFP_KERNEL);
> > if (gbl == NULL)
> > @@ -93,7 +94,11 @@ static int gpio_backlight_probe(struct platform_device *pdev)
> > else
> > bl->props.power = FB_BLANK_UNBLANK;
> >
> > - bl->props.brightness = 1;
> > + ret = device_property_read_u32(dev, "default-brightness-level", &value);
> > + if (!ret && value <= props.max_brightness)
> > + bl->props.brightness = value;
> > + else
> > + bl->props.brightness = 1;
> >
> > init_brightness = backlight_get_brightness(bl);
> > ret = gpiod_direction_output(gbl->gpiod, init_brightness);
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: backlight: document new property default-brightness-level
2023-05-26 10:20 ` Daniel Thompson
@ 2023-05-29 7:03 ` Alexandru Ardelean
0 siblings, 0 replies; 9+ messages in thread
From: Alexandru Ardelean @ 2023-05-29 7:03 UTC (permalink / raw)
To: Daniel Thompson
Cc: dri-devel, linux-leds, devicetree, linux-kernel, linux-fbdev,
lee, jingoohan1, pavel, robh+dt, krzysztof.kozlowski+dt,
conor+dt, deller, Yannick Fertre, Philippe CORNU
On Fri, May 26, 2023 at 1:20 PM Daniel Thompson
<daniel.thompson@linaro.org> wrote:
>
> On Fri, May 19, 2023 at 11:05:20PM +0300, Alexandru Ardelean wrote:
> > From: Yannick Fertre <yannick.fertre@foss.st.com>
> >
> > Add documentation for new default-brightness-level property.
> >
> > Reviewed-by: Philippe CORNU <philippe.cornu@foss.st.com>
> > Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> > Signed-off-by: Alexandru Ardelean <alex@shruggie.ro>
> > ---
> >
> > Link to original patch:
> > https://github.com/STMicroelectronics/linux/commit/c4067d7bd883c6fa14ffd49892c4ce663cdafe98
> >
> > .../bindings/leds/backlight/gpio-backlight.yaml | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> > index 584030b6b0b9..b96c08cff0f0 100644
> > --- a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> > +++ b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> > @@ -23,6 +23,15 @@ properties:
> > description: enable the backlight at boot.
> > type: boolean
> >
> > + default-brightness-level:
> > + description:
> > + The default brightness level (index into the array defined by the
> > + "brightness-levels" property).
>
> gpio-backlight does not have a brightness-levels array property!
>
> I think it is also necessary to improve the docs of both properties to
> distinguish between the meaning of default-on and
> default-brightness-level. The result of setting default-on and
> default-brightness level to zero is that the GPIO will be off (this is
> correct behaviour but hard to figure out from the current text).
>
> default-on is a control that can "enable" the backlight at boot when it
> is not linked to a display in the DT (e.g. it is mostly for legacy
> cases). When the backlight is linked to a display then the backlight
> enable state will be automatically linked to the display enable state
> instead.
>
> default-brightness-level is useful for handling displays that
> are still readable with the backlight off (e-ink, reflective/
> transflexive LCD, etc), otherwise is should be absent or set to 1.
ack
will do in V2
thanks
>
>
> > + $ref: /schemas/types.yaml#/definitions/uint32
> > +
> > +dependencies:
> > + default-brightness-level: [ "brightness-levels" ]
>
> As above, depending on brightness-levels doesn't make any sense here.
>
>
> Daniel.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: backlight: document new property default-brightness-level
2023-05-26 12:05 ` Philippe CORNU
@ 2023-05-29 7:03 ` Alexandru Ardelean
0 siblings, 0 replies; 9+ messages in thread
From: Alexandru Ardelean @ 2023-05-29 7:03 UTC (permalink / raw)
To: Philippe CORNU
Cc: dri-devel, linux-leds, devicetree, linux-kernel, linux-fbdev,
lee, daniel.thompson, jingoohan1, pavel, robh+dt,
krzysztof.kozlowski+dt, conor+dt, deller, Yannick Fertre
On Fri, May 26, 2023 at 3:05 PM Philippe CORNU
<philippe.cornu@foss.st.com> wrote:
>
>
>
> On 5/19/23 22:05, Alexandru Ardelean wrote:
> > From: Yannick Fertre <yannick.fertre@foss.st.com>
> >
> > Add documentation for new default-brightness-level property.
> >
> > Reviewed-by: Philippe CORNU <philippe.cornu@foss.st.com>
>
> Hi Alexandru,
> same comments as for the 1/2 patch.
Ack
Will do
Thanks
Alexandru
> Many thanks
> Philippe :-)
>
> > Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> > Signed-off-by: Alexandru Ardelean <alex@shruggie.ro>
> > ---
> >
> > Link to original patch:
> > https://github.com/STMicroelectronics/linux/commit/c4067d7bd883c6fa14ffd49892c4ce663cdafe98
> >
> > .../bindings/leds/backlight/gpio-backlight.yaml | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> > index 584030b6b0b9..b96c08cff0f0 100644
> > --- a/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> > +++ b/Documentation/devicetree/bindings/leds/backlight/gpio-backlight.yaml
> > @@ -23,6 +23,15 @@ properties:
> > description: enable the backlight at boot.
> > type: boolean
> >
> > + default-brightness-level:
> > + description:
> > + The default brightness level (index into the array defined by the
> > + "brightness-levels" property).
> > + $ref: /schemas/types.yaml#/definitions/uint32
> > +
> > +dependencies:
> > + default-brightness-level: [ "brightness-levels" ]
> > +
> > required:
> > - compatible
> > - gpios
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2023-05-29 7:04 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-19 20:05 [PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level Alexandru Ardelean
2023-05-19 20:05 ` [PATCH 2/2] dt-bindings: backlight: document " Alexandru Ardelean
2023-05-26 10:20 ` Daniel Thompson
2023-05-29 7:03 ` Alexandru Ardelean
2023-05-26 12:05 ` Philippe CORNU
2023-05-29 7:03 ` Alexandru Ardelean
2023-05-26 10:27 ` [PATCH 1/2] backlight: gpio_backlight: add " Daniel Thompson
2023-05-26 12:04 ` Philippe CORNU
2023-05-29 7:02 ` Alexandru Ardelean
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).