evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

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

Browse source 
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>
This commit is contained in:
Kyle Yan 2017-11-07 22:14:39 -08:00 • committed by Gerrit - the friendly Code Review server
parent f2170a2917
commit 6c06d8ddf7
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
drivers/soc/qcom/service-locator.c
View file

@ -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);
}