Re: [patch] SB700 contains more than one IDE channel
On Thursday 27 September 2007, Shane Huang wrote: > From: [EMAIL PROTECTED] > > SB700 supports one physical IDE channel, but SB700 SATA controller > supports combined mode. When the SATA combined mode is enabled, > two SATA ports(port4 and port5) share one IDE channel from IDE > controller, and PATA will share the other IDE channel. > > Our previous patch adding SB700 IDE device ID only support one > IDE channel, which contains bug. The attached patch fix the bug. > > > Signed-off-by: [EMAIL PROTECTED] applied The inlined patch was word-wrapped and the attached one lacked patch description so manual fixing was required anyway. Not a big problem but next time please try to use some known-to-be-good mail client to inline patches. Thanks, Bart - 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
RE: [patch] SB700 contains more than one IDE channel
Hi Sergei: Thanks for your reply. > The inlined patch has suffered from word wrapping, OK, I think the cause is my mail client MS outlook's setting. I try to change my outlook settings and attach the patch again at the end of this mail. Please check it again. > and the attached one > luckily didn't; yet the attachment has a wrong MIME type -- patches should be > text/plain. As to this case, I don't know why. The patch was generated with diff on linux, and copied to my windows mail machine to be sent. All my previous patches with the same method do NOT meet such problem. So if the patch in this mail does not work either, would you please help to change and apply it for me? So that we can save some time, otherwise I'll have to try to use gmail to send it at home. Thanks Best Regards Shane Signed-off-by: [EMAIL PROTECTED] -- diff -ruN linux-2.6.23-rc5_orig/drivers/ide/pci/atiixp.c linux-2.6.23-rc5/drivers/ide/pci/atiixp.c --- linux-2.6.23-rc5_orig/drivers/ide/pci/atiixp.c 2007-09-01 14:08:24.0 +0800 +++ linux-2.6.23-rc5/drivers/ide/pci/atiixp.c 2007-09-28 00:35:55.0 +0800 @@ -325,7 +325,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}, + { PCI_VENDOR_ID_ATI, PCI_DEVICE_ID_ATI_IXP700_IDE, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, { 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
Re: [patch] SB700 contains more than one IDE channel
Hello. Shane Huang wrote: SB700 supports one physical IDE channel, but SB700 SATA controller supports combined mode. When the SATA combined mode is enabled, two SATA ports(port4 and port5) share one IDE channel from IDE controller, and PATA will share the other IDE channel. Our previous patch adding SB700 IDE device ID only support one IDE channel, which contains bug. The attached patch fix the bug. Signed-off-by: [EMAIL PROTECTED] Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> -- diff -ruN linux-2.6.23-rc5_orig/drivers/ide/pci/atiixp.c linux-2.6.23-rc5/drivers/ide/pci/atiixp.c --- linux-2.6.23-rc5_orig/drivers/ide/pci/atiixp.c 2007-09-01 14:08:24.0 +0800 +++ linux-2.6.23-rc5/drivers/ide/pci/atiixp.c 2007-09-28 00:35:55.0 +0800 @@ -325,7 +325,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}, + { PCI_VENDOR_ID_ATI, PCI_DEVICE_ID_ATI_IXP700_IDE, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, { 0, }, }; MODULE_DEVICE_TABLE(pci, atiixp_pci_tbl); The inlined patch has suffered from word wrapping, and the attached one luckily didn't; yet the attachment has a wrong MIME type -- patches should be text/plain. MBR, Sergei - 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