linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 2.6.13] drivers/ide: Enable basic VIA VT6410 IDE functionality
@ 2005-09-21 21:38 Robert Kesterson
  2005-09-22 14:28 ` Alan Cox
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Robert Kesterson @ 2005-09-21 21:38 UTC (permalink / raw)
  To: Bartlomiej Zolnierkiewicz; +Cc: linux-kernel, linux-ide

This patch enables plain IDE functionality on the VIA VT6410 IDE controller
(such as used on the Asus P4P800 Deluxe motherboard).  This is not a RAID
driver, but you can use Linux's software RAID once the drives are visible.
I did not write the original version of this patch, which I found on the
internet back in the days of kernel 2.6.2.  I have been unable to identify
the original author to give him/her proper credit.  I have been maintaining
and updating this patch since then and have released several updates which
have been successfully used by others who have downloaded it from my
website.

It seems appropriate that this minimal functionality should be in the
mainstream kernel.

Signed-off-by: Robert Kesterson <robertk@robertk.com>

diff -rup linux-2.6.13.2.orig/drivers/ide/pci/generic.c linux-2.6.13.2/drivers/ide/pci/generic.c
--- linux-2.6.13.2.orig/drivers/ide/pci/generic.c	2005-09-16 20:02:12.000000000 -0500
+++ linux-2.6.13.2/drivers/ide/pci/generic.c	2005-09-21 16:27:38.000000000 -0500
@@ -179,6 +179,12 @@ static ide_pci_device_t generic_chipsets
  		.channels	= 2,
  		.autodma	= AUTODMA,
  		.bootable	= OFF_BOARD,
+	},{	/* 15 */
+		.name		= "VIA_610",
+		.init_hwif	= init_hwif_generic,
+		.channels	= 2,
+		.autodma	= AUTODMA,
+		.bootable	= ON_BOARD,
  	}
  };

@@ -238,6 +244,7 @@ static struct pci_device_id generic_pci_
  	{ PCI_VENDOR_ID_TOSHIBA,PCI_DEVICE_ID_TOSHIBA_PICCOLO_1,   PCI_ANY_ID, PCI_ANY_ID, 0, 0, 12},
  	{ PCI_VENDOR_ID_TOSHIBA,PCI_DEVICE_ID_TOSHIBA_PICCOLO_2,   PCI_ANY_ID, PCI_ANY_ID, 0, 0, 13},
  	{ PCI_VENDOR_ID_NETCELL,PCI_DEVICE_ID_REVOLUTION,          PCI_ANY_ID, PCI_ANY_ID, 0, 0, 14},
+	{ PCI_VENDOR_ID_VIA,    PCI_DEVICE_ID_VIA_610,             PCI_ANY_ID, PCI_ANY_ID, 0, 0, 15},
  	/* Must come last. If you add entries adjust this table appropriately and the init_one code */
  	{ PCI_ANY_ID,		PCI_ANY_ID,			   PCI_ANY_ID, PCI_ANY_ID, PCI_CLASS_STORAGE_IDE << 8, 0xFFFFFF00UL, 0},
  	{ 0, },
diff -rup linux-2.6.13.2.orig/include/linux/pci_ids.h linux-2.6.13.2/include/linux/pci_ids.h
--- linux-2.6.13.2.orig/include/linux/pci_ids.h	2005-09-16 20:02:12.000000000 -0500
+++ linux-2.6.13.2/include/linux/pci_ids.h	2005-09-21 16:27:38.000000000 -0500
@@ -1413,6 +1413,7 @@
  #define PCI_DEVICE_ID_VIA_8703_51_0	0x3148
  #define PCI_DEVICE_ID_VIA_8237_SATA	0x3149
  #define PCI_DEVICE_ID_VIA_XN266		0x3156
+#define PCI_DEVICE_ID_VIA_610		0x3164
  #define PCI_DEVICE_ID_VIA_8754C_0	0x3168
  #define PCI_DEVICE_ID_VIA_8235		0x3177
  #define PCI_DEVICE_ID_VIA_P4N333	0x3178

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2.6.13] drivers/ide: Enable basic VIA VT6410 IDE functionality
  2005-09-21 21:38 [PATCH 2.6.13] drivers/ide: Enable basic VIA VT6410 IDE functionality Robert Kesterson
@ 2005-09-22 14:28 ` Alan Cox
  2005-09-23 15:40   ` Bartlomiej Zolnierkiewicz
  2005-09-22 18:41 ` Bill Davidsen
  2005-10-03 18:41 ` Bill Davidsen
  2 siblings, 1 reply; 5+ messages in thread
From: Alan Cox @ 2005-09-22 14:28 UTC (permalink / raw)
  To: Robert Kesterson; +Cc: Bartlomiej Zolnierkiewicz, linux-kernel, linux-ide

On Mer, 2005-09-21 at 16:38 -0500, Robert Kesterson wrote:
> This patch enables plain IDE functionality on the VIA VT6410 IDE controller
> (such as used on the Asus P4P800 Deluxe motherboard). 

Mathias Kretschmer posted a patch several months ago, and Daniel Drake
reposted it on August 30th if you check the archive. Bartlomiej was
asked to respond to/merge it but if he responded I didn't see it so it
was probably a private reply.

You can pull it from the Gentoo tree and it'll drop into other vendor
trees cleanly.

Alan


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2.6.13] drivers/ide: Enable basic VIA VT6410 IDE functionality
  2005-09-21 21:38 [PATCH 2.6.13] drivers/ide: Enable basic VIA VT6410 IDE functionality Robert Kesterson
  2005-09-22 14:28 ` Alan Cox
