linux-kernel - Re: [2.6 patch] PCMCIA mustn't select HAVE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080415221002.GL1677@cs181133002.pp.htv.fi>
Date:	Wed, 16 Apr 2008 01:10:02 +0300
From:	Adrian Bunk <bunk@...nel.org>
To:	Bartlomiej Zolnierkiewicz <bzolnier@...il.com>,
	Sam Ravnborg <sam@...nborg.org>, linux-ide@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-pcmcia@...ts.infradead.org
Subject: Re: [2.6 patch] PCMCIA mustn't select HAVE_IDE

On Tue, Apr 15, 2008 at 11:03:45PM +0100, Russell King wrote:
> On Wed, Apr 16, 2008 at 12:52:23AM +0300, Adrian Bunk wrote:
>...
> So this is a only impacting ARM wrt. PCMCIA, and given that ARM supplies
> an asm/ide.h, it's _entirely_ reasonable that if a platform has PCMCIA
> then it supports IDE.
> 
> > We could simply always select HAVE_IDE on arm instead of manually 
> > setting which platforms could possibly get IDE support (e.g. are there
> > any boards with PCI slots for which HAVE_IDE is currently not 
> > selected?).
> 
> You could, if there was a demand for it.  As no one has added that,
> I conclude that its less common for people to stick an IDE controller
> into a PCI backplane.

People can always enable code for stuff they don't use.

But instead of having 14 ARM platforms plus PCMCIA (which is offered 
unconditionally on all ARM platforms...) select HAVE_IDE it's simpler
to select it once for all ARM platforms.

> In fact, there are only three classes of ARM platforms which have PCI
> selected but not HAVE_IDE - IOP13xx, IXP2000, and Orion.  I suspect
> the only reason they don't select it because they now use the ATA code
> rather than the old IDE code - that's certainly true of Orion.

The libata options are offered unconditionally on all platforms...

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

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