From: Arun Sharma Use sizeof() in the ioctl encoding, not sizeof(sizeof()). 25-akpm/drivers/video/aty/aty128fb.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff -puN drivers/video/aty/aty128fb.c~sizeof-in-ioctl-fix drivers/video/aty/aty128fb.c --- 25/drivers/video/aty/aty128fb.c~sizeof-in-ioctl-fix Tue Sep 30 14:04:12 2003 +++ 25-akpm/drivers/video/aty/aty128fb.c Tue Sep 30 14:04:12 2003 @@ -2041,9 +2041,9 @@ aty128fb_setcolreg(u_int regno, u_int re #define ATY_MIRROR_CRT_ON 0x00000002 /* out param: u32* backlight value: 0 to 15 */ -#define FBIO_ATY128_GET_MIRROR _IOR('@', 1, sizeof(__u32*)) +#define FBIO_ATY128_GET_MIRROR _IOR('@', 1, __u32*) /* in param: u32* backlight value: 0 to 15 */ -#define FBIO_ATY128_SET_MIRROR _IOW('@', 2, sizeof(__u32*)) +#define FBIO_ATY128_SET_MIRROR _IOW('@', 2, __u32*) static int aty128fb_ioctl(struct inode *inode, struct file *file, u_int cmd, u_long arg, struct fb_info *info) _