[U-Boot] [PATCH 03/21] common/lcd: Add command for writing to lcd-display

[Nikita Kiryanov]

Hi Hannes,

On 01/30/2015 03:25 PM, Hannes Petermaier wrote:
> We need this function if we want to make some outputs out of u-boot scripts.
>

I think this commit message is missing information. What makes this necessary?
Why can't your script use regular echo commands with the lcd console enabled?

> Signed-off-by: Hannes Petermaier <oe5hpm at oevsv.at>
> ---
>   common/lcd.c |   17 +++++++++++++++++
>   1 file changed, 17 insertions(+)
>
> diff --git a/common/lcd.c b/common/lcd.c
> index f418da9..755388f 100644
> --- a/common/lcd.c
> +++ b/common/lcd.c
> @@ -279,6 +279,17 @@ static int do_lcd_clear(cmd_tbl_t *cmdtp, int flag, int argc,
>   	return 0;
>   }
>
> +static int do_lcd_puts(cmd_tbl_t *cmdtp, int flag, int argc,
> +		       char *const argv[])
> +{
> +	if (argc != 2)
> +		return CMD_RET_USAGE;
> +
> +	lcd_puts(argv[1]);
> +
> +	return 0;
> +}
> +
>   static int do_lcd_setcursor(cmd_tbl_t *cmdtp, int flag, int argc,
>   			    char *const argv[])
>   {
> @@ -306,6 +317,12 @@ U_BOOT_CMD(
>   	"    <col> <row> in character"
>   );
>
> +U_BOOT_CMD(
> +	puts,	2,	1,	do_lcd_puts,

"puts" is too generic for an lcd specific function. I would expect to see something
with an "lcd" prefix. Also, this code seems better suited for lcd_console.c

> +	"print string on lcd-framebuffer",
> +	"    <string>"
> +);
> +
>   /*----------------------------------------------------------------------*/
>
>   static int lcd_init(void *lcdbase)
>

-- 
Regards,
Nikita Kiryanov