[U-Boot] [PATCH 5/5] lib: libfdt: wrap scripts/dtc/libfdt/* where possible
Simon Glass
sjg at chromium.org
Sun Oct 22 14:35:08 UTC 2017
Hi Masahiro,
On 17 October 2017 at 15:30, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> lib/libfdt/ and scripts/dtc/libfdt have the same copies for the
> followings 6 files:
> fdt.c fdt_addresses.c fdt_empty_tree.c fdt_overlay.c fdt_strerr.c
> fdt_sw.c
>
> Make them a wrapper of scripts/dtc/libfdt/*. This is exactly what
> Linux does to sync libfdt. In order to make is possible, import
> <linux/libfdt.h> and <linux/libfdt_env.h> from Linux 4.14-rc5.
>
> Unfortunately, U-Boot locally modified the following 3 files:
> fdt_ro.c fdt_wip.c fdt_rw.c
>
> The fdt_region.c is U-Boot own file.
>
> I did not touch them in order to avoid unpredictable impact.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> ---
>
> include/libfdt_env.h | 6 -
> include/linux/libfdt.h | 17 +
> include/linux/libfdt_env.h | 22 ++
> lib/libfdt/Makefile | 17 +-
> lib/libfdt/fdt.c | 212 +----------
> lib/libfdt/fdt_addresses.c | 57 +--
> lib/libfdt/fdt_empty_tree.c | 39 +-
> lib/libfdt/fdt_overlay.c | 863 +-------------------------------------------
> lib/libfdt/fdt_strerror.c | 63 +---
> lib/libfdt/fdt_sw.c | 256 +------------
> 10 files changed, 63 insertions(+), 1489 deletions(-)
> create mode 100644 include/linux/libfdt.h
> create mode 100644 include/linux/libfdt_env.h
>
I am not hugely keen on this. At present I can compare upstream with
U-Boot directly. Why do I need to worry about scripts/? Why are the
files even duplicated in there?
Regards,
Simon
More information about the U-Boot
mailing list