Dne 2.5.2017 v 16:43 Robert Jarzmik napsal(a): > Petr Cvek <petr.cvek@xxxxxx> writes: > >> During the transfer from the soc_camera a test in pxa_mbus_image_size() >> got removed. Without it any PXA_MBUS_LAYOUT_PACKED format causes either >> the return of a wrong value (PXA_MBUS_PACKING_2X8_PADHI doubles >> the correct value) or EINVAL (PXA_MBUS_PACKING_NONE and >> PXA_MBUS_PACKING_EXTEND16). This was observed in an error from the ffmpeg >> (for some of the YUYV subvariants). >> >> This patch re-adds the same test as in soc_camera version. >> >> Signed-off-by: Petr Cvek <petr.cvek@xxxxxx> > Did you test that with YUV422P format ? > If yes, then you can have my ack. pxa27x-camera pxa27x-camera.0: s_fmt_vid_cap(pix=320x240:50323234) And mplayer to framebuffer "somewhat" works (it timeouts after some time but it does regardless on format, ffmpeg is fine). Anyway the patch does not affect V4L2_PIX_FMT_YUV422P in any way as the .layout field is PXA_MBUS_LAYOUT_PLANAR_2Y_U_V and test is only for "== PXA_MBUS_LAYOUT_PACKED" > > And you should add Hans to the reviewers list, it's his call ultimately, and his > tree which should carry it on. > > Cheers. > > -- > Robert > Petr
