Hi Atish

> From: U-Boot [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Atish Patra
> Sent: Friday, February 28, 2020 5:01 AM
> To: u-boot@lists.denx.de
> Cc: Atish Patra; Alexander Graf; Anup Patel; Bin Meng; Joe Hershberger; Loic 
> Pallardy; Lukas Auer; Marek BehĂșn; Marek Vasut; Patrick Delaunay; Peng Fan; 
> Philippe Reynes; Simon Glass; Simon Goldschmidt; Stefano Babic; Thierry 
> Reding; tr...@konsulko.com; Heinrich Schuchardt; Ard Biesheuvel; 
> l...@nuviainc.com; abner.ch...@hpe.com; daniel.schae...@hpe.com
> Subject: [v1 PATCH 1/1] riscv: Add boot hartid to Device tree
>
> Linux booting protocol mandates that register "a0" contains the hartid.
> However, U-boot can not pass the hartid via a0 during via standard UEFI 
> protocol. DT nodes are commonly used to pass such information to the OS.
>
> Add a DT node under chosen node to indicate the boot hartid. EFI stub in 
> Linux kernel will parse this node and pass it to the real kernel in "a0" 
> before jumping to it.
>
> Signed-off-by: Atish Patra <atish.pa...@wdc.com>
> ---
>  arch/riscv/lib/bootm.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
>
> diff --git a/arch/riscv/lib/bootm.c b/arch/riscv/lib/bootm.c index 
> fad16901c5f2..dd359a45c2e1 100644
> --- a/arch/riscv/lib/bootm.c
> +++ b/arch/riscv/lib/bootm.c
> @@ -28,6 +28,19 @@ __weak void board_quiesce_devices(void)
>
>  int arch_fixup_fdt(void *blob)
>  {
> +       u32 size;
> +       int chosen_offset, err;
> +
> +       size = fdt_totalsize(blob);
> +       err  = fdt_open_into(blob, blob, size + 32);
> +       if (err < 0) {
> +               printf("Device Tree can't be expanded to accommodate new 
> node");
> +               return -1;
> +       }
> +       chosen_offset = fdt_path_offset(blob, "/chosen");
> +       fdt_setprop_u32(blob, chosen_offset, "boot-hartid",
> +                          gd->arch.boot_hart);

Is it also OK for RV64 ?

Other than that,
Reviewed-by: Rick Chen <r...@andestech.com>

> +
>         return 0;
>  }
>
> --
> 2.24.0
>

Reply via email to