Skip to content

Commit

Permalink
power: supply: bq24190_charger: Longer delay while polling reset flag
Browse files Browse the repository at this point in the history
On chip reset, polling loop used udelay(10) which is too short
to be useful. Instead, use usleep_range(100, 200).

Signed-off-by: Liam Breck <kernel@networkimprov.net>
Acked-by: Tony Lindgren <tony@atomide.com>
Acked-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
  • Loading branch information
Liam Breck authored and Sebastian Reichel committed Apr 13, 2017
1 parent e3ebc38 commit 61489b0
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions drivers/power/supply/bq24190_charger.c
Original file line number Diff line number Diff line change
Expand Up @@ -550,16 +550,13 @@ static int bq24190_register_reset(struct bq24190_dev_info *bdi)
if (ret < 0)
return ret;

if (!v)
break;
if (v == 0)
return 0;

udelay(10);
usleep_range(100, 200);
} while (--limit);

if (!limit)
return -EIO;

return 0;
return -EIO;
}

/* Charger power supply property routines */
Expand Down

0 comments on commit 61489b0

Please sign in to comment.