Messages in this thread | | | From | Josua Mayer <> | Subject | Re: [PATCH net-next 2/2] net: phy: adin: add support for setting led-, link-status-pin polarity | Date | Sat, 20 Apr 2024 09:10:57 +0000 |
| |
Am 19.04.24 um 17:47 schrieb Andrew Lunn: > On Fri, Apr 19, 2024 at 05:35:18PM +0200, Josua Mayer wrote: >> ADIN1300 supports software control over pin polarity for both LED_0 and >> LINK_ST pins. >> >> Configure the polarity during probe based on device-tree properties. >> >> Led polarity is only set if specified in device-tree, otherwise the phy >> can choose either active-low or active-high based on external line >> voltage. Link-status polarity is set to active-high as default if not >> specified, which is always the reset-default. >> >> Signed-off-by: Josua Mayer <josua@solid-run.com> > Hi Josua > > Please take a look at: > > commit 447b80a9330ef2d9a94fc5a9bf35b6eac061f38b > Author: Alexander Stein <alexander.stein@ew.tq-group.com> > Date: Wed Jan 31 08:50:48 2024 +0100 > > net: phy: dp83867: Add support for active-low LEDs > > Add the led_polarity_set callback for setting LED polarity. > > Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com> > Reviewed-by: Andrew Lunn <andrew@lunn.ch> > Signed-off-by: David S. Miller <davem@davemloft.net> > > > Andrew > > ---
Hi Andrew,
That looks very much related!
I was already planning to investigate adding led support ... .
1. for the LINK_ST pin I believe we still need a non-led-framework device property for setting polarity, as it is a fixed function signal that we can't even turn on or off from software.
2. LED_0 control not currently supported by adin driver. The phy supports what data-sheet calls extended configuration (disabled by default) for controlling led state (on, off, patterns).
Since it is not default, I see the polarity setting separate from leds. However I do believe the led_polarity_set callback is an acceptable solution.
I might prepare a reduced v2 for only the fixed-function link-status pin.
sincerely Josua Mayer
| |