* Re: [PATCH 07/11] dmaengine: qcom-adm: stop abusing slave_id config
[not found] <20211115085403.360194-8-arnd@kernel.org>
@ 2021-11-25 7:57 ` kernel test robot
2021-11-25 8:25 ` Arnd Bergmann
0 siblings, 1 reply; 3+ messages in thread
From: kernel test robot @ 2021-11-25 7:57 UTC (permalink / raw)
To: Arnd Bergmann, Vinod Koul
Cc: llvm, kbuild-all, alsa-devel, dri-devel, Bjorn Andersson,
linux-tegra, Thierry Reding, linux-mtd, Laurent Pinchart,
linux-spi, Robert Jarzmik
Hi Arnd,
I love your patch! Perhaps something to improve:
[auto build test WARNING on vkoul-dmaengine/next]
[also build test WARNING on tiwai-sound/for-next staging/staging-testing linus/master v5.16-rc2 next-20211125]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Arnd-Bergmann/dmaengine-kill-off-dma_slave_config-slave_id/20211115-165731
base: https://git.kernel.org/pub/scm/linux/kernel/git/vkoul/dmaengine.git next
config: riscv-randconfig-r016-20211115 (https://download.01.org/0day-ci/archive/20211125/202111251538.x6sJNCka-lkp@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project fbe72e41b99dc7994daac300d208a955be3e4a0a)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
# https://github.com/0day-ci/linux/commit/f2e7f9ee67ce784864f75db39f20d1060c932279
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Arnd-Bergmann/dmaengine-kill-off-dma_slave_config-slave_id/20211115-165731
git checkout f2e7f9ee67ce784864f75db39f20d1060c932279
# save the config file to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=riscv
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/dma/qcom/qcom_adm.c:712:18: warning: no previous prototype for function 'adm_dma_xlate' [-Wmissing-prototypes]
struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
^
drivers/dma/qcom/qcom_adm.c:712:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
^
static
1 warning generated.
vim +/adm_dma_xlate +712 drivers/dma/qcom/qcom_adm.c
700
701 /**
702 * adm_dma_xlate
703 * @dma_spec: pointer to DMA specifier as found in the device tree
704 * @ofdma: pointer to DMA controller data
705 *
706 * This can use either 1-cell or 2-cell formats, the first cell
707 * identifies the slave device, while the optional second cell
708 * contains the crci value.
709 *
710 * Returns pointer to appropriate dma channel on success or NULL on error.
711 */
> 712 struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
713 struct of_dma *ofdma)
714 {
715 struct dma_device *dev = ofdma->of_dma_data;
716 struct dma_chan *chan, *candidate = NULL;
717 struct adm_chan *achan;
718
719 if (!dev || dma_spec->args_count > 2)
720 return NULL;
721
722 list_for_each_entry(chan, &dev->channels, device_node)
723 if (chan->chan_id == dma_spec->args[0]) {
724 candidate = chan;
725 break;
726 }
727
728 if (!candidate)
729 return NULL;
730
731 achan = to_adm_chan(candidate);
732 if (dma_spec->args_count == 2)
733 achan->crci = dma_spec->args[1];
734 else
735 achan->crci = 0;
736
737 return dma_get_slave_channel(candidate);
738 }
739
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 07/11] dmaengine: qcom-adm: stop abusing slave_id config
2021-11-25 7:57 ` [PATCH 07/11] dmaengine: qcom-adm: stop abusing slave_id config kernel test robot
@ 2021-11-25 8:25 ` Arnd Bergmann
2021-11-25 9:32 ` Vinod Koul
0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2021-11-25 8:25 UTC (permalink / raw)
To: kernel test robot
Cc: Vinod Koul, llvm, kbuild-all, ALSA Development Mailing List,
dri-devel, Bjorn Andersson, open list:TEGRA ARCHITECTURE SUPPORT,
Thierry Reding, linux-mtd, Laurent Pinchart, linux-spi,
Robert Jarzmik
On Thu, Nov 25, 2021 at 8:57 AM kernel test robot <lkp@intel.com> wrote:
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):
>
> >> drivers/dma/qcom/qcom_adm.c:712:18: warning: no previous prototype for function 'adm_dma_xlate' [-Wmissing-prototypes]
> struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
> ^
> drivers/dma/qcom/qcom_adm.c:712:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
> struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
> ^
> static
> 1 warning generated.
I noticed this mistake slipped into v2 as well, the function needs to
be marked 'static'.
Vinod, let me know how you want me to address this. Should I just fold
the fix (see below)
and the final Acks into the patch and send an updated pull request, or
do a complete v3
submission?
Arnd
8<---
diff --git a/drivers/dma/qcom/qcom_adm.c b/drivers/dma/qcom/qcom_adm.c
index bb338b303af6..65697bee4db0 100644
--- a/drivers/dma/qcom/qcom_adm.c
+++ b/drivers/dma/qcom/qcom_adm.c
@@ -709,8 +709,8 @@ static void adm_channel_init(struct adm_device
*adev, struct adm_chan *achan,
*
* Returns pointer to appropriate dma channel on success or NULL on error.
*/
-struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
- struct of_dma *ofdma)
+static struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
+ struct of_dma *ofdma)
{
struct dma_device *dev = ofdma->of_dma_data;
struct dma_chan *chan, *candidate = NULL;
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 07/11] dmaengine: qcom-adm: stop abusing slave_id config
2021-11-25 8:25 ` Arnd Bergmann
@ 2021-11-25 9:32 ` Vinod Koul
0 siblings, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2021-11-25 9:32 UTC (permalink / raw)
To: Arnd Bergmann
Cc: kernel test robot, llvm, kbuild-all,
ALSA Development Mailing List, dri-devel, Bjorn Andersson,
open list:TEGRA ARCHITECTURE SUPPORT, Thierry Reding, linux-mtd,
Laurent Pinchart, linux-spi, Robert Jarzmik
On 25-11-21, 09:25, Arnd Bergmann wrote:
> On Thu, Nov 25, 2021 at 8:57 AM kernel test robot <lkp@intel.com> wrote:
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@intel.com>
> >
> > All warnings (new ones prefixed by >>):
> >
> > >> drivers/dma/qcom/qcom_adm.c:712:18: warning: no previous prototype for function 'adm_dma_xlate' [-Wmissing-prototypes]
> > struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
> > ^
> > drivers/dma/qcom/qcom_adm.c:712:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
> > struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
> > ^
> > static
> > 1 warning generated.
>
> I noticed this mistake slipped into v2 as well, the function needs to
> be marked 'static'.
>
> Vinod, let me know how you want me to address this. Should I just fold
> the fix (see below)
> and the final Acks into the patch and send an updated pull request, or
> do a complete v3
> submission?
I can fold this while applying, the series lgtm, I will wait a day
before applying...
Thanks
>
> Arnd
>
> 8<---
> diff --git a/drivers/dma/qcom/qcom_adm.c b/drivers/dma/qcom/qcom_adm.c
> index bb338b303af6..65697bee4db0 100644
> --- a/drivers/dma/qcom/qcom_adm.c
> +++ b/drivers/dma/qcom/qcom_adm.c
> @@ -709,8 +709,8 @@ static void adm_channel_init(struct adm_device
> *adev, struct adm_chan *achan,
> *
> * Returns pointer to appropriate dma channel on success or NULL on error.
> */
> -struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
> - struct of_dma *ofdma)
> +static struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
> + struct of_dma *ofdma)
> {
> struct dma_device *dev = ofdma->of_dma_data;
> struct dma_chan *chan, *candidate = NULL;
--
~Vinod
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-11-25 9:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20211115085403.360194-8-arnd@kernel.org>
2021-11-25 7:57 ` [PATCH 07/11] dmaengine: qcom-adm: stop abusing slave_id config kernel test robot
2021-11-25 8:25 ` Arnd Bergmann
2021-11-25 9:32 ` Vinod Koul
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).