lkml.org 
[lkml]   [2014]   [Aug]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/4] mfd: arizona: Add additional dummy IRQ callbacks
On Thu, Aug 21, 2014 at 12:56:31PM +0100, Lee Jones wrote:
> On Tue, 12 Aug 2014, Charles Keepax wrote:
> > static struct irq_chip arizona_irq_chip = {
> > .name = "arizona",
> > - .irq_disable = arizona_irq_disable,
> > - .irq_enable = arizona_irq_enable,
> > + .irq_disable = arizona_irq_dummy,
> > + .irq_enable = arizona_irq_dummy,
> > + .irq_ack = arizona_irq_dummy,
> > + .irq_mask = arizona_irq_dummy,
> > + .irq_unmask = arizona_irq_dummy,
>
> If you provide .irq_enable(), then .irq_unmask becomes redundant
> and/or is checked for before invoking. There is a chance of
> .irq_mask() being called, but if this is a problem, it should be fixed
> in the IRQ Chip code. There is also one unprotected invocation of
> .irq_ack(), but I think this should be fixed rather than forcing each
> user of IRQ Chip to provide all of these call-backs.

Cool I will look at doing some fixups in the IRQ code and see
where that gets me to.

Thanks,
Charles


\
 
 \ /
  Last update: 2014-08-21 14:41    [W:0.831 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site