On Thu, Sep 1, 2011 at 6:14 PM, Enrico <ebutera@xxxxxxxxxxxxxxxx> wrote:
> On Thu, Sep 1, 2011 at 5:16 PM, Gary Thomas <gary@xxxxxxxxxxxx> wrote:
>>
>> - entity 16: tvp5150m1 2-005c (1 pad, 1 link)
>> type V4L2 subdev subtype Unknown
>> device node name /dev/v4l-subdev8
>> pad0: Output [unknown 720x480 (1,1)/720x480]
>> -> 'OMAP3 ISP CCDC':pad0 [ACTIVE]
>>
>> Ideas where to look for the 'unknown' mode?
>
> I didn't notice that, if you are using UYVY8_2X8 the reason is in
> media-ctl main.c:
>
> { "UYVY", V4L2_MBUS_FMT_UYVY8_1X16 },
>
> You can add a line like:
>
> { "UYVY2X8", V4L2_MBUS_FMT_UYVY8_2X8 },
>
> recompile and it should work, i'll try it now.
That worked, but now there is another problem.
yavta will set UYVY (PIX_FMT), this will cause a call to
ispvideo.c:isp_video_pix_to_mbus(..), that will do this:
for (i = 0; i < ARRAY_SIZE(formats); ++i) {
if (formats[i].pixelformat == pix->pixelformat)
break;
}
that is it will stop at the first matching array item, and that's:
{ V4L2_MBUS_FMT_UYVY8_1X16, V4L2_MBUS_FMT_UYVY8_1X16,
V4L2_MBUS_FMT_UYVY8_1X16, 0,
V4L2_PIX_FMT_UYVY, 16, 16, },
but you wanted this:
{ V4L2_MBUS_FMT_UYVY8_2X8, V4L2_MBUS_FMT_UYVY8_2X8,
V4L2_MBUS_FMT_UYVY8_2X8, 0,
V4L2_PIX_FMT_UYVY, 8, 16, },
so a better check could be to check for width too, but i don't know if
it's possibile to pass a width requirement or if it's already there in
some struct passed to the function.
Enrico
--
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
