Page MenuHomeFreeBSD
Differential D45862

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

Authored by christos on Wed, Jul 3, 7:10 PM.
Tags
None
Referenced Files
F87618392: D45862.id.diff
Fri, Jul 5, 10:18 PM
F87615817: D45862.id.diff
Fri, Jul 5, 9:21 PM
F87606609: D45862.id140532.diff
Fri, Jul 5, 6:33 PM
Unknown Object (File)
Thu, Jul 4, 11:25 AM
Subscribers
imp

Details

Reviewers
markj
emaste
dev_submerge.ch
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 58487
Build 55375: 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
2124

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
2124

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
2210–2212

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
2210–2212

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

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
20 lines

Diff 140535

View Options

sys/dev/sound/pcm/dsp.c

Loading...