From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 9 May 2012 07:13:52 -0700 Subject: [U-Boot] [PATCH 2/2] ARM: omap3: Add CONFIG_SPL_BOARD_INIT for CONFIG_SPL_MMC_SUPPORT In-Reply-To: <4FAA2068.3040306@gmail.com> References: <1336498172-14063-1-git-send-email-trini@ti.com> <1336498172-14063-2-git-send-email-trini@ti.com> <4FAA2068.3040306@gmail.com> Message-ID: <4FAA7BA0.6040000@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 05/09/2012 12:44 AM, Thomas Weber wrote: > Hello Tom & Peter, > > On 05/08/12 19:29, Tom Rini wrote: >> As originally reported against beagleboard we currently have the >> following error message reported in SPL: >> >> U-Boot SPL 2012.04-00020-gb8310b9-dirty (Apr 25 2012 - 18:49:57) >> Texas Instruments Revision detection unimplemented >> OMAP SD/MMC: 0 >> timed out in wait_for_bb: I2C_STAT=1000 >> reading u-boot.img >> .... >> >> The reason for above message is that when booting from MMC, I2C needs to >> be initialized to talk with the TWL4030. On OMAP3 I2C is only >> initalized in SPL if CONFIG_SPL_BOARD_INIT is set. > On Tricorder there is no such message. I thought the I2C to TWL4030 is > initialized by the ROM code. > And why can the SPL read the u-boot.img in the next step? Things must be setup to a certain extent or we couldn't have been loaded by ROM, yes. Since am35x parts are a different family from the omap3 boards, I'll double check what the am3517 evm I have says here as well. For correctness however, we currently are turning on the i2c support in SPL and then not using it. So something should be changed :) -- Tom