Hi allen, I love your patch! Yet something to improve: [auto build test ERROR on robh/for-next] [also build test ERROR on drm-intel/for-linux-next linus/master v5.8 next-20200810] [cannot apply to drm-tip/drm-tip drm/drm-next] [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/allen/IT6505-cover-letter/20200810-181446 base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next config: powerpc-allyesconfig (attached as .config) compiler: powerpc64-linux-gcc (GCC) 9.3.0 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 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All error/warnings (new ones prefixed by >>): drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_calc_video_info': drivers/gpu/drm/bridge/ite-it6505.c:662:14: warning: variable 'vdes' set but not used [-Wunused-but-set-variable] 662 | int vtotal, vdes, vdew, vfph, vsyncw; | ^~~~ drivers/gpu/drm/bridge/ite-it6505.c:661:14: warning: variable 'hdes' set but not used [-Wunused-but-set-variable] 661 | int htotal, hdes, hdew, hfph, hsyncw; | ^~~~ drivers/gpu/drm/bridge/ite-it6505.c:660:28: warning: variable 'interlaced' set but not used [-Wunused-but-set-variable] 660 | int hsync_pol, vsync_pol, interlaced; | ^~~~~~~~~~ drivers/gpu/drm/bridge/ite-it6505.c:660:17: warning: variable 'vsync_pol' set but not used [-Wunused-but-set-variable] 660 | int hsync_pol, vsync_pol, interlaced; | ^~~~~~~~~ drivers/gpu/drm/bridge/ite-it6505.c:660:6: warning: variable 'hsync_pol' set but not used [-Wunused-but-set-variable] 660 | int hsync_pol, vsync_pol, interlaced; | ^~~~~~~~~ In file included from drivers/gpu/drm/bridge/ite-it6505.c:32: drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_setup_sha1_input': >> drivers/gpu/drm/bridge/ite-it6505.c:1935:28: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=] 1935 | DRM_DEV_DEBUG_DRIVER(dev, "binfo:0x%*ph", ARRAY_SIZE(binfo), binfo); | ^~~~~~~~~~~~~~ include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER' 386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__) | ^~~ drivers/gpu/drm/bridge/ite-it6505.c:1935:38: note: format string is defined here 1935 | DRM_DEV_DEBUG_DRIVER(dev, "binfo:0x%*ph", ARRAY_SIZE(binfo), binfo); | ~^~ | | | int In file included from drivers/gpu/drm/bridge/ite-it6505.c:32: drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_hdcp_work': drivers/gpu/drm/bridge/ite-it6505.c:2070:28: warning: field width specifier '*' expects argument of type 'int', but argument 5 has type 'long unsigned int' [-Wformat=] 2070 | DRM_DEV_DEBUG_DRIVER(dev, "ret: %d link_status: %*ph", ret, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ 2071 | sizeof(link_status), link_status); | ~~~~~~~~~~~~~~~~~~~ | | | long unsigned int include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER' 386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__) | ^~~ drivers/gpu/drm/bridge/ite-it6505.c:2070:51: note: format string is defined here 2070 | DRM_DEV_DEBUG_DRIVER(dev, "ret: %d link_status: %*ph", ret, | ~^~ | | | int In file included from drivers/gpu/drm/bridge/ite-it6505.c:32: drivers/gpu/drm/bridge/ite-it6505.c:2087:28: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=] 2087 | DRM_DEV_DEBUG_DRIVER(dev, "bksv = 0x%*ph", | ^~~~~~~~~~~~~~~ include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER' 386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__) | ^~~ drivers/gpu/drm/bridge/ite-it6505.c:2087:39: note: format string is defined here 2087 | DRM_DEV_DEBUG_DRIVER(dev, "bksv = 0x%*ph", | ~^~ | | | int drivers/gpu/drm/bridge/ite-it6505.c: At top level: >> drivers/gpu/drm/bridge/ite-it6505.c:2099:6: warning: no previous prototype for 'it6505_show_hdcp_info' [-Wmissing-prototypes] 2099 | void it6505_show_hdcp_info(struct it6505 *it6505) | ^~~~~~~~~~~~~~~~~~~~~ In file included from drivers/gpu/drm/bridge/ite-it6505.c:32: drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_show_hdcp_info': drivers/gpu/drm/bridge/ite-it6505.c:2107:28: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=] 2107 | DRM_DEV_DEBUG_DRIVER(dev, "bksv = 0x%*ph", ARRAY_SIZE(it6505->bksvs), | ^~~~~~~~~~~~~~~ include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER' 386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__) | ^~~ drivers/gpu/drm/bridge/ite-it6505.c:2107:39: note: format string is defined here 2107 | DRM_DEV_DEBUG_DRIVER(dev, "bksv = 0x%*ph", ARRAY_SIZE(it6505->bksvs), | ~^~ | | | int In file included from drivers/gpu/drm/bridge/ite-it6505.c:32: drivers/gpu/drm/bridge/ite-it6505.c:2113:29: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=] 2113 | DRM_DEV_DEBUG_DRIVER(dev, "sha1_input: 0x%*ph", | ^~~~~~~~~~~~~~~~~~~~ include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER' 386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__) | ^~~ drivers/gpu/drm/bridge/ite-it6505.c:2113:45: note: format string is defined here 2113 | DRM_DEV_DEBUG_DRIVER(dev, "sha1_input: 0x%*ph", | ~^~ | | | int drivers/gpu/drm/bridge/ite-it6505.c: At top level: >> drivers/gpu/drm/bridge/ite-it6505.c:2965:16: error: initialization of 'enum drm_mode_status (*)(struct drm_bridge *, const struct drm_display_mode *)' from incompatible pointer type 'enum drm_mode_status (*)(struct drm_bridge *, const struct drm_display_info *, const struct drm_display_mode *)' [-Werror=incompatible-pointer-types] 2965 | .mode_valid = it6505_bridge_mode_valid, | ^~~~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/bridge/ite-it6505.c:2965:16: note: (near initialization for 'it6505_bridge_funcs.mode_valid') drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_i2c_probe': drivers/gpu/drm/bridge/ite-it6505.c:3140:31: warning: variable 'pdata' set but not used [-Wunused-but-set-variable] 3140 | struct it6505_platform_data *pdata; | ^~~~~ cc1: some warnings being treated as errors vim +2965 drivers/gpu/drm/bridge/ite-it6505.c 2961 2962 static const struct drm_bridge_funcs it6505_bridge_funcs = { 2963 .attach = it6505_bridge_attach, 2964 .detach = it6505_bridge_detach, > 2965 .mode_valid = it6505_bridge_mode_valid, 2966 .mode_set = it6505_bridge_mode_set, 2967 .enable = it6505_bridge_enable, 2968 .disable = it6505_bridge_disable, 2969 }; 2970 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org