Hi Giuseppe,

On Mon, Nov 11, 2013 at 3:11 PM, Giuseppe Pagano
<giuseppe.pag...@seco.com> wrote:
> Previous uDoo configuration adopts register settings for DDR3, clock, muxing, 
> etc. taken from Nitrogen6x.  uDoo schematics is rather different from that 
> board, and it needs customized setting for most of the registers.
> All this changes can be considered atomical since it is part of initial 
> support of the board.
>
> Patch changes uDoo configuration files path to a specific one, and adopt 
> optimized value for every configured register.
>
> Signed-off-by: Giuseppe Pagano <giuseppe.pag...@seco.com>
> CC: Stefano Babic <sba...@denx.de>
> CC: Fabio Estevam <fabio.este...@freescale.com>

Tested-by: Fabio Estevam <fabio.este...@freescale.com>

Thanks.

Only one minor nit below:

> ---
>  board/udoo/1066mhz_4x256mx16.cfg |   56 ++++++++++++++++++++++++
>  board/udoo/clocks.cfg            |   32 ++++++++++++++
>  board/udoo/ddr-setup.cfg         |   87 
> ++++++++++++++++++++++++++++++++++++++
>  board/udoo/udoo.cfg              |   29 +++++++++++++
>  boards.cfg                       |    2 +-
>  5 files changed, 205 insertions(+), 1 deletion(-)
>  create mode 100644 board/udoo/1066mhz_4x256mx16.cfg
>  create mode 100644 board/udoo/clocks.cfg
>  create mode 100644 board/udoo/ddr-setup.cfg
>  create mode 100644 board/udoo/udoo.cfg
>
> diff --git a/board/udoo/1066mhz_4x256mx16.cfg 
> b/board/udoo/1066mhz_4x256mx16.cfg
> new file mode 100644
> index 0000000..539e3f6
> --- /dev/null
> +++ b/board/udoo/1066mhz_4x256mx16.cfg
> @@ -0,0 +1,56 @@
> +/*
> + * Copyright (C) 2013 Boundary Devices
> + *
> + * SPDX-License-Identifier:    GPL-2.0+
> + */
> +
> +

No need to add two lines here. One line is enough.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to