From: kbuild test robot <lkp@intel.com>
To: Bjorn Andersson <bjorn.andersson@linaro.org>,
Andy Gross <agross@kernel.org>, Ohad Ben-Cohen <ohad@wizery.com>,
Rob Herring <robh+dt@kernel.org>
Cc: kbuild-all@lists.01.org, linux-arm-msm@vger.kernel.org,
linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v5 3/5] remoteproc: qcom: Update PIL relocation info on load
Date: Fri, 15 May 2020 02:48:35 +0800 [thread overview]
Message-ID: <202005150200.DX18JJaQ%lkp@intel.com> (raw)
In-Reply-To: <20200513055641.1413100-4-bjorn.andersson@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 2336 bytes --]
Hi Bjorn,
I love your patch! Yet something to improve:
[auto build test ERROR on robh/for-next]
[also build test ERROR on linus/master v5.7-rc5 next-20200514]
[cannot apply to agross-msm/qcom/for-next remoteproc/for-next rpmsg/for-next hwspinlock/for-next]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Bjorn-Andersson/remoteproc-qcom-PIL-info-support/20200514-161851
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arm64-allyesconfig (attached as .config)
compiler: aarch64-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
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day GCC_VERSION=9.3.0 make.cross ARCH=arm64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
drivers/remoteproc/qcom_q6v5_wcss.c: In function 'q6v5_wcss_load':
>> drivers/remoteproc/qcom_q6v5_wcss.c:433:2: error: implicit declaration of function 'qcom_pil_info_store' [-Werror=implicit-function-declaration]
433 | qcom_pil_info_store("wcnss", wcss->mem_reloc, wcss->mem_size);
| ^~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/qcom_pil_info_store +433 drivers/remoteproc/qcom_q6v5_wcss.c
420
421 static int q6v5_wcss_load(struct rproc *rproc, const struct firmware *fw)
422 {
423 struct q6v5_wcss *wcss = rproc->priv;
424 int ret;
425
426 ret = qcom_mdt_load_no_init(wcss->dev, fw, rproc->firmware,
427 0, wcss->mem_region, wcss->mem_phys,
428 wcss->mem_size, &wcss->mem_reloc);
429 if (ret)
430 return ret;
431
432 /* Failures only affect post mortem debugging, so ignore return value */
> 433 qcom_pil_info_store("wcnss", wcss->mem_reloc, wcss->mem_size);
434
435 return ret;
436 }
437
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 71781 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v5 3/5] remoteproc: qcom: Update PIL relocation info on load
Date: Fri, 15 May 2020 02:48:35 +0800 [thread overview]
Message-ID: <202005150200.DX18JJaQ%lkp@intel.com> (raw)
In-Reply-To: <20200513055641.1413100-4-bjorn.andersson@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 2394 bytes --]
Hi Bjorn,
I love your patch! Yet something to improve:
[auto build test ERROR on robh/for-next]
[also build test ERROR on linus/master v5.7-rc5 next-20200514]
[cannot apply to agross-msm/qcom/for-next remoteproc/for-next rpmsg/for-next hwspinlock/for-next]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Bjorn-Andersson/remoteproc-qcom-PIL-info-support/20200514-161851
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arm64-allyesconfig (attached as .config)
compiler: aarch64-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
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day GCC_VERSION=9.3.0 make.cross ARCH=arm64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
drivers/remoteproc/qcom_q6v5_wcss.c: In function 'q6v5_wcss_load':
>> drivers/remoteproc/qcom_q6v5_wcss.c:433:2: error: implicit declaration of function 'qcom_pil_info_store' [-Werror=implicit-function-declaration]
433 | qcom_pil_info_store("wcnss", wcss->mem_reloc, wcss->mem_size);
| ^~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/qcom_pil_info_store +433 drivers/remoteproc/qcom_q6v5_wcss.c
420
421 static int q6v5_wcss_load(struct rproc *rproc, const struct firmware *fw)
422 {
423 struct q6v5_wcss *wcss = rproc->priv;
424 int ret;
425
426 ret = qcom_mdt_load_no_init(wcss->dev, fw, rproc->firmware,
427 0, wcss->mem_region, wcss->mem_phys,
428 wcss->mem_size, &wcss->mem_reloc);
429 if (ret)
430 return ret;
431
432 /* Failures only affect post mortem debugging, so ignore return value */
> 433 qcom_pil_info_store("wcnss", wcss->mem_reloc, wcss->mem_size);
434
435 return ret;
436 }
437
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 71781 bytes --]
next prev parent reply other threads:[~2020-05-14 19:02 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-13 5:56 [PATCH v5 0/5] remoteproc: qcom: PIL info support Bjorn Andersson
2020-05-13 5:56 ` [PATCH v5 1/5] dt-bindings: remoteproc: Add Qualcomm PIL info binding Bjorn Andersson
2020-05-20 17:59 ` Mathieu Poirier
2020-05-26 22:32 ` Rob Herring
2020-05-13 5:56 ` [PATCH v5 2/5] remoteproc: qcom: Introduce helper to store pil info in IMEM Bjorn Andersson
2020-05-20 18:00 ` Mathieu Poirier
2020-05-13 5:56 ` [PATCH v5 3/5] remoteproc: qcom: Update PIL relocation info on load Bjorn Andersson
2020-05-14 18:48 ` kbuild test robot [this message]
2020-05-14 18:48 ` kbuild test robot
2020-05-19 18:14 ` rishabhb
2020-05-19 23:07 ` Bjorn Andersson
2020-05-13 5:56 ` [PATCH v5 4/5] arm64: dts: qcom: qcs404: Add IMEM and PIL info region Bjorn Andersson
2020-05-13 5:56 ` [PATCH v5 5/5] arm64: dts: qcom: sdm845: " Bjorn Andersson
2020-05-14 16:53 ` [PATCH v5 0/5] remoteproc: qcom: PIL info support Vinod Koul
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202005150200.DX18JJaQ%lkp@intel.com \
--to=lkp@intel.com \
--cc=agross@kernel.org \
--cc=bjorn.andersson@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=kbuild-all@lists.01.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-remoteproc@vger.kernel.org \
--cc=ohad@wizery.com \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.