All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabio Estevam <festevam@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2] mx7_common: Define CONFIG_SYS_MALLOC_LEN in the board file
Date: Sat, 26 Mar 2016 13:30:49 -0300	[thread overview]
Message-ID: <1459009849-30149-2-git-send-email-festevam@gmail.com> (raw)
In-Reply-To: <1459009849-30149-1-git-send-email-festevam@gmail.com>

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*/
-- 
1.9.1

  reply	other threads:[~2016-03-26 16:30 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 ` Fabio Estevam [this message]
2016-04-03 17:17   ` [U-Boot] [PATCH 2/2] mx7_common: Define CONFIG_SYS_MALLOC_LEN in the board file Stefano Babic
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=1459009849-30149-2-git-send-email-festevam@gmail.com \
    --to=festevam@gmail.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.