Page MenuHomeFreeBSD
Differential D45862

sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO
ClosedPublic
Actions

Authored by christos on Wed, Jul 3, 7:10 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 24, 4:23 AM
Unknown Object (File)
Fri, Jul 19, 5:07 AM
Unknown Object (File)
Tue, Jul 16, 11:30 AM
Unknown Object (File)
Tue, Jul 9, 5:14 PM
Unknown Object (File)
Sat, Jul 6, 5:07 PM
Unknown Object (File)
Fri, Jul 5, 10:18 PM
Unknown Object (File)
Fri, Jul 5, 9:21 PM
Unknown Object (File)
Fri, Jul 5, 6:33 PM
View All 9 Files
Subscribers
imp

Details

Reviewers
markj
emaste
dev_submerge.ch
Commits
rG95d3abff7375: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO
rG86585210fd56: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO
Summary

Since we allow feeding of any rate within the [feeder_rate_min,
feeder_rate_max] range, report this as the min/max rate as well. Only
exceptions are when we the device is opened in exclusive or bitperfect
mode.

Sponsored by: The FreeBSD Foundation
MFC after: 2 days

Diff Detail

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

Event Timeline

christos created this revision.Wed, Jul 3, 7:10 PM
Herald added a subscriber: imp. · View Herald TranscriptWed, Jul 3, 7:10 PM
christos requested review of this revision.Wed, Jul 3, 7:10 PM
Harbormaster completed remote builds in B58485: Diff 140532.Wed, Jul 3, 7:10 PM
christos mentioned this in D45664: sound: Fix min/max sample rate assignment for VCHANs.Wed, Jul 3, 7:10 PM
christos added inline comments.
sys/dev/sound/pcm/dsp.c
2123

I did this for AUDIOINFO since we can feed arbitrary rates to physical channels as well.

christos added inline comments.Wed, Jul 3, 7:21 PM
sys/dev/sound/pcm/dsp.c
2123

Didn't get the EXCL and BITPERFECT cases into account. Uploading fixed diff in a sec.

christos updated this revision to Diff 140535.Wed, Jul 3, 7:40 PM

Handle EXCLUSIVE and BITPERFECT cases.

Harbormaster completed remote builds in B58487: Diff 140535.Wed, Jul 3, 7:40 PM
christos edited the summary of this revision. (Show Details)Wed, Jul 3, 7:41 PM
christos added a comment.Wed, Jul 3, 7:47 PM

sndstat_get_caps() probably also needs fixing, but I will do this in a separate patch.

dev_submerge.ch accepted this revision.Thu, Jul 4, 3:23 PM

Looks good to me, in my tests with ossinfo this returns the expected sample rate values.

sys/dev/sound/pcm/dsp.c
2203–2205

I think this comment should be adapted on occasion.

This revision is now accepted and ready to land.Thu, Jul 4, 3:23 PM
christos added inline comments.Thu, Jul 4, 3:28 PM
sys/dev/sound/pcm/dsp.c
2203–2205

I think we could even get rid of it. It also exists in AUDIOINFO.

christos added a comment.Thu, Jul 4, 4:00 PM
In D45862#1045693, @christos wrote:

sndstat_get_caps() probably also needs fixing, but I will do this in a separate patch.

D45872

Closed by commit rG86585210fd56: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO (authored by christos). · Explain WhySat, Jul 6, 6:24 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rG86585210fd56: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO.
christos added a commit: rG95d3abff7375: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO.Wed, Jul 10, 4:49 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
11 lines

Diff 140532

View Options

sys/dev/sound/pcm/dsp.c

Loading...