* [PATCH v1] dt-bindings: backlight: add brightness-levels related common properties
@ 2023-10-16 15:05 Flavio Suligoi
2023-10-16 19:35 ` Krzysztof Kozlowski
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Flavio Suligoi @ 2023-10-16 15:05 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Pavel Machek, Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: devicetree, Flavio Suligoi, linux-leds, dri-devel, linux-kernel
Both files pwm-backlight.yaml and led-backlight.yaml contain properties
in common with each other, regarding the brightness levels:
- brightness-levels
- default-brightness-level
These properties can then be moved to backlight/common.yaml.
Signed-off-by: Flavio Suligoi <f.suligoi@asem.it>
---
.../bindings/leds/backlight/common.yaml | 17 ++++++++++++++++
.../leds/backlight/led-backlight.yaml | 19 ++++--------------
.../leds/backlight/pwm-backlight.yaml | 20 ++++---------------
3 files changed, 25 insertions(+), 31 deletions(-)
diff --git a/Documentation/devicetree/bindings/leds/backlight/common.yaml b/Documentation/devicetree/bindings/leds/backlight/common.yaml
index 3b60afbab68b..e0983e44934c 100644
--- a/Documentation/devicetree/bindings/leds/backlight/common.yaml
+++ b/Documentation/devicetree/bindings/leds/backlight/common.yaml
@@ -33,4 +33,21 @@ properties:
due to restrictions in a specific system, such as mounting conditions.
$ref: /schemas/types.yaml#/definitions/uint32
+ brightness-levels:
+ description:
+ Array of distinct brightness levels. The levels must be in the range
+ accepted by the underlying LED device. Typically these are in the range
+ from 0 to 255, but any range starting at 0 will do, as long as they are
+ accepted by the LED.
+ The 0 value means a 0% of brightness (darkest/off), while the last value
+ in the array represents a full 100% brightness (brightest).
+ If this array is not provided, the driver default mapping is used.
+ $ref: /schemas/types.yaml#/definitions/uint32-array
+
+ default-brightness-level:
+ description:
+ The default brightness level (index into the array defined by the
+ "brightness-levels" property).
+ $ref: /schemas/types.yaml#/definitions/uint32
+
additionalProperties: true
diff --git a/Documentation/devicetree/bindings/leds/backlight/led-backlight.yaml b/Documentation/devicetree/bindings/leds/backlight/led-backlight.yaml
index d7b78198abc2..f5554da6bc6c 100644
--- a/Documentation/devicetree/bindings/leds/backlight/led-backlight.yaml
+++ b/Documentation/devicetree/bindings/leds/backlight/led-backlight.yaml
@@ -16,6 +16,9 @@ description:
can also be used to describe a backlight device controlled by the output of
a LED driver.
+allOf:
+ - $ref: common.yaml#
+
properties:
compatible:
const: led-backlight
@@ -26,25 +29,11 @@ properties:
items:
maxItems: 1
- brightness-levels:
- description:
- Array of distinct brightness levels. The levels must be in the range
- accepted by the underlying LED devices. This is used to translate a
- backlight brightness level into a LED brightness level. If it is not
- provided, the identity mapping is used.
- $ref: /schemas/types.yaml#/definitions/uint32-array
-
- default-brightness-level:
- description:
- The default brightness level (index into the array defined by the
- "brightness-levels" property).
- $ref: /schemas/types.yaml#/definitions/uint32
-
required:
- compatible
- leds
-additionalProperties: false
+unevaluatedProperties: false
examples:
- |
diff --git a/Documentation/devicetree/bindings/leds/backlight/pwm-backlight.yaml b/Documentation/devicetree/bindings/leds/backlight/pwm-backlight.yaml
index 535690288990..b71f6454a4ac 100644
--- a/Documentation/devicetree/bindings/leds/backlight/pwm-backlight.yaml
+++ b/Documentation/devicetree/bindings/leds/backlight/pwm-backlight.yaml
@@ -11,6 +11,9 @@ maintainers:
- Daniel Thompson <daniel.thompson@linaro.org>
- Jingoo Han <jingoohan1@gmail.com>
+allOf:
+ - $ref: common.yaml#
+
properties:
compatible:
const: pwm-backlight
@@ -39,21 +42,6 @@ properties:
Delay in ms between disabling the backlight using GPIO and setting PWM
value to 0.
- brightness-levels:
- description:
- Array of distinct brightness levels. Typically these are in the range
- from 0 to 255, but any range starting at 0 will do. The actual brightness
- level (PWM duty cycle) will be interpolated from these values. 0 means a
- 0% duty cycle (darkest/off), while the last value in the array represents
- a 100% duty cycle (brightest).
- $ref: /schemas/types.yaml#/definitions/uint32-array
-
- default-brightness-level:
- description:
- The default brightness level (index into the array defined by the
- "brightness-levels" property).
- $ref: /schemas/types.yaml#/definitions/uint32
-
num-interpolated-steps:
description:
Number of interpolated steps between each value of brightness-levels
@@ -69,7 +57,7 @@ required:
- compatible
- pwms
-additionalProperties: false
+unevaluatedProperties: false
examples:
- |
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v1] dt-bindings: backlight: add brightness-levels related common properties
2023-10-16 15:05 [PATCH v1] dt-bindings: backlight: add brightness-levels related common properties Flavio Suligoi
@ 2023-10-16 19:35 ` Krzysztof Kozlowski
2023-10-17 18:04 ` Rob Herring
2023-10-19 12:27 ` (subset) " Lee Jones
2 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2023-10-16 19:35 UTC (permalink / raw)
To: Flavio Suligoi, Lee Jones, Daniel Thompson, Jingoo Han,
Helge Deller, Pavel Machek, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: devicetree, linux-leds, dri-devel, linux-kernel
On 16/10/2023 17:05, Flavio Suligoi wrote:
> Both files pwm-backlight.yaml and led-backlight.yaml contain properties
> in common with each other, regarding the brightness levels:
>
> - brightness-levels
> - default-brightness-level
>
> These properties can then be moved to backlight/common.yaml.
>
> Signed-off-by: Flavio Suligoi <f.suligoi@asem.it>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v1] dt-bindings: backlight: add brightness-levels related common properties
2023-10-16 15:05 [PATCH v1] dt-bindings: backlight: add brightness-levels related common properties Flavio Suligoi
2023-10-16 19:35 ` Krzysztof Kozlowski
@ 2023-10-17 18:04 ` Rob Herring
2023-10-19 12:27 ` (subset) " Lee Jones
2 siblings, 0 replies; 4+ messages in thread
From: Rob Herring @ 2023-10-17 18:04 UTC (permalink / raw)
To: Flavio Suligoi
Cc: devicetree, Daniel Thompson, Pavel Machek, Jingoo Han,
Helge Deller, Lee Jones, Conor Dooley, dri-devel, linux-kernel,
Rob Herring, Krzysztof Kozlowski, linux-leds
On Mon, 16 Oct 2023 17:05:54 +0200, Flavio Suligoi wrote:
> Both files pwm-backlight.yaml and led-backlight.yaml contain properties
> in common with each other, regarding the brightness levels:
>
> - brightness-levels
> - default-brightness-level
>
> These properties can then be moved to backlight/common.yaml.
>
> Signed-off-by: Flavio Suligoi <f.suligoi@asem.it>
> ---
> .../bindings/leds/backlight/common.yaml | 17 ++++++++++++++++
> .../leds/backlight/led-backlight.yaml | 19 ++++--------------
> .../leds/backlight/pwm-backlight.yaml | 20 ++++---------------
> 3 files changed, 25 insertions(+), 31 deletions(-)
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: (subset) [PATCH v1] dt-bindings: backlight: add brightness-levels related common properties
2023-10-16 15:05 [PATCH v1] dt-bindings: backlight: add brightness-levels related common properties Flavio Suligoi
2023-10-16 19:35 ` Krzysztof Kozlowski
2023-10-17 18:04 ` Rob Herring
@ 2023-10-19 12:27 ` Lee Jones
2 siblings, 0 replies; 4+ messages in thread
From: Lee Jones @ 2023-10-19 12:27 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Pavel Machek, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Flavio Suligoi
Cc: devicetree, linux-leds, dri-devel, linux-kernel
On Mon, 16 Oct 2023 17:05:54 +0200, Flavio Suligoi wrote:
> Both files pwm-backlight.yaml and led-backlight.yaml contain properties
> in common with each other, regarding the brightness levels:
>
> - brightness-levels
> - default-brightness-level
>
> These properties can then be moved to backlight/common.yaml.
>
> [...]
Applied, thanks!
[1/1] dt-bindings: backlight: add brightness-levels related common properties
commit: d5272d39995f4150062a67e6f2cef556edece740
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-10-19 12:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-16 15:05 [PATCH v1] dt-bindings: backlight: add brightness-levels related common properties Flavio Suligoi
2023-10-16 19:35 ` Krzysztof Kozlowski
2023-10-17 18:04 ` Rob Herring
2023-10-19 12:27 ` (subset) " Lee Jones
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).