On Thu, Apr 1, 2021 at 12:27 PM Leon Romanovsky <leon@xxxxxxxxxx> wrote: > > From: Leon Romanovsky <leonro@xxxxxxxxxx> > > The "select" kconfig keyword provides reverse dependency, however it > doesn't check that selected symbol meets its own dependencies. Usually > "select" is used for non-visible symbols, so instead of trying to keep > dependencies in sync with BNXT ethernet driver, simply "depends on" it, > like Kconfig documentation suggest. > > * CONFIG_PCI is already required by BNXT > * CONFIG_NETDEVICES and CONFIG_ETHERNET are needed to chose BNXT > > Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxx> > --- > drivers/infiniband/hw/bnxt_re/Kconfig | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/infiniband/hw/bnxt_re/Kconfig b/drivers/infiniband/hw/bnxt_re/Kconfig > index 0feac5132ce1..6a17f5cdb020 100644 > --- a/drivers/infiniband/hw/bnxt_re/Kconfig > +++ b/drivers/infiniband/hw/bnxt_re/Kconfig > @@ -2,9 +2,7 @@ > config INFINIBAND_BNXT_RE > tristate "Broadcom Netxtreme HCA support" > depends on 64BIT > - depends on ETHERNET && NETDEVICES && PCI && INET && DCB > - select NET_VENDOR_BROADCOM > - select BNXT > + depends on INET && DCB && BNXT > help > This driver supports Broadcom NetXtreme-E 10/25/40/50 gigabit > RoCE HCAs. To compile this driver as a module, choose M here: > -- > 2.30.2 > Acked-By: Devesh Sharma <devesh.sharma@xxxxxxxxxxxx> -- -Regards Devesh
Attachment:
smime.p7s
Description: S/MIME Cryptographic Signature
