Christian, On Wed, Aug 20, 2014 at 9:21 AM, Christian Riesch <christian.rie...@omicron.at> wrote: > Benoît, > > On Tue, Aug 19, 2014 at 8:35 PM, Benoît Thébaudeau > <benoit.thebaudeau....@gmail.com> wrote: >> Commit 41623c9 'arm: move exception handling out of start.S files' missed >> some >> linker scripts. Hence, some boards no longer had exception handling linked >> since >> this commit. Restore the original behavior by adding the .vectors section to >> these linker scripts. >> >> Signed-off-by: Benoît Thébaudeau <benoit.thebaudeau....@gmail.com> >> Cc: Albert ARIBAUD <albert.u.b...@aribaud.net> > > Does this really _fix_ exception handling for all of these boards (as > your message subject states) or just restore the state before commit > 41623c9? I believe exception handling has never worked for most of > these boards, they don't use it. See the discussion in [1], in > particular [2] and [3]. > > Regards, Christian > > [1] http://marc.info/?t=140247105400001&r=1&w=2 > [2] http://marc.info/?l=u-boot&m=140309659906074&w=2 > [3] http://marc.info/?l=u-boot&m=140450616517545&w=2
The issue is that commit 41623c9 silently removed exception handling for many boards. Whether it was working or not for these boards before this commit is another issue. My patch only fixes _missing_ exception handling, not exception handling itself. I had discussed that with Albert here: http://lists.denx.de/pipermail/u-boot/2014-July/183073.html What do you suggest? - Changing the wording? In which way? - Dropping some boards from this patch? Which ones? I think that there should be an explicit commit from the board maintainers doing that, rather than just commit 41623c9. Regards, Benoît _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
