Re: [PATCH 3/5] arm/tms570: include TMS570_USE_HWINIT_STARTUP option to select bare metal startup and selftest.

2016-09-14 Thread Gedare Bloom
Can you provide some more elaboration in the RTEMS_BSPOPTS_HELP
description of the option? Such as, why to turn it on or to leave it
off.

On Mon, Sep 12, 2016 at 5:47 PM,   wrote:
> From: Pavel Pisa 
>
> ---
>  c/src/lib/libbsp/arm/tms570/Makefile.am  | 21 +
>  c/src/lib/libbsp/arm/tms570/configure.ac |  4 
>  2 files changed, 25 insertions(+)
>
> diff --git a/c/src/lib/libbsp/arm/tms570/Makefile.am 
> b/c/src/lib/libbsp/arm/tms570/Makefile.am
> index c4d39bc..012ce23 100644
> --- a/c/src/lib/libbsp/arm/tms570/Makefile.am
> +++ b/c/src/lib/libbsp/arm/tms570/Makefile.am
> @@ -168,8 +168,29 @@ libbsp_a_SOURCES += 
> ../../../libcpu/shared/src/cache_manager.c
>  libbsp_a_SOURCES += ../../../libcpu/arm/shared/include/cache_.h
>  libbsp_a_CPPFLAGS += -I$(srcdir)/../../../libcpu/arm/shared/include
>
> +if TMS570_USE_HWINIT_STARTUP
> +libbsp_a_SOURCES += hwinit/tms570_sys_core.S
> +libbsp_a_SOURCES += hwinit/fail_notification.c
> +libbsp_a_SOURCES += hwinit/init_pinmux.c
> +libbsp_a_SOURCES += hwinit/init_system.c
> +libbsp_a_SOURCES += hwinit/init_emif_sdram.c
> +libbsp_a_SOURCES += hwinit/init_esm.c
> +libbsp_a_SOURCES += hwinit/tms570_selftest.c
> +libbsp_a_SOURCES += hwinit/tms570_parity_tests.c
> +libbsp_a_SOURCES += hwinit/tms570_parity_std.c
> +libbsp_a_SOURCES += hwinit/tms570_parity_can.c
> +libbsp_a_SOURCES += hwinit/tms570_parity_mibspi.c
> +
> +include_bsp_HEADERS += hwinit/tms570_parity_tests.h
> +include_bsp_HEADERS += hwinit/tms570_hwinit.h
> +include_bsp_HEADERS += hwinit/tms570_selftest.h
> +
> +# Start hooks
> +libbsp_a_SOURCES += hwinit/bspstarthooks-hwinit.c
> +else
>  # Start hooks
>  libbsp_a_SOURCES += startup/bspstarthooks.c
> +endif
>
>  # 
>  # --  Special Rules
> diff --git a/c/src/lib/libbsp/arm/tms570/configure.ac 
> b/c/src/lib/libbsp/arm/tms570/configure.ac
> index e06d4e4..7662064 100644
> --- a/c/src/lib/libbsp/arm/tms570/configure.ac
> +++ b/c/src/lib/libbsp/arm/tms570/configure.ac
> @@ -50,6 +50,10 @@ RTEMS_BSPOPTS_HELP([TMS570_OSCILLATOR_RTC],[RTC oscillator 
> frequency in Hz])
>  RTEMS_BSPOPTS_SET([TMS570_CCLK],[*],[9600U])
>  RTEMS_BSPOPTS_HELP([TMS570_CCLK],[CPU clock in Hz])
>
> +RTEMS_BSPOPTS_SET([TMS570_USE_HWINIT_STARTUP],[*],[0])
> +RTEMS_BSPOPTS_HELP([TMS570_USE_HWINIT_STARTUP],[use minimalized startup code 
> for TMS570LS3137])
> +AM_CONDITIONAL(TMS570_USE_HWINIT_STARTUP,test "$TMS570_USE_HWINIT_STARTUP" = 
> "1")
> +
>  RTEMS_BSP_CLEANUP_OPTIONS(0, 1)
>  RTEMS_BSP_LINKCMDS
>
> --
> 1.9.1
>
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel


