| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <202512102347.N5BM3QL7-lkp@intel.com>
Date: Thu, 11 Dec 2025 00:00:54 +0800
From: kernel test robot <lkp@...el.com>
To: Himanshu Bhavani <himanshu.bhavani@...iconsignals.io>,
sakari.ailus@...ux.intel.com, krzk+dt@...nel.org, robh@...nel.org
Cc: llvm@...ts.linux.dev, oe-kbuild-all@...ts.linux.dev,
Himanshu Bhavani <himanshu.bhavani@...iconsignals.io>,
Elgin Perumbilly <elgin.perumbilly@...iconsignals.io>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
linux-media@...r.kernel.org, Conor Dooley <conor+dt@...nel.org>,
Hans Verkuil <hverkuil@...all.nl>, Hans de Goede <hansg@...nel.org>,
Mehdi Djait <mehdi.djait@...ux.intel.com>,
Ricardo Ribalda <ribalda@...omium.org>,
Laurent Pinchart <laurent.pinchart@...asonboard.com>,
Vladimir Zapolskiy <vladimir.zapolskiy@...aro.org>,
André Apitzsch <git@...tzsch.eu>,
Dongcheng Yan <dongcheng.yan@...el.com>,
Sylvain Petinot <sylvain.petinot@...s.st.com>,
Benjamin Mugnier <benjamin.mugnier@...s.st.com>,
Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
Hardevsinh Palaniya <hardevsinh.palaniya@...iconsignals.io>,
Jingjing Xiong <jingjing.xiong@...el.com>,
Heimir Thor Sverrisson <heimir.sverrisson@...il.com>,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] media: i2c: add os05b10 image sensor driver
Hi Himanshu,
kernel test robot noticed the following build warnings:
[auto build test WARNING on sailus-media-tree/master]
[also build test WARNING on robh/for-next linus/master v6.18 next-20251210]
[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/Himanshu-Bhavani/dt-bindings-media-i2c-Add-os05b10-sensor/20251209-180746
base: git://linuxtv.org/sailus/media_tree.git master
patch link: https://lore.kernel.org/r/20251209100017.302518-3-himanshu.bhavani%40siliconsignals.io
patch subject: [PATCH 2/2] media: i2c: add os05b10 image sensor driver
config: um-allmodconfig (https://download.01.org/0day-ci/archive/20251210/202512102347.N5BM3QL7-lkp@intel.com/config)
compiler: clang version 19.1.7 (https://github.com/llvm/llvm-project cd708029e0b2869e80abe31ddb175f7c35361f90)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251210/202512102347.N5BM3QL7-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@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202512102347.N5BM3QL7-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from drivers/media/i2c/os05b10.c:19:
In file included from include/linux/i2c.h:19:
In file included from include/linux/regulator/consumer.h:35:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:27:
In file included from include/linux/kernel_stat.h:8:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:11:
In file included from arch/um/include/asm/hardirq.h:5:
In file included from include/asm-generic/hardirq.h:17:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:12:
In file included from arch/um/include/asm/io.h:24:
include/asm-generic/io.h:1209:55: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
1209 | return (port > MMIO_UPPER_LIMIT) ? NULL : PCI_IOBASE + port;
| ~~~~~~~~~~ ^
>> drivers/media/i2c/os05b10.c:986:44: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
986 | "failed to initialize CCI: %d\n", ret);
| ^~~
drivers/media/i2c/os05b10.c:972:9: note: initialize the variable 'ret' to silence this warning
972 | int ret;
| ^
| = 0
2 warnings generated.
vim +/ret +986 drivers/media/i2c/os05b10.c
967
968 static int os05b10_probe(struct i2c_client *client)
969 {
970 struct os05b10 *os05b10;
971 unsigned int xclk_freq;
972 int ret;
973
974 os05b10 = devm_kzalloc(&client->dev, sizeof(*os05b10), GFP_KERNEL);
975 if (!os05b10)
976 return -ENOMEM;
977
978 os05b10->client = client;
979 os05b10->dev = &client->dev;
980
981 v4l2_i2c_subdev_init(&os05b10->sd, client, &os05b10_subdev_ops);
982
983 os05b10->cci = devm_cci_regmap_init_i2c(client, 16);
984 if (IS_ERR(os05b10->cci))
985 return dev_err_probe(os05b10->dev, PTR_ERR(os05b10->cci),
> 986 "failed to initialize CCI: %d\n", ret);
987
988 /* Get system clock (xclk) */
989 os05b10->xclk = devm_v4l2_sensor_clk_get(os05b10->dev, NULL);
990 if (IS_ERR(os05b10->xclk))
991 return dev_err_probe(os05b10->dev, PTR_ERR(os05b10->xclk),
992 "failed to get xclk\n");
993
994 xclk_freq = clk_get_rate(os05b10->xclk);
995 if (xclk_freq != OS05B10_XCLK_FREQ)
996 return dev_err_probe(os05b10->dev, -EINVAL,
997 "xclk frequency not supported: %d Hz\n",
998 xclk_freq);
999
1000 ret = os05b10_get_regulators(os05b10);
1001 if (ret)
1002 return dev_err_probe(os05b10->dev, ret, "failed to get regulators\n");
1003
1004 ret = os05b10_parse_endpoint(os05b10);
1005 if (ret) {
1006 dev_err(os05b10->dev, "failed to parse endpoint configuration\n");
1007 return ret;
1008 }
1009
1010 os05b10->reset_gpio = devm_gpiod_get(&client->dev, "reset", GPIOD_OUT_LOW);
1011 if (IS_ERR(os05b10->reset_gpio))
1012 return dev_err_probe(os05b10->dev, PTR_ERR(os05b10->reset_gpio),
1013 "failed to get reset GPIO\n");
1014
1015 ret = os05b10_power_on(os05b10->dev);
1016 if (ret)
1017 return ret;
1018
1019 ret = os05b10_identify_module(os05b10);
1020 if (ret)
1021 goto error_power_off;
1022
1023 /* This needs the pm runtime to be registered. */
1024 ret = os05b10_init_controls(os05b10);
1025 if (ret)
1026 goto error_power_off;
1027
1028 /* Initialize subdev */
1029 os05b10->sd.flags |= V4L2_SUBDEV_FL_HAS_DEVNODE;
1030 os05b10->sd.entity.function = MEDIA_ENT_F_CAM_SENSOR;
1031 os05b10->sd.internal_ops = &os05b10_internal_ops;
1032 os05b10->pad.flags = MEDIA_PAD_FL_SOURCE;
1033
1034 ret = media_entity_pads_init(&os05b10->sd.entity, 1, &os05b10->pad);
1035 if (ret) {
1036 dev_err_probe(os05b10->dev, ret, "failed to init entity pads\n");
1037 goto error_handler_free;
1038 }
1039
1040 os05b10->sd.state_lock = os05b10->handler.lock;
1041 ret = v4l2_subdev_init_finalize(&os05b10->sd);
1042 if (ret < 0) {
1043 dev_err_probe(os05b10->dev, ret, "subdev init error\n");
1044 goto error_media_entity;
1045 }
1046
1047 pm_runtime_set_active(os05b10->dev);
1048 pm_runtime_enable(os05b10->dev);
1049
1050 ret = v4l2_async_register_subdev_sensor(&os05b10->sd);
1051 if (ret < 0) {
1052 dev_err_probe(os05b10->dev, ret,
1053 "failed to register os05b10 sub-device\n");
1054 goto error_subdev_cleanup;
1055 }
1056
1057 pm_runtime_idle(os05b10->dev);
1058
1059 return 0;
1060
1061 error_subdev_cleanup:
1062 v4l2_subdev_cleanup(&os05b10->sd);
1063 pm_runtime_disable(os05b10->dev);
1064 pm_runtime_set_suspended(os05b10->dev);
1065
1066 error_media_entity:
1067 media_entity_cleanup(&os05b10->sd.entity);
1068
1069 error_handler_free:
1070 v4l2_ctrl_handler_free(os05b10->sd.ctrl_handler);
1071
1072 error_power_off:
1073 os05b10_power_off(os05b10->dev);
1074
1075 return ret;
1076 }
1077
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists