[v2] dt-bindings: mt76: add active-low property for led

Commit Message

Frank Wunderlich Feb. 7, 2023, 1:35 p.m. UTC

From: Frank Wunderlich <[e-mail]>

LEDs can be in low-active mode, driver already supports it, but
documentation is missing. Add documentation for the dt property.

Signed-off-by: Frank Wunderlich <[e-mail]>
---
v2:
- change commit message to mention that driver already support this
---
 .../devicetree/bindings/net/wireless/mediatek,mt76.yaml      | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Krzysztof Kozlowski Feb. 7, 2023, 1:36 p.m. UTC | #1

On 07/02/2023 14:35, Frank Wunderlich wrote:
> From: Frank Wunderlich <[e-mail]>
> 
> LEDs can be in low-active mode, driver already supports it, but
> documentation is missing. Add documentation for the dt property.
> 
> Signed-off-by: Frank Wunderlich <[e-mail]>\


Acked-by: Krzysztof Kozlowski <[e-mail]>

Best regards,
Krzysztof

Matthias Brugger March 30, 2023, 5:09 p.m. UTC | #2

Hi Felix and Kalle,

On 07/02/2023 14:36, Krzysztof Kozlowski wrote:
> On 07/02/2023 14:35, Frank Wunderlich wrote:
>> From: Frank Wunderlich <[e-mail]>
>>
>> LEDs can be in low-active mode, driver already supports it, but
>> documentation is missing. Add documentation for the dt property.
>>
>> Signed-off-by: Frank Wunderlich <[e-mail]>\
> 
> 
> Acked-by: Krzysztof Kozlowski <[e-mail]>
> 

Can you take that through your tree or are fine if I take it through mine?

Regards,
Matthias

Kalle Valo March 31, 2023, 2:22 p.m. UTC | #3

Matthias Brugger <[e-mail]> writes:

> On 07/02/2023 14:36, Krzysztof Kozlowski wrote:
>> On 07/02/2023 14:35, Frank Wunderlich wrote:
>>> From: Frank Wunderlich <[e-mail]>
>>>
>>> LEDs can be in low-active mode, driver already supports it, but
>>> documentation is missing. Add documentation for the dt property.
>>>
>>> Signed-off-by: Frank Wunderlich <[e-mail]>\
>>
>>
>> Acked-by: Krzysztof Kozlowski <[e-mail]>
>>
>
> Can you take that through your tree or are fine if I take it through mine?

It would be safest if Felix can take this to his tree, less conflicts
the better.

Patch

diff --git a/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.yaml b/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.yaml
index f0c78f994491..212508672979 100644
--- a/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.yaml
+++ b/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.yaml
@@ -112,6 +112,11 @@  properties:
     $ref: /schemas/leds/common.yaml#
     additionalProperties: false
     properties:
+      led-active-low:
+        description:
+          LED is enabled with ground signal.
+        type: boolean
+
       led-sources:
         maxItems: 1