Re: FOR COMMENT: void __iomem * and similar casts are Bad News

[Russell King - ARM Linux <linux@arm.linux.org.uk>]

On Wed, Sep 03, 2008 at 02:35:52PM -0700, David Brownell wrote:
> On Wednesday 03 September 2008, Russell King - ARM Linux wrote:
> > - virtual addresses are pointer like.
> > - physical addresses are integer like.
> > 
> > So, if it's a physical address, it should be stored in an integer type
> > large enough to contain it, and that means something like u32, or
> > unsigned long.
> 
> And to complete the puzzle:  in the absense of IOMMU technologies
> what's the difference between a "physical address" and a dma_addr_t?
> (DMA addreses being integer-like.)

That's a valid point, and related, but one which I don't wish to get
into at present (mainly because it isn't a problem on current hardware.)

I'm not out to solve world hunger at this point, just to shake out some
of the real problems that OMAP currently suffers from, which we can use
the standard compiler to find.  That's why I'm trying to keep this thread
narrowly focussed.