Hi Tanmay,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on robh/for-next] [also build test WARNING on drm-intel/for-linux-next linus/master v5.7 next-20200608] [cannot apply to robclark/msm-next] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Tanmay-Shah/dt-bindings-msm-dp-add-... base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next config: arm-allyesconfig (attached as .config) compiler: arm-linux-gnueabi-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=arm
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 >>, old ones prefixed by <<):
drivers/gpu/drm/msm/dp/dp_aux.c:35:8: warning: type qualifiers ignored on function return type [-Wignored-qualifiers]
35 | static const char * const dp_aux_get_error(u32 aux_error) | ^~~~~ --
drivers/gpu/drm/msm/dp/dp_catalog.c:352:6: warning: no previous prototype for 'dump_regs' [-Wmissing-prototypes]
352 | void dump_regs(void * __iomem *base, int len) | ^~~~~~~~~ In file included from include/linux/printk.h:7, from include/linux/kernel.h:15, from include/linux/delay.h:22, from drivers/gpu/drm/msm/dp/dp_catalog.c:8: drivers/gpu/drm/msm/dp/dp_catalog.c: In function 'dump_regs':
include/linux/kern_levels.h:5:18: warning: format '%x' expects argument of type 'unsigned int', but argument 3 has type 'void **' [-Wformat=]
5 | #define KERN_SOH "001" /* ASCII Start Of Header */ | ^~~~~~ include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH' 14 | #define KERN_INFO KERN_SOH "6" /* informational */ | ^~~~~~~~ include/linux/printk.h:305:9: note: in expansion of macro 'KERN_INFO' 305 | printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__) | ^~~~~~~~~
drivers/gpu/drm/msm/dp/dp_catalog.c:364:3: note: in expansion of macro 'pr_info'
364 | pr_info("%08x: %08x %08x %08x %08x", base, x0, x4, x8, xc); | ^~~~~~~ drivers/gpu/drm/msm/dp/dp_catalog.c:364:15: note: format string is defined here 364 | pr_info("%08x: %08x %08x %08x %08x", base, x0, x4, x8, xc); | ~~~^ | | | unsigned int At top level: drivers/gpu/drm/msm/dp/dp_catalog.c:76:17: warning: 'vm_voltage_swing_hbr3_hbr2' defined but not used [-Wunused-const-variable=] 76 | static u8 const vm_voltage_swing_hbr3_hbr2[4][4] = { | ^~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/msm/dp/dp_catalog.c:69:17: warning: 'vm_pre_emphasis_hbr3_hbr2' defined but not used [-Wunused-const-variable=] 69 | static u8 const vm_pre_emphasis_hbr3_hbr2[4][4] = { | ^~~~~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/msm/dp/dp_catalog.c:62:17: warning: 'vm_voltage_swing' defined but not used [-Wunused-const-variable=] 62 | static u8 const vm_voltage_swing[4][4] = { | ^~~~~~~~~~~~~~~~ drivers/gpu/drm/msm/dp/dp_catalog.c:54:17: warning: 'vm_pre_emphasis' defined but not used [-Wunused-const-variable=] 54 | static u8 const vm_pre_emphasis[4][4] = { | ^~~~~~~~~~~~~~~ -- drivers/gpu/drm/msm/dp/dp_ctrl.c: In function '_tu_param_compare':
drivers/gpu/drm/msm/dp/dp_ctrl.c:254:13: warning: variable 'b_frac' set but not used [-Wunused-but-set-variable]
254 | u32 b_int, b_frac, b_sign; | ^~~~~~
drivers/gpu/drm/msm/dp/dp_ctrl.c:254:6: warning: variable 'b_int' set but not used [-Wunused-but-set-variable]
254 | u32 b_int, b_frac, b_sign; | ^~~~~
drivers/gpu/drm/msm/dp/dp_ctrl.c:253:13: warning: variable 'a_frac' set but not used [-Wunused-but-set-variable]
253 | u32 a_int, a_frac, a_sign; | ^~~~~~
drivers/gpu/drm/msm/dp/dp_ctrl.c:253:6: warning: variable 'a_int' set but not used [-Wunused-but-set-variable]
253 | u32 a_int, a_frac, a_sign; | ^~~~~ drivers/gpu/drm/msm/dp/dp_ctrl.c: In function 'dp_panel_update_tu_timings':
drivers/gpu/drm/msm/dp/dp_ctrl.c:301:6: warning: variable 'overhead_dsc' set but not used [-Wunused-but-set-variable]
301 | s64 overhead_dsc; | ^~~~~~~~~~~~ -- drivers/gpu/drm/msm/dp/dp_link.c: In function 'dp_link_parse_phy_test_params': drivers/gpu/drm/msm/dp/dp_link.c:576:47: error: 'DP_PHY_TEST_PATTERN' undeclared (first use in this function); did you mean 'DP_NO_TEST_PATTERN'? 576 | rlen = drm_dp_dpcd_readb(link->aux->drm_aux, DP_PHY_TEST_PATTERN, | ^~~~~~~~~~~~~~~~~~~ | DP_NO_TEST_PATTERN drivers/gpu/drm/msm/dp/dp_link.c:576:47: note: each undeclared identifier is reported only once for each function it appears in drivers/gpu/drm/msm/dp/dp_link.c: In function 'dp_link_adjust_levels':
drivers/gpu/drm/msm/dp/dp_link.c:1106:26: warning: variable 'link' set but not used [-Wunused-but-set-variable]
1106 | struct dp_link_private *link; | ^~~~ -- drivers/gpu/drm/msm/dp/dp_panel.c: In function 'dp_panel_read_dpcd':
drivers/gpu/drm/msm/dp/dp_panel.c:39:11: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
39 | if (rlen < 0) { | ^ drivers/gpu/drm/msm/dp/dp_panel.c: At top level:
drivers/gpu/drm/msm/dp/dp_panel.c:283:4: warning: no previous prototype for 'dp_panel_get_edid_checksum' [-Wmissing-prototypes]
283 | u8 dp_panel_get_edid_checksum(struct edid *edid) | ^~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/msm/dp/dp_panel.c: In function 'dp_panel_put':
drivers/gpu/drm/msm/dp/dp_panel.c:482:27: warning: variable 'panel' set but not used [-Wunused-but-set-variable]
482 | struct dp_panel_private *panel; | ^~~~~ -- drivers/gpu/drm/msm/dp/dp_parser.c: In function 'dp_parser_init_clk_data':
drivers/gpu/drm/msm/dp/dp_parser.c:251:26: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
251 | int num_clk = 0, i = 0, rc = 0; | ^~ -- drivers/gpu/drm/msm/dp/dp_power.c: In function 'dp_power_clk_enable':
drivers/gpu/drm/msm/dp/dp_power.c:172:27: warning: variable 'mp' set but not used [-Wunused-but-set-variable]
172 | struct dss_module_power *mp; | ^~
vim +35 drivers/gpu/drm/msm/dp/dp_aux.c
34
35 static const char * const dp_aux_get_error(u32 aux_error)
36 { 37 switch (aux_error) { 38 case DP_AUX_ERR_NONE: 39 return DP_AUX_ENUM_STR(DP_AUX_ERR_NONE); 40 case DP_AUX_ERR_ADDR: 41 return DP_AUX_ENUM_STR(DP_AUX_ERR_ADDR); 42 case DP_AUX_ERR_TOUT: 43 return DP_AUX_ENUM_STR(DP_AUX_ERR_TOUT); 44 case DP_AUX_ERR_NACK: 45 return DP_AUX_ENUM_STR(DP_AUX_ERR_NACK); 46 case DP_AUX_ERR_DEFER: 47 return DP_AUX_ENUM_STR(DP_AUX_ERR_DEFER); 48 case DP_AUX_ERR_NACK_DEFER: 49 return DP_AUX_ENUM_STR(DP_AUX_ERR_NACK_DEFER); 50 default: 51 return "unknown"; 52 } 53 } 54
--- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org