Skip to contentSkip to conversation

clk: samsung: fix several typos to fix boot on s3c2410

PATCH New Vasily Khoruzhick 3 participants 10 messages Jun 23, 2014 – Jun 30, 2014

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410

There's a several typos in a driver: 2410 instead of S3C2410 and wrong argument to ARRAY_SIZE(). They prevent s3c2410 from properly booting.

---
 drivers/clk/samsung/clk-s3c2410.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/clk/samsung/clk-s3c2410.c +2 -2
index ba07168..bd9a873 100644
@@ -378,7 +378,7 @@ void __init s3c2410_common_clk_init(struct device_node *np, unsigned long xti_f,
 	if (!np)
 		s3c2410_common_clk_register_fixed_ext(ctx, xti_f);
 
-	if (current_soc == 2410) {
+	if (current_soc == S3C2410) {
 		if (_get_rate("xti") == 12 * MHZ) {
 			s3c2410_plls[mpll].rate_table = pll_s3c2410_12mhz_tbl;
 			s3c2410_plls[upll].rate_table = pll_s3c2410_12mhz_tbl;
@@ -432,7 +432,7 @@ void __init s3c2410_common_clk_init(struct device_node *np, unsigned long xti_f,
 		samsung_clk_register_fixed_factor(ctx, s3c2410_ffactor,
 				ARRAY_SIZE(s3c2410_ffactor));
 		samsung_clk_register_alias(ctx, s3c2410_aliases,
-			ARRAY_SIZE(s3c2410_common_aliases));
+			ARRAY_SIZE(s3c2410_aliases));
 		break;
 	case S3C2440:
 		samsung_clk_register_mux(ctx, s3c2440_muxes,
Signature · 1 line
2.0.0
#1 lore raw

PATCH 2/2 clk: samsung: add more aliases for s3c24xx

Without these aliases clock lookup fails in s3c2410fb, s3cmci, s3c2410-nand, s3c24xx-i2s, and i2c-s3c2410 drivers.

---
 drivers/clk/samsung/clk-s3c2410.c | 5 +++++
 1 file changed, 5 insertions(+)
drivers/clk/samsung/clk-s3c2410.c +5 -0
index bd9a873..140f473 100644
@@ -152,6 +152,11 @@ struct samsung_clock_alias s3c2410_common_aliases[] __initdata = {
 	ALIAS(HCLK, NULL, "hclk"),
 	ALIAS(MPLL, NULL, "mpll"),
 	ALIAS(FCLK, NULL, "fclk"),
+	ALIAS(PCLK, NULL, "watchdog"),
+	ALIAS(PCLK_SDI, NULL, "sdi"),
+	ALIAS(HCLK_NAND, NULL, "nand"),
+	ALIAS(PCLK_I2S, NULL, "iis"),
+	ALIAS(PCLK_I2C, NULL, "i2c"),
 };
 
 /* S3C2410 specific clocks */
Signature · 1 line
2.0.0
#2 lore raw

PATCH 2/2 clk: samsung: add more aliases for s3c24xx

Am Montag, 23. Juni 2014, 23:29:10 schrieb Vasily Khoruzhick:

Quoted 4 lines

Without these aliases clock lookup fails in s3c2410fb, s3cmci, s3c2410-nand, s3c24xx-i2s, and i2c-s3c2410 drivers.

Quoted 20 lines
---
 drivers/clk/samsung/clk-s3c2410.c | 5 +++++
 1 file changed, 5 insertions(+)
(quoted diff) +0 -0

b/drivers/clk/samsung/clk-s3c2410.c index bd9a873..140f473 100644

drivers/clk/samsung/clk-s3c2410.c +5 -0
@@ -152,6 +152,11 @@ struct samsung_clock_alias s3c2410_common_aliases[]
 __initdata = { ALIAS(HCLK, NULL, "hclk"),
 	ALIAS(MPLL, NULL, "mpll"),
 	ALIAS(FCLK, NULL, "fclk"),
+	ALIAS(PCLK, NULL, "watchdog"),
+	ALIAS(PCLK_SDI, NULL, "sdi"),
+	ALIAS(HCLK_NAND, NULL, "nand"),
+	ALIAS(PCLK_I2S, NULL, "iis"),
+	ALIAS(PCLK_I2C, NULL, "i2c"),
 };
 
 /* S3C2410 specific clocks */
Applied
#3 lore raw

PATCH 2/2 clk: samsung: add more aliases for s3c24xx

In reply to Vasily Khoruzhick
Quoted 7 lines · Vasily Khoruzhick

Without these aliases clock lookup fails in s3c2410fb, s3cmci, s3c2410-nand, s3c24xx-i2s, and i2c-s3c2410 drivers.

---
 drivers/clk/samsung/clk-s3c2410.c | 5 +++++
 1 file changed, 5 insertions(+)

Applied as a fix for 3.16.

Best regards,
Tomasz

#4 lore raw

PATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410

In reply to Vasily Khoruzhick

Am Montag, 23. Juni 2014, 23:29:09 schrieb Vasily Khoruzhick:

Quoted 5 lines

There's a several typos in a driver: 2410 instead of S3C2410 and wrong argument to ARRAY_SIZE(). They prevent s3c2410 from properly booting.

Thanks for catching these.

Quoted 27 lines
---
 drivers/clk/samsung/clk-s3c2410.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
(quoted diff) +0 -0

b/drivers/clk/samsung/clk-s3c2410.c index ba07168..bd9a873 100644

drivers/clk/samsung/clk-s3c2410.c +2 -2
@@ -378,7 +378,7 @@ void __init s3c2410_common_clk_init(struct device_node
 *np, unsigned long xti_f, if (!np)
 		s3c2410_common_clk_register_fixed_ext(ctx, xti_f);
 
-	if (current_soc == 2410) {
+	if (current_soc == S3C2410) {
 		if (_get_rate("xti") == 12 * MHZ) {
 			s3c2410_plls[mpll].rate_table = pll_s3c2410_12mhz_tbl;
 			s3c2410_plls[upll].rate_table = pll_s3c2410_12mhz_tbl;
@@ -432,7 +432,7 @@ void __init s3c2410_common_clk_init(struct device_node
 *np, unsigned long xti_f, samsung_clk_register_fixed_factor(ctx,
 s3c2410_ffactor,
 				ARRAY_SIZE(s3c2410_ffactor));
 		samsung_clk_register_alias(ctx, s3c2410_aliases,
-			ARRAY_SIZE(s3c2410_common_aliases));
+			ARRAY_SIZE(s3c2410_aliases));
 		break;
 	case S3C2440:

samsung_clk_register_mux(ctx, s3c2440_muxes,

#5 lore raw

PATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410

In reply to Vasily Khoruzhick

Hi Vasily,

Quoted 3 lines · Vasily Khoruzhick

There's a several typos in a driver: 2410 instead of S3C2410 and wrong argument to ARRAY_SIZE(). They prevent s3c2410 from properly booting.

Thanks for fixing this. I will apply and send to Mike with other fixes. Same for patch 2/2.

Best regards,
Tomasz

Applied
#6 lore raw

PATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410

In reply to Vasily Khoruzhick
Quoted 8 lines · Vasily Khoruzhick

There's a several typos in a driver: 2410 instead of S3C2410 and wrong argument to ARRAY_SIZE(). They prevent s3c2410 from properly booting.

---
 drivers/clk/samsung/clk-s3c2410.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Applied as a fix for 3.16.

Best regards,
Tomasz

#7 lore raw

PATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410

Quoted 5 lines · On 30 June 2014 15:49:58 Tomasz Figa wrote:
Quoted 4 lines · Vasily Khoruzhick

There's a several typos in a driver: 2410 instead of S3C2410 and wrong argument to ARRAY_SIZE(). They prevent s3c2410 from properly booting.

Quoted 7 lines
Quoted 5 lines

---

 drivers/clk/samsung/clk-s3c2410.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Applied as a fix for 3.16.

Hi Tomasz,

I also have clk fixes for s3c24xx touchscreen, framebuffer, adc, usb and nand drivers (to replace clk_enable with clk_prepare_enable and clk_disable with clk_disable_unprepare),

Should they be merged through different trees or it's possible to merge them through a single tree?

Quoted 3 lines

Best regards,
Tomasz

Regards
Vasily

#8 lore raw

PATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410

Hi Vasily,

Quoted 14 lines · Vasily Khoruzhick
Quoted 5 lines · On 30 June 2014 15:49:58 Tomasz Figa wrote:
Quoted 4 lines · Vasily Khoruzhick

There's a several typos in a driver: 2410 instead of S3C2410 and wrong argument to ARRAY_SIZE(). They prevent s3c2410 from properly booting.

Quoted 7 lines
Quoted 5 lines

---

 drivers/clk/samsung/clk-s3c2410.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Applied as a fix for 3.16.

Quoted 8 lines

Hi Tomasz,

I also have clk fixes for s3c24xx touchscreen, framebuffer, adc, usb and nand drivers (to replace clk_enable with clk_prepare_enable and clk_disable with clk_disable_unprepare),

Should they be merged through different trees or it's possible to merge them through a single tree?

I'd say they should all go through respective subsystem trees, unless they directly depend on patches I already applied, which is not the case as far as I can see.

Best regards,
Tomasz

#9 lore raw

PATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410

Quoted 23 lines · On 30 June 2014 16:07:24 Tomasz Figa wrote:

Hi Vasily,

Quoted 20 lines · Vasily Khoruzhick
Quoted 5 lines · On 30 June 2014 15:49:58 Tomasz Figa wrote:
Quoted 4 lines · Vasily Khoruzhick

There's a several typos in a driver: 2410 instead of S3C2410 and wrong argument to ARRAY_SIZE(). They prevent s3c2410 from properly booting.

Quoted 7 lines
Quoted 5 lines

---

 drivers/clk/samsung/clk-s3c2410.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Applied as a fix for 3.16.

Hi Tomasz,

I also have clk fixes for s3c24xx touchscreen, framebuffer, adc, usb and nand drivers (to replace clk_enable with clk_prepare_enable and clk_disable with clk_disable_unprepare),

Quoted 6 lines
Quoted 2 lines

Should they be merged through different trees or it's possible to merge them through a single tree?

I'd say they should all go through respective subsystem trees, unless they directly depend on patches I already applied, which is not the case as far as I can see.

Hm, they aren't necessary for compiling the kernel, but system fails to boot without these patches due to merge of common clk changes for s3c24xx.

Quoted 2 lines

Best regards,
Tomasz

P.S. Sorry for 2nd mail, Tomasz, I've occassionally omited MLs.