Commit f20415af authored Nov 07, 2017 by Kyle Yan

soc: qcom: service-locator: Fix error message prints for receiving messages



ENOMSG is not a real failure return code for qmi_recv_msg. Instead ENOMSG
signifies that we have finished reading the entirety of the message buffer.
Update the error condition to print errors only when any error message
other than ENOMSG is returned.

Change-Id: Id6b42df182cb02f2cbffaae9698363b7dafce4e4
Signed-off-by: Kyle Yan <kyan@codeaurora.org>

parent 0364b310

drivers/soc/qcom/service-locator.c

+4 −5

Original line number	Diff line number	Diff line
		@@ -149,11 +149,10 @@ static void service_locator_recv_msg(struct work_struct *work)

		do {
		pr_debug("Notified about a Receive event\n");
		ret = qmi_recv_msg(service_locator.clnt_handle);
		if (ret < 0)
		pr_err("Error receiving message rc:%d. Retrying...\n",
		ret);
		} while (ret == 0);
		} while ((ret = qmi_recv_msg(service_locator.clnt_handle)) == 0);

		if (ret != -ENOMSG)
		pr_err("Error receiving message rc:%d\n", ret);

		}