On Mon 14 Sep 22:26 CDT 2020, Liu Shixin wrote: > Simplify the return expression. > > Signed-off-by: Liu Shixin <liushixin2@xxxxxxxxxx> Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > --- > drivers/power/avs/qcom-cpr.c | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/drivers/power/avs/qcom-cpr.c b/drivers/power/avs/qcom-cpr.c > index bd7c3e48b386..b24cc77d1889 100644 > --- a/drivers/power/avs/qcom-cpr.c > +++ b/drivers/power/avs/qcom-cpr.c > @@ -665,8 +665,6 @@ static int cpr_enable(struct cpr_drv *drv) > > static int cpr_disable(struct cpr_drv *drv) > { > - int ret; > - > mutex_lock(&drv->lock); > > if (cpr_is_allowed(drv)) { > @@ -676,11 +674,7 @@ static int cpr_disable(struct cpr_drv *drv) > > mutex_unlock(&drv->lock); > > - ret = regulator_disable(drv->vdd_apc); > - if (ret) > - return ret; > - > - return 0; > + return regulator_disable(drv->vdd_apc); > } > > static int cpr_config(struct cpr_drv *drv) > -- > 2.25.1 >