Re: [parisc-linux] Results of (IDE) kernel tests with modified io.c

[Randolph Chung <randolph@tausq.org>]

> But what I'm wondering about is why the Silicon Image driver
> (always) detects the Seagate ST311022A hard drive as "TS130220A2".
> Since it uses mmio instead of PIO as the other drivers, maybe there is
> some big-endian problem in the Linux/PA-RISC mmio code? But then this
> problem must be somewhere in the IDE code, since all mmio in general
> seems to be working fine!

oh, for mmio you need to fix the macros in include/asm-parisc/ide.h. the
ones in io.c are for pio access. the mmio ones are new, i think, and 
haven't been fixed properly (__ide_mm_insw, etc) -- you can try 
replacing readw with __raw_readw, etc.

can you also try the native-mode fixup? something like this (sorry 
it's not in patch format, my tree is a bit of a mess right now and it's
late....)

in arch/parisc/kernel/pci.c, change:

struct pci_fixup pcibios_fixups[] = {
        { PCI_FIXUP_HEADER, PCI_ANY_ID, PCI_ANY_ID, parisc_fixup_ide },
        { 0 }
};

parisc_fixup_ide is something like this:

static void __devinit parisc_fixup_ide(struct pci_dev *dev)
{
    u8 prog;

    if ((dev->class >> 8) != PCI_CLASS_STORAGE_IDE)
        return;

    pci_read_config_byte(dev, 8, &prog);
    pci_write_config_byte(dev, 8, prog | 0x5);
    dev->class |= 0x5;
}

not tested, YMMV :)

Grant tells me that he suspects there's a bug somewhere in the pci
config accessor functions tho... on my box i saw that, using something
similar to the above sequence, i read 0x8a into prog (this is the 
default), but after the write if i read it again the value is still 
0x8a :(

randolph
-- 
Randolph Chung
Debian GNU/Linux Developer, hppa/ia64 ports
http://www.tausq.org/