On 7/19/05, Adrian Bunk <[email protected]> wrote:
> [ The subject was adapted to linux-kernel spam filters... ]
>
> On Sat, Jul 16, 2005 at 07:26:44PM +0200, Jindrich Makovicka wrote:
> > Andrew Vasquez wrote:
> > > Yes, quite. How about the following to correct the intention.
> > >
> > >
> > >
> > > Add correct Kconfig option for ISP24xx support.
> > >
> > > Signed-off-by: Andrew Vasquez <[email protected]>
> > > ---
> > >
> > > diff --git a/drivers/scsi/qla2xxx/Kconfig b/drivers/scsi/qla2xxx/Kconfig
> > > --- a/drivers/scsi/qla2xxx/Kconfig
> > > +++ b/drivers/scsi/qla2xxx/Kconfig
> > > @@ -39,3 +39,11 @@ config SCSI_QLA6312
> > > ---help---
> > > This driver supports the QLogic 63xx (ISP6312 and ISP6322) host
> > > adapter family.
> > > +
> > > +config SCSI_QLA24XX
> > > + tristate "QLogic ISP24xx host adapter family support"
> > > + depends on SCSI_QLA2XXX
> > > + select SCSI_FC_ATTRS
> >
> > there should be also "select FW_LOADER", as it uses request_firmware &
> > release_firmware
> >...
>
> You are right, patch below.
>
> > Jindrich Makovicka
>
> cu
> Adrian
>
>
> <-- snip -->
>
>
> qla_init.c now uses code that requires FW_LOADER.
>
> Additionally, this patch removes spaces instead of tabs at the
> SCSI_FC_ATTRS selects.
>
> Signed-off-by: Adrian Bunk <[email protected]>
>
> --- linux-2.6.13-rc3-mm1-full/drivers/scsi/qla2xxx/Kconfig.old 2005-07-17 15:44:26.000000000 +0200
> +++ linux-2.6.13-rc3-mm1-full/drivers/scsi/qla2xxx/Kconfig 2005-07-17 15:45:45.000000000 +0200
> @@ -1,49 +1,55 @@
> config SCSI_QLA2XXX
> tristate
> depends on SCSI && PCI
> default y
>
> config SCSI_QLA21XX
> tristate "QLogic ISP2100 host adapter family support"
> depends on SCSI_QLA2XXX
> - select SCSI_FC_ATTRS
> + select SCSI_FC_ATTRS
> + select FW_LOADER
> ---help---
> This driver supports the QLogic 21xx (ISP2100) host adapter family.
>
> config SCSI_QLA22XX
> tristate "QLogic ISP2200 host adapter family support"
> depends on SCSI_QLA2XXX
> - select SCSI_FC_ATTRS
> + select SCSI_FC_ATTRS
> + select FW_LOADER
> ---help---
> This driver supports the QLogic 22xx (ISP2200) host adapter family.
>
> config SCSI_QLA2300
> tristate "QLogic ISP2300 host adapter family support"
> depends on SCSI_QLA2XXX
> - select SCSI_FC_ATTRS
> + select SCSI_FC_ATTRS
> + select FW_LOADER
> ---help---
> This driver supports the QLogic 2300 (ISP2300 and ISP2312) host
> adapter family.
>
> config SCSI_QLA2322
> tristate "QLogic ISP2322 host adapter family support"
> depends on SCSI_QLA2XXX
> - select SCSI_FC_ATTRS
> + select SCSI_FC_ATTRS
> + select FW_LOADER
> ---help---
> This driver supports the QLogic 2322 (ISP2322) host adapter family.
>
> config SCSI_QLA6312
> tristate "QLogic ISP63xx host adapter family support"
> depends on SCSI_QLA2XXX
> - select SCSI_FC_ATTRS
> + select SCSI_FC_ATTRS
> + select FW_LOADER
> ---help---
> This driver supports the QLogic 63xx (ISP6312 and ISP6322) host
> adapter family.
>
> config SCSI_QLA24XX
> tristate "QLogic ISP24xx host adapter family support"
> depends on SCSI_QLA2XXX
> - select SCSI_FC_ATTRS
> + select SCSI_FC_ATTRS
> + select FW_LOADER
> ---help---
> This driver supports the QLogic 24xx (ISP2422 and ISP2432) host
> adapter family.
I send a patch for this yesterday that lets SCSI_QLA2XXX select
FW_LOADER. I believe that's a bit better since the other options
depend on SCSI_QLA2XXX anyway, there's no point in having them all set
FW_LOADER. My patch also fixes another little issue; that you cannot
disable SCSI_QLA2XXX if you don't need it.
See the patch here: http://lkml.org/lkml/2005/7/19/147
The mail contains 3 patches, but the third one is the best fix IMHO.
--
Jesper Juhl <[email protected]>
Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please http://www.expita.com/nomime.html
-
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]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
| |
 |