starfive-tech/opensbi.git/lib, branch StarFive_JH7100_generic

lib: utils: missing initialization in thead_reset_init

2021-06-11T13:50:16+00:00

If property csr-copy does not exist, fdt_getprop() will return NULL and cnt will have a random value from the stack. Call clone_csrs() only if cnt is initialized to a non-zero value. Fixes: 49e422c5ad7a ("lib: utils: reset: Add T-HEAD sample platform reset driver") Signed-off-by: Heinrich Schuchardt Reviewed-by: Anup Patel

lib: sbi_scratch: zero out scratch memory on all harts

2021-06-11T08:29:48+00:00

In sbi_scratch_init() we determine the last hart. The index of the last hart cannot exceed SBI_HARTMASK_MAX_BITS - 1. We should not initialize last_hartid_having_scratch to a higher number to avoid buffer overflows when using this value before calling sbi_scratch_init(). When allocating scratch memory in sbi_scratch_alloc_offset() we zero out the allocated memory for all harts except for the last one. We should not skip the last hart. Signed-off-by: Heinrich Schuchardt Reviewed-by: Anup Patel

lib: sbi_scratch: remove owner from sbi_scratch_alloc_offset

2021-06-02T11:37:26+00:00

The parameter owner of function sbi_scratch_alloc_offset() is never used. The scratch memory is small. We should not use it for debug information in future. Hence eliminate the parameter. Signed-off-by: Heinrich Schuchardt Reviewed-by: Xiang W Reviewed-by: Anup Patel

lib: utils: consider ':' in stdout-path

2021-06-02T11:36:17+00:00

The value of the /chosen/stdout-path devicetree property is used to determine the UART used by openSBI. According to the devicetree specification the value may contain a hyphen, e.g. chosen { stdout-path = "/serial@f00:115200"; }; If the character ':' is present, it terminates the path of the device. Signed-off-by: Heinrich Schuchardt Reviewed-by: Atish Patra

lib: utils: Try other FDT drivers when we see SBI_ENODEV

2021-05-24T10:19:21+00:00

We should try other FDT drivers when we see SBI_ENODEV returned by cold_init() of FDT driver. Signed-off-by: Anup Patel Reviewed-by: Atish Patra Reviewed-by: Xiang W

lib: Check region base for merging in sbi_domain_root_add_memregion()

2021-05-24T10:17:41+00:00

We can merge region B onto region A only if base of region A is aligned to region A order + 1. Signed-off-by: Anup Patel Reviewed-by: Atish Patra Reviewed-by: Xiang W

lib: utils/fdt: Replace strcmp with strncmp

2021-05-19T12:05:38+00:00

Use strncmp() instead of strcmp() in __fixup_find_domain_offset() so that it compiles fine when linking with external firmware (such as EDK2). Signed-off-by: Daniel Schaefer Reviewed-by: Xiang W Reviewed-by: Abner Chang Reviewed-by: Anup Patel

lib: utils/fdt: Don't use sbi_string functions

2021-05-19T12:04:06+00:00

When SBI is built by external firmware, we need to use their functions, defined in libfdt_env.h. Just like 2cfd2fc9048806353298a1b967abf985901e36e8 Signed-off-by: Daniel Schaefer Reviewed-by: Abner Chang Reviewed-by: Xiang W Reviewed-by: Anup Patel

lib: utils: Add strncpy macro to libfdt_env.h

2021-05-19T12:03:33+00:00

we want to use sbi_strncpy as strncpy in the OpenSBI implementation for libfdt. Just like 2845d2d2cf4fb74a89452ba223995aa4a118c07e Signed-off-by: Daniel Schaefer Reviewed-by: Abner Chang Reviewed-by: Xiang W Reviewed-by: Anup Patel

lib: utils/serial: Support Synopsys DesignWare APB UART

2021-05-19T07:33:19+00:00

Synopsys DesignWare APB UART is seen on the StarFive JH7100 SoC. Its programming interface is compatible with the existing 8250 UART driver. Simply add its compatible string to the driver makes it work with the StarFive JH7100 SoC on a BeagleV board. With this patch, the generic platform firmware can be used out of the box on the BeagleV board. Signed-off-by: Bin Meng Reviewed-by: Anup Patel