* [PATCH] sparc: fix drivers/video/tcx.c warning
@ 2008-04-26 3:22 Robert Reif
2008-04-27 5:29 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Robert Reif @ 2008-04-26 3:22 UTC (permalink / raw)
To: sparclinux
[-- Attachment #1: Type: text/plain, Size: 428 bytes --]
Fix compile warning:
CC drivers/video/tcx.o
drivers/video/tcx.c: In function ‘tcx_init_one’:
drivers/video/tcx.c:477: warning: format ‘%lx’ expects type ‘long
unsigned int’, but argument 4 has type ‘resource_size_t’
This was the only sparc driver to use the resource directly in the
printk so I changed it to physbase like the other drivers.
Boot tested on SS4.
Signed-off-by: Robert Reif <reif@earthlink.net>
[-- Attachment #2: tcx.diff.txt --]
[-- Type: text/plain, Size: 717 bytes --]
diff --git a/drivers/video/tcx.c b/drivers/video/tcx.c
index e5a9ddb..fd94dfb 100644
--- a/drivers/video/tcx.c
+++ b/drivers/video/tcx.c
@@ -419,7 +419,7 @@ static int __devinit tcx_init_one(struct of_device *op)
par->mmap_map[6].size = SBUS_MMAP_EMPTY;
}
- par->physbase = 0;
+ par->physbase = op->resource[0].start;
par->which_io = op->resource[0].flags & IORESOURCE_BITS;
for (i = 0; i < TCX_MMAP_ENTRIES; i++) {
@@ -473,7 +473,7 @@ static int __devinit tcx_init_one(struct of_device *op)
printk("%s: TCX at %lx:%lx, %s\n",
dp->full_name,
par->which_io,
- op->resource[0].start,
+ par->physbase,
par->lowdepth ? "8-bit only" : "24-bit depth");
return 0;
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] sparc: fix drivers/video/tcx.c warning
2008-04-26 3:22 [PATCH] sparc: fix drivers/video/tcx.c warning Robert Reif
@ 2008-04-27 5:29 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2008-04-27 5:29 UTC (permalink / raw)
To: sparclinux
From: Robert Reif <reif@earthlink.net>
Date: Fri, 25 Apr 2008 23:22:28 -0400
> Fix compile warning:
>
> CC drivers/video/tcx.o
> drivers/video/tcx.c: In function ‘tcx_init_one’:
> drivers/video/tcx.c:477: warning: format ‘%lx’ expects type ‘long
> unsigned int’, but argument 4 has type ‘resource_size_t’
>
> This was the only sparc driver to use the resource directly in the
> printk so I changed it to physbase like the other drivers.
> Boot tested on SS4.
>
> Signed-off-by: Robert Reif <reif@earthlink.net>
Thanks a lot for this bug fix, patch applied!
ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¬¥ªÜ–)îÇø§¶\x17›¡Ü¨}©ž²Æ zÚ&j:+v‰¨þø\x1e¯ù\x1e®w¥þŠà2ŠÞ™¨èÚ&¢)ß¡«a¶Úÿÿûàz¿äz¹Þ—ú+ƒùšŽŠÝ¢jÿŠwèþ^[f
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-04-27 5:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-04-26 3:22 [PATCH] sparc: fix drivers/video/tcx.c warning Robert Reif
2008-04-27 5:29 ` David Miller
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.