CVS commit: src/sys/dev/pci

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

Module Name:    src
Committed By:   andvar
Date:           Sun Apr 20 09:44:40 UTC 2025

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

Log Message:
viaide(4): Improve VT8251 integrated SATA controller support in IDE/RAID modes.

Use via_chip_map() to attach this controller in IDE and RAID modes, similarly
to VX800. This allows drives to be identified and attached successfully on all
four ports (two channels with master/slave ports).

Switch channel setup to sata_setup_channel(), replacing via_setup_channel().
This avoids writes to the 0x50 register, which is only relevant for IDE
controllers. Writes to SATA controller registers caused drives on the IDE
controller (0x0571) to fail to attach. Apply this fix also to VX900 and VT8261.

Rename VT8237R_SATA to VT8251_SATA (0x3349 is not used for VT8237R, AFAIK).
This PCI ID is used in RAID mode on newer (CE) chipset revision and in all
modes (IDE/RAID/AHCI) on older revision.
Add the 0x5287 PCI ID, used in IDE mode on newer (CE) chipset revision.
Rename VT8251_SATA to VT8251_AHCI (0x6287 is used only in AHCI mode).
Add VT8251_AHCI to the ahcisata(4) quirk list, same as VT8251_SATA.

For more details, see the tech-kern thread:
https://mail-index.netbsd.org/tech-kern/2025/04/13/msg030365.html

Should fix and close PR kern/37517.

Tested on Asus A8V-VM (CD) and MSI MS-7318 (CE) motherboards.

Reviewed by bad.


To generate a diff of this commit:
cvs rdiff -u -r1.71 -r1.72 src/sys/dev/pci/ahcisata_pci.c
cvs rdiff -u -r1.1521 -r1.1522 src/sys/dev/pci/pcidevs
cvs rdiff -u -r1.96 -r1.97 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.