Re: [ptxdist] [PATCH v2] host-openocd: add initial support

[Roland Hieber <rhi@pengutronix.de>]

On Mon, Jun 17, 2019 at 03:23:12PM +0200, Michael Grzeschik wrote:
> Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
> ---
> v1 -> v2: - fixed indentation
>           - removed buildflags
> 
>  patches/openocd-af952850/autogen.sh |  3 +
>  rules/host-openocd.in               | 12 ++++
>  rules/host-openocd.make             | 87 +++++++++++++++++++++++++++++
>  3 files changed, 102 insertions(+)
>  create mode 100755 patches/openocd-af952850/autogen.sh
>  create mode 100644 rules/host-openocd.in
>  create mode 100644 rules/host-openocd.make
> 
> diff --git a/patches/openocd-af952850/autogen.sh b/patches/openocd-af952850/autogen.sh
> new file mode 100755
> index 000000000..7d344d32f
> --- /dev/null
> +++ b/patches/openocd-af952850/autogen.sh
> @@ -0,0 +1,3 @@
> +#!/bin/bash
> +
> +exec ./bootstrap
> diff --git a/rules/host-openocd.in b/rules/host-openocd.in
> new file mode 100644
> index 000000000..8c863598e
> --- /dev/null
> +++ b/rules/host-openocd.in
> @@ -0,0 +1,12 @@
> +## SECTION=hosttools
> +
> +menuconfig HOST_OPENOCD
> +	tristate
> +	prompt "openocd"
> +	select HOST_LIBUSB
> +	select HOST_JIMTCL
> +	select HOST_LIBFTDI1
> +	help
> +	  The "Open On-Chip Debugger" provides JTAG/SWD access from GDB
> +	  (or directly with TCL scripts) to processors with ARM and MIPS
> +	  based cores.
> diff --git a/rules/host-openocd.make b/rules/host-openocd.make
> new file mode 100644
> index 000000000..10ac4d12b
> --- /dev/null
> +++ b/rules/host-openocd.make
> @@ -0,0 +1,87 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2013 by Michael Grzeschik <mgr@pengutronix.de>
> +#
> +# See CREDITS for details about who has contributed to this project.
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +HOST_PACKAGES-$(PTXCONF_HOST_OPENOCD) += host-openocd
> +
> +#
> +# Paths and names
> +#
> +HOST_OPENOCD_VERSION	:= af952850
> +HOST_OPENOCD_MD5	:= fa6841d4e4e76d1626406a99d5b1851d
> +HOST_OPENOCD		:= openocd-$(HOST_OPENOCD_VERSION)
> +HOST_OPENOCD_SUFFIX	:= tar.bz2
> +HOST_OPENOCD_URL	:= git://git.code.sf.net/p/openocd/code;tag=$(HOST_OPENOCD_VERSION)

When trying to build this package, SF.net seems horribly slow to me.
Care to use repo.or.cz instead? It is about 4 times faster, and openocd
recommends it as a mirror in their README.

Also it would be nice to have a more descriptive version, so I suggest:

HOST_OPENOCD_VERSION := 0.10.0-870-gaf952850b549
HOST_OPENOCD_URL     := git://repo.or.cz/openocd;tag=v$(HOST_OPENOCD_VERSION)

and to move the patches to the appropriate location too.

 - Roland


> +HOST_OPENOCD_SOURCE	:= $(SRCDIR)/$(HOST_OPENOCD).$(HOST_OPENOCD_SUFFIX)
> +HOST_OPENOCD_DIR	:= $(HOST_BUILDDIR)/$(HOST_OPENOCD)
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +#
> +# autoconf
> +#
> +HOST_OPENOCD_CONF_TOOL	:= autoconf
> +HOST_OPENOCD_CONF_ENV	:= \
> +	CCACHE=none
> +HOST_OPENOCD_CONF_OPT	:= \
> +	$(HOST_AUTOCONF) \
> +	--enable-shared \
> +	--disable-static \
> +	--disable-werror \
> +	--disable-internal-jimtcl \
> +	--disable-internal-libjaylink \
> +	--enable-dummy \
> +	--enable-ftdi \
> +	--enable-stlink \
> +	--disable-ti-icdi \
> +	--disable-ulink \
> +	--disable-usb-blaster-2 \
> +	--disable-ft232r \
> +	--disable-vsllink \
> +	--disable-xds110 \
> +	--disable-osbdm \
> +	--disable-opendous \
> +	--disable-aice \
> +	--disable-usbprog \
> +	--disable-rlink \
> +	--disable-armjtagew \
> +	--disable-cmsis-dap \
> +	--disable-kitprog \
> +	--disable-usb_blaster \
> +	--disable-presto\
> +	--disable-openjtag \
> +	--disable-jlink \
> +	--disable-parport \
> +	--disable-parport-ppdev \
> +	--disable-parport-giveio \
> +	--disable-jtag_vpi \
> +	--disable-amtjtagaccel \
> +	--disable-zy1000 \
> +	--disable-zy1000-master \
> +	--disable-ioutil \
> +	--disable-ep93xx \
> +	--disable-at91rm9200 \
> +	--disable-bcm2835gpio \
> +	--disable-imx_gpio \
> +	--disable-gw16012 \
> +	--disable-oocd_trace \
> +	--disable-buspirate \
> +	--disable-sysfsgpio \
> +	--disable-minidriver-dummy \
> +	--disable-target-64 \
> +	--disable-remote-bitbang \
> +	--disable-doxygen-pdf \
> +	--disable-doxygen-html
> +
> +# vim: syntax=make
> -- 
> 2.20.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Roland Hieber                     | r.hieber@pengutronix.de     |
Pengutronix e.K.                  | https://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 |
Amtsgericht Hildesheim, HRA 2686  | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de