Page MenuHomeFreeBSD
Differential D27830

Add SoC audio framework
ClosedPublic
Actions

Authored by gonzo on Dec 30 2020, 12:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 28, 2:23 PM
Unknown Object (File)
Sat, Jul 25, 4:40 AM
Unknown Object (File)
Wed, Jul 22, 7:28 PM
Unknown Object (File)
Sun, Jul 19, 9:52 PM
Unknown Object (File)
Jul 2 2026, 10:44 PM
Unknown Object (File)
Jun 29 2026, 12:59 PM
Unknown Object (File)
Jun 24 2026, 2:42 AM
Unknown Object (File)
Jun 23 2026, 3:01 AM
View All Files
Subscribers
andrew
avg
emaste
gbe
• hselasky
imp

Details

Reviewers
manu
Commits
rGcb25a9455802: arm64: allwinner: Add i2s and codec support
rG2cfe870acdec: arm64: Add Soc audio framework
Summary

This framework is initial implementation of the simple-audio-card compatible
audio driver framework. It provides glue for CPU/codec/aux device.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 35799
Build 32688: arc lint + arc unit

Event Timeline

gonzo created this revision.Dec 30 2020, 12:20 AM
Herald added subscribers: emaste, andrew, imp. · View Herald TranscriptDec 30 2020, 12:20 AM
gonzo requested review of this revision.Dec 30 2020, 12:20 AM
Harbormaster completed remote builds in B35772: Diff 81346.Dec 30 2020, 12:20 AM
gbe added a subscriber: gbe.Dec 30 2020, 8:51 AM
manu added a comment.Dec 30 2020, 9:32 AM

Looks nice to me, do you have some example drivers so it might be easier to review ?

sys/dev/sound/fdt/simple_amplifier.c
99

enable-gpios is optional.

gonzo updated this revision to Diff 81382.Dec 30 2020, 7:33 PM
  • Add Allwinner codecs and I2S driver as an example for the framework usage
  • simple-amplifier: make gpio-enable optional
Harbormaster completed remote builds in B35799: Diff 81382.Dec 30 2020, 7:33 PM
manu added a comment.Dec 31 2020, 10:46 AM

Thanks for the example driver.
This looks ok for me but I know almost nothing about sound :P

sys/arm/allwinner/aw_i2s.c
339

Not sure this is correct :)

• hselasky added a subscriber: • hselasky.Jan 11 2021, 8:39 AM

I'd like to review this.

avg added a subscriber: avg.Jun 4 2021, 11:47 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jun 11 2021, 7:07 PM
Closed by commit rG2cfe870acdec: arm64: Add Soc audio framework (authored by gonzo, committed by manu). · Explain Why
This revision was automatically updated to reflect the committed changes.
manu added a commit: rG2cfe870acdec: arm64: Add Soc audio framework.
manu added a commit: rGcb25a9455802: arm64: allwinner: Add i2s and codec support.

Revision Contents
Changeset List

PathSize
sys/
arm/
allwinner/
a64/
488 lines
813 lines
417 lines
conf/
7 lines
dev/
sound/
fdt/
72 lines
95 lines
546 lines
127 lines
206 lines

Diff 81382

View Options

sys/arm/allwinner/a64/sun50i_a64_acodec.c

Loading...
View Options

sys/arm/allwinner/aw_i2s.c

Loading...
View Options

sys/arm/allwinner/sun8i_codec.c

Loading...
View Options

sys/conf/files.arm64

Loading...
View Options

sys/dev/sound/fdt/audio_dai.h

Loading...
View Options

sys/dev/sound/fdt/audio_dai_if.m

Loading...
View Options

sys/dev/sound/fdt/audio_soc.c

Loading...
View Options

sys/dev/sound/fdt/dummy_codec.c

Loading...
View Options

sys/dev/sound/fdt/simple_amplifier.c

Loading...