On 06/15/2016 06:16 AM, Linus Walleij wrote: > On Tue, Jun 14, 2016 at 8:55 PM, Stephen Boyd <sboyd@xxxxxxxxxxxxxx> wrote: > >> >> Just leave these out? RPM shouldn't be touching s0 or s1. > Really? The msm-3.4 tree does: > > /* RPM early regulator constraints */ > static struct rpm_regulator_init_data > rpm_regulator_early_init_data[] = { > /* ID a_on pd ss min_uV max_uV init_ip freq */ > RPM_SMPS(PM8058_S0, 0, 1, 1, 500000, 1325000, SMPS_HMIN, 1p60), > RPM_SMPS(PM8058_S1, 0, 1, 1, 500000, 1250000, SMPS_HMIN, 1p60), > > Are you sure you don't mean s0+s1 on PM8901 (which I left out)? > Ah yes, I meant pm8901. -- 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-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html