mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Roland Hieber <rhi@pengutronix.de>
To: Ahmad Fatoum <a.fatoum@pengutronix.de>
Cc: barebox@lists.infradead.org, ejo@pengutronix.de,
	uol@pengutronix.de, renaud.barbier@abaco.com
Subject: Re: [PATCH 2/4] common: add new menu for target tools
Date: Tue, 14 Sep 2021 18:00:23 +0200
Message-ID: <20210914160023.ngxk6mlzyvlvgyly@pengutronix.de> (raw)
In-Reply-To: <20210914132016.22572-2-a.fatoum@pengutronix.de>

On Tue, Sep 14, 2021 at 03:20:14PM +0200, Ahmad Fatoum wrote:
> We have four target tools and will add 3 more in a follow up commit.
> Add a new menu to collect them.
> 
> Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>

Looks very useful to me.

Acked-by: Roland Hieber <rhi@pengutronix.de>

> ---
>  common/Kconfig  | 26 --------------------------
>  scripts/Kconfig | 30 ++++++++++++++++++++++++++++++
>  2 files changed, 30 insertions(+), 26 deletions(-)
> 
> diff --git a/common/Kconfig b/common/Kconfig
> index 222a438ee545..9dab7532e15e 100644
> --- a/common/Kconfig
> +++ b/common/Kconfig
> @@ -734,18 +734,6 @@ config IMD
>  	select CRC32
>  	bool "barebox metadata support"
>  
> -config IMD_TARGET
> -	bool "build bareboximd target tool"
> -	depends on IMD
> -
> -config KERNEL_INSTALL_TARGET
> -	bool
> -	prompt "Build kernel-install utility for the target"
> -	help
> -	  Enable this to compile the kernel-install script using the cross
> -	  compiler. The utility for the target will be under
> -	  scripts/kernel-install-target
> -
>  choice
>  	prompt "console support"
>  	default CONSOLE_FULL
> @@ -980,20 +968,6 @@ config DEFAULT_ENVIRONMENT_PATH
>  	  be taken. Relative paths will be relative to the barebox top-level
>  	  directory, but absolute paths are fine as well.
>  
> -config BAREBOXENV_TARGET
> -	bool
> -	prompt "build bareboxenv tool for target"
> -	help
> -	  'bareboxenv' is a tool to access the barebox environment from a running Linux
> -	  system. Say yes here to build it for the target.
> -
> -config BAREBOXCRC32_TARGET
> -	bool
> -	prompt "build bareboxcrc32 tool for target"
> -	help
> -	  'bareboxcrc32' is a userspacetool to generate the crc32 checksums the same way
> -	  barebox does. Say yes here to build it for the target.
> -
>  config HAS_SCHED
>  	bool
>  
> diff --git a/scripts/Kconfig b/scripts/Kconfig
> index b903486ecd8f..f7ed775fbc66 100644
> --- a/scripts/Kconfig
> +++ b/scripts/Kconfig
> @@ -40,3 +40,33 @@ config OMAP4_HOSTTOOL_USBBOOT
>  	  You need libusb-1.0 to compile this tool.
>  
>  endmenu
> +
> +menu "Target Tools"
> +
> +config IMD_TARGET
> +	bool "build bareboximd target tool"
> +	depends on IMD
> +
> +config KERNEL_INSTALL_TARGET
> +	bool
> +	prompt "Build kernel-install utility for the target"
> +	help
> +	  Enable this to compile the kernel-install script using the cross
> +	  compiler. The utility for the target will be under
> +	  scripts/kernel-install-target
> +
> +config BAREBOXENV_TARGET
> +	bool
> +	prompt "build bareboxenv tool for target"
> +	help
> +	  'bareboxenv' is a tool to access the barebox environment from a running Linux
> +	  system. Say yes here to build it for the target.
> +
> +config BAREBOXCRC32_TARGET
> +	bool
> +	prompt "build bareboxcrc32 tool for target"
> +	help
> +	  'bareboxcrc32' is a userspacetool to generate the crc32 checksums the same way
> +	  barebox does. Say yes here to build it for the target.
> +
> +endmenu
> -- 
> 2.30.2
> 
> 

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
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


  reply	other threads:[~2021-09-14 16:03 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-14 13:20 [PATCH 1/4] common: remove !SANDBOX dependency " Ahmad Fatoum
2021-09-14 13:20 ` [PATCH 2/4] common: add new menu " Ahmad Fatoum
2021-09-14 16:00   ` Roland Hieber [this message]
2021-09-14 13:20 ` [PATCH 3/4] scripts: unify libusb.h inclusion Ahmad Fatoum
2021-09-14 19:22   ` Trent Piepho
2021-09-15  8:55     ` Ahmad Fatoum
2021-09-14 13:20 ` [PATCH 4/4] scripts: allow building USB loader tools for target as well Ahmad Fatoum
2021-09-14 19:11   ` Trent Piepho
2021-09-15  8:50     ` Ahmad Fatoum
2021-09-15  9:38       ` Trent Piepho
2021-09-15 10:23         ` Ahmad Fatoum
2021-09-15  8:44   ` Enrico Jörns

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210914160023.ngxk6mlzyvlvgyly@pengutronix.de \
    --to=rhi@pengutronix.de \
    --cc=a.fatoum@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=ejo@pengutronix.de \
    --cc=renaud.barbier@abaco.com \
    --cc=uol@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

mail archive of the barebox mailing list

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://lore.barebox.org/barebox/0 barebox/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 barebox barebox/ https://lore.barebox.org/barebox \
		barebox@lists.infradead.org barebox@lists.infradead.org
	public-inbox-index barebox

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git