* [PATCH] ARM: clk: imx: remove the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags
@ 2014-05-26 7:31 Huang Shijie
2014-05-26 9:03 ` Philipp Zabel
2014-05-29 8:10 ` Shawn Guo
0 siblings, 2 replies; 5+ messages in thread
From: Huang Shijie @ 2014-05-26 7:31 UTC (permalink / raw)
To: linux-arm-kernel
The clocks for Quadspi controller may be different when different
NOR flashes are connected to the board.
But the CLK_SET_RATE_NO_REPARENT flag in the current imx_clk_mux_flags
makes the clock do not have the re-parent capability.
This patch removes the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags.
It makes the Quadspi happy.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
arch/arm/mach-imx/clk.h | 3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/arch/arm/mach-imx/clk.h b/arch/arm/mach-imx/clk.h
index e29f6eb..32d6253 100644
--- a/arch/arm/mach-imx/clk.h
+++ b/arch/arm/mach-imx/clk.h
@@ -108,8 +108,7 @@ static inline struct clk *imx_clk_mux_flags(const char *name,
int num_parents, unsigned long flags)
{
return clk_register_mux(NULL, name, parents, num_parents,
- flags | CLK_SET_RATE_NO_REPARENT, reg, shift, width, 0,
- &imx_ccm_lock);
+ flags, reg, shift, width, 0, &imx_ccm_lock);
}
static inline struct clk *imx_clk_fixed_factor(const char *name,
--
1.7.8
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] ARM: clk: imx: remove the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags
2014-05-26 9:03 ` Philipp Zabel
@ 2014-05-26 7:46 ` Huang Shijie
2014-05-26 9:11 ` Shawn Guo
1 sibling, 0 replies; 5+ messages in thread
From: Huang Shijie @ 2014-05-26 7:46 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, May 26, 2014 at 11:03:22AM +0200, Philipp Zabel wrote:
> Am Montag, den 26.05.2014, 15:31 +0800 schrieb Huang Shijie:
> > The clocks for Quadspi controller may be different when different
> > NOR flashes are connected to the board.
> >
> > But the CLK_SET_RATE_NO_REPARENT flag in the current imx_clk_mux_flags
> > makes the clock do not have the re-parent capability.
> >
> > This patch removes the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags.
> > It makes the Quadspi happy.
> >
> > Signed-off-by: Huang Shijie <b32955@freescale.com>
> > ---
> > arch/arm/mach-imx/clk.h | 3 +--
> > 1 files changed, 1 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/mach-imx/clk.h b/arch/arm/mach-imx/clk.h
> > index e29f6eb..32d6253 100644
> > --- a/arch/arm/mach-imx/clk.h
> > +++ b/arch/arm/mach-imx/clk.h
> > @@ -108,8 +108,7 @@ static inline struct clk *imx_clk_mux_flags(const char *name,
> > int num_parents, unsigned long flags)
> > {
> > return clk_register_mux(NULL, name, parents, num_parents,
> > - flags | CLK_SET_RATE_NO_REPARENT, reg, shift, width, 0,
> > - &imx_ccm_lock);
> > + flags, reg, shift, width, 0, &imx_ccm_lock);
> > }
> >
> > static inline struct clk *imx_clk_fixed_factor(const char *name,
>
> We need CLK_SET_RATE_NO_REPARENT on the audio/video multiplexers.
> I think this change is the right thing for imx_clk_mux_flags, but at the
> same you should make sure not to drop this flag where we want to control
> a video PLL through the mux.
thanks for the quick response.
Could you tell me the clocks the audio/video used?
we can add the CLK_SET_RATE_NO_REPARENT to them.
thanks
Huang Shijie
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: clk: imx: remove the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags
2014-05-26 7:31 [PATCH] ARM: clk: imx: remove the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags Huang Shijie
@ 2014-05-26 9:03 ` Philipp Zabel
2014-05-26 7:46 ` Huang Shijie
2014-05-26 9:11 ` Shawn Guo
2014-05-29 8:10 ` Shawn Guo
1 sibling, 2 replies; 5+ messages in thread
From: Philipp Zabel @ 2014-05-26 9:03 UTC (permalink / raw)
To: linux-arm-kernel
Am Montag, den 26.05.2014, 15:31 +0800 schrieb Huang Shijie:
> The clocks for Quadspi controller may be different when different
> NOR flashes are connected to the board.
>
> But the CLK_SET_RATE_NO_REPARENT flag in the current imx_clk_mux_flags
> makes the clock do not have the re-parent capability.
>
> This patch removes the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags.
> It makes the Quadspi happy.
>
> Signed-off-by: Huang Shijie <b32955@freescale.com>
> ---
> arch/arm/mach-imx/clk.h | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-imx/clk.h b/arch/arm/mach-imx/clk.h
> index e29f6eb..32d6253 100644
> --- a/arch/arm/mach-imx/clk.h
> +++ b/arch/arm/mach-imx/clk.h
> @@ -108,8 +108,7 @@ static inline struct clk *imx_clk_mux_flags(const char *name,
> int num_parents, unsigned long flags)
> {
> return clk_register_mux(NULL, name, parents, num_parents,
> - flags | CLK_SET_RATE_NO_REPARENT, reg, shift, width, 0,
> - &imx_ccm_lock);
> + flags, reg, shift, width, 0, &imx_ccm_lock);
> }
>
> static inline struct clk *imx_clk_fixed_factor(const char *name,
We need CLK_SET_RATE_NO_REPARENT on the audio/video multiplexers.
I think this change is the right thing for imx_clk_mux_flags, but at the
same you should make sure not to drop this flag where we want to control
a video PLL through the mux.
regards
Philipp
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: clk: imx: remove the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags
2014-05-26 9:03 ` Philipp Zabel
2014-05-26 7:46 ` Huang Shijie
@ 2014-05-26 9:11 ` Shawn Guo
1 sibling, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2014-05-26 9:11 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, May 26, 2014 at 11:03:22AM +0200, Philipp Zabel wrote:
> > @@ -108,8 +108,7 @@ static inline struct clk *imx_clk_mux_flags(const char *name,
> > int num_parents, unsigned long flags)
> > {
> > return clk_register_mux(NULL, name, parents, num_parents,
> > - flags | CLK_SET_RATE_NO_REPARENT, reg, shift, width, 0,
> > - &imx_ccm_lock);
> > + flags, reg, shift, width, 0, &imx_ccm_lock);
> > }
> >
> > static inline struct clk *imx_clk_fixed_factor(const char *name,
>
> We need CLK_SET_RATE_NO_REPARENT on the audio/video multiplexers.
> I think this change is the right thing for imx_clk_mux_flags, but at the
> same you should make sure not to drop this flag where we want to control
> a video PLL through the mux.
Philipp,
Are you saying that you see a different clock tree topology after this
change?
Shawn
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: clk: imx: remove the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags
2014-05-26 7:31 [PATCH] ARM: clk: imx: remove the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags Huang Shijie
2014-05-26 9:03 ` Philipp Zabel
@ 2014-05-29 8:10 ` Shawn Guo
1 sibling, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2014-05-29 8:10 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, May 26, 2014 at 03:31:14PM +0800, Huang Shijie wrote:
> The clocks for Quadspi controller may be different when different
> NOR flashes are connected to the board.
>
> But the CLK_SET_RATE_NO_REPARENT flag in the current imx_clk_mux_flags
> makes the clock do not have the re-parent capability.
>
> This patch removes the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags.
> It makes the Quadspi happy.
>
> Signed-off-by: Huang Shijie <b32955@freescale.com>
Applied it for now. Let's if the change will cause any issue on
linux-next.
Shawn
> ---
> arch/arm/mach-imx/clk.h | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-imx/clk.h b/arch/arm/mach-imx/clk.h
> index e29f6eb..32d6253 100644
> --- a/arch/arm/mach-imx/clk.h
> +++ b/arch/arm/mach-imx/clk.h
> @@ -108,8 +108,7 @@ static inline struct clk *imx_clk_mux_flags(const char *name,
> int num_parents, unsigned long flags)
> {
> return clk_register_mux(NULL, name, parents, num_parents,
> - flags | CLK_SET_RATE_NO_REPARENT, reg, shift, width, 0,
> - &imx_ccm_lock);
> + flags, reg, shift, width, 0, &imx_ccm_lock);
> }
>
> static inline struct clk *imx_clk_fixed_factor(const char *name,
> --
> 1.7.8
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-05-29 8:10 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-26 7:31 [PATCH] ARM: clk: imx: remove the CLK_SET_RATE_NO_REPARENT for imx_clk_mux_flags Huang Shijie
2014-05-26 9:03 ` Philipp Zabel
2014-05-26 7:46 ` Huang Shijie
2014-05-26 9:11 ` Shawn Guo
2014-05-29 8:10 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).