On Wed, Mar 25, 2020 at 11:43:16PM +0200, Andy Shevchenko wrote:
> On Mon, Mar 16, 2020 at 06:07:21PM +0200, Andy Shevchenko wrote:
> > Refactor pca954x_irq_handler() to:
> > - use for_each_set_bit() macro
> > - use IRQ_RETVAL() macro
> >
> > Above change makes code easy to read and understand.
>
> Peter, does this series good in your opinion?
Peter, Wolfram, this missed previous cycle, any work needs to be done?
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > ---
> > v2: masked pending to prevent handling of spurious IRQs (Peter)
> > drivers/i2c/muxes/i2c-mux-pca954x.c | 17 +++++++----------
> > 1 file changed, 7 insertions(+), 10 deletions(-)
> >
> > diff --git a/drivers/i2c/muxes/i2c-mux-pca954x.c b/drivers/i2c/muxes/i2c-mux-pca954x.c
> > index a0d926ae3f86..b764c7c746e9 100644
> > --- a/drivers/i2c/muxes/i2c-mux-pca954x.c
> > +++ b/drivers/i2c/muxes/i2c-mux-pca954x.c
> > @@ -327,21 +327,18 @@ static DEVICE_ATTR_RW(idle_state);
> > static irqreturn_t pca954x_irq_handler(int irq, void *dev_id)
> > {
> > struct pca954x *data = dev_id;
> > - unsigned int child_irq;
> > - int ret, i, handled = 0;
> > + unsigned long pending;
> > + int ret, i;
> >
> > ret = i2c_smbus_read_byte(data->client);
> > if (ret < 0)
> > return IRQ_NONE;
> >
> > - for (i = 0; i < data->chip->nchans; i++) {
> > - if (ret & BIT(PCA954X_IRQ_OFFSET + i)) {
> > - child_irq = irq_linear_revmap(data->irq, i);
> > - handle_nested_irq(child_irq);
> > - handled++;
> > - }
> > - }
> > - return handled ? IRQ_HANDLED : IRQ_NONE;
> > + pending = (ret >> PCA954X_IRQ_OFFSET) & (BIT(data->chip->nchans) - 1);
> > + for_each_set_bit(i, &pending, data->chip->nchans)
> > + handle_nested_irq(irq_linear_revmap(data->irq, i));
> > +
> > + return IRQ_RETVAL(pending);
> > }
> >
> > static int pca954x_irq_set_type(struct irq_data *idata, unsigned int type)
> > --
> > 2.25.1
> >
>
> --
> With Best Regards,
> Andy Shevchenko
>
>
--
With Best Regards,
Andy Shevchenko
