On 01/09/2013 02:35 PM, Laurent Pinchart wrote:
On Tuesday 08 January 2013 14:49:41 William Swanson wrote:
I believe the data is combined in a single buffer, with alternate fields
interleaved.
Could you please double-check that ? I'd like to be sure, not just believe :-)
Sorry for the delay in getting back to you. I have checked it, and the
fields are indeed interlaced into a single buffer. On the other hand,
looking at this caused me to discover another problem with the patch.
According to the TI documentation, the CCDC_SDOFST register controls the
deinterlacing process. My patch never configures this register, however,
which is surprising. The reason things work on our boards is because we
are carrying a separate patch which changes the register by accident.
Oops! I have fixed this, and will be sending another patch with the
CCDC_SDOFST changes.
In that case the OMAP3 ISP driver should set the v4l2_pix_format::field to
V4L2_FIELD_INTERLACED in the format-related ioctl when an interlaced format is
used. I suppose this could be added later - Sakari, any opinion ?
I don't have a lot of time to work on this stuff, so my main focus is
just getting the data to flow. Changing the output format information
involves other parts of the driver that I am not familiar with, so I
don't know if I will be able to work on that bit.
By the way, thanks for taking the time to review this, Laurent.
-William
--
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