Hi Hans, thank you for your comments. On 11 July 2012 11:45, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > Hi Javier! > > Thanks for the patch. > > On Wed 11 July 2012 10:55:03 Javier Martin wrote: >> Some video decoders such as tvp5150 provide separate >> video fields (V4L2_FIELD_SEQ_TB). This driver uses >> dmaengine to convert this format to V4L2_FIELD_INTERLACED_TB >> (weaving) or V4L2_FIELD_NONE (line doubling) > > Which field is used for the line doubling? Top or bottom? Or is each field > doubled, thus doubling the framerate? No, just top field is used. I don't know if it's worth defining a new field format for doubling fields. > I also recommend adding SEQ_BT/INTERLACED_BT support: NTSC transmits the bottom > field first, so it is useful to have support for that. Adding that is quite easy but I cannot test it. Maybe someone could add it later? -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de los Castros s/n 39005- Santander. Cantabria. Spain +34 942 25 32 60 www.vista-silicon.com -- 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