src/sys/dev/sound/pci, branch mainFreeBSD source treehttps://cgit-dev.freebsd.org/src/atom?h=main2026-06-15T15:03:48Zsound: Create all device nodes with GID_AUDIO2026-06-15T15:03:48ZChristos Margiolischristos@FreeBSD.org2026-05-21T18:03:31Zurn:sha1:a6aa95eba6ee2dd199d6cb8063dbaccd8a6f7512
Commit 6024e3f99a1e ("Add audio group") introduced GID_AUDIO, initially
for virtual_oss(8) loopback devices. Now make all of them with
GID_AUDIO.
Relnotes: yes
Sponsored by: The FreeBSD Foundation
Reviewed by: emaste
Pull-Request: https://ron-dev.freebsd.org/FreeBSD/src/pulls/35
snd_hda: Patch Dell WYSE 70402026-05-26T10:32:56ZChristos Margiolischristos@FreeBSD.org2026-05-25T11:40:05Zurn:sha1:c6cefcbbbb3fd5bbe9e0c36c8c553364d444e2d8
https://forums.freebsd.org/threads/dell-wyse-7040-thin-client-intel-i5-6500te-built-in-sound-alc255-mini-how-to.102656/
Reported by: razif (on the forum)
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
snd_hda: Reassign duplicate HDMI/DP pin sequences instead of disabling2026-05-19T12:25:51ZAbdelkader Boudihfreebsd@seuros.com2026-05-19T12:24:14Zurn:sha1:65341ec3172936804b081c8ceca9bae88f5c7192
Some firmware (e.g. Apple EFI on Sandy Bridge Mac hardware) programs all
HDMI/DP output pins in an association with identical sequence numbers.
The existing code disables the entire association on the first
duplicate, leaving HDMI/DP audio non-functional.
For digital output pins (HDMI/DP) with seq=0 duplicates, search for the
next free sequence slot and reassign the duplicate rather than
disabling.
The seq=0 restriction targets the known Apple firmware pattern; any
other duplicate sequence is more likely a genuine firmware error and the
association is still disabled.
Update first after reassignment so that hpredir is not left pointing at
a stale sequence. Non-digital and input associations retain the existing
disable behaviour.
MFC after: 1 week
Reviewed by: christos
Differential Revision: https://reviews.freebsd.org/D55473
chore: replace {0, 0} with {DEV,KOBJ}METHOD_END2026-02-25T05:59:34ZEnji Cooperngie@FreeBSD.org2026-02-21T06:14:46Zurn:sha1:0fc6c3f731a2cca3120798806c330a3081c9424b
Both of the aforementioned macros have been present in FreeBSD
for well over a decade: 2009 for `KOBJMETHOD_END`; 2011 for
`DEVMETHOD_END`.
Adapt all hardcoded references of `{0, 0}` with `DEVMETHOD_END`
and `KOBJMETHOD_END` as appropriate. This helps ensure that
future adaptations to drivers following patterns documented
in driver(9) can be made more easily/without issue.
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D55414
bus: Document special ranges of IVARs2026-02-17T20:45:13ZJohn Baldwinjhb@FreeBSD.org2026-02-17T20:45:13Zurn:sha1:6cf4e30252fe48b230b9d76cac20576d5b3d2ffa
Some IVAR indices are special in that they have global meaning across
multiple buses where as other IVARs are always private to the local bus.
Try to document this a bit and add constants for the various ranges to
avoid future conflicts.
This is a no-op, but IVAR indices are now generally defined as enums
as that makes it easier to define them in terms of ranges.
Reviewed by: imp, royger, andrew
Differential Revision: https://reviews.freebsd.org/D54159
sound: Replace MIN() with min()2026-01-12T13:38:39ZChristos Margiolischristos@FreeBSD.org2026-01-12T13:38:39Zurn:sha1:eccd366b0a8ba7d902fcf0b1bec447926a75c36c
We use min() in most places.
No functional change intended.
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
sound: Do not check for NULL before free()2026-01-02T16:58:06ZChristos Margiolischristos@FreeBSD.org2026-01-02T16:56:52Zurn:sha1:792251295cdf6c3e1cbb1aa6291434539632912f
No functional change intended.
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D54174
sound: Get rid of unnecessary breaks2026-01-02T16:58:04ZChristos Margiolischristos@FreeBSD.org2026-01-02T16:55:54Zurn:sha1:21a2aa785d7d7610b3e90e028ae782893d8c095a
No functional change intended.
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
snd_hda: Default to 256 when corb/rirb is invalid2026-01-02T15:49:33ZChristos Margiolischristos@FreeBSD.org2026-01-02T15:49:33Zurn:sha1:2455db44ef088797d113397f863d46c50e77134f
PR: 289284
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: emaste
Differential Revision: https://reviews.freebsd.org/D54339
bus_alloc_resource: Pass rid by value to BUS_ALLOC_RESOURCE DEVMETHOD2025-12-09T20:00:06ZJohn Baldwinjhb@FreeBSD.org2025-12-09T19:59:21Zurn:sha1:575639548cef58590a1d70c29e47aae0e8d44153
The wrapper functions such as bus_alloc_resource_any() still support
passing the rid by value or pointer, but the underlying implementation
now passes by value.
Reviewed by: imp
Differential Revision: https://reviews.freebsd.org/D53402