From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/3] arm: add interrupt support
Date: Wed, 24 Sep 2014 09:20:04 +0200 [thread overview]
Message-ID: <E1XWgrt-0007b3-Az@janus> (raw)
In-Reply-To: <CAMhH57TKeka8T216inP1ku0vKXidZjvCz-i+XiXb3mYHFzBN5g@mail.gmail.com>
Hi Masahiro,
On Tue, 23 Sep 2014 12:17:28 +0900, Masahiro YAMADA
<yamada.m@jp.panasonic.com> wrote:
> Hi Georges,
>
> 2014-09-23 3:24 GMT+09:00 Georges Savoundararadj <savoundg@gmail.com>:
> > Le 22/09/2014 03:35, Masahiro Yamada a ?crit :
> >>
> >> On Sun, 21 Sep 2014 23:33:47 +0200
> >> Georges Savoundararadj<savoundg@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.
next prev parent reply other threads:[~2014-09-24 7:20 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-21 21:33 [U-Boot] [PATCH 0/3] arm: add interrupt support Georges Savoundararadj
2014-09-21 21:33 ` [U-Boot] [PATCH 1/3] arm: make .vectors section allocatable Georges Savoundararadj
2014-09-24 7:34 ` Albert ARIBAUD
2014-09-25 20:11 ` Georges Savoundararadj
2014-09-21 21:33 ` [U-Boot] [PATCH 2/3] arm1176: move exception vectors after relocation Georges Savoundararadj
2014-09-21 21:33 ` [U-Boot] [PATCH 3/3] arm: enable_interrupts: set sp in IRQ/FIQ modes Georges Savoundararadj
2014-09-22 1:35 ` [U-Boot] [PATCH 0/3] arm: add interrupt support Masahiro Yamada
2014-09-22 18:24 ` Georges Savoundararadj
2014-09-23 3:17 ` Masahiro YAMADA
2014-09-24 7:20 ` Albert ARIBAUD [this message]
2014-09-24 7:22 ` Albert ARIBAUD
2014-09-27 19:48 ` [U-Boot] [PATCH v2 0/3] arm: fix exception handling Georges Savoundararadj
2014-09-27 19:48 ` [U-Boot] [PATCH v2 1/3] arm: make .vectors section allocatable Georges Savoundararadj
2014-09-27 19:48 ` [U-Boot] [PATCH v2 2/3] arm: relocate the exception vectors Georges Savoundararadj
2014-10-11 10:47 ` Albert ARIBAUD
2014-10-14 20:02 ` Georges Savoundararadj
2014-10-14 22:11 ` Albert ARIBAUD
2014-10-20 21:08 ` Georges Savoundararadj
2014-10-21 5:41 ` Masahiro Yamada
2014-10-21 14:05 ` Albert ARIBAUD
2014-10-22 9:50 ` Masahiro Yamada
2014-10-22 20:29 ` Georges Savoundararadj
2014-10-21 13:54 ` Albert ARIBAUD
2014-10-22 9:54 ` Masahiro Yamada
2014-10-22 20:52 ` Georges Savoundararadj
2014-09-27 19:58 ` [U-Boot] [PATCH v2 3/3] arm: interrupt_init: set sp in IRQ/FIQ modes Georges Savoundararadj
2014-10-26 22:25 ` [U-Boot] [PATCH v3 0/4] arm: fix exception handling Georges Savoundararadj
2014-10-26 22:25 ` [U-Boot] [PATCH v3 1/4] kconfig: arm: introduce symbol for ARM CPUs Georges Savoundararadj
2014-10-27 16:50 ` Masahiro YAMADA
2014-10-27 17:09 ` Albert ARIBAUD
2014-10-27 17:23 ` Masahiro YAMADA
2014-10-28 7:01 ` Albert ARIBAUD
2014-10-28 16:10 ` Masahiro YAMADA
2014-10-28 15:57 ` Masahiro YAMADA
2014-10-28 18:58 ` Georges Savoundararadj
2014-10-26 22:25 ` [U-Boot] [PATCH v3 2/4] arm: make .vectors section allocatable Georges Savoundararadj
2014-10-26 23:20 ` Albert ARIBAUD
2014-10-26 22:25 ` [U-Boot] [PATCH v3 3/4] arm: relocate the exception vectors Georges Savoundararadj
2014-10-26 22:25 ` [U-Boot] [PATCH v3 4/4] arm: interrupt_init: set sp in IRQ/FIQ modes Georges Savoundararadj
2014-10-26 23:16 ` Albert ARIBAUD
2014-10-26 23:32 ` Albert ARIBAUD
2014-10-28 22:16 ` [U-Boot] [PATCH v4 0/4] arm: fix exception handling Georges Savoundararadj
2014-10-28 22:16 ` [U-Boot] [PATCH v4 1/4] kconfig: arm: introduce symbol for ARM CPUs Georges Savoundararadj
2014-10-29 8:07 ` Masahiro Yamada
2014-10-29 12:37 ` Tom Rini
2014-10-29 12:50 ` Albert ARIBAUD
2014-10-29 15:22 ` Tom Rini
2014-10-28 22:16 ` [U-Boot] [PATCH v4 2/4] arm: make .vectors section allocatable Georges Savoundararadj
2014-10-29 12:49 ` Albert ARIBAUD
2014-10-29 15:22 ` Tom Rini
2014-10-28 22:16 ` [U-Boot] [PATCH v4 3/4] arm: relocate the exception vectors Georges Savoundararadj
2014-10-29 12:49 ` Albert ARIBAUD
2014-10-29 15:22 ` Tom Rini
2014-10-28 22:16 ` [U-Boot] [PATCH v4 4/4] arm: interrupt_init: set sp in IRQ/FIQ modes Georges Savoundararadj
2014-10-29 12:49 ` Albert ARIBAUD
2014-10-29 15:22 ` Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=E1XWgrt-0007b3-Az@janus \
--to=albert.u.boot@aribaud.net \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.