starfive-tech/opensbi.git, 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

Makefile: fix build with binutils 2.38

2022-02-04T05:17:24+00:00

From version 2.38, binutils default to ISA spec version 20191213. This means that the csr read/write (csrr*/csrw*) instructions and fence.i instruction has separated from the `I` extension, become two standalone extensions: Zicsr and Zifencei. As the kernel uses those instruction, this causes the following build failure: CC lib/sbi/sbi_tlb.o <>/lib/sbi/sbi_tlb.c: Assembler messages: <>/lib/sbi/sbi_tlb.c:190: Error: unrecognized opcode `fence.i' make: *** [Makefile:431: <>/build/lib/sbi/sbi_tlb.o] Error 1 The fix is to specify those extensions explicitly in -march. However as older binutils version do not support this, we first need to detect that. Signed-off-by: Aurelien Jarno Reviewed-by: Bin Meng Tested-by: Alexandre Ghiti 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

platform: sifive_fu740: fix reset when watchdog is running

2022-01-21T16:14:57+00:00

When the watchdog is running the HiFive Unmatched board does not reboot properly and shuts down itself a few seconds after reboot, in the early stages of the u-boot loading. On a Linux kernel this happens when the da9063_wdt module is loaded. This does not happen if the module is unloaded before reboot or if the watchdog module is loaded with "stop_on_reboot=1". Fix that by stopping the watchdog before attempting to reset the board. This is done by zeroing the TWDSCALE field of CONTROL_D register, unless it was already set to 0. Reported-by: Tianon Gravi Signed-off-by: Aurelien Jarno Reviewed-by: Nikita Shubin Tested-by: Nikita Shubin Reviewed-by: Anup Patel

platform: sifive_fu740: do not use a global in da9063_reset/shutdown

2022-01-21T16:11:33+00:00

da9063_reset() and da9063_shutdown() take the chip address in argument (like similar functions), but in practice use the da9063 global struct instead. Fix that. Signed-off-by: Aurelien Jarno Reviewed-by: Nikita Shubin Reviewed-by: Anup Patel Reviewed-by: Xiang W

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