Re: [linux-next:master 9419/10007] drivers/mtd/nand/raw/arasan-nand-controller.c:1453:33: warning: shift count >= width of type

[Miquel Raynal <miquel.raynal@bootlin.com>]

Hello,

kernel test robot <lkp@intel.com> wrote on Wed, 16 Jun 2021 18:33:22
+0800:

> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head:   19ae1f2bd9c091059f80646604ccef8a1e614f57
> commit: 236fd3e0427070413e7059dc3c52c3f9c89f7cc0 [9419/10007] Merge remote-tracking branch 'nand/nand/next'
> config: riscv-randconfig-r013-20210615 (attached as .config)
> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
> 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://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=236fd3e0427070413e7059dc3c52c3f9c89f7cc0
>         git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
>         git fetch --no-tags linux-next master
>         git checkout 236fd3e0427070413e7059dc3c52c3f9c89f7cc0
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross 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 >>):
> 
>    In file included from drivers/mtd/nand/raw/arasan-nand-controller.c:17:
>    In file included from include/linux/dma-mapping.h:10:
>    In file included from include/linux/scatterlist.h:8:
>    In file included from include/linux/mm.h:33:
>    In file included from include/linux/pgtable.h:6:
>    arch/riscv/include/asm/pgtable.h:521:9: error: implicit declaration of function 'pfn_pmd' [-Werror,-Wimplicit-function-declaration]
>            return pfn_pmd(page_to_pfn(page), prot);
>                   ^
>    arch/riscv/include/asm/pgtable.h:521:9: note: did you mean 'pfn_pgd'?
>    arch/riscv/include/asm/pgtable.h:222:21: note: 'pfn_pgd' declared here
>    static inline pgd_t pfn_pgd(unsigned long pfn, pgprot_t prot)
>                        ^
>    arch/riscv/include/asm/pgtable.h:521:17: error: implicit declaration of function 'page_to_section' [-Werror,-Wimplicit-function-declaration]
>            return pfn_pmd(page_to_pfn(page), prot);
>                           ^
>    include/asm-generic/memory_model.h:81:21: note: expanded from macro 'page_to_pfn'
>    #define page_to_pfn __page_to_pfn
>                        ^
>    include/asm-generic/memory_model.h:64:14: note: expanded from macro '__page_to_pfn'
>            int __sec = page_to_section(__pg);                      \
>                        ^
>    arch/riscv/include/asm/pgtable.h:521:17: note: did you mean '__nr_to_section'?
>    include/asm-generic/memory_model.h:81:21: note: expanded from macro 'page_to_pfn'
>    #define page_to_pfn __page_to_pfn
>                        ^
>    include/asm-generic/memory_model.h:64:14: note: expanded from macro '__page_to_pfn'
>            int __sec = page_to_section(__pg);                      \
>                        ^
>    include/linux/mmzone.h:1314:35: note: '__nr_to_section' declared here
>    static inline struct mem_section *__nr_to_section(unsigned long nr)
>                                      ^
>    In file included from drivers/mtd/nand/raw/arasan-nand-controller.c:17:
>    In file included from include/linux/dma-mapping.h:10:
>    In file included from include/linux/scatterlist.h:8:
>    In file included from include/linux/mm.h:33:
>    In file included from include/linux/pgtable.h:6:
>    arch/riscv/include/asm/pgtable.h:521:9: error: returning 'int' from a function with incompatible result type 'pmd_t'
>            return pfn_pmd(page_to_pfn(page), prot);
>                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>    In file included from drivers/mtd/nand/raw/arasan-nand-controller.c:17:
>    In file included from include/linux/dma-mapping.h:10:
>    In file included from include/linux/scatterlist.h:8:
>    include/linux/mm.h:1552:29: error: static declaration of 'page_to_section' follows non-static declaration
>    static inline unsigned long page_to_section(const struct page *page)
>                                ^
>    arch/riscv/include/asm/pgtable.h:521:17: note: previous implicit declaration is here
>            return pfn_pmd(page_to_pfn(page), prot);
>                           ^
>    include/asm-generic/memory_model.h:81:21: note: expanded from macro 'page_to_pfn'
>    #define page_to_pfn __page_to_pfn
>                        ^
>    include/asm-generic/memory_model.h:64:14: note: expanded from macro '__page_to_pfn'
>            int __sec = page_to_section(__pg);                      \
>                        ^
> >> drivers/mtd/nand/raw/arasan-nand-controller.c:1453:33: warning: shift count >= width of type [-Wshift-count-overflow]  
>            ret = dma_set_mask(&pdev->dev, DMA_BIT_MASK(64));
>                                           ^~~~~~~~~~~~~~~~
>    include/linux/dma-mapping.h:76:54: note: expanded from macro 'DMA_BIT_MASK'
>    #define DMA_BIT_MASK(n) (((n) == 64) ? ~0ULL : ((1ULL<<(n))-1))
>                                                         ^ ~~~
>    1 warning and 4 errors generated.

I'm not sure to understand what the proper fix here. I am also unable
to reproduce the warning with the above steps.

Thanks,
Miquèl