drivers: qcom: rpmh: add nord support - #65
Conversation
Adding Nord support, following target specific RMPh configuration structure. Unit tested on Nord device.
IDineshChoudhary
left a comment
IDineshChoudhary
left a comment
There was a problem hiding this comment.
you can revisit this once lemans SW refactoring goes through.
| CFG_TEE_RAM_VA_SIZE ?= 0x00200000 | ||
| CFG_TA_RAM_VA_SIZE ?= 0x07B80000 | ||
|
|
||
| ifneq ($(CFG_INSECURE),y) |
There was a problem hiding this comment.
QFPROM in OTPEE isn't applicable for Nord, you can make it default NO.
Or you can clean up this part completely from you pull request onus is on platform or client driver not us to update the same unless it's causing issues in rpmh/cmd-db driver enablement
| CFG_QCOM_QFPROM_FUSEPROV ?= y | ||
| endif | ||
|
|
||
| ifeq ($(CFG_QCOM_QFPROM_FUSEPROV),y) |
There was a problem hiding this comment.
rpmh/cmd_db initialization shouldn't be dependent on qfprom, it should be independent cleanup top level check
| $(call force,CFG_QCOM_RPMH_CLIENT,y) | ||
| endif | ||
|
|
||
| ifneq ($(filter y,$(CFG_QCOM_QFPROM_FUSEPROV) $(CFG_QCOM_FUSE_PTA)),) |
|
|
||
| #define GICD_BASE UL(0x17000000) | ||
| #define GICR_BASE UL(0x17080000) | ||
| #define AOP_CMD_DB_BASE UL(0x87148000) |
There was a problem hiding this comment.
correct indent throughout the file
| #define DRAM1_SIZE ULL(0x780000000) | ||
| #define DRAM2_BASE ULL(0x8800000000) | ||
| #define DRAM2_SIZE ULL(0x3800000000) | ||
| #define DRAM0_SIZE UL(0x80000000) |
There was a problem hiding this comment.
these aren't expected to be used in rpmh/cmd-db drivers, restore this change.
|
|
||
| static const struct drv_config_data optee_config_data = { | ||
| .drvs_count = 1, | ||
| .init_clks_count = 0, |
There was a problem hiding this comment.
in parent driver these struct members may be used but they aren't expected to - review and change the design at right place.
| @@ -0,0 +1,57 @@ | |||
| /* SPDX-License-Identifier: BSD-2-Clause */ | |||
| RPMH_TCS_MAX = 4 /* Wake TCS end, Max TCS count */ | ||
| }; | ||
|
|
||
| #define RPMH_MAX_CMDS_PER_TCS 16 |
| } | ||
|
|
||
| dict_addr = base + AOP_MSG_RAM_SIZE - MSG_RAM_SECTION_SIZE; | ||
| dict_addr = base + 15 * MSG_RAM_SECTION_SIZE; |
2 participants
Adding Nord support, following target specific RMPh configuration structure.
Includes MSG_RAM_SECTION changes, and other target specific data changes.
Unit tested on Nord device.
Stacked on #55.