[U-Boot] [PATCH 6/8] nhk8815: added keypad

Wolfgang Denk wd at denx.de
Sun Nov 22 23:49:25 CET 2009


Dear Alessandro Rubini,

In message <5a9bb8fb15cdb47d2ccd77af62f759ed493de944.1255086085.git.rubini@ unipv.it> you wrote:
> From: Alessandro Rubini <rubini at unipv.it>
> 
> Signed-off-by: Alessandro Rubini <rubini at unipv.it>
> Acked-by: Andrea Gallo <andrea.gallo at stericsson.com>

Please see previous comments about coding style.

Additionally:

> +int nhk8815_keypad_init(void)
> +{
> +        struct stdio_dev dev;

Indentation by TAB only, please.

> +	printf("%s:%s\n", __FILE__, __func__);

Guess this should be debug() instead?



Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
As long as we're going to reinvent the wheel again, we might as  well
try making it round this time.                        - Mike Dennison


More information about the U-Boot mailing list