Module Name:    src
Committed By:   andvar
Date:           Sun Mar 16 11:26:39 UTC 2025

Modified Files:
        src/sys/dev/pci: viaide.c

Log Message:
viaide(4): add IDE controller identification by the VT8237S ISA bridge.

The controller's PATA interface uses PCI ID 0x0571, which is shared among
many VIA southbridges. The ISA bridge is used to identify capabilities,
including for the VT8237S.

Unknown PATA controllers disable UDMA by default. This change ensures PATA
drives attach at full speed.

Pullups to netbsd-9 and netbsd-10 are needed.


To generate a diff of this commit:
cvs rdiff -u -r1.93 -r1.94 src/sys/dev/pci/viaide.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/dev/pci/viaide.c
diff -u src/sys/dev/pci/viaide.c:1.93 src/sys/dev/pci/viaide.c:1.94
--- src/sys/dev/pci/viaide.c:1.93	Tue Mar 11 16:35:03 2025
+++ src/sys/dev/pci/viaide.c	Sun Mar 16 11:26:39 2025
@@ -1,4 +1,4 @@
-/*	$NetBSD: viaide.c,v 1.93 2025/03/11 16:35:03 andvar Exp $	*/
+/*	$NetBSD: viaide.c,v 1.94 2025/03/16 11:26:39 andvar Exp $	*/
 
 /*
  * Copyright (c) 1999, 2000, 2001 Manuel Bouyer.
@@ -26,7 +26,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: viaide.c,v 1.93 2025/03/11 16:35:03 andvar Exp $");
+__KERNEL_RCSID(0, "$NetBSD: viaide.c,v 1.94 2025/03/16 11:26:39 andvar Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -556,6 +556,10 @@ via_chip_map(struct pciide_softc *sc, co
 				aprint_normal("VT8237A ATA133 controller\n");
 				sc->sc_wdcdev.sc_atac.atac_udma_cap = 6;
 				break;
+			case PCI_PRODUCT_VIATECH_VT8237S_ISA:
+				aprint_normal("VT8237S ATA133 controller\n");
+				sc->sc_wdcdev.sc_atac.atac_udma_cap = 6;
+				break;
 			case PCI_PRODUCT_VIATECH_CX700:
 				aprint_normal("CX700 ATA133 controller\n");
 				sc->sc_wdcdev.sc_atac.atac_udma_cap = 6;

Reply via email to