audiovideo_V4L2: Added more pixel formats.

audiovideo_V4L2: Added more pixel formats.

Added the rest of the YUYV ordering formats, and also YU12.

Change-Id: Id04a780b237c33e985c1750acf36eb925b65b042

BUG=none
TEST=none

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=dfe7f67

[jiesun1, 2011-04-27 22:47:01]

LGTM, if you had tested.

[Andrew Chew, 2011-04-28 00:03:04]

I tested YUYV, YVYU, UYVY, and VYUY, and they all look good.  I also tested
YU12, and I THINK the test is doing the right thing, but I'm not 100% sure. 
Still debugging, but it doesn't look like the chroma planes are being used
properly.

[Andrew Chew, 2011-04-28 02:56:32]

Okay, I think the problem with YU12 right now is in the test.  I forced the test
to dump the raw Y, U, and V planes, then concatenated them together, and used a
different program to convert from YU12 to RGB, and the image looks good.

[jiesun, 2011-04-28 17:28:27]

some errors and nits.

http://codereview.chromium.org/6883202/diff/3001/client/site_tests/audiovideo...
File client/site_tests/audiovideo_V4L2/src/media_v4l2_test.cc (right):

http://codereview.chromium.org/6883202/diff/3001/client/site_tests/audiovideo...
client/site_tests/audiovideo_V4L2/src/media_v4l2_test.cc:158: }
I actually prefer to do the branching outside the main loop.
using y0_offset, y1_offset, and u_offset, and v_offset...
save some cpu..

http://codereview.chromium.org/6883202/diff/3001/client/site_tests/audiovideo...
client/site_tests/audiovideo_V4L2/src/media_v4l2_test.cc:194: int32_t u =
u_plane[j>>1];
apparently that I had forgetten to -128 here and the line below.

http://codereview.chromium.org/6883202/diff/3001/client/site_tests/audiovideo...
client/site_tests/audiovideo_V4L2/src/media_v4l2_test.cc:195: int32_t v =
v_plane[j>>1];
please also add space around binary operator >>.

[jiesun, 2011-04-28 17:32:31]

YV12?

http://codereview.chromium.org/6883202/diff/3001/client/site_tests/audiovideo...
File client/site_tests/audiovideo_V4L2/src/media_v4l2_test.cc (right):

http://codereview.chromium.org/6883202/diff/3001/client/site_tests/audiovideo...
client/site_tests/audiovideo_V4L2/src/media_v4l2_test.cc:181: } else if (ifmt ==
v4l2_fourcc('Y', 'U', '1', '2')) {
|| ifmt == v4l2_fourcc('Y', 'V', '1', '2')
since in another patch, you said YV12 is done.

http://codereview.chromium.org/6883202/diff/3001/client/site_tests/audiovideo...
client/site_tests/audiovideo_V4L2/src/media_v4l2_test.cc:190: uint8_t* v_plane =
u_plane + height * istride / 4;
if (ifmt == v4l2_fourcc('Y', 'V', '1', '2')) swap the u_plane and v_plane.

[Andrew Chew, 2011-04-28 20:50:07]

Done.  Retested YUYV, YVYU, UYVY, VYUY, YU12, and YV12, and they all look good.

[jiesun1, 2011-04-28 21:16:31]

LGTM