opensbi.git/platform/template/platform.c, branch v0.8

lib: utils: Allow CLINT functions to be used for multiple CLINTs

2020-05-23T05:06:43Z

We extend CLINT cold init function to have a "struct clint_data *" parameter pointing to CLINT details. This allows platforms to use CLINT functions for multiple CLINT instances. When multiple CLINTs are present, the platform can also provide one of the CLINT as reference CLINT for other CLINTs. This will help CLINTs to sync their time value with reference CLINT using a time_delta computed in warm init function. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: utils: Allow PLIC functions to be used for multiple PLICs

2020-05-23T05:06:29Z

We extend all PLIC functions to have a "struct plic_data *" parameter pointing to PLIC details. This allows platforms to use these functions for multiple PLIC instances. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: utils: Remove redundant parameters from PLIC init functions

2020-05-23T05:06:26Z

The "target_hart" and "hart_count" parameters of PLIC cold and warm init functions are only used for sanity checks and not required in PLIC initialization. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

include: sbi_platform: Combine reboot and shutdown into one callback

2020-04-27T09:05:29Z

We can achieve shutdown, cold reboot, and warm reboot using just one sbi_platform callback so we combine system_reboot() and system_shutdown() callbacks into one system_reset() callback. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: No need to provide default PMP region using platform callbacks

2020-04-27T08:27:27Z

The default (usually last) PMP region provides S-mode access to complete memory range not covered by other PMP regions. Currently, the default PMP region is described as platform specific PMP region which is not appropriate because all platforms need it and default PMP region should be part of generic library. Signed-off-by: Anup Patel Reviewed-by: Atish Patra Reviewed-by: Bin Meng

platform: Use one unified per-HART stack size macro for all platforms

2020-03-14T03:44:00Z

As of today all platforms use 8KB of per-HART stack hence there is no need for each platform to define its own macro or use the magic number. Create one macro for all platforms. Platform still can use its own version if needed. Signed-off-by: Bin Meng Reviewed-by: Anup Patel

platform: Set per-HART stack size to 8KB in the template platform codes

2020-03-14T03:43:20Z

The template platform codes should set per-HART stack size to 8KB to avoid possible mistakes of future platform ports. Signed-off-by: Bin Meng Reviewed-by: Anup Patel

include: Make sbi_current_hartid() as macro in riscv_asm.h

2020-03-13T06:52:02Z

The sbi_current_hartid() being a regular function is quite expensive because for callers it is a function call instead of a direct CSR read. This patch converts sbi_current_hartid() into a macro in riscv_asm.h. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

include: Remove disabled_hart_mask from sbi_platform

2020-03-11T10:00:12Z

The disabled_hard_mask in sbi_platform is only 64bits wide so we cannot disable a HART with HARTID > 63. To tackle this, we remove disabled_hart_mask and replace it with hart_disabled() platform callback. Signed-off-by: Anup Patel Reviewed-by: Bin Meng Reviewed-by: Atish Patra

platform: Add an platform ops to return platform specific tlb flush limit

2020-02-05T05:07:11Z

If a platform requires to perform a tlb full flush, they should set the tlb_range_flush_limit value to zero. However, corresponding platform API ignore the value and continue to return the default value. Add a platform ops to retrieve platform specific tlb range flush limit. The platform variable becomes redundant in presence of the platform ops. Take this opportunity to remove the variable as well. The default is still set to smallest page size in RISC-V (4KB), as there is no way to figure out a best value for all platforms. Individual platform should set it to the optimal value for their platform. Signed-off-by: Atish Patra Reviewed-by: Anup Patel