On Wed 27 Jun 18:19 PDT 2018, Chris Lew wrote: > When wrapping around the FIFO, the remote expects the tail pointer to > be reset to 0 on the edge case where the tail equals the FIFO length. >
Applied Thanks, Bjorn > Signed-off-by: Chris Lew <[email protected]> > --- > drivers/rpmsg/qcom_glink_smem.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/rpmsg/qcom_glink_smem.c b/drivers/rpmsg/qcom_glink_smem.c > index 2b5cf2790954..7a276be598d6 100644 > --- a/drivers/rpmsg/qcom_glink_smem.c > +++ b/drivers/rpmsg/qcom_glink_smem.c > @@ -109,7 +109,7 @@ static void glink_smem_rx_advance(struct qcom_glink_pipe > *np, > tail = le32_to_cpu(*pipe->tail); > > tail += count; > - if (tail > pipe->native.length) > + if (tail >= pipe->native.length) > tail -= pipe->native.length; > > *pipe->tail = cpu_to_le32(tail); > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project >
