On Mon, 9 Mar 2009, Robert Jarzmik wrote: > > Ok, this one will change I presume - new alignment calculations and > > line-breaking. In fact, if you adjust width and height earlier in set_fmt, > > maybe you'll just remove any rounding here completely. > Helas, not fully. > The problem is with passthrough and rgb formats, where I don't enforce > width/height. In the newest form of the patch I have this : > if (pcdev->channels == 3) > *size = icd->width * icd->height * 2; > else > *size = roundup(icd->width * icd->height * > ((icd->current_fmt->depth + 7) >> 3), 8); If icd->current_fmt->depth could be set to 16 for planar formats, then you could get rid of the special case here. -- 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