Re: [patch 02/66] irqchip/gic: Convert to hotplug state machine
From: Ingo Molnar
Date: Mon Jul 11 2016 - 10:34:40 EST
* Anna-Maria Gleixner <anna-maria@xxxxxxxxxxxxx> wrote:
> More or less straightforward, although this driver sports some very
> interesting SMP setup code. Regarding the callback ordering, this
> deleted comment is interesting:
>
> ... the GIC needs to be up before the ARM generic timers.
>
> That comment is half baken as the same requirement is true for perf.
>
> Signed-off-by: Richard Cochran <rcochran@xxxxxxxxxxxxx>
> Reviewed-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>
> Cc: Jason Cooper <jason@xxxxxxxxxxxxxx>
> Cc: Marc Zyngier <marc.zyngier@xxxxxxx>
> Signed-off-by: Anna-Maria Gleixner <anna-maria@xxxxxxxxxxxxx>
The SOB sequence is unusal. Is this patch missing this line:
From: Richard Cochran <rcochran@xxxxxxxxxxxxx>
... or should the SOB be:
Acked-by: Richard Cochran <rcochran@xxxxxxxxxxxxx>
?
Thanks,
Ingo