diff options
author | Arun Kumar Neelakantam <aneela@codeaurora.org> | 2018-10-03 07:38:20 -0400 |
---|---|---|
committer | Bjorn Andersson <bjorn.andersson@linaro.org> | 2018-10-03 20:04:32 -0400 |
commit | 928002a5e9dab2ddc1a0fe3e00739e89be30dc6b (patch) | |
tree | 65ad096c69f31845d38a574a2f7b49b91fbbdf61 /drivers/rpmsg | |
parent | 940c620d6af8fca7d115de40f19870fba415efac (diff) |
rpmsg: glink: smem: Support rx peak for size less than 4 bytes
The current rx peak function fails to read the data if size is
less than 4bytes.
Use memcpy_fromio to support data reads of size less than 4 bytes.
Cc: stable@vger.kernel.org
Fixes: f0beb4ba9b18 ("rpmsg: glink: Remove chunk size word align warning")
Signed-off-by: Arun Kumar Neelakantam <aneela@codeaurora.org>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Diffstat (limited to 'drivers/rpmsg')
-rw-r--r-- | drivers/rpmsg/qcom_glink_smem.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/drivers/rpmsg/qcom_glink_smem.c b/drivers/rpmsg/qcom_glink_smem.c index ab23da3d7131..64a5ce324c7f 100644 --- a/drivers/rpmsg/qcom_glink_smem.c +++ b/drivers/rpmsg/qcom_glink_smem.c | |||
@@ -89,15 +89,11 @@ static void glink_smem_rx_peak(struct qcom_glink_pipe *np, | |||
89 | tail -= pipe->native.length; | 89 | tail -= pipe->native.length; |
90 | 90 | ||
91 | len = min_t(size_t, count, pipe->native.length - tail); | 91 | len = min_t(size_t, count, pipe->native.length - tail); |
92 | if (len) { | 92 | if (len) |
93 | __ioread32_copy(data, pipe->fifo + tail, | 93 | memcpy_fromio(data, pipe->fifo + tail, len); |
94 | len / sizeof(u32)); | ||
95 | } | ||
96 | 94 | ||
97 | if (len != count) { | 95 | if (len != count) |
98 | __ioread32_copy(data + len, pipe->fifo, | 96 | memcpy_fromio(data + len, pipe->fifo, (count - len)); |
99 | (count - len) / sizeof(u32)); | ||
100 | } | ||
101 | } | 97 | } |
102 | 98 | ||
103 | static void glink_smem_rx_advance(struct qcom_glink_pipe *np, | 99 | static void glink_smem_rx_advance(struct qcom_glink_pipe *np, |