Messages in this thread |  | | | Date | Wed, 20 Oct 2010 04:02:35 +0100 | | From | Mark Brown <> | | Subject | Re: [PATCH 1/1] power: bq24617: Adding initial charger support |
| |
On Tue, Oct 19, 2010 at 07:17:11PM -0700, rklein@nvidia.com wrote: > From: Rhyland Klein <rklein@nvidia.com> > > Initial checkin adding basic support for the TI BQ24617 battery charger on the > Nvidia Tegra architecture.
Why is this driver dependant on the CPU? I can't see anything in the code that makes it so.
> + if (old_status != -1 && > + old_status != new_status) { > + dev_dbg(&chip->pdev->dev, > + "%s: %i -> %i\n", __func__, old_status, > + new_status); > + kobject_uevent(&chip->power_supply.dev->kobj, KOBJ_CHANGE);
power_supply_changed().
> +static irqreturn_t bq24617_irq_switch(int irq, void *devid) > +{ > + struct bq24617_info *chip = devid; > + > + schedule_work(&chip->ac_work); > + > + return IRQ_HANDLED; > +}
You're looking for a threaded IRQ handler here - use request_threaded_irq() with no primary handler.
|  |