On Tue, 2010-05-04 at 10:30 -0700, Randy Dunlap wrote: > From: Randy Dunlap <randy.dunlap@xxxxxxxxxx> > > bnx2i should depend on NET since it selects SCSI_ISCSI_ATTRS, > which depends on NET. > > Also move the dependencies together. > > The "depends" change fixes multiple build errors when CONFIG_NET is > not enabled: > ERROR: "skb_trim" [drivers/scsi/scsi_transport_iscsi.ko] undefined! > ERROR: "netlink_kernel_create" [drivers/scsi/scsi_transport_iscsi.ko] undefined! > ERROR: "netlink_kernel_release" [drivers/scsi/scsi_transport_iscsi.ko] undefined! > ERROR: "skb_pull" [drivers/scsi/scsi_transport_iscsi.ko] undefined! > ERROR: "init_net" [drivers/scsi/scsi_transport_iscsi.ko] undefined! > ERROR: "__alloc_skb" [drivers/scsi/scsi_transport_iscsi.ko] undefined! > ERROR: "netlink_broadcast" [drivers/scsi/scsi_transport_iscsi.ko] undefined! > ERROR: "kfree_skb" [drivers/scsi/scsi_transport_iscsi.ko] undefined! > ERROR: "skb_put" [drivers/scsi/scsi_transport_iscsi.ko] undefined! > > Signed-off-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx> > Cc: Anil Veerabhadrappa <anilgv@xxxxxxxxxxxx> > --- > drivers/scsi/bnx2i/Kconfig | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > --- lnx-2634-rc6.orig/drivers/scsi/bnx2i/Kconfig > +++ lnx-2634-rc6/drivers/scsi/bnx2i/Kconfig > @@ -1,10 +1,11 @@ > config SCSI_BNX2_ISCSI > tristate "Broadcom NetXtreme II iSCSI support" > + depends on NET > + depends on PCI Actually, let's not fix it this way. While we still have this annoying bug in the Kconfig system where it doesn't respect the dependencies of a select, let's not build traps like this ... the rule has to be selected attributes must select their dependencies, that way the selection just works(tm). Of course, fixing Kconfig would be the better option ... James --- diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig index 75f2336..804ba3d 100644 --- a/drivers/scsi/Kconfig +++ b/drivers/scsi/Kconfig @@ -308,7 +308,8 @@ config SCSI_FC_TGT_ATTRS config SCSI_ISCSI_ATTRS tristate "iSCSI Transport Attributes" - depends on SCSI && NET + depends on SCSI + select NET help If you wish to export transport-specific information about each attached iSCSI device to sysfs, say Y. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
