Re: [PATCH] Drop ISA dependencies from IRDA drivers

From: Andi Kleen
Date: Thu Jul 15 2004 - 15:54:15 EST


On Thu, Jul 15, 2004 at 12:48:07PM -0400, Jeff Garzik wrote:
> Andi Kleen wrote:
> >http://bugme.osdl.org/show_bug.cgi?id=3077
> >
> >Some IRDA chipsets currently don't work on x86-64, because
> >they're dependent on CONFIG_ISA and x86-64 doesn't set this.
> >CONFIG_ISA means real ISA slots, and I doubt these chips
> >come on real ISA cards, so I just removed the bogus
> >dependency.
>
> Honestly, the issue and patch need more thought, IMO.
>
> Regardless of theory, CONFIG_ISA is currently also used to indicate
> legacy ISA devices that are today integrated into southbridges.

I don't think so. I did most of the original CONFIG_ISA annotations
and I only added it to real ISA devices.

And the LPC devices in southbridges are normally not marked
CONFIG_ISA.

>
> And with legacy ISA devices still around, I don't see a whole lot of
> value in differentiating between "I have ISA slots" and "I have ISA
> devices".

There is great value. Basically most ISA drivers are not 64bit
clean (if they even still work on i386 which is also often doubtful
in 2.6) and it is a great way for 64bit archs to get rid of a lot
of not working code.

-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/