public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] serial: arm_dcc: Convert to use default_serial_puts
@ 2013-08-17  7:39 Axel Lin
  2013-08-18 20:19 ` Marek Vasut
  2013-08-19 13:49 ` Tom Rini
  0 siblings, 2 replies; 4+ messages in thread
From: Axel Lin @ 2013-08-17  7:39 UTC (permalink / raw)
  To: u-boot

Use default_serial_puts() instead of its own implementation.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
 drivers/serial/arm_dcc.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/drivers/serial/arm_dcc.c b/drivers/serial/arm_dcc.c
index 29d9295..5dfb02f 100644
--- a/drivers/serial/arm_dcc.c
+++ b/drivers/serial/arm_dcc.c
@@ -123,12 +123,6 @@ static void arm_dcc_putc(char ch)
 		write_dcc(ch);
 }
 
-static void arm_dcc_puts(const char *s)
-{
-	while (*s)
-		arm_dcc_putc(*s++);
-}
-
 static int arm_dcc_tstc(void)
 {
 	register unsigned int reg;
@@ -148,7 +142,7 @@ static struct serial_device arm_dcc_drv = {
 	.stop	= NULL,
 	.setbrg	= arm_dcc_setbrg,
 	.putc	= arm_dcc_putc,
-	.puts	= arm_dcc_puts,
+	.puts	= default_serial_puts,
 	.getc	= arm_dcc_getc,
 	.tstc	= arm_dcc_tstc,
 };
-- 
1.8.1.2

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

* [U-Boot] [PATCH] serial: arm_dcc: Convert to use default_serial_puts
  2013-08-17  7:39 [U-Boot] [PATCH] serial: arm_dcc: Convert to use default_serial_puts Axel Lin
@ 2013-08-18 20:19 ` Marek Vasut
  2013-08-19  5:30   ` Michal Simek
  2013-08-19 13:49 ` Tom Rini
  1 sibling, 1 reply; 4+ messages in thread
From: Marek Vasut @ 2013-08-18 20:19 UTC (permalink / raw)
  To: u-boot

Dear Axel Lin,

> Use default_serial_puts() instead of its own implementation.
> 
> Signed-off-by: Axel Lin <axel.lin@ingics.com>

Thanks,

Acked-by: Marek Vasut <marex@denx.de>

> ---
>  drivers/serial/arm_dcc.c | 8 +-------
>  1 file changed, 1 insertion(+), 7 deletions(-)
> 
> diff --git a/drivers/serial/arm_dcc.c b/drivers/serial/arm_dcc.c
> index 29d9295..5dfb02f 100644
> --- a/drivers/serial/arm_dcc.c
> +++ b/drivers/serial/arm_dcc.c
> @@ -123,12 +123,6 @@ static void arm_dcc_putc(char ch)
>  		write_dcc(ch);
>  }
> 
> -static void arm_dcc_puts(const char *s)
> -{
> -	while (*s)
> -		arm_dcc_putc(*s++);
> -}
> -
>  static int arm_dcc_tstc(void)
>  {
>  	register unsigned int reg;
> @@ -148,7 +142,7 @@ static struct serial_device arm_dcc_drv = {
>  	.stop	= NULL,
>  	.setbrg	= arm_dcc_setbrg,
>  	.putc	= arm_dcc_putc,
> -	.puts	= arm_dcc_puts,
> +	.puts	= default_serial_puts,
>  	.getc	= arm_dcc_getc,
>  	.tstc	= arm_dcc_tstc,
>  };

Best regards,
Marek Vasut

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

* [U-Boot] [PATCH] serial: arm_dcc: Convert to use default_serial_puts
  2013-08-18 20:19 ` Marek Vasut
@ 2013-08-19  5:30   ` Michal Simek
  0 siblings, 0 replies; 4+ messages in thread
From: Michal Simek @ 2013-08-19  5:30 UTC (permalink / raw)
  To: u-boot

On 08/18/2013 10:19 PM, Marek Vasut wrote:
> Dear Axel Lin,
> 
>> Use default_serial_puts() instead of its own implementation.
>>
>> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> 
> Thanks,
> 
> Acked-by: Marek Vasut <marex@denx.de>

Ah yeah.

Acked-by: Michal Simek <monstr@monstr.eu>

Tom: Can you please apply this patch directly to your tree?

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130819/27334863/attachment.pgp>

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

* [U-Boot] [PATCH] serial: arm_dcc: Convert to use default_serial_puts
  2013-08-17  7:39 [U-Boot] [PATCH] serial: arm_dcc: Convert to use default_serial_puts Axel Lin
  2013-08-18 20:19 ` Marek Vasut
@ 2013-08-19 13:49 ` Tom Rini
  1 sibling, 0 replies; 4+ messages in thread
From: Tom Rini @ 2013-08-19 13:49 UTC (permalink / raw)
  To: u-boot

On Sat, Aug 17, 2013 at 03:39:34PM +0800, Axel Lin wrote:

> Use default_serial_puts() instead of its own implementation.
> 
> Signed-off-by: Axel Lin <axel.lin@ingics.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130819/1c6135f9/attachment.pgp>

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

end of thread, other threads:[~2013-08-19 13:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-17  7:39 [U-Boot] [PATCH] serial: arm_dcc: Convert to use default_serial_puts Axel Lin
2013-08-18 20:19 ` Marek Vasut
2013-08-19  5:30   ` Michal Simek
2013-08-19 13:49 ` Tom Rini

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