Message ID | 20170411115957.14253-5-liam@networkimprov.net (mailing list archive) |
---|---|
State | Not Applicable, archived |
Headers | show |
* Liam Breck <liam@networkimprov.net> [170411 05:02]: > From: Liam Breck <kernel@networkimprov.net> > > On chip reset, polling loop used udelay(10) which is too short > to be useful. Instead, use usleep_range(100, 200). Good to see this get rid of udelay in favor of usleep: Acked-by: Tony Lindgren <tony@atomide.com> > Cc: Tony Lindgren <tony@atomide.com> > Cc: Hans de Goede <hdegoede@redhat.com> > Signed-off-by: Liam Breck <kernel@networkimprov.net> > --- > drivers/power/supply/bq24190_charger.c | 12 ++++-------- > 1 file changed, 4 insertions(+), 8 deletions(-) > > diff --git a/drivers/power/supply/bq24190_charger.c b/drivers/power/supply/bq24190_charger.c > index a904abb..f581042 100644 > --- a/drivers/power/supply/bq24190_charger.c > +++ b/drivers/power/supply/bq24190_charger.c > @@ -550,16 +550,13 @@ static int bq24190_register_reset(struct bq24190_dev_info *bdi) > if (ret < 0) > return ret; > > + if (v == 0) > + return 0; > - if (!v) > - break; > > - udelay(10); > + usleep_range(100, 200); > } while (--limit); > > - if (!limit) > - return -EIO; > - > - return 0; > + return -EIO; > } > > /* Charger power supply property routines */ > -- > 2.9.3 >
diff --git a/drivers/power/supply/bq24190_charger.c b/drivers/power/supply/bq24190_charger.c index a904abb..f581042 100644 --- a/drivers/power/supply/bq24190_charger.c +++ b/drivers/power/supply/bq24190_charger.c @@ -550,16 +550,13 @@ static int bq24190_register_reset(struct bq24190_dev_info *bdi) if (ret < 0) return ret; + if (v == 0) + return 0; - if (!v) - break; - udelay(10); + usleep_range(100, 200); } while (--limit); - if (!limit) - return -EIO; - - return 0; + return -EIO; } /* Charger power supply property routines */