2024-05-29 14:16:33

by Rengarajan S

[permalink] [raw]
Subject: [PATCH net-next v3 0/2] lan78xx: Enable 125 MHz CLK and Auto Speed configuration for LAN7801 if NO EEPROM is detected

This patch series adds the support for 125 MHz clock, Auto speed and
auto duplex configuration for LAN7801 in the absence of EEPROM.

Rengarajan S (2):
lan78xx: Enable 125 MHz CLK configuration for LAN7801 if NO EEPROM is
detected
lan78xx: Enable Auto Speed and Auto Duplex configuration for LAN7801
if NO EEPROM is detected

v3
Resubmitting the patch. No changes.
v2
Split the patches into 125 MHz clock support and Auto speed config
support for LAN7801.
v1
Initial Commit.

drivers/net/usb/lan78xx.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)

--
2.25.1



2024-05-29 14:17:01

by Rengarajan S

[permalink] [raw]
Subject: [PATCH net-next v3 2/2] lan78xx: Enable Auto Speed and Auto Duplex configuration for LAN7801 if NO EEPROM is detected

Enabled ASD/ADD configuration for LAN7801 in the absence of EEPROM.
After the lite reset these contents go back to defaults where ASD/
ADD is disabled. The check is already available for LAN7800.

Reviewed-by: Simon Horman <[email protected]>
Signed-off-by: Rengarajan S <[email protected]>
---
drivers/net/usb/lan78xx.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/lan78xx.c
index 7ac540cc3686..62dbfff8dad4 100644
--- a/drivers/net/usb/lan78xx.c
+++ b/drivers/net/usb/lan78xx.c
@@ -3034,8 +3034,11 @@ static int lan78xx_reset(struct lan78xx_net *dev)
return ret;

/* LAN7801 only has RGMII mode */
- if (dev->chipid == ID_REV_CHIP_ID_7801_)
+ if (dev->chipid == ID_REV_CHIP_ID_7801_) {
buf &= ~MAC_CR_GMII_EN_;
+ /* Enable Auto Duplex and Auto speed */
+ buf |= MAC_CR_AUTO_DUPLEX_ | MAC_CR_AUTO_SPEED_;
+ }

if (dev->chipid == ID_REV_CHIP_ID_7800_ ||
dev->chipid == ID_REV_CHIP_ID_7850_) {
--
2.25.1


2024-06-01 23:25:46

by Jakub Kicinski

[permalink] [raw]
Subject: Re: [PATCH net-next v3 0/2] lan78xx: Enable 125 MHz CLK and Auto Speed configuration for LAN7801 if NO EEPROM is detected

On Wed, 29 May 2024 19:32:54 +0530 Rengarajan S wrote:
> This patch series adds the support for 125 MHz clock, Auto speed and
> auto duplex configuration for LAN7801 in the absence of EEPROM.

I had to look up why this is not a fix.

When someone asks you a question, please edit the commit message /
cover letter so that the answer is obvious for the next revision...

2024-06-01 23:30:46

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH net-next v3 0/2] lan78xx: Enable 125 MHz CLK and Auto Speed configuration for LAN7801 if NO EEPROM is detected

Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <[email protected]>:

On Wed, 29 May 2024 19:32:54 +0530 you wrote:
> This patch series adds the support for 125 MHz clock, Auto speed and
> auto duplex configuration for LAN7801 in the absence of EEPROM.
>
> Rengarajan S (2):
> lan78xx: Enable 125 MHz CLK configuration for LAN7801 if NO EEPROM is
> detected
> lan78xx: Enable Auto Speed and Auto Duplex configuration for LAN7801
> if NO EEPROM is detected
>
> [...]

Here is the summary with links:
- [net-next,v3,1/2] lan78xx: Enable 125 MHz CLK configuration for LAN7801 if NO EEPROM is detected
https://git.kernel.org/netdev/net-next/c/5160b129f65f
- [net-next,v3,2/2] lan78xx: Enable Auto Speed and Auto Duplex configuration for LAN7801 if NO EEPROM is detected
https://git.kernel.org/netdev/net-next/c/799f532de136

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html