opensbi.git/include/sbi/sbi_platform.h, branch v0.8

lib: sbi: Improve get_feature_str() implementation and usage

2020-05-19T03:49:34Z

We do following improvements for get_feature_str(): 1. We should return "none" from get_feature_str() no features available instead of sbi_boot_prints() explicitly handling failure. 2. We don't need to return failure (just like misa_xlen()) because we are returning "none" for no features and we are truncating output when space is not available. 3. Based on 1 and 2, the sbi_boot_prints() can be further simplified. 4. No need for two char[] in sbi_boot_prints() Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: Add platform features in boot time print

2020-05-10T04:54:08Z

We have now clear distinction between platform and hart features. Modify the boot print messages to print platform features in a string format. In the process, this patch moved relatively larger functions to its own file from platform.h header file. Signed-off-by: Atish Patra Tested-by: Jonathan Balkind Reviewed-by: Anup Patel

platform: Move platform features to hart

2020-05-10T04:35:41Z

PMP & performance counters belong to a hart rather than a platform. In addition to that, these features enable reading/writing from a particular csr. Thus, they can be detected and set at runtime rather than compile time. Move PMP/MCOUNTEREN/SCOUNTEREN features to hart and detect them at runtime. Signed-off-by: Atish Patra Tested-by: Jonathan Balkind Reviewed-by: Anup Patel

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

include: sbi_platform: Remove priv parameter from hart_start() callback

2020-03-28T08:02:26Z

The priv parameter in hart_start() platform callback is redundant hence we remove it. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

include: sbi_platform: Introduce HART index to HART id table

2020-03-19T04:00:01Z

A platform can have discontinuous and/or sparse HART ids so we cannot always assume a set of HARTs with continuous HART ids. This patch adds support for discontinuous and sparse HART ids by introducing HART index to HART id table. This table has platform hart_count entries and it maps HART index to HART id. The HART index to HART id table has only two restrictions: 1. HART index < sbi_platform hart_count 2. HART id < SBI_HARTMASK_MAX_BITS Example1: Let's say we have a platform with 2 HART ids 11 and 22, for such a a platform: hart_count = 2 hart_index2id[0] = 11 hart_index2id[1] = 22 Example2: Let's say we have a platform with 5 HARTs ids 0, 1, 2, 3, and 4 but out of these HART with id 0 is not usable so for such a platform: hart_count = 5 hart_index2id[0] = -1U hart_index2id[1] = 1 hart_index2id[2] = 2 hart_index2id[3] = 3 hart_index2id[4] = 4 OR hart_count = 4 hart_index2id[0] = 1 hart_index2id[1] = 2 hart_index2id[2] = 3 hart_index2id[3] = 4 With HART index to HART id table in place, the hart_disabled() callback is now redundant so we remove it as well. Signed-off-by: Anup Patel Reviewed-by: Bin Meng Reviewed-by: Atish Patra

include: sbi_platform: Improve sbi_platform_hart_disabled() API

2020-03-19T03:45:29Z

The sbi_platform_hart_disabled() should return TRUE for HART id greater than platform hart count. Signed-off-by: Anup Patel 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

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