Re: [PATCH v6 10/10] media: i2c: Add driver for mlx7502x ToF sensor

[kernel test robot <lkp@intel.com>]

Hi Volodymyr,

kernel test robot noticed the following build warnings:

[auto build test WARNING on 8771b7f31b7fff91a998e6afdb60650d4bac59a5]

url:    https://github.com/intel-lab-lkp/linux/commits/Volodymyr-Kharuk/media-uapi-ctrls-Add-camera-trigger-controls/20240517-004536
base:   8771b7f31b7fff91a998e6afdb60650d4bac59a5
patch link:    https://lore.kernel.org/r/6f666d475da17b3469fd0471531bc615f6fd797a.1715871189.git.vkh%40melexis.com
patch subject: [PATCH v6 10/10] media: i2c: Add driver for mlx7502x ToF sensor
config: csky-randconfig-r123-20240518 (https://download.01.org/0day-ci/archive/20240518/202405181143.nHERYo41-lkp@intel.com/config)
compiler: csky-linux-gcc (GCC) 13.2.0
reproduce: (https://download.01.org/0day-ci/archive/20240518/202405181143.nHERYo41-lkp@intel.com/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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202405181143.nHERYo41-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/media/i2c/mlx7502x.c:480:12: warning: 'mlx7502x_read' defined but not used [-Wunused-function]
     480 | static int mlx7502x_read(struct v4l2_subdev *sd, u16 reg, u8 *val, int val_size)
         |            ^~~~~~~~~~~~~


vim +/mlx7502x_read +480 drivers/media/i2c/mlx7502x.c

   479	
 > 480	static int mlx7502x_read(struct v4l2_subdev *sd, u16 reg, u8 *val, int val_size)
   481	{
   482		int ret;
   483		unsigned char data_w[2];
   484		struct i2c_client *client = v4l2_get_subdevdata(sd);
   485	
   486		struct i2c_msg msg[] = {
   487			{
   488				.addr = client->addr,
   489				.flags = 0,
   490				.len = 2,
   491				.buf = data_w,
   492			}, {
   493				.addr = client->addr,
   494				.flags = I2C_M_RD,
   495				.len = val_size,
   496				.buf = val,
   497			}
   498		};
   499	
   500		/* write reg address into first msg */
   501		put_unaligned_be16(reg, data_w);
   502	
   503		/* Using transfer allows skip STOP between messages
   504		 * so we have repeated Start here
   505		 */
   506		ret = i2c_transfer(client->adapter, msg, 2);
   507	
   508		return ret != 2 ? -EIO : 0;
   509	}
   510	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki