Hi Javier On Mon, 2 Jan 2012, javier Martin wrote: > Hi Guennadi, > > On 29 December 2011 09:38, Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote: > > Hi Mauro > > > > Please pull a couple of soc-camera patches for 3.3. This is going to be a > > _much_ quieter pull, than the previous one:-) I didn't have time to > > continue the work on the soc-camera Media Controller conversion, so, that > > will have to wait at least until 3.4. > > > > Interestingly, Javier Martin has fixed field_count handling in mx2_camera, > > but, apparently, it also has to be fixed in other soc-camera drivers. So, > > a patch for that might follow later in the 3.3 cycle. > > > > The following changes since commit 1a5cd29631a6b75e49e6ad8a770ab9d69cda0fa2: > > > > [media] tda10021: Add support for DVB-C Annex C (2011-12-20 14:01:08 -0200) > > > > are available in the git repository at: > > git://linuxtv.org/gliakhovetski/v4l-dvb.git for-3.3 > > > > Guennadi Liakhovetski (4): > > V4L: soc-camera: remove redundant parameter from the .set_bus_param() method > > V4L: mt9m111: cleanly separate register contexts > > V4L: mt9m111: power down most circuits when suspended > > V4L: mt9m111: properly implement .s_crop and .s_fmt(), reset on STREAMON > > > > Javier Martin (2): > > media i.MX27 camera: add support for YUV420 format. > > media i.MX27 camera: Fix field_count handling. > > So, you already applied my patch related to "field_count" handling. I > was preparing a v3 for that patch to fix the frame_count type issue > and to really provide frame loss detection but I can prepare it on top > of the old one if you want. Yes, the type issue is not that severe. Matching a type of another variable or a struct field, with which this variable is connected, is a valid reason, so, we can keep u32, if you like. As for frame loss detection - yes, please, provide it as an incremental patch. Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
