Re: [PATCH] rpmsg: glink: initialize ret to zero to ensure error status check is correct

2017-09-04 Thread Bjorn Andersson
On Sun 03 Sep 06:06 PDT 2017, Colin King wrote: > From: Colin Ian King > > The new switch cases for RPM_CMD_RX_DONE, RPM_CMD_RX_DONE_W_REUSE, > RPM_CMD_RX_INTENT_REQ_ACK, RPM_CMD_INTENT and RPM_CMD_RX_INTENT_REQ from > 4 recent commits are not setting ret and so a later non-zero check on ret > i

[PATCH] rpmsg: glink: initialize ret to zero to ensure error status check is correct

2017-09-03 Thread Colin King
From: Colin Ian King The new switch cases for RPM_CMD_RX_DONE, RPM_CMD_RX_DONE_W_REUSE, RPM_CMD_RX_INTENT_REQ_ACK, RPM_CMD_INTENT and RPM_CMD_RX_INTENT_REQ from 4 recent commits are not setting ret and so a later non-zero check on ret is testing on a garbage value in ret. Fix this by initializing