Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-15 Thread Stefano Babic
Hi Tim, On 15/05/2014 00:32, Tim Harvey wrote: I figured this one out - it has nothing to do with the order of calling arch_cpu_init() its that the MMDC isn't always 'ready' by the time the BSS is cleared and thus in my failure case the BSS isn't getting entirely cleared which causes the

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-14 Thread Tim Harvey
On Tue, May 13, 2014 at 10:03 PM, Tim Harvey thar...@gateworks.com wrote: On Tue, May 13, 2014 at 9:58 PM, Tim Harvey thar...@gateworks.com wrote: On Wed, May 7, 2014 at 2:29 AM, Stefano Babic sba...@denx.de wrote: Hi Tim, On 06/05/2014 20:18, Tim Harvey wrote: Stefano / York,

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-13 Thread Tim Harvey
On Wed, May 7, 2014 at 2:29 AM, Stefano Babic sba...@denx.de wrote: Hi Tim, On 06/05/2014 20:18, Tim Harvey wrote: Stefano / York, While preparing for a v3 patch series of my IMX6 SPL bootloader, I find that commit dec1861be90c948ea9fb771927d3d26a994d2e20 [1] breaks the above code because

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-13 Thread Tim Harvey
On Tue, May 13, 2014 at 9:58 PM, Tim Harvey thar...@gateworks.com wrote: On Wed, May 7, 2014 at 2:29 AM, Stefano Babic sba...@denx.de wrote: Hi Tim, On 06/05/2014 20:18, Tim Harvey wrote: Stefano / York, While preparing for a v3 patch series of my IMX6 SPL bootloader, I find that commit

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-07 Thread Stefano Babic
Hi Tim, On 06/05/2014 20:18, Tim Harvey wrote: Stefano / York, While preparing for a v3 patch series of my IMX6 SPL bootloader, I find that commit dec1861be90c948ea9fb771927d3d26a994d2e20 [1] breaks the above code because gd is now needed within setup_i2c. I've always been a bit fuzzy

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-07 Thread York Sun
On 05/06/2014 04:35 PM, Tim Harvey wrote: On Tue, May 6, 2014 at 12:11 PM, Jeroen Hofstee dasub...@myspectrum.nl wrote: Hello Tim, snip Since Crt0.S already created gd on the stack before calling board_init_f, can't the assignment of gd not simply be removed? Is there anything special

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-07 Thread Jeroen Hofstee
Hi, On wo, 2014-05-07 at 09:14 -0700, York Sun wrote: snip Since Crt0.S already created gd on the stack before calling board_init_f, can't the assignment of gd not simply be removed? Is there anything special about gdata? That does make sense, but what I find is that York's ocmmit

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-07 Thread Tim Harvey
On Wed, May 7, 2014 at 9:14 AM, York Sun york...@freescale.com wrote: On 05/06/2014 04:35 PM, Tim Harvey wrote: On Tue, May 6, 2014 at 12:11 PM, Jeroen Hofstee dasub...@myspectrum.nl wrote: Hello Tim, snip Since Crt0.S already created gd on the stack before calling board_init_f, can't

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-07 Thread York Sun
On 05/07/2014 01:27 PM, Tim Harvey wrote: On Wed, May 7, 2014 at 9:14 AM, York Sun york...@freescale.com wrote: On 05/06/2014 04:35 PM, Tim Harvey wrote: On Tue, May 6, 2014 at 12:11 PM, Jeroen Hofstee dasub...@myspectrum.nl wrote: Hello Tim, snip Since Crt0.S already created gd on the

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-07 Thread Tim Harvey
On Wed, May 7, 2014 at 1:29 PM, York Sun york...@freescale.com wrote: On 05/07/2014 01:27 PM, Tim Harvey wrote: On Wed, May 7, 2014 at 9:14 AM, York Sun york...@freescale.com wrote: On 05/06/2014 04:35 PM, Tim Harvey wrote: On Tue, May 6, 2014 at 12:11 PM, Jeroen Hofstee dasub...@myspectrum.nl

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-07 Thread York Sun
On 05/07/2014 01:35 PM, Tim Harvey wrote: On Wed, May 7, 2014 at 1:29 PM, York Sun york...@freescale.com wrote: On 05/07/2014 01:27 PM, Tim Harvey wrote: On Wed, May 7, 2014 at 9:14 AM, York Sun york...@freescale.com wrote: On 05/06/2014 04:35 PM, Tim Harvey wrote: On Tue, May 6, 2014 at

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-06 Thread Tim Harvey
On Mon, Apr 28, 2014 at 1:17 PM, Tim Harvey thar...@gateworks.com wrote: Switch to an SPL image. The SPL for Ventana does the following: - setup i2c and read the factory programmed EEPROM to obtain DRAM config and model for board-specific calibration data - configure DRAM per

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-06 Thread York Sun
On 05/06/2014 11:18 AM, Tim Harvey wrote: On Mon, Apr 28, 2014 at 1:17 PM, Tim Harvey thar...@gateworks.com wrote: Switch to an SPL image. The SPL for Ventana does the following: - setup i2c and read the factory programmed EEPROM to obtain DRAM config and model for board-specific

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-06 Thread Jeroen Hofstee
Hello Tim, On di, 2014-05-06 at 11:18 -0700, Tim Harvey wrote: On Mon, Apr 28, 2014 at 1:17 PM, Tim Harvey thar...@gateworks.com wrote: void board_init_f(ulong dummy) { struct ventana_board_info ventana_info; int board_model; /* Set global data pointer. */

Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-06 Thread Tim Harvey
On Tue, May 6, 2014 at 12:11 PM, Jeroen Hofstee dasub...@myspectrum.nl wrote: Hello Tim, On di, 2014-05-06 at 11:18 -0700, Tim Harvey wrote: On Mon, Apr 28, 2014 at 1:17 PM, Tim Harvey thar...@gateworks.com wrote: void board_init_f(ulong dummy) { struct ventana_board_info