Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-12-01 Thread Stefano Babic
Hi Marek, On 01/12/2015 00:31, Marek Vasut wrote: >> >> Perhaps the use case for a >1MB malloc area is falcon mode. While a >> 1MB area certainly works for loading and bootstrapping to u-boot.img I >> have used anywhere from 1MB to 8MB images for falcon mode. > > Does the falcon mode load the im

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-12-01 Thread Marek Vasut
On Tuesday, December 01, 2015 at 03:13:17 PM, Tim Harvey wrote: > On Mon, Nov 30, 2015 at 3:31 PM, Marek Vasut wrote: > >> Perhaps the use case for a >1MB malloc area is falcon mode. While a > >> 1MB area certainly works for loading and bootstrapping to u-boot.img I > >> have used anywhere from 1M

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-12-01 Thread Tim Harvey
On Mon, Nov 30, 2015 at 3:31 PM, Marek Vasut wrote: >> >> Perhaps the use case for a >1MB malloc area is falcon mode. While a >> 1MB area certainly works for loading and bootstrapping to u-boot.img I >> have used anywhere from 1MB to 8MB images for falcon mode. > > Does the falcon mode load the im

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-30 Thread Marek Vasut
On Monday, November 30, 2015 at 07:56:55 PM, Tim Harvey wrote: > On Wed, Nov 25, 2015 at 5:32 AM, Marek Vasut wrote: > > On Wednesday, November 25, 2015 at 02:15:13 PM, Przemyslaw Marczak wrote: > >> On 11/25/2015 01:16 PM, Marek Vasut wrote: > >> > On Wednesday, November 25, 2015 at 01:00:41 PM,

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-30 Thread Tim Harvey
On Wed, Nov 25, 2015 at 5:32 AM, Marek Vasut wrote: > On Wednesday, November 25, 2015 at 02:15:13 PM, Przemyslaw Marczak wrote: >> On 11/25/2015 01:16 PM, Marek Vasut wrote: >> > On Wednesday, November 25, 2015 at 01:00:41 PM, Przemyslaw Marczak wrote: >> >> Hello Marek, >> >> >> >> On 11/25/2015

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-25 Thread Marek Vasut
On Wednesday, November 25, 2015 at 02:15:13 PM, Przemyslaw Marczak wrote: > On 11/25/2015 01:16 PM, Marek Vasut wrote: > > On Wednesday, November 25, 2015 at 01:00:41 PM, Przemyslaw Marczak wrote: > >> Hello Marek, > >> > >> On 11/25/2015 11:56 AM, Marek Vasut wrote: > >>> On Wednesday, November 2

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-25 Thread Przemyslaw Marczak
On 11/25/2015 01:16 PM, Marek Vasut wrote: On Wednesday, November 25, 2015 at 01:00:41 PM, Przemyslaw Marczak wrote: Hello Marek, On 11/25/2015 11:56 AM, Marek Vasut wrote: On Wednesday, November 25, 2015 at 11:40:36 AM, Przemyslaw Marczak wrote: Hello Tim, Marek On 11/20/2015 10:40 PM, Tim

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-25 Thread Marek Vasut
On Wednesday, November 25, 2015 at 01:00:41 PM, Przemyslaw Marczak wrote: > Hello Marek, > > On 11/25/2015 11:56 AM, Marek Vasut wrote: > > On Wednesday, November 25, 2015 at 11:40:36 AM, Przemyslaw Marczak wrote: > >> Hello Tim, Marek > >> > >> On 11/20/2015 10:40 PM, Tim Harvey wrote: > >>> On

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-25 Thread Przemyslaw Marczak
Hello Marek, On 11/25/2015 11:56 AM, Marek Vasut wrote: On Wednesday, November 25, 2015 at 11:40:36 AM, Przemyslaw Marczak wrote: Hello Tim, Marek On 11/20/2015 10:40 PM, Tim Harvey wrote: On Fri, Nov 20, 2015 at 12:43 PM, Marek Vasut wrote: Using 50 MiB malloc pool in SPL is nonsense. Sinc

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-25 Thread Marek Vasut
On Wednesday, November 25, 2015 at 11:40:36 AM, Przemyslaw Marczak wrote: > Hello Tim, Marek > > On 11/20/2015 10:40 PM, Tim Harvey wrote: > > On Fri, Nov 20, 2015 at 12:43 PM, Marek Vasut wrote: > >> Using 50 MiB malloc pool in SPL is nonsense. Since the caches are not > >> enabled in SPL, it ta

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-25 Thread Przemyslaw Marczak
Hello Tim, Marek On 11/20/2015 10:40 PM, Tim Harvey wrote: On Fri, Nov 20, 2015 at 12:43 PM, Marek Vasut wrote: Using 50 MiB malloc pool in SPL is nonsense. Since the caches are not enabled in SPL, it takes 2 seconds to init the pool and has no obvious benefit. Reduce the size to 1 MiB. Signe

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-20 Thread Tim Harvey
On Fri, Nov 20, 2015 at 12:43 PM, Marek Vasut wrote: > Using 50 MiB malloc pool in SPL is nonsense. Since the caches are not > enabled in SPL, it takes 2 seconds to init the pool and has no obvious > benefit. Reduce the size to 1 MiB. > > Signed-off-by: Marek Vasut > Cc: Stefano Babic > Cc: Tim

Re: [U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-20 Thread Stefano Babic
On 20/11/2015 21:43, Marek Vasut wrote: > Using 50 MiB malloc pool in SPL is nonsense. Since the caches are not > enabled in SPL, it takes 2 seconds to init the pool and has no obvious > benefit. Reduce the size to 1 MiB. You're absolutely right ! Nice catch ! > > Signed-off-by: Marek Vasut > C

[U-Boot] [PATCH] arm: mx6: Reduce SPL malloc pool size

2015-11-20 Thread Marek Vasut
Using 50 MiB malloc pool in SPL is nonsense. Since the caches are not enabled in SPL, it takes 2 seconds to init the pool and has no obvious benefit. Reduce the size to 1 MiB. Signed-off-by: Marek Vasut Cc: Stefano Babic Cc: Tim Harvey --- include/configs/imx6_spl.h | 6 +++--- 1 file changed,