From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Kocialkowski Date: Fri, 12 Jun 2015 13:32:56 +0200 Subject: [U-Boot] [PATCH v2 0/8] omap-common: Common boot code OMAP3 support and SYS_BOOT-based fallback boot device In-Reply-To: <20150610113556.GE31214@pali> References: <1433351007-1704-2-git-send-email-contact@paulk.fr> <1433798667-9171-1-git-send-email-contact@paulk.fr> <201506091828.29152@pali> <201506092034.26048@pali> <1433930040.2418.3.camel@aldrin> <20150610103439.GC31214@pali> <20150610104246.GD31214@pali> <1433933891.2418.11.camel@aldrin> <20150610113556.GE31214@pali> Message-ID: <1434108776.4299.2.camel@collins> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Le mercredi 10 juin 2015 ? 13:35 +0200, Pali Roh?r a ?crit : > On Wednesday 10 June 2015 12:58:11 Paul Kocialkowski wrote: > > Le mercredi 10 juin 2015 ? 12:42 +0200, Pali Roh?r a ?crit : > > > On Wednesday 10 June 2015 12:34:39 Pali Roh?r wrote: > > > > On Wednesday 10 June 2015 11:54:00 Paul Kocialkowski wrote: > > > > > Le mardi 09 juin 2015 ? 20:34 +0200, Pali Roh?r a ?crit : > > > > > > On Tuesday 09 June 2015 18:28:29 Pali Roh?r wrote: > > > > > > > On Monday 08 June 2015 23:24:18 Paul Kocialkowski wrote: > > > > > > > > I would be very glad to see board maintainers give a go > > > > > > > > at the changeset before it gets merged, especially on devices like > > > > > > > > the Nokia RX-51 (N900) where some specific adaptation was needed. > > > > > > > > > > > > > > So U-Boot is broken since e11c6c279d823dc0d2f470c5c2e3c0a9854a640f > > > > > > > (see other email thread). Until somebody fix that broken commit, I > > > > > > > cannot test your new patches in qemu or on (real) Nokia N900. > > > > > > > > > > > > Now I tested this patch series on top of u-boot master with applied my > > > > > > patch "Nokia RX-51: Fix calculating return address in save_boot_params". > > > > > > > > > > > > And it really as I thought broke booting U-Boot on Nokia N900 in qemu. > > > > > > > > > > I'm confused here -- did you try booting on the actual device or in > > > > > qemu? > > > > > > > > > > > > > I tested your changes only in qemu. But because you removed (or better > > > > masked) required lowlevel asm code, it will not work on real n900 too. > > > > > > > > > > So this patch series is NAK from my side. > > > > > > > > > > Thanks for testing. Of course, the point is to make another version of > > > > > the patch set that fits the N900 too, since we really need to integrate > > > > > the omap3 to the common omap boot mechanism anyways. > > > > > > > > > > > > > Make sure that asm function in rx51 lowlevel asm file is called > > > > immediately from start.S. It is required. Basically no modification to > > > > that function should be needed (maybe just fixing return address in lr). > > > > The problem is that this declaration conflicts with the one from > > omap-common's lowlevel_init.S. I understand that it is not needed for > > the RX-51 since it is not loaded by the U-Boot SPL or by the bootrom. > > > > Yes. > > > Perhaps the best way to do things here would be to have a config option > > clearly stating that it should not expect the booting device information > > structure in r0. This situation is not specific to the RX-51 anyways. > > Sounds good. I just sent out v3 that fixes n900 support the way I suggested. I was able to get my hands on the device and along with the patch you sent out earlier this week, everything seems to work fine. -- Paul Kocialkowski, Replicant developer Replicant is a fully free Android distribution running on several devices, a free software mobile operating system putting the emphasis on freedom and privacy/security. Website: http://www.replicant.us/ Blog: http://blog.replicant.us/ Wiki/tracker/forums: http://redmine.replicant.us/ -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: This is a digitally signed message part URL: