Hi Masahiro,
On 21 March 2018 at 03:03, Masahiro Yamada yamada.masahiro@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@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