From patchwork Tue Nov 16 21:35:14 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vijai Kumar K X-Patchwork-Id: 1041 Return-Path: Received: from shymkent.ilbers.de ([unix socket]) by shymkent (Cyrus 2.5.10-Debian-2.5.10-3) with LMTPA; Wed, 17 Nov 2021 08:35:44 +0100 X-Sieve: CMU Sieve 2.4 Received: from mail-oi1-f185.google.com (mail-oi1-f185.google.com [209.85.167.185]) by shymkent.ilbers.de (8.15.2/8.15.2/Debian-8) with ESMTPS id 1AH7ZhZ8010383 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT) for ; Wed, 17 Nov 2021 08:35:44 +0100 Received: by mail-oi1-f185.google.com with SMTP id j85-20020acaeb58000000b002a815919bb4sf1399399oih.13 for ; Tue, 16 Nov 2021 23:35:44 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1637134538; cv=pass; d=google.com; s=arc-20160816; b=G+w+k+bCtqJFsM3aMD+IooXOw43ODXA2UNjPLZy5uiAwykxkOO7zoAeWv7xBsvRqrM 6zC3LHMwJTeS1jb9S8wq696pfzXQaQddbdcIwTUTGOR/pNLfobe5F+BeC+Vms3/BXppr TkuHcyd+QezlvN7JGJ2wzMhnoG3tQ8vxhbCVcbh5611dlJM1csSaYovbgh7GZR+IguHf l3s5w1Lx5AIczGMWSKdlnMApgblYLqD4OVgWLT9ngpFR3jYbv9lTNfS2ReJYSQ65ZyQB U1UtSnYwtI6YfrZx+92zMeYsX2urCeB/u6hcHacqrUMh7hi+1SggC/Yd/SlcMQ3mBQxL zB2Q== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-unsubscribe:list-subscribe:list-archive:list-help:list-post :list-id:mailing-list:precedence:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:ironport-sdr:ironport-sdr:sender :dkim-signature; bh=uMuhK9hm/OvDX7KcrgwX/Ci6yVlfEcUm/5jwZ+7N7bo=; b=zaf1vwLsoD3a5G8j5i0aPZyyhLMDXPXvkqT0EdcJ9hdppCau6Ua6m3DkoJ/d9PzI1y 7YmZ2BdxbN1DUch3tebXM111LLEbLppLMsr2pXqrV7Uich6Y+ExxWhK08PrIPiosmrjY SC0/g7pY533s5Mx6ukDo86j9XXU+seHvC3OAU7RRvNd4MDIw4+csqKZ4CIIyuKWT79DU 3jB97Ax2mohpYWZbPyHz35FIJHA8FwI+TFrGh6+DqPtUad6VG2t437a5vx67kxZmGVXS HIlcYOxnm4rOU6LDV9LHqCqmjEVsZ2xgjj6n8P1rOuMZp505xF9NY6XtHAk3xsyklJRe HD0g== ARC-Authentication-Results: i=2; gmr-mx.google.com; spf=pass (google.com: domain of vijaikumar_kanagarajan@mentor.com designates 68.232.129.153 as permitted sender) smtp.mailfrom=Vijaikumar_Kanagarajan@mentor.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlegroups.com; s=20210112; h=sender:ironport-sdr:ironport-sdr:from:to:cc:subject:date:message-id :in-reply-to:references:mime-version:x-original-sender :x-original-authentication-results:precedence:mailing-list:list-id :list-post:list-help:list-archive:list-subscribe:list-unsubscribe; bh=uMuhK9hm/OvDX7KcrgwX/Ci6yVlfEcUm/5jwZ+7N7bo=; b=PFW/74FDvUM4eYlklgrY+OTTdoBn+sldDDoARdMHCBgUWpVcwc3s8d48Di7pIvuW5b T98XAx2v3XVUClkPYmJ33tQO4/0OI4/uYCKKzAO0cq5fbO7THc+db9FbpwVL1CYIFYa8 Pale4OH9DEFGn3qySd1ko5d6Y3MW9SXEPW0BNkIqnLgR0hb8FEVoQoSYVWTCxFzI4kYt DD8OhSqGy5o5oCThG773d2OgTOnapBOE6VPaqLtuJFM/JhuiJBYsK83aOiEVGBlKQttL bLWmYXRVIN/FCZB0JjnRIHaK9Lmi4cioC+SyGnjvRGNni0XeULCYj2lRMcEJi2ilQLRQ EIjg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=sender:x-gm-message-state:ironport-sdr:ironport-sdr:from:to:cc :subject:date:message-id:in-reply-to:references:mime-version :x-original-sender:x-original-authentication-results:precedence :mailing-list:list-id:x-spam-checked-in-group:list-post:list-help :list-archive:list-subscribe:list-unsubscribe; bh=uMuhK9hm/OvDX7KcrgwX/Ci6yVlfEcUm/5jwZ+7N7bo=; b=x/qECjw9Qd0Tf811wvy2tRQY6QZmAcolllTeccdTWwCklwBYn2AN6FDd5V2EXvjeAQ cR5tiskuFcWmeuP57CyDo3Z6sVruXKUIFnCDlr770pusOrnEttQa9qISjvcE/NYFjCfr DPuyFMxgWjMVml9z1G9WpRj8Bwm7Yg5DUKYQi6ACFjf9eZj1YLr0XzmI5U2UTaLXtUZp eLTRfcTB6JMEtkRLhNJvvJxBpd5QAlvOKIfCyYvStioei/1ZvwDV7gPC1/gF43rgTUFf OVrecuGzpcFkL8TGnRUNndinjHFnvrCaKAuZ4OjI8zSRNY3pPRxbFEhcXQkFC5wswdUs BrqA== Sender: isar-users@googlegroups.com X-Gm-Message-State: AOAM5322k7sEtV5+5ApAAhe38FY1F8Bzld4VlMGdK1o0G8fdTBU5hhPL O5CLYurFL04+P6ZYCTmlXMM= X-Google-Smtp-Source: ABdhPJxyfqwolyHvCh1QmYFMwHBQDbYPxKVRXPsnbBm3lOgEh+2Zk2a5fOuNiJuFIArCmEUobWz6kg== X-Received: by 2002:a05:6808:170e:: with SMTP id bc14mr12495003oib.86.1637134537856; Tue, 16 Nov 2021 23:35:37 -0800 (PST) X-BeenThere: isar-users@googlegroups.com Received: by 2002:a9d:4599:: with SMTP id x25ls7101698ote.9.gmail; Tue, 16 Nov 2021 23:35:37 -0800 (PST) X-Received: by 2002:a9d:6ac7:: with SMTP id m7mr11812691otq.306.1637134537509; Tue, 16 Nov 2021 23:35:37 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1637134537; cv=none; d=google.com; s=arc-20160816; b=wKPKoNr+YUGnM4bCotZr99krdh7qOJytO/IGFVeWgcVVoDjvEWJGBAvacVPIpVI0Jv 27cxi8+LH7OkTJvIIRiePIjFw7loc3l6Clv1o4h05AibDvtq5mjPQWgvxF1YDb4N5IIw BhirdR9miY7Ls3MWNk0gAlNDno1YlMy7BclT50Kppfu/buVqJ9FogEp40AcaQiIazKYb sqygPvzC0NOSMsI+RRI0dVyTjkJg27FOHetWhcpItf8ptYmkwc108x9PSYl+PKdtWUqi eTqWBRTQJwB79Wxp67HnZLi+/mlVuBXYyH93wt0K/pqsKVdZirgk5LOao9lCzaVbc/Zj EG0A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:ironport-sdr:ironport-sdr; bh=XU9jHQ85CgfUHuezvnpVcyZyvwKdlnX1iNc0XDZWn2c=; b=UaxdFsmzN9wiLE9w9cUW6Sa6sw6ytXGpyoJ1MRHH8khU9hDOTzgDlcWPKbhho3m5cQ dNhT8ecAnG3HqJGdkMUsR8sIbdnCPSXOyvz7OljPMDJMr6BWk55+Ijy3a7xblaZtjJH0 G+ZprEYSoKg9755p+5e7d9WkDc6I0TpK/6fmvnpjJEr9lRkKk+5RK7ZlcOBeXhVwnr7K Z6ko9YGk0B/r/NLv/wxv6qBtrnngewkCMAYunageyLLjSAjx7aaU+XwKWg9I/bPbkWaP z3XaYS42HMG/hd4awQK6RuVUMviH81Bvkg9OMdlLMc2uGcqlGD/je0Sy/aTq5zzd0jtX 3MMQ== ARC-Authentication-Results: i=1; gmr-mx.google.com; spf=pass (google.com: domain of vijaikumar_kanagarajan@mentor.com designates 68.232.129.153 as permitted sender) smtp.mailfrom=Vijaikumar_Kanagarajan@mentor.com Received: from esa1.mentor.iphmx.com (esa1.mentor.iphmx.com. [68.232.129.153]) by gmr-mx.google.com with ESMTPS id u27si1911814ots.2.2021.11.16.23.35.37 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 16 Nov 2021 23:35:37 -0800 (PST) Received-SPF: pass (google.com: domain of vijaikumar_kanagarajan@mentor.com designates 68.232.129.153 as permitted sender) client-ip=68.232.129.153; IronPort-SDR: +F2R55uWQsZJTTd4x3WDlI5qkTBZKEGym6TttjDP8n5OlKElvhTKghAlrJbsYK0UrjD+pE+Kie rQIH2Vw4nd8r7nazpSYaxobBInc5zzmGsa0x+x5IBA15XUmxYA6kTINMqJoKAwQ5yUJW7JhNUc hcLINhMpl/6wtlNLODv2kiEzYnE7WGUfsgUR51V6HSyuRTx+CP+sNIjhv3kM4ySv2k++EtMnqx MmUJGTZ5HcrELYoQL8srkwQyQcrDl+ei+nJicyA9jeF8W2Ddk8FxnGThRpzQDBXEwRTWhNt4cn Ze9aSA9DKOpCtpY6l9Cu7Qnk X-IronPort-AV: E=Sophos;i="5.87,240,1631606400"; d="scan'208";a="71060788" Received: from orw-gwy-02-in.mentorg.com ([192.94.38.167]) by esa1.mentor.iphmx.com with ESMTP; 16 Nov 2021 23:35:37 -0800 IronPort-SDR: 7SEVfO1YyPZrzTJ5QETxfvrzUyyrIOXDhBdmZUkiVo4u9B4FrVxkM4NRcP16A0FrMqCsWOO/2j cKVKQma9kHyYTz1E8Vk7Go22NNHtPfY8yAVqVGJwQXfEoc3B/oyxFGxwmoptXKqYzMaQPwUi8Y anprLeHQbBNNSZYds8hvnUuNtbSDaJHge80GfDrqQaF/kADdgW9/bmF+PpUQnC1o1kF8HlivVt GlZ00A1uVRsC4Avy49gn+szCaUBstk9zSIWCdynxrSoRta8sThKWPn7D2xQ83iixD9+nqO1VGM FtY= From: Vijai Kumar K To: CC: Vijai Kumar K Subject: [PATCH v3 1/2] u-boot: Switch to use DEB_BUILD_PROFILES Date: Wed, 17 Nov 2021 13:05:14 +0530 Message-ID: <20211117073515.773106-2-Vijaikumar_Kanagarajan@mentor.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20211117073515.773106-1-Vijaikumar_Kanagarajan@mentor.com> References: <20211117073515.773106-1-Vijaikumar_Kanagarajan@mentor.com> MIME-Version: 1.0 X-ClientProxiedBy: svr-orw-mbx-11.mgc.mentorg.com (147.34.90.211) To svr-orw-mbx-01.mgc.mentorg.com (147.34.90.201) X-Original-Sender: vijaikumar_kanagarajan@mentor.com X-Original-Authentication-Results: gmr-mx.google.com; spf=pass (google.com: domain of vijaikumar_kanagarajan@mentor.com designates 68.232.129.153 as permitted sender) smtp.mailfrom=Vijaikumar_Kanagarajan@mentor.com Precedence: list Mailing-list: list isar-users@googlegroups.com; contact isar-users+owners@googlegroups.com List-ID: X-Spam-Checked-In-Group: isar-users@googlegroups.com X-Google-Group-Id: 914930254986 List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_MED, DKIM_SIGNED, DKIM_VALID,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_MSPIKE_H2,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on shymkent.ilbers.de X-getmail-retrieved-from-mailbox: INBOX Use DEB_BUILD_PROFILES to select packages to build. This patch drops the U_BOOT_TOOLS_PACKAGE and U_BOOT_CONFIG_PACKAGE variables and introduces U_BOOT_BUILD_PROFILES through which we can enable particular packages like u-boot-tools, u-boot-dev and u-boot-config. Also, provide backward compatibility with U_BOOT_*_PACKAGES variable and prompt a deprecation warning to user. Signed-off-by: Vijai Kumar K --- RECIPE-API-CHANGELOG.md | 12 +++++ .../u-boot/u-boot-de0-nano-soc_2021.10.bb | 2 +- .../u-boot/files/debian/control.tmpl | 13 +++++ meta/recipes-bsp/u-boot/files/debian/rules | 9 +++- meta/recipes-bsp/u-boot/u-boot-custom.inc | 52 ++++++++++--------- 5 files changed, 61 insertions(+), 27 deletions(-) diff --git a/RECIPE-API-CHANGELOG.md b/RECIPE-API-CHANGELOG.md index 7312d4d..3b926bd 100644 --- a/RECIPE-API-CHANGELOG.md +++ b/RECIPE-API-CHANGELOG.md @@ -301,3 +301,15 @@ Kernel update with "apt-get" will not work since bootloader configuration will not be updated. It used to "kind of work" for grub and efi, that hack is gone. When using the plugins it is advised to name the partition "/boot" and to exclude boot from the follwing rootfs to not waste space. + +### Deprecate U_BOOT_TOOLS_PACKAGE and U_BOOT_CONFIG_PACKAGE + +Use U_BOOT_BUILD_PROFILES instead of U_BOOT_TOOLS_PACKAGE and U_BOOT_CONFIG_PACKAGE + +U_BOOT_TOOLS_PACKAGE = "1" is achieved by U_BOOT_BUILD_PROFILES += "tools" +U_BOOT_CONFIG_PACKAGE = "1" is achieved by U_BOOT_BUILD_PROFILES += "config" + +u-boot-${MACHINE}-dev package build can also be controlled now. Enabled by default +in u-boot-custom.inc. To remove use the below code in your recipe. + +U_BOOT_BUILD_PROFILES_remove = "dev" diff --git a/meta-isar/recipes-bsp/u-boot/u-boot-de0-nano-soc_2021.10.bb b/meta-isar/recipes-bsp/u-boot/u-boot-de0-nano-soc_2021.10.bb index dcb76c5..640b7ea 100644 --- a/meta-isar/recipes-bsp/u-boot/u-boot-de0-nano-soc_2021.10.bb +++ b/meta-isar/recipes-bsp/u-boot/u-boot-de0-nano-soc_2021.10.bb @@ -6,4 +6,4 @@ require u-boot-${PV}.inc # Just for testing purposes, distro package would be recent enough -U_BOOT_TOOLS_PACKAGE = "1" +U_BOOT_BUILD_PROFILES += "tools" diff --git a/meta/recipes-bsp/u-boot/files/debian/control.tmpl b/meta/recipes-bsp/u-boot/files/debian/control.tmpl index 9379be7..7ac11ad 100644 --- a/meta/recipes-bsp/u-boot/files/debian/control.tmpl +++ b/meta/recipes-bsp/u-boot/files/debian/control.tmpl @@ -11,4 +11,17 @@ Description: ${DESCRIPTION}, bootloader binaries Package: u-boot-${MACHINE}-dev Architecture: ${DISTRO_ARCH} +Build-Profiles: Description: ${DESCRIPTION}, bootloader libraries + +Package: u-boot-tools +Architecture: linux-any +Build-Profiles: +Depends: ${shlibs:Depends}, ${misc:Depends} +Description: ${DESCRIPTION}, companion tools + +Package: u-boot-${MACHINE}-config +Build-Profiles: +Provides: u-boot-config +Architecture: ${DISTRO_ARCH} +Description: ${DESCRIPTION}, environment configuration diff --git a/meta/recipes-bsp/u-boot/files/debian/rules b/meta/recipes-bsp/u-boot/files/debian/rules index 3d66762..121b00e 100755 --- a/meta/recipes-bsp/u-boot/files/debian/rules +++ b/meta/recipes-bsp/u-boot/files/debian/rules @@ -20,10 +20,17 @@ override_dh_auto_build: else \ ./scripts/get_default_envs.sh >u-boot-initial-env; \ fi - $(MAKE) $(PARALLEL_MAKE) $(SET_CROSS_BUILD_TOOLS) NO_SDL=1 tools-only envtools +ifneq (,$(filter dev,$(DEB_BUILD_PROFILES))) + $(MAKE) $(PARALLEL_MAKE) $(SET_CROSS_BUILD_TOOLS) NO_SDL=1 envtools +endif +ifneq (,$(filter tools,$(DEB_BUILD_PROFILES))) + $(MAKE) $(PARALLEL_MAKE) $(SET_CROSS_BUILD_TOOLS) NO_SDL=1 tools-only +endif override_dh_auto_install: +ifneq (,$(filter dev,$(DEB_BUILD_PROFILES))) mv tools/env/lib.a tools/env/libubootenv.a +endif override_dh_auto_test: diff --git a/meta/recipes-bsp/u-boot/u-boot-custom.inc b/meta/recipes-bsp/u-boot/u-boot-custom.inc index 57d4f36..f7a7dac 100644 --- a/meta/recipes-bsp/u-boot/u-boot-custom.inc +++ b/meta/recipes-bsp/u-boot/u-boot-custom.inc @@ -9,10 +9,13 @@ FILESEXTRAPATHS_prepend := "${FILE_DIRNAME}/files:" DESCRIPTION ?= "Custom U-Boot" -PROVIDES += "u-boot-${MACHINE} u-boot-${MACHINE}-dev" -PROVIDES += "${@'u-boot-tools' if d.getVar('U_BOOT_TOOLS_PACKAGE') == '1' else ''}" +PROVIDES += "u-boot-${MACHINE}" +PROVIDES += "${@'u-boot-tools' \ + if bb.utils.contains('U_BOOT_BUILD_PROFILES', 'tools', 1, 0, d) else ''}" PROVIDES += "${@('u-boot-config u-boot-' + d.getVar('MACHINE') + '-config') \ - if d.getVar('U_BOOT_CONFIG_PACKAGE') == '1' else ''}" + if bb.utils.contains('U_BOOT_BUILD_PROFILES', 'config', 1, 0, d) else ''}" +PROVIDES += "${@('u-boot-' + d.getVar('MACHINE') + '-dev') \ + if bb.utils.contains('U_BOOT_BUILD_PROFILES', 'dev', 1, 0, d) else ''}" inherit dpkg @@ -22,6 +25,20 @@ python() { if d.getVar('BUILD_DEPENDS'): bb.warn("u-boot-custom: Deprecated use of BUILD_DEPENDS, please switch to DEBIAN_BUILD_DEPENDS") d.setVar('DEBIAN_BUILD_DEPENDS', d.getVar('BUILD_DEPENDS')) + + if d.getVar('U_BOOT_TOOLS_PACKAGE') is not None: + bb.warn("u-boot-custom: Deprecated use of U_BOOT_TOOLS_PACKAGE, please switch to U_BOOT_BUILD_PROFILES") + if d.getVar('U_BOOT_TOOLS_PACKAGE') == "1": + d.appendVar('U_BOOT_BUILD_PROFILES', " tools") + else: + d.setVar('U_BOOT_BUILD_PROFILES_remove', "tools") + + if d.getVar('U_BOOT_CONFIG_PACKAGE') is not None: + bb.warn("u-boot-custom: Deprecated use of U_BOOT_CONFIG_PACKAGE, please switch to U_BOOT_BUILD_PROFILES") + if d.getVar('U_BOOT_CONFIG_PACKAGE') == "1": + d.appendVar('U_BOOT_BUILD_PROFILES', " config") + else: + d.setVar('U_BOOT_BUILD_PROFILES_remove', "config") } DEBIAN_BUILD_DEPENDS ?= "bc, bison, flex, device-tree-compiler, git, libssl-dev:native" @@ -31,8 +48,8 @@ DEBIAN_BUILD_DEPENDS += "${@', libssl-dev' if d.getVar('ISAR_CROSS_COMPILE') == TEMPLATE_FILES = "debian/control.tmpl" TEMPLATE_VARS += "MACHINE DEBIAN_BUILD_DEPENDS" -U_BOOT_TOOLS_PACKAGE ?= "0" -U_BOOT_CONFIG_PACKAGE ?= "0" + +U_BOOT_BUILD_PROFILES ?= "dev" do_prepare_build() { cp -r ${WORKDIR}/debian ${S}/ @@ -42,31 +59,15 @@ do_prepare_build() { echo "${U_BOOT_BIN} /usr/lib/u-boot/${MACHINE}" > \ ${S}/debian/u-boot-${MACHINE}.install - echo "tools/env/libubootenv.a usr/lib" > \ - ${S}/debian/u-boot-${MACHINE}-dev.install - - if [ "${U_BOOT_TOOLS_PACKAGE}" = "1" ]; then - cat <>${S}/debian/control - -Package: u-boot-tools -Architecture: linux-any -Depends: \${shlibs:Depends}, \${misc:Depends} -Description: ${DESCRIPTION}, companion tools -EOF + if [ "${@bb.utils.contains('U_BOOT_BUILD_PROFILES', 'dev', 'yes', 'no', d)}" = "yes" ];then + echo "tools/env/libubootenv.a usr/lib" > \ + ${S}/debian/u-boot-${MACHINE}-dev.install fi - if [ "${U_BOOT_CONFIG_PACKAGE}" = "1" ]; then + if [ "${@bb.utils.contains('U_BOOT_BUILD_PROFILES', 'config', 'yes', 'no', d)}" = "yes" ];then cp ${WORKDIR}/fw_env.config ${S}/ || \ die "U_BOOT_CONFIG_PACKAGE requires a fw_env.config in SRC_URI" - cat <>${S}/debian/control - -Package: u-boot-${MACHINE}-config -Provides: u-boot-config -Architecture: ${DISTRO_ARCH} -Description: ${DESCRIPTION}, environment configuration -EOF - cat <>${S}/debian/u-boot-${MACHINE}-config.install u-boot-initial-env /etc fw_env.config /etc @@ -77,4 +78,5 @@ EOF dpkg_runbuild_prepend() { export U_BOOT_CONFIG="${U_BOOT_CONFIG}" export U_BOOT_BIN="${U_BOOT_BIN}" + export DEB_BUILD_PROFILES="${U_BOOT_BUILD_PROFILES}" }