drivers/soc/qcom/smd-rpm.c:177:47: sparse: sparse: incorrect type in argument 2 (different address spaces)

From: kernel test robot
Date: Sat Jul 04 2020 - 23:14:18 EST

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 35e884f89df4c48566d745dc5a97a0d058d04263
commit: 670d0a4b10704667765f7d18f7592993d02783aa sparse: use identifiers to define address spaces
date: 2 weeks ago
config: ia64-randconfig-s031-20200705 (attached as .config)
compiler: ia64-linux-gcc (GCC) 9.3.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# apt-get install sparse
# sparse version: v0.6.2-3-gfa153962-dirty
git checkout 670d0a4b10704667765f7d18f7592993d02783aa
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=ia64

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>


sparse warnings: (new ones prefixed by >>)

>> drivers/soc/qcom/smd-rpm.c:177:47: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void const volatile [noderef] __iomem *src @@ got unsigned char const * @@
>> drivers/soc/qcom/smd-rpm.c:177:47: sparse: expected void const volatile [noderef] __iomem *src
drivers/soc/qcom/smd-rpm.c:177:47: sparse: got unsigned char const *

vim +177 drivers/soc/qcom/smd-rpm.c

936f14cf4e6716 Bjorn Andersson 2015-07-27 146
5052de8deff561 Bjorn Andersson 2017-03-27 147 static int qcom_smd_rpm_callback(struct rpmsg_device *rpdev,
5052de8deff561 Bjorn Andersson 2017-03-27 148 void *data,
5052de8deff561 Bjorn Andersson 2017-03-27 149 int count,
5052de8deff561 Bjorn Andersson 2017-03-27 150 void *priv,
5052de8deff561 Bjorn Andersson 2017-03-27 151 u32 addr)
936f14cf4e6716 Bjorn Andersson 2015-07-27 152 {
936f14cf4e6716 Bjorn Andersson 2015-07-27 153 const struct qcom_rpm_header *hdr = data;
30b7ea5eda0afb Stephen Boyd 2015-09-02 154 size_t hdr_length = le32_to_cpu(hdr->length);
936f14cf4e6716 Bjorn Andersson 2015-07-27 155 const struct qcom_rpm_message *msg;
5052de8deff561 Bjorn Andersson 2017-03-27 156 struct qcom_smd_rpm *rpm = dev_get_drvdata(&rpdev->dev);
936f14cf4e6716 Bjorn Andersson 2015-07-27 157 const u8 *buf = data + sizeof(struct qcom_rpm_header);
30b7ea5eda0afb Stephen Boyd 2015-09-02 158 const u8 *end = buf + hdr_length;
936f14cf4e6716 Bjorn Andersson 2015-07-27 159 char msgbuf[32];
936f14cf4e6716 Bjorn Andersson 2015-07-27 160 int status = 0;
30b7ea5eda0afb Stephen Boyd 2015-09-02 161 u32 len, msg_length;
936f14cf4e6716 Bjorn Andersson 2015-07-27 162
30b7ea5eda0afb Stephen Boyd 2015-09-02 163 if (le32_to_cpu(hdr->service_type) != RPM_SERVICE_TYPE_REQUEST ||
30b7ea5eda0afb Stephen Boyd 2015-09-02 164 hdr_length < sizeof(struct qcom_rpm_message)) {
b853cb9628bfbc Bjorn Andersson 2016-03-28 165 dev_err(rpm->dev, "invalid request\n");
936f14cf4e6716 Bjorn Andersson 2015-07-27 166 return 0;
936f14cf4e6716 Bjorn Andersson 2015-07-27 167 }
936f14cf4e6716 Bjorn Andersson 2015-07-27 168
936f14cf4e6716 Bjorn Andersson 2015-07-27 169 while (buf < end) {
936f14cf4e6716 Bjorn Andersson 2015-07-27 170 msg = (struct qcom_rpm_message *)buf;
30b7ea5eda0afb Stephen Boyd 2015-09-02 171 msg_length = le32_to_cpu(msg->length);
30b7ea5eda0afb Stephen Boyd 2015-09-02 172 switch (le32_to_cpu(msg->msg_type)) {
936f14cf4e6716 Bjorn Andersson 2015-07-27 173 case RPM_MSG_TYPE_MSG_ID:
936f14cf4e6716 Bjorn Andersson 2015-07-27 174 break;
936f14cf4e6716 Bjorn Andersson 2015-07-27 175 case RPM_MSG_TYPE_ERR:
30b7ea5eda0afb Stephen Boyd 2015-09-02 176 len = min_t(u32, ALIGN(msg_length, 4), sizeof(msgbuf));
936f14cf4e6716 Bjorn Andersson 2015-07-27 @177 memcpy_fromio(msgbuf, msg->message, len);
936f14cf4e6716 Bjorn Andersson 2015-07-27 178 msgbuf[len - 1] = 0;
936f14cf4e6716 Bjorn Andersson 2015-07-27 179
936f14cf4e6716 Bjorn Andersson 2015-07-27 180 if (!strcmp(msgbuf, "resource does not exist"))
936f14cf4e6716 Bjorn Andersson 2015-07-27 181 status = -ENXIO;
936f14cf4e6716 Bjorn Andersson 2015-07-27 182 else
936f14cf4e6716 Bjorn Andersson 2015-07-27 183 status = -EINVAL;
936f14cf4e6716 Bjorn Andersson 2015-07-27 184 break;
936f14cf4e6716 Bjorn Andersson 2015-07-27 185 }
936f14cf4e6716 Bjorn Andersson 2015-07-27 186
30b7ea5eda0afb Stephen Boyd 2015-09-02 187 buf = PTR_ALIGN(buf + 2 * sizeof(u32) + msg_length, 4);
936f14cf4e6716 Bjorn Andersson 2015-07-27 188 }
936f14cf4e6716 Bjorn Andersson 2015-07-27 189
936f14cf4e6716 Bjorn Andersson 2015-07-27 190 rpm->ack_status = status;
936f14cf4e6716 Bjorn Andersson 2015-07-27 191 complete(&rpm->ack);
936f14cf4e6716 Bjorn Andersson 2015-07-27 192 return 0;
936f14cf4e6716 Bjorn Andersson 2015-07-27 193 }
936f14cf4e6716 Bjorn Andersson 2015-07-27 194

:::::: The code at line 177 was first introduced by commit
:::::: 936f14cf4e67168fcd37f10cebf5a475f490fb6e soc: qcom: Driver for the Qualcomm RPM over SMD

:::::: TO: Bjorn Andersson <bjorn.andersson@xxxxxxxxxxxxxx>
:::::: CC: Andy Gross <agross@xxxxxxxxxxxxxx>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip