[U-Boot] [PATCH v4] add new board nas62x0

[Igor Grinberg <grinberg@compulab.co.il>]

Hi Luka,

On 03/21/12 02:53, Luka Perkov wrote:
> Add support for new boards RaidSonic ICY BOX NAS6210 and NAS6220.
> 
> Only difference between boards is number of SATA ports.
> 
> Signed-off-by: Luka Perkov <uboot@lukaperkov.net>
> Signed-off-by: Gerald Kerma <dreagle@doukki.net>
> ---
> 
> Changes from version v3:
> 
>  * don't use utf8 for Geralds name
>  * don't mix upper/lower case in kwbimage.cfg
>  * fix prompt
> 
> Found problems that should be looked at (this is not related with adding
> support for this board):
> 
>  * IB62x0_OE_LOW and IB62x0_OE_HIGH issue
>  * CONFIG_SKIP_LOWLEVEL_INIT issue
> 
> In separate patch I'll resend fix for CONFIG_SKIP_LOWLEVEL_INIT.
> 
> 
>  MAINTAINERS                       |    4 +
>  board/Marvell/ib62x0/Makefile     |   51 +++++++++++
>  board/Marvell/ib62x0/ib62x0.c     |  114 ++++++++++++++++++++++++
>  board/Marvell/ib62x0/ib62x0.h     |   41 +++++++++
>  board/Marvell/ib62x0/kwbimage.cfg |  167 +++++++++++++++++++++++++++++++++++
>  boards.cfg                        |    2 +
>  include/configs/ib62x0.h          |  176 +++++++++++++++++++++++++++++++++++++
>  7 files changed, 555 insertions(+)

[...]

> diff --git a/board/Marvell/ib62x0/Makefile b/board/Marvell/ib62x0/Makefile
> new file mode 100644
> index 0000000..3d82f27
> --- /dev/null
> +++ b/board/Marvell/ib62x0/Makefile
> @@ -0,0 +1,51 @@
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> +# MA 02110-1301 USA

It is better to remove the postal address, as it is subject to change.
Please, check all files you add.

> +clean:
> +	rm -f $(SOBJS) $(OBJS)
> +
> +distclean:	clean
> +	rm -f $(LIB) core *.bak .depend

clean and distclean have no effect here and should be removed.

[...]

> diff --git a/board/Marvell/ib62x0/ib62x0.c b/board/Marvell/ib62x0/ib62x0.c
> new file mode 100644
> index 0000000..e6853e7
> --- /dev/null
> +++ b/board/Marvell/ib62x0/ib62x0.c

[...]

> +int board_init(void)
> +{
> +	/*
> +	 * arch number of board
> +	 */
> +	gd->bd->bi_arch_number = MACH_TYPE_NAS6210;

This should be handled in a generic way:
#define CONFIG_MACH_TYPE MACH_TYPE_NAS6210
in your board config file.

> +
> +	/* adress of boot parameters */
> +	gd->bd->bi_boot_params = kw_sdram_bar(0) + 0x100;
> +
> +	return 0;
> +}

[...]


-- 
Regards,
Igor.