Re: [U-Boot] [PATCH] powerpc/83xx: increment malloc heap size for the MPC832x MDS boards

2012-06-15 Thread Kim Phillips
On Sat, 17 Mar 2012 17:44:00 -0500 Timur Tabi ti...@freescale.com wrote: The malloc buffer is not large enough to hold a flash sector (0x2 bytes) in addition to whatever else it normally holds, so double its size. This fixes a failure trying to save the environment: = save Saving

Re: [U-Boot] [PATCH] powerpc/83xx: increment malloc heap size for the MPC832x MDS boards

2012-03-20 Thread Scott Wood
On 03/18/2012 04:07 AM, Wolfgang Denk wrote: Dear Tabi Timur-B04825, In message CAOZdJXUem_-mgDSqF+p4R0npZQ6qu14G1yexQCBUe2G=nr2...@mail.gmail.com you wrote: Doubling it is kind of aggressive strategy. You know exactly how much free room needs to be guaranteed, so why don't you

Re: [U-Boot] [PATCH] powerpc/83xx: increment malloc heap size for the MPC832x MDS boards

2012-03-20 Thread Wolfgang Denk
Dear Scott Wood, In message 4f68cf30.8000...@freescale.com you wrote: Make your code more robust. The robust thing to do would be to not be stingy with the malloc size, and change all boards to have at least 1 MiB for malloc (except ones with very small amounts of RAM). Maybe have a

Re: [U-Boot] [PATCH] powerpc/83xx: increment malloc heap size for the MPC832x MDS boards

2012-03-18 Thread Wolfgang Denk
Dear Tabi Timur-B04825, In message CAOZdJXUem_-mgDSqF+p4R0npZQ6qu14G1yexQCBUe2G=nr2...@mail.gmail.com you wrote: Doubling it is kind of aggressive strategy. You know exactly how much free room needs to be guaranteed, so why don't you auto-adjust the size? -#define

[U-Boot] [PATCH] powerpc/83xx: increment malloc heap size for the MPC832x MDS boards

2012-03-17 Thread Timur Tabi
The malloc buffer is not large enough to hold a flash sector (0x2 bytes) in addition to whatever else it normally holds, so double its size. This fixes a failure trying to save the environment: = save Saving Environment to Flash... Unable to save the rest of sector (122880) . done Protected

Re: [U-Boot] [PATCH] powerpc/83xx: increment malloc heap size for the MPC832x MDS boards

2012-03-17 Thread Wolfgang Denk
Dear Timur Tabi, In message 1332024240-23286-1-git-send-email-ti...@freescale.com you wrote: The malloc buffer is not large enough to hold a flash sector (0x2 bytes) in addition to whatever else it normally holds, so double its size. This fixes a failure trying to save the environment:

Re: [U-Boot] [PATCH] powerpc/83xx: increment malloc heap size for the MPC832x MDS boards

2012-03-17 Thread Tabi Timur-B04825
On Sat, Mar 17, 2012 at 6:19 PM, Wolfgang Denk w...@denx.de wrote: Doubling it is kind of aggressive strategy.  You know exactly how much free room needs to be guaranteed, so why don't you auto-adjust the size? -#define CONFIG_SYS_MALLOC_LEN        (128 * 1024)    /* Reserved for malloc */