* [PATCH] drivers/ide/ppc/mpc8xx.c - volatile fix
@ 2004-12-11 14:48 Magnus Damm
0 siblings, 0 replies; only message in thread
From: Magnus Damm @ 2004-12-11 14:48 UTC (permalink / raw)
To: linuxppc-dev, linuxppc-embedded
[-- Attachment #1: Type: text/plain, Size: 377 bytes --]
Hello,
I fixed this in a local tree for a customer half a year ago (mvl-3.1)
and forgot to report it to the lists. Now I came across it in vanilla
2.5.10-rc3. I have not tested the fix but it is pretty obvious that
access with a non-volatile pointer to hardware is bad, at least with
gcc-3 or above.
Please apply. Or ignore if already fixed. =)
Thanks.
/ magnus (needs CC)
[-- Attachment #2: linux-2.6.9-mpc8xx-volatile.patch --]
[-- Type: application/octet-stream, Size: 377 bytes --]
--- linux-2.6.9/drivers/ide/ppc/mpc8xx.c 2004-12-11 13:05:09.000000000 +0100
+++ linux-2.6.9-mpc8xx-volatile/drivers/ide/ppc/mpc8xx.c 2004-12-11 15:49:16.000000000 +0100
@@ -172,7 +172,7 @@
volatile pcmcia_win_t *win;
volatile pcmconf8xx_t *pcmp;
- uint *pgcrx;
+ volatile uint *pgcrx;
u32 pcmcia_phy_base;
u32 pcmcia_phy_end;
static unsigned long pcmcia_base = 0;
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2004-12-11 14:54 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-12-11 14:48 [PATCH] drivers/ide/ppc/mpc8xx.c - volatile fix Magnus Damm
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.