* Re: [PATCH] irqchip: or1k-pic: Fix interrupt system.
[not found] <20170630113356.GA20194@gmail.com>
@ 2017-06-30 13:04 ` Stafford Horne
2017-06-30 13:09 ` Marc Zyngier
0 siblings, 1 reply; 3+ messages in thread
From: Stafford Horne @ 2017-06-30 13:04 UTC (permalink / raw)
To: Pedro H. Penna
Cc: tglx, jason, marc.zyngier, stefan.kristiansson, jonas, LKML, Openrisc
+CC Kernel List
On Fri, Jun 30, 2017 at 08:33:56AM -0300, Pedro H. Penna wrote:
> Usually, hardware implicitly acknowledges interruts when
> reading them. However, if this is not the case, the IRQ
> gets fired over and over again in the current implementation.
>
> This patch uses the right mask acknowledge function to handle the
> aforementioned situation on or1k processors that interact with
> such kind of hardware.
>
> Signed-off-by: Pedro H. Penna <pedrohenriquepenna@gmail.com>
Acked-by: Stafford Horne
I am ok to take this patch into my tree if no one objects. But if someone
wants to take it through the drive tree I am fine too.
> ---
> drivers/irqchip/irq-or1k-pic.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/irqchip/irq-or1k-pic.c b/drivers/irqchip/irq-or1k-pic.c
> index 6a9a3e7..dd9d5d1 100644
> --- a/drivers/irqchip/irq-or1k-pic.c
> +++ b/drivers/irqchip/irq-or1k-pic.c
> @@ -70,7 +70,7 @@ static struct or1k_pic_dev or1k_pic_level = {
> .name = "or1k-PIC-level",
> .irq_unmask = or1k_pic_unmask,
> .irq_mask = or1k_pic_mask,
> - .irq_mask_ack = or1k_pic_mask,
> + .irq_mask_ack = or1k_pic_mask_ack,
> },
> .handle = handle_level_irq,
> .flags = IRQ_LEVEL | IRQ_NOPROBE,
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] irqchip: or1k-pic: Fix interrupt system.
2017-06-30 13:04 ` [PATCH] irqchip: or1k-pic: Fix interrupt system Stafford Horne
@ 2017-06-30 13:09 ` Marc Zyngier
2017-06-30 13:45 ` Stafford Horne
0 siblings, 1 reply; 3+ messages in thread
From: Marc Zyngier @ 2017-06-30 13:09 UTC (permalink / raw)
To: Stafford Horne, Pedro H. Penna
Cc: tglx, jason, stefan.kristiansson, jonas, LKML, Openrisc
On 30/06/17 14:04, Stafford Horne wrote:
> +CC Kernel List
>
> On Fri, Jun 30, 2017 at 08:33:56AM -0300, Pedro H. Penna wrote:
>> Usually, hardware implicitly acknowledges interruts when
>> reading them. However, if this is not the case, the IRQ
>> gets fired over and over again in the current implementation.
>>
>> This patch uses the right mask acknowledge function to handle the
>> aforementioned situation on or1k processors that interact with
>> such kind of hardware.
>>
>> Signed-off-by: Pedro H. Penna <pedrohenriquepenna@gmail.com>
>
> Acked-by: Stafford Horne
Can you please Ack it with an email address?
>
> I am ok to take this patch into my tree if no one objects. But if someone
> wants to take it through the drive tree I am fine too.
I'll queue it through my tree, as I have some last minute fixes to send
to Thomas.
Thanks,
M.
--
Jazz is not dead. It just smells funny...
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] irqchip: or1k-pic: Fix interrupt system.
2017-06-30 13:09 ` Marc Zyngier
@ 2017-06-30 13:45 ` Stafford Horne
0 siblings, 0 replies; 3+ messages in thread
From: Stafford Horne @ 2017-06-30 13:45 UTC (permalink / raw)
To: Marc Zyngier
Cc: Pedro H. Penna, tglx, jason, stefan.kristiansson, jonas, LKML, Openrisc
On Fri, Jun 30, 2017 at 02:09:32PM +0100, Marc Zyngier wrote:
> On 30/06/17 14:04, Stafford Horne wrote:
> > +CC Kernel List
> >
> > On Fri, Jun 30, 2017 at 08:33:56AM -0300, Pedro H. Penna wrote:
> >> Usually, hardware implicitly acknowledges interruts when
> >> reading them. However, if this is not the case, the IRQ
> >> gets fired over and over again in the current implementation.
> >>
> >> This patch uses the right mask acknowledge function to handle the
> >> aforementioned situation on or1k processors that interact with
> >> such kind of hardware.
> >>
> >> Signed-off-by: Pedro H. Penna <pedrohenriquepenna@gmail.com>
> >
> > Acked-by: Stafford Horne
>
> Can you please Ack it with an email address?
Sorry about that:
Acked-by: Stafford Horne <shorne@gmail.com>
> >
> > I am ok to take this patch into my tree if no one objects. But if someone
> > wants to take it through the drive tree I am fine too.
>
> I'll queue it through my tree, as I have some last minute fixes to send
> to Thomas.
Thanks,
-Stafford
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-06-30 13:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20170630113356.GA20194@gmail.com>
2017-06-30 13:04 ` [PATCH] irqchip: or1k-pic: Fix interrupt system Stafford Horne
2017-06-30 13:09 ` Marc Zyngier
2017-06-30 13:45 ` Stafford Horne
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).