Santosh Shilimkar <[email protected]> writes:

> Minor comment.
>> -----Original Message-----
>> From: [email protected] [mailto:linux-omap-
>> [email protected]] On Behalf Of Kevin Hilman
>> Sent: Saturday, March 19, 2011 5:49 AM
>> To: [email protected]
>> Cc: Paul Walmsely; Benoit Cousson
>> Subject: [PATCH 2/7] OMAP2+: voltage: move PRCM mod offets into VDD
>> structure
>>
>> Eliminate need for global variables for the various PRM module
>> offsets by
>> making them part of the VDD structure.
>>
>> Eventually, these will likely be moved again, or more likely
>> removed,
>> but for now just getting rid of them as global variabes so that the
>> voltage domain initialization can be cleaned up.
>>
>> Signed-off-by: Kevin Hilman <[email protected]>
>> ---
>>  arch/arm/mach-omap2/voltage.c                 |  109 ++++++++++++--
>> -----------
>>  arch/arm/mach-omap2/voltage.h                 |    6 +-
>>  arch/arm/mach-omap2/voltagedomains3xxx_data.c |   10 +-
>>  arch/arm/mach-omap2/voltagedomains44xx_data.c |   12 ++-
>>  4 files changed, 67 insertions(+), 70 deletions(-)
>>
> [....]
>
>> diff --git a/arch/arm/mach-omap2/voltage.h b/arch/arm/mach-
>> omap2/voltage.h
>> index e9f5408..44edc1e 100644
>> --- a/arch/arm/mach-omap2/voltage.h
>> +++ b/arch/arm/mach-omap2/voltage.h
>> @@ -133,6 +133,9 @@ struct omap_vdd_info {
>>      struct dentry *debug_dir;
>>      u32 curr_volt;
>>      bool vp_enabled;
>> +
>> +    s16 prm_mod;
>> +    s16 prm_irqst_mod;
>
> Can you also add kernel doc for above two
> entries?
>

Yes, thanks.

Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to