[kbuild-all] [stffrdhrn:starfive-bad-dcache-flush 20/39] drivers/video/fbdev/starfive/starfive_fb.c:424:6: warning: variable 'ret' set but not used

tree: https://github.com/stffrdhrn/linux.git starfive-bad-dcache-flush head: a5c948a9cc2b8944f00056c067fb7bde19b34843 commit: eb1c3c7d70ee34a4f65bf77035c8c557caf559ac [20/39] drivers/media and video: update fb driver(flush cache) and cam sensor drivers config: arc-allyesconfig (attached as .config) compiler: arceb-elf-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 # https://github.com/stffrdhrn/linux/commit/eb1c3c7d70ee34a4f65bf77035c8c55... git remote add stffrdhrn https://github.com/stffrdhrn/linux.git git fetch --no-tags stffrdhrn starfive-bad-dcache-flush git checkout eb1c3c7d70ee34a4f65bf77035c8c557caf559ac # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arc 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/video/fbdev/starfive/starfive_fb.c: In function 'sf_fb_setcolreg': 424 | int ret = 1; | ^~~ -- 45 | void lcdc_mode_cfg(struct sf_fb_data *sf_dev, uint32_t workMode, int dotEdge, int syncEdge, int r2yBypass, | ^~~~~~~~~~~~~ 64 | void lcdc_timing_cfg(struct sf_fb_data *sf_dev, int vunit) | ^~~~~~~~~~~~~~~ 87 | void lcdc_desize_cfg(struct sf_fb_data *sf_dev) | ^~~~~~~~~~~~~~~ 97 | void lcdc_rgb_dclk_cfg(struct sf_fb_data *sf_dev, int dot_clk_sel) | ^~~~~~~~~~~~~~~~~ 111 | void lcdc_win_cfgA(struct sf_fb_data *sf_dev, int winNum, int layEn, int clorTab, | ^~~~~~~~~~~~~ 151 | void lcdc_win_cfgB(struct sf_fb_data *sf_dev, int winNum, int xpos, int ypos, int argbOrd) | ^~~~~~~~~~~~~ 176 | void lcdc_win_cfgC(struct sf_fb_data *sf_dev, int winNum, int colorKey) | ^~~~~~~~~~~~~ 184 | void lcdc_win_srcSize(struct sf_fb_data *sf_dev, int winNum) | ^~~~~~~~~~~~~~~~ 205 | void lcdc_alphaVal_cfg(struct sf_fb_data *sf_dev, int val1, int val2, int val3, int val4, int sel) | ^~~~~~~~~~~~~~~~~ 217 | void lcdc_panel_cfg(struct sf_fb_data *sf_dev, int buswid, int depth, int txcycle, int pixpcycle, | ^~~~~~~~~~~~~~ 232 | void lcdc_win02Addr_cfg(struct sf_fb_data *sf_dev, int addr0, int addr1) | ^~~~~~~~~~~~~~~~~~ drivers/video/fbdev/starfive/starfive_lcdc.c: In function 'lcdc_isr_handler': 303 | u32 intr_status = 0; | ^~~~~~~~~~~ In file included from drivers/video/fbdev/starfive/starfive_lcdc.c:22: At top level: drivers/video/fbdev/starfive/starfive_comm_regs.h:74:12: warning: 'sf_fb_cfgread32' defined but not used [-Wunused-function] 74 | static u32 sf_fb_cfgread32(struct sf_fb_data *sf_dev, u32 reg) | ^~~~~~~~~~~~~~~ -- 77 | void pp_output_cfg(struct sf_fb_data *sf_dev, int ppNum, int outSel, int progInter, int desformat, int ptMode) | ^~~~~~~~~~~~~ 89 | void pp_srcfmt_cfg(struct sf_fb_data *sf_dev, int ppNum, int srcformat, int yuv420Inter, int yuv422_mode, | ^~~~~~~~~~~~~ 103 | void pp_r2yscal_bypass(struct sf_fb_data *sf_dev, int ppNum, int r2yByp, int scalByp, int y2rByp) | ^~~~~~~~~~~~~~~~~ 111 | void pp_argb_alpha(struct sf_fb_data *sf_dev, int ppNum, int alpha) | ^~~~~~~~~~~~~ 119 | void pp_r2y_coeff(struct sf_fb_data *sf_dev, int ppNum, int coefNum, int rcoef, int gcoef, int bcoef, int off) | ^~~~~~~~~~~~ 131 | void pp_output_fmt_cfg(struct sf_fb_data *sf_dev, int ppNum, int yuv420Inter, int yuv422_mode) | ^~~~~~~~~~~~~~~~~ 140 | void pp_lockTrans_cfg(struct sf_fb_data *sf_dev, int ppNum, int lockTrans) | ^~~~~~~~~~~~~~~~ 147 | void pp_int_interval_cfg(struct sf_fb_data *sf_dev, int ppNum, int interval) | ^~~~~~~~~~~~~~~~~~~ 154 | void pp_srcSize_cfg(struct sf_fb_data *sf_dev, int ppNum, int hsize, int vsize) | ^~~~~~~~~~~~~~ 162 | void pp_drop_cfg(struct sf_fb_data *sf_dev, int ppNum, int hdrop, int vdrop) | ^~~~~~~~~~~ 170 | void pp_desSize_cfg(struct sf_fb_data *sf_dev, int ppNum, int hsize, int vsize) | ^~~~~~~~~~~~~~ 177 | void pp_desAddr_cfg(struct sf_fb_data *sf_dev, int ppNum, int yaddr, int uaddr, int vaddr) | ^~~~~~~~~~~~~~ 185 | void pp_desOffset_cfg(struct sf_fb_data *sf_dev, int ppNum, int yoff, int uoff, int voff) | ^~~~~~~~~~~~~~~~ 363 | void pp_format_set(struct sf_fb_data *sf_dev, int ppNum, struct pp_video_mode *src, struct pp_video_mode *dst) | ^~~~~~~~~~~~~ 408 | void pp_size_set(struct sf_fb_data *sf_dev, int ppNum, struct pp_video_mode *src, struct pp_video_mode *dst) | ^~~~~~~~~~~ -- 341 | void reset_dphy(struct sf_fb_data *sf_dev, int resetb) | ^~~~~~~~~~ 362 | void polling_dphy_lock(struct sf_fb_data *sf_dev) | ^~~~~~~~~~~~~~~~~ 388 | void release_txbyte_rst(struct sf_fb_data *sf_dev) | ^~~~~~~~~~~~~~~~~~ 404 | void vid_size_cfg_update(struct sf_fb_data *sf_dev) | ^~~~~~~~~~~~~~~~~~~ 437 | void vid_blktime_cfg_update(struct sf_fb_data *sf_dev) | ^~~~~~~~~~~~~~~~~~~~~~ 486 | void dsi_main_cfg(struct sf_fb_data *sf_dev) | ^~~~~~~~~~~~ 545 | int dsitx_dcs_write(struct sf_fb_data *sf_dev, int n, ...) | ^~~~~~~~~~~~~~~ 626 | void dpi_cfg(struct sf_fb_data *sf_dev, int int_en) { | ^~~~~~~ drivers/video/fbdev/starfive/starfive_mipi_tx.c: In function 'dsitx_dcs_write': drivers/video/fbdev/starfive/starfive_mipi_tx.c:564:5: warning: second parameter of 'va_start' not last named argument [-Wvarargs] 564 | va_start(ap, cmd_size); | ^~~~~~~~ -- 1058 | int tda998x_audio_mute_stream(struct device *dev, void *data, bool enable, | ^~~~~~~~~~~~~~~~~~~~~~~~~ vim +/ret +424 drivers/video/fbdev/starfive/starfive_fb.c ff76d633fd1fcc jack.zhu 2021-01-11 418 ff76d633fd1fcc jack.zhu 2021-01-11 419 static int sf_fb_setcolreg(u_int regno, u_int red, u_int green, ff76d633fd1fcc jack.zhu 2021-01-11 420 u_int blue, u_int trans, struct fb_info *info) ff76d633fd1fcc jack.zhu 2021-01-11 421 { ff76d633fd1fcc jack.zhu 2021-01-11 422 struct sf_fb_data *sf_dev = container_of(info, struct sf_fb_data, fb); ff76d633fd1fcc jack.zhu 2021-01-11 423 unsigned int val; ff76d633fd1fcc jack.zhu 2021-01-11 @424 int ret = 1; ff76d633fd1fcc jack.zhu 2021-01-11 425 ff76d633fd1fcc jack.zhu 2021-01-11 426 FB_PRT("%s,%d\n",__func__, __LINE__); ff76d633fd1fcc jack.zhu 2021-01-11 427 /* ff76d633fd1fcc jack.zhu 2021-01-11 428 * If inverse mode was selected, invert all the colours ff76d633fd1fcc jack.zhu 2021-01-11 429 * rather than the register number. The register number ff76d633fd1fcc jack.zhu 2021-01-11 430 * is what you poke into the framebuffer to produce the ff76d633fd1fcc jack.zhu 2021-01-11 431 * colour you requested. ff76d633fd1fcc jack.zhu 2021-01-11 432 */ ff76d633fd1fcc jack.zhu 2021-01-11 433 if (sf_dev->cmap_inverse) { ff76d633fd1fcc jack.zhu 2021-01-11 434 red = 0xffff - red; ff76d633fd1fcc jack.zhu 2021-01-11 435 green = 0xffff - green; ff76d633fd1fcc jack.zhu 2021-01-11 436 blue = 0xffff - blue; ff76d633fd1fcc jack.zhu 2021-01-11 437 } ff76d633fd1fcc jack.zhu 2021-01-11 438 ff76d633fd1fcc jack.zhu 2021-01-11 439 /* ff76d633fd1fcc jack.zhu 2021-01-11 440 * If greyscale is true, then we convert the RGB value ff76d633fd1fcc jack.zhu 2021-01-11 441 * to greyscale no matter what visual we are using. ff76d633fd1fcc jack.zhu 2021-01-11 442 */ ff76d633fd1fcc jack.zhu 2021-01-11 443 if (sf_dev->fb.var.grayscale) ff76d633fd1fcc jack.zhu 2021-01-11 444 red = green = blue = (19595 * red + 38470 * green + ff76d633fd1fcc jack.zhu 2021-01-11 445 7471 * blue) >> 16; ff76d633fd1fcc jack.zhu 2021-01-11 446 ff76d633fd1fcc jack.zhu 2021-01-11 447 switch (sf_dev->fb.fix.visual) { ff76d633fd1fcc jack.zhu 2021-01-11 448 case FB_VISUAL_TRUECOLOR: ff76d633fd1fcc jack.zhu 2021-01-11 449 /* ff76d633fd1fcc jack.zhu 2021-01-11 450 * 16-bit True Colour. We encode the RGB value ff76d633fd1fcc jack.zhu 2021-01-11 451 * according to the RGB bitfield information. ff76d633fd1fcc jack.zhu 2021-01-11 452 */ ff76d633fd1fcc jack.zhu 2021-01-11 453 if (regno < 16) { ff76d633fd1fcc jack.zhu 2021-01-11 454 u32 *pal = sf_dev->fb.pseudo_palette; ff76d633fd1fcc jack.zhu 2021-01-11 455 ff76d633fd1fcc jack.zhu 2021-01-11 456 val = sf_chan_to_field(red, &sf_dev->fb.var.red); ff76d633fd1fcc jack.zhu 2021-01-11 457 val |= sf_chan_to_field(green, &sf_dev->fb.var.green); ff76d633fd1fcc jack.zhu 2021-01-11 458 val |= sf_chan_to_field(blue, &sf_dev->fb.var.blue); ff76d633fd1fcc jack.zhu 2021-01-11 459 ff76d633fd1fcc jack.zhu 2021-01-11 460 pal[regno] = val; ff76d633fd1fcc jack.zhu 2021-01-11 461 ret = 0; ff76d633fd1fcc jack.zhu 2021-01-11 462 } ff76d633fd1fcc jack.zhu 2021-01-11 463 break; ff76d633fd1fcc jack.zhu 2021-01-11 464 ff76d633fd1fcc jack.zhu 2021-01-11 465 case FB_VISUAL_STATIC_PSEUDOCOLOR: ff76d633fd1fcc jack.zhu 2021-01-11 466 case FB_VISUAL_PSEUDOCOLOR: ff76d633fd1fcc jack.zhu 2021-01-11 467 /* haven't support this function. */ ff76d633fd1fcc jack.zhu 2021-01-11 468 break; ff76d633fd1fcc jack.zhu 2021-01-11 469 } ff76d633fd1fcc jack.zhu 2021-01-11 470 ff76d633fd1fcc jack.zhu 2021-01-11 471 return 0; ff76d633fd1fcc jack.zhu 2021-01-11 472 } ff76d633fd1fcc jack.zhu 2021-01-11 473 :::::: The code at line 424 was first introduced by commit :::::: ff76d633fd1fcc65037af860aa54b91cac544d2d drivers/video/fbdev and drivers/media/platform: starfive drivers added :::::: TO: jack.zhu <jack.zhu(a)starfivetech.com&gt; :::::: CC: Emil Renner Berthing <kernel(a)esmil.dk&gt; --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org