public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 15/19] dm: powerpc: ppc4xx: Move glacier to use driver model for serial
Date: Mon, 26 Jan 2015 10:23:33 +0100	[thread overview]
Message-ID: <54C60795.9030500@denx.de> (raw)
In-Reply-To: <1418653194-16873-16-git-send-email-sjg@chromium.org>

Hi Simon,

finally I'm testing your patchset. Again sorry for the big delay
here.

On 15.12.2014 15:19, Simon Glass wrote:
> Adjust Kconfig to default to driver model for glacier, canyonlands and
> arches.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
> 
>   board/amcc/canyonlands/Kconfig | 9 +++++++++
>   1 file changed, 9 insertions(+)
> 
> diff --git a/board/amcc/canyonlands/Kconfig b/board/amcc/canyonlands/Kconfig
> index 0fc6877..cbc5ff9 100644
> --- a/board/amcc/canyonlands/Kconfig
> +++ b/board/amcc/canyonlands/Kconfig
> @@ -29,4 +29,13 @@ config ARCHES
>   
>   endchoice
>   
> +config DM
> +	default y
> +
> +config DM_SERIAL
> +	default y
> +
> +config SYS_MALLOC_F_LEN
> +	default 0x400
> +

Hmmm. This does not seem to work. With your patch series applied on
current top-of-tree I get this error (for glacier_ramboot or
canyonlands):

$ make -s -j10
board/amcc/canyonlands/Kconfig:38:warning: config symbol defined without type
drivers/serial/serial-uclass.c:29:2: error: #error "Serial is required before relocation - define CONFIG_SYS_MALLOC_F_LEN to make this work"
make[1]: *** [drivers/serial/serial-uclass.o] Error 1

Somehow the CONFIG_SYS_MALLOC_F_LEN define is not available in the
.config used. Do you have any idea why it might be missing?

Thanks,
Stefan

  reply	other threads:[~2015-01-26  9:23 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-15 14:19 [U-Boot] [PATCH 0/19] powerpc: Introduce device tree control and driver model Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 01/19] Introduce board_init_f_mem() to handle early memory layout Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 02/19] powerpc: Permit device tree control of U-Boot (CONFIG_OF_CONTROL) Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 03/19] powerpc: ppc4xx: canyonlands: config: Tidy up CONFIGs and config.mk Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 04/19] powerpc: ppc4xx: Move CANYONLANDS/GLACIER/ARCHES to Kconfig Simon Glass
2015-01-06  7:29   ` Masahiro Yamada
2014-12-15 14:19 ` [U-Boot] [PATCH 05/19] powerpc: ppc4xx: Add ramboot config for glacier Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 06/19] WIP: powerpc: ppc4xx: Somehow BSS is not cleared in RAMBOOT case Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 07/19] powerpc: ppc4xx: canyonlands: Move to generic board Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 08/19] powerpc: ppc4xx: dts: Bring in canyonlands device tree files Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 09/19] powerpc: ppc4xx: Call board_init_f_mem() for generic board Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 10/19] powerpc: ppc4xx: Add a gpio.h header file Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 11/19] powerpc: ppc4xx: Allow the end of u-boot.bin to be found Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 12/19] powerpc: ppc4xx: Use CONFIG_OF_CONTROL for canyonlands boards Simon Glass
2015-01-26 13:17   ` Stefan Roese
2015-01-26 13:31     ` Simon Glass
2015-01-26 13:39       ` Bin Meng
2014-12-15 14:19 ` [U-Boot] [PATCH 13/19] ppc: amcc: Omit unneeded ns16550 CONFIG if using driver model Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 14/19] powerpc: Add serial driver for " Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 15/19] dm: powerpc: ppc4xx: Move glacier to use driver model for serial Simon Glass
2015-01-26  9:23   ` Stefan Roese [this message]
2015-01-26  9:35     ` Stefan Roese
2015-01-26 13:33       ` Simon Glass
2015-01-26 13:38         ` Stefan Roese
2014-12-15 14:19 ` [U-Boot] [PATCH 16/19] powerpc: Add linkage.h file Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 17/19] serial: Support an early UART for debugging Simon Glass
2014-12-16  4:38   ` Masahiro Yamada
2014-12-17  4:46     ` Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 18/19] serial: ns16550: Support debug UART Simon Glass
2014-12-16  4:39   ` Masahiro Yamada
2014-12-17  4:47     ` Simon Glass
2014-12-15 14:19 ` [U-Boot] [PATCH 19/19] powerpc: ppc4xx: Provide early debug UART defaults Simon Glass
2015-01-24  4:06 ` [U-Boot] [PATCH 0/19] powerpc: Introduce device tree control and driver model Simon Glass

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=54C60795.9030500@denx.de \
    --to=sr@denx.de \
    --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