Paul,

> -----Original Message-----
> From: Paul Walmsley [mailto:p...@pwsan.com] 
> Sent: Tuesday, March 24, 2009 2:58 PM
> To: Kevin Hilman
> Cc: Gupta, Ramesh; linux-omap@vger.kernel.org
> Subject: Re: [PATCH 1/1] Export notifier register functions 
> for kernel module building
> 
> Hello Kevin, Ramesh,
> 
> On Thu, 12 Feb 2009, Kevin Hilman wrote:
> 
> > "Gupta, Ramesh" <grgu...@ti.com> writes:
> > 
> > > This Patch exports symbols 
> clk_notifier_register/unregister function 
> > > for other kernel modules usage.
> > >
> > > Signed-off-by: Ramesh Gupta G <grgu...@ti.com>
> > 
> > Thanks, pushed to PM branch.
> 
> As an aside, this patch should be reverted.  DSPBridge and 
> other drivers needing clock notifiers should pass function pointers to
> clk_notifier_{register,unregister}() in their struct 
> platform_data, rather than exporting those symbols.  This 
> will keep the drivers platform-agnostic, since system-wide 
> clock notifiers are not yet upstream.

I agree on this, I think the latest patch set from Rajendra Naik([1]) removes 
the EXPORT_SYMBOL for the clk notifier functions.

Ref[1]: http://marc.info/?l=linux-omap&m=123755561914202&w=2
Ref[2]: http://marc.info/?l=linux-omap&m=123755561914205&w=2

We will send a patch to dspbridge sources to adopt these changes.

Please let me know your comments.

Thanks
Ramesh Gupta G--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to