On Fri, Aug 23, 2013 at 05:03:45PM +0200, Sylwester Nawrocki wrote:
> This patch adds common __clk_get(), __clk_put() clkdev helpers which
> replace their platform specific counterparts when the common clock
> API is enabled.
>
> The owner module pointer field is added to struct clk so a reference
> to the clock supplier module can be taken by the clock consumers.
>
> Signed-off-by: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx>
> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
I'm mostly happy with this now.
> +int __clk_get(struct clk *clk)
> +{
> + if (clk && !try_module_get(clk->owner))
> + return 0;
> +
> + return 1;
> +}
> +EXPORT_SYMBOL(__clk_get);
> +
> +void __clk_put(struct clk *clk)
> +{
> + if (WARN_ON_ONCE(IS_ERR(clk)))
> + return;
> +
> + if (clk)
> + module_put(clk->owner);
> +}
> +EXPORT_SYMBOL(__clk_put);
Why are these exported? clkdev can only be built into the kernel, as can
the common clk framework - they can't be modular. So why would a module
wish to access these directly?
