[kbuild-all] [hverkuil-media-tree:for-v5.18f 76/82] drivers/media/platform/aspeed-video.c:977:5: warning: format specifies type 'unsigned int' but the argument has type 'dma_addr_t' (aka 'unsigned long long')

Saturday, 15 January 2022

tree:   git://linuxtv.org/hverkuil/media_tree.git for-v5.18f
head:   97dd85f0caf1ad96c275147583c46f8ee12308fc
commit: 63c049607882d11ee1cce0c739113c8dfd7fed02 [76/82] media: aspeed: add more debug log
messages
config: x86_64-randconfig-a012
(https://download.01.org/0day-ci/archive/20220115/202201150524.D0pO65rp-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
82c8aca93488730ce8f66101e0f3538f14b551dd)
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
        git remote add hverkuil-media-tree git://linuxtv.org/hverkuil/media_tree.git
        git fetch --no-tags hverkuil-media-tree for-v5.18f
        git checkout 63c049607882d11ee1cce0c739113c8dfd7fed02
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir
ARCH=x86_64 SHELL=/bin/bash drivers/media/platform/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com&gt;

All warnings (new ones prefixed by >>):

...
> drivers/media/platform/aspeed-video.c:977:5: warning: format
specifies type 'unsigned int' but the argument has type 'dma_addr_t' (aka
'unsigned long long') [-Wformat]                            
video->srcs[0].dma, video->srcs[0].size);
                            ^~~~~~~~~~~~~~~~~~
   include/media/v4l2-common.h:74:42: note: expanded from macro 'v4l2_dbg'
                           v4l2_printk(KERN_DEBUG, dev, fmt , ## arg);     \
                                                        ~~~      ^~~
   include/media/v4l2-common.h:58:44: note: expanded from macro 'v4l2_printk'
           printk(level "%s: " fmt, (dev)->name , ## arg)
                               ~~~                   ^~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   drivers/media/platform/aspeed-video.c:979:5: warning: format specifies type
'unsigned int' but the argument has type 'dma_addr_t' (aka 'unsigned
long long') [-Wformat]
                            video->srcs[1].dma, video->srcs[1].size);
                            ^~~~~~~~~~~~~~~~~~
   include/media/v4l2-common.h:74:42: note: expanded from macro 'v4l2_dbg'
                           v4l2_printk(KERN_DEBUG, dev, fmt , ## arg);     \
                                                        ~~~      ^~~
   include/media/v4l2-common.h:58:44: note: expanded from macro 'v4l2_printk'
           printk(level "%s: " fmt, (dev)->name , ## arg)
                               ~~~                   ^~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   drivers/media/platform/aspeed-video.c:1806:25: warning: format specifies type
'unsigned int' but the argument has type 'dma_addr_t' (aka 'unsigned
long long') [-Wformat]
                    VE_JPEG_HEADER_SIZE, video->jpeg.dma);
                                         ^~~~~~~~~~~~~~~
   include/linux/dev_printk.h:150:67: note: expanded from macro 'dev_info'
           dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
                                                                    ~~~     ^~~~~~~~~~~
   include/linux/dev_printk.h:110:23: note: expanded from macro
'dev_printk_index_wrap'
                   _p_func(dev, fmt, ##__VA_ARGS__);                       \
                                ~~~    ^~~~~~~~~~~
   3 warnings generated.

vim +977 drivers/media/platform/aspeed-video.c

   916	
   917	static void aspeed_video_set_resolution(struct aspeed_video *video)
   918	{
   919		struct v4l2_bt_timings *act = &video->active_timings;
   920		unsigned int size = act->width * act->height;
   921	
   922		/* Set capture/compression frame sizes */
   923		aspeed_video_calc_compressed_size(video, size);
   924	
   925		if (video->active_timings.width == 1680) {
   926			/*
   927			 * This is a workaround to fix a silicon bug on A1 and A2
   928			 * revisions. Since it doesn't break capturing operation of
   929			 * other revisions, use it for all revisions without checking
   930			 * the revision ID. It picked 1728 which is a very next
   931			 * 64-pixels aligned value to 1680 to minimize memory bandwidth
   932			 * and to get better access speed from video engine.
   933			 */
   934			aspeed_video_write(video, VE_CAP_WINDOW,
   935					   1728 << 16 | act->height);
   936			size += (1728 - 1680) * video->active_timings.height;
   937		} else {
   938			aspeed_video_write(video, VE_CAP_WINDOW,
   939					   act->width << 16 | act->height);
   940		}
   941		aspeed_video_write(video, VE_COMP_WINDOW,
   942				   act->width << 16 | act->height);
   943		aspeed_video_write(video, VE_SRC_SCANLINE_OFFSET, act->width * 4);
   944	
   945		/* Don't use direct mode below 1024 x 768 (irqs don't fire) */
   946		if (size < DIRECT_FETCH_THRESHOLD) {
   947			v4l2_dbg(1, debug, &video->v4l2_dev, "Capture: Sync Mode\n");
   948			aspeed_video_write(video, VE_TGS_0,
   949					   FIELD_PREP(VE_TGS_FIRST,
   950						      video->frame_left - 1) |
   951					   FIELD_PREP(VE_TGS_LAST,
   952						      video->frame_right));
   953			aspeed_video_write(video, VE_TGS_1,
   954					   FIELD_PREP(VE_TGS_FIRST, video->frame_top) |
   955					   FIELD_PREP(VE_TGS_LAST,
   956						      video->frame_bottom + 1));
   957			aspeed_video_update(video, VE_CTRL, 0, VE_CTRL_INT_DE);
   958		} else {
   959			v4l2_dbg(1, debug, &video->v4l2_dev, "Capture: Direct Mode\n");
   960			aspeed_video_update(video, VE_CTRL, 0, VE_CTRL_DIRECT_FETCH);
   961		}
   962	
   963		size *= 4;
   964	
   965		if (size != video->srcs[0].size) {
   966			if (video->srcs[0].size)
   967				aspeed_video_free_buf(video, &video->srcs[0]);
   968			if (video->srcs[1].size)
   969				aspeed_video_free_buf(video, &video->srcs[1]);
   970	
   971			if (!aspeed_video_alloc_buf(video, &video->srcs[0], size))
   972				goto err_mem;
   973			if (!aspeed_video_alloc_buf(video, &video->srcs[1], size))
   974				goto err_mem;
   975	
   976			v4l2_dbg(1, debug, &video->v4l2_dev, "src buf0 addr(%#x)
size(%d)\n",
...
 977				 video->srcs[0].dma, video->srcs[0].size);   
978			v4l2_dbg(1, debug, &video->v4l2_dev, "src buf1 addr(%#x)
size(%d)\n",
   979				 video->srcs[1].dma, video->srcs[1].size);
   980			aspeed_video_write(video, VE_SRC0_ADDR, video->srcs[0].dma);
   981			aspeed_video_write(video, VE_SRC1_ADDR, video->srcs[1].dma);
   982		}
   983	
   984		return;
   985	
   986	err_mem:
   987		dev_err(video->dev, "Failed to allocate source buffers\n");
   988	
   989		if (video->srcs[0].size)
   990			aspeed_video_free_buf(video, &video->srcs[0]);
   991	}
   992	

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

2022

2021

2020

2019

[kbuild-all] [hverkuil-media-tree:for-v5.18f 76/82] drivers/media/platform/aspeed-video.c:977:5: warning: format specifies type 'unsigned int' but the argument has type 'dma_addr_t' (aka 'unsigned long long')