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

Tom Rini
2012-06-28T18:02:45Z

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@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>