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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 7 Feb 2022 09:25:42 +0000 From: Joel Stanley <joel@....id.au> To: Cai Huoqing <cai.huoqing@...ux.dev> Cc: "David S. Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>, Karol Gugala <kgugala@...micro.com>, Mateusz Holenko <mholenko@...micro.com>, Gabriel Somlo <gsomlo@...il.com>, Cai Huoqing <caihuoqing@...du.com>, Networking <netdev@...r.kernel.org>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org> Subject: Re: [PATCH] net: ethernet: litex: Add the dependency on HAS_IOMEM On Mon, 7 Feb 2022 at 08:49, Cai Huoqing <cai.huoqing@...ux.dev> wrote: > > The helper function devm_platform_ioremap_resource_xxx() > needs HAS_IOMEM enabled, so add the dependency on HAS_IOMEM. > > Fixes: 464a57281f29 ("net/mlxbf_gige: Make use of devm_platform_ioremap_resourcexxx()") That looks wrong... $ git show --oneline --stat 464a57281f29 464a57281f29 net/mlxbf_gige: Make use of devm_platform_ioremap_resourcexxx() drivers/net/ethernet/litex/litex_liteeth.c | 7 ++----- drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c | 21 +++------------------ drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_mdio.c | 7 +------ drivers/net/ethernet/ni/nixge.c That's a strange commit message for the litex driver. Similarly for the ni driver. Did something go wrong there? A better fixes line would be ee7da21ac4c3be1f618b6358e0a38739a5d1773e, as the original driver addition also has the iomem dependency. > Signed-off-by: Cai Huoqing <cai.huoqing@...ux.dev> > --- > drivers/net/ethernet/litex/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/litex/Kconfig b/drivers/net/ethernet/litex/Kconfig > index f99adbf26ab4..04345b929d8e 100644 > --- a/drivers/net/ethernet/litex/Kconfig > +++ b/drivers/net/ethernet/litex/Kconfig > @@ -17,7 +17,7 @@ if NET_VENDOR_LITEX > > config LITEX_LITEETH > tristate "LiteX Ethernet support" > - depends on OF > + depends on OF && HAS_IOMEM > help > If you wish to compile a kernel for hardware with a LiteX LiteEth > device then you should answer Y to this. > -- > 2.25.1 >
Powered by blists - more mailing lists