Hi Jacopo, kernel test robot noticed the following build warnings: [auto build test WARNING on media-tree/master] [also build test WARNING on linus/master v6.4 next-20230704] [cannot apply to sailus-media-tree/streams] [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#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Jacopo-Mondi/media-i2c-imx219-Rename-mbus-codes-array/20230704-184252 base: git://linuxtv.org/media_tree.git master patch link: https://lore.kernel.org/r/20230704104057.149837-6-jacopo.mondi%40ideasonboard.com patch subject: [PATCH 5/5] media: i2c: imx219: Simplify code handling in s_fmt config: i386-allyesconfig (https://download.01.org/0day-ci/archive/20230704/202307042337.lqiNqeJW-lkp@xxxxxxxxx/config) compiler: gcc-12 (Debian 12.2.0-14) 12.2.0 reproduce: (https://download.01.org/0day-ci/archive/20230704/202307042337.lqiNqeJW-lkp@xxxxxxxxx/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202307042337.lqiNqeJW-lkp@xxxxxxxxx/ All warnings (new ones prefixed by >>): drivers/media/i2c/imx219.c: In function 'imx219_set_pad_format': >> drivers/media/i2c/imx219.c:755:22: warning: unused variable 'i' [-Wunused-variable] 755 | unsigned int i; | ^ vim +/i +755 drivers/media/i2c/imx219.c 1283b3b8f82b90 Dave Stevenson 2020-01-20 747 1283b3b8f82b90 Dave Stevenson 2020-01-20 748 static int imx219_set_pad_format(struct v4l2_subdev *sd, 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 749 struct v4l2_subdev_state *sd_state, 1283b3b8f82b90 Dave Stevenson 2020-01-20 750 struct v4l2_subdev_format *fmt) 1283b3b8f82b90 Dave Stevenson 2020-01-20 751 { 1283b3b8f82b90 Dave Stevenson 2020-01-20 752 struct imx219 *imx219 = to_imx219(sd); 1283b3b8f82b90 Dave Stevenson 2020-01-20 753 const struct imx219_mode *mode; 1283b3b8f82b90 Dave Stevenson 2020-01-20 754 int exposure_max, exposure_def, hblank; 22da1d56e98215 Lad Prabhakar 2020-03-10 @755 unsigned int i; 1283b3b8f82b90 Dave Stevenson 2020-01-20 756 1283b3b8f82b90 Dave Stevenson 2020-01-20 757 mode = v4l2_find_nearest_size(supported_modes, 1283b3b8f82b90 Dave Stevenson 2020-01-20 758 ARRAY_SIZE(supported_modes), 1283b3b8f82b90 Dave Stevenson 2020-01-20 759 width, height, 1283b3b8f82b90 Dave Stevenson 2020-01-20 760 fmt->format.width, fmt->format.height); 563219f153d5f7 Jacopo Mondi 2023-07-04 761 7471d0495584f7 Jacopo Mondi 2023-07-04 762 imx219_update_pad_format(imx219, mode, &fmt->format, fmt->format.code); 563219f153d5f7 Jacopo Mondi 2023-07-04 763 563219f153d5f7 Jacopo Mondi 2023-07-04 764 if (fmt->which == V4L2_SUBDEV_FORMAT_ACTIVE) { 1283b3b8f82b90 Dave Stevenson 2020-01-20 765 imx219->mode = mode; 1283b3b8f82b90 Dave Stevenson 2020-01-20 766 /* Update limits and set FPS to default */ 1283b3b8f82b90 Dave Stevenson 2020-01-20 767 __v4l2_ctrl_modify_range(imx219->vblank, IMX219_VBLANK_MIN, 1283b3b8f82b90 Dave Stevenson 2020-01-20 768 IMX219_VTS_MAX - mode->height, 1, 1283b3b8f82b90 Dave Stevenson 2020-01-20 769 mode->vts_def - mode->height); 1283b3b8f82b90 Dave Stevenson 2020-01-20 770 __v4l2_ctrl_s_ctrl(imx219->vblank, 1283b3b8f82b90 Dave Stevenson 2020-01-20 771 mode->vts_def - mode->height); 1283b3b8f82b90 Dave Stevenson 2020-01-20 772 /* Update max exposure while meeting expected vblanking */ 1283b3b8f82b90 Dave Stevenson 2020-01-20 773 exposure_max = mode->vts_def - 4; 1283b3b8f82b90 Dave Stevenson 2020-01-20 774 exposure_def = (exposure_max < IMX219_EXPOSURE_DEFAULT) ? 1283b3b8f82b90 Dave Stevenson 2020-01-20 775 exposure_max : IMX219_EXPOSURE_DEFAULT; 1283b3b8f82b90 Dave Stevenson 2020-01-20 776 __v4l2_ctrl_modify_range(imx219->exposure, 1283b3b8f82b90 Dave Stevenson 2020-01-20 777 imx219->exposure->minimum, 1283b3b8f82b90 Dave Stevenson 2020-01-20 778 exposure_max, imx219->exposure->step, 1283b3b8f82b90 Dave Stevenson 2020-01-20 779 exposure_def); 1283b3b8f82b90 Dave Stevenson 2020-01-20 780 /* 1283b3b8f82b90 Dave Stevenson 2020-01-20 781 * Currently PPL is fixed to IMX219_PPL_DEFAULT, so hblank 1283b3b8f82b90 Dave Stevenson 2020-01-20 782 * depends on mode->width only, and is not changeble in any 1283b3b8f82b90 Dave Stevenson 2020-01-20 783 * way other than changing the mode. 1283b3b8f82b90 Dave Stevenson 2020-01-20 784 */ 1283b3b8f82b90 Dave Stevenson 2020-01-20 785 hblank = IMX219_PPL_DEFAULT - mode->width; 1283b3b8f82b90 Dave Stevenson 2020-01-20 786 __v4l2_ctrl_modify_range(imx219->hblank, hblank, hblank, 1, 1283b3b8f82b90 Dave Stevenson 2020-01-20 787 hblank); 1283b3b8f82b90 Dave Stevenson 2020-01-20 788 } 1283b3b8f82b90 Dave Stevenson 2020-01-20 789 563219f153d5f7 Jacopo Mondi 2023-07-04 790 *v4l2_subdev_get_pad_format(sd, sd_state, 0) = fmt->format; 1283b3b8f82b90 Dave Stevenson 2020-01-20 791 1283b3b8f82b90 Dave Stevenson 2020-01-20 792 return 0; 1283b3b8f82b90 Dave Stevenson 2020-01-20 793 } 1283b3b8f82b90 Dave Stevenson 2020-01-20 794 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki