All of lore.kernel.org
 help / color / mirror / Atom feed
From: Devarsh Thakkar <devarsht@ti.com>
To: Simon Glass <sjg@chromium.org>,
	U-Boot Mailing List <u-boot@lists.denx.de>
Cc: Tom Rini <trini@konsulko.com>,
	Heinrich Schuchardt <xypron.glpk@gmx.de>,
	Bin Meng <bmeng.cn@gmail.com>,
	Fabrice Gasnier <fabrice.gasnier@foss.st.com>,
	Hugo Villeneuve <hvilleneuve@dimonoff.com>,
	Nikhil M Jain <n-jain1@ti.com>,
	Patrice Chotard <patrice.chotard@foss.st.com>,
	Patrick Delaunay <patrick.delaunay@foss.st.com>,
	Rasmus Villemoes <rasmus.villemoes@prevas.dk>,
	Stefan Roese <sr@denx.de>
Subject: Re: [PATCH 08/19] boot: Move fdt_support to boot/
Date: Thu, 31 Aug 2023 11:32:40 +0530	[thread overview]
Message-ID: <d04cf2fd-92d0-46d1-5041-9ec4480bd6fd@ti.com> (raw)
In-Reply-To: <20230824195917.1687886-9-sjg@chromium.org>

Hi Simon,

On 25/08/23 01:28, Simon Glass wrote:
> This relates to booting since it fixes up the devicetree for the OS. Move
> it into the boot/ directory.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
> 
>  boot/Makefile                  | 3 +++
>  {common => boot}/fdt_support.c | 0
>  common/Makefile                | 2 --
>  3 files changed, 3 insertions(+), 2 deletions(-)
>  rename {common => boot}/fdt_support.c (100%)
> 
> diff --git a/boot/Makefile b/boot/Makefile
> index 10f015722378..f15a161614ff 100644
> --- a/boot/Makefile
> +++ b/boot/Makefile
> @@ -16,6 +16,7 @@ obj-$(CONFIG_QFW) += bootmeth_qfw.o
>  endif
>  
>  obj-y += image.o image-board.o
> +
>  obj-$(CONFIG_ANDROID_AB) += android_ab.o
>  obj-$(CONFIG_ANDROID_BOOT_IMAGE) += image-android.o image-android-dt.o
>  
> @@ -37,6 +38,8 @@ obj-$(CONFIG_$(SPL_TPL_)BOOTSTD) += bootflow_menu.o
>  obj-$(CONFIG_$(SPL_TPL_)CEDIT) += cedit.o
>  endif
>  
> +obj-$(CONFIG_$(SPL_TPL_)OF_LIBFDT) += fdt_support.o
> +
>  obj-$(CONFIG_$(SPL_TPL_)OF_LIBFDT) += image-fdt.o

Can this be,
obj-$(CONFIG_$(SPL_TPL_)OF_LIBFDT) += fdt_support.o image-fdt.o ?

Regards
Devarsh

