CVS commit: [netbsd-10] src/sys/dev/pci

["Martin Husemann" <martin%netbsd.org@localhost>]

Module Name:    src
Committed By:   martin
Date:           Sat Jun  7 15:52:24 UTC 2025

Modified Files:
        src/sys/dev/pci [netbsd-10]: viaide.c

Log Message:
Pull up following revision(s) (requested by andvar in ticket #1128):

        sys/dev/pci/viaide.c: revision 1.101
        sys/dev/pci/viaide.c: revision 1.102

viaide(4): use via_chip_map() to attach VT8261 SATA controller in RAID mode.

Similarly to VT8251 and CX700, VT8261 has two ports per channel, while
via_sata_chip_map supports only one drive per channel. This caused disks on
secondary ports (2, 4) to not be detected, which is addressed by this change.
viaide(4): explicitly enable channels for VT6410 add-in cards.

VT6410 and VT6415 have different reasons for "missing" enable bits.

VT6410 uses the standard 0x40 offset for channel enable bits, but some add-in
cards have both channels disabled by default.  This requires explicitly
writing the enable bits; otherwise, devices will not be detected.  The same
bug is present in Linux and OpenBSD, where the bits are also simulated.

It was reported in Linux but closed as no-fix due to an unidentified root cause.

VT6415 returns 0 at offsets 0x40-0x43 and likely uses a different location
for its single channel enable bit, or it may not expose it at all.  Until
the correct offset is identified (if ever), the driver must simulate the
enable bits.


To generate a diff of this commit:
cvs rdiff -u -r1.89.28.1 -r1.89.28.2 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.