[PATCH] libata: fix combined mode
authorAlan <alan@lxorguk.ukuu.org.uk>
Tue, 2 Jan 2007 11:58:34 +0000 (11:58 +0000)
committerLinus Torvalds <torvalds@woody.osdl.org>
Tue, 2 Jan 2007 16:20:51 +0000 (08:20 -0800)
commitdc3c3377f03634d351fafdfe35b237b283586c04
treee97c4a7dfeb45cd965c46cfa146fa426199104e8
parente22a9a8b703d05f13366c3f2e7e1aa0550bb5ca6
[PATCH] libata: fix combined mode

This is a slight variant on the patch I posted December 16th to fix
libata combined mode handling. The only real change is that we now
correctly also reserve BAR1,2,4. That is basically a neatness issue.

Jeff was unhappy about two things

1. That it didn't work in the case of one channel native one channel
legacy.

This is a silly complaint because the SFF layer in libata doesn't handle
this case yet anyway.

2. The case where combined mode is in use and IDE=n.

In this case the libata quirk code reserves the resources in question
correctly already.

Once the combined mode stuff is redone properly (2.6.21) then the entire
mess turns into a single pci_request_regions() for all cases and all the
ugly resource hackery goes away.

I'm sending this now rather than after running full test suites so that
it can get the maximal testing in a short time. I'll be running tests on
this after lunch.

Signed-off-by: Alan Cox <alan@redhat.com>
Cc: Jeff Garzik <jgarzik@pobox.com>
Acked-by: Alessandro Suardi <alessandro.suardi@gmail.com>
Acked-by: Theodore Tso <tytso@mit.edu>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
drivers/ata/libata-sff.c