Re: [PATCH 13/22] OMAP2+: voltage: keep track of powerdomains in each voltagedomain

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

 



Jean Pihet <jean.pihet@xxxxxxxxxxxxxx> writes:

> Hi Kevin,
>
> On Mon, Aug 29, 2011 at 7:35 PM, Kevin Hilman <khilman@xxxxxx> wrote:
>> When a powerdomain is registered and it has an associated voltage domain,
>> add the powerdomain to the voltagedomain using voltdm_add_pwrdm().
>>
>> Also add voltagedomain iterator helper functions to iterate over all
>> registered voltagedomains and all powerdomains associated with a
>> voltagedomain.
>>
>> Modeled after a similar relationship between clockdomains and powerdomains.
>>
>> Signed-off-by: Kevin Hilman <khilman@xxxxxx>
>> ---
>>  arch/arm/mach-omap2/powerdomain.c |    2 +
>>  arch/arm/mach-omap2/powerdomain.h |    2 +
>>  arch/arm/mach-omap2/voltage.c     |   80 +++++++++++++++++++++++++++++++++++++
>>  arch/arm/mach-omap2/voltage.h     |   10 +++++
>>  4 files changed, 94 insertions(+), 0 deletions(-)
>>
> ...
>
>> diff --git a/arch/arm/mach-omap2/voltage.h b/arch/arm/mach-omap2/voltage.h
>> index 966aa88..b41d9f1 100644
>> --- a/arch/arm/mach-omap2/voltage.h
>> +++ b/arch/arm/mach-omap2/voltage.h
>> @@ -19,6 +19,8 @@
>>  #include "vc.h"
>>  #include "vp.h"
>>
>> +struct powerdomain;
>> +
>>  /* XXX document */
>>  #define VOLTSCALE_VPFORCEUPDATE                1
>>  #define VOLTSCALE_VCBYPASS             2
>> @@ -55,12 +57,15 @@ struct omap_vfsm_instance_data {
>>  * @name: Name of the voltage domain which can be used as a unique identifier.
>>  * @scalable: Whether or not this voltage domain is scalable
>>  * @node: list_head linking all voltage domains
>> + * @pwrdm_node: list_head linking all powerdomains in this voltagedomain
>>  * @vdd: to be removed
>> + * @pwrdms: powerdomains in this voltagedomain
>>  */
>>  struct voltagedomain {
>>        char *name;
>>        bool scalable;
>>        struct list_head node;
>> +       struct list_head pwrdm_list;
>>        struct omap_vdd_info *vdd;
>>  };
> The kerneldoc is not reflecting the code for struct voltagedomain.

hmm, looks like kerneldoc is out of sync in a few places.

Thanks for catching.  Will fixup.

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