Patchwork ARM: i.MX6Q clocks: Fix CKIH clock name

login
register
mail settings
Submitter Alexander Shiyan
Date July 27, 2013, 6:56 a.m.
Message ID <1374908212-32008-1-git-send-email-shc_work@mail.ru>
Download mbox | patch
Permalink /patch/262381/
State New
Headers show

Comments

Alexander Shiyan - July 27, 2013, 6:56 a.m.
i.MX6Q have only one High-frequency reference clock (CKIH).
Patch removes index from CKIH and makes name according datasheet.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
 arch/arm/mach-imx/clk-imx6q.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Dirk Behme - July 29, 2013, 8:07 a.m.
On 27.07.2013 08:56, Alexander Shiyan wrote:
> i.MX6Q have only one High-frequency reference clock (CKIH).
> Patch removes index from CKIH and makes name according datasheet.
>
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> ---
>   arch/arm/mach-imx/clk-imx6q.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-imx/clk-imx6q.c b/arch/arm/mach-imx/clk-imx6q.c
> index 9181a24..8f2a8b5 100644
> --- a/arch/arm/mach-imx/clk-imx6q.c
> +++ b/arch/arm/mach-imx/clk-imx6q.c
> @@ -292,7 +292,7 @@ static void __init imx6q_clocks_init(struct device_node *ccm_node)
>
>   	clk[dummy] = imx_clk_fixed("dummy", 0);
>   	clk[ckil] = imx_obtain_fixed_clock("ckil", 0);
> -	clk[ckih] = imx_obtain_fixed_clock("ckih1", 0);
> +	clk[ckih] = imx_obtain_fixed_clock("ckih", 0);
>   	clk[osc] = imx_obtain_fixed_clock("osc", 0);
>
>   	np = of_find_compatible_node(NULL, NULL, "fsl,imx6q-anatop");

Just for my understanding: This is a cosmetic change, only? And doesn't 
change any functionality? Correct?

Thanks,

Dirk
Alexander Shiyan - July 29, 2013, 3:42 p.m.
On Mon, 29 Jul 2013 10:07:45 +0200
Dirk Behme <dirk.behme@de.bosch.com> wrote:

> On 27.07.2013 08:56, Alexander Shiyan wrote:
> > i.MX6Q have only one High-frequency reference clock (CKIH).
> > Patch removes index from CKIH and makes name according datasheet.
> >
> > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > ---
> >   arch/arm/mach-imx/clk-imx6q.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/mach-imx/clk-imx6q.c b/arch/arm/mach-imx/clk-imx6q.c
> > index 9181a24..8f2a8b5 100644
> > --- a/arch/arm/mach-imx/clk-imx6q.c
> > +++ b/arch/arm/mach-imx/clk-imx6q.c
> > @@ -292,7 +292,7 @@ static void __init imx6q_clocks_init(struct device_node *ccm_node)
> >
> >   	clk[dummy] = imx_clk_fixed("dummy", 0);
> >   	clk[ckil] = imx_obtain_fixed_clock("ckil", 0);
> > -	clk[ckih] = imx_obtain_fixed_clock("ckih1", 0);
> > +	clk[ckih] = imx_obtain_fixed_clock("ckih", 0);
> >   	clk[osc] = imx_obtain_fixed_clock("osc", 0);
> >
> >   	np = of_find_compatible_node(NULL, NULL, "fsl,imx6q-anatop");
> 
> Just for my understanding: This is a cosmetic change, only? And doesn't 
> change any functionality? Correct?

Yes, but in any case this is an incomplete patch, because I forgot to change
the relevant DTS.

Patch

diff --git a/arch/arm/mach-imx/clk-imx6q.c b/arch/arm/mach-imx/clk-imx6q.c
index 9181a24..8f2a8b5 100644
--- a/arch/arm/mach-imx/clk-imx6q.c
+++ b/arch/arm/mach-imx/clk-imx6q.c
@@ -292,7 +292,7 @@  static void __init imx6q_clocks_init(struct device_node *ccm_node)
 
 	clk[dummy] = imx_clk_fixed("dummy", 0);
 	clk[ckil] = imx_obtain_fixed_clock("ckil", 0);
-	clk[ckih] = imx_obtain_fixed_clock("ckih1", 0);
+	clk[ckih] = imx_obtain_fixed_clock("ckih", 0);
 	clk[osc] = imx_obtain_fixed_clock("osc", 0);
 
 	np = of_find_compatible_node(NULL, NULL, "fsl,imx6q-anatop");