Hi Tommaso, kernel test robot noticed the following build warnings: [auto build test WARNING on media-tree/master] [also build test WARNING on robh/for-next linus/master v6.4-rc5 next-20230608] [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/Tommaso-Merciai/dt-bindings-vendor-prefixes-Add-prefix-alliedvision/20230608-163423 base: git://linuxtv.org/media_tree.git master patch link: https://lore.kernel.org/r/20230608083127.545750-4-tomm.merciai%40gmail.com patch subject: [PATCH v5 3/3] media: i2c: Add support for alvium camera config: alpha-allyesconfig (https://download.01.org/0day-ci/archive/20230609/202306090401.jEkSrYuJ-lkp@xxxxxxxxx/config) compiler: alpha-linux-gcc (GCC) 12.3.0 reproduce (this is a W=1 build): mkdir -p ~/bin wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git remote add media-tree git://linuxtv.org/media_tree.git git fetch media-tree master git checkout media-tree/master b4 shazam https://lore.kernel.org/r/20230608083127.545750-4-tomm.merciai@xxxxxxxxx # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.3.0 ~/bin/make.cross W=1 O=build_dir ARCH=alpha olddefconfig COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.3.0 ~/bin/make.cross W=1 O=build_dir ARCH=alpha SHELL=/bin/bash drivers/media/i2c/ 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/202306090401.jEkSrYuJ-lkp@xxxxxxxxx/ All warnings (new ones prefixed by >>): drivers/media/i2c/alvium-csi2.c: In function 'alvium_get_dt_data': >> drivers/media/i2c/alvium-csi2.c:3245:13: warning: variable 'ret' set but not used [-Wunused-but-set-variable] 3245 | int ret = 0; | ^~~ vim +/ret +3245 drivers/media/i2c/alvium-csi2.c 3239 3240 static int alvium_get_dt_data(struct alvium_dev *alvium) 3241 { 3242 struct device *dev = &alvium->i2c_client->dev; 3243 struct device_node *node = dev->of_node; 3244 struct fwnode_handle *endpoint; > 3245 int ret = 0; 3246 3247 if (!node) 3248 return -EINVAL; 3249 3250 ret = fwnode_property_read_u32(dev_fwnode(dev), 3251 "alliedvision,lp2hs-delay-us", 3252 &alvium->lp2hs_delay); 3253 3254 endpoint = fwnode_graph_get_next_endpoint(dev_fwnode(dev), NULL); 3255 if (!endpoint) { 3256 dev_err(dev, "endpoint node not found\n"); 3257 return -EINVAL; 3258 } 3259 3260 if (v4l2_fwnode_endpoint_alloc_parse(endpoint, &alvium->ep)) { 3261 dev_err(dev, "could not parse endpoint\n"); 3262 return 0; 3263 } 3264 3265 if (alvium->ep.bus_type != V4L2_MBUS_CSI2_DPHY) { 3266 dev_err(dev, "unsupported bus type\n"); 3267 return -EINVAL; 3268 } 3269 3270 if (!alvium->ep.nr_of_link_frequencies) { 3271 dev_err(dev, "no link frequencies defined"); 3272 return -EINVAL; 3273 } 3274 3275 dev_info(dev, "freq: %llu\n", 3276 alvium->ep.link_frequencies[0]); 3277 dev_info(dev, "lanes: %d\n", 3278 alvium->ep.bus.mipi_csi2.num_data_lanes); 3279 3280 return 0; 3281 } 3282 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
