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 -- 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
