Kevin, On Mon, Mar 21, 2011 at 9:02 PM, Kevin Hilman <khilman@xxxxxx> wrote: > "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) Even if we try to remove 'vdd_name', how should we get 'struct voltagedomain' info corresponding to sr_mpu, sr_core in sr_dev_init API? As functional clock of SR1 and SR2 are from wake up domain, so even if we try to use current Clockdomain and Powerdomain framework APIs, to get 'clkdomain' pointer and then retrieve 'pwrdomain' pointer, finally getting 'voltdomain' pointer we would get 'WKUP' voltage domain ptr not the 'mpu' or 'core' voltdomain pointer. > 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 > -- Thanks, Regards, Shweta -- 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