[tiwai-sound:test/pcm-iov_iter 3/25] sound/core/memory.c:108:53: warning: passing argument 2 of 'copy_from_iter' makes integer from pointer without a cast

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git test/pcm-iov_iter
head:   da1be13e050f24d088d24fd65517d56f27a2eeb1
commit: 516e284c8b17a5129fcfbfb1f2f4e11093e13d3f [3/25] ALSA: core: Add memory copy helpers between iov_iter and iomem
config: i386-randconfig-r033-20230812 (https://download.01.org/0day-ci/archive/20230812/202308120931.Aj1gaY7K-lkp@xxxxxxxxx/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce: (https://download.01.org/0day-ci/archive/20230812/202308120931.Aj1gaY7K-lkp@xxxxxxxxx/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202308120931.Aj1gaY7K-lkp@xxxxxxxxx/

All warnings (new ones prefixed by >>):

   sound/core/memory.c: In function 'copy_from_iter_toio':
>> sound/core/memory.c:108:53: warning: passing argument 2 of 'copy_from_iter' makes integer from pointer without a cast [-Wint-conversion]
     108 |         return !copy_from_iter((void __force *)dst, src, count) ? -EFAULT : 0;
         |                                                     ^~~
         |                                                     |
         |                                                     struct iov_iter *
   In file included from include/sound/pcm.h:19,
                    from sound/core/memory.c:12:
   include/linux/uio.h:199:42: note: expected 'size_t' {aka 'unsigned int'} but argument is of type 'struct iov_iter *'
     199 | size_t copy_from_iter(void *addr, size_t bytes, struct iov_iter *i)
         |                                   ~~~~~~~^~~~~
>> sound/core/memory.c:108:58: warning: passing argument 3 of 'copy_from_iter' makes pointer from integer without a cast [-Wint-conversion]
     108 |         return !copy_from_iter((void __force *)dst, src, count) ? -EFAULT : 0;
         |                                                          ^~~~~
         |                                                          |
         |                                                          size_t {aka unsigned int}
   include/linux/uio.h:199:66: note: expected 'struct iov_iter *' but argument is of type 'size_t' {aka 'unsigned int'}
     199 | size_t copy_from_iter(void *addr, size_t bytes, struct iov_iter *i)
         |                                                 ~~~~~~~~~~~~~~~~~^


vim +/copy_from_iter +108 sound/core/memory.c

    93	
    94	/**
    95	 * copy_from_iter_toio - copy data from iov_iter to mmio-space
    96	 * @dst: the destination pointer on mmio-space
    97	 * @src: the source iov_iter
    98	 * @count: the data size to copy in bytes
    99	 *
   100	 * Copies the data from iov_iter to mmio-space.
   101	 * Only the single segment iov_iter is assumed for now.
   102	 *
   103	 * Return: Zero if successful, or non-zero on failure.
   104	 */
   105	int copy_from_iter_toio(void __iomem *dst, struct iov_iter *src, size_t count)
   106	{
   107	#if defined(__i386__) || defined(CONFIG_SPARC32)
 > 108		return !copy_from_iter((void __force *)dst, src, count) ? -EFAULT : 0;

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux