RE: [media-ctl] [omap3camera:devel] How to use the app?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent,

> -----Original Message-----
> From: Laurent Pinchart [mailto:laurent.pinchart@xxxxxxxxxxxxxxxx]
> Sent: Tuesday, June 29, 2010 5:23 AM
> To: Aguirre, Sergio
> Cc: Sakari Ailus; linux-media@xxxxxxxxxxxxxxx
> Subject: Re: [media-ctl] [omap3camera:devel] How to use the app?
> 
> Hi Sergio,
> 
> On Tuesday 29 June 2010 01:34:01 Aguirre, Sergio wrote:
> > Hi Laurent/Sakari,
> >
> > I have been attempting to migrate my IMX046 sensor driver, that I had
> > working on my Zoom3(OMAP3630 ES1.1) with older codebase, to work with
> the
> > latest omap3camera tree, 'devel' branch:
> >
> > http://gitorious.org/omap3camera/mainline/commits/devel
> >
> > And for that, I'm trying to also understand how to use your test tool:
> > "media-ctl":
> >
> > http://git.ideasonboard.org/?p=media-ctl.git;a=summary
> >
> > Now, the thing is that, I don't see any guide to learn how to write the
> > Proper format for some of the parameters, like to build links in
> > interactive mode (-i), or to set formats (-f).
> >
> > Can you please detail about a typical usage for this tool? (example on
> how
> > to build a link, set link format, etc.)
> 
> I've pushed a new patch to the media-ctl repository that makes the help
> message a bit more verbose when running media-ctl -h. It describes the
> links
> and formats as follows:
> 
> Links and formats are defined as
> 	link            = pad, '->', pad, '[', flags, ']' ;
> 	format          = pad, '[', fcc, ' ', size, [ ' ', crop ], ']' ;
> 	pad             = entity, ':', pad number ;
> 	entity          = entity number | ( '\"', entity name, '\"' ) ;
> 	size            = width, 'x', height ;
> 	crop            = left, ',', top, '/', size ;
> where the fields are
> 	entity number   Entity numeric identifier
> 	entity name     Entify name (string)
> 	pad number      Pad numeric identifier
> 	flags           Link flags (0: inactive, 1: active)
> 	fcc             Format FourCC
> 	width           Image width in pixels
> 	height          Image height in pixels
> 
> For instance, to set the CCDC to preview link as active, the link
> specifier
> would be
> 
> '"OMAP3 ISP CCDC":2 -> "OMAP3 ISP preview":0 [1]'
> 
> To set the format on the preview output pad to YUYV 1280x720, the format
> specifier would be
> 
> '"OMAP3 ISP preview":2 [YUYV 1280x720]'
> 
> Spaces are optional.

Thanks a lot for the explanation, and for the verbose help patch!
That's much better :)

> 
> > So far, my progress is pushed into this branch:
> >
> > http://dev.omapzoom.org/?p=saaguirre/linux-omap-
> camera.git;a=shortlog;h=ref
> > s/heads/mc_migration_wip
> 
> Thanks for the link.
> 
> > And with that, after I boot, I get the following topology:
> 
> [snip]
> 
> You will find a set of patches that remove the legacy video nodes attached
> to
> this e-mail. They haven't been applied to the omap3camera tree yet, as we
> still haven't fixed all userspace components yet, but they should get
> there in
> a few weeks hopefully. You should probably apply them to your tree to make
> sure you don't start using the legacy video nodes by mistake. They also
> remove
> a lot of code, which is always good, and remove the hardcoded number of
> sensors.

I had following compilation error:

drivers/media/video/isp/ispvideo.c: In function 'isp_video_streamon':
drivers/media/video/isp/ispvideo.c:780: error: 'const struct isp_video_operations' has no member named 'stream_off'
drivers/media/video/isp/ispvideo.c:781: error: 'const struct isp_video_operations' has no member named 'stream_off'
make[4]: *** [drivers/media/video/isp/ispvideo.o] Error 1
make[3]: *** [drivers/media/video/isp] Error 2
make[3]: *** Waiting for unfinished jobs....
make[2]: *** [drivers/media/video] Error 2
make[1]: *** [drivers/media] Error 2
make: *** [drivers] Error 2

Which I solved with the attached patch. You might want to squash it with your patch "omap3isp: video: Remove the init, cleanup and stream_off operations"

I'll continue trying to bring up my sensor, and let you know if I have other query.

Thanks for your time!

Regards,
Sergio

> 
> --
> Regards,
> 
> Laurent Pinchart

Attachment: 0001-SQUASH-omap3isp-video-Remove-the-init-cleanup-and-st.patch
Description: 0001-SQUASH-omap3isp-video-Remove-the-init-cleanup-and-st.patch


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux