Re: + revert-iscsi-build-failure-use-depends-instead-of.patch added to -mm tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> From: Randy Dunlap <[email protected]>
> 
> Remove all (2) "select NET" instances from all kernel Kconfig files.  Use
> "depends on" for them instead.  As Roman Zippel commented, "please don't
> select NET, it's way too broad."
> 
> This affects all IEEE1394 drivers and some SCSI drivers.  Networking (NET)
> will have to be enabled before any IEEE1394 or a few SCSI drivers can be
> enabled.  In particular for SCSI, the QLogic ISP4xxx driver
> (SCSI_QLA_ISCSI) and any FiberChannel drivers are affected.
> 
> Signed-off-by: Randy Dunlap <[email protected]>
> Cc: James Bottomley <[email protected]>
> Cc: Stefan Richter <[email protected]>
> Cc: Roman Zippel <[email protected]>
> Signed-off-by: Andrew Morton <[email protected]>
> ---
> 
>  drivers/ieee1394/Kconfig     |    5 ++++-
>  drivers/scsi/Kconfig         |   13 +++++++++----
>  drivers/scsi/qla4xxx/Kconfig |    8 ++++----
>  3 files changed, 17 insertions(+), 9 deletions(-)
[...]

I'm OK with that change. You could have submitted the IEEE 1394 part
separately though.

A comment on the patch description: Only the eth1394 driver and the
ieee1394 core driver depend directly on NET. (All other FireWire drivers
depend on ieee1394 of course.) ieee1394's dependency on NET is planned
to be removed by a minor change to its implementation but we don't have
a deadline for this, as usual...
-- 
Stefan Richter
-=====-=-==- =-== ---=-
http://arcgraph.de/sr/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux