| Message ID | 20220314110254.12498-1-anders.roxell@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 231fdac3e58f4e52e387930c73bf535439607563 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [PATCHv2] net: phy: Kconfig: micrel_phy: fix dependency issue | expand |
On Mon, Mar 14, 2022 at 12:02:54PM +0100, Anders Roxell wrote: > When building driver CONFIG_MICREL_PHY the follow error shows up: > > aarch64-linux-gnu-ld: drivers/net/phy/micrel.o: in function `lan8814_ts_info': > micrel.c:(.text+0x1764): undefined reference to `ptp_clock_index' > micrel.c:(.text+0x1764): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `ptp_clock_index' > aarch64-linux-gnu-ld: drivers/net/phy/micrel.o: in function `lan8814_probe': > micrel.c:(.text+0x4720): undefined reference to `ptp_clock_register' > micrel.c:(.text+0x4720): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `ptp_clock_register' > > Rework Kconfig for MICREL_PHY to depend on 'PTP_1588_CLOCK_OPTIONAL'. > Arnd describes in a good way why its needed to add this depends in patch > e5f31552674e ("ethernet: fix PTP_1588_CLOCK dependencies"). > > Reported-by: kernel test robot <lkp@intel.com> > Fixes: ece19502834d ("net: phy: micrel: 1588 support for LAN8814 phy") > Signed-off-by: Anders Roxell <anders.roxell@linaro.org> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Andrew
Hello: This patch was applied to netdev/net-next.git (master) by Paolo Abeni <pabeni@redhat.com>: On Mon, 14 Mar 2022 12:02:54 +0100 you wrote: > When building driver CONFIG_MICREL_PHY the follow error shows up: > > aarch64-linux-gnu-ld: drivers/net/phy/micrel.o: in function `lan8814_ts_info': > micrel.c:(.text+0x1764): undefined reference to `ptp_clock_index' > micrel.c:(.text+0x1764): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `ptp_clock_index' > aarch64-linux-gnu-ld: drivers/net/phy/micrel.o: in function `lan8814_probe': > micrel.c:(.text+0x4720): undefined reference to `ptp_clock_register' > micrel.c:(.text+0x4720): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `ptp_clock_register' > > [...] Here is the summary with links: - [PATCHv2] net: phy: Kconfig: micrel_phy: fix dependency issue https://git.kernel.org/netdev/net-next/c/231fdac3e58f You are awesome, thank you!
diff --git a/drivers/net/phy/Kconfig b/drivers/net/phy/Kconfig index 902495afcb38..ea7571a2b39b 100644 --- a/drivers/net/phy/Kconfig +++ b/drivers/net/phy/Kconfig @@ -220,6 +220,7 @@ config MEDIATEK_GE_PHY config MICREL_PHY tristate "Micrel PHYs" + depends on PTP_1588_CLOCK_OPTIONAL help Supports the KSZ9021, VSC8201, KS8001 PHYs.
When building driver CONFIG_MICREL_PHY the follow error shows up: aarch64-linux-gnu-ld: drivers/net/phy/micrel.o: in function `lan8814_ts_info': micrel.c:(.text+0x1764): undefined reference to `ptp_clock_index' micrel.c:(.text+0x1764): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `ptp_clock_index' aarch64-linux-gnu-ld: drivers/net/phy/micrel.o: in function `lan8814_probe': micrel.c:(.text+0x4720): undefined reference to `ptp_clock_register' micrel.c:(.text+0x4720): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `ptp_clock_register' Rework Kconfig for MICREL_PHY to depend on 'PTP_1588_CLOCK_OPTIONAL'. Arnd describes in a good way why its needed to add this depends in patch e5f31552674e ("ethernet: fix PTP_1588_CLOCK dependencies"). Reported-by: kernel test robot <lkp@intel.com> Fixes: ece19502834d ("net: phy: micrel: 1588 support for LAN8814 phy") Signed-off-by: Anders Roxell <anders.roxell@linaro.org> --- drivers/net/phy/Kconfig | 1 + 1 file changed, 1 insertion(+)