@ 2005-09-22 18:41 ` Bill Davidsen
  2005-10-03 18:41 ` Bill Davidsen
  2 siblings, 0 replies; 5+ messages in thread
From: Bill Davidsen @ 2005-09-22 18:41 UTC (permalink / raw)
  To: Robert Kesterson; +Cc: linux-kernel, linux-ide

Robert Kesterson wrote:
> This patch enables plain IDE functionality on the VIA VT6410 IDE controller
> (such as used on the Asus P4P800 Deluxe motherboard).  This is not a RAID
> driver, but you can use Linux's software RAID once the drives are visible.
> I did not write the original version of this patch, which I found on the
> internet back in the days of kernel 2.6.2.  I have been unable to identify
> the original author to give him/her proper credit.  I have been maintaining
> and updating this patch since then and have released several updates which
> have been successfully used by others who have downloaded it from my
> website.
> 
> It seems appropriate that this minimal functionality should be in the
> mainstream kernel.

Thank you, I have been running with a similar patch for 2.4 series, but 
have been running unpatched kernels for testing and have not had access 
to my other drives. Hopefully this can get into the mainstream without 
too much delay.

-- 
    -bill davidsen (davidsen@tmr.com)
"The secret to procrastination is to put things off until the
  last possible moment - but no longer"  -me

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2.6.13] drivers/ide: Enable basic VIA VT6410 IDE functionality
  2005-09-22 14:28 ` Alan Cox
@ 2005-09-23 15:40   ` Bartlomiej Zolnierkiewicz
  0 siblings, 0 replies; 5+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2005-09-23 15:40 UTC (permalink / raw)
  To: Alan Cox; +Cc: Robert Kesterson, linux-kernel, linux-ide

On 9/22/05, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
> On Mer, 2005-09-21 at 16:38 -0500, Robert Kesterson wrote:
> > This patch enables plain IDE functionality on the VIA VT6410 IDE controller
> > (such as used on the Asus P4P800 Deluxe motherboard).
>
> Mathias Kretschmer posted a patch several months ago, and Daniel Drake
> reposted it on August 30th if you check the archive. Bartlomiej was

No, this patch is an earlier reincarnation
(it changes generic.c instead of via82cxxx.c).

> asked to respond to/merge it but if he responded I didn't see it so it
> was probably a private reply.

Bartlomiej responded the same day (cc:ing linux-kernel and linux-ide)
and asked Daniel to fix via82cxxx.c to support multiple controllers.

Daniel sent me patch 2 days later but I still need to go through it
(I've been away for 3 weeks).

Bartlomiej

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2.6.13] drivers/ide: Enable basic VIA VT6410 IDE functionality
  2005-09-21 21:38 [PATCH 2.6.13] drivers/ide: Enable basic VIA VT6410 IDE functionality Robert Kesterson
  2005-09-22 14:28 ` Alan Cox
  2005-09-22 18:41 ` Bill Davidsen
@ 2005-10-03 18:41 ` Bill Davidsen
  2 siblings, 0 replies; 5+ messages in thread
From: Bill Davidsen @ 2005-10-03 18:41 UTC (permalink / raw)
  To: Robert Kesterson; +Cc: linux-kernel, linux-ide

Robert Kesterson wrote:
> This patch enables plain IDE functionality on the VIA VT6410 IDE controller
> (such as used on the Asus P4P800 Deluxe motherboard).  This is not a RAID
> driver, but you can use Linux's software RAID once the drives are visible.
> I did not write the original version of this patch, which I found on the
> internet back in the days of kernel 2.6.2.  I have been unable to identify
> the original author to give him/her proper credit.  I have been maintaining
> and updating this patch since then and have released several updates which
> have been successfully used by others who have downloaded it from my
> website.
> 
> It seems appropriate that this minimal functionality should be in the
> mainstream kernel.

After using this patch (it works), it comes to me that the 2.4 version 
was in a separate driver, so I can control my order of controller 
recognition. Just having access to the devices is useful, but it would 
be more useful in the module version.
-- 
    -bill davidsen (davidsen@tmr.com)
"The secret to procrastination is to put things off until the
  last possible moment - but no longer"  -me

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2005-10-03 18:40 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-09-21 21:38 [PATCH 2.6.13] drivers/ide: Enable basic VIA VT6410 IDE functionality Robert Kesterson
2005-09-22 14:28 ` Alan Cox
2005-09-23 15:40   ` Bartlomiej Zolnierkiewicz
2005-09-22 18:41 ` Bill Davidsen
2005-10-03 18:41 ` Bill Davidsen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).