Re: Marvell SATA driver

[Thor Lancelot Simon <tls%panix.com@localhost>]

On Sun, Jul 19, 2009 at 11:36:01PM +0900, Izumi Tsutsui wrote:
> 
> Are you saying you choose different method (#ifdef or cf_flags)
> because implementation is different? I don't see the particular point.

I thought his reasoning was pretty clear: the DMA code in mvsata isn't
shared with the other drivers, so he wanted to provide a way to have
this code not be compiled into the kernel while still retaining minimal
driver functionality

I don't generally like this kind of conditional compilation in our
kernel, but this driver in particular is used on pretty small machines
and the space savings could be worth it.

-- 
Thor Lancelot Simon                                        
tls%rek.tjls.com@localhost
    "Even experienced UNIX users occasionally enter rm *.* at the UNIX
     prompt only to realize too late that they have removed the wrong
     segment of the directory structure." - Microsoft WSS whitepaper