>  obj-$(CONFIG_$(SPL_TPL_)FIT_SIGNATURE) += fdt_region.o
>  obj-$(CONFIG_$(SPL_TPL_)FIT) += image-fit.o
> diff --git a/common/fdt_support.c b/boot/fdt_support.c
> similarity index 100%
> rename from common/fdt_support.c
> rename to boot/fdt_support.c
> diff --git a/common/Makefile b/common/Makefile
> index 0a3f75f2f1c8..0948721d0b47 100644
> --- a/common/Makefile
> +++ b/common/Makefile
> @@ -18,7 +18,6 @@ obj-$(CONFIG_DISPLAY_BOARDINFO) += board_info.o
>  obj-$(CONFIG_DISPLAY_BOARDINFO_LATE) += board_info.o
>  
>  obj-$(CONFIG_FDT_SIMPLEFB) += fdt_simplefb.o
> -obj-$(CONFIG_$(SPL_TPL_)OF_LIBFDT) += fdt_support.o
>  obj-$(CONFIG_MII) += miiphyutil.o
>  obj-$(CONFIG_CMD_MII) += miiphyutil.o
>  obj-$(CONFIG_PHYLIB) += miiphyutil.o
> @@ -51,7 +50,6 @@ ifdef CONFIG_SPL_DFU
>  obj-$(CONFIG_DFU_OVER_USB) += dfu.o
>  endif
>  obj-$(CONFIG_SPL_NET) += miiphyutil.o
> -obj-$(CONFIG_$(SPL_TPL_)OF_LIBFDT) += fdt_support.o
>  
>  obj-$(CONFIG_SPL_USB_HOST) += usb.o usb_hub.o
>  obj-$(CONFIG_SPL_USB_STORAGE) += usb_storage.o

  reply	other threads:[~2023-08-31  6:03 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-24 19:58 [PATCH 00/19] Kconfig: Tidy up some options Simon Glass
2023-08-24 19:58 ` [PATCH 01/19] lib: rational: Move the Kconfigs into the correct place Simon Glass
2023-08-24 19:58 ` [PATCH 02/19] Kconfig: Move API into general setup Simon Glass
2023-08-24 19:58 ` [PATCH 03/19] video: Hide the BMP options Simon Glass
2023-08-24 19:58 ` [PATCH 04/19] video: Move BMP options and code to video directory Simon Glass
2023-08-31  6:00   ` Devarsh Thakkar
2023-08-24 19:58 ` [PATCH 05/19] net: Move SYS_RX_ETH_BUFFER into the network menu Simon Glass
2023-08-24 19:58 ` [PATCH 06/19] FMU: Avoid showing an unselectable menu option Simon Glass
2023-08-25 12:00   ` Sughosh Ganu
2023-08-25 14:55     ` Tom Rini
2023-08-24 19:58 ` [PATCH 07/19] test: Move POST under a renamed Testing section Simon Glass
2023-08-24 19:58 ` [PATCH 08/19] boot: Move fdt_support to boot/ Simon Glass
2023-08-31  6:02   ` Devarsh Thakkar [this message]
2023-09-04 16:48     ` Simon Glass
2023-08-24 19:59 ` [PATCH 09/19] Move fdt_simplefb " Simon Glass
2023-08-31  6:12   ` Devarsh Thakkar
2023-08-24 19:59 ` [PATCH 10/19] boot: Move some other fdt-fixup options to the same menu Simon Glass
2023-08-24 19:59 ` [PATCH 11/19] boot: Rename Android-boot text Simon Glass
2023-08-24 19:59 ` [PATCH 12/19] Kconfig: Create a menu for FIT Simon Glass
2023-08-24 19:59 ` [PATCH 13/19] Kconfig: Move SPL_FIT under FIT Simon Glass
2023-08-24 19:59 ` [PATCH 14/19] spl: Tidy up load address in spl_ram Simon Glass
2023-08-24 19:59 ` [PATCH 15/19] boot: Make standard boot a menu Simon Glass
2023-08-24 19:59 ` [PATCH 16/19] Kconfig: Move TEXT_BASE et al under general setup Simon Glass
2023-08-24 19:59 ` [PATCH 17/19] Make DISTRO_DEFAULTS as deprecated Simon Glass
2023-08-28 14:41   ` Mark Kettenis
2023-08-28 14:57     ` Tom Rini
2023-08-24 19:59 ` [PATCH 18/19] boot: Join FDT_FIXUP_PARTITIONS with related options Simon Glass
2023-08-24 19:59 ` [PATCH 19/19] boot: Join ARCH_FIXUP_FDT_MEMORY " Simon Glass

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=d04cf2fd-92d0-46d1-5041-9ec4480bd6fd@ti.com \
    --to=devarsht@ti.com \
    --cc=bmeng.cn@gmail.com \
    --cc=fabrice.gasnier@foss.st.com \
    --cc=hvilleneuve@dimonoff.com \
    --cc=n-jain1@ti.com \
    --cc=patrice.chotard@foss.st.com \
    --cc=patrick.delaunay@foss.st.com \
    --cc=rasmus.villemoes@prevas.dk \
    --cc=sjg@chromium.org \
    --cc=sr@denx.de \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=xypron.glpk@gmx.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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.