[PATCH] doc: Fix fw_payload.md

Anup Patel anup at brainfault.org
Thu Oct 5 21:38:53 PDT 2023


On Mon, Sep 25, 2023 at 3:19 AM Hoa Nguyen <hn at hnpl.org> wrote:
>
> The base of .text is defined by `FW_TEXT_START`, not `FW_TEXT_BASE`.
>
> Signed-off-by: Hoa Nguyen <hn at hnpl.org>

Looks good to me.

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

Applied this patch to the riscv/opensbi repo.

Thanks,
Anup

> ---
>  docs/firmware/fw_payload.md | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/docs/firmware/fw_payload.md b/docs/firmware/fw_payload.md
> index 3d50ddc..113604a 100644
> --- a/docs/firmware/fw_payload.md
> +++ b/docs/firmware/fw_payload.md
> @@ -36,7 +36,7 @@ options. These configuration parameters can be defined using either the top
>  level `make` command line or the target platform *objects.mk* configuration
>  file. The parameters currently defined are as follows:
>
> -* **FW_PAYLOAD_OFFSET** - Offset from *FW_TEXT_BASE* where the payload binary
> +* **FW_PAYLOAD_OFFSET** - Offset from *FW_TEXT_START* where the payload binary
>    will be linked in the final *FW_PAYLOAD* firmware binary image. This
>    configuration parameter is mandatory if *FW_PAYLOAD_ALIGN* is not defined.
>    Compilation errors will result from an incorrect definition of
> --
> 2.25.1
>
>
> --
> opensbi mailing list
> opensbi at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi



More information about the opensbi mailing list