diff mbox series

[1/1] examples: make examples customizable

Message ID 20200922180350.46636-2-xypron.glpk@gmx.de
State Superseded, archived
Delegated to: Tom Rini
Headers show
Series [1/1] examples: make examples customizable | expand

Commit Message

Heinrich Schuchardt Sept. 22, 2020, 6:03 p.m. UTC
Most users don't need the standalone API examples. Distributions like Suse
do not supply libgcc for cross-compiling and we cannot do without on ARMv8
for building examples/.

Make examples customizable via symbol CONFIG_EXAMPLES. It only defaults to
yes to ensure that we compile it on Gitlab CI.

Cc: Matthias Brugger <mbrugger@suse.com>
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
I wonder if we should not completely drop the standalone API. The UEFI API
has enough to run a standalone application.
---
 Kconfig           | 8 ++++++++
 examples/Makefile | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

--
2.28.0

Comments

Matthias Brugger Sept. 23, 2020, 11:37 a.m. UTC | #1
On 22/09/2020 20:03, Heinrich Schuchardt wrote:
> Most users don't need the standalone API examples. Distributions like Suse

Nit: It's SUSE not Suse

Regards,
Matthias

> do not supply libgcc for cross-compiling and we cannot do without on ARMv8
> for building examples/.
> 
> Make examples customizable via symbol CONFIG_EXAMPLES. It only defaults to
> yes to ensure that we compile it on Gitlab CI.
> 
> Cc: Matthias Brugger <mbrugger@suse.com>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> I wonder if we should not completely drop the standalone API. The UEFI API
> has enough to run a standalone application.
> ---
>   Kconfig           | 8 ++++++++
>   examples/Makefile | 2 +-
>   2 files changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/Kconfig b/Kconfig
> index 883e3f71d0..d0eae2ceba 100644
> --- a/Kconfig
> +++ b/Kconfig
> @@ -422,6 +422,14 @@ config SYS_SRAM_SIZE
>   	default 0x10000 if TARGET_TRICORDER
>   	default 0x0
> 
> +config EXAMPLES
> +	bool "Compile API examples"
> +	depends on !SANDBOX
> +	default y
> +	help
> +	  U-Boot provides an API for standalone applications. Examples are
> +	  provided in directory examples/.
> +
>   endmenu		# General setup
> 
>   menu "Boot images"
> diff --git a/examples/Makefile b/examples/Makefile
> index d440bc5655..bf518bd221 100644
> --- a/examples/Makefile
> +++ b/examples/Makefile
> @@ -1,6 +1,6 @@
>   # SPDX-License-Identifier: GPL-2.0+
> 
> -ifndef CONFIG_SANDBOX
> +ifdef CONFIG_EXAMPLES
> 
>   ifdef FTRACE
>   subdir-ccflags-y += -finstrument-functions -DFTRACE
> --
> 2.28.0
>
diff mbox series

Patch

diff --git a/Kconfig b/Kconfig
index 883e3f71d0..d0eae2ceba 100644
--- a/Kconfig
+++ b/Kconfig
@@ -422,6 +422,14 @@  config SYS_SRAM_SIZE
 	default 0x10000 if TARGET_TRICORDER
 	default 0x0

+config EXAMPLES
+	bool "Compile API examples"
+	depends on !SANDBOX
+	default y
+	help
+	  U-Boot provides an API for standalone applications. Examples are
+	  provided in directory examples/.
+
 endmenu		# General setup

 menu "Boot images"
diff --git a/examples/Makefile b/examples/Makefile
index d440bc5655..bf518bd221 100644
--- a/examples/Makefile
+++ b/examples/Makefile
@@ -1,6 +1,6 @@ 
 # SPDX-License-Identifier: GPL-2.0+

-ifndef CONFIG_SANDBOX
+ifdef CONFIG_EXAMPLES

 ifdef FTRACE
 subdir-ccflags-y += -finstrument-functions -DFTRACE