[x-loader][PATCHv2] OMAP3: Name refactoring for current and future IGEP boards

1 view
Skip to first unread message

Enric Balletbo i Serra

unread,
May 17, 2011, 4:09:53 AM5/17/11
to x-lo...@googlegroups.com, Enric Balletbo i Serra
Refactor IGEP2 support and add support for IGEP3 board. These boards are
very similar and can be merged into one file (igep00x0). This
is an attempt to reduce and not duplicate code unnecessarily.

Signed-off-by: Enric Balletbo i Serra <ebal...@gmail.com>
---
MAINTAINERS | 2 +-
Makefile | 4 ++--
board/{igep0020 => igep00x0}/Makefile | 2 +-
board/{igep0020 => igep00x0}/config.mk | 0
board/{igep0020/igep0020.c => igep00x0/igep00x0.c} | 0
board/{igep0020 => igep00x0}/platform.S | 0
board/{igep0020 => igep00x0}/x-load.lds | 0
include/configs/{igep0020.h => igep00x0.h} | 0
8 files changed, 4 insertions(+), 4 deletions(-)
rename board/{igep0020 => igep00x0}/Makefile (98%)
rename board/{igep0020 => igep00x0}/config.mk (100%)
rename board/{igep0020/igep0020.c => igep00x0/igep00x0.c} (100%)
rename board/{igep0020 => igep00x0}/platform.S (100%)
rename board/{igep0020 => igep00x0}/x-load.lds (100%)
rename include/configs/{igep0020.h => igep00x0.h} (100%)

diff --git a/MAINTAINERS b/MAINTAINERS
index a089c34..ccc5c95 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -16,7 +16,7 @@
#########################################################################

Enric Balletbo i Serra, <ebal...@iseebcn.com>
- igep0020 ARM ARMV7 (OMAP3xx SoC)
+ igep00x0 ARM ARMV7 (OMAP3xx SoC)

#########################################################################
# OMAP4 Systems: #
diff --git a/Makefile b/Makefile
index 1930db5..237bc0f 100644
--- a/Makefile
+++ b/Makefile
@@ -225,8 +225,8 @@ overo_config : unconfig
omap3530beagle_config : unconfig
@$(MKCONFIG) $(@:_config=) arm omap3 omap3530beagle

-igep0020_config : unconfig
- @$(MKCONFIG) $(@:_config=) arm omap3 igep0020
+igep00x0_config : unconfig
+ @$(MKCONFIG) $(@:_config=) arm omap3 igep00x0

#########################################################################
## OMAP4 (ARM-CortexA9) Systems
diff --git a/board/igep0020/Makefile b/board/igep00x0/Makefile
similarity index 98%
rename from board/igep0020/Makefile
rename to board/igep00x0/Makefile
index c9a4dad..5382288 100644
--- a/board/igep0020/Makefile
+++ b/board/igep00x0/Makefile
@@ -25,7 +25,7 @@ include $(TOPDIR)/config.mk

LIB = $(obj)lib$(BOARD).a

-COBJS := igep0020.o
+COBJS := igep00x0.o
SOBJS := platform.o

SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c)
diff --git a/board/igep0020/config.mk b/board/igep00x0/config.mk
similarity index 100%
rename from board/igep0020/config.mk
rename to board/igep00x0/config.mk
diff --git a/board/igep0020/igep0020.c b/board/igep00x0/igep00x0.c
similarity index 100%
rename from board/igep0020/igep0020.c
rename to board/igep00x0/igep00x0.c
diff --git a/board/igep0020/platform.S b/board/igep00x0/platform.S
similarity index 100%
rename from board/igep0020/platform.S
rename to board/igep00x0/platform.S
diff --git a/board/igep0020/x-load.lds b/board/igep00x0/x-load.lds
similarity index 100%
rename from board/igep0020/x-load.lds
rename to board/igep00x0/x-load.lds
diff --git a/include/configs/igep0020.h b/include/configs/igep00x0.h
similarity index 100%
rename from include/configs/igep0020.h
rename to include/configs/igep00x0.h
--
1.7.0.4

Nishanth Menon

unread,
May 17, 2011, 7:47:43 AM5/17/11
to x-lo...@googlegroups.com, Enric Balletbo i Serra
Much better.. thanks.. looks fine to me
--
Regards,
Nishanth Menon

Gadiyar, Anand

unread,
May 23, 2011, 7:24:24 AM5/23/11
to x-lo...@googlegroups.com, Enric Balletbo i Serra
Nishanth Menon wrote:
> On 05/17/2011 03:09 AM, Enric Balletbo i Serra wrote:
> > Refactor IGEP2 support and add support for IGEP3 board. These boards are
> > very similar and can be merged into one file (igep00x0). This
> > is an attempt to reduce and not duplicate code unnecessarily.
> >
> > Signed-off-by: Enric Balletbo i Serra<ebal...@gmail.com>
> > ---
> > MAINTAINERS | 2 +-
> > Makefile | 4 ++--
> > board/{igep0020 => igep00x0}/Makefile | 2 +-
> > board/{igep0020 => igep00x0}/config.mk | 0
> > board/{igep0020/igep0020.c => igep00x0/igep00x0.c} | 0
> > board/{igep0020 => igep00x0}/platform.S | 0
> > board/{igep0020 => igep00x0}/x-load.lds | 0
> > include/configs/{igep0020.h => igep00x0.h} | 0
> > 8 files changed, 4 insertions(+), 4 deletions(-)
> > rename board/{igep0020 => igep00x0}/Makefile (98%)
> > rename board/{igep0020 => igep00x0}/config.mk (100%)
> > rename board/{igep0020/igep0020.c => igep00x0/igep00x0.c} (100%)
> > rename board/{igep0020 => igep00x0}/platform.S (100%)
> > rename board/{igep0020 => igep00x0}/x-load.lds (100%)
> > rename include/configs/{igep0020.h => igep00x0.h} (100%)

...

> Much better.. thanks.. looks fine to me

Applied, thanks!

Reply all
Reply to author
Forward
0 new messages