Re: [PATCH] mwl8k: interrupt handling changes

[Nishant Sarmukadam <nishants@marvell.com>]

On Fri, 2011-04-08 at 11:19 -0700, Lennert Buytenhek wrote:
> On Fri, Apr 08, 2011 at 02:38:27PM +0530, Nishant Sarmukadam wrote:
> 
> > We do not need to enable all the interrupts in mwl8k_probe_hw.
> > We need to enable only MWL8K_A2H_INT_OPC_DONE interrupt for sending
> > commands to the firmware. Keep the other interrupts masked in
> > mwl8k_probe_hw.
> 
> This part of the description matches the patch.  But..
> 
> 
> > Also, in mwl8k_start, where we expect other interrupts,
> > enable only those interrupts we are interested in.
> 
> ..this comment does not match the code.
> 
> 
> >  	/* Enable interrupts */
> >  	iowrite32(MWL8K_A2H_EVENTS, priv->regs + MWL8K_HIU_A2H_INTERRUPT_MASK);
> > +	iowrite32(MWL8K_A2H_EVENTS,
> > +		  priv->regs + MWL8K_HIU_A2H_INTERRUPT_STATUS_MASK);
> 
> This writes to the status register.  Why does it do that?

Here we are writing to the 'status mask' register and not the 'status'
register. Interrupts will not be enabled until we write 1's in the
status mask register. Earlier, we were writing 0xffffffff in this
register, whereas enabling only interrupts specified in MWL8K_A2H_EVENTS
should be sufficient since driver is interested in only those
interrupts. Hence the description.