Re: [PATCH] libata: PIIX3 support

2007-01-19 Thread Jeff Garzik
Alan wrote: This I believe completes the PIIX range of support for libata This adds the table entries needed for the PIIX3, both a new PCI identifier and a new mode list. It also fixes an erroneous access to PCI configuration 0x48 on non UDMA capable chips. Signed-off-by: Alan Cox <[EMAIL

Re: [PATCH] libata: PIIX3 support

2007-01-19 Thread Jeff Garzik
Alan wrote: This I believe completes the PIIX range of support for libata This adds the table entries needed for the PIIX3, both a new PCI identifier and a new mode list. It also fixes an erroneous access to PCI configuration 0x48 on non UDMA capable chips. Signed-off-by: Alan Cox [EMAIL

Re: [PATCH] libata: PIIX3 support

2007-01-13 Thread Robert Hancock
Mikael Pettersson wrote: On Wed, 10 Jan 2007 17:13:38 +, Alan <[EMAIL PROTECTED]> wrote: This I believe completes the PIIX range of support for libata This adds the table entries needed for the PIIX3, both a new PCI identifier and a new mode list. It also fixes an erroneous access to PCI

Re: [PATCH] libata: PIIX3 support

2007-01-13 Thread Robert Hancock
Mikael Pettersson wrote: On Wed, 10 Jan 2007 17:13:38 +, Alan [EMAIL PROTECTED] wrote: This I believe completes the PIIX range of support for libata This adds the table entries needed for the PIIX3, both a new PCI identifier and a new mode list. It also fixes an erroneous access to PCI

Re: [PATCH] libata: PIIX3 support

2007-01-11 Thread Alan
On Fri, 12 Jan 2007 02:08:12 +0100 (MET) Mikael Pettersson <[EMAIL PROTECTED]> wrote: > On Wed, 10 Jan 2007 17:13:38 +, Alan <[EMAIL PROTECTED]> wrote: > >This I believe completes the PIIX range of support for libata > > > >This adds the table entries needed for the PIIX3, both a new PCI >

Re: [PATCH] libata: PIIX3 support

2007-01-11 Thread Mikael Pettersson
On Wed, 10 Jan 2007 17:13:38 +, Alan <[EMAIL PROTECTED]> wrote: >This I believe completes the PIIX range of support for libata > >This adds the table entries needed for the PIIX3, both a new PCI >identifier and a new mode list. It also fixes an erroneous access to PCI >configuration 0x48 on

Re: [PATCH] libata: PIIX3 support

2007-01-11 Thread Mikael Pettersson
On Wed, 10 Jan 2007 17:13:38 +, Alan [EMAIL PROTECTED] wrote: This I believe completes the PIIX range of support for libata This adds the table entries needed for the PIIX3, both a new PCI identifier and a new mode list. It also fixes an erroneous access to PCI configuration 0x48 on non UDMA

Re: [PATCH] libata: PIIX3 support

2007-01-11 Thread Alan
On Fri, 12 Jan 2007 02:08:12 +0100 (MET) Mikael Pettersson [EMAIL PROTECTED] wrote: On Wed, 10 Jan 2007 17:13:38 +, Alan [EMAIL PROTECTED] wrote: This I believe completes the PIIX range of support for libata This adds the table entries needed for the PIIX3, both a new PCI identifier and

Re: [PATCH] libata: PIIX3 support

2007-01-10 Thread Jeff Garzik
Alan wrote: @@ -786,7 +797,8 @@ { 2, 3 }, }; pci_read_config_word(dev, master_port, _data); - pci_read_config_byte(dev, 0x48, _enable); + if (ap->udma_mask) + pci_read_config_byte(dev, 0x48, _enable); if (speed >= XFER_UDMA_0) {

[PATCH] libata: PIIX3 support

2007-01-10 Thread Alan
This I believe completes the PIIX range of support for libata This adds the table entries needed for the PIIX3, both a new PCI identifier and a new mode list. It also fixes an erroneous access to PCI configuration 0x48 on non UDMA capable chips. Signed-off-by: Alan Cox <[EMAIL PROTECTED]> ---

[PATCH] libata: PIIX3 support

2007-01-10 Thread Alan
This I believe completes the PIIX range of support for libata This adds the table entries needed for the PIIX3, both a new PCI identifier and a new mode list. It also fixes an erroneous access to PCI configuration 0x48 on non UDMA capable chips. Signed-off-by: Alan Cox [EMAIL PROTECTED] ---

Re: [PATCH] libata: PIIX3 support

2007-01-10 Thread Jeff Garzik
Alan wrote: @@ -786,7 +797,8 @@ { 2, 3 }, }; pci_read_config_word(dev, master_port, master_data); - pci_read_config_byte(dev, 0x48, udma_enable); + if (ap-udma_mask) + pci_read_config_byte(dev, 0x48, udma_enable); if (speed =