Page MenuHomeFreeBSD
Differential D58091

fwcam: set ISO speed from device link speed
ClosedPublic
Actions

Authored by seuros on Jul 7 2026, 5:54 PM.
Tags
Referenced Files
F171264160: D58091.diff
Wed, Sep 9, 11:16 PM
F171263845: D58091.diff
Wed, Sep 9, 11:11 PM
Unknown Object (File)
Tue, Sep 8, 3:00 PM
Unknown Object (File)
Thu, Sep 3, 3:33 PM
Unknown Object (File)
Tue, Sep 1, 10:24 PM
Unknown Object (File)
Tue, Sep 1, 8:02 PM
Unknown Object (File)
Fri, Aug 28, 11:58 PM
Unknown Object (File)
Fri, Aug 28, 8:55 PM
View All 54 Files
Subscribers
imp

Details

Reviewers
adrian
Group Reviewers
firewire
Commits
rG4e0fc4d44dc1: fwcam: set ISO speed from device link speed
Summary

iso_speed was never initialized, defaulting to S100 regardless of the
camera's actual link speed. Some cameras firmwares reject ISO_EN
when the speed field in the ISO_CHANNEL register
does not match their capabilities.

Diff Detail

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

Event Timeline

seuros created this revision.Jul 7 2026, 5:54 PM
Herald added a subscriber: imp. · View Herald TranscriptJul 7 2026, 5:54 PM
seuros requested review of this revision.Jul 7 2026, 5:54 PM
Harbormaster completed remote builds in B74596: Diff 181470.Jul 7 2026, 5:54 PM
seuros added a reviewer: firewire.Jul 7 2026, 5:54 PM
seuros added projects: firewire, drivers.
seuros added a comment.Jul 7 2026, 5:58 PM

S400 is the default in non apple hardware.

Apple firmware handle the negotiation automatically from what it seems.

This fix is for non-apple cards.

seuros added a parent revision: D58090: fwcam: add dynamic resolution and frame rate support.Jul 7 2026, 8:11 PM
seuros added a child revision: D58092: fwcam: write video mode registers before enabling ISO streaming.
adrian accepted this revision.Jul 7 2026, 8:44 PM

accepted; tested!

This revision is now accepted and ready to land.Jul 7 2026, 8:44 PM
Closed by commit rG4e0fc4d44dc1: fwcam: set ISO speed from device link speed (authored by seuros, committed by adrian). · Explain WhyJul 8 2026, 5:42 AM
This revision was automatically updated to reflect the committed changes.
adrian added a commit: rG4e0fc4d44dc1: fwcam: set ISO speed from device link speed.

Revision Contents
Changeset List

PathSize
sys/
dev/
firewire/
1 line

Diff 181470

View Options

sys/dev/firewire/fwcam.c

Loading...