[FB PATCH]

[FB PATCH]

[Richard Henderson]

I got link errors due to a missing fb_blank function.  The best
I can figure is that you were intending to call the fb_blank 
fbops hook.  Certainly that seems to work.

I'm not sure how no one else ran into this. 



r~



You can import this changeset into BK by piping this whole message to:
'| bk receive [path to repository]' or apply the patch as usual.

===================================================================


ChangeSet@1.957, 2002-12-27 15:04:08-08:00, rth@are.twiddle.net
  [FB] fb_blank is an fbops hook, not a standalone function.


 drivers/video/console/fbcon.c |    2 +-
 include/linux/fb.h            |    1 -
 2 files changed, 1 insertion, 2 deletions


diff -Nru a/drivers/video/console/fbcon.c b/drivers/video/console/fbcon.c
--- a/drivers/video/console/fbcon.c	Fri Dec 27 15:05:10 2002
+++ b/drivers/video/console/fbcon.c	Fri Dec 27 15:05:10 2002
@@ -1993,7 +1993,7 @@
 			update_screen(vc->vc_num);
 		return 0;
 	} else
-		return fb_blank(blank, info);
+		return info->fbops->fb_blank(blank, info);
 }
 
 static void fbcon_free_font(struct display *p)
diff -Nru a/include/linux/fb.h b/include/linux/fb.h
--- a/include/linux/fb.h	Fri Dec 27 15:05:10 2002
+++ b/include/linux/fb.h	Fri Dec 27 15:05:10 2002
@@ -449,7 +449,6 @@
 
 extern int fb_set_var(struct fb_var_screeninfo *var, struct fb_info *info); 
 extern int fb_pan_display(struct fb_var_screeninfo *var, struct fb_info *info); 
-extern int fb_blank(int blank, struct fb_info *info);
 extern int soft_cursor(struct fb_info *info, struct fb_cursor *cursor);
 extern void cfb_fillrect(struct fb_info *info, struct fb_fillrect *rect); 
 extern void cfb_copyarea(struct fb_info *info, struct fb_copyarea *area); 

===================================================================


This BitKeeper patch contains the following changesets:
+
## Wrapped with gzip_uu ##


begin 664 bkpatch2996
M'XL(`*;<##X``[55VVZ<,!!]7G^%I;RT:H"Q,==JHS3I+4JE1EOEJ:HJ`[.!
M+M@1F$TJ\?'UDC;73;9)&T`@;#,^,^><88L>=]BFD]:49(M^U)U))[)%UYQ5
M15&CJ]#8\9G6=MPK=8.>7>GM'7KF1#K<#8B=/9(F+^D2VRZ=,->_'#$_3S&=
MS-Y]./[T9D;(=$KW2ZE.\`L:.IT2H]NEK(MN5YJRULHUK51=@T:ZN6Z&RZ4#
M!^#V#%CD0Q`.+`01#3DK&)."80%<Q*$@%M;N+>"W8C#.0^X'(42#+R*(R%O*
MW"2(*'"/<8]'E`4IB!1B!^(4@*X)25]QZ@#9H_\7_#[)Z=?W>]_H//N>U5(M
M:-51J>RK/NUHJ?5BFRIMJ*2=D:J0=D>D\U[EIK)[DT/J^R*)R=%5@8GSR(,0
MD$!V-B16J;SN"_3J2O7GWCQSR^L9)H(-$"8^'T+@2584&681HI!B72WO"V9Y
MBK@/?K`*$P';"*IHJY7XO&55H+9Q<KLBOX9+``L''M@*#3P*YTD""<;9/!09
M6XOK9CP;K=,UWHA[#:(?1'$\BOO!SS8+_A^R(#^ZJFGLAKN-/#_#NG91VL1L
M71=_G1`DW+J"Q]8;(<2C-X+DIC7\-.`/68-1ASV+-0[44B_PRAS+2EY88Y3^
M2,!GZK1GXV6E?/0P%T_PQEN6)"%EY.#W<S)IT?2MHI6::V=G!+-Z7`!\,=ZW
MQ\F7KT=QW-7Z9D4\U6QK>^']9K--<47]```QC,3S1Q(/ST7\#!N]1-JKOL/B
MBO\"\UJV\D_SN^@2MQ1P-]^GT"X";MF^_*/E)>:+KF^F++:E$WE`?@&"R\7O
$0`<`````
`
end

Re: [FB PATCH]

[Geert Uytterhoeven]

On Fri, 27 Dec 2002, Richard Henderson wrote:
> I got link errors due to a missing fb_blank function.  The best
> I can figure is that you were intending to call the fb_blank 
> fbops hook.  Certainly that seems to work.
> 
> I'm not sure how no one else ran into this. 

Strange, it's defined in drivers/video/fbmem.c in my copy of 2.5.53.

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: [FB PATCH]

[Richard Henderson]

On Sat, Dec 28, 2002 at 12:25:58AM +0100, Geert Uytterhoeven wrote:
> Strange, it's defined in drivers/video/fbmem.c in my copy of 2.5.53.

Ah.  It's not exported, so fbcon as a module fails.
Not sure how I missed it with a grep...


r~


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: [Linux-fbdev-devel] [FB PATCH]

[James Simmons]

> > Strange, it's defined in drivers/video/fbmem.c in my copy of 2.5.53.
> 
> Ah.  It's not exported, so fbcon as a module fails.
> Not sure how I missed it with a grep...

That problem is fixed already in the latest BK tree. I will push today the 
most recent fixes.

Re: [Linux-fbdev-devel] [FB PATCH]

[James Simmons]

> I got link errors due to a missing fb_blank function.  The best
> I can figure is that you were intending to call the fb_blank 
> fbops hook.  Certainly that seems to work.
> 
> I'm not sure how no one else ran into this. 

Okay that is weird. Yes it was suppose to call fb_blank the function.This 
function exist in fbmem.c and it is even exported for modular fbdev 
drivers.