[U-Boot] [PATCH 0/3] arm: add interrupt support

Albert ARIBAUD albert.u.boot at aribaud.net
Wed Sep 24 09:20:04 CEST 2014


Hi Masahiro,

On Tue, 23 Sep 2014 12:17:28 +0900, Masahiro YAMADA
<yamada.m at jp.panasonic.com> wrote:

> Hi Georges,
> 
> 2014-09-23 3:24 GMT+09:00 Georges Savoundararadj <savoundg at gmail.com>:
> > Le 22/09/2014 03:35, Masahiro Yamada a écrit :
> >>
> >> On Sun, 21 Sep 2014 23:33:47 +0200
> >> Georges Savoundararadj<savoundg at gmail.com>  wrote:
> >>
> >>> Hi folks,
> >>>
> >>> I wanted to use interrupt on U-Boot on my Raspberry Pi but I have
> >>> found that it did not work properly.
> >>>
> >>> My patches intend to make interrupt work.
> >>
> >>
> >> I am not sure if interrupt feature is necessary for a boot loader.
> >
> > Of course, It is not necessary.
> >
> > But for U-boot, I think it is still important to handle exception correctly
> > even if interrupts are not used. Don't you?
> >
> > For instance, without these patches an undefined instruction or any other
> > exception causes an unexpected behavior. It is better to handle properly
> > these cases (dump registers and reset CPU, see functions do_* in
> > arch/arm/lib/interrupts.c).
> > My patches should better be considered as *bugfixes*.
> >
> 
> Sorry, totally my misunderstanding.
> (I thought you were trying to implement a full interrupt feature.)
> 
> As long as they are bug fixes of exception handlers, they are fine.
> 
> I should have read your patches more carefully before my response.
> Again, my apologies..

But then, the patch series needs at east to be reworded, since it does
mention interrupts where actually all exceptions are involved.

Note to Georges: I am reading the patches right now and will comment on
them.

> Best Regards
> Masahiro Yamada

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list