Re: [U-Boot] [PATCH 2/2 v6] Add support for MINI2440 (s3c2440).

2012-04-30 Thread Marek Vasut
Dear Gabriel Huau,

I detect line over 80 in the configs/s3c2440.h-whatever file, but just 
visually, 
so you might want to prove me wrong by running checkpatch ;-)

> Signed-off-by: Gabriel Huau 
> 
> ---
> Changes for v2:
>   - Coding style cleanup
>   - Remove unnecessary files modification
>   - Remove unnecessary board configuration set
> 
> Changes for v3:
>   - Coding style cleanup
>   - Move some macro definition from lowlevel_init.S
>   to a new header
>   - Remove some "magic bloat" with I/O board initialization
>   - Add a pll_delay and replace loop by it
>   - Somme cleanup in the configuration file
>   - Cancel modifications on an SoC specific header
>   - Add my name to copyright
> 
> Changes for v4:
>   - Move dram init to dram_init() instead low_levelinit
>   - Remove u-boot env from configuration file and change
>   the address of initial SP
>   - Remove PLL init, now it's SoC specific
> 
> Changes for v5:
>   - Clean up configuration file
>   - Add a MAINTAINERS entry
>   - Add a README.mini2440 file
>   - Use gpio/iomux interface in case of magic numbers
>   - Use get_ram_size()
> 
> Changes for v6:
>   - Coding style cleanup
>   - Remove some unused define in the board config
> ---
>  MAINTAINERS   |4 +
>  board/friendlyarm/mini2440/Makefile   |   44 
>  board/friendlyarm/mini2440/mini2440.c |  132 +++
>  board/friendlyarm/mini2440/mini2440.h |  144 +
>  boards.cfg|1 +
>  doc/README.mini2440   |   28 +
>  include/configs/mini2440.h|  186
> + 7 files changed, 539 insertions(+)
>  create mode 100644 board/friendlyarm/mini2440/Makefile
>  create mode 100644 board/friendlyarm/mini2440/mini2440.c
>  create mode 100644 board/friendlyarm/mini2440/mini2440.h
>  create mode 100644 doc/README.mini2440
>  create mode 100644 include/configs/mini2440.h
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 708ded7..2611fb5 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -566,6 +566,10 @@ Unknown / orphaned boards:
>  #Board   CPU #
>  #
> 
> +Gabriel HUAU 
> +
> + mini2440s3c2440
> +
>  Albert ARIBAUD 
> 
>   edminiv2ARM926EJS (Orion5x SoC)
> diff --git a/board/friendlyarm/mini2440/Makefile
> b/board/friendlyarm/mini2440/Makefile new file mode 100644
> index 000..b88e569
> --- /dev/null
> +++ b/board/friendlyarm/mini2440/Makefile
> @@ -0,0 +1,44 @@
> +#
> +# (C) Copyright 2012
> +# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
> +#
> +# See file CREDITS for list of people who contributed to this
> +# project.
> +#
> +# This program is free software; you can redistribute it and/or
> +# modify it under the terms of the GNU General Public License as
> +# published by the Free Software Foundation; either version 2 of
> +# the License, or (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +# GNU General Public License for more details.
> +#
> +# 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., 59 Temple Place, Suite 330, Boston,
> +# MA 02111-1307 USA
> +#
> +
> +include $(TOPDIR)/config.mk
> +
> +LIB  = $(obj)lib$(BOARD).o
> +
> +COBJS:= mini2440.o
> +
> +SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c)
> +OBJS := $(addprefix $(obj),$(COBJS))
> +SOBJS:= $(addprefix $(obj),$(SOBJS))
> +
> +$(LIB):  $(obj).depend $(OBJS) $(SOBJS)
> + $(call cmd_link_o_target, $(OBJS) $(SOBJS))
> +
> +#
> +
> +# defines $(obj).depend target
> +include $(SRCTREE)/rules.mk
> +
> +sinclude $(obj).depend
> +
> +#
> diff --git a/board/friendlyarm/mini2440/mini2440.c
> b/board/friendlyarm/mini2440/mini2440.c new file mode 100644
> index 000..bd726e7
> --- /dev/null
> +++ b/board/friendlyarm/mini2440/mini2440.c
> @@ -0,0 +1,132 @@
> +/*
> + * (C) Copyright 2002
> + * Sysgo Real-Time Solutions, GmbH 
> + * Marius Groeger 
> + *
> + * (C) Copyright 2002
> + * David Mueller, ELSOFT AG, 
> + *
> + * (C) Copyright 2009
> + * Michel Pollet 
> + *
> + * (C) Copyright 2012
> + * Gabriel Huau 
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundat

[U-Boot] [PATCH 2/2 v6] Add support for MINI2440 (s3c2440).

2012-04-30 Thread Gabriel Huau
Signed-off-by: Gabriel Huau 

---
Changes for v2:
- Coding style cleanup
- Remove unnecessary files modification
- Remove unnecessary board configuration set

Changes for v3:
- Coding style cleanup
- Move some macro definition from lowlevel_init.S
to a new header
- Remove some "magic bloat" with I/O board initialization
- Add a pll_delay and replace loop by it
- Somme cleanup in the configuration file
- Cancel modifications on an SoC specific header
- Add my name to copyright

Changes for v4:
- Move dram init to dram_init() instead low_levelinit
- Remove u-boot env from configuration file and change
the address of initial SP
- Remove PLL init, now it's SoC specific

Changes for v5:
- Clean up configuration file
- Add a MAINTAINERS entry
- Add a README.mini2440 file
- Use gpio/iomux interface in case of magic numbers
- Use get_ram_size()

Changes for v6:
- Coding style cleanup
- Remove some unused define in the board config
---
 MAINTAINERS   |4 +
 board/friendlyarm/mini2440/Makefile   |   44 
 board/friendlyarm/mini2440/mini2440.c |  132 +++
 board/friendlyarm/mini2440/mini2440.h |  144 +
 boards.cfg|1 +
 doc/README.mini2440   |   28 +
 include/configs/mini2440.h|  186 +
 7 files changed, 539 insertions(+)
 create mode 100644 board/friendlyarm/mini2440/Makefile
 create mode 100644 board/friendlyarm/mini2440/mini2440.c
 create mode 100644 board/friendlyarm/mini2440/mini2440.h
 create mode 100644 doc/README.mini2440
 create mode 100644 include/configs/mini2440.h

diff --git a/MAINTAINERS b/MAINTAINERS
index 708ded7..2611fb5 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -566,6 +566,10 @@ Unknown / orphaned boards:
 #  Board   CPU #
 #
 
+Gabriel HUAU 
+
+   mini2440s3c2440
+
 Albert ARIBAUD 
 
edminiv2ARM926EJS (Orion5x SoC)
diff --git a/board/friendlyarm/mini2440/Makefile 
b/board/friendlyarm/mini2440/Makefile
new file mode 100644
index 000..b88e569
--- /dev/null
+++ b/board/friendlyarm/mini2440/Makefile
@@ -0,0 +1,44 @@
+#
+# (C) Copyright 2012
+# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
+#
+# See file CREDITS for list of people who contributed to this
+# project.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation; either version 2 of
+# the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# 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., 59 Temple Place, Suite 330, Boston,
+# MA 02111-1307 USA
+#
+
+include $(TOPDIR)/config.mk
+
+LIB= $(obj)lib$(BOARD).o
+
+COBJS  := mini2440.o
+
+SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
+OBJS   := $(addprefix $(obj),$(COBJS))
+SOBJS  := $(addprefix $(obj),$(SOBJS))
+
+$(LIB):$(obj).depend $(OBJS) $(SOBJS)
+   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
+
+#
+
+# defines $(obj).depend target
+include $(SRCTREE)/rules.mk
+
+sinclude $(obj).depend
+
+#
diff --git a/board/friendlyarm/mini2440/mini2440.c 
b/board/friendlyarm/mini2440/mini2440.c
new file mode 100644
index 000..bd726e7
--- /dev/null
+++ b/board/friendlyarm/mini2440/mini2440.c
@@ -0,0 +1,132 @@
+/*
+ * (C) Copyright 2002
+ * Sysgo Real-Time Solutions, GmbH 
+ * Marius Groeger 
+ *
+ * (C) Copyright 2002
+ * David Mueller, ELSOFT AG, 
+ *
+ * (C) Copyright 2009
+ * Michel Pollet 
+ *
+ * (C) Copyright 2012
+ * Gabriel Huau 
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General