Re: PATCH & call for help: Marking ISA only drivers

From: Alan Cox (alan@lxorguk.ukuu.org.uk)
Date: Thu May 09 2002 - 15:33:21 EST


> BTW I think CONFIG_ISA would be an useful configuration option for
> i386 too - at least most modern PCs do not come with ISA slots anymore.

ISA slots != ISA devices

And ISA stuff hanging off things like chipset glue busses is everywhere

> +if [ "$CONFIG_ISA" = "y" ]; then
> + dep_tristate 'Adaptec AHA152X/2825 support' CONFIG_SCSI_AHA152X $CONFIG_SCSI

2825 is not ISA bus

> +# does not use pci dma and seems to be isa/onboard only for old machines
> +if [ "$CONFIG_X86_64" != "y" ]; then
> + dep_tristate 'AM53/79C974 PCI SCSI support' CONFIG_SCSI_AM53C974 $CONFIG_SCSI $CONFIG_PCI
> +fi

Thats PCI.

> +if [ "$CONFIG_ISA" = "y" ]; then
> + dep_tristate 'Generic NCR5380/53c400 SCSI support' CONFIG_SCSI_GENERIC_NCR5380 $CONFIG_SCSI
> + if [ "$CONFIG_SCSI_GENERIC_NCR5380" != "n" ]; then

This is used in multiple non ISA situations.

> +if [ "$CONFIG_X86" = "y" -a "$CONFIG_ISA" = "y" ]; then
> dep_tristate 'UltraStor SCSI support' CONFIG_SCSI_ULTRASTOR $CONFIG_SCSI
> fi

It makes an ugly mess of Config.in - I guess 2.5 is the time to do what
Russell and Keith wanted and make undefined imply "N" in dep_*

Alan
-
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 : Tue May 14 2002 - 12:00:12 EST