Re: [project-aspen-dev] Re: [PATCH 1/3] clk: hisilicon: add hisi phase clock support

2017-11-15 Thread Shawn Guo
On Thu, Nov 16, 2017 at 11:40:18AM +0800, Jiancheng Xue wrote: > >> +struct clk_hisi_phase { > >> + struct clk_hw hw; > >> + void __iomem*reg; > >> + u32 *phase_values; > >> + u32 *phase_regs; > >> + u8 phase_num; > > > > I do not think this

Re: [project-aspen-dev] Re: [PATCH 1/3] clk: hisilicon: add hisi phase clock support

2017-11-15 Thread Shawn Guo
On Thu, Nov 16, 2017 at 11:40:18AM +0800, Jiancheng Xue wrote: > >> +struct clk_hisi_phase { > >> + struct clk_hw hw; > >> + void __iomem*reg; > >> + u32 *phase_values; > >> + u32 *phase_regs; > >> + u8 phase_num; > > > > I do not think this

Re: [project-aspen-dev] Re: [PATCH 1/3] clk: hisilicon: add hisi phase clock support

2017-11-15 Thread Jiancheng Xue
Hi Shawn, On 2017/11/16 10:31, Shawn Guo wrote: > On Wed, Oct 18, 2017 at 07:00:27AM -0400, Jiancheng Xue wrote: >> From: tianshuliang >> >> Add a phase clock type for HiSilicon SoCs,which supports >> clk_set_phase operation. > > As the pair of phase operation, I

Re: [project-aspen-dev] Re: [PATCH 1/3] clk: hisilicon: add hisi phase clock support

2017-11-15 Thread Jiancheng Xue
Hi Shawn, On 2017/11/16 10:31, Shawn Guo wrote: > On Wed, Oct 18, 2017 at 07:00:27AM -0400, Jiancheng Xue wrote: >> From: tianshuliang >> >> Add a phase clock type for HiSilicon SoCs,which supports >> clk_set_phase operation. > > As the pair of phase operation, I don't see why clk_get_phase

Re: [PATCH 1/3] clk: hisilicon: add hisi phase clock support

2017-11-15 Thread Shawn Guo
On Wed, Oct 18, 2017 at 07:00:27AM -0400, Jiancheng Xue wrote: > From: tianshuliang > > Add a phase clock type for HiSilicon SoCs,which supports > clk_set_phase operation. As the pair of phase operation, I don't see why clk_get_phase operation is missing. > >

Re: [PATCH 1/3] clk: hisilicon: add hisi phase clock support

2017-11-15 Thread Shawn Guo
On Wed, Oct 18, 2017 at 07:00:27AM -0400, Jiancheng Xue wrote: > From: tianshuliang > > Add a phase clock type for HiSilicon SoCs,which supports > clk_set_phase operation. As the pair of phase operation, I don't see why clk_get_phase operation is missing. > > Signed-off-by: tianshuliang >

[PATCH 1/3] clk: hisilicon: add hisi phase clock support

2017-10-17 Thread Jiancheng Xue
From: tianshuliang Add a phase clock type for HiSilicon SoCs,which supports clk_set_phase operation. Signed-off-by: tianshuliang Signed-off-by: Jiancheng Xue --- drivers/clk/hisilicon/Makefile | 2

[PATCH 1/3] clk: hisilicon: add hisi phase clock support

2017-10-17 Thread Jiancheng Xue
From: tianshuliang Add a phase clock type for HiSilicon SoCs,which supports clk_set_phase operation. Signed-off-by: tianshuliang Signed-off-by: Jiancheng Xue --- drivers/clk/hisilicon/Makefile | 2 +- drivers/clk/hisilicon/clk-hisi-phase.c | 117 +