[PATCH 3/3] docs: domain_support: Update the DT example

Anup Patel anup at brainfault.org
Mon Feb 27 05:55:15 PST 2023


On Fri, Feb 24, 2023 at 10:02 AM Bin Meng <bmeng at tinylab.org> wrote:
>
> commit 3e2f573e707e ("lib: utils: Disallow non-root domains from adding M-mode regions")
> added access permission check in __fdt_parse_region(). With the
> existing DT example in the doc OpenSBI won't boot anymore.
>
> Let's update the DT example so that it can work out of the box.
>
> Signed-off-by: Bin Meng <bmeng at tinylab.org>

Looks good to me.

Reviewed-by: Anup Patel <anup at brainfault.org>

Applied this patch to the riscv/opensbi repo.

Thanks,
Anup

> ---
>
>  docs/domain_support.md | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/docs/domain_support.md b/docs/domain_support.md
> index 4ccbcff..2a9ca3d 100644
> --- a/docs/domain_support.md
> +++ b/docs/domain_support.md
> @@ -254,7 +254,7 @@ be done:
>              tdomain: trusted-domain {
>                  compatible = "opensbi,domain,instance";
>                  possible-harts = <&cpu0>;
> -                regions = <&tmem 0x7>, <&tuart 0x7>;
> +                regions = <&tmem 0x3f>, <&tuart 0x3f>;
>                  boot-hart = <&cpu0>;
>                  next-arg1 = <0x0 0x0>;
>                  next-addr = <0x0 0x80100000>;
> @@ -265,7 +265,7 @@ be done:
>              udomain: untrusted-domain {
>                  compatible = "opensbi,domain,instance";
>                  possible-harts = <&cpu1 &cpu2 &cpu3 &cpu4>;
> -                regions = <&tmem 0x0>, <&tuart 0x0>, <&allmem 0x7>;
> +                regions = <&tmem 0x0>, <&tuart 0x0>, <&allmem 0x3f>;
>              };
>          };
>      };
> --
> 2.25.1
>



More information about the opensbi mailing list