I have a fix in the fbdev BK repository. I will soon begin porting that
driver over to the new api.
. ---
|o_o |
|:_/ | Give Micro$oft the Bird!!!!
// \ \ Use Linux!!!!
(| | )
/'\_ _/`\
\___)=(___/
On Sun, 9 Jun 2002, A Guy Called Tyketto wrote:
>
> I'll throw one in too:
>
> make[2]: Entering directory `/usr/src/linux-2.5.20/drivers/video'
> gcc -Wp,-MD,.aty128fb.o.d -D__KERNEL__ -I/usr/src/linux-2.5.20/include -Wall
> -Wstrict-prototypes -Wno-trigraphs -O2 -fomit-frame-pointer
> -fno-strict-aliasing -fno-common -pipe -mpreferred-stack-boundary=2
> -march=i686 -malign-functions=4 -nostdinc -iwithprefix include
> -DKBUILD_BASENAME=aty128fb -c -o aty128fb.o aty128fb.c
> aty128fb.c: In function `aty128_init':
> aty128fb.c:1775: `con' undeclared (first use in this function)
> aty128fb.c:1775: (Each undeclared identifier is reported only once
> aty128fb.c:1775: for each function it appears in.)
> aty128fb.c:1776: incompatible type for argument 1 of `fb_alloc_cmap'
> aty128fb.c:1662: warning: `size' might be used uninitialized in this function
> make[2]: *** [aty128fb.o] Error 1
> make[2]: Leaving directory `/usr/src/linux-2.5.20/drivers/video'
> make[1]: *** [_subdir_video] Error 2
> make[1]: Leaving directory `/usr/src/linux-2.5.20/drivers'
> make: *** [drivers] Error 2
>
> lines 1775-1776 are:
>
> size = (fb_display[con].var.bits_per_pixel <= 8) ? 256 : 32;
> fb_alloc_cmap(info->fb_info.cmap, size, 0);
>
> fb_display[] seems to be the culprit. It's an array, in which has the
> 'con' field, which is of type int. In all other functions where this array is
> used, 'con' is passed to the function as type int. It seems to be missing for
> this one.
>
> Backing out of the change seems to work, as this was added in the
> 2.5.21 patch, but I'd rather like to know what the proper solution is..
>
> BL.
>
> Relevant parts of config below.
>
> CONFIG_FB=y
> CONFIG_DUMMY_CONSOLE=y
> CONFIG_VIDEO_SELECT=y
> CONFIG_FB_ATY=y
> CONFIG_FB_ATY_CT=y
> CONFIG_FB_RADEON=y
> CONFIG_FB_ATY128=y
> CONFIG_FBCON_ADVANCED=y
> CONFIG_FBCON_CFB8=y
> CONFIG_FBCON_CFB16=y
> CONFIG_FBCON_CFB24=y
> CONFIG_FBCON_CFB32=y
> CONFIG_FONT_8x8=y
> CONFIG_FONT_8x16=y
>
> --
> Brad Littlejohn | Email: tyketto@wizard.com
> Unix Systems Administrator, | tyketto@ozemail.com.au
> Web + NewsMaster, BOFH.. Smeghead! :) | http://www.wizard.com/~tyketto
> PGP: 1024D/E319F0BF 6980 AAD6 7329 E9E6 D569 F620 C819 199A E319 F0BF
>
> -
> 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/
>
-
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/
This archive was generated by hypermail 2b29 : Sat Jun 15 2002 - 22:00:32 EST