diff options
| author | Antonio Borneo <borneo.antonio@gmail.com> | 2020-11-01 15:39:23 +0100 |
|---|---|---|
| committer | Antonio Borneo <borneo.antonio@gmail.com> | 2020-11-07 20:53:39 +0000 |
| commit | 384e90b06b5005aa04d695b8a67c81a0af646082 (patch) | |
| tree | b2ec4a213ed69862e8cd2523779b3192adbdd450 | |
| parent | ad06fba6f07ceaaf88845658c47a8f50888e413e (diff) | |
| download | riscv-openocd-384e90b06b5005aa04d695b8a67c81a0af646082.zip riscv-openocd-384e90b06b5005aa04d695b8a67c81a0af646082.tar.gz riscv-openocd-384e90b06b5005aa04d695b8a67c81a0af646082.tar.bz2 | |
build: fix build with --enable-minidriver-dummy
Commit 462c01206692 ("Add complete JTAG debug logging.") breaks
the build for minidriver, that is enabled either on zy1000 build
and on minidriver-dummy build. The check on BUILD_ZY1000 was added
to pass the auto-build in jenkins. While the build issue with
minidriver-dummy was known, as reported in the comment, it was not
addressed and got ignored for slightly more than one year.
Use the macro HAVE_JTAG_MINIDRIVER_H in place of BUILD_ZY1000 to
take in account both builds that require the minidriver.
Fix also the build in case configure enables the HLA drivers due
to autodetection of libusb. The HLA drivers would not be in the
build and the function transport_is_hla() would be missing.
Change-Id: I1d85c5fa247bf4a85aba29b233c0b573b46665bc
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: http://openocd.zylin.com/5911
Tested-by: jenkins
| -rw-r--r-- | src/jtag/core.c | 4 | ||||
| -rw-r--r-- | src/transport/transport.h | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/jtag/core.c b/src/jtag/core.c index f0f74eb..5abf832 100644 --- a/src/jtag/core.c +++ b/src/jtag/core.c @@ -953,12 +953,12 @@ int default_interface_jtag_execute_queue(void) int result = jtag->jtag_ops->execute_queue(); -#if !BUILD_ZY1000 +#if !HAVE_JTAG_MINIDRIVER_H /* Only build this if we use a regular driver with a command queue. * Otherwise jtag_command_queue won't be found at compile/link time. Its * definition is in jtag/commands.c, which is only built/linked by * jtag/Makefile.am if MINIDRIVER_DUMMY || !MINIDRIVER, but those variables - * aren't accessible here. */ + * aren't accessible here. Use HAVE_JTAG_MINIDRIVER_H */ struct jtag_command *cmd = jtag_command_queue; while (debug_level >= LOG_LVL_DEBUG_IO && cmd) { switch (cmd->type) { diff --git a/src/transport/transport.h b/src/transport/transport.h index e04f780..6bf6aac 100644 --- a/src/transport/transport.h +++ b/src/transport/transport.h @@ -98,7 +98,7 @@ bool transport_is_dapdirect_jtag(void); bool transport_is_dapdirect_swd(void); bool transport_is_swim(void); -#if BUILD_HLADAPTER +#if BUILD_HLADAPTER && !HAVE_JTAG_MINIDRIVER_H bool transport_is_hla(void); #else static inline bool transport_is_hla(void) |