[PATCH SCP] module/scmi_pin_control: allow variable length responses

[Dan Carpenter <dan.carpenter@linaro.org>]

The PINCTRL_LIST_ASSOCIATIONS and PINCTRL_SETTINGS_GET commands can
return variable length responses.  Originally, they were hard coded as
returning only one response.  Change the length to -1 and change
scmi_message_validation() to skip the length checking for lengths when
it's -1.

Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
---
 module/scmi/src/mod_scmi.c                         | 5 ++++-
 module/scmi_pin_control/src/mod_scmi_pin_control.c | 6 ++----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/module/scmi/src/mod_scmi.c b/module/scmi/src/mod_scmi.c
index dfaf441fbae5..da4ecc09e13b 100644
--- a/module/scmi/src/mod_scmi.c
+++ b/module/scmi/src/mod_scmi.c
@@ -434,8 +434,11 @@ static int scmi_message_validation(
         return (int)SCMI_NOT_FOUND;
     }
 
-    if (payload_size != payload_size_table[message_id]) {
+    if (payload_size_table[message_id] != -1 &&
+        payload_size != payload_size_table[message_id]) {
         /* Incorrect payload size or message is not supported */
+        FWK_LOG_ERR("invalid message size: message_id=%ld payload_size=%ld vs %ld\n",
+		    message_id, payload_size, payload_size_table[message_id]);
         return (int)SCMI_PROTOCOL_ERROR;
     }
 
diff --git a/module/scmi_pin_control/src/mod_scmi_pin_control.c b/module/scmi_pin_control/src/mod_scmi_pin_control.c
index 058c89f9512f..2a1095dba9b4 100644
--- a/module/scmi_pin_control/src/mod_scmi_pin_control.c
+++ b/module/scmi_pin_control/src/mod_scmi_pin_control.c
@@ -99,12 +99,10 @@ static const size_t payload_size_table[MOD_SCMI_PIN_CONTROL_COMMAND_COUNT] = {
         (unsigned int)sizeof(struct scmi_protocol_message_attributes_a2p),
     [MOD_SCMI_PIN_CONTROL_ATTRIBUTES] =
         (unsigned int)sizeof(struct scmi_pin_control_attributes_a2p),
-    [MOD_SCMI_PIN_CONTROL_LIST_ASSOCIATIONS] =
-        (unsigned int)sizeof(struct scmi_pin_control_list_associations_a2p),
-    [MOD_SCMI_PIN_CONTROL_SETTINGS_GET] =
-        (unsigned int)sizeof(struct scmi_pin_control_settings_get_a2p),
     [MOD_SCMI_PIN_CONTROL_SETTINGS_CONFIGURE] =
         (unsigned int)sizeof(struct scmi_pin_control_settings_configure_a2p),
+    [MOD_SCMI_PIN_CONTROL_LIST_ASSOCIATIONS] = -1,
+    [MOD_SCMI_PIN_CONTROL_SETTINGS_GET] = -1,
     [MOD_SCMI_PIN_CONTROL_REQUEST] =
         (unsigned int)sizeof(struct scmi_pin_control_request_a2p),
     [MOD_SCMI_PIN_CONTROL_RELEASE] =
-- 
2.47.2