From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 9 May 2012 11:54:21 -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: <4FAABD5D.9000807@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? I think we need this still, on correctness grounds. We include the i2c code (and twl4030 code) to have the MMC driver link and behave as in full U-Boot. I don't see this timed out message on my am3517 evm nor my "omap3evm" with an am37x SOM (iirc anyhow). I do see it on cold power on, on my beagle however, but not warm (which might be why I missed this before). -- Tom