Differential D58100

fwcam: defer ISO streaming to first read
ClosedPublic
Actions

Authored by guest-seuros on Wed, Jul 8, 8:08 AM.

Tags

Referenced Files

	F162323239: D58100.diff
	Sun, Jul 12, 1:33 AM

	F162302003: D58100.diff
	Sat, Jul 11, 8:43 PM

	Unknown Object (File)
	Wed, Jul 8, 11:39 AM

Subscribers

Details

Reviewers

Group Reviewers

Commits

rG56d4dee82de1: fwcam: defer ISO streaming to first read

Summary

Moved ISO start to first usage. Opening the device now
only validates state and increments the open count, allowing info
queries and mode changes without starting the camera. ISO streaming
begins on demand when userland first reads frame data.
This avoid the camera led to turn-on at attach.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

guest-seuros created this revision.Wed, Jul 8, 8:08 AM

Herald added a subscriber: imp. · View Herald TranscriptWed, Jul 8, 8:08 AM

guest-seuros requested review of this revision.Wed, Jul 8, 8:08 AM

Harbormaster completed remote builds in B74610: Diff 181524.Wed, Jul 8, 8:08 AM

guest-seuros added a reviewer: firewire.Wed, Jul 8, 8:26 AM

guest-seuros added projects: firewire, drivers.

adrian accepted this revision.Thu, Jul 9, 12:56 AM

This revision is now accepted and ready to land.Thu, Jul 9, 12:56 AM

adrian added a child revision: D58101: fwcam: retry ISO enable after re-powering camera.Thu, Jul 9, 12:56 AM

Closed by commit rG56d4dee82de1: fwcam: defer ISO streaming to first read (authored by guest-seuros, committed by adrian). · Explain WhyThu, Jul 9, 2:54 AM

This revision was automatically updated to reflect the committed changes.

adrian added a commit: rG56d4dee82de1: fwcam: defer ISO streaming to first read.

Revision Contents
Changeset List

Path

Size

sys/

dev/

firewire/

24 lines

Diff 181580

sys/dev/firewire/fwcam.c

Loading...