Re: [PATCH/RFC 12/19] OMAP2+: powerdomain: add voltage domain lookup during register

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux