sound: Call device_get_name() and device_get_unit() only once in mixer_init()
ClosedPublic
Actions

Authored by christos on Thu, Apr 18, 5:42 PM.

Details

Reviewers

markj
emaste

Commits

rG41ab202ff74e: sound: Call device_get_name() and device_get_unit() only once in mixer_init()

Summary

No functional change intended.

Sponsored by: The FreeBSD Foundation
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

christos created this revision.Thu, Apr 18, 5:42 PM

Herald added a subscriber: imp. · View Herald TranscriptThu, Apr 18, 5:42 PM

christos requested review of this revision.Thu, Apr 18, 5:42 PM

Harbormaster completed remote builds in B57179: Diff 137318.Thu, Apr 18, 5:42 PM

emaste added inline comments.Thu, Apr 18, 5:51 PM

sys/dev/sound/pcm/mixer.c
736–740	probably swap the order here just for consistency (we think of <name><unit>)

Address Ed's comment.

Harbormaster completed remote builds in B57197: Diff 137337.Thu, Apr 18, 6:05 PM

emaste accepted this revision.Thu, Apr 18, 6:16 PM

This revision is now accepted and ready to land.Thu, Apr 18, 6:16 PM

markj accepted this revision.Thu, Apr 18, 7:10 PM

Closed by commit rG41ab202ff74e: sound: Call device_get_name() and device_get_unit() only once in mixer_init() (authored by christos). · Explain WhyThu, Apr 18, 8:36 PM

This revision was automatically updated to reflect the committed changes.

christos added a commit: rG41ab202ff74e: sound: Call device_get_name() and device_get_unit() only once in mixer_init().

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

mixer.c

12 lines

Diff 137363

View Options

sound: Call device_get_name() and device_get_unit() only once in mixer_init()ClosedPublicActions