From: "Mario Gaucher" <zadiglist@zadig.ca>
To: adaplas@pol.net
Cc: "Linux-Kernel" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] fbdev: Fix crash if fb_set_var() called before register_framebuffer()
Date: Fri, 26 Nov 2004 18:17:36 -0500
Message-Id: <20041126231251.M19516@zadig.ca>
In-Reply-To: <200411260845.04618.adaplas@hotpop.com>
References: <200411250115.50895.adaplas@hotpop.com> <20041125235955.M86030@zadig.ca> <200411260845.04618.adaplas@hotpop.com>
MIME-Version: 1.0
Content-Type: text/plain;
	charset=iso-8859-1
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1799
Lines: 55


From: "Antonino A. Daplas" <adaplas@hotpop.com>
> > but I still has some problem with my Matrox Millenium PCI card using
> > matroxfb driver... the kernel boot... but I get corrupted characters on
> > the console... X load ok and display ok...
> 
> Try this first.
> 
> 1. Open drivers/video/matrox/matrofb_accel.c
> 2. At the end of the file is this function:
> 
> static void matroxfb_imageblit(struct fb_info* info, const struct
fb_image* image) {
> 	MINFO_FROM_INFO(info);
> 
> 	DBG_HEAVY(__FUNCTION__);
> 
> 	if (image->depth == 1) {
> 		u_int32_t fgx, bgx;
> 
> 		fgx = ((u_int32_t*)info->pseudo_palette)[image->fg_color];
> 		bgx = ((u_int32_t*)info->pseudo_palette)[image->bg_color];
> 		matroxfb_1bpp_imageblit(PMINFO fgx, bgx, image->data, image->width,
image->height, image->dy, image->dx);
> 	} else {
> 		/* Danger! image->depth is useless: logo painting code always
> 		   passes framebuffer color depth here, although logo data are
> 		   always 8bpp and info->pseudo_palette is changed to contain
> 		   logo palette to be used (but only for true/direct-color... sic...).
> 		   So do it completely in software... */
> 		cfb_imageblit(info, image);
> 	}
> }
> 
> 3. Replace the above function to use software drawing so it becomes like
this:
> 
> static void matroxfb_imageblit(struct fb_info* info, const struct
fb_image* image) {
> 	cfb_imageblit(info, image);
> }


this patch works fine on my PowerMac 7300 with my Matrox Millenium PCI
card... 

so I think that both patch you sent me should be included in the kernel... 

thank you

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