On Mon, 31 May 2010, Robert Jarzmik wrote: > Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> writes: > > > Hi Robert, > > > > I have digged around in the Datasheet and if I understand it correctly > > the PXA swaps red/blue in RGB mode. So if we do not use rgb mode but yuv > > (which should be a pass through) we should be able to support rgb on PXA > > aswell. Robert, can you confirm that with the following patch applied > > you still get an image but with red/blue swapped? > I can confirm the color swap. > If you want to follow that path, I would suggest instead : > cicr1 |= CICR1_COLOR_SP_VAL(0); > > There is no difference from a processing point of view, it's just that > CICR1_COLOR_SP_VAL(0) is "raw colorspace", with means "pass through", and that > seems to be your goal here. That would be the default case in that switch, but raw only supports 8, 9, or 10 bpp, so, you'd have to use 8bpp but then fake the pixels-per-line field. But that would be the cleanest way, yes. Would that work like that? > Note that the patch would have to be completed with the BGR565 into RGB565 > conversion, if the sensor was to provide only BGR565. But that could very well > be for another patch. Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- 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
