On Monday 18 July 2011 00:14:21 Guennadi Liakhovetski wrote: > On Sun, 17 Jul 2011, Laurent Pinchart wrote: > > Hi Guennadi, > > > > On Saturday 16 July 2011 23:40:23 Guennadi Liakhovetski wrote: > > > On Sat, 16 Jul 2011, Laurent Pinchart wrote: > > > > On Saturday 16 July 2011 01:11:28 Guennadi Liakhovetski wrote: > > > > > On Fri, 15 Jul 2011, Laurent Pinchart wrote: > > > > > > Compute crop rectangle boundaries to ensure a GRBG Bayer pattern. > > > > > > > > > > > > Signed-off-by: Laurent Pinchart > > > > > > <laurent.pinchart@xxxxxxxxxxxxxxxx> --- > > > > > > > > > > > > drivers/media/video/mt9v032.c | 20 ++++++++++---------- > > > > > > 1 files changed, 10 insertions(+), 10 deletions(-) > > > > > > > > > > > > If there's no comment I'll send a pull request for this patch in > > > > > > a couple of days. > > > > > > > > > > Hm, I might have a comment: why?... Isn't it natural to accept the > > > > > fact, that different sensors put a different Bayer pixel at their > > > > > sensor matrix origin? Isn't that why we have all possible Bayer > > > > > formats? Maybe you just have to choose a different output format? > > > > > > > > That's the other solution. The driver currently claims the device > > > > outputs SGRBG, but configures it to output SGBGR. This is clearly a > > > > bug. Is it better to modify the format than the crop rectangle > > > > location ? > > > > > > Actually, it is interesting. I just looked (again) in the mt9v032 and > > > some other Aptina Bayer sensor datasheets, and they actually have > > > _odd_ numbers of rows and columns. So, mt9v032 actually has 753x481 > > > active pixels. And that extra pixel is explicitly provided to adjust > > > the origin colour. Ok, they write, it is for uniformity with the > > > mirrored image, but who believes them?;-) So, maybe you should adjust > > > your max values to the above ones, then taking one pixel out of your > > > image will not reduce your useful image size. > > > > I'm not sure what you mean. Even though the pixel array is bigger than > > that, the maximum output width/height are 752x480 according to the > > datasheet. > > Have a look at the "Pixel array structure" (p.10 in my version) section. I've seen that, but the sensor is still unable to output an image bigger than 752x480. See registers 3 and 4 maximum values on page 24 (in my version :-)). -- Regards, Laurent Pinchart -- 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
