On 01/21/2017 04:50 AM, Stephen Boyd wrote: > On 01/10, Stanimir Varbanov wrote: >> >>> + udelay(1); >>> + >>> + reg = sc->gds_hw_ctrl ? sc->gds_hw_ctrl : sc->gdscr; >>> + ret = gdsc_poll_status(sc, reg, 0); >> >> This should be gdsc_poll_status(sc, reg, true) because after disabling >> hw_control we expect that the GDSC is in power_on state. >> >>> + if (ret) >>> + return ret; >>> } >>> >>> if (sc->pwrsts & PWRSTS_OFF) >>> > > I expect this patch is going for v2? Yes, sorry, coming up shortly. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html