Re: Linux 2.6.0-test3: logo patch

[Martin Schlemmer]

On Sat, 2003-08-09 at 17:38, Russell King wrote:
> On Sat, Aug 09, 2003 at 11:00:57AM -0500, Thomas Molina wrote:
> > The following patch has been floating around forever.  Can we get it in 
> > mainstream sometime in the near future?
> > 
> > --- linux-2.5-tm/drivers/video/cfbimgblt.c.orig	2003-08-08 17:42:16.000000000 -0500
> > +++ linux-2.5-tm/drivers/video/cfbimgblt.c	2003-08-08 17:42:30.000000000 -0500
> > @@ -325,7 +325,7 @@
> >  		else 
> >  			slow_imageblit(image, p, dst1, fgcolor, bgcolor,
> >  					start_index, pitch_index);
> > -	} else if (image->depth == bpp) 
> > +	} else if (image->depth <= bpp) 
> >  		color_imageblit(image, p, dst1, start_index, pitch_index);
> >  }
> >  
> 
> Is this patch _still_ not in the kernel.
> 
> Linus - please merge this patch - its required for several ARM framebuffer
> drivers, and several other drivers.  James has indicated that this is the
> correct fix back in May:
> 

Right, this fixes the 'no logo' issue - anybody have any idea why
I now have two logo's ? :)

If need be, I can give dmesg/etc/etc, but not at home right now.

> On Tue, May 13, 2003 at 11:41:34PM +0100, James Simmons wrote:
> > At the very bottom of cfbimgblt.c change
> >
> > } else if (image->depth == bpp)
> >
> > to
> >
> > } else if (image->depth <= bpp)
> >
> > and tell me if this works.
-- 
Martin Schlemmer
Gentoo Linux Developer, Desktop Team
Cape Town, South Africa

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/