firmware: qcom_scm: Fix legacy convention SCM accessors
authorJonathan McDowell <noodles@earth.li>
Sat, 4 Jul 2020 17:23:34 +0000 (18:23 +0100)
committerBjorn Andersson <bjorn.andersson@linaro.org>
Mon, 13 Jul 2020 18:41:43 +0000 (11:41 -0700)
commitb88c28280c3f7097546db93824686db1e7dceee1
tree6e3c5f8710669544eea6fd412716511250415e98
parent943c8a80830b88b7b203fa3e2d755620d85129d1
firmware: qcom_scm: Fix legacy convention SCM accessors

The move to a combined driver for the QCOM SCM hardware changed the
io_writel and io_readl helpers to use non-atomic calls, despite the
commit message saying that atomic was a better option. This breaks these
helpers on hardware that uses the old legacy convention (access fails
with a -95 return code). Switch back to using the atomic calls.

Observed as a failure routing GPIO interrupts to the Apps processor on
an IPQ8064; fix is confirmed as correctly allowing the interrupts to be
routed and observed.

Reviewed-by: Elliot Berman <eberman@codeaurora.org>
Fixes: 57d3b816718c ("firmware: qcom_scm: Remove thin wrappers")
Cc: stable@vger.kernel.org
Signed-off-by: Jonathan McDowell <noodles@earth.li>
Link: https://lore.kernel.org/r/20200704172334.GA759@earth.li
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
drivers/firmware/qcom_scm.c