clk: samsung: fix several typos to fix boot on s3c2410
Subsystems (3)
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
@@ -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
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
@@ -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
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.
Signed-off-by Vasily Khoruzhick <[email protected]>
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.cindex bd9a873..140f473 100644drivers/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 */
Tomasz Figa
AppliedPATCH 2/2 clk: samsung: add more aliases for s3c24xx
In reply to Vasily KhoruzhickQuoted 7 lines · Vasily Khoruzhick
Without these aliases clock lookup fails in s3c2410fb, s3cmci, s3c2410-nand, s3c24xx-i2s, and i2c-s3c2410 drivers.
Signed-off-by Vasily Khoruzhick <[email protected]>--- drivers/clk/samsung/clk-s3c2410.c | 5 +++++ 1 file changed, 5 insertions(+)
Applied as a fix for 3.16.
Best regards,
Tomasz
PATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410
In reply to Vasily KhoruzhickAm 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.Signed-off-by Vasily Khoruzhick <[email protected]>
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.cindex ba07168..bd9a873 100644drivers/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,
PATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410
In reply to Vasily KhoruzhickHi 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
Tomasz Figa
AppliedPATCH 1/2 clk: samsung: fix several typos to fix boot on s3c2410
In reply to Vasily KhoruzhickQuoted 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.Signed-off-by Vasily Khoruzhick <[email protected]>--- drivers/clk/samsung/clk-s3c2410.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Applied as a fix for 3.16.
Best regards,
Tomasz
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
Signed-off-by Vasily Khoruzhick <[email protected]>---
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
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
Signed-off-by Vasily Khoruzhick <[email protected]>---
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_enablewithclk_prepare_enableandclk_disablewithclk_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
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
Signed-off-by Vasily Khoruzhick <[email protected]>---
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_enablewithclk_prepare_enableandclk_disablewithclk_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.