Re: [patch] libata: use correct map_db values for ICH8

[Kristen Carlson Accardi <kristen.c.accardi@intel.com>]

On Fri, 20 Oct 2006 18:06:53 +0900
Tejun Heo <htejun@gmail.com> wrote:

> Hello, Kristen.
> 
> Kristen Carlson Accardi wrote:
> > Use valid values for ICH8 map_db.  With the old values, when the 
> > controller was in Native mode, and SCC was 1 (drives configured for
> > IDE), any drive plugged into a slave port was not recognized.  For
> > Combined Mode (and SCC is still 1), 2 is a value value for MAP.map_value,
> > and needs to be recognized.
> > 
> > Signed-off-by:  Kristen Carlson Accardi <kristen.c.accardi@intel.com>
> 
> Do you guys have doc update related to this?  The doc and spec update 
> still indicate that MAP value is reserved to 00b.  Anyways, if you say 
> that's right...
> 
> Acked-by: Tejun Heo <htejun@gmail.com>
> 
> -- 
> tejun
As far as I know, this has always been documented.
The datasheet is located here:
http://developer.intel.com/design/chipsets/datashts/313056.htm

Indicates that 10b is valid for combined mode.  Make sure you are looking
at device 31 function 2 - for 31 function 5 it is hardwired to 00b, but
for function 2, it can be 00 or 10.  This was not very clear, so it's
easy to understand how this could have been misunderstood.  See 
section 11.1.33 in the notes, or do a search on "combined mode" through
the doc, and you'll see that MV can be 10b when SCC is 01 on device 31
function 2.