starfive-tech/opensbi.git/lib, branch SDK_v2.1.0

lib: utils/serial: Round UART8250 baud rate divisor to nearest integer

2022-02-04T05:46:29+00:00

Previously, it was rounded down and that gives suboptimal results when non-standard clock sources or baud rates are used. Signed-off-by: Jakub Luzny Reviewed-by: Anup Patel

lib: fix compilation when strings.h is included

2022-02-04T05:34:19+00:00

In a systems that provide strings.h and it is included together with sbi_bitops.h the compilation error appears. The ffs() and fls() are provided by strings.h Signed-off-by: Petro Karashchenko Reviewed-by: Anup Patel

lib: sbi: Disable interrupt and inhibit counting in M-mode during init

2022-02-04T04:40:28+00:00

Currently, the mhpmevent CSRs are untouched during hart init during cold/warm boot. Ideally, we should clear out all the bits except overflow and MINH bit. That is required to disable overflow interrupt and inhibit counting in M-mode to avoid any spurious interrupts before perf start. Signed-off-by: Atish Patra Reviewed-by: Anup Patel

lib: sbi: Disable interrupt during config matching

2022-02-04T04:38:57+00:00

PMU overflow interrupt should be disabled durinig initial configuration of counters. They should be enabled while starting counters. Fixes: 730f01bb41a6 ("lib: sbi: Support sscofpmf extension in OpenSBI") Signed-off-by: Atish Patra Reviewed-by: Anup Patel

lib: sbi: Map only the counters enabled in hardware

2022-02-04T04:37:07+00:00

The counter mapping in DT may be incorrect if all the counters specified in the mapping are actually not physically present in the hardware. OpenSBI should only keep a mapping of counters enabled in hardware and defined in DT. This assume that all the programmable hpmcounters are consecutive as it doesn't make sense to build a system with sparse hpmcounters. Signed-off-by: Atish Patra Reviewed-by: Anup Patel

lib: fix pointer of type 'void *' used in arithmetic

2022-01-21T16:28:12+00:00

Using "void *" in arithmetic causes errors with strict compiler settings: "error: pointer of type 'void *' used in arithmetic [-Werror=pointer-arith]" Avoid these by calculating on "char *" where 1-byte data size is assumed. Signed-off-by: Jukka Laitinen Reviewed-by: Dong Du Reviewed-by: Xiang W Reviewed-by: Anup Patel

lib: utils/sys: Extend HTIF library to allow custom base address

2022-01-11T12:47:49+00:00

Some of RISC-V emulators provide HTIF at fixed base address so for such emulators users have to hard-code HTIF base address in the linker script. To address this problem, we let users optionally provide fixed HTIF base address via platform support (or device tree). Signed-off-by: Anup Patel Reviewed-by: Dong Du Reviewed-by: Atish Patra

lib: utils/ipi: Fix size check in aclint_mswi_cold_init()

2022-01-11T12:47:07+00:00

Currently, the ACLINT MSWI size check is forcing size to be at least 0x4000. This is inappropriate check because most systems will never utilize full 16KB for a single ACLINT MSWI device so instead we should check that ACLINT MSWI size is enough for on the associated HARTs. Signed-off-by: Anup Patel Reviewed-by: Dong Du Reviewed-by: Alistair Francis

firmware: Move memcpy/memset mapping to fw_base.S

2021-12-23T12:16:09+00:00

Some of the external firmwares using OpenSBI as library are facing issues with the weak memcpy() and memset() aliases in libsbi.a so we move these to fw_base.S. This way mapping of implicit memcpy() or memset() calls to sbi_memcpy() or sbi_memset() will only be done for OpenSBI firmwares. (Refer, https://github.com/riscv-software-src/opensbi/issues/234) In addition, we also add memmove() and memcmp() mappings in fw_base.S because as-per the GCC documentation the freestanding environment must provide memcpy(), memmove(), memset(), and memcmp(). Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: sbi: simplify pmp_set(), pmp_get()

2021-12-22T03:35:05+00:00

pmpcfg_csr and pmpcfg_shift are only negative for an unexpected value of __riscv_xlen. We can immediately return in this case. Signed-off-by: Heinrich Schuchardt Reviewed-by: Dong Du Reviewed-by: Anup Patel