[PATCH v4] Add basic address decoding support for Marvell 370/XP
arnd at arndb.de
Tue Sep 11 15:29:39 EDT 2012
On Tuesday 11 September 2012, Thomas Petazzoni wrote:
> Le Tue, 11 Sep 2012 13:10:00 +0000,
> Arnd Bergmann <arnd at arndb.de> a écrit :
> > Excellent!
> > Acked-by: Arnd Bergmann <arnd at arndb.de>
> Related to this patch set, I have a question: wouldn't it make sense to
> make the .virtual field of struct map_desc anvoid __iomem pointer as
> well instead of an unsigned long? This would avoid all the (unsigned
> long) casts in map_descs array definitions, and would be a bit more
> consistent, no?
I've actually tried this before, and got stuck at the point where we set
up mappings that are not __iomem in mm/mmu.c: We have a bunch of calls
to create_mapping that are not at all related to MMIO access, and as Russell
pointed out back then, those should not take an __iomem pointer.
We can probably solve this by changing the prototype for create_mapping to
static void __init create_mapping(unsigned long virtual, unsigned long pfn,
unsigned long length, unsigned int type);
and leave a single type cast in the iotable_init() function. There are
a few callers that we'd have to change manually then:
My feeling is that we can special-case dma_contiguous_remap by introducing
a different helper like
extern void __init create_dma_ready_mapping(unsigned long virtual, size_t length);
and add casts to __iomem for the other four.
More information about the linux-arm-kernel