diff mbox

[U-Boot,v2,2/2] README: Add more clarification about CONFIG_SYS_SPL_MALLOC_START

Message ID 1447333811-31720-2-git-send-email-fabio.estevam@freescale.com
State Superseded
Delegated to: Tom Rini
Headers show

Commit Message

Fabio Estevam Nov. 12, 2015, 1:10 p.m. UTC
Add some text to clarify the usage of full malloc (when
CONFIG_SYS_SPL_MALLOC_START is selected) and also that simple malloc()
can be used prior to DRAM initialization if CONFIG_SYS_MALLOC_F is defined.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
Changes since v1:
- Improve README and commit log (Simon)

 README | 3 +++
 1 file changed, 3 insertions(+)

Comments

Albert ARIBAUD Nov. 12, 2015, 1:41 p.m. UTC | #1
Hello Fabio,

On Thu, 12 Nov 2015 11:10:11 -0200, Fabio Estevam
<fabio.estevam@freescale.com> wrote:
> Add some text to clarify the usage of full malloc (when
> CONFIG_SYS_SPL_MALLOC_START is selected) and also that simple malloc()
> can be used prior to DRAM initialization if CONFIG_SYS_MALLOC_F is defined.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> Changes since v1:
> - Improve README and commit log (Simon)

Not sure why you made this a 2-patch series -- a single patch would
have been fine.

Amicalement,
Tom Rini Nov. 12, 2015, 2:29 p.m. UTC | #2
On Thu, Nov 12, 2015 at 02:41:45PM +0100, Albert ARIBAUD wrote:
> Hello Fabio,
> 
> On Thu, 12 Nov 2015 11:10:11 -0200, Fabio Estevam
> <fabio.estevam@freescale.com> wrote:
> > Add some text to clarify the usage of full malloc (when
> > CONFIG_SYS_SPL_MALLOC_START is selected) and also that simple malloc()
> > can be used prior to DRAM initialization if CONFIG_SYS_MALLOC_F is defined.
> > 
> > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > ---
> > Changes since v1:
> > - Improve README and commit log (Simon)
> 
> Not sure why you made this a 2-patch series -- a single patch would
> have been fine.

Indeed.  I'll squash them once Simon acks.
diff mbox

Patch

diff --git a/README b/README
index ef8d437..92d36e6 100644
--- a/README
+++ b/README
@@ -3568,6 +3568,9 @@  FIT uImage format:
 
 		CONFIG_SYS_SPL_MALLOC_START
 		Starting address of the malloc pool used in SPL.
+		When this option is set the full malloc is used in SPL and
+		it is set up by spl_init() and before that, the simple malloc()
+		can be used if CONFIG_SYS_MALLOC_F is defined.
 
 		CONFIG_SYS_SPL_MALLOC_SIZE
 		The size of the malloc pool used in SPL.