PR/59010 CVS commit: src/sys/dev/pci

["Andrius Varanavicius" <andvar%netbsd.org@localhost>]

The following reply was made to PR kern/59010; it has been noted by GNATS.

From: "Andrius Varanavicius" <andvar%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/59010 CVS commit: src/sys/dev/pci
Date: Mon, 3 Mar 2025 22:22:37 +0000

 Module Name:	src
 Committed By:	andvar
 Date:		Mon Mar  3 22:22:37 UTC 2025
 
 Modified Files:
 	src/sys/dev/pci: viaide.c
 
 Log Message:
 viaide(4): use via_chip_map() instead of via_sata_chip_map_new() to attach
 CX700/VX800 IDE/SATA RAID controllers and set interface flags the same way as
 the VT6410_RAID case for native interrupts required by RAID mode.
 
 Device descriptions were moved to pciide_via_products and updated to cascade
 VT6410_RAID and CX700_IDE options.
 
 via_sata_chip_map_new previously worked more by "accident," as it did not
 return early on failure to map SATA registers and defaulted to the generic
 drive probe function. However, it failed to attach PATA drives (endless
 timeouts) and incorrectly detected "three" channels as VT6421, despite the
 controller having only two.
 
 Fixes PR kern/59010.
 
 Pull-ups to netbsd-9, 10 are needed.
 
 Reviewed and approved by jakllsch@.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.91 -r1.92 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.