Re: [PATCH] neofb patches

On Wed, Apr 21, 2004 at 06:47:16PM +0100, James Simmons wrote:

quoted

I have the same chipset. I have the new driver posted at my website. I 
tried your patchs but I had no luck.

I wasn't aware of your patches earlier (or where to get them).. I found the URL
you posted in another message and have just downloaded the diff and will look
into integrating things..

What is your LCD resolution, BTW?

1024x768. Its a IBM thinkpad 600X.

quoted

quoted

neofb.2.imageblit.patch - Fix imageblit for color images

  It looks like the imageblit routine was written with only mono blits in 
  mind. This patch adds support for color image transfers (Hi Tux!).

I tried this patch. My boot wouldn't boot with this patch. Mind you I 
never got the imageblit to work right.

Could you be a little more specific?

Were you applying this to a stock kernel or to your patched driver?  What
exactly happened when you tried it?  You were booting in 8-bit color mode, I
assume?

Patched driver. I manually applied your patches. I start in 8 bpp mode. 
Nothing appeared on the screen.

quoted

This can be handled by struct fb_pixmap. It padded image scan_lines to 
what the hardware can handle or is optimized to.

Ok, I'll take a look at this.. I was planning on going back and investigating
this further anyway.  Initially I was just concentrating on getting working
functionality.

Okay. Cool We can work on a solution for this problem.
 

quoted

quoted

neofb.5.modedb.patch    - Change to use modedb for video mode selection

  Allows specifying the video mode on the kernel/insmod command line now too.

Also barfed my machine :-( I really like to get i2c support going. Perhaps 
we should ask neomagic for the full specs for this chipset.

Hmm.. again, what does "barfed" mean, exactly?  Did you try specifying
different video modes on the command line?

Blank screen.

quoted

quoted

neofb.6.blanking.patch  - Add support for DPMS blanking

Ug. The new neofb driver has been moved over to use the new vga core code.
See vgastate.c and vga.h in include/video.

Ah, ok.. I'll make the appropriate adjustmen

-alex

-------------------------------------------------------
This SF.net email is sponsored by: The Robotic Monkeys at ThinkGeek
For a limited time only, get FREE Ground shipping on all orders of $35
or more. Hurry up and shop folks, this offer expires April 30th!
http://www.thinkgeek.com/freeshipping/?cpg=12297