Skip to content

Commit

Permalink
fix(drivers/scmi-msg): base: fix protocol list response size
Browse files Browse the repository at this point in the history
Corrects the size of the SCMI response payload when querying the list
of the supported protocol. This response payload size depends on the
number of protocols enumerated by the response.

Change-Id: Ib01eb5cec6c6656dfd7d88ccdd5a720c1deee7a3
Reported-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/tf-a/+/252335
Tested-by: Yann GAUTIER <yann.gautier@foss.st.com>
Reviewed-by: Yann GAUTIER <yann.gautier@foss.st.com>
  • Loading branch information
etienne-lms authored and spasdeloup committed May 23, 2022
1 parent 5b3acfe commit c6da179
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/scmi-msg/base.c
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
// SPDX-License-Identifier: BSD-3-Clause
/*
* Copyright (c) 2015-2019, Arm Limited and Contributors. All rights reserved.
* Copyright (c) 2019-2020, Linaro Limited
* Copyright (c) 2019-2022, Linaro Limited
*/
#include <assert.h>
#include <string.h>
Expand Down Expand Up @@ -165,7 +165,7 @@ static void discover_list_protocols(struct scmi_msg *msg)
memcpy(outargs, &p2a, sizeof(p2a));
memcpy(outargs + sizeof(p2a), list + a2p->skip, count);

scmi_write_response(msg, outargs, sizeof(outargs));
scmi_write_response(msg, outargs, sizeof(p2a) + round_up(count, sizeof(uint32_t)));
}

static const scmi_msg_handler_t scmi_base_handler_table[] = {
Expand Down

0 comments on commit c6da179

Please sign in to comment.