* [PATCH] net: phy: broadcom: Configure LEDs on BCM54210E
@ 2024-01-01 21:07 Marek Vasut
2024-03-28 15:09 ` Tom Rini
0 siblings, 1 reply; 3+ messages in thread
From: Marek Vasut @ 2024-01-01 21:07 UTC (permalink / raw)
To: u-boot
Cc: Marek Vasut, Joe Hershberger, Rafał Miłecki,
Ramon Fried, Rasmus Villemoes
Configure LEDs on BCM54210E so they would blink on activity
and indicate link speed. Without this the LEDs are always on
if cable is plugged in.
Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Joe Hershberger <joe.hershberger@ni.com>
Cc: Rafał Miłecki <rafal@milecki.pl>
Cc: Ramon Fried <rfried.dev@gmail.com>
Cc: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
---
drivers/net/phy/broadcom.c | 17 ++++++++++++++++-
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/drivers/net/phy/broadcom.c b/drivers/net/phy/broadcom.c
index 82e3bbef7dd..ecccb7c3b54 100644
--- a/drivers/net/phy/broadcom.c
+++ b/drivers/net/phy/broadcom.c
@@ -42,6 +42,12 @@
#define BCM54810_SHD_CLK_CTL 0x3
#define BCM54810_SHD_CLK_CTL_GTXCLK_EN BIT(9)
+#define BCM54XX_SHD_LEDS1 0x0d
+#define BCM_LED_SRC_LINKSPD2 0x1
+#define BCM_LED_SRC_ACTIVITYLED 0x3
+#define BCM54XX_SHD_LEDS1_LED3(src) (((src) & 0xf) << 4)
+#define BCM54XX_SHD_LEDS1_LED1(src) (((src) & 0xf) << 0)
+
static int bcm54xx_auxctl_read(struct phy_device *phydev, u16 regnum)
{
/* The register must be written to both the Shadow Register Select and
@@ -148,7 +154,16 @@ static int bcm54210e_config(struct phy_device *phydev)
if (ret < 0)
return ret;
- return bcm5461_config(phydev);
+ ret = bcm5461_config(phydev);
+ if (ret < 0)
+ return ret;
+
+ /* Configure LEDs to blink. */
+ bcm_phy_write_shadow(phydev, BCM54XX_SHD_LEDS1,
+ BCM54XX_SHD_LEDS1_LED1(BCM_LED_SRC_ACTIVITYLED) |
+ BCM54XX_SHD_LEDS1_LED3(BCM_LED_SRC_LINKSPD2));
+
+ return 0;
}
static int bcm54xx_parse_status(struct phy_device *phydev)
--
2.43.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] net: phy: broadcom: Configure LEDs on BCM54210E
2024-01-01 21:07 [PATCH] net: phy: broadcom: Configure LEDs on BCM54210E Marek Vasut
@ 2024-03-28 15:09 ` Tom Rini
2024-04-02 8:11 ` Michael Walle
0 siblings, 1 reply; 3+ messages in thread
From: Tom Rini @ 2024-03-28 15:09 UTC (permalink / raw)
To: Marek Vasut
Cc: u-boot, Joe Hershberger, Rafał Miłecki, Ramon Fried,
Rasmus Villemoes
[-- Attachment #1: Type: text/plain, Size: 315 bytes --]
On Mon, Jan 01, 2024 at 10:07:47PM +0100, Marek Vasut wrote:
> Configure LEDs on BCM54210E so they would blink on activity
> and indicate link speed. Without this the LEDs are always on
> if cable is plugged in.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
Applied to u-boot/next, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] net: phy: broadcom: Configure LEDs on BCM54210E
2024-03-28 15:09 ` Tom Rini
@ 2024-04-02 8:11 ` Michael Walle
0 siblings, 0 replies; 3+ messages in thread
From: Michael Walle @ 2024-04-02 8:11 UTC (permalink / raw)
To: Tom Rini, Marek Vasut
Cc: u-boot, Joe Hershberger, Rafał Miłecki, Ramon Fried,
Rasmus Villemoes
[-- Attachment #1: Type: text/plain, Size: 666 bytes --]
On Thu Mar 28, 2024 at 4:09 PM CET, Tom Rini wrote:
> On Mon, Jan 01, 2024 at 10:07:47PM +0100, Marek Vasut wrote:
>
> > Configure LEDs on BCM54210E so they would blink on activity
> > and indicate link speed. Without this the LEDs are always on
> > if cable is plugged in.
> >
> > Signed-off-by: Marek Vasut <marex@denx.de>
>
> Applied to u-boot/next, thanks!
Pretty late and I'm not implying this should be reverted. I just
want to point out, that this is really board dependent and might
even break boards which have this PHY and are using its default
configuration for the attached LEDs.
FWIW, linux now have LED PHY DT bindings.
-michael
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 297 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-04-02 8:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-01 21:07 [PATCH] net: phy: broadcom: Configure LEDs on BCM54210E Marek Vasut
2024-03-28 15:09 ` Tom Rini
2024-04-02 8:11 ` Michael Walle
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.