From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 3/3] package/openpowerlink2: add demo mn console application
Date: Sat, 02 May 2015 17:38:07 +0200 [thread overview]
Message-ID: <5544EF5F.2010201@mind.be> (raw)
In-Reply-To: <1430494560-5196-3-git-send-email-romain.naour@openwide.fr>
On 01/05/15 17:36, Romain Naour wrote:
> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
> ---
> v2: add missing -DCFG_DEBUG_LVL in CONF_OPTS
> ---
> package/openpowerlink2/Config.in | 1 +
> .../openpowerlink2/openpowerlink2-apps/Config.in | 2 +
> .../openpowerlink2-apps-demo-mn-console/Config.in | 5 +++
> .../openpowerlink2-apps-demo-mn-console.mk | 45 ++++++++++++++++++++++
> .../openpowerlink2-apps/openpowerlink2-apps.mk | 2 +
> package/openpowerlink2/openpowerlink2.mk | 1 +
> 6 files changed, 56 insertions(+)
> create mode 100644 package/openpowerlink2/openpowerlink2-apps/Config.in
> create mode 100644 package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps-demo-mn-console/Config.in
> create mode 100644 package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps-demo-mn-console/openpowerlink2-apps-demo-mn-console.mk
> create mode 100644 package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps.mk
>
> diff --git a/package/openpowerlink2/Config.in b/package/openpowerlink2/Config.in
> index c651031..e82e738 100644
> --- a/package/openpowerlink2/Config.in
> +++ b/package/openpowerlink2/Config.in
> @@ -35,6 +35,7 @@ choice
>
> endchoice
>
> +source "package/openpowerlink2/openpowerlink2-apps/Config.in"
> source "package/openpowerlink2/openpowerlink2-drivers/Config.in"
> source "package/openpowerlink2/openpowerlink2-stack/Config.in"
Actually, shouldn't the _DEBUG_LEVEL option come before the sourcing of the
other packages?
>
> diff --git a/package/openpowerlink2/openpowerlink2-apps/Config.in b/package/openpowerlink2/openpowerlink2-apps/Config.in
> new file mode 100644
> index 0000000..6f10a6a
> --- /dev/null
> +++ b/package/openpowerlink2/openpowerlink2-apps/Config.in
> @@ -0,0 +1,2 @@
> +
> +source "package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps-demo-mn-console/Config.in"
> diff --git a/package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps-demo-mn-console/Config.in b/package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps-demo-mn-console/Config.in
> new file mode 100644
> index 0000000..dcf2605
> --- /dev/null
> +++ b/package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps-demo-mn-console/Config.in
> @@ -0,0 +1,5 @@
> +
> +config BR2_PACKAGE_OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE
> + bool "demo_mn_console"
Shouldn't this depend on BR2_PACKAGE_OPENPOWERLINK2_MN and either select or
depend on the stack?
> + help
> + Managing Node console demo application
> diff --git a/package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps-demo-mn-console/openpowerlink2-apps-demo-mn-console.mk b/package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps-demo-mn-console/openpowerlink2-apps-demo-mn-console.mk
> new file mode 100644
> index 0000000..23c17b5
> --- /dev/null
> +++ b/package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps-demo-mn-console/openpowerlink2-apps-demo-mn-console.mk
> @@ -0,0 +1,45 @@
> +################################################################################
> +#
> +# openpowerlink2-apps-demo-mn-console
> +#
> +################################################################################
> +
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_VERSION = $(OPENPOWERLINK2_VERSION)
> +
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_LICENSE = $(OPENPOWERLINK2_LICENSE)
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_LICENSE_FILES = $(OPENPOWERLINK2_LICENSE_FILES)
> +
> +# We want to use the same archive for all sub-packages.
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_SOURCE =
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_DEPENDENCIES = openpowerlink2-stack
> +
> +define OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_SYMLINK_TO_SRC_HOOK
> + ln -s $(OPENPOWERLINK2_DIR) $(OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_DIR)/src
> +endef
> +
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_POST_EXTRACT_HOOKS += \
> + OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_SYMLINK_TO_SRC_HOOK
> +
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_SUBDIR = src/apps/demo_mn_console/
> +
> +# See apps/common/cmake/configure-linux.cmake for available options list.
> +
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_CONF_OPTS = \
> + -DCFG_DEBUG_LVL=$(call qstrip,$(BR2_PACKAGE_OPENPOWERLINK2_DEBUG_LEVEL))
> +
> +# PCAP is used for directlink or userspace daemon
Why is this here?
> +ifeq ($(BR2_PACKAGE_OPENPOWERLINK2_STACK_MONOLITHIC_USER_STACK_LIB),y)
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_CONF_OPTS += \
> + -DCFG_BUILD_KERNEL_STACK="Link to Application"
> +else ifeq ($(BR2_PACKAGE_OPENPOWERLINK2_STACK_USERSPACE_DAEMON_LIB),y)
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_CONF_OPTS += \
> + -DCFG_BUILD_KERNEL_STACK="Linux Userspace Daemon" \
> + -DCFG_DEMO_MN_CONSOLE_USE_SYNCTHREAD=ON
> +else ifeq ($(BR2_PACKAGE_OPENPOWERLINK2_STACK_KERNEL_STACK_LIB),y)
> +# kernel stack
Well, yes, obviously...
Regards,
Arnout
> +OPENPOWERLINK2_APPS_DEMO_MN_CONSOLE_CONF_OPTS += \
> + -DCFG_BUILD_KERNEL_STACK="Linux Kernel Module" \
> + -DCFG_DEMO_MN_CONSOLE_USE_SYNCTHREAD=ON
> +endif
> +
> +$(eval $(cmake-package))
> diff --git a/package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps.mk b/package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps.mk
> new file mode 100644
> index 0000000..802dcbf
> --- /dev/null
> +++ b/package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps.mk
> @@ -0,0 +1,2 @@
> +
> +include $(sort $(wildcard package/openpowerlink2/openpowerlink2-apps/*/*.mk))
> diff --git a/package/openpowerlink2/openpowerlink2.mk b/package/openpowerlink2/openpowerlink2.mk
> index 144fa98..ba0d418 100644
> --- a/package/openpowerlink2/openpowerlink2.mk
> +++ b/package/openpowerlink2/openpowerlink2.mk
> @@ -14,5 +14,6 @@ OPENPOWERLINK2_LICENSE_FILES = license.md
>
> $(eval $(generic-package))
>
> +include package/openpowerlink2/openpowerlink2-apps/openpowerlink2-apps.mk
> include package/openpowerlink2/openpowerlink2-drivers/openpowerlink2-drivers.mk
> include package/openpowerlink2/openpowerlink2-stack/openpowerlink2-stack.mk
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
next prev parent reply other threads:[~2015-05-02 15:38 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-01 15:35 [Buildroot] [PATCH v2 1/3] package/openpowerlink2: new package Romain Naour
2015-05-01 15:35 ` [Buildroot] [PATCH v2 2/3] package/openpowerlink2: add drivers packages Romain Naour
2015-05-02 15:12 ` Arnout Vandecappelle
2015-05-07 14:46 ` Romain Naour
2015-05-01 15:36 ` [Buildroot] [PATCH v2 3/3] package/openpowerlink2: add demo mn console application Romain Naour
2015-05-02 15:38 ` Arnout Vandecappelle [this message]
2015-05-02 14:53 ` [Buildroot] [PATCH v2 1/3] package/openpowerlink2: new package Arnout Vandecappelle
2015-05-06 12:20 ` Romain Naour
2015-05-07 11:13 ` Romain Naour
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=5544EF5F.2010201@mind.be \
--to=arnout@mind.be \
--cc=buildroot@busybox.net \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox