Re: [PATCH 1/2] NET: PHY: Add PHY LED control binding.

On 06/09/2016 08:12 AM, John Crispin wrote:

On 09/06/2016 08:06, Alexander Stein wrote:

quoted

On Wednesday 08 June 2016 14:30:08, Rob Herring wrote:

quoted

quoted

diff --git a/Documentation/devicetree/bindings/phy/phy-leds.txt

b/Documentation/devicetree/bindings/phy/phy-leds.txt new file mode 100644
index 0000000..1a35e3d

--- /dev/null
+++ b/Documentation/devicetree/bindings/phy/phy-leds.txt

@@ -0,0 +1,52 @@
+LED configuration for Ethernet phys
+
+All these properties are optional, not all properties are supported by
+all PHYs. When more then one property name is define for one LED the
+order they get applied is device depended.
+Property names:
+	led-const-on: conditions the LED should be constant on
+	led-pulse: condition the LED should be pulsed on
+	led-blink-slow: condition the LED should slowly blink

How slow is slow?

This depends on the MMD.INTERNAL.LEDCH.SBF setting which is 2 Hz by default.

quoted

quoted

+	led-blink-fast: condition the LED should fast blink

How fast is fast?

This depends on the MMD.INTERNAL.LEDCH.FBF setting which is 16 Hz by default.

Both can be set independently to 2, 4, 8 or 16 Hz.

and both are intel/lantiq implementation specific and hence should not
be part of a generic led-phy binding.

Ok, I can remove them, I think the constant on and the pulse are used by
many Ethernet PHYs.

imho these leds should be exposed via a led driver and the configurtion
should be exposed via a led driver specific trigger, in the same manner
in which wireless macs do it.

Where is a good example on how this is done?
Is this then also triggered by the hardware or does the software has to
trigger it?

Hauke

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html