Skip to contentSkip to conversation

ARM: i.MX6Q clocks: Fix CKIH clock name

PATCH New Alexander Shiyan 2 participants 3 messages Jul 27, 2013 – Jul 29, 2013

Subsystems (3)
Arm (32-bit) Arm sub-architectures Arm i.MX and Layerscape
lore
Share & tools
lore

Alexander Shiyan

author
#0 lore raw

PATCH 1/1 ARM: i.MX6Q clocks: Fix CKIH clock name

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 <[email protected]>
---
 arch/arm/mach-imx/clk-imx6q.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/mach-imx/clk-imx6q.c +1 -1
index 9181a24..8f2a8b5 100644
@@ -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");
Signature · 1 line
1.8.1.5
#1 lore raw

PATCH 1/1 ARM: i.MX6Q clocks: Fix CKIH clock name

Quoted 13 lines · Alexander Shiyan

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 <[email protected]>
---
  arch/arm/mach-imx/clk-imx6q.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/mach-imx/clk-imx6q.c +0 -0
index 9181a24..8f2a8b5 100644

arch/arm/mach-imx/clk-imx6q.c static void __init imx6q_clocks_init(struct device_node *ccm_node) · line 298

Quoted 8 lines
arch/arm/mach-imx/clk-imx6q.c +1 -1
@@ -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

author
#2 lore raw

PATCH 1/1 ARM: i.MX6Q clocks: Fix CKIH clock name

Quoted 21 lines · On Mon, 29 Jul 2013 10:07:45 +0200 Dirk Behme <[email protected]> wrote:
Quoted 20 lines · Alexander Shiyan

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 <[email protected]>
---
  arch/arm/mach-imx/clk-imx6q.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/mach-imx/clk-imx6q.c +1 -1
index 9181a24..8f2a8b5 100644
@@ -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);
Quoted 4 lines

arch/arm/mach-imx/clk-imx6q.c static void __init imx6q_clocks_init(struct device_node *ccm_node) · line 298

Quoted 1 line
arch/arm/mach-imx/clk-imx6q.c +0 -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.

Signature · 1 line
Alexander Shiyan <[email protected]>