* [PATCH] board: sifive: overwrite board_fdt_blob_setup in u-boot proper
@ 2021-09-11 14:27 Bin Meng
2021-09-11 14:28 ` Bin Meng
0 siblings, 1 reply; 2+ messages in thread
From: Bin Meng @ 2021-09-11 14:27 UTC (permalink / raw)
To: Zong Li, Leo Yu-Chi Liang, Rick Chen, u-boot
Commit 47d73ba4f4a4 ("board: sifive: overwrite board_fdt_blob_setup in u-boot proper")
added a board-specific implementation of board_fdt_blob_setup() which
takes a pointer as the return value, but it does not return anything
if CONFIG_OF_SEPARATE is not enabled. This will cause a build warning
seen when testing booting S-mode U-Boot directly from QEMU, per the
instructions in [1]:
board/sifive/unleashed/unleashed.c: In function ‘board_fdt_blob_setup’:
board/sifive/unleashed/unleashed.c:125:1: warning: control reaches end of non-void function [-Wreturn-type]
Return &_end as the default case.
[1] https://qemu.readthedocs.io/en/latest/system/riscv/sifive_u.html#running-u-boot
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
---
board/sifive/unleashed/unleashed.c | 4 ++--
board/sifive/unmatched/unmatched.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/board/sifive/unleashed/unleashed.c b/board/sifive/unleashed/unleashed.c
index 8cd514df30..33baeda986 100644
--- a/board/sifive/unleashed/unleashed.c
+++ b/board/sifive/unleashed/unleashed.c
@@ -119,9 +119,9 @@ void *board_fdt_blob_setup(void)
if (IS_ENABLED(CONFIG_OF_SEPARATE)) {
if (gd->arch.firmware_fdt_addr)
return (ulong *)gd->arch.firmware_fdt_addr;
- else
- return (ulong *)&_end;
}
+
+ return (ulong *)&_end;
}
int board_init(void)
diff --git a/board/sifive/unmatched/unmatched.c b/board/sifive/unmatched/unmatched.c
index d90b252bae..8773b660fa 100644
--- a/board/sifive/unmatched/unmatched.c
+++ b/board/sifive/unmatched/unmatched.c
@@ -16,9 +16,9 @@ void *board_fdt_blob_setup(void)
if (IS_ENABLED(CONFIG_OF_SEPARATE)) {
if (gd->arch.firmware_fdt_addr)
return (ulong *)gd->arch.firmware_fdt_addr;
- else
- return (ulong *)&_end;
}
+
+ return (ulong *)&_end;
}
int board_init(void)
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] board: sifive: overwrite board_fdt_blob_setup in u-boot proper
2021-09-11 14:27 [PATCH] board: sifive: overwrite board_fdt_blob_setup in u-boot proper Bin Meng
@ 2021-09-11 14:28 ` Bin Meng
0 siblings, 0 replies; 2+ messages in thread
From: Bin Meng @ 2021-09-11 14:28 UTC (permalink / raw)
To: Zong Li, Leo Yu-Chi Liang, Rick Chen, U-Boot Mailing List
On Sat, Sep 11, 2021 at 10:27 PM Bin Meng <bmeng.cn@gmail.com> wrote:
>
> Commit 47d73ba4f4a4 ("board: sifive: overwrite board_fdt_blob_setup in u-boot proper")
> added a board-specific implementation of board_fdt_blob_setup() which
> takes a pointer as the return value, but it does not return anything
> if CONFIG_OF_SEPARATE is not enabled. This will cause a build warning
> seen when testing booting S-mode U-Boot directly from QEMU, per the
> instructions in [1]:
>
> board/sifive/unleashed/unleashed.c: In function ‘board_fdt_blob_setup’:
> board/sifive/unleashed/unleashed.c:125:1: warning: control reaches end of non-void function [-Wreturn-type]
>
> Return &_end as the default case.
>
> [1] https://qemu.readthedocs.io/en/latest/system/riscv/sifive_u.html#running-u-boot
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> ---
>
> board/sifive/unleashed/unleashed.c | 4 ++--
> board/sifive/unmatched/unmatched.c | 4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
Oops, wrong commit title, please ignore.
Regards,
Bin
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-09-11 14:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-11 14:27 [PATCH] board: sifive: overwrite board_fdt_blob_setup in u-boot proper Bin Meng
2021-09-11 14:28 ` Bin Meng
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).