From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from arroyo.ext.ti.com ([192.94.94.40]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1Rn85X-0002q8-G6 for barebox@lists.infradead.org; Tue, 17 Jan 2012 12:24:32 +0000 From: "Premi, Sanjeev" Date: Tue, 17 Jan 2012 12:24:26 +0000 Message-ID: References: <1326709100-24106-1-git-send-email-s.hauer@pengutronix.de> <20120116115228.GR5446@pengutronix.de> <20120116154626.GS5446@pengutronix.de> <20120117121246.GY5446@pengutronix.de> In-Reply-To: <20120117121246.GY5446@pengutronix.de> Content-Language: en-US MIME-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: barebox-bounces@lists.infradead.org Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: RE: [PATCH] omap3 towards thumb2 support To: Sascha Hauer Cc: "barebox@lists.infradead.org" > -----Original Message----- > From: Sascha Hauer [mailto:s.hauer@pengutronix.de] > Sent: Tuesday, January 17, 2012 5:43 PM > To: Premi, Sanjeev > Cc: barebox@lists.infradead.org > Subject: Re: [PATCH] omap3 towards thumb2 support > > On Tue, Jan 17, 2012 at 11:14:31AM +0000, Premi, Sanjeev wrote: [snip] > > > > > > I attached my binary MLO file. Could you check if it > works for you? I > > > just checked this binary and it comes up with: > > > > > > [sp] I will try tomorrow morning. > > > > Sascha, > > Your MLO boots fine. But I couldn't make repeat success on either > > beagle or the omap3evem on my side. > > > > Each time there is an exception on entry of setup_auxcr() on both > > boards. See attached image. > > > > I noticed that the size of the generated MLO for beagle > were different > > for us - 35224(yours) vs. 35572(mine). > > > > I am building with CodeSourcery 2011.03-41. > > Ok, got it. Please replace the setup_auxcr patch with the > following one > (I also updated the work/omap3 branch). The problem was the missing > ENDPROC(setup_auxcr) directive which tells the assembler that > setup_auxcr is a function. With this the function is correctly called > with 'blx' instead of 'bl'. 'blx' is needed because setup_auxcr is > compiled in arm mode instead of thumb2 mode. Somehow my toolchain > got that right without ENDPROC but the codesourcery toolchain didn't. > With this patch the binary also works with the codesourcery toolchain. > I replaced the contents of auxcr.S and I am able to boot fine on the OMAP3EVM. BTW, I noticed that this file has no license header. Is this okay? ~sanjeev _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox