[PATCH 4/4] ARM: Xilinx: base header files and assembly macros

-----Original Message-----
From: Russell King - ARM Linux [mailto:linux at arm.linux.org.uk]
Sent: Tuesday, February 08, 2011 1:27 AM
To: John Linn
Cc: linux-arm-kernel at lists.infradead.org; catalin.marinas at arm.com;

glikely at secretlab.ca

Subject: Re: [PATCH 4/4] ARM: Xilinx: base header files and assembly

macros

On Mon, Feb 07, 2011 at 05:14:13PM -0700, John Linn wrote:

quoted

I'm assuming you meant to change all addresses of devices (#defines)

to

quoted

use the IOMEM.

I did that and it's ok. You also have to cast them to ints when

putting

quoted

the addresses into
the resource for a platform device.

Hang on.  The reason I mentioned it was because you were passing some
of these into readl() et.al.  readl() takes the mapped cookie, not the
physical address.

Physical addresses should be integer like, virtual addresses should be
pointer like.

Thanks for clarifying.

It seems like it's better to leave the #defines without IOMEM, then use
IOMEM where a variable, 
such as the address of the timer, is initialized with the #define.

I see what you mean with regard to readl taking the cookie but yet
physical addresses are also
needed.

This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.