"Cousson, Benoit" <b-cousson@xxxxxx> writes:
> Hi Kevin,
>
> On 3/19/2011 1:18 AM, Hilman, Kevin wrote:
>> The voltage domain pointer currently in struct omap_hwmod is not used
>> and does not belong here. Instead, voltage domains will be associated
>> with powerdomains in forthcoming patches.
>
> Technically speaking, an IP, and thus the hwmod does belong to a
> voltage domain, a power domain and a clock domain.
>
> What is maybe important to add is that since clkdm < pwrdm < voltdm,
> we can potentially avoid providing the information for the each domain
> type.
>
> At some point the hwmod will have to contain a clkdm instead of
> relying on the main_clk to get it... but that's another topic...
>
>> Signed-off-by: Kevin Hilman<khilman@xxxxxx>
>> ---
>> arch/arm/plat-omap/include/plat/omap_hwmod.h | 1 -
>> 1 files changed, 0 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/plat-omap/include/plat/omap_hwmod.h b/arch/arm/plat-omap/include/plat/omap_hwmod.h
>> index 1adea9c..a5fa7c1 100644
>> --- a/arch/arm/plat-omap/include/plat/omap_hwmod.h
>> +++ b/arch/arm/plat-omap/include/plat/omap_hwmod.h
>> @@ -520,7 +520,6 @@ struct omap_hwmod {
>> struct clk *_clk;
>> struct omap_hwmod_opt_clk *opt_clks;
>> char *vdd_name;
>
> And what about vdd_name? It should probably be removed as well.
>
Yes, but it's currently used by the SR layer (currently the only user.)
Removing it required cleaning up the SR layer as well, so I decided to
leave the SR cleanups for someone else for the moment while I focus on
the voltage layer(s)
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
