On Wed, Nov 01 2006, Jeff Garzik wrote:
> Andrew Morton wrote:
> >On Wed, 01 Nov 2006 14:30:35 +0000
> >Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> wrote:
> >
> >>Ar Mer, 2006-11-01 am 09:06 -0500, ysgrifennodd John Stoffel:
> >>>Jeff> + { 0x8086, 0x7110, PCI_ANY_ID, PCI_ANY_ID, 0, 0, piix_pata_33
> >>>},
> >>>Jeff> { 0x8086, 0x7111, PCI_ANY_ID, PCI_ANY_ID, 0, 0, piix_pata_33
> >>>},
> >>>
> >>>Umm, according to lspci -nn on my 440GX box, isn't the 0x8086/0x7110
> >>>an ISA bridge, not a PIIX? controller?
> >>Correct - the 7110 doesn't belong on that list.
> >
> >So should it be moved elsewhere, or simply removed?
>
> Well, according to Jens' own comment message, the PCI ID he needed was
> already in the driver (my eyes didn't catch this).
>
> It looks like it should be reverted, based on this thread and also the
> patch's commit message itself.
I think so, I must have botched the lspci -> lspci -n read. I'll retest
and see what went wrong on this notebook, but I can definitely ack that
0x7110 is an isa bridge here as well.
--
Jens Axboe
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
