Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-20 Thread Vlad Zakharov
Hi Stephen, On Wed, 2017-04-19 at 09:49 -0700, sb...@codeaurora.org wrote: > On 04/05, Vlad Zakharov wrote: > > > > Hi Stephen, > > > > On Tue, 2017-04-04 at 18:35 -0700, Stephen Boyd wrote: > > > > > > > > > > > + .pll_table = (struct pll_of_table []){ > > > > + { > > > > +   

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-20 Thread Vlad Zakharov
Hi Stephen, On Wed, 2017-04-19 at 09:49 -0700, sb...@codeaurora.org wrote: > On 04/05, Vlad Zakharov wrote: > > > > Hi Stephen, > > > > On Tue, 2017-04-04 at 18:35 -0700, Stephen Boyd wrote: > > > > > > > > > > > + .pll_table = (struct pll_of_table []){ > > > > + { > > > > +   

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-19 Thread sb...@codeaurora.org
On 04/05, Vlad Zakharov wrote: > Hi Stephen, > > On Tue, 2017-04-04 at 18:35 -0700, Stephen Boyd wrote: > > > + .pll_table = (struct pll_of_table []){ > > > + { > > > + .prate = 2700, > > > > Can this be another clk in the framework instead of hardcoding >

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-19 Thread sb...@codeaurora.org
On 04/05, Vlad Zakharov wrote: > Hi Stephen, > > On Tue, 2017-04-04 at 18:35 -0700, Stephen Boyd wrote: > > > + .pll_table = (struct pll_of_table []){ > > > + { > > > + .prate = 2700, > > > > Can this be another clk in the framework instead of hardcoding >

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-05 Thread Vlad Zakharov
Hi Stephen, On Tue, 2017-04-04 at 18:35 -0700, Stephen Boyd wrote: > > + .pll_table = (struct pll_of_table []){ > > + { > > + .prate = 2700, > > Can this be another clk in the framework instead of hardcoding > the parent rate? In fact there is another clk

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-05 Thread Vlad Zakharov
Hi Stephen, On Tue, 2017-04-04 at 18:35 -0700, Stephen Boyd wrote: > > + .pll_table = (struct pll_of_table []){ > > + { > > + .prate = 2700, > > Can this be another clk in the framework instead of hardcoding > the parent rate? In fact there is another clk

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-04 Thread Stephen Boyd
On 02/21, Vlad Zakharov wrote: > diff --git a/Documentation/devicetree/bindings/clock/snps,pll-clock.txt > b/Documentation/devicetree/bindings/clock/snps,pll-clock.txt > new file mode 100644 > index 000..5706246 > --- /dev/null > +++

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-04 Thread Stephen Boyd
On 02/21, Vlad Zakharov wrote: > diff --git a/Documentation/devicetree/bindings/clock/snps,pll-clock.txt > b/Documentation/devicetree/bindings/clock/snps,pll-clock.txt > new file mode 100644 > index 000..5706246 > --- /dev/null > +++

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-03 Thread Jose Abreu
Hi Vlad, On 21-02-2017 13:11, Vlad Zakharov wrote: > AXS10X boards manages it's clocks using various PLLs. These PLL has same > dividers and corresponding control registers mapped to different addresses. > So we add one common driver for such PLLs. > > Each PLL on AXS10X board consist of three

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-04-03 Thread Jose Abreu
Hi Vlad, On 21-02-2017 13:11, Vlad Zakharov wrote: > AXS10X boards manages it's clocks using various PLLs. These PLL has same > dividers and corresponding control registers mapped to different addresses. > So we add one common driver for such PLLs. > > Each PLL on AXS10X board consist of three

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-03-29 Thread Vlad Zakharov
Hi Stephen, Michael, On Fri, 2017-03-03 at 15:50 -0800, Stephen Boyd wrote: > On 03/03, Vlad Zakharov wrote: > > > > Hi Michael, Stephen, > > > > On Tue, 2017-02-21 at 16:11 +0300, Vlad Zakharov wrote: > > > > > > AXS10X boards manages it's clocks using various PLLs. These PLL has same > > >

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-03-29 Thread Vlad Zakharov
Hi Stephen, Michael, On Fri, 2017-03-03 at 15:50 -0800, Stephen Boyd wrote: > On 03/03, Vlad Zakharov wrote: > > > > Hi Michael, Stephen, > > > > On Tue, 2017-02-21 at 16:11 +0300, Vlad Zakharov wrote: > > > > > > AXS10X boards manages it's clocks using various PLLs. These PLL has same > > >

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-03-03 Thread Stephen Boyd
On 03/03, Vlad Zakharov wrote: > Hi Michael, Stephen, > > On Tue, 2017-02-21 at 16:11 +0300, Vlad Zakharov wrote: > > AXS10X boards manages it's clocks using various PLLs. These PLL has same > > dividers and corresponding control registers mapped to different addresses. > > So we add one common

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-03-03 Thread Stephen Boyd
On 03/03, Vlad Zakharov wrote: > Hi Michael, Stephen, > > On Tue, 2017-02-21 at 16:11 +0300, Vlad Zakharov wrote: > > AXS10X boards manages it's clocks using various PLLs. These PLL has same > > dividers and corresponding control registers mapped to different addresses. > > So we add one common

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-03-03 Thread Vlad Zakharov
Hi Michael, Stephen, On Tue, 2017-02-21 at 16:11 +0300, Vlad Zakharov wrote: > AXS10X boards manages it's clocks using various PLLs. These PLL has same > dividers and corresponding control registers mapped to different addresses. > So we add one common driver for such PLLs. > > Each PLL on

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-03-03 Thread Vlad Zakharov
Hi Michael, Stephen, On Tue, 2017-02-21 at 16:11 +0300, Vlad Zakharov wrote: > AXS10X boards manages it's clocks using various PLLs. These PLL has same > dividers and corresponding control registers mapped to different addresses. > So we add one common driver for such PLLs. > > Each PLL on

[PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-02-21 Thread Vlad Zakharov
AXS10X boards manages it's clocks using various PLLs. These PLL has same dividers and corresponding control registers mapped to different addresses. So we add one common driver for such PLLs. Each PLL on AXS10X board consist of three dividers: IDIV, FBDIV and ODIV. Output clock value is managed

[PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-02-21 Thread Vlad Zakharov
AXS10X boards manages it's clocks using various PLLs. These PLL has same dividers and corresponding control registers mapped to different addresses. So we add one common driver for such PLLs. Each PLL on AXS10X board consist of three dividers: IDIV, FBDIV and ODIV. Output clock value is managed