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-rc1 next-20211118]
[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: xtensa-buildonly-randconfig-r005-20211115 (attached as .config)
compiler: xtensa-linux-gcc (GCC) 11.2.0
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
        # 
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 attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross 
ARCH=xtensa 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <l...@intel.com>

All warnings (new ones prefixed by >>):

>> drivers/dma/qcom/qcom_adm.c:712:18: warning: no previous prototype for 
>> 'adm_dma_xlate' [-Wmissing-prototypes]
     712 | struct dma_chan *adm_dma_xlate(struct of_phandle_args *dma_spec,
         |                  ^~~~~~~~~~~~~


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-...@lists.01.org

Attachment: .config.gz
Description: application/gzip

Reply via email to