On Fri, 6 Mar 2009 robert.jarzmik@xxxxxxx wrote: > ----- Mail Original ----- > De: "Guennadi Liakhovetski" <g.liakhovetski@xxxxxx> > À: "Trent Piepho" <xyzzy@xxxxxxxxxxxxx> > Cc: "Robert Jarzmik" <robert.jarzmik@xxxxxxx>, mike@xxxxxxxxxxxxxx, "Linux Media Mailing List" <linux-media@xxxxxxxxxxxxxxx> > Envoyé: Jeudi 5 Mars 2009 23h15:03 GMT +01:00 Amsterdam / Berlin / Berne / Rome / Stockholm / Vienne > Objet: Re: [PATCH 1/4] pxa_camera: Remove YUV planar formats hole > > > Yes, adjusting both is also what I was suggesting in my original review. > > How about aligning the bigger of the two to 4 bytes and the smaller to 2? > > Yes, sounds good. > > I remade my calculations : > - if (width x height % 8 == 0) : > => frame size = width x height x 2 > => U plane size = frame size / 4 = width x height /2 > => U plane size is a multiple of 4 > As the last DMA load from QIF fifo will return 8 bytes (and not 4 as > we would expect, cf. PXA Developer's Manual, 27.4.4.1), this is not > good. So it's only a requirement that the total size of all three planes put together is a multiple of 8? Or is it a requirement that each plane is a multiple of 8? And are you using 4:2:2 or 4:2:0? -- 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
