diff mbox

[U-Boot] km/common: remove printfs for i2c deblocking code

Message ID 1343401093-31790-1-git-send-email-gerlando.falauto@keymile.com
State Accepted
Commit e69e482bbf16795539af112d6a21d4ddaee35105
Delegated to: Heiko Schocher
Headers show

Commit Message

Gerlando Falauto July 27, 2012, 2:58 p.m. UTC
From: Holger Brunck <holger.brunck@keymile.com>

This code will also be used before reallocation and during this time we
are not allowed to do these printings.

Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
---
 board/keymile/common/common.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

Comments

Prafulla Wadaskar July 30, 2012, 6:34 a.m. UTC | #1
> -----Original Message-----
> From: u-boot-bounces@lists.denx.de [mailto:u-boot-
> bounces@lists.denx.de] On Behalf Of Gerlando Falauto
> Sent: 27 July 2012 20:28
> To: u-boot@lists.denx.de
> Cc: Holger Brunck
> Subject: [U-Boot] [PATCH] km/common: remove printfs for i2c deblocking
> code
> 
> From: Holger Brunck <holger.brunck@keymile.com>
> 
> This code will also be used before reallocation and during this time
> we
> are not allowed to do these printings.
> 
> Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
> ---
>  board/keymile/common/common.c |    3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
> 
> diff --git a/board/keymile/common/common.c
> b/board/keymile/common/common.c
> index 1013f42..a90f112 100644
> --- a/board/keymile/common/common.c
> +++ b/board/keymile/common/common.c
> @@ -151,7 +151,6 @@ int i2c_make_abort(void)
>  			sda_state = get_sda();
>  			if (scl_state && sda_state) {
>  				ret = 0;
> -				printf("[INFO] i2c abort after %d clocks\n", i);
>  				break;
>  			}
>  		}
> @@ -159,8 +158,6 @@ int i2c_make_abort(void)
>  	if (ret == 0)
>  		for (i = 0; i < 5; i++)
>  			i2c_write_start_seq();
> -	else
> -		printf("[ERROR] i2c abort failed\n");
> 
>  	/* respect stop setup time */
>  	udelay(DELAY_ABORT_SEQ);
> --

Acked-By: Prafulla Wadaskar <Prafulla@marvell.com>

Regards...
Prafulla . . .
Heiko Schocher July 30, 2012, 9:22 a.m. UTC | #2
Hello Gerlando,

On 27.07.2012 16:58, Gerlando Falauto wrote:
> From: Holger Brunck<holger.brunck@keymile.com>
>
> This code will also be used before reallocation and during this time we
> are not allowed to do these printings.
>
> Signed-off-by: Holger Brunck<holger.brunck@keymile.com>
> ---
>   board/keymile/common/common.c |    3 ---
>   1 files changed, 0 insertions(+), 3 deletions(-)

Acked-by: Heiko Schocher<hs@denx.de>

@Prafulla: Is it OK for you, if I pick this patch up to my i2c tree?

bye,
Heiko
Prafulla Wadaskar July 30, 2012, 6:15 p.m. UTC | #3
> -----Original Message-----
> From: Heiko Schocher [mailto:hs@denx.de]
> Sent: 30 July 2012 14:53
> To: Gerlando Falauto
> Cc: u-boot@lists.denx.de; Holger Brunck; Prafulla Wadaskar
> Subject: Re: [U-Boot] [PATCH] km/common: remove printfs for i2c
> deblocking code
> 
> Hello Gerlando,
> 
> On 27.07.2012 16:58, Gerlando Falauto wrote:
> > From: Holger Brunck<holger.brunck@keymile.com>
> >
> > This code will also be used before reallocation and during this time
> we
> > are not allowed to do these printings.
> >
> > Signed-off-by: Holger Brunck<holger.brunck@keymile.com>
> > ---
> >   board/keymile/common/common.c |    3 ---
> >   1 files changed, 0 insertions(+), 3 deletions(-)
> 
> Acked-by: Heiko Schocher<hs@denx.de>
> 
> @Prafulla: Is it OK for you, if I pick this patch up to my i2c tree?
> 

Hi Heiko
Yes, I am okay with it, please pick it.

Regards...
Prafulla . . .
diff mbox

Patch

diff --git a/board/keymile/common/common.c b/board/keymile/common/common.c
index 1013f42..a90f112 100644
--- a/board/keymile/common/common.c
+++ b/board/keymile/common/common.c
@@ -151,7 +151,6 @@  int i2c_make_abort(void)
 			sda_state = get_sda();
 			if (scl_state && sda_state) {
 				ret = 0;
-				printf("[INFO] i2c abort after %d clocks\n", i);
 				break;
 			}
 		}
@@ -159,8 +158,6 @@  int i2c_make_abort(void)
 	if (ret == 0)
 		for (i = 0; i < 5; i++)
 			i2c_write_start_seq();
-	else
-		printf("[ERROR] i2c abort failed\n");
 
 	/* respect stop setup time */
 	udelay(DELAY_ABORT_SEQ);