[U-Boot] [PATCH v2 1/4] add TI DA8xx support: Add new directory for da8xx cpu functions

[Wolfgang Denk]

Dear Sandeep,

In message <0554BEF07D437848AF01B9C9B5F0BC5D9380761E at dlee01.ent.ti.com> you wrote:
> 
> > It looks like this patch is duplicating much of what is already
> > in arm926ejs/davinci.  Unless there is a really good reason
> > for a separate cpu dir, this patch should change to merge
> > with davinci.
> Tom,
> 
> There was a similar discussion in the Linux-Davinci ML when DA8xx
> was being added. Finally as things stand currently, we re use most of
> the stuff from DaVinci for DA8xx, i.e DM355,Dm365,DM6467,DM6446 and
> DA8xx are all considered DaVinci SOC's

Avoiding code duplication is definitely a good thing.

> If we do the same thing in U-Boot, then the DA8xx patches submitted will need re organization.

Please do this, then. I don't want to see the code duplicated without
really good reasons.

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
"There is no statute of limitations on stupidity."
- Randomly produced by a computer program called Markov3.