On Wed, Sep 17, 2014 at 06:17:03PM -0400, Ashwin Chaugule wrote:
> + while (!(readw_relaxed(&generic_comm_base->status) &
> PCC_CMD_COMPLETE)) {
> + if (retries++ < 5)
> + udelay(cmd_delay);
> + else {
> + pr_err("PCC platform did not respond.\n");
> + return false;
> + }
> + }
You should have { } on both branches of the if but otherwise that looks
fine to me (assuming that there are enough retries).
Attachment:
signature.asc
Description: Digital signature
