Re: [PATCH V1 03/12] spmi: pmic-arb: clean up pmic_arb_find_apid function

From: Stephen Boyd
Date: Thu Jul 27 2017 - 21:18:13 EST


On 07/20, Kiran Gunda wrote:
> Clean up the pmic_arb_find_apid() by using the local
> variables to improve the code readability.
>
> Signed-off-by: Kiran Gunda <kgunda@xxxxxxxxxxxxxx>
> ---

Reviewed-by: Stephen Boyd <sboyd@xxxxxxxxxxxxxx>

One nit below:

> break;
>
> regval = readl_relaxed(pmic_arb->cnfg +
> - SPMI_OWNERSHIP_TABLE_REG(apid));
> - pmic_arb->apid_data[apid].owner =
> - SPMI_OWNERSHIP_PERIPH2OWNER(regval);
> + SPMI_OWNERSHIP_TABLE_REG(apid));

This should be 7 spaces and not a tab? Originally looks like it
was 6 spaces

--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project