diff mbox series

dt-bindings: leds: common: Add mmc0 as default trigger

Message ID 20200830111115.32623-1-krzk@kernel.org
State New
Headers show
Series dt-bindings: leds: common: Add mmc0 as default trigger | expand

Commit Message

Krzysztof Kozlowski Aug. 30, 2020, 11:11 a.m. UTC
MMC could be a default trigger so add a pattern to match it and fix
dtbs_check warnings like:

  arch/arm/boot/dts/exynos4412-odroidx.dt.yaml: leds: led2:linux,default-trigger:0:
    'mmc0' is not one of ['backlight', 'default-on', 'heartbeat', 'disk-activity', 'ide-disk', 'timer', 'pattern']
    From schema: Documentation/devicetree/bindings/leds/leds-gpio.yaml

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../devicetree/bindings/leds/common.yaml      | 39 ++++++++++---------
 1 file changed, 20 insertions(+), 19 deletions(-)

Comments

Rob Herring (Arm) Sept. 14, 2020, 6:20 p.m. UTC | #1
On Sun, Aug 30, 2020 at 01:11:15PM +0200, Krzysztof Kozlowski wrote:
> MMC could be a default trigger so add a pattern to match it and fix

> dtbs_check warnings like:

> 

>   arch/arm/boot/dts/exynos4412-odroidx.dt.yaml: leds: led2:linux,default-trigger:0:

>     'mmc0' is not one of ['backlight', 'default-on', 'heartbeat', 'disk-activity', 'ide-disk', 'timer', 'pattern']

>     From schema: Documentation/devicetree/bindings/leds/leds-gpio.yaml


The preference is to move from 'linux,default-trigger' to 'function' 
with some standardization of the names or use the trigger-source binding.

> 

> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

> ---

>  .../devicetree/bindings/leds/common.yaml      | 39 ++++++++++---------

>  1 file changed, 20 insertions(+), 19 deletions(-)

> 

> diff --git a/Documentation/devicetree/bindings/leds/common.yaml b/Documentation/devicetree/bindings/leds/common.yaml

> index a2a541bca73c..6b38f9f3792c 100644

> --- a/Documentation/devicetree/bindings/leds/common.yaml

> +++ b/Documentation/devicetree/bindings/leds/common.yaml

> @@ -78,25 +78,26 @@ properties:

>        This parameter, if present, is a string defining the trigger assigned to

>        the LED.

>      $ref: /schemas/types.yaml#definitions/string

> -

> -    enum:

> -        # LED will act as a back-light, controlled by the framebuffer system

> -      - backlight

> -        # LED will turn on (but for leds-gpio see "default-state" property in

> -        # Documentation/devicetree/bindings/leds/leds-gpio.yaml)

> -      - default-on

> -        # LED "double" flashes at a load average based rate

> -      - heartbeat

> -        # LED indicates disk activity

> -      - disk-activity

> -        # LED indicates IDE disk activity (deprecated), in new implementations

> -        # use "disk-activity"

> -      - ide-disk

> -        # LED flashes at a fixed, configurable rate

> -      - timer

> -        # LED alters the brightness for the specified duration with one software

> -        # timer (requires "led-pattern" property)

> -      - pattern

> +    oneOf:

> +      - enum:

> +            # LED will act as a back-light, controlled by the framebuffer system

> +          - backlight

> +            # LED will turn on (but for leds-gpio see "default-state" property in

> +            # Documentation/devicetree/bindings/leds/leds-gpio.yaml)

> +          - default-on

> +            # LED "double" flashes at a load average based rate

> +          - heartbeat

> +            # LED indicates disk activity

> +          - disk-activity

> +            # LED indicates IDE disk activity (deprecated), in new implementations

> +            # use "disk-activity"

> +          - ide-disk

> +            # LED flashes at a fixed, configurable rate

> +          - timer

> +            # LED alters the brightness for the specified duration with one software

> +            # timer (requires "led-pattern" property)

> +          - pattern

> +      - pattern: "^mmc[0-9]+$"

>  

>    led-pattern:

>      description: |

> -- 

> 2.17.1

>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/leds/common.yaml b/Documentation/devicetree/bindings/leds/common.yaml
index a2a541bca73c..6b38f9f3792c 100644
--- a/Documentation/devicetree/bindings/leds/common.yaml
+++ b/Documentation/devicetree/bindings/leds/common.yaml
@@ -78,25 +78,26 @@  properties:
       This parameter, if present, is a string defining the trigger assigned to
       the LED.
     $ref: /schemas/types.yaml#definitions/string
-
-    enum:
-        # LED will act as a back-light, controlled by the framebuffer system
-      - backlight
-        # LED will turn on (but for leds-gpio see "default-state" property in
-        # Documentation/devicetree/bindings/leds/leds-gpio.yaml)
-      - default-on
-        # LED "double" flashes at a load average based rate
-      - heartbeat
-        # LED indicates disk activity
-      - disk-activity
-        # LED indicates IDE disk activity (deprecated), in new implementations
-        # use "disk-activity"
-      - ide-disk
-        # LED flashes at a fixed, configurable rate
-      - timer
-        # LED alters the brightness for the specified duration with one software
-        # timer (requires "led-pattern" property)
-      - pattern
+    oneOf:
+      - enum:
+            # LED will act as a back-light, controlled by the framebuffer system
+          - backlight
+            # LED will turn on (but for leds-gpio see "default-state" property in
+            # Documentation/devicetree/bindings/leds/leds-gpio.yaml)
+          - default-on
+            # LED "double" flashes at a load average based rate
+          - heartbeat
+            # LED indicates disk activity
+          - disk-activity
+            # LED indicates IDE disk activity (deprecated), in new implementations
+            # use "disk-activity"
+          - ide-disk
+            # LED flashes at a fixed, configurable rate
+          - timer
+            # LED alters the brightness for the specified duration with one software
+            # timer (requires "led-pattern" property)
+          - pattern
+      - pattern: "^mmc[0-9]+$"
 
   led-pattern:
     description: |