Re: [PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-26 Thread Saravana Kannan
On Tue, April 17, 2012 12:17 am, Shawn Guo wrote: On 17 April 2012 11:50, Turquette, Mike mturque...@ti.com wrote: That is a good question.  I think it is worth waiting on Saravana's patch which exposes non-private members of struct clk via struct clk_hw.  This will have an effect on both

Re: [PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-20 Thread Saravana Kannan
On 04/17/2012 12:17 AM, Shawn Guo wrote: On 17 April 2012 11:50, Turquette, Mikemturque...@ti.com wrote: That is a good question. I think it is worth waiting on Saravana's patch which exposes non-private members of struct clk via struct clk_hw. This will have an effect on both platform clock

Re: [PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-17 Thread Shawn Guo
On 17 April 2012 11:50, Turquette, Mike mturque...@ti.com wrote: That is a good question.  I think it is worth waiting on Saravana's patch which exposes non-private members of struct clk via struct clk_hw.  This will have an effect on both platform clock data and code. Saravana, (*nudge*)

Re: [PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-16 Thread Sascha Hauer
On Wed, Apr 11, 2012 at 06:02:51PM -0700, Mike Turquette wrote: This patch is the basic clk version of 'clk: core: copy parent_names return error codes'. The registration functions are changed to allow the core code to copy the array of strings and allow platforms to declare those arrays as

Re: [PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-16 Thread Turquette, Mike
On Wed, Apr 11, 2012 at 11:49 PM, Shawn Guo shawn@linaro.org wrote: On Wed, Apr 11, 2012 at 06:02:51PM -0700, Mike Turquette wrote: ... @@ -175,23 +188,32 @@ struct clk *clk_register_divider(struct device *dev, const char *name,       div-flags = clk_divider_flags;       div-lock =

Re: [PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-16 Thread Turquette, Mike
On Mon, Apr 16, 2012 at 1:52 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Wed, Apr 11, 2012 at 06:02:51PM -0700, Mike Turquette wrote: This patch is the basic clk version of 'clk: core: copy parent_names return error codes'. The registration functions are changed to allow the core code

Re: [PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-16 Thread Shawn Guo
On 17 April 2012 07:10, Turquette, Mike mturque...@ti.com wrote: ... Yes, this was a braindead change on my part.  I'll remove the kstrdup in my next series (the rest of this patch will stay in). Do you have an ETA on that? A few platform porting are waiting for a stable branch with all

Re: [PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-16 Thread Turquette, Mike
On Mon, Apr 16, 2012 at 6:46 PM, Shawn Guo shawn@linaro.org wrote: On 17 April 2012 07:10, Turquette, Mike mturque...@ti.com wrote: ... Yes, this was a braindead change on my part.  I'll remove the kstrdup in my next series (the rest of this patch will stay in). Do you have an ETA on

Re: [PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-12 Thread Shawn Guo
On Wed, Apr 11, 2012 at 06:02:51PM -0700, Mike Turquette wrote: ... @@ -175,23 +188,32 @@ struct clk *clk_register_divider(struct device *dev, const char *name, div-flags = clk_divider_flags; div-lock = lock; + /* allocate the temporary parent_names */ if

[PATCH 13/13] clk: basic: improve parent_names return errors

2012-04-11 Thread Mike Turquette
This patch is the basic clk version of 'clk: core: copy parent_names return error codes'. The registration functions are changed to allow the core code to copy the array of strings and allow platforms to declare those arrays as __initdata. This patch also converts all of the basic clk