[RESEND][PATCH] sata_via: Add SATA support for vt8237a

[RESEND][PATCH] sata_via: Add SATA support for vt8237a

[Jay Cliburn]

The initial submittal of this patch was erroneously sent to the wrong
list (linux-scsi), and I neglected to cc the maintainer.  Never
received an ACK.

This patch adds support for the VIA Technologies VT8237A SATA controller.

Signed-off-by: Jay Cliburn <jacliburn@bellsouth.net>

---
diff --git a/drivers/scsi/sata_via.c b/drivers/scsi/sata_via.c
index 03baec2..d939fc8 100644
--- a/drivers/scsi/sata_via.c
+++ b/drivers/scsi/sata_via.c
@@ -76,6 +76,7 @@ static u32 svia_scr_read (struct ata_por
 static void svia_scr_write (struct ata_port *ap, unsigned int sc_reg, u32 val);
 
 static const struct pci_device_id svia_pci_tbl[] = {
+	{ 0x1106, 0x0591, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6420 },
 	{ 0x1106, 0x3149, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6420 },
 	{ 0x1106, 0x3249, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6421 },
 

Re: [RESEND][PATCH] sata_via: Add SATA support for vt8237a

[Martin Michlmayr]

* Jay Cliburn <jacliburn@bellsouth.net> [2006-08-13 22:10]:
> This patch adds support for the VIA Technologies VT8237A SATA controller.
> 
> Signed-off-by: Jay Cliburn <jacliburn@bellsouth.net>
> 
>  static const struct pci_device_id svia_pci_tbl[] = {
> +	{ 0x1106, 0x0591, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6420 },
>  	{ 0x1106, 0x3149, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6420 },
>  	{ 0x1106, 0x3249, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6421 },

What's the status of this patch?  Someone just reported a bug to
Debian saying that their controller is not supported and that this
patch works.
-- 
Martin Michlmayr
tbm@cyrius.com

Re: [RESEND][PATCH] sata_via: Add SATA support for vt8237a

[Jay Cliburn]

Martin Michlmayr wrote:
> * Jay Cliburn <jacliburn@bellsouth.net> [2006-08-13 22:10]:
>> This patch adds support for the VIA Technologies VT8237A SATA controller.
>>
>> Signed-off-by: Jay Cliburn <jacliburn@bellsouth.net>
>>
>>  static const struct pci_device_id svia_pci_tbl[] = {
>> +	{ 0x1106, 0x0591, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6420 },
>>  	{ 0x1106, 0x3149, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6420 },
>>  	{ 0x1106, 0x3249, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6421 },
> 
> What's the status of this patch?  Someone just reported a bug to
> Debian saying that their controller is not supported and that this
> patch works.

It has not been ACKed, NACKed, or applied.  I can only assume it's queued.

Jay

Re: [RESEND][PATCH] sata_via: Add SATA support for vt8237a

[Jay Cliburn]

Jay Cliburn wrote:
> The initial submittal of this patch was erroneously sent to the wrong
> list (linux-scsi), and I neglected to cc the maintainer.  Never
> received an ACK.
> 
> This patch adds support for the VIA Technologies VT8237A SATA controller.
> 
> Signed-off-by: Jay Cliburn <jacliburn@bellsouth.net>
> 
> ---
> diff --git a/drivers/scsi/sata_via.c b/drivers/scsi/sata_via.c
> index 03baec2..d939fc8 100644
> --- a/drivers/scsi/sata_via.c
> +++ b/drivers/scsi/sata_via.c
> @@ -76,6 +76,7 @@ static u32 svia_scr_read (struct ata_por
>  static void svia_scr_write (struct ata_port *ap, unsigned int sc_reg, u32 val);
>  
>  static const struct pci_device_id svia_pci_tbl[] = {
> +	{ 0x1106, 0x0591, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6420 },
>  	{ 0x1106, 0x3149, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6420 },
>  	{ 0x1106, 0x3249, PCI_ANY_ID, PCI_ANY_ID, 0, 0, vt6421 },
>  
> -

Please apply this patch.  It's trivial and it works.  I own a
motherboard that contains the VT8237A SATA controller, and I've tested
the patch with Fedora Core 5, Fedora Core 6 test 2, and current Fedora
Core rawhide (2.6.18rc4-git1).  I'm receiving personal requests for
assistance in getting kernels to recognize and boot from the 8237A, and
it would really help if the patch were applied.

If there's something wrong with the patch, or if you'd rather receive it
from a more trusted source, just say so and I'll comply.  Any feedback
at all would be greatly appreciated.

Best regards,
Jay