[PATCH] PowerPC: MPIC ack interrupts at mpic_teardown_this_cpu()
olof at lixom.net
Sat Apr 5 06:39:42 EST 2008
On Fri, Apr 04, 2008 at 08:55:01PM +0200, Segher Boessenkool wrote:
>>> platforms/maple/setup.c and platforms/powermac/pic.c use
>>> Is there a reason why we don't do that on every MPIC? If there is
>>> a reason, the default should be to reset, only pseries and chrp and
>>> and now pasemi do not use it. It's the only sane way to get an MPIC
>>> a sane known state starting from <whatever> state.
>> I used to have the reset there on pasemi but due to some other issues I
>> ended up taking it out.
>> It's not like doing a final EOI is a big deal, I really don't see why
>> there's hassle about this patch.
> The patch is fine, don't get me wrong. I'm just saying MPIC_WANTS_RESET
> should be the default since pretty much everything wants it. The few
> platforms that don't can document why; with the current code, we cannot
> see if you just forgot it or not.
It was documented in the changeset, see de0c74e7f98d8a06b784e120103eb388da8194f6.
But yeah, that's hard to tell at some point later in time, a short
comment by the call could be useful. I'll add one next time I touch that
More information about the Linuxppc-dev