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
| ||
|
Message-ID: <a0353e85-8604-a268-5776-2f28b092e57b@linaro.org> Date: Fri, 21 Oct 2022 09:38:23 -0400 From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org> To: Maxime Chevallier <maxime.chevallier@...tlin.com>, davem@...emloft.net, Rob Herring <robh+dt@...nel.org> Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org, devicetree@...r.kernel.org, thomas.petazzoni@...tlin.com, Andrew Lunn <andrew@...n.ch>, Florian Fainelli <f.fainelli@...il.com>, Heiner Kallweit <hkallweit1@...il.com>, Russell King <linux@...linux.org.uk>, linux-arm-kernel@...ts.infradead.org, Vladimir Oltean <vladimir.oltean@....com>, Luka Perkov <luka.perkov@...tura.hr>, Robert Marko <robert.marko@...tura.hr> Subject: Re: [PATCH net-next v5 2/5] net: ipqess: introduce the Qualcomm IPQESS driver On 21/10/2022 08:45, Maxime Chevallier wrote: > The Qualcomm IPQESS controller is a simple 1G Ethernet controller found > on the IPQ4019 chip. This controller has some specificities, in that the > IPQ4019 platform that includes that controller also has an internal > switch, based on the QCA8K IP. Thank you for your patch. There is something to discuss/improve. > diff --git a/MAINTAINERS b/MAINTAINERS > index 5c6ce094e55e..46fdd3c523c5 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -17037,6 +17037,12 @@ L: netdev@...r.kernel.org > S: Maintained > F: drivers/net/ethernet/qualcomm/emac/ > > +QUALCOMM IPQESS ETHERNET DRIVER > +M: Maxime Chevallier <maxime.chevallier@...tlin.com> > +L: netdev@...r.kernel.org > +S: Maintained > +F: drivers/net/ethernet/qualcomm/ipqess/ Add also bindings. > + > QUALCOMM ETHQOS ETHERNET DRIVER > M: Vinod Koul <vkoul@...nel.org> > R: Bhupesh Sharma <bhupesh.sharma@...aro.org> > diff --git a/drivers/net/ethernet/qualcomm/Kconfig b/drivers/net/ethernet/qualcomm/Kconfig > index a4434eb38950..a723ddbea248 100644 > --- a/drivers/net/ethernet/qualcomm/Kconfig > +++ b/drivers/net/ethernet/qualcomm/Kconfig > @@ -60,6 +60,17 @@ config QCOM_EMAC > low power, Receive-Side Scaling (RSS), and IEEE 1588-2008 > Precision Clock Synchronization Protocol. > > +config QCOM_IPQ4019_ESS_EDMA > + tristate "Qualcomm Atheros IPQ4019 ESS EDMA support" > + depends on OF I think this is present only on systems where Qualcomm IPQ4019 SoCs is the main SoC (AP)? If so, I propose not to offer this to non-Qualcomm SoC builds, because they cannot use it and it makes life of distro-vendors configurators difficult: depends on ARCH_QCOM || COMPILE_TEST > + select PHYLINK > + help > + This driver supports the Qualcomm Atheros IPQ40xx built-in > + ESS EDMA ethernet controller. > + > + To compile this driver as a module, choose M here: the > + module will be called ipqess. > + > source "drivers/net/ethernet/qualcomm/rmnet/Kconfig" Best regards, Krzysztof
Powered by blists - more mailing lists