Re: [PATCHv4 1/5] clk: mvebu: support for 98DX3236 SoC

2017-01-23 Thread Stephen Boyd
On 01/22/2017 11:53 PM, Chris Packham wrote: > Not 100% sure what you mean. Something like this? > > +static void __init of_mv98dx3236_cpu_clk_setup(struct device_node *node) > +{ > + of_clk_add_provider(node, of_clk_src_simple_get, NULL); > +} > > Seems to work as expected (i.e. does

Re: [PATCHv4 1/5] clk: mvebu: support for 98DX3236 SoC

2017-01-22 Thread Chris Packham
On 21/01/17 13:48, Stephen Boyd wrote: > On 01/13, Chris Packham wrote: >> @@ -158,6 +170,14 @@ static const struct coreclk_soc_desc axp_coreclks = { >> .num_ratios = ARRAY_SIZE(axp_coreclk_ratios), >> }; >> >> +static const struct coreclk_soc_desc mv98dx3236_coreclks = { >> +

Re: [PATCHv4 1/5] clk: mvebu: support for 98DX3236 SoC

2017-01-20 Thread Stephen Boyd
On 01/13, Chris Packham wrote: > @@ -158,6 +170,14 @@ static const struct coreclk_soc_desc axp_coreclks = { > .num_ratios = ARRAY_SIZE(axp_coreclk_ratios), > }; > > +static const struct coreclk_soc_desc mv98dx3236_coreclks = { > + .get_tclk_freq = mv98dx3236_get_tclk_freq, > +

Re: [PATCHv4 1/5] clk: mvebu: support for 98DX3236 SoC

2017-01-18 Thread Chris Packham
On 19/01/17 11:25, Rob Herring wrote: > On Fri, Jan 13, 2017 at 10:12:16PM +1300, Chris Packham wrote: >> The 98DX3236, 98DX3336, 98DX4521 and variants have a different TCLK from >> the Armada XP (200MHz vs 250MHz). The CPU core clock is fixed at 800MHz. >> >> The clock gating options are a subset

Re: [PATCHv4 1/5] clk: mvebu: support for 98DX3236 SoC

2017-01-18 Thread Rob Herring
On Fri, Jan 13, 2017 at 10:12:16PM +1300, Chris Packham wrote: > The 98DX3236, 98DX3336, 98DX4521 and variants have a different TCLK from > the Armada XP (200MHz vs 250MHz). The CPU core clock is fixed at 800MHz. > > The clock gating options are a subset of those on the Armada XP. > > The core

[PATCHv4 1/5] clk: mvebu: support for 98DX3236 SoC

2017-01-13 Thread Chris Packham
The 98DX3236, 98DX3336, 98DX4521 and variants have a different TCLK from the Armada XP (200MHz vs 250MHz). The CPU core clock is fixed at 800MHz. The clock gating options are a subset of those on the Armada XP. The core clock divider is different to the Armada XP also. Signed-off-by: Chris