Re: Linux 2.4.8-ac10

From: Andrzej Krzysztofowicz (ankry@green.mif.pg.gda.pl)
Date: Fri Aug 24 2001 - 12:14:39 EST


>
> > Linux 2.4.8-ac10
> >
> > From: Alan Cox (laughing@shared-source.org)
> > o Put config hooks in to make qlogicfc firmware (me)
> > optionally loadable for weird hardware
> > | Needs a suitable firmware file adding ..
>
> shouldn't the question about the FW only be asked if the FC driver is
> actually requested? I have not requested the driver, but get asked (make
> oldconfig) about the FW.

Somebodyd did a typo:

   if [ "%CONFIG_SCSI_QLOGIC_FC" != "n" ]; then
        ^^^
Also indentation of question texts should step by 2, not 3.

BTW, wouldn't it better to use deb_mbool here ?
Patch for all three fixes follows.

Andrzej
*******************************************
--- drivers/scsi/Config.in.old Fri Aug 24 19:03:51 2001
+++ drivers/scsi/Config.in Fri Aug 24 19:08:05 2001
@@ -153,9 +153,7 @@
 if [ "$CONFIG_PCI" = "y" ]; then
    dep_tristate 'Qlogic ISP SCSI support' CONFIG_SCSI_QLOGIC_ISP $CONFIG_SCSI
    dep_tristate 'Qlogic ISP FC SCSI support' CONFIG_SCSI_QLOGIC_FC $CONFIG_SCSI
- if [ "%CONFIG_SCSI_QLOGIC_FC" != "n" ]; then
- bool ' Include loadable firmware in driver' CONFIG_SCSI_QLOGIC_FC_FIRMWARE
- fi
+ dep_mbool ' Include loadable firmware in driver' CONFIG_SCSI_QLOGIC_FC_FIRMWARE $CONFIG_SCSI_QLOGIC_FC
    dep_tristate 'Qlogic QLA 1280 SCSI support' CONFIG_SCSI_QLOGIC_1280 $CONFIG_SCSI
 fi
 if [ "$CONFIG_X86" = "y" ]; then

-- 
=======================================================================
  Andrzej M. Krzysztofowicz               ankry@mif.pg.gda.pl
  phone (48)(58) 347 14 61
Faculty of Applied Phys. & Math.,   Technical University of Gdansk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Fri Aug 31 2001 - 21:00:11 EST