[PATCH 3/5] arm/tms570: include TMS570_USE_HWINIT_STARTUP option to select bare metal startup and selftest.

2016-09-12 Thread pisa
From: Pavel Pisa 

---
 c/src/lib/libbsp/arm/tms570/Makefile.am  | 21 +
 c/src/lib/libbsp/arm/tms570/configure.ac |  4 
 2 files changed, 25 insertions(+)

diff --git a/c/src/lib/libbsp/arm/tms570/Makefile.am 
b/c/src/lib/libbsp/arm/tms570/Makefile.am
index c4d39bc..012ce23 100644
--- a/c/src/lib/libbsp/arm/tms570/Makefile.am
+++ b/c/src/lib/libbsp/arm/tms570/Makefile.am
@@ -168,8 +168,29 @@ libbsp_a_SOURCES += 
../../../libcpu/shared/src/cache_manager.c
 libbsp_a_SOURCES += ../../../libcpu/arm/shared/include/cache_.h
 libbsp_a_CPPFLAGS += -I$(srcdir)/../../../libcpu/arm/shared/include
 
+if TMS570_USE_HWINIT_STARTUP
+libbsp_a_SOURCES += hwinit/tms570_sys_core.S
+libbsp_a_SOURCES += hwinit/fail_notification.c
+libbsp_a_SOURCES += hwinit/init_pinmux.c
+libbsp_a_SOURCES += hwinit/init_system.c
+libbsp_a_SOURCES += hwinit/init_emif_sdram.c
+libbsp_a_SOURCES += hwinit/init_esm.c
+libbsp_a_SOURCES += hwinit/tms570_selftest.c
+libbsp_a_SOURCES += hwinit/tms570_parity_tests.c
+libbsp_a_SOURCES += hwinit/tms570_parity_std.c
+libbsp_a_SOURCES += hwinit/tms570_parity_can.c
+libbsp_a_SOURCES += hwinit/tms570_parity_mibspi.c
+
+include_bsp_HEADERS += hwinit/tms570_parity_tests.h
+include_bsp_HEADERS += hwinit/tms570_hwinit.h
+include_bsp_HEADERS += hwinit/tms570_selftest.h
+
+# Start hooks
+libbsp_a_SOURCES += hwinit/bspstarthooks-hwinit.c
+else
 # Start hooks
 libbsp_a_SOURCES += startup/bspstarthooks.c
+endif
 
 # 
 # --  Special Rules
diff --git a/c/src/lib/libbsp/arm/tms570/configure.ac 
b/c/src/lib/libbsp/arm/tms570/configure.ac
index e06d4e4..7662064 100644
--- a/c/src/lib/libbsp/arm/tms570/configure.ac
+++ b/c/src/lib/libbsp/arm/tms570/configure.ac
@@ -50,6 +50,10 @@ RTEMS_BSPOPTS_HELP([TMS570_OSCILLATOR_RTC],[RTC oscillator 
frequency in Hz])
 RTEMS_BSPOPTS_SET([TMS570_CCLK],[*],[9600U])
 RTEMS_BSPOPTS_HELP([TMS570_CCLK],[CPU clock in Hz])
 
+RTEMS_BSPOPTS_SET([TMS570_USE_HWINIT_STARTUP],[*],[0])
+RTEMS_BSPOPTS_HELP([TMS570_USE_HWINIT_STARTUP],[use minimalized startup code 
for TMS570LS3137])
+AM_CONDITIONAL(TMS570_USE_HWINIT_STARTUP,test "$TMS570_USE_HWINIT_STARTUP" = 
"1")
+
 RTEMS_BSP_CLEANUP_OPTIONS(0, 1)
 RTEMS_BSP_LINKCMDS
 
-- 
1.9.1

___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel