public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c clk freq
@ 2013-03-25  9:02 Akshay Saraswat
  2013-03-25  9:02 ` [U-Boot] [PATCH] exynos: uart: Use generic api to the uart " Akshay Saraswat
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Akshay Saraswat @ 2013-03-25  9:02 UTC (permalink / raw)
  To: u-boot

From: Padmavathi Venna <padma.v@samsung.com>

Use generic api to get the i2c clock frequency.
Tested on smdk5250 with the help of following commands.
run
i2c dev 1
i2c dev 2
i2c speed

This patch depends on:
http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/154930

Signed-off-by: Padmavathi Venna <padma.v@samsung.com>
Signed-off-by: Akshay Saraswat <akshay.s@samsung.com>
---
 drivers/i2c/s3c24x0_i2c.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/s3c24x0_i2c.c b/drivers/i2c/s3c24x0_i2c.c
index 769a2ba..a9f19e7 100644
--- a/drivers/i2c/s3c24x0_i2c.c
+++ b/drivers/i2c/s3c24x0_i2c.c
@@ -151,7 +151,7 @@ static void i2c_ch_init(struct s3c24x0_i2c *i2c, int speed, int slaveadd)
 {
 	ulong freq, pres = 16, div;
 #if (defined CONFIG_EXYNOS4 || defined CONFIG_EXYNOS5)
-	freq = get_i2c_clk();
+	freq = clock_get_periph_rate(PERIPH_ID_I2C0);
 #else
 	freq = get_PCLK();
 #endif
-- 
1.8.0

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] exynos: uart: Use generic api to the uart clk freq
  2013-03-25  9:02 [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c clk freq Akshay Saraswat
@ 2013-03-25  9:02 ` Akshay Saraswat
  2013-06-04  8:02   ` Minkyu Kang
  2013-06-04  7:53 ` [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c " Minkyu Kang
  2013-06-04  8:01 ` Minkyu Kang
  2 siblings, 1 reply; 5+ messages in thread
From: Akshay Saraswat @ 2013-03-25  9:02 UTC (permalink / raw)
  To: u-boot

From: Padmavathi Venna <padma.v@samsung.com>

Use generic api to get the uart clock frequency

This patch depends on:
http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/154930

Signed-off-by: Padmavathi Venna <padma.v@samsung.com>
Signed-off-by: Akshay Saraswat <akshay.s@samsung.com>
---
 drivers/serial/serial_s5p.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/serial/serial_s5p.c b/drivers/serial/serial_s5p.c
index 3c41242..7058d55 100644
--- a/drivers/serial/serial_s5p.c
+++ b/drivers/serial/serial_s5p.c
@@ -65,7 +65,7 @@ static const int udivslot[] = {
 void serial_setbrg_dev(const int dev_index)
 {
 	struct s5p_uart *const uart = s5p_get_base_uart(dev_index);
-	u32 uclk = get_uart_clk(dev_index);
+	u32 uclk = clock_get_periph_rate(PERIPH_ID_UART0 + dev_index);
 	u32 baudrate = gd->baudrate;
 	u32 val;
 
-- 
1.8.0

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c clk freq
  2013-03-25  9:02 [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c clk freq Akshay Saraswat
  2013-03-25  9:02 ` [U-Boot] [PATCH] exynos: uart: Use generic api to the uart " Akshay Saraswat
@ 2013-06-04  7:53 ` Minkyu Kang
  2013-06-04  8:01 ` Minkyu Kang
  2 siblings, 0 replies; 5+ messages in thread
From: Minkyu Kang @ 2013-06-04  7:53 UTC (permalink / raw)
  To: u-boot

On 25/03/13 18:02, Akshay Saraswat wrote:
> From: Padmavathi Venna <padma.v@samsung.com>
> 
> Use generic api to get the i2c clock frequency.

Please move following comments to under "---" line.

> Tested on smdk5250 with the help of following commands.
> run
> i2c dev 1
> i2c dev 2
> i2c speed
> 
> This patch depends on:
> http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/154930
> 
> Signed-off-by: Padmavathi Venna <padma.v@samsung.com>
> Signed-off-by: Akshay Saraswat <akshay.s@samsung.com>
> ---
>  drivers/i2c/s3c24x0_i2c.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/i2c/s3c24x0_i2c.c b/drivers/i2c/s3c24x0_i2c.c
> index 769a2ba..a9f19e7 100644
> --- a/drivers/i2c/s3c24x0_i2c.c
> +++ b/drivers/i2c/s3c24x0_i2c.c
> @@ -151,7 +151,7 @@ static void i2c_ch_init(struct s3c24x0_i2c *i2c, int speed, int slaveadd)
>  {
>  	ulong freq, pres = 16, div;
>  #if (defined CONFIG_EXYNOS4 || defined CONFIG_EXYNOS5)
> -	freq = get_i2c_clk();
> +	freq = clock_get_periph_rate(PERIPH_ID_I2C0);
>  #else
>  	freq = get_PCLK();
>  #endif
> 

Thanks,
Minkyu Kang.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c clk freq
  2013-03-25  9:02 [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c clk freq Akshay Saraswat
  2013-03-25  9:02 ` [U-Boot] [PATCH] exynos: uart: Use generic api to the uart " Akshay Saraswat
  2013-06-04  7:53 ` [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c " Minkyu Kang
@ 2013-06-04  8:01 ` Minkyu Kang
  2 siblings, 0 replies; 5+ messages in thread
From: Minkyu Kang @ 2013-06-04  8:01 UTC (permalink / raw)
  To: u-boot

On 25/03/13 18:02, Akshay Saraswat wrote:
> From: Padmavathi Venna <padma.v@samsung.com>
> 
> Use generic api to get the i2c clock frequency.
> Tested on smdk5250 with the help of following commands.
> run
> i2c dev 1
> i2c dev 2
> i2c speed
> 
> This patch depends on:
> http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/154930
> 
> Signed-off-by: Padmavathi Venna <padma.v@samsung.com>
> Signed-off-by: Akshay Saraswat <akshay.s@samsung.com>
> ---
>  drivers/i2c/s3c24x0_i2c.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/i2c/s3c24x0_i2c.c b/drivers/i2c/s3c24x0_i2c.c
> index 769a2ba..a9f19e7 100644
> --- a/drivers/i2c/s3c24x0_i2c.c
> +++ b/drivers/i2c/s3c24x0_i2c.c
> @@ -151,7 +151,7 @@ static void i2c_ch_init(struct s3c24x0_i2c *i2c, int speed, int slaveadd)
>  {
>  	ulong freq, pres = 16, div;
>  #if (defined CONFIG_EXYNOS4 || defined CONFIG_EXYNOS5)
> -	freq = get_i2c_clk();
> +	freq = clock_get_periph_rate(PERIPH_ID_I2C0);

Since clock_get_periph_rate did not implemented on exynos4, exynos4 will get zero.
So, I'll not accept this patch.

>  #else
>  	freq = get_PCLK();
>  #endif
> 

Thanks,
Minkyu Kang.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] exynos: uart: Use generic api to the uart clk freq
  2013-03-25  9:02 ` [U-Boot] [PATCH] exynos: uart: Use generic api to the uart " Akshay Saraswat
@ 2013-06-04  8:02   ` Minkyu Kang
  0 siblings, 0 replies; 5+ messages in thread
From: Minkyu Kang @ 2013-06-04  8:02 UTC (permalink / raw)
  To: u-boot

On 25/03/13 18:02, Akshay Saraswat wrote:
> From: Padmavathi Venna <padma.v@samsung.com>
> 
> Use generic api to get the uart clock frequency
> 
> This patch depends on:
> http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/154930
> 
> Signed-off-by: Padmavathi Venna <padma.v@samsung.com>
> Signed-off-by: Akshay Saraswat <akshay.s@samsung.com>
> ---
>  drivers/serial/serial_s5p.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/serial/serial_s5p.c b/drivers/serial/serial_s5p.c
> index 3c41242..7058d55 100644
> --- a/drivers/serial/serial_s5p.c
> +++ b/drivers/serial/serial_s5p.c
> @@ -65,7 +65,7 @@ static const int udivslot[] = {
>  void serial_setbrg_dev(const int dev_index)
>  {
>  	struct s5p_uart *const uart = s5p_get_base_uart(dev_index);
> -	u32 uclk = get_uart_clk(dev_index);
> +	u32 uclk = clock_get_periph_rate(PERIPH_ID_UART0 + dev_index);
>  	u32 baudrate = gd->baudrate;
>  	u32 val;
>  
> 

Since clock_get_periph_rate did not implemented on exynos4, exynos4 will get zero.
So, I'll not accept this patch.

Thanks,
Minkyu Kang.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-06-04  8:02 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-25  9:02 [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c clk freq Akshay Saraswat
2013-03-25  9:02 ` [U-Boot] [PATCH] exynos: uart: Use generic api to the uart " Akshay Saraswat
2013-06-04  8:02   ` Minkyu Kang
2013-06-04  7:53 ` [U-Boot] [PATCH] exynos: i2c: Use generic api to get i2c " Minkyu Kang
2013-06-04  8:01 ` Minkyu Kang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox