Page MenuHomeFreeBSD

sound: Retire volch
ClosedPublic

Authored by christos on Apr 25 2025, 5:26 PM.
Tags
None
Referenced Files
F134567941: D50022.diff
Mon, Nov 3, 6:38 AM
Unknown Object (File)
Wed, Oct 29, 4:47 AM
Unknown Object (File)
Wed, Oct 29, 4:45 AM
Unknown Object (File)
Wed, Oct 29, 4:45 AM
Unknown Object (File)
Wed, Oct 29, 4:41 AM
Unknown Object (File)
Sun, Oct 26, 5:19 PM
Unknown Object (File)
Sat, Oct 25, 12:59 AM
Unknown Object (File)
Mon, Oct 13, 2:21 AM
Subscribers

Details

Summary

volch is a variable used internally in dsp_ioctl_channel() which
corresponds to either priv->rdch or priv->wrch. As a result, priv->volch
is also a pointer to one of those two channels, but is also not used
anywhere outside dsp_ioctl_channel(). Since the operation of figuring
out which channel we want to operate on inside dsp_ioctl_channel() is
cheap enough, retire volch to avoid unnecessary complexity and
confusion.

While here, rename "volch" to just "ch".

Sponsored by: The FreeBSD Foundation
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 63744
Build 60628: arc lint + arc unit

Event Timeline

sys/dev/sound/pcm/dsp.c
578

Delete this comment too? It's not adding anything anymore.

700

Can't you just pass NULL here?

christos marked 2 inline comments as done.

Address Mark's comments.

This revision is now accepted and ready to land.Apr 27 2025, 1:47 PM
This revision was automatically updated to reflect the committed changes.