NetBSD-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

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



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

From: "Andrius Varanavicius" <andvar%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/45917 CVS commit: src/sys/dev/pci
Date: Thu, 3 Apr 2025 15:35:54 +0000

 Module Name:	src
 Committed By:	andvar
 Date:		Thu Apr  3 15:35:54 UTC 2025
 
 Modified Files:
 	src/sys/dev/pci: pcidevs pciide_apollo_reg.h viaide.c
 
 Log Message:
 viaide(4): Add support for VIA VT6415/VT6330 single-channel IDE controllers.
 
 Beyond adding the usual identification code, several adjustments were required:
 
 1) Introduced 'single_channel` to set 'nchannels` to 1 for this controller.
 
 2) Added 'APO_IDECONF_ALWAYS_EN` definition and the 'no_ideconf` flag to mimic
    enabled channel bits, as this controller's enable chip register is unset.
    Applied the same to VT6410, following OpenBSD and Linux, since some
    controllers are known to not set this register as well.
 
 3) Combined all VIA controllers identified by PCI ID (not ISA bus) using switch
    fallthrough, as they currently share the same UDMA setting. Consequently,
    the VX900 name printing was moved to the device description and adjusted
    to better reflect the device.
 
 4) Moved setting interface bits for RAID controllers under the RAID capability
    check, enabling the above fallthrough and paving the way for support of other
    RAID-capable controllers in upcoming changes.
 
 5) The VT6330 is a combo FireWire/IDE controller. Added its FireWire controller
    PCI ID to pcidevs.
 
 Tested on ASRock P5B-DE.
 
 Reviewed by jak and bad.
 
 Addresses PR kern/45917
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.1519 -r1.1520 src/sys/dev/pci/pcidevs
 cvs rdiff -u -r1.20 -r1.21 src/sys/dev/pci/pciide_apollo_reg.h
 cvs rdiff -u -r1.94 -r1.95 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.
 


Home | Main Index | Thread Index | Old Index