Skip to contentSkip to conversation

Combined mode on Dell Inspiron 6000 - patch for 2.6.14-rc5 - ata_piix.c

DISCUSSION Werner Backes 2 participants 2 messages Oct 21, 2005 libata

lore
Share & tools
lore
#0 lore raw

Hi,

I'm no kernel expert at all, but since the combined mode for 2.6.13.x was working and for 2.6.14-rcx not, I looked at the differences in the code. I changed the following line in ata_piix.c. The original code is commented out. Since you only switched the values of both defines the line from 2.6.13.x seemed to make sense....
Without this change the drive is found, but I cannot use it. No mount, no burning, ...
I hope this will help somehow ...

Werner Backes

static int piix_sata_probe (struct ata_port *ap)
{
....

/* TODO: this is vaguely wrong for ICH6 combined mode,
* where only two of the four SATA ports are mapped
* onto a single ATA channel. It is also vaguely inaccurate
* for ICH5, which has only two ports. However, this is ok,
* as further device presence detection code will handle
* any false positives produced here.
*/

for (i = 0; i < 4; i++) {
mask = (PIIX_PORT_PRESENT << i) | (PIIX_PORT_ENABLED << i);
/* mask = (PIIX_PORT_ENABLED << i); */

                if ((orig_mask & mask) == mask)
                        if (combined || (i == ap->hard_port_no))
                                return 1;
        }
#1 lore raw
Quoted 4 lines · Werner Backes

Hi,

I'm no kernel expert at all, but since the combined mode for 2.6.13.x was working and for 2.6.14-rcx not, I looked at the differences in the code. I changed the following line in ata_piix.c. The original code is commented out. Since you only switched the values of both defines the line from 2.6.13.x seemed to make sense.... Without this change the drive is found, but I cannot use it. No mount, no burning, ... I hope this will help somehow ...

Have you tried 2.6.14-rc5?

It contains a fix for combined mode.

Jeff