[U-Boot] [PATCH 04/11 v1] ARM: OMAP3: Add assembly startup and sys_info common files

[Wolfgang Denk <wd@denx.de>]

Dear Dirk Behme,

In message <48CFF803.3080105@googlemail.com> you wrote:
>
> Yes, from strict technical point of view you are correct. Cortex A8 is 
> the 'CPU' and OMAP3 is the SoC.
> 
> But what do we win adding an additional directory layer 
> cpu/arm_cortexa8/omap3? Let's assume we will see an additional 
> Cortex-A8 core xxx in the future. Then we would have
> 
> a)
> 
> cpu/omap3/
> cpu/xxx/

... which probably means code duplication.

> *or*
> 
> b)
> 
> cpu/arm_cortexa8/omap3
> cpu/arm_cortexa8/xxx
> 
> What's better?

Instead, we might probably have just a "cpu/arm_cortexa8/" with common
code for "omap3" and "xxx" ?

> If U-Boot maintainers want cpu/arm_cortexa8/xxx, I wonder if we could 
> do this using git to move the files once the patches are applied? It 

I think we shoud try and start without the xxx part.


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The human race is a race of cowards; and I am not  only  marching  in
that procession but carrying a banner.                   - Mark Twain