[U-Boot] [PATCH 5/5] libfdt: split fdt_region declarations out to <fdt_region.h>

[Simon Glass]

Hi Masahiro,

On 21 March 2018 at 03:03, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> fdt_region APIs are not part of libfdt.  They are U-Boot extension
> for the verified boot.  Split the declartions related to fdt_region
> out ot <fdt_region.h>.  This allows <linux/libfdt.h> to become a
> simple wrapper file, like Linux does.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> ---
>
>  common/fdt_region.c    |   1 +
>  common/image-sig.c     |   1 +
>  include/fdt_region.h   | 303 +++++++++++++++++++++++++++++++++++++++++++++++++
>  include/linux/libfdt.h | 299 ------------------------------------------------
>  tools/fdtgrep.c        |   1 +
>  tools/image-host.c     |   1 +
>  6 files changed, 307 insertions(+), 299 deletions(-)
>  create mode 100644 include/fdt_region.h

I think moving the code into a separate header is fine, but I'm not
keen on putting the implementation in common/ as it is a feature of
libfdt. How does this actually help us?

Regards,
Simon