Re: [PATCH v3 3/3] dmaengine: sf-pdma: Get number of channel by device tree

[kernel test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 3836 bytes --]

Hi Zong,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on vkoul-dmaengine/next]
[also build test ERROR on v5.16]
[cannot apply to robh/for-next next-20220114]
[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/Zong-Li/Determine-the-number-of-DMA-channels-by-dma-channels-property/20220114-172017
base:   https://git.kernel.org/pub/scm/linux/kernel/git/vkoul/dmaengine.git next
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20220115/202201150313.YDVezSOq-lkp(a)intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
        # https://github.com/0day-ci/linux/commit/efa33512fd463fd6e29750992df1d40513eb5a7c
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Zong-Li/Determine-the-number-of-DMA-channels-by-dma-channels-property/20220114-172017
        git checkout efa33512fd463fd6e29750992df1d40513eb5a7c
        # save the config file to linux build tree
        mkdir build_dir
        make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash

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

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/compiler_types.h:85,
                    from <command-line>:
   arch/x86/include/asm/jump_label.h: In function 'vchan_cookie_complete':
>> include/linux/compiler-gcc.h:86:38: warning: asm operand 0 probably doesn't match constraints
      86 | #define asm_volatile_goto(x...) do { asm goto(x); asm (""); } while (0)
         |                                      ^~~
   arch/x86/include/asm/jump_label.h:27:2: note: in expansion of macro 'asm_volatile_goto'
      27 |  asm_volatile_goto("1:"
         |  ^~~~~~~~~~~~~~~~~
   include/linux/compiler-gcc.h:86:38: warning: asm operand 1 probably doesn't match constraints
      86 | #define asm_volatile_goto(x...) do { asm goto(x); asm (""); } while (0)
         |                                      ^~~
   arch/x86/include/asm/jump_label.h:27:2: note: in expansion of macro 'asm_volatile_goto'
      27 |  asm_volatile_goto("1:"
         |  ^~~~~~~~~~~~~~~~~
>> include/linux/compiler-gcc.h:86:38: error: impossible constraint in 'asm'
      86 | #define asm_volatile_goto(x...) do { asm goto(x); asm (""); } while (0)
         |                                      ^~~
   arch/x86/include/asm/jump_label.h:27:2: note: in expansion of macro 'asm_volatile_goto'
      27 |  asm_volatile_goto("1:"
         |  ^~~~~~~~~~~~~~~~~


vim +/asm +86 include/linux/compiler-gcc.h

313dd1b629219d Kees Cook   2017-05-05  76  
cb984d101b30eb Joe Perches 2015-06-25  77  /*
cb984d101b30eb Joe Perches 2015-06-25  78   * GCC 'asm goto' miscompiles certain code sequences:
cb984d101b30eb Joe Perches 2015-06-25  79   *
cb984d101b30eb Joe Perches 2015-06-25  80   *   http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58670
cb984d101b30eb Joe Perches 2015-06-25  81   *
cb984d101b30eb Joe Perches 2015-06-25  82   * Work it around via a compiler barrier quirk suggested by Jakub Jelinek.
cb984d101b30eb Joe Perches 2015-06-25  83   *
cb984d101b30eb Joe Perches 2015-06-25  84   * (asm goto is automatically volatile - the naming reflects this.)
cb984d101b30eb Joe Perches 2015-06-25  85   */
cb984d101b30eb Joe Perches 2015-06-25 @86  #define asm_volatile_goto(x...)	do { asm goto(x); asm (""); } while (0)
cb984d101b30eb Joe Perches 2015-06-25  87  

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