[U-Boot] [PATCH] omap3/am33xx: mux: fix several checkpatch issues

Stefan Roese sr at denx.de
Tue Oct 21 17:36:11 CEST 2014


On 21.10.2014 17:25, Igor Grinberg wrote:
> Fix the following checkpatch issues:
>
> CHECK: No space is necessary after a cast
> \#39: FILE: arch/arm/include/asm/arch-am33xx/mux.h:39:
> +#define PAD_CTRL_BASE  0x800
> +#define OFFSET(x)      (unsigned int) (&((struct pad_signals *) \
>
> CHECK: Avoid CamelCase: <CONTROL_PADCONF_JTAG_nTRST>
> \#284: FILE: arch/arm/include/asm/arch-omap3/mux.h:284:
> +#define CONTROL_PADCONF_JTAG_nTRST     0x0A1C
>
> ERROR: space required after that ',' (ctx:VxV)
> \#446: FILE: arch/arm/include/asm/arch-omap3/mux.h:446:
> +#define MUX_VAL(OFFSET,VALUE)\
>                         ^
> Cc: Raphael Assenat <raph at 8d.com>
> Cc: Ilya Yanok <yanok at emcraft.com>
> Cc: Vaibhav Hiremath <hvaibhav at ti.com>
> Cc: Peter Barada <peter.barada at logicpd.com>
> Cc: Grazvydas Ignotas <notasas at gmail.com>
> Cc: Stefan Roese <sr at denx.de>
> Cc: Stefano Babic <sbabic at denx.de>
> Cc: Nagendra T S <nagendra at mistralsolutions.com>
> Cc: Nishanth Menon <nm at ti.com>
> Cc: Tom Rini <trini at ti.com>
> Signed-off-by: Igor Grinberg <grinberg at compulab.co.il>

Looks good, so:

Acked-by: Stefan Roese <sr at denx.de>

Thanks,
Stefan



More information about the U-Boot mailing list