Page MenuHomeFreeBSD

fwisound: add Apple FireWire audio driver
ClosedPublic

Authored by guest-seuros on Wed, Jul 8, 12:56 PM.
Referenced Files
F162304217: D58109.diff
Sat, Jul 11, 9:16 PM
F162278402: D58109.id.diff
Sat, Jul 11, 3:44 PM
Unknown Object (File)
Fri, Jul 10, 1:42 PM
Subscribers

Details

Summary

Expose audio capture from Apple FireWire devices as a standard
pcm(4)/dsp(4) device via the newpcm framework.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 74636
Build 71519: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Thu, Jul 9, 12:57 AM
adrian added inline comments.
sys/dev/firewire/fwisound.c
396

What's 475 for?

This revision was automatically updated to reflect the committed changes.
guest-seuros added inline comments.
sys/dev/firewire/fwisound.c
396

Damn. i should have used a constant #define FWISOUND_MAX_SAMPLES 475

That from Clemens Ladisch's RE: https://github.com/torvalds/linux/blob/29b9667982e4df2ed7744f86b1144f8bb58eb698/sound/firewire/isight.c#L45

The hard upper bound from the packet size would be (2048 - 16) / 4 = 508 . But i didn't want to diverge .

We can retry it in another round later with 508 value. (maybe it was a isight firmware in that time, or simply an Apple quirk.

sys/dev/firewire/fwisound.c
396

its ok, just add it to another cleanup diff later. no biggie.