hi Bart,
Thanks for your kindly reply, it's seems that something wrong with the MS 
outlook,
the IXP700 device ID is defined to /linux/pci_ids.h in another patch,I will 
resend the  
complete patch.

BRS,
Henry

-----Original Message-----
From: Bartlomiej Zolnierkiewicz [mailto:[EMAIL PROTECTED] 
Sent: Wednesday, May 16, 2007 6:37 AM
To: gmail_ati
Cc: Henry Su; linux-ide@vger.kernel.org
Subject: Re: [PATCH]add the IDE device ID for ATI SB700


Hi,

On Monday 14 May 2007, gmail_ati wrote:
> From: [EMAIL PROTECTED]
> add the IDE device ids to atiixp_pci_tbl struct in atiixp.c for ATI SB700.
> Signed-off-by: Luugi Marsan <[EMAIL PROTECTED]>

This patch is somehow missing the chunk adding PCI_DEVICE_ID_ATI_IXP700_IDE
define to <linux/pci_ids.h> and thus makes atiixp.c fail to compile.

Please resend the complete patch.

PS your mailer seems to encode patches in some strange way (whitespaces are
mangled and saved email contained MIME encoded text instead of plain-text) so
I had to apply this change by hand.  Not really a problem for this particular
patch ;) but definitively something needing fixing.

Thanks,
Bart

> -----------------------------------------------------
> diff -Nur linux-2.6.21.1.org/drivers/ide/pci/atiixp.c 
> linux-2.6.21.1/drivers/ide/pci/atiixp.c
> --- linux-2.6.21.1.org/drivers/ide/pci/atiixp.c 2007-05-13 04:47:25.000000000 
> +0800
> +++ linux-2.6.21.1/drivers/ide/pci/atiixp.c 2007-05-13 05:14:08.000000000 
> +0800
> @@ -353,6 +353,7 @@
>   { PCI_VENDOR_ID_ATI, PCI_DEVICE_ID_ATI_IXP300_IDE, PCI_ANY_ID, PCI_ANY_ID, 
> 0, 0, 0},
>   { PCI_VENDOR_ID_ATI, PCI_DEVICE_ID_ATI_IXP400_IDE, PCI_ANY_ID, PCI_ANY_ID, 
> 0, 0, 0},
>   { PCI_VENDOR_ID_ATI, PCI_DEVICE_ID_ATI_IXP600_IDE, PCI_ANY_ID, PCI_ANY_ID, 
> 0, 0, 1},
> + { PCI_VENDOR_ID_ATI, PCI_DEVICE_ID_ATI_IXP700_IDE, PCI_ANY_ID, PCI_ANY_ID, 
> 0, 0, 1},
>   { 0, },
>  };
>  MODULE_DEVICE_TABLE(pci, atiixp_pci_tbl);



-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to