* James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> [2008-12-30 17:12:04]: > On Tue, 2008-12-30 at 08:54 -0800, Randy Dunlap wrote: > > James Bottomley wrote: > > > diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig > > > index 0e5e084..9f4cc8e 100644 > > > --- a/drivers/scsi/Kconfig > > > +++ b/drivers/scsi/Kconfig > > > @@ -607,13 +607,12 @@ config SCSI_FLASHPOINT > > > > > > config LIBFC > > > tristate "LibFC module" > > > - depends on SCSI && SCSI_FC_ATTRS > > > + select SCSI_FC_ATTRS > > > ---help--- > > > Fibre Channel library module > > > > > > config FCOE > > > tristate "FCoE module" > > > - depends on SCSI > > > select LIBFC > > > ---help--- > > > Fibre Channel over Ethernet module > > > > > > > > > > This still has a build error when CONFIG_PCI=n: > > > > > > linux-next-20081229/drivers/scsi/fcoe/fc_transport_fcoe.c: In function 'fcoe_load_transport_driver': > > linux-next-20081229/drivers/scsi/fcoe/fc_transport_fcoe.c:324: error: 'pci_bus_type' undeclared (first use in this function) > > make[4]: *** [drivers/scsi/fcoe/fc_transport_fcoe.o] Error 1 > > OK, so take two would make FCOE depend on SCSI, like this. > > James > Hi James, Thanks, the patch fixes same build failure, with !CONFIG_PCI, which I reported some time back today. > --- > > From: James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> > Subject: [SCSI] fcoe: fix configuration problems > > fcoe selects libfc and requires SCSI and PCI (the SCSI requirement is > implicitly covered by an enclosing if). Fix them both up so they > cannot be configured in an invalid state: make LIBFC select > SCSI_FC_ATTRS and make FCOE depend on PCI and select LIBFC. > Tested-by: Kamalesh Babulal <kamalesh@xxxxxxxxxxxxxxxxxx> > Reported-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx> > Signed-off-by: James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> > --- > drivers/scsi/Kconfig | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig > index 0e5e084..152d4aa 100644 > --- a/drivers/scsi/Kconfig > +++ b/drivers/scsi/Kconfig > @@ -607,13 +607,13 @@ config SCSI_FLASHPOINT > > config LIBFC > tristate "LibFC module" > - depends on SCSI && SCSI_FC_ATTRS > + select SCSI_FC_ATTRS > ---help--- > Fibre Channel library module > > config FCOE > tristate "FCoE module" > - depends on SCSI > + depends on PCI > select LIBFC > ---help--- > Fibre Channel over Ethernet module > -- > 1.5.6.6 > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-next" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Thanks & Regards, Kamalesh Babulal, Linux Technology Center, IBM, ISTL. -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
