public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Masahiro Yamada <yamada.m@jp.panasonic.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH 04/17] kconfig: add defconfig files for all boards
Date: Fri, 28 Mar 2014 11:25:03 +0900	[thread overview]
Message-ID: <20140328112503.DB34.AA925319@jp.panasonic.com> (raw)
In-Reply-To: <CACUy__XwDv5Yi9mUXp3u5zd4gbcY5btMrHgroTv4jU6JHQdQEw@mail.gmail.com>

Hi Daniel,
Sorry not to have followed up earlier. I was sick in bed.


On Mon, 24 Mar 2014 21:35:15 +0100
Daniel Schwierzeck <daniel.schwierzeck@gmail.com> wrote:

> 2014-03-24 7:35 GMT+01:00 Masahiro Yamada <yamada.m@jp.panasonic.com>:
> >
> > Thanks for your effort. I think basicly we're going to the right direction.
> > But I am afraind more careful check is necessary before posting the
> > series.
> >
> >
> > I did build test on your patches on kconfig-support branch.
> > I noticed many boards would be broken:
> >
> > - ARM
> >  cm_t335 am335x_boneblack am335x_evm am335x_evm_nor am335x_evm_norboot
> > am335x_evm_spiboot am335x_evm_uart1 am335x_evm_uart2 am335x_evm_uart3
> > am335x_evm_uart4 am335x_evm_uart5 am335x_evm_usbspl h2200 palmld palmtc
> > palmtreo680 zipitz2 vpac270_ond_256 mx31ads lubbock
> >
> > - AVR32
> > favr-32-ezkit
> >
> > - Blackfin
> >  bf527-ad7160-eval
> >
> > - PowerPC
> > MVBLUE  hermes svm_sc8xx ESTEEM192E ELPT860 FPS850L FPS860L NSCU SM850
> > TK885D TQM823L TQM823L_LCD TQM823M TQM850L TQM850M TQM855L TQM855M
> > TQM860L TQM860M TQM862L TQM862M TQM866M TQM885D TTTech virtlab2 wtk
> > MVSMR GENIETV )
> 
> Those boards have linker errors, but I do not understand yet why. For
> example on ARM the linker complains about multiple definition of
> board_init() and board_eth_init(). The solely difference is that there
> are one or two additional intermediate objects (board/built-in.o and
> if necessary board/VENDOR/built-in.o). But board/built-in.o and
> board/BOARD/built-in.o respectively board/VENDOR/BOARD/built-in.o
> contain the same symbols and object code. The strange thing is that
> the error only affects some boards.

Have you figured out the multiple definition error?

They are nasty ones and have their own linker scripts.   :-(
For cm_t335 board, board/compulab/cm_t335/u-boot.lds needs modifying.

  --- a/board/compulab/cm_t335/u-boot.lds
  +++ b/board/compulab/cm_t335/u-boot.lds
  @@ -19,7 +19,7 @@ SECTIONS
          {
                  *(.__image_copy_start)
                  CPUDIR/start.o (.text*)
  -               board/compulab/cm_t335/built-in.o (.text*)
  +               board/built-in.o (.text*)
                  *(.text*)
          }



Best Regards
Masahiro Yamada

  reply	other threads:[~2014-03-28  2:25 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-17  8:52 [U-Boot] [RFC PATCH 0/17] Version 0 of Kconfig for U-Boot Masahiro Yamada
2014-03-17  8:52 ` [U-Boot] [RFC PATCH 01/17] kconfig: import Kconfig files from Linux v3.13 tag Masahiro Yamada
2014-03-17  8:52 ` [U-Boot] [RFC PATCH 02/17] kconfig: add basic Kconfig files Masahiro Yamada
2014-03-17  8:52 ` [U-Boot] [RFC PATCH 03/17] Do not apply: tools: add gendefconfigs Masahiro Yamada
2014-03-17  8:52 ` [U-Boot] [RFC PATCH 04/17] kconfig: add defconfig files for all boards Masahiro Yamada
2014-03-18  3:08   ` Stephen Warren
2014-03-19  3:16     ` Masahiro Yamada
2014-03-19  3:39       ` Stephen Warren
2014-03-19  4:50         ` Masahiro Yamada
2014-03-19  9:56           ` Wolfgang Denk
2014-03-19 10:51             ` Masahiro Yamada
2014-03-19 12:37               ` Daniel Schwierzeck
2014-03-19 14:20                 ` Wolfgang Denk
2014-03-19 16:06                   ` Tom Rini
2014-03-19 23:48                     ` Masahiro Yamada
2014-03-21 13:41                       ` Tom Rini
2014-03-20  0:11                 ` Masahiro Yamada
2014-03-20 13:17                   ` Daniel Schwierzeck
2014-03-21 18:05                     ` Tom Rini
2014-03-22 17:14                       ` Daniel Schwierzeck
2014-03-24  6:35                         ` Masahiro Yamada
2014-03-24 20:35                           ` Daniel Schwierzeck
2014-03-28  2:25                             ` Masahiro Yamada [this message]
2014-03-28 20:35                               ` Daniel Schwierzeck
2014-03-31  8:56                                 ` Masahiro Yamada
2014-03-19 19:58           ` Stephen Warren
2014-03-19 12:54   ` Tom Rini
2014-03-19 22:58     ` Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 05/17] include: define CONFIG_SPL and CONFIG_TPL as 1 Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 06/17] m68k: define processor family CONFIGs " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 07/17] kconfig: switch over to Kconfig Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 08/17] MAKEALL: adjust for Kconfig Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 09/17] buildman: " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 10/17] kconfig: delete redundant CONFIG_${ARCH} definition Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 11/17] sh: remove redundant CPU family definition Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 12/17] sparc: " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 13/17] m68k: " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 14/17] powerpc: " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 15/17] kbuild: remove CONFIG_SPL/CONFIG_TPL definition in config headers Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 16/17] kconfig: remove old script Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 17/17] kconfig: add CONFIG_CROSS_COMPILE Masahiro Yamada
2014-03-21  2:15 ` [U-Boot] [RFC PATCH 0/17] Version 0 of Kconfig for U-Boot Simon Glass
2014-03-24  5:58   ` Masahiro Yamada
2014-03-24  7:30     ` Wolfgang Denk
2014-03-24  7:52       ` Masahiro Yamada

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140328112503.DB34.AA925319@jp.panasonic.com \
    --to=yamada.m@jp.panasonic.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox