From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 4 Aug 2015 17:45:57 +0800 From: Dong Aisheng To: Dong Aisheng CC: , , , , , , , , , Subject: Re: [PATCH V3 0/5] clk: support clocks which requires parent clock on during operation Message-ID: <20150804094556.GE24870@shlinux1.ap.freescale.net> References: <1438089585-30103-1-git-send-email-aisheng.dong@freescale.com> <20150728143824.GB16916@shlinux1.ap.freescale.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" In-Reply-To: <20150728143824.GB16916@shlinux1.ap.freescale.net> Return-Path: aisheng.dong@freescale.com List-ID: On Tue, Jul 28, 2015 at 10:38:25PM +0800, Dong Aisheng wrote: > On Tue, Jul 28, 2015 at 09:19:40PM +0800, Dong Aisheng wrote: > > This patch series adds support in clock framework for clocks which operations > > requires its parent clock is on. > > > > Such clock type is initially met on Freescale i.MX7D platform that all clocks > > operations, including enable/disable, rate change and re-parent, requires its > > parent clock on. No sure if any other SoC has the similar clock type. > > > > Current clock core can not support such type of clock well. > > > > This patch introduce a new flag CLK_SET_PARENT_ON to handle this special case > > in clock core that enable its parent clock firstly for each operation and disable > > it later after operation complete. > > > > The most special case is for set_parent() operation which requires both parent, > > old one and new one, to be enabled at the same time during the operation. > > > > The patch series is based on for-next branch of Michael's git: > > git://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git > > > > Change Log: > > v2->v3: > > * rebased version, no other changes. > > > > v1->v2: > > Mainly addressed Stephen Boyd's comments > > * remove dupliciated code with __clk_set_parent_after > > * introduce more clk_core_x APIs for core easily use > > * move clk_disable_unused code position > > * use clk_core_x API to make code more clean and easily read > > > > Dong Aisheng (5): > > clk: remove duplicated code with __clk_set_parent_after > > clk: introduce clk_core_enable_lock and clk_core_disable_lock > > functions > > clk: move clk_disable_unused after clk_core_disable_unprepare function > > clk: core: add CLK_OPS_PARENT_ON flags to support clocks require > > parent on > > clk: core: add CLK_OPS_PARENT_ON flags to support clocks require > > parent on > > > > drivers/clk/clk.c | 338 +++++++++++++++++++++++++------------------ > > include/linux/clk-provider.h | 5 + > > 2 files changed, 200 insertions(+), 143 deletions(-) > > > > Sorry, forgot to update Mike's new email address. > Updated. > Ping... Regards Dong Aisheng > Regards > Dong Aisheng > > > -- > > 1.9.1 > >