2024-02-26 07:48:48

by Randy Dunlap

[permalink] [raw]
Subject: [PATCH] net: ethernet: adi: move PHYLIB from vendor to driver symbol

In a previous patch I added "select PHYLIB" at the wrong place for the
ADIN1110 driver symbol, so move it to its correct place under the
ADIN1110 kconfig symbol.

Fixes: a9f80df4f514 ("net: ethernet: adi: requires PHYLIB support")
Signed-off-by: Randy Dunlap <[email protected]>
Reported-by: Michal Kubecek <[email protected]>
Closes: https://lore.kernel.org/lkml/[email protected]/T/#m8ba397484738711edc0ad607b2c63ca02244e3c3
Cc: Lennart Franzen <[email protected]>
Cc: "David S. Miller" <[email protected]>
Cc: Eric Dumazet <[email protected]>
Cc: Jakub Kicinski <[email protected]>
Cc: Paolo Abeni <[email protected]>
Cc: [email protected]
Cc: Nuno Sa <[email protected]>
---
Thanks, Michal.

drivers/net/ethernet/adi/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff -- a/drivers/net/ethernet/adi/Kconfig b/drivers/net/ethernet/adi/Kconfig
--- a/drivers/net/ethernet/adi/Kconfig
+++ b/drivers/net/ethernet/adi/Kconfig
@@ -7,7 +7,6 @@ config NET_VENDOR_ADI
bool "Analog Devices devices"
default y
depends on SPI
- select PHYLIB
help
If you have a network (Ethernet) card belonging to this class, say Y.

@@ -22,6 +21,7 @@ config ADIN1110
tristate "Analog Devices ADIN1110 MAC-PHY"
depends on SPI && NET_SWITCHDEV
select CRC8
+ select PHYLIB
help
Say yes here to build support for Analog Devices ADIN1110
Low Power 10BASE-T1L Ethernet MAC-PHY.


2024-02-26 15:29:41

by Michal Kubecek

[permalink] [raw]
Subject: Re: [PATCH] net: ethernet: adi: move PHYLIB from vendor to driver symbol

On Sun, Feb 25, 2024 at 11:48:20PM -0800, Randy Dunlap wrote:
> In a previous patch I added "select PHYLIB" at the wrong place for the
> ADIN1110 driver symbol, so move it to its correct place under the
> ADIN1110 kconfig symbol.
>
> Fixes: a9f80df4f514 ("net: ethernet: adi: requires PHYLIB support")
> Signed-off-by: Randy Dunlap <[email protected]>
> Reported-by: Michal Kubecek <[email protected]>
> Closes: https://lore.kernel.org/lkml/[email protected]/T/#m8ba397484738711edc0ad607b2c63ca02244e3c3
> Cc: Lennart Franzen <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Eric Dumazet <[email protected]>
> Cc: Jakub Kicinski <[email protected]>
> Cc: Paolo Abeni <[email protected]>
> Cc: [email protected]
> Cc: Nuno Sa <[email protected]>

Thank you for the quick update, now the dependencies work as expected.

Tested-by: Michal Kubecek <[email protected]>


Attachments:
(No filename) (952.00 B)
signature.asc (499.00 B)
Download all attachments

2024-02-28 11:22:18

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH] net: ethernet: adi: move PHYLIB from vendor to driver symbol

Hello:

This patch was applied to netdev/net.git (main)
by David S. Miller <[email protected]>:

On Sun, 25 Feb 2024 23:48:20 -0800 you wrote:
> In a previous patch I added "select PHYLIB" at the wrong place for the
> ADIN1110 driver symbol, so move it to its correct place under the
> ADIN1110 kconfig symbol.
>
> Fixes: a9f80df4f514 ("net: ethernet: adi: requires PHYLIB support")
> Signed-off-by: Randy Dunlap <[email protected]>
> Reported-by: Michal Kubecek <[email protected]>
> Closes: https://lore.kernel.org/lkml/[email protected]/T/#m8ba397484738711edc0ad607b2c63ca02244e3c3
> Cc: Lennart Franzen <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Eric Dumazet <[email protected]>
> Cc: Jakub Kicinski <[email protected]>
> Cc: Paolo Abeni <[email protected]>
> Cc: [email protected]
> Cc: Nuno Sa <[email protected]>
>
> [...]

Here is the summary with links:
- net: ethernet: adi: move PHYLIB from vendor to driver symbol
https://git.kernel.org/netdev/net/c/943d4bd67950

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