[U-Boot] [PATCH v3 0/4] Add support new arch: c6x

[Tom Rini]

On Thu, Jun 28, 2012 at 09:43:57PM +0400, Dmitry Bondar wrote:
> On Mon, 25/06/2012 at 15:31 -0700, Tom Rini wrote:
> > On Mon, Jun 25, 2012 at 10:02:37PM +0400, Dmitry Bondar wrote:
> > 
> > > C6X (C6000) is family of fixed and floating-point DSPs manufactured by Texas Instruments.
> > > ---
> > > Changes for v2:
> > > 	- reset_c.c rewritten with readl/writel.
> > >  	- Tom Rini <trini at ti.com> added to cc
> > > 
> > > Changes for v3:
> > > 	- fix coding style
> > > 	- arch/c6x/include/asm/io.h - add memory barriers on readl/writel ()
> > > 
> > > 
> > > Dmitry Bondar (4):
> > >   Add support new arch: c6x
> > >   c6x: Add support c674x CPUs
> > >   c6x: Add suport build davinci SOC with c674x CPUs
> > >   c6x: Add support c6745-som board
> > 
> > I found a good toolchain and gave building the series a whirl, and
> > here's a few things that need fixing:
> > - The davinci symlink isn't safe, running make again results in ln
> >   complaining.
> > - The symlink doesn't work in conjunction with separate object
> >   directories as it is in the main src tree.
> > - With O= we get arch/arm/cpu/arm926ejs/davinci/libdavinci.o because of
> >   the above, which makes link fail.
> 
> May be i must copy pinmux.c,reset.c,timer.c,cpu.c from
> arch/arm/arm926ejs/davinci to arch/c6x/c674x/c674x, remove from them all
> not used ifdef (#ifdef CONFIG_SOC_DM355) and don't set in boards.cfg SOC
> for board?
> This allow don't use symlink.

No, I think you just need to rework the mkconfig logic you're adding.
It should be, roughly, outside of the ! -z ${soc} check:
If c6x and davinci
  rm -f arch/c6x/cpu/c674x/davinci
  rm -f asm/arch
  if srctree != objtree
    set symlinks one way
  else
    set symlinks another way
  fi
fi

Then once that works, see if you can shrink the logic down to just set a
prefix variable or two and always set the symlinks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120628/29680a47/attachment.pgp>