On 00:17 Thu 02 Sep     , Russell King - ARM Linux wrote:

On Wed, Sep 01, 2010 at 01:13:02PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:

quoted

On 10:12 Wed 01 Sep     , Russell King - ARM Linux wrote:

quoted

On Tue, Aug 31, 2010 at 01:16:42PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:

quoted

factorise some generic infrastructure to assist looking up struct clks
for the ARM & SH architecture.
+/*
+ * Returns a clock. Note that we first try to use device id on the bus
+ * and clock name. If this fails, we try to use clock name only.
+ */
 struct clk *clk_get(struct device *dev, const char *con_id)
 {
 	const char *dev_id = dev ? dev_name(dev) : NULL;
 
 	return clk_get_sys(dev_id, con_id);
 }
-EXPORT_SYMBOL(clk_get);
+EXPORT_SYMBOL_GPL(clk_get);
 
 void clk_put(struct clk *clk)
 {
 	__clk_put(clk);
 }
-EXPORT_SYMBOL(clk_put);
+EXPORT_SYMBOL_GPL(clk_put);

Please leave these as-is.

ok but on SH it's GPL so it is ok with you Paul too?

If you want to get that pedantic... I created clkdev, and Sascha Hauer
extended it with some refactoring to create the clk_get_sys() interface.

personnaly It's fine for I
I just get in mind to knon if Paul have a specific reason to put it as GPL
EXPORT_SYMBOL

Best Regards,
J.