Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-09-13 Thread Linus Walleij
On Thu, Sep 12, 2013 at 4:56 PM, Lee Jones wrote: > On Thu, 12 Sep 2013, Linus Walleij wrote: >> But if you still don't like this, let me cook a counter-patch so >> I can realized on my own how terribly wrong I am... > > I'm going to yank all of the clk_register_clkdev() calls out > imminently an

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-09-12 Thread Lee Jones
On Thu, 12 Sep 2013, Linus Walleij wrote: > On Tue, Aug 27, 2013 at 10:23 AM, Lee Jones wrote: > > On Fri, 23 Aug 2013, Linus Walleij wrote: > >> On Thu, Aug 22, 2013 at 11:21 AM, Lee Jones wrote: > > >> > 1. Duplicate each of the; clk_reg_prcmu_*(), clk_reg_prcc_pclk(), > >> > clk_reg_prcc_kcl

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-09-12 Thread Linus Walleij
On Tue, Aug 27, 2013 at 10:23 AM, Lee Jones wrote: > On Fri, 23 Aug 2013, Linus Walleij wrote: >> On Thu, Aug 22, 2013 at 11:21 AM, Lee Jones wrote: >> > 1. Duplicate each of the; clk_reg_prcmu_*(), clk_reg_prcc_pclk(), >> > clk_reg_prcc_kclk() calls into your proposed u8500_clk_init_dt(), >> >

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-08-27 Thread Lee Jones
On Fri, 23 Aug 2013, Linus Walleij wrote: > On Thu, Aug 22, 2013 at 11:21 AM, Lee Jones wrote: > > >> I really do not like the approach of uglifying something and then > >> beautifying it later... I prefer each step in isolation to be good > >> looking, or you will be confused when traversing th

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-08-24 Thread Linus Walleij
On Sat, Aug 24, 2013 at 10:00 AM, Arnd Bergmann wrote: > You just reminded me that I'm still sitting on this old patch > to add lots of #ifdef CONFIG_ATAGS to ux500 in the places that need > to get cut out. I don't think I'm able to update that patch at > the moment, but I can send you the old ve

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-08-24 Thread Arnd Bergmann
On Friday 23 August 2013, Linus Walleij wrote: > I think this is perfectly realistic. > > You're not going to duplicate each clk_register_clkdev(), > which makes it way smaller than the original function, > and since one of the function will be inside a > > #ifdef CONFIG_OF > #endif > > After we

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-08-23 Thread Linus Walleij
On Thu, Aug 22, 2013 at 11:21 AM, Lee Jones wrote: >> I really do not like the approach of uglifying something and then >> beautifying it later... I prefer each step in isolation to be good >> looking, or you will be confused when traversing the history. > > So then we have a few options, some mo

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-08-22 Thread Lee Jones
On Thu, 22 Aug 2013, Linus Walleij wrote: > On Wed, Aug 21, 2013 at 12:14 PM, Lee Jones wrote: > > On Wed, 21 Aug 2013, Linus Walleij wrote: > > >> Isn't it possible to fork a function u8500_clk_init_dt() to add all the > >> clocks in the DT probe path and keep this function > >> u8500_clk_init(

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-08-21 Thread Linus Walleij
On Wed, Aug 21, 2013 at 12:14 PM, Lee Jones wrote: > On Wed, 21 Aug 2013, Linus Walleij wrote: >> Isn't it possible to fork a function u8500_clk_init_dt() to add all the >> clocks in the DT probe path and keep this function >> u8500_clk_init() as it is? > > Yes, we probably could do that, but as

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-08-21 Thread Lee Jones
On Wed, 21 Aug 2013, Linus Walleij wrote: > On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones wrote: > > > This patch enables clocks to be specified from Device Tree via phandles > > to the "prcc-kernel-clock" node. > > > > Cc: Mike Turquette > > Cc: Ulf Hansson > > Signed-off-by: Lee Jones > (...) >

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-08-21 Thread Linus Walleij
On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones wrote: > This patch enables clocks to be specified from Device Tree via phandles > to the "prcc-kernel-clock" node. > > Cc: Mike Turquette > Cc: Ulf Hansson > Signed-off-by: Lee Jones (...) > static struct clk *prcmu_clk[PRCMU_NUM_CLKS]; > static str

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-06-11 Thread Lee Jones
> > This patch enables clocks to be specified from Device Tree via phandles > > to the "prcc-kernel-clock" node. > > > > Cc: Mike Turquette > > Cc: Ulf Hansson > > Signed-off-by: Lee Jones > > Could you please fold this patch into a "common PRCC device tree > support" patch instead. Thus handli

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-06-10 Thread Ulf Hansson
On 6 June 2013 14:17, Lee Jones wrote: > This patch enables clocks to be specified from Device Tree via phandles > to the "prcc-kernel-clock" node. > > Cc: Mike Turquette > Cc: Ulf Hansson > Signed-off-by: Lee Jones Could you please fold this patch into a "common PRCC device tree support" patc