diff options
| author | Luca Vizzarro <Luca.Vizzarro@arm.com> | 2020-10-08 19:52:10 +0100 |
|---|---|---|
| committer | nicola-mazzucato-arm <42373140+nicola-mazzucato-arm@users.noreply.github.com> | 2020-10-22 14:39:59 +0100 |
| commit | 586092c41b113a8936e4acb90e5227f4e8f99d49 (patch) | |
| tree | a583ec42e5b8644c1545520b83fcb13a9c6324e1 | |
| parent | 274aa3f44a30601bc7da2d7451464734126c9521 (diff) | |
mod_scmi_sys_power: Fix PSCI access level
The POWER_STATE_NOTIFY command erroneously denies access to the PSCI
agent when there is no explicit declaration in the SCMI v2 spec that its
access is indeed denied. This commit removes this condition, therefore
enabling the PSCI aent to access this command.
Change-Id: Ie356def6b313091f3ba113bd806be21adeac9d09
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
| -rw-r--r-- | module/scmi_system_power/src/mod_scmi_system_power.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/module/scmi_system_power/src/mod_scmi_system_power.c b/module/scmi_system_power/src/mod_scmi_system_power.c index 7b13e3b7..9ecc85c4 100644 --- a/module/scmi_system_power/src/mod_scmi_system_power.c +++ b/module/scmi_system_power/src/mod_scmi_system_power.c @@ -441,7 +441,6 @@ static int scmi_sys_power_state_notify_handler(fwk_id_t service_id, const uint32_t *payload) { unsigned int agent_id; - enum scmi_agent_type agent_type; const struct scmi_sys_power_state_notify_a2p *parameters; struct scmi_sys_power_state_notify_p2a return_values = { .status = SCMI_GENERIC_ERROR, @@ -452,15 +451,6 @@ static int scmi_sys_power_state_notify_handler(fwk_id_t service_id, if (status != FWK_SUCCESS) goto exit; - status = scmi_sys_power_ctx.scmi_api->get_agent_type(agent_id, &agent_type); - if (status != FWK_SUCCESS) - goto exit; - - if (agent_type == SCMI_AGENT_TYPE_PSCI) { - return_values.status = SCMI_NOT_SUPPORTED; - goto exit; - } - parameters = (const struct scmi_sys_power_state_notify_a2p *)payload; if (parameters->flags & (~STATE_NOTIFY_FLAGS_MASK)) { |