It is added to svwks_csb_check -

@@ -109,6 +114,7 @@ static u8 svwks_csb_check (struct pci_de
                case PCI_DEVICE_ID_SERVERWORKS_CSB5IDE:
                case PCI_DEVICE_ID_SERVERWORKS_CSB6IDE:
                case PCI_DEVICE_ID_SERVERWORKS_CSB6IDE2:
+               case PCI_DEVICE_ID_SERVERWORKS_HT1000IDE:
                        return 1;
                default:
                        break; 


Thanks

Naren Sankar
System Architect
Chipset Product Line
Broadcom Corporation
2451 Mission College Blvd
Santa Clara, CA 95054
Office:  (408) 922-7143
Email:  [EMAIL PROTECTED] 

-----Original Message-----
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] On Behalf Of Bartlomiej
Zolnierkiewicz
Sent: Wednesday, August 17, 2005 12:14 PM
To: Naren (Narendra) Sankar
Cc: [email protected]
Subject: Re: [PATCH 2.6.13-rc6 ] (updated) serverworks: Add support for new
southbridge IDE

On 8/17/05, Naren (Narendra) Sankar <[EMAIL PROTECTED]> wrote:
> Hi Everyone

Hi,

> Description -
> 
> BCM5785 (HT1000) is a Opteron Southbridge from Serverworks/Broadcom 
> that Incorporates a single channel ATA100 IDE controller that is 
> functionally Identical to the Serverworks CSB6 IDE controller. This 
> patch adds support For the new PCI device ID and also the support for this
controller.

Patch looks fine but don't you need to also add HT1000 ID to
swvks_csb_check()?

Bartlomiej
-
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


-
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