All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.