Hi, On 01/11/2013 12:08 PM, Sebastian Dröge wrote: > I can't test the patch right now but it should do almost the right > thing. IMHO for the chroma planes the bytesperline should be (width > +1)/2, otherwise you'll miss one chroma value per line for odd widths.
Odd widths are not allowed, the driver will adjust width to be multiple of 16 pixels. However, you can adjust the usable area more precisely with VIDIOC_S_CROP or VIDIOC_S_SELECTION ioctl. I still need to do some work to define properly the selection ioctl on mem-to-mem devices in the V4L2 documentation. > However I also noticed another bug. Currently S_FMT happily allows > V4L2_PIX_FMT_BGR32, V4L2_PIX_FMT_BGR24, V4L2_PIX_FMT_RGB24 and probably > others. But the output will be distorted and useless. > (V4L2_PIX_FMT_RGB32 works perfectly fine) This shouldn't really happen. Are you checking pixelformat after VIDIOC_S_FMT call ? Isn't it adjusted to some valid and supported by the driver format ? -- Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
