src/sys/dev/sound, branch mainFreeBSD source treehttps://cgit-dev.freebsd.org/src/atom?h=main2026-04-07T15:44:39Zsound: Make chn_reset() control flow clearer2026-04-07T15:44:39ZChristos Margiolischristos@FreeBSD.org2026-04-07T15:44:39Zurn:sha1:8007a78bfe1f41b7055f8677f8ac9ae4cc2fbc7c
I think this is cleaner than playing around with return values.
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D56264
sound: De-macro array definitions in chn_calclatency()2026-04-07T15:44:34ZChristos Margiolischristos@FreeBSD.org2026-04-07T15:44:34Zurn:sha1:66a82f5fa471b8eda018b7566c3725a6742dfc0c
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D56263
sound: Simplify parts of chn_notify()2026-04-07T15:44:29ZChristos Margiolischristos@FreeBSD.org2026-04-07T15:44:29Zurn:sha1:8b18fc456bb136112e166508a9182718cdd44be2
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D56240
sound: Retire unused CHN_N_* defines2026-04-07T15:44:21ZChristos Margiolischristos@FreeBSD.org2026-04-07T15:44:21Zurn:sha1:2d84c8a3d3d673081c803f8aadc4eae8f6a221d8
These still haven't been implemented by the original author, and there
doesn't seem to be much use for them anyway.
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D56239
sound: Mark some snd_fmt* functions as static2026-04-07T15:44:15ZChristos Margiolischristos@FreeBSD.org2026-04-07T15:44:15Zurn:sha1:f369d4148b93410fe7f8ea989f66790403d23ce8
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D56238
snd_uaudio: Retire sndcard_func usage2026-03-30T14:13:16ZChristos Margiolischristos@FreeBSD.org2026-03-30T14:13:16Zurn:sha1:0efd33382504d3172734fa21325fcabef9c7f063
This is effectively a no-op, as it does not make use of the
sndcard_func->varinfo field, so eventually ua_probe() always succeeds.
Also change ua_probe()'s value to 0. There is no need to return
BUS_PROBE_DEFAULT, because snd_uaudio() attaches the sound(4)'s children
with bus_attach_children().
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D56160
sound: Remove unimplemented chn_reinit() definition2026-03-24T10:38:42ZChristos Margiolischristos@FreeBSD.org2026-03-24T10:32:38Zurn:sha1:2e1cf242292a33a71839c512d222115d36f8fd40
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
sound: Remove dead code in pcm/ac97.c2026-03-24T10:38:42ZChristos Margiolischristos@FreeBSD.org2026-03-24T10:32:36Zurn:sha1:d048cec5fb41cf9529dd0b637a51c7ee72b59b3b
ac97_uninitmixer() does not exist also.
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D56045
sound: Address some XXX comments regarding AC'97 IDs2026-03-24T10:38:41ZChristos Margiolischristos@FreeBSD.org2026-03-24T10:32:31Zurn:sha1:edad981e1e8673b6dd14656fe75b9f239fb777ce
These seem harmless to address. Not sure why the original author did not
just assign the appropriate names if he knew they were wrong.
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D56044
sound: Retire Z_PARANOID2026-03-24T10:38:41ZChristos Margiolischristos@FreeBSD.org2026-03-24T10:32:22Zurn:sha1:6b91c8d5d2c35c4c6231eb172b514f95ce6e10d7
It is always defined.
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D56042