From: Jean Thomas <jean.thomas@wifirst.fr>
Date: Tue, 5 Dec 2023 10:12:37 +0000 (+0100)
Subject: uqmi: improve response detection
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=dfa612e421cbe4c940e08aa467718fae81fd53db;p=project%2Fuqmi.git

uqmi: improve response detection

Create a helper which checks the flags depending on the service
indicated in the received message, as the response flag is not the same
for the CTL service and the other services. This avoids considering
a CTL indication as a valid response.

Also use a mask in order to check the flags, as they are a bitmap.

Signed-off-by: Jean Thomas <jean.thomas@wifirst.fr>
---

diff --git a/dev.c b/dev.c
index bd10207..2d1597c 100644
--- a/dev.c
+++ b/dev.c
@@ -62,6 +62,20 @@ qmi_get_service_idx(QmiService svc)
 	return -1;
 }
 
+static bool qmi_message_is_response(struct qmi_msg *msg)
+{
+	if (msg->qmux.service == QMI_SERVICE_CTL) {
+		if (msg->flags & QMI_CTL_FLAG_RESPONSE)
+			return true;
+	}
+	else {
+		if (msg->flags & QMI_SERVICE_FLAG_RESPONSE)
+			return true;
+	}
+
+	return false;
+}
+
 static void __qmi_request_complete(struct qmi_dev *qmi, struct qmi_request *req, struct qmi_msg *msg)
 {
 	void *tlv_buf;
@@ -96,7 +110,7 @@ static void qmi_process_msg(struct qmi_dev *qmi, struct qmi_msg *msg)
 	struct qmi_request *req;
 	uint16_t tid;
 
-	if (msg->flags != QMI_CTL_FLAG_RESPONSE && msg->flags != QMI_SERVICE_FLAG_RESPONSE)
+	if (!qmi_message_is_response(msg))
 		return;
 
 	if (msg->qmux.service == QMI_SERVICE_CTL)