diff mbox series

docs: firmware: Document FW_PIC compile time option

Message ID 20210712004802.23368-1-bmeng.cn@gmail.com
State Accepted
Headers show
Series docs: firmware: Document FW_PIC compile time option | expand

Commit Message

Bin Meng July 12, 2021, 12:48 a.m. UTC
FW_PIC=y is on by default, but the doc is missing when this was
introduced. Add some description for it.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
---

 docs/firmware/fw.md | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Anup Patel July 17, 2021, 12:16 p.m. UTC | #1
On Mon, Jul 12, 2021 at 6:18 AM Bin Meng <bmeng.cn@gmail.com> wrote:
>
> FW_PIC=y is on by default, but the doc is missing when this was
> introduced. Add some description for it.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>

Thanks for adding this.

Reviewed-by: Anup Patel <anup.patel@wdc.com>

Applied this patch to the riscv/opensbi repo.

Thanks,
Anup

> ---
>
>  docs/firmware/fw.md | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/docs/firmware/fw.md b/docs/firmware/fw.md
> index a3bb70c..db9a31c 100644
> --- a/docs/firmware/fw.md
> +++ b/docs/firmware/fw.md
> @@ -69,6 +69,12 @@ parameters:
>    argument by the prior booting stage.
>  * **FW_FDT_PADDING** - Optional zero bytes padding to the embedded flattened
>    device tree binary file specified by **FW_FDT_PATH** option.
> +* **FW_PIC** - "FW_PIC=y" generates position independent executable firmware
> +  images. OpenSBI can run at arbitrary address with appropriate alignment.
> +  Therefore, the original relocation mechanism ("FW_PIC=n") will be skipped.
> +  In other words, OpenSBI will directly run at the load address without any
> +  code movement. This option requires a toolchain with PIE support, and it
> +  is on by default.
>
>  Additionally, each firmware type as a set of type specific configuration
>  parameters. Detailed information for each firmware type can be found in the
> --
> 2.25.1
>
>
> --
> opensbi mailing list
> opensbi@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi
diff mbox series

Patch

diff --git a/docs/firmware/fw.md b/docs/firmware/fw.md
index a3bb70c..db9a31c 100644
--- a/docs/firmware/fw.md
+++ b/docs/firmware/fw.md
@@ -69,6 +69,12 @@  parameters:
   argument by the prior booting stage.
 * **FW_FDT_PADDING** - Optional zero bytes padding to the embedded flattened
   device tree binary file specified by **FW_FDT_PATH** option.
+* **FW_PIC** - "FW_PIC=y" generates position independent executable firmware
+  images. OpenSBI can run at arbitrary address with appropriate alignment.
+  Therefore, the original relocation mechanism ("FW_PIC=n") will be skipped.
+  In other words, OpenSBI will directly run at the load address without any
+  code movement. This option requires a toolchain with PIE support, and it
+  is on by default.
 
 Additionally, each firmware type as a set of type specific configuration
 parameters. Detailed information for each firmware type can be found in the