mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] Documentation: zii-vf610-dev: Fix 'halt' in openocd configuration script
@ 2019-08-20 22:17 Cory Tusar
  2019-08-20 22:25 ` Andrey Smirnov
  2019-08-23  8:58 ` Sascha Hauer
  0 siblings, 2 replies; 3+ messages in thread
From: Cory Tusar @ 2019-08-20 22:17 UTC (permalink / raw)
  To: barebox; +Cc: Chris Healy

When attempting to use the existing configuration file to bootstrap
an SCU4 AIB board, openocd would consistently fail on the 'halt' command
with a "Target not examined yet" error.  Dumping target information
showed that vf610.cpu1 was the active target, rather that vf610.cpu0.

Adding a command prior to the halt to explicitly set the target core
resulted in proper operation.

Tested with openocd-0.10.0+dev-00924-g16496488 on a Zodiac SCU4 AIB.

Signed-off-by: Cory Tusar <cory.tusar@zii.aero>
---
 Documentation/boards/imx/zii-vf610-dev/openocd.cfg | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/boards/imx/zii-vf610-dev/openocd.cfg b/Documentation/boards/imx/zii-vf610-dev/openocd.cfg
index 222f48711..5127c7623 100644
--- a/Documentation/boards/imx/zii-vf610-dev/openocd.cfg
+++ b/Documentation/boards/imx/zii-vf610-dev/openocd.cfg
@@ -258,6 +258,7 @@ proc safe_reset {} {
 proc start_barebox { } {
 	set VF610_DDR_BASE_ADDR 0x80000000
 	echo "Bootstrap: Loading Barebox"
+	targets vf610.cpu0
 	halt
 	load_image images/barebox-zii-vf610-dev.img $VF610_DDR_BASE_ADDR bin
 	arm core_state arm
-- 
2.23.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Documentation: zii-vf610-dev: Fix 'halt' in openocd configuration script
  2019-08-20 22:17 [PATCH] Documentation: zii-vf610-dev: Fix 'halt' in openocd configuration script Cory Tusar
@ 2019-08-20 22:25 ` Andrey Smirnov
  2019-08-23  8:58 ` Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Andrey Smirnov @ 2019-08-20 22:25 UTC (permalink / raw)
  To: Cory Tusar; +Cc: barebox, Chris Healy

On Tue, Aug 20, 2019 at 3:17 PM Cory Tusar <Cory.Tusar@zii.aero> wrote:
>
> When attempting to use the existing configuration file to bootstrap
> an SCU4 AIB board, openocd would consistently fail on the 'halt' command
> with a "Target not examined yet" error.  Dumping target information
> showed that vf610.cpu1 was the active target, rather that vf610.cpu0.
>
> Adding a command prior to the halt to explicitly set the target core
> resulted in proper operation.
>
> Tested with openocd-0.10.0+dev-00924-g16496488 on a Zodiac SCU4 AIB.
>
> Signed-off-by: Cory Tusar <cory.tusar@zii.aero>
> ---
>  Documentation/boards/imx/zii-vf610-dev/openocd.cfg | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/boards/imx/zii-vf610-dev/openocd.cfg b/Documentation/boards/imx/zii-vf610-dev/openocd.cfg
> index 222f48711..5127c7623 100644
> --- a/Documentation/boards/imx/zii-vf610-dev/openocd.cfg
> +++ b/Documentation/boards/imx/zii-vf610-dev/openocd.cfg
> @@ -258,6 +258,7 @@ proc safe_reset {} {
>  proc start_barebox { } {
>         set VF610_DDR_BASE_ADDR 0x80000000
>         echo "Bootstrap: Loading Barebox"
> +       targets vf610.cpu0

There's nothing wrong with this fix as is, however I think this is
something that should be fixed in OpenOCD itself. Both
tcl/target/imx6.cfg and tcl/target/vybrid_vf6xx.cfg are missing proper
"targets" directive, compared to e.g. tcl/target/imx7.cfg.

Thanks,
Andrey Smirnov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Documentation: zii-vf610-dev: Fix 'halt' in openocd configuration script
  2019-08-20 22:17 [PATCH] Documentation: zii-vf610-dev: Fix 'halt' in openocd configuration script Cory Tusar
  2019-08-20 22:25 ` Andrey Smirnov
@ 2019-08-23  8:58 ` Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Sascha Hauer @ 2019-08-23  8:58 UTC (permalink / raw)
  To: Cory Tusar; +Cc: barebox, Chris Healy

On Tue, Aug 20, 2019 at 10:17:36PM +0000, Cory Tusar wrote:
> When attempting to use the existing configuration file to bootstrap
> an SCU4 AIB board, openocd would consistently fail on the 'halt' command
> with a "Target not examined yet" error.  Dumping target information
> showed that vf610.cpu1 was the active target, rather that vf610.cpu0.
> 
> Adding a command prior to the halt to explicitly set the target core
> resulted in proper operation.
> 
> Tested with openocd-0.10.0+dev-00924-g16496488 on a Zodiac SCU4 AIB.
> 
> Signed-off-by: Cory Tusar <cory.tusar@zii.aero>
> ---
>  Documentation/boards/imx/zii-vf610-dev/openocd.cfg | 1 +
>  1 file changed, 1 insertion(+)

Applied, thanks

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-08-23  8:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-20 22:17 [PATCH] Documentation: zii-vf610-dev: Fix 'halt' in openocd configuration script Cory Tusar
2019-08-20 22:25 ` Andrey Smirnov
2019-08-23  8:58 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox