On Fri, 2011-11-04 at 13:13 -0500, Punya Prakash wrote: Hi, > RGB color format does not have any alpha components, adding alpha length to depth could yield wrong results. > > Signed-off-by: Punya Prakash <pprakash at ti.com> > --- > sys/fbdev/gstfbdevsink.c | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/sys/fbdev/gstfbdevsink.c b/sys/fbdev/gstfbdevsink.c > index 8925f3d..568f091 100644 > --- a/sys/fbdev/gstfbdevsink.c > +++ b/sys/fbdev/gstfbdevsink.c > @@ -185,8 +185,7 @@ gst_fbdevsink_getcaps (GstBaseSink * bsink) > "bpp", G_TYPE_INT, fbdevsink->varinfo.bits_per_pixel, > "depth", G_TYPE_INT, fbdevsink->varinfo.red.length + > fbdevsink->varinfo.green.length + > - fbdevsink->varinfo.blue.length + > - fbdevsink->varinfo.transp.length, > + fbdevsink->varinfo.blue.length, > "endianness", G_TYPE_INT, endianness, > "red_mask", G_TYPE_INT, rmask, > "green_mask", G_TYPE_INT, gmask, "blue_mask", G_TYPE_INT, bmask, NULL); Thanks for the patch, but could you please put it as git format-patch style patch into bugzilla (or ping the bug if there is one already)? Thanks! Cheers -Tim