Hi Guernnadi, Christoph, On 12 March 2013 09:25, Christoph Fritz <chf.fritz@xxxxxxxxxxxxxx> wrote: > On Tue, 2013-03-12 at 08:58 +0100, Guennadi Liakhovetski wrote: >> On Thu, 7 Mar 2013, javier Martin wrote: > >> > What mbus format are you using? Could you please check if the s_width >> > value that your sensor mt9m001 returns is correct? Remember it should >> > be in pixels, not in bytes. >> >> Thanks for looking at this. But here's my question: for a pass-through >> mode mx2_camera uses a 16-bpp (RGB565) format. But what if it's actually >> an 8-bpp format, don't you then have to adjust line-width register >> settings? If you don't do that, the camera interface would expect a double >> number of bytes per line, so, it could get confused by HSYNC coming after >> half-line? You are right. > To emphasize this: I'm using here a mt9m001 (monochrome) camera with an > 8-bpp format. Ok, now that makes sense. Then, what you should do is apply your patch conditionally so that you don't break other working cases: - Channel 1 is being used. - Channel 1 is in pass-through mode. - The sensor uses an 8-bpp format. Regards. -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de los Castros s/n 39005- Santander. Cantabria. Spain +34 942 25 32 60 www.vista-silicon.com -- 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
