lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 16 Mar 2021 16:47:29 +0100 From: Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com> To: Dmitry Osipenko <digetx@...il.com> Cc: Thierry Reding <thierry.reding@...il.com>, Jonathan Hunter <jonathanh@...dia.com>, linux-tegra@...r.kernel.org, Felipe Balbi <balbi@...nel.org>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org, Krzysztof Kozlowski <krzk@...nel.org>, kernel test robot <lkp@...el.com> Subject: Re: [PATCH v2] phy: tegra: depend on COMMON_CLK to fix compile tests On Tue, 16 Mar 2021 at 16:43, Dmitry Osipenko <digetx@...il.com> wrote: > > 16.03.2021 10:55, Krzysztof Kozlowski пишет: > > From: Krzysztof Kozlowski <krzk@...nel.org> > > > > The Tegra USB PHY driver uses Common Clock Framework thus it cannot be > > built on platforms without it (e.g. compile test on MIPS with RALINK and > > SOC_RT305X): > > > > /usr/bin/mips-linux-gnu-ld: drivers/usb/phy/phy-tegra-usb.o: in function `tegra_usb_phy_init': > > phy-tegra-usb.c:(.text+0x1dd4): undefined reference to `clk_get_parent' > > > > Reported-by: kernel test robot <lkp@...el.com> > > Signed-off-by: Krzysztof Kozlowski <krzk@...nel.org> > > > > --- > > > > Changes since v1: > > 1. Depend on COMMON_CLK always, not only for compile test. > > --- > > drivers/usb/phy/Kconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/usb/phy/Kconfig b/drivers/usb/phy/Kconfig > > index 52eebcb88c1f..7500e77a7d01 100644 > > --- a/drivers/usb/phy/Kconfig > > +++ b/drivers/usb/phy/Kconfig > > @@ -163,6 +163,7 @@ config USB_MXS_PHY > > config USB_TEGRA_PHY > > tristate "NVIDIA Tegra USB PHY Driver" > > depends on ARCH_TEGRA || COMPILE_TEST > > + depends on COMMON_CLK > > select USB_COMMON > > select USB_PHY > > select USB_ULPI > > > > But if COMMON_CLK is disabled, then include/linux/clk.h provides a stub > for clk_get_parent(), meaning that MIPS has its own COMMON_CLK, no? Hi, It depends on the platform. Not all of them implement every clk API, so you can have failures: https://lore.kernel.org/lkml/202102170017.MgPVy7aZ-lkp@intel.com/ Best regards, Krzysztof
Powered by blists - more mailing lists