On Fri, 6 Mar 2009 [email protected] wrote: > ----- Mail Original ----- > De: "Guennadi Liakhovetski" <[email protected]> > ??: "Trent Piepho" <[email protected]> > Cc: "Robert Jarzmik" <[email protected]>, [email protected], "Linux > Media Mailing List" <[email protected]> > 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
