Messages in this thread | | | Date | Sat, 15 Jan 2022 05:28:29 +0800 | From | kernel test robot <> | Subject | [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') |
| |
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-lkp@intel.com/config) 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@intel.com>
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
| |