[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4] arm: Add Kconfig entry to select CONFIG_DTB_FILE


  • To: Michal Orzel <Michal.Orzel@xxxxxxx>
  • From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
  • Date: Wed, 10 Mar 2021 08:05:46 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=aJRcvDglW3nsBhwz9XMyxQjlKduvqxHw916myrBus/4=; b=D2nGvgrOTWAsfQfaYKEuFVTS/uE5qHRw8iaTJpUvu0wPGZJrmysldnHnAUfm3ai5UUmhezTGg610lUFQJz9U/2vzFIthW3iJqcVFXiflNVImux9qbBUAHkbUXDn3Bq0KsqxyGmbI7ut3sMwXp+X2HKjk5ARVGmRqt3A2v6uGF0zYVG2GuEwpAkwaU+07KJ1TpiFuaa4B4YH/lTACP/o404pvyXJ1vEhCPbHbQo+HSeRAX1nMgHeQlh/T9ae+EWC3apwT8fO7mqpurYd/ZDdJG8JSeJkaUCLyWcGIdrY4UIcB9IxaAwjYfFKNlDIL4TGfx1RMW0aLienvtRfwBEZgog==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hay59Qmu4Qr7Zw3wh9anufjKcdFtNYnXKRgiRcgYzek21brXMuxq0jLmAaW/P2E36r4YwPaKVukSJEh4VpPSIcXu8OPI5L7YJALvnryAog6AC6mNLKNXPtvNNAqCvP4+NPEe8TO/othh4Nz3HSo/ov3kxROyCulL7R4b/+n8lZEdKanR31oNIxmvYYbteIUWC5RtMggWx7ABL4C6wNKJdhs91oLEwac98qgO49RdUV4vNmZ0kgMA707vNUlmTLe4iZ0F6kXt+MYO+YTAv3BM7IIhusimCbWy8vXJOfPpIauKaOqfN6Dem1ozcDghLxYzAtcia8HpF3Fy+o+CEyYrtg==
  • Authentication-results-original: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=arm.com;
  • Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Wed, 10 Mar 2021 08:06:13 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHXFXrOKr1ioWnJ5USYV383dKI5L6p83WMA
  • Thread-topic: [PATCH v4] arm: Add Kconfig entry to select CONFIG_DTB_FILE

Hi,

> On 10 Mar 2021, at 07:58, Michal Orzel <Michal.Orzel@xxxxxxx> wrote:
> 
> Currently in order to link existing DTB into Xen image
> we need to either specify option CONFIG_DTB_FILE on the
> command line or manually add it into .config.
> Add Kconfig entry: CONFIG_DTB_FILE to be able to
> provide the path to DTB we want to embed into Xen image.
> If no path provided - the dtb will not be embedded.
> 
> Remove the line: AFLAGS-y += -DCONFIG_DTB_FILE=\"$(CONFIG_DTB_FILE)\"
> as it is not needed since Kconfig will define it in a header
> with all the other config options.
> 
> Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>
Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>

Cheers
Bertrand

> ---
> xen/arch/arm/Makefile | 5 ++---
> xen/common/Kconfig    | 8 ++++++++
> 2 files changed, 10 insertions(+), 3 deletions(-)
> 
> diff --git a/xen/arch/arm/Makefile b/xen/arch/arm/Makefile
> index 16e6523e2c..46e6a95fec 100644
> --- a/xen/arch/arm/Makefile
> +++ b/xen/arch/arm/Makefile
> @@ -68,9 +68,8 @@ extra-y += $(TARGET_SUBARCH)/head.o
> 
> #obj-bin-y += ....o
> 
> -ifdef CONFIG_DTB_FILE
> +ifneq ($(CONFIG_DTB_FILE),"")
> obj-y += dtb.o
> -AFLAGS-y += -DCONFIG_DTB_FILE=\"$(CONFIG_DTB_FILE)\"
> endif
> 
> ALL_OBJS := $(TARGET_SUBARCH)/head.o $(ALL_OBJS)
> @@ -137,7 +136,7 @@ asm-offsets.s: $(TARGET_SUBARCH)/asm-offsets.c
> xen.lds: xen.lds.S
>       $(CPP) -P $(a_flags) -MQ $@ -o $@ $<
> 
> -dtb.o: $(CONFIG_DTB_FILE)
> +dtb.o: $(patsubst "%",%,$(CONFIG_DTB_FILE))
> 
> .PHONY: clean
> clean::
> diff --git a/xen/common/Kconfig b/xen/common/Kconfig
> index eb953d171e..a27836bf47 100644
> --- a/xen/common/Kconfig
> +++ b/xen/common/Kconfig
> @@ -400,6 +400,14 @@ config DOM0_MEM
> 
>         Leave empty if you are not sure what to specify.
> 
> +config DTB_FILE
> +     string "Absolute path to device tree blob"
> +     depends on HAS_DEVICE_TREE
> +     ---help---
> +       When using a bootloader that has no device tree support or when there
> +       is no bootloader at all, use this option to specify the absolute path
> +       to a device tree that will be linked directly inside Xen binary.
> +
> config TRACEBUFFER
>       bool "Enable tracing infrastructure" if EXPERT
>       default y
> -- 
> 2.29.0
> 




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.