Thread (22 messages) flat view 22 messages, 6 authors, 2012-08-29

[v3] Add basic address decoding support for Marvell 370/XP

From: sebastian.hesselbarth@gmail.com (Sebastian Hesselbarth)
Date: 2012-08-21 12:45:23

On 8/21/12, Thomas Petazzoni [off-list ref] wrote:
Ok, I will look at your code, but the purpose of
PLAT_ORION/PLAT_ORION_LEGACY was not only to re-use the addr-map code,
but also to allow drivers to access the header files containing their
platform_data structure and other stuff (i.e, all the header files from
arch/arm/plat-orion/include/). How do you handle this problem if
MACH_MVEBU is not a PLAT_ORION platform?
Hmm, ok I wasn't aware of that. But I have a bad feeling about joining
PLAT_ORION into MACH_MVEBU if you want to have ORION replaced
by MVEBU in the future. Maybe it is better here to put all you need into
mach-mvebu/include/mach/legacy.h that can be easily abandoned when
everything is ported to something more general.

I know it is code duplication but it is only for the purpose of removing
PLAT_ORION later on.

Sebastian
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help