From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yuval Shaia Subject: Re: [PATCH] IB: Move PCI dependency from root KConfig to HW's KConfigs Date: Mon, 26 Jun 2017 13:38:52 +0300 Message-ID: <20170626103850.GA6506@yuvallap> References: <20170626085313.6052-1-yuval.shaia@oracle.com> <20170626100231.GC1248@mtr-leonro.local> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20170626100231.GC1248-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Leon Romanovsky Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, swise-ut6Up61K2wZBDgjK7y7TUQ@public.gmane.org, mike.marciniszyn-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, dennis.dalessandro-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, faisal.latif-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, shiraz.saleem-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, rajesh.borundia-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org, Ram.Amrani-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org, linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org On Mon, Jun 26, 2017 at 01:02:31PM +0300, Leon Romanovsky wrote: > On Mon, Jun 26, 2017 at 11:53:13AM +0300, Yuval Shaia wrote: > > No reason to have dependency on PCI for the entire infiniband stack so > > move it to KConfig of only the drivers that actually using PCI. > > > > Signed-off-by: Yuval Shaia > > --- > > drivers/infiniband/Kconfig | 1 - > > drivers/infiniband/hw/cxgb3/Kconfig | 1 + > > drivers/infiniband/hw/cxgb4/Kconfig | 1 + > > drivers/infiniband/hw/hfi1/Kconfig | 1 + > > drivers/infiniband/hw/i40iw/Kconfig | 1 + > > drivers/infiniband/hw/qedr/Kconfig | 1 + > > drivers/infiniband/hw/qib/Kconfig | 1 + > > drivers/infiniband/sw/rdmavt/Kconfig | 1 + > > Why do you need PCI for rdmavt? Saw calls such as get_pci_dev so just suspect, but you are right, i don't see why SW device needs PCI. > > > drivers/infiniband/ulp/ipoib/Kconfig | 1 + > > 9 files changed, 8 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/infiniband/Kconfig b/drivers/infiniband/Kconfig > > index 234fe01..568088a 100644 > > --- a/drivers/infiniband/Kconfig > > +++ b/drivers/infiniband/Kconfig > > @@ -1,6 +1,5 @@ > > menuconfig INFINIBAND > > tristate "InfiniBand support" > > - depends on PCI || BROKEN > > By the way, do you know why are we dependent on BROKEN? No, assume that it is like if PCI is not defined the "break" configuration or something like that. Will be happy if you can shad some light on that. > > Thanks -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html