Jean Pihet <jean.pihet@xxxxxxxxxxxxxx> writes: [...] >> @@ -94,6 +95,14 @@ static int _pwrdm_register(struct powerdomain *pwrdm) >> Â Â Â Âif (_pwrdm_lookup(pwrdm->name)) >> Â Â Â Â Â Â Â Âreturn -EEXIST; >> >> + Â Â Â voltdm = voltdm_lookup(pwrdm->voltdm.name); >> + Â Â Â if (!voltdm) { >> + Â Â Â Â Â Â Â pr_err("powerdomain: %s: voltagedomain %s does not exist\n", >> + Â Â Â Â Â Â Â Â Â Â Âpwrdm->name, pwrdm->voltdm.name); >> + Â Â Â Â Â Â Â return -EINVAL; > > Per patch 10/19 some power domains do not have an associated voltage > domain struct filled in. This will result in failure of the power > domain registration. I think this is not expected. > Right. I'll fix up the changelog in patch 10. Now, all powerdomains are required to have a voltage domain. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html