From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2] mx7_common: Define CONFIG_SYS_MALLOC_LEN in the board file
Date: Sun, 3 Apr 2016 19:17:36 +0200 [thread overview]
Message-ID: <57015030.1020409@denx.de> (raw)
In-Reply-To: <1459009849-30149-2-git-send-email-festevam@gmail.com>
On 26/03/2016 17:30, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@nxp.com>
>
> Having CONFIG_SYS_MALLOC_LEN in mx7_common.h is not a good idea,
> because the malloc() pool size is board dependent.
>
> For example: if a certain board has support for splashscreen or DFU,
> it may be necessary to adjust CONFIG_SYS_MALLOC_LEN to a larger value.
>
> So define CONFIG_SYS_MALLOC_LEN in each board config file.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
> ---
> include/configs/mx7_common.h | 3 ---
> include/configs/mx7dsabresd.h | 3 +++
> include/configs/warp7.h | 4 ++++
> 3 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/include/configs/mx7_common.h b/include/configs/mx7_common.h
> index 5034464..9bd6114 100644
> --- a/include/configs/mx7_common.h
> +++ b/include/configs/mx7_common.h
> @@ -28,9 +28,6 @@
> /* Enable iomux-lpsr support */
> #define CONFIG_IOMUX_LPSR
>
> -/* Size of malloc() pool */
> -#define CONFIG_SYS_MALLOC_LEN (32 * SZ_1M)
> -
> #define CONFIG_DISPLAY_CPUINFO
> #define CONFIG_DISPLAY_BOARDINFO
>
> diff --git a/include/configs/mx7dsabresd.h b/include/configs/mx7dsabresd.h
> index 988d363..25ed29d 100644
> --- a/include/configs/mx7dsabresd.h
> +++ b/include/configs/mx7dsabresd.h
> @@ -16,6 +16,9 @@
>
> #define CONFIG_MXC_UART_BASE UART1_IPS_BASE_ADDR
>
> +/* Size of malloc() pool */
> +#define CONFIG_SYS_MALLOC_LEN (32 * SZ_1M)
> +
> #define CONFIG_BOARD_EARLY_INIT_F
> #define CONFIG_BOARD_LATE_INIT
>
> diff --git a/include/configs/warp7.h b/include/configs/warp7.h
> index 5ea5512..d1404b2 100644
> --- a/include/configs/warp7.h
> +++ b/include/configs/warp7.h
> @@ -16,6 +16,10 @@
>
> #define CONFIG_MXC_UART_BASE UART1_IPS_BASE_ADDR
>
> +
> +/* Size of malloc() pool */
> +#define CONFIG_SYS_MALLOC_LEN (32 * SZ_1M)
> +
> #define CONFIG_BOARD_EARLY_INIT_F
>
> /* MMC Config*/
>
Applied to u-boot-imx, thanks !
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
next prev parent reply other threads:[~2016-04-03 17:17 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-26 16:30 [U-Boot] [PATCH 1/2] warp7: Pass the UART base definition Fabio Estevam
2016-03-26 16:30 ` [U-Boot] [PATCH 2/2] mx7_common: Define CONFIG_SYS_MALLOC_LEN in the board file Fabio Estevam
2016-04-03 17:17 ` Stefano Babic [this message]
2016-04-03 17:17 ` [U-Boot] [PATCH 1/2] warp7: Pass the UART base definition Stefano Babic
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=57015030.1020409@denx.de \
--to=sbabic@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 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.