This patch series adds some LED_FUNCTION_* definitions mainly for router
devices.
Those definitions are useful for OpenWrt or something.
v1 -> v2
- fix sort order of LED_FUNCTION_MOBILE
- improve the commit description of the first commit
INAGAKI Hiroshi (2):
dt-bindings: leds: add LED_FUNCTION_MOBILE for mobile network
dt-bindings: leds: add LED_FUNCTION_SPEED_* for link speed on LAN/WAN
include/dt-bindings/leds/common.h | 3 +++
1 file changed, 3 insertions(+)
--
2.25.1
Add LED_FUNCTION_MOBILE for LEDs that indicate status of mobile network
connection. This is useful to distinguish those LEDs from LEDs that
indicates status of wired "wan" connection.
example (on stock fw):
IIJ SA-W2 has "Mobile" LEDs that indicate status (no signal, too low,
low, good) of mobile network connection via dongle connected to USB
port.
- no signal: (none, turned off)
- too low: green:mobile & red:mobile (amber, blink)
- low: green:mobile & red:mobile (amber, turned on)
- good: green:mobile (turned on)
Suggested-by: Hauke Mehrtens <[email protected]>
Signed-off-by: INAGAKI Hiroshi <[email protected]>
---
v1 -> v2
- move _MOBILE before _MTD
- improve the commit description
include/dt-bindings/leds/common.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/dt-bindings/leds/common.h b/include/dt-bindings/leds/common.h
index ecea167930d9..6216ecdb06c7 100644
--- a/include/dt-bindings/leds/common.h
+++ b/include/dt-bindings/leds/common.h
@@ -90,6 +90,7 @@
#define LED_FUNCTION_INDICATOR "indicator"
#define LED_FUNCTION_LAN "lan"
#define LED_FUNCTION_MAIL "mail"
+#define LED_FUNCTION_MOBILE "mobile"
#define LED_FUNCTION_MTD "mtd"
#define LED_FUNCTION_PANIC "panic"
#define LED_FUNCTION_PROGRAMMING "programming"
--
2.25.1
Add LED_FUNCTION_SPEED_LAN and LED_FUNCTION_SPEED_WAN for LEDs that
indicate link speed of ethernet ports on LAN/WAN. This is useful to
distinguish those LEDs from LEDs that indicate link status (up/down).
example:
Fortinet FortiGate 30E/50E have LEDs that indicate link speed on each
of the ethernet ports in addition to LEDs that indicate link status
(up/down).
- 1000 Mbps: green:speed-(lan|wan)-N
- 100 Mbps: amber:speed-(lan|wan)-N
- 10 Mbps: (none, turned off)
Reviewed-by: Rob Herring <[email protected]>
Signed-off-by: INAGAKI Hiroshi <[email protected]>
---
v1 -> v2
(no changes)
include/dt-bindings/leds/common.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/dt-bindings/leds/common.h b/include/dt-bindings/leds/common.h
index 6216ecdb06c7..82a5769725ce 100644
--- a/include/dt-bindings/leds/common.h
+++ b/include/dt-bindings/leds/common.h
@@ -96,6 +96,8 @@
#define LED_FUNCTION_PROGRAMMING "programming"
#define LED_FUNCTION_RX "rx"
#define LED_FUNCTION_SD "sd"
+#define LED_FUNCTION_SPEED_LAN "speed-lan"
+#define LED_FUNCTION_SPEED_WAN "speed-wan"
#define LED_FUNCTION_STANDBY "standby"
#define LED_FUNCTION_TORCH "torch"
#define LED_FUNCTION_TX "tx"
--
2.25.1
On 23/03/2024 08:36, INAGAKI Hiroshi wrote:
> Add LED_FUNCTION_MOBILE for LEDs that indicate status of mobile network
> connection. This is useful to distinguish those LEDs from LEDs that
> indicates status of wired "wan" connection.
>
> example (on stock fw):
>
Acked-by: Krzysztof Kozlowski <[email protected]>
Best regards,
Krzysztof
On Sat, 23 Mar 2024 16:36:09 +0900, INAGAKI Hiroshi wrote:
> Add LED_FUNCTION_MOBILE for LEDs that indicate status of mobile network
> connection. This is useful to distinguish those LEDs from LEDs that
> indicates status of wired "wan" connection.
>
> example (on stock fw):
>
> IIJ SA-W2 has "Mobile" LEDs that indicate status (no signal, too low,
> low, good) of mobile network connection via dongle connected to USB
> port.
>
> - no signal: (none, turned off)
> - too low: green:mobile & red:mobile (amber, blink)
> - low: green:mobile & red:mobile (amber, turned on)
> - good: green:mobile (turned on)
>
> Suggested-by: Hauke Mehrtens <[email protected]>
> Signed-off-by: INAGAKI Hiroshi <[email protected]>
> ---
> v1 -> v2
>
> - move _MOBILE before _MTD
> - improve the commit description
>
> include/dt-bindings/leds/common.h | 1 +
> 1 file changed, 1 insertion(+)
>
Reviewed-by: Rob Herring <[email protected]>
On Sat, 23 Mar 2024 16:36:08 +0900, INAGAKI Hiroshi wrote:
> This patch series adds some LED_FUNCTION_* definitions mainly for router
> devices.
> Those definitions are useful for OpenWrt or something.
>
> v1 -> v2
>
> - fix sort order of LED_FUNCTION_MOBILE
> - improve the commit description of the first commit
>
> [...]
Applied, thanks!
[1/2] dt-bindings: leds: add LED_FUNCTION_MOBILE for mobile network
commit: c332f0450f33c123a538a8fcc69fa8e4e5aedfbb
[2/2] dt-bindings: leds: add LED_FUNCTION_SPEED_* for link speed on LAN/WAN
commit: 2588a5d98532244a7faf4514f79dc684fddf4c14
--
Lee Jones [李琼斯]
On Sat, 23 Mar 2024 16:36:08 +0900, INAGAKI Hiroshi wrote:
> This patch series adds some LED_FUNCTION_* definitions mainly for router
> devices.
> Those definitions are useful for OpenWrt or something.
>
> v1 -> v2
>
> - fix sort order of LED_FUNCTION_MOBILE
> - improve the commit description of the first commit
>
> [...]
Applied, thanks!
[1/2] dt-bindings: leds: add LED_FUNCTION_MOBILE for mobile network
commit: b65a10938791d90c88ef4f3ecddee22b9fc23b2e
[2/2] dt-bindings: leds: add LED_FUNCTION_SPEED_* for link speed on LAN/WAN
commit: 03075af4c95133dfa14f434dab3b7b97e6cb9b56
--
Lee Jones [李琼斯]