[U-Boot] [PATCH] imx: imx31_phycore.h: fix checkpatch warnings

Fabio Estevam festevam at gmail.com
Sun Oct 16 00:18:49 CEST 2011


Hi Anatolij,

On Sat, Oct 15, 2011 at 7:00 PM, Anatolij Gustschin <agust at denx.de> wrote:
> Cleanup board config file and fix issues reported by
> checkpatch.pl script.
>
> Signed-off-by: Anatolij Gustschin <agust at denx.de>
> Cc: Stefano Babic <sbabic at denx.de>
> ---
>  include/configs/imx31_phycore.h |  112 +++++++++++++++++++++++----------------
>  1 files changed, 67 insertions(+), 45 deletions(-)
>
> diff --git a/include/configs/imx31_phycore.h b/include/configs/imx31_phycore.h
> index 48bd50b..f8cc5ec 100644
> --- a/include/configs/imx31_phycore.h
> +++ b/include/configs/imx31_phycore.h
> @@ -30,7 +30,7 @@
>
>  #include <asm/arch/imx-regs.h>
>
> - /* High Level Configuration Options */
> +/* High Level Configuration Options */
>  #define CONFIG_ARM1136         1    /* This is an arm1136 CPU core */
>  #define CONFIG_MX31            1    /* in a mx31 */

While you are at it, maybe it is also a good idea to change:

#define ABCD 1

to

#define ABCD

(ie, remove the "1")

that appears several times in this file.

Regards,

Fabio Estevam


More information about the U-Boot mailing list