diff mbox series

[U-Boot] fdt: restore board_fdt_blob_setup() declaration

Message ID 409cd2580def16ae9af9849e6f6ad1b0e73b405b.1540730474.git.baruch@tkos.co.il
State Accepted
Commit b23644858bec1a21743debccb6643af7df5c48b4
Delegated to: Simon Glass
Headers show
Series [U-Boot] fdt: restore board_fdt_blob_setup() declaration | expand

Commit Message

Baruch Siach Oct. 28, 2018, 12:41 p.m. UTC
Commit 90c08fa038451d (fdt: Add device tree memory bindings) removed the
prototype declaration of board_fdt_blob_setup(), most likely by mistake.
This didn't break the build because the only file calling this function
(lib/fdtdec.c) provides a local weak definition. Restore the
declaration.

Cc: Michael Pratt <mpratt@chromium.org>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 include/fdtdec.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Simon Glass Nov. 16, 2018, 2:05 a.m. UTC | #1
On 28 October 2018 at 05:41, Baruch Siach <baruch@tkos.co.il> wrote:
> Commit 90c08fa038451d (fdt: Add device tree memory bindings) removed the
> prototype declaration of board_fdt_blob_setup(), most likely by mistake.
> This didn't break the build because the only file calling this function
> (lib/fdtdec.c) provides a local weak definition. Restore the
> declaration.
>
> Cc: Michael Pratt <mpratt@chromium.org>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  include/fdtdec.h | 1 +
>  1 file changed, 1 insertion(+)
>

Reviewed-by: Simon Glass <sjg@chromium.org>
Simon Glass Nov. 22, 2018, 8:21 p.m. UTC | #2
On 28 October 2018 at 05:41, Baruch Siach <baruch@tkos.co.il> wrote:
> Commit 90c08fa038451d (fdt: Add device tree memory bindings) removed the
> prototype declaration of board_fdt_blob_setup(), most likely by mistake.
> This didn't break the build because the only file calling this function
> (lib/fdtdec.c) provides a local weak definition. Restore the
> declaration.
>
> Cc: Michael Pratt <mpratt@chromium.org>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  include/fdtdec.h | 1 +
>  1 file changed, 1 insertion(+)
>

Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot-dm/master, thanks!
diff mbox series

Patch

diff --git a/include/fdtdec.h b/include/fdtdec.h
index 83be06403d8b..250df41e636d 100644
--- a/include/fdtdec.h
+++ b/include/fdtdec.h
@@ -1001,6 +1001,7 @@  int fdtdec_setup(void);
  * Called when CONFIG_OF_BOARD is defined, or if CONFIG_OF_SEPARATE is defined
  * and the board implements it.
  */
+void *board_fdt_blob_setup(void);
 
 /*
  * Decode the size of memory