diff options
| author | Alistair Francis <alistair.francis@wdc.com> | 2019-06-18 17:55:39 -0700 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-06-19 22:13:39 +0100 |
| commit | ef82d0230e2286a22500aeb544277bef4c9dca15 (patch) | |
| tree | cdedd7600583498e3986e76e86558c176efe82fe /meta/recipes-bsp/opensbi | |
| parent | 462ae94d701a871fed7dc9868e03bae335f5e2d3 (diff) | |
| download | poky-ef82d0230e2286a22500aeb544277bef4c9dca15.tar.gz | |
opensbi: Initial commit of OpenSBI
This is used as a bootloader for RISC-V QEMU machines.
(From OE-Core rev: 112ca2174dd97f5ca9ea25f83007d44054abc487)
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-bsp/opensbi')
4 files changed, 252 insertions, 0 deletions
diff --git a/meta/recipes-bsp/opensbi/files/0001-Makefile-Don-t-specify-mabi-or-march.patch b/meta/recipes-bsp/opensbi/files/0001-Makefile-Don-t-specify-mabi-or-march.patch new file mode 100644 index 0000000000..9c2d0eb479 --- /dev/null +++ b/meta/recipes-bsp/opensbi/files/0001-Makefile-Don-t-specify-mabi-or-march.patch | |||
| @@ -0,0 +1,38 @@ | |||
| 1 | From f5871e1f3650d6c8a032928cb5d8ca00c275c377 Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Alistair Francis <alistair.francis@wdc.com> | ||
| 3 | Date: Fri, 15 Feb 2019 14:57:41 -0800 | ||
| 4 | Subject: [PATCH] Makefile: Don't specify mabi or march | ||
| 5 | |||
| 6 | To avoid | ||
| 7 | can't link double-float modules with soft-float modules | ||
| 8 | errors when building 32-bit openSBI don't specify mabi or march. | ||
| 9 | |||
| 10 | Upstream-Status: Inappropriate [Fixes a 32-bit Yocto flow bug] | ||
| 11 | Signed-off-by: Alistair Francis <alistair.francis@wdc.com> | ||
| 12 | --- | ||
| 13 | Makefile | 2 -- | ||
| 14 | 1 file changed, 2 deletions(-) | ||
| 15 | |||
| 16 | diff --git a/Makefile b/Makefile | ||
| 17 | index ae68f55..10851fc 100644 | ||
| 18 | --- a/Makefile | ||
| 19 | +++ b/Makefile | ||
| 20 | @@ -145,7 +145,6 @@ GENFLAGS += $(firmware-genflags-y) | ||
| 21 | CFLAGS = -g -Wall -Werror -nostdlib -fno-strict-aliasing -O2 | ||
| 22 | CFLAGS += -fno-omit-frame-pointer -fno-optimize-sibling-calls | ||
| 23 | CFLAGS += -mno-save-restore -mstrict-align | ||
| 24 | -CFLAGS += -mabi=$(PLATFORM_RISCV_ABI) -march=$(PLATFORM_RISCV_ISA) | ||
| 25 | CFLAGS += -mcmodel=$(PLATFORM_RISCV_CODE_MODEL) | ||
| 26 | CFLAGS += $(GENFLAGS) | ||
| 27 | CFLAGS += $(platform-cflags-y) | ||
| 28 | @@ -158,7 +157,6 @@ CPPFLAGS += $(firmware-cppflags-y) | ||
| 29 | ASFLAGS = -g -Wall -nostdlib -D__ASSEMBLY__ | ||
| 30 | ASFLAGS += -fno-omit-frame-pointer -fno-optimize-sibling-calls | ||
| 31 | ASFLAGS += -mno-save-restore -mstrict-align | ||
| 32 | -ASFLAGS += -mabi=$(PLATFORM_RISCV_ABI) -march=$(PLATFORM_RISCV_ISA) | ||
| 33 | ASFLAGS += -mcmodel=$(PLATFORM_RISCV_CODE_MODEL) | ||
| 34 | ASFLAGS += $(GENFLAGS) | ||
| 35 | ASFLAGS += $(platform-asflags-y) | ||
| 36 | -- | ||
| 37 | 2.20.1 | ||
| 38 | |||
diff --git a/meta/recipes-bsp/opensbi/files/0002-lib-Create-a-sbi_ipi_data-structure.patch b/meta/recipes-bsp/opensbi/files/0002-lib-Create-a-sbi_ipi_data-structure.patch new file mode 100644 index 0000000000..3c846bde62 --- /dev/null +++ b/meta/recipes-bsp/opensbi/files/0002-lib-Create-a-sbi_ipi_data-structure.patch | |||
| @@ -0,0 +1,128 @@ | |||
| 1 | From 508a27204cbbca0a9430236e56681e5e0d343fb9 Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Alistair Francis <alistair.francis@wdc.com> | ||
| 3 | Date: Fri, 8 Mar 2019 11:22:22 -0800 | ||
| 4 | Subject: [PATCH] lib: Create a sbi_ipi_data structure | ||
| 5 | |||
| 6 | Create a sbi_ipi_data structure that holds unpacked IPI information. At | ||
| 7 | the same time remove ipi_type from the sbi_scratch struct and use a | ||
| 8 | fixed offset to access it. | ||
| 9 | |||
| 10 | This structure fits in behind the sbi_scratch structure. | ||
| 11 | |||
| 12 | This fixes https://github.com/riscv/opensbi/issues/81 | ||
| 13 | |||
| 14 | Upstream-Status: Backport [https://github.com/riscv/opensbi/commit/508a27204cbbca0a9430236e56681e5e0d343fb9] | ||
| 15 | Signed-off-by: Alistair Francis <alistair.francis@wdc.com> | ||
| 16 | --- | ||
| 17 | firmware/fw_base.S | 1 - | ||
| 18 | include/sbi/sbi_scratch.h | 23 ++++++++++++++++------- | ||
| 19 | lib/sbi_ipi.c | 8 +++++--- | ||
| 20 | 3 files changed, 21 insertions(+), 11 deletions(-) | ||
| 21 | |||
| 22 | diff --git a/firmware/fw_base.S b/firmware/fw_base.S | ||
| 23 | index cf2c6a0..69cfc90 100644 | ||
| 24 | --- a/firmware/fw_base.S | ||
| 25 | +++ b/firmware/fw_base.S | ||
| 26 | @@ -183,7 +183,6 @@ _start_warm: | ||
| 27 | REG_S a4, SBI_SCRATCH_PLATFORM_ADDR_OFFSET(tp) | ||
| 28 | la a4, _hartid_to_scratch | ||
| 29 | REG_S a4, SBI_SCRATCH_HARTID_TO_SCRATCH_OFFSET(tp) | ||
| 30 | - REG_S zero, SBI_SCRATCH_IPI_TYPE_OFFSET(tp) | ||
| 31 | REG_S zero, SBI_SCRATCH_TMP0_OFFSET(tp) | ||
| 32 | |||
| 33 | /* Setup stack */ | ||
| 34 | diff --git a/include/sbi/sbi_scratch.h b/include/sbi/sbi_scratch.h | ||
| 35 | index 8389ef3..70ab384 100644 | ||
| 36 | --- a/include/sbi/sbi_scratch.h | ||
| 37 | +++ b/include/sbi/sbi_scratch.h | ||
| 38 | @@ -28,17 +28,24 @@ | ||
| 39 | #define SBI_SCRATCH_PLATFORM_ADDR_OFFSET (6 * __SIZEOF_POINTER__) | ||
| 40 | /** Offset of hartid_to_scratch member in sbi_scratch */ | ||
| 41 | #define SBI_SCRATCH_HARTID_TO_SCRATCH_OFFSET (7 * __SIZEOF_POINTER__) | ||
| 42 | -/** Offset of ipi_type member in sbi_scratch */ | ||
| 43 | -#define SBI_SCRATCH_IPI_TYPE_OFFSET (8 * __SIZEOF_POINTER__) | ||
| 44 | /** Offset of tmp0 member in sbi_scratch */ | ||
| 45 | -#define SBI_SCRATCH_TMP0_OFFSET (9 * __SIZEOF_POINTER__) | ||
| 46 | -/** Maximum size of sbi_scratch */ | ||
| 47 | -#define SBI_SCRATCH_SIZE 256 | ||
| 48 | +#define SBI_SCRATCH_TMP0_OFFSET (8 * __SIZEOF_POINTER__) | ||
| 49 | + | ||
| 50 | +/** sbi_ipi_data is located behind sbi_scratch. This struct is not packed. */ | ||
| 51 | +/** Offset of ipi_type in sbi_ipi_data */ | ||
| 52 | +#define SBI_IPI_DATA_IPI_TYPE_OFFSET (15 * __SIZEOF_POINTER__) | ||
| 53 | + | ||
| 54 | +/** Maximum size of sbi_scratch and sbi_ipi_data */ | ||
| 55 | +#define SBI_SCRATCH_SIZE (32 * __SIZEOF_POINTER__) | ||
| 56 | |||
| 57 | #ifndef __ASSEMBLY__ | ||
| 58 | |||
| 59 | #include <sbi/sbi_types.h> | ||
| 60 | |||
| 61 | +struct sbi_ipi_data { | ||
| 62 | + unsigned long ipi_type; | ||
| 63 | +}; | ||
| 64 | + | ||
| 65 | /** Representation of per-HART scratch space */ | ||
| 66 | struct sbi_scratch { | ||
| 67 | /** Start (or base) address of firmware linked to OpenSBI library */ | ||
| 68 | @@ -57,8 +64,6 @@ struct sbi_scratch { | ||
| 69 | unsigned long platform_addr; | ||
| 70 | /** Address of HART ID to sbi_scratch conversion function */ | ||
| 71 | unsigned long hartid_to_scratch; | ||
| 72 | - /** IPI type (or flags) */ | ||
| 73 | - unsigned long ipi_type; | ||
| 74 | /** Temporary storage */ | ||
| 75 | unsigned long tmp0; | ||
| 76 | } __packed; | ||
| 77 | @@ -71,6 +76,10 @@ struct sbi_scratch { | ||
| 78 | #define sbi_scratch_thishart_arg1_ptr() \ | ||
| 79 | ((void *)(sbi_scratch_thishart_ptr()->next_arg1)) | ||
| 80 | |||
| 81 | +/** Get pointer to sbi_ipi_data from sbi_scratch */ | ||
| 82 | +#define sbi_ipi_data_ptr(scratch) \ | ||
| 83 | +((struct sbi_ipi_data *)(void*)scratch + SBI_IPI_DATA_IPI_TYPE_OFFSET) | ||
| 84 | + | ||
| 85 | #endif | ||
| 86 | |||
| 87 | #endif | ||
| 88 | diff --git a/lib/sbi_ipi.c b/lib/sbi_ipi.c | ||
| 89 | index 0e371d7..e0f2a19 100644 | ||
| 90 | --- a/lib/sbi_ipi.c | ||
| 91 | +++ b/lib/sbi_ipi.c | ||
| 92 | @@ -31,7 +31,7 @@ static int sbi_ipi_send(struct sbi_scratch *scratch, u32 hartid, u32 event) | ||
| 93 | * trigger the interrupt | ||
| 94 | */ | ||
| 95 | remote_scratch = sbi_hart_id_to_scratch(scratch, hartid); | ||
| 96 | - atomic_raw_set_bit(event, &remote_scratch->ipi_type); | ||
| 97 | + atomic_raw_set_bit(event, &sbi_ipi_data_ptr(remote_scratch)->ipi_type); | ||
| 98 | mb(); | ||
| 99 | sbi_platform_ipi_send(plat, hartid); | ||
| 100 | if (event != SBI_IPI_EVENT_SOFT) | ||
| 101 | @@ -80,7 +80,7 @@ void sbi_ipi_process(struct sbi_scratch *scratch) | ||
| 102 | sbi_platform_ipi_clear(plat, hartid); | ||
| 103 | |||
| 104 | do { | ||
| 105 | - ipi_type = scratch->ipi_type; | ||
| 106 | + ipi_type = sbi_ipi_data_ptr(scratch)->ipi_type; | ||
| 107 | rmb(); | ||
| 108 | ipi_event = __ffs(ipi_type); | ||
| 109 | switch (ipi_event) { | ||
| 110 | @@ -97,12 +97,14 @@ void sbi_ipi_process(struct sbi_scratch *scratch) | ||
| 111 | sbi_hart_hang(); | ||
| 112 | break; | ||
| 113 | }; | ||
| 114 | - ipi_type = atomic_raw_clear_bit(ipi_event, &scratch->ipi_type); | ||
| 115 | + ipi_type = atomic_raw_clear_bit(ipi_event, &sbi_ipi_data_ptr(scratch)->ipi_type); | ||
| 116 | } while(ipi_type > 0); | ||
| 117 | } | ||
| 118 | |||
| 119 | int sbi_ipi_init(struct sbi_scratch *scratch, bool cold_boot) | ||
| 120 | { | ||
| 121 | + sbi_ipi_data_ptr(scratch)->ipi_type = 0x00; | ||
| 122 | + | ||
| 123 | /* Enable software interrupts */ | ||
| 124 | csr_set(CSR_MIE, MIP_MSIP); | ||
| 125 | |||
| 126 | -- | ||
| 127 | 2.22.0 | ||
| 128 | |||
diff --git a/meta/recipes-bsp/opensbi/opensbi-payloads.inc b/meta/recipes-bsp/opensbi/opensbi-payloads.inc new file mode 100644 index 0000000000..7df168325a --- /dev/null +++ b/meta/recipes-bsp/opensbi/opensbi-payloads.inc | |||
| @@ -0,0 +1,37 @@ | |||
| 1 | def riscv_get_extra_oemake_image(d): | ||
| 2 | sbi_payload = d.getVar('RISCV_SBI_PAYLOAD') | ||
| 3 | deploy_dir = d.getVar('DEPLOY_DIR_IMAGE') | ||
| 4 | |||
| 5 | if sbi_payload is None: | ||
| 6 | return "" | ||
| 7 | |||
| 8 | if d.getVar('RISCV_SBI_PAYLOAD'): | ||
| 9 | return "FW_PAYLOAD_PATH=" + deploy_dir + "/" + sbi_payload | ||
| 10 | |||
| 11 | return "" | ||
| 12 | |||
| 13 | def riscv_get_extra_oemake_fdt(d): | ||
| 14 | sbi_fdt_payload = d.getVar('RISCV_SBI_FDT') | ||
| 15 | deploy_dir = d.getVar('DEPLOY_DIR_IMAGE') | ||
| 16 | |||
| 17 | if sbi_fdt_payload is None: | ||
| 18 | return "" | ||
| 19 | |||
| 20 | if d.getVar('RISCV_SBI_PAYLOAD'): | ||
| 21 | # This is internal to openSBI, not a full path | ||
| 22 | return "FW_PAYLOAD_FDT=" + sbi_fdt_payload | ||
| 23 | |||
| 24 | return "" | ||
| 25 | |||
| 26 | def riscv_get_do_compile_depends(d): | ||
| 27 | sbi_payload = d.getVar('RISCV_SBI_PAYLOAD') | ||
| 28 | |||
| 29 | if sbi_payload is None: | ||
| 30 | return "" | ||
| 31 | |||
| 32 | if 'linux' in sbi_payload or 'Image' in sbi_payload: | ||
| 33 | return "virtual/kernel:do_deploy" | ||
| 34 | if 'u-boot' in sbi_payload: | ||
| 35 | return "virtual/bootloader:do_deploy" | ||
| 36 | |||
| 37 | return "" | ||
diff --git a/meta/recipes-bsp/opensbi/opensbi_0.3.bb b/meta/recipes-bsp/opensbi/opensbi_0.3.bb new file mode 100644 index 0000000000..2b4fb97df2 --- /dev/null +++ b/meta/recipes-bsp/opensbi/opensbi_0.3.bb | |||
| @@ -0,0 +1,49 @@ | |||
| 1 | SUMMARY = "RISC-V Open Source Supervisor Binary Interface (OpenSBI)" | ||
| 2 | DESCRIPTION = "OpenSBI aims to provide an open-source and extensible implementation of the RISC-V SBI specification for a platform specific firmware (M-mode) and a general purpose OS, hypervisor or bootloader (S-mode or HS-mode). OpenSBI implementation can be easily extended by RISC-V platform or System-on-Chip vendors to fit a particular hadware configuration." | ||
| 3 | LICENSE = "BSD-2-Clause" | ||
| 4 | LIC_FILES_CHKSUM = "file://COPYING.BSD;md5=c36118b4f615f9da37635f2a7ac8ccaf" | ||
| 5 | DEPENDS += "dtc-native" | ||
| 6 | |||
| 7 | require opensbi-payloads.inc | ||
| 8 | |||
| 9 | inherit autotools-brokensep | ||
| 10 | |||
| 11 | SRCREV = "ca20ac0cd4c099006d4eea4d9ac7bd7b58e2ae0f" | ||
| 12 | SRC_URI = "git://github.com/riscv/opensbi.git \ | ||
| 13 | file://0001-Makefile-Don-t-specify-mabi-or-march.patch \ | ||
| 14 | file://0002-lib-Create-a-sbi_ipi_data-structure.patch \ | ||
| 15 | " | ||
| 16 | |||
| 17 | S = "${WORKDIR}/git" | ||
| 18 | |||
| 19 | SRC_URI[md5sum] = "621f38d8205ef5fb185e4055025e73df" | ||
| 20 | SRC_URI[sha256sum] = "07f18b73abf3b85aabe5bead19a923716c100d30eb58033459f39c3a224be300" | ||
| 21 | |||
| 22 | EXTRA_OEMAKE += "PLATFORM=${RISCV_SBI_PLAT} I=${D}" | ||
| 23 | # If RISCV_SBI_PAYLOAD is set then include it as a payload | ||
| 24 | EXTRA_OEMAKE_append = " ${@riscv_get_extra_oemake_image(d)} ${@riscv_get_extra_oemake_fdt(d)}" | ||
| 25 | |||
| 26 | # Required if specifying a custom payload | ||
| 27 | do_compile[depends] += "${@riscv_get_do_compile_depends(d)}" | ||
| 28 | |||
| 29 | do_install_append() { | ||
| 30 | # In the future these might be required as a dependency for other packages. | ||
| 31 | # At the moment just delete them to avoid warnings | ||
| 32 | rm -r ${D}/include | ||
| 33 | rm -r ${D}/platform/${RISCV_SBI_PLAT}/lib | ||
| 34 | rm -r ${D}/platform/${RISCV_SBI_PLAT}/firmware/payloads | ||
| 35 | } | ||
| 36 | |||
| 37 | do_deploy () { | ||
| 38 | install -d ${DEPLOY_DIR_IMAGE} | ||
| 39 | install -m 755 ${D}/platform/${RISCV_SBI_PLAT}/firmware/fw_payload.* ${DEPLOY_DIR_IMAGE}/ | ||
| 40 | install -m 755 ${D}/platform/${RISCV_SBI_PLAT}/firmware/fw_jump.* ${DEPLOY_DIR_IMAGE}/ | ||
| 41 | } | ||
| 42 | |||
| 43 | addtask deploy after do_install | ||
| 44 | |||
| 45 | FILES_${PN} += "/platform/${RISCV_SBI_PLAT}/firmware/fw_jump.*" | ||
| 46 | FILES_${PN} += "/platform/${RISCV_SBI_PLAT}/firmware/fw_payload.*" | ||
| 47 | |||
| 48 | COMPATIBLE_HOST = "(riscv64|riscv32).*" | ||
| 49 | INHIBIT_PACKAGE_STRIP = "1" | ||
