| To: | "David S. Miller" <davem@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] Fix 64bit bugs in dscc44.c |
| From: | Andi Kleen <ak@xxxxxxx> |
| Date: | Wed, 8 Oct 2003 17:55:48 +0200 |
| Cc: | Andi Kleen <ak@xxxxxx>, ncorbic@xxxxxxxxxxx, netdev@xxxxxxxxxxx |
| In-reply-to: | <20031008084205.5c81faac.davem@redhat.com> |
| References: | <20031007175953.GA1802@averell> <20031008084205.5c81faac.davem@redhat.com> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Wed, Oct 08, 2003 at 08:42:05AM -0700, David S. Miller wrote: > On Tue, 7 Oct 2003 19:59:53 +0200 > Andi Kleen <ak@xxxxxx> wrote: > > > ioremap_nocache returns unsigned long, not u32. This makes a difference > > on 64bit hosts. > > Incorrect, like ioremap() it returns "void *". I knew someone would nit pick this ;-) I agree it would have been better to write: "ioremap and ioremap_nocache return pointers and that should be stored in a pointer variable. However when you really want to store them in a integer variable for unknown reasons always use unsigned long, not u32 or int" However that was just too long, so I didn't write it. -Andi |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] Fix 64bit bugs in dscc44.c II, David S. Miller |
|---|---|
| Next by Date: | Re: [PATCH] Fix 64bit bugs in dscc44.c, Andi Kleen |
| Previous by Thread: | Re: [PATCH] Fix 64bit bugs in dscc44.c, David S. Miller |
| Next by Thread: | Re: [PATCH] Fix 64bit bugs in dscc44.c, David S. Miller |
| Indexes: | [Date] [Thread] [Top] [All Lists] |