diff options
| author | Rodolfo Giometti <giometti@linux.it> | 2006-05-31 00:26:57 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-05-31 19:27:11 -0400 |
| commit | c05b7f3d12b9455d746b69b7078ed34d777f560b (patch) | |
| tree | fcfb05e9c1398f18ae5c3408c78afdab2117df11 | |
| parent | 5a47d749e3d067e057d276075fed1d91749d3841 (diff) | |
[PATCH] au1100fb: Fix compilation
From: Rodolfo Giometti <giometti@linux.it>
Fix the following warning on compilation:
drivers/video/au1100fb.c: In function `au1100fb_fb_setcolreg':
drivers/video/au1100fb.c:219: warning: ISO C90 forbids mixed declarations and code
drivers/video/au1100fb.c: In function `au1100fb_fb_pan_display':
drivers/video/au1100fb.c:321: warning: ISO C90 forbids mixed declarations and code
drivers/video/au1100fb.c: In function `au1100fb_fb_mmap':
drivers/video/au1100fb.c:387: warning: ISO C90 forbids mixed declarations and code
drivers/video/au1100fb.c: In function `au1100fb_drv_probe':
drivers/video/au1100fb.c:471: warning: unsigned int format, long unsigned int arg (arg 2)
drivers/video/au1100fb.c: At top level:
drivers/video/au1100fb.c:617: warning: initialization from incompatible pointer type
drivers/video/au1100fb.c:618: warning: initialization from incompatible pointer type
Signed-off-by: Rodolfo Giometti <giometti@linux.it>
Signed-off-by: Antonino Daplas <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
| -rw-r--r-- | drivers/video/au1100fb.c | 21 |
1 files changed, 14 insertions, 7 deletions
diff --git a/drivers/video/au1100fb.c b/drivers/video/au1100fb.c index 3d04b2def0f1..789450bb0bc9 100644 --- a/drivers/video/au1100fb.c +++ b/drivers/video/au1100fb.c | |||
| @@ -214,10 +214,13 @@ int au1100fb_setmode(struct au1100fb_device *fbdev) | |||
| 214 | */ | 214 | */ |
| 215 | int au1100fb_fb_setcolreg(unsigned regno, unsigned red, unsigned green, unsigned blue, unsigned transp, struct fb_info *fbi) | 215 | int au1100fb_fb_setcolreg(unsigned regno, unsigned red, unsigned green, unsigned blue, unsigned transp, struct fb_info *fbi) |
| 216 | { | 216 | { |
| 217 | struct au1100fb_device *fbdev = to_au1100fb_device(fbi); | 217 | struct au1100fb_device *fbdev; |
| 218 | u32 *palette = fbdev->regs->lcd_pallettebase; | 218 | u32 *palette; |
| 219 | u32 value; | 219 | u32 value; |
| 220 | 220 | ||
| 221 | fbdev = to_au1100fb_device(fbi); | ||
| 222 | palette = fbdev->regs->lcd_pallettebase; | ||
| 223 | |||
| 221 | if (regno > (AU1100_LCD_NBR_PALETTE_ENTRIES - 1)) | 224 | if (regno > (AU1100_LCD_NBR_PALETTE_ENTRIES - 1)) |
| 222 | return -EINVAL; | 225 | return -EINVAL; |
| 223 | 226 | ||
| @@ -316,9 +319,11 @@ int au1100fb_fb_blank(int blank_mode, struct fb_info *fbi) | |||
| 316 | */ | 319 | */ |
| 317 | int au1100fb_fb_pan_display(struct fb_var_screeninfo *var, struct fb_info *fbi) | 320 | int au1100fb_fb_pan_display(struct fb_var_screeninfo *var, struct fb_info *fbi) |
| 318 | { | 321 | { |
| 319 | struct au1100fb_device *fbdev = to_au1100fb_device(fbi); | 322 | struct au1100fb_device *fbdev; |
| 320 | int dy; | 323 | int dy; |
| 321 | 324 | ||
| 325 | fbdev = to_au1100fb_device(fbi); | ||
| 326 | |||
| 322 | print_dbg("fb_pan_display %p %p", var, fbi); | 327 | print_dbg("fb_pan_display %p %p", var, fbi); |
| 323 | 328 | ||
| 324 | if (!var || !fbdev) { | 329 | if (!var || !fbdev) { |
| @@ -382,10 +387,12 @@ void au1100fb_fb_rotate(struct fb_info *fbi, int angle) | |||
| 382 | */ | 387 | */ |
| 383 | int au1100fb_fb_mmap(struct fb_info *fbi, struct vm_area_struct *vma) | 388 | int au1100fb_fb_mmap(struct fb_info *fbi, struct vm_area_struct *vma) |
| 384 | { | 389 | { |
| 385 | struct au1100fb_device *fbdev = to_au1100fb_device(fbi); | 390 | struct au1100fb_device *fbdev; |
| 386 | unsigned int len; | 391 | unsigned int len; |
| 387 | unsigned long start=0, off; | 392 | unsigned long start=0, off; |
| 388 | 393 | ||
| 394 | fbdev = to_au1100fb_device(fbi); | ||
| 395 | |||
| 389 | if (vma->vm_pgoff > (~0UL >> PAGE_SHIFT)) { | 396 | if (vma->vm_pgoff > (~0UL >> PAGE_SHIFT)) { |
| 390 | return -EINVAL; | 397 | return -EINVAL; |
| 391 | } | 398 | } |
| @@ -467,7 +474,7 @@ int au1100fb_drv_probe(struct device *dev) | |||
| 467 | 474 | ||
| 468 | if (!request_mem_region(au1100fb_fix.mmio_start, au1100fb_fix.mmio_len, | 475 | if (!request_mem_region(au1100fb_fix.mmio_start, au1100fb_fix.mmio_len, |
| 469 | DRIVER_NAME)) { | 476 | DRIVER_NAME)) { |
| 470 | print_err("fail to lock memory region at 0x%08x", | 477 | print_err("fail to lock memory region at 0x%08lx", |
| 471 | au1100fb_fix.mmio_start); | 478 | au1100fb_fix.mmio_start); |
| 472 | return -EBUSY; | 479 | return -EBUSY; |
| 473 | } | 480 | } |
| @@ -595,13 +602,13 @@ int au1100fb_drv_remove(struct device *dev) | |||
| 595 | return 0; | 602 | return 0; |
| 596 | } | 603 | } |
| 597 | 604 | ||
| 598 | int au1100fb_drv_suspend(struct device *dev, u32 state, u32 level) | 605 | int au1100fb_drv_suspend(struct device *dev, pm_message_t state) |
| 599 | { | 606 | { |
| 600 | /* TODO */ | 607 | /* TODO */ |
| 601 | return 0; | 608 | return 0; |
| 602 | } | 609 | } |
| 603 | 610 | ||
| 604 | int au1100fb_drv_resume(struct device *dev, u32 level) | 611 | int au1100fb_drv_resume(struct device *dev) |
| 605 | { | 612 | { |
| 606 | /* TODO */ | 613 | /* TODO */ |
| 607 | return 0; | 614 | return 0; |
