[U-Boot] [PATCH] README: Give an explicit example of adding a new board.

From: Dirk Behme <dirk.behme@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] README: Give an explicit example of adding a new	board.
Date: Tue, 17 Nov 2009 11:02:55 +0100	[thread overview]
Message-ID: <4B0274CF.3040605@googlemail.com> (raw)
In-Reply-To: <alpine.LFD.2.00.0911170419590.9417@localhost>

Robert P. J. Day wrote:
> Explicitly demonstrate an example (BeagleBoard) of what was
> added/changed to support a new board.
> 
> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> 
> ---
> 
>   obviously not critical, but it might be useful for readers to see
> explicitly what constitutes adding support for a new board, file by
> file.  your choice to apply it or not.  i *think* i got everything.
> 
> 
> diff --git a/README b/README
> index 2c77687..cc701b1 100644
> --- a/README
> +++ b/README
> @@ -2874,6 +2874,73 @@ steps:
>      [Of course, this last step is much harder than it sounds.]
> 
> 
> +A real-life example of adding a new board
> +=========================================
> +
> +  As a working example of adding a new board, consider the files
> +and directories that were added/changed to support the OMAP3-based
> +BeagleBoard.
> +
> +  First, the new files:
> +
> +  ./board/ti/beagle/
> +      beagle.c
> +      beagle.h
> +      config.mk
> +      Makefile
> +  ./include/configs/omap3_beagle.h

Most probably the naming convention guys here want some hints about 
correct directory and config file names ;)

> +Followed by changes to existing files:
> +
> +	Makefile
> +	========
> +
> +  omap3_beagle_config :   unconfig
> +        @$(MKCONFIG) $(@:_config=) arm arm_cortexa8 beagle ti omap3
> +
> +	MAKEALL
> +	=======
> +
> +  LIST_ARM_CORTEX_A8="            \
> +        devkit8000              \
> +        omap3_beagle            \
> +        omap3_overo             \
> +        ...
> +
> +	include/asm-arm-types.h
> +	=======================

No. Never ever touch this file manually. Instead, add an entry to 
Russell's machine registry and then ask ARM maintainer to import this 
file.

Most probably you have to mention that this is ARM specific.

> +  #ifdef CONFIG_MACH_OMAP3_BEAGLE
> +  # ifdef machine_arch_type
> +  #  undef machine_arch_type
> +  #  define machine_arch_type     __machine_arch_type
> +  # else
> +  #  define machine_arch_type     MACH_TYPE_OMAP3_BEAGLE
> +  # endif
> +  # define machine_is_omap3_beagle() (machine_arch_type == MACH_TYPE_OMAP3_BEAGLE)
> +  #else
> +  # define machine_is_omap3_beagle()      (0)
> +  #endif
> +
> +	doc/README.omap3
> +	================
> +
> +  Build
> +  =====
> +
> +  * BeagleBoard:
> +
> +  make omap3_beagle_config
> +  make

Or './MAKEALL omap3_beagle' to easily catch compiler warnings.

Best regards

Dirk

> +	MAINTAINERS
> +	===========
> +
> +  Dirk Behme <dirk.behme@gmail.com>
> +
> +        omap3_beagle    ARM CORTEX-A8 (OMAP3530 SoC)
> +
> +
>  Testing of U-Boot Modifications, Ports to New Hardware, etc.:
>  ==============================================================
> 
> rday
> --
> 
> ========================================================================
> Robert P. J. Day                               Waterloo, Ontario, CANADA
> 
>             Linux Consulting, Training and Kernel Pedantry.
> 
> Web page:                                          http://crashcourse.ca
> Twitter:                                       http://twitter.com/rpjday
> ========================================================================
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
> 