Re: [PATCH V1 5/5] Add support for Awinic sar sensor.

From: Dan Carpenter
Date: Fri May 31 2024 - 04:13:41 EST

Next message: Huang Ying: "[PATCH 1/3] mm,swap: fix a theoretical underflow in readahead window calculation"
Previous message: Huang Ying: "[PATCH 0/3] mm,swap: cleanup VMA based swap readahead window calculation"
In reply to: kernel test robot: "Re: [PATCH V1 5/5] Add support for Awinic sar sensor."
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi,

kernel test robot noticed the following build warnings:

url: https://github.com/intel-lab-lkp/linux/commits/wangshuaijie-awinic-com/dt-bindings-input-Add-YAML-to-Awinic-sar-sensor/20240529-211303
base: e0cce98fe279b64f4a7d81b7f5c3a23d80b92fbc
patch link: https://lore.kernel.org/r/20240529130608.783624-6-wangshuaijie%40awinic.com
patch subject: [PATCH V1 5/5] Add support for Awinic sar sensor.
config: riscv-randconfig-r071-20240530 (https://download.01.org/0day-ci/archive/20240531/202405310138.ry5jf9hL-lkp@xxxxxxxxx/config)
compiler: riscv64-linux-gcc (GCC) 13.2.0

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>
| Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
| Closes: https://lore.kernel.org/r/202405310138.ry5jf9hL-lkp@xxxxxxxxx/

New smatch warnings:
drivers/input/misc/aw_sar/aw_sar.c:221 aw_sar_load_bin_comm() warn: 'fw' from request_firmware() not released on lines: 217.
drivers/input/misc/aw_sar/aw_sar.c:283 aw_sar_irq() error: uninitialized symbol 'irq_status'.
drivers/input/misc/aw_sar/./aw9610x/aw9610x.c:479 aw9610x_get_chip_version() error: __builtin_memcpy() 'aw9610x->chip_name[__builtin_choose_expr((4 == 1), __builtin_strlen(aw9610x->chip_name), __fortify_strlen(aw9610x->chip_name))]' too small (1 vs 2)
drivers/input/misc/aw_sar/./aw963xx/aw963xx.c:227 aw963xx_sram_data_write() error: uninitialized symbol 'ret'.
drivers/input/misc/aw_sar/./aw963xx/aw963xx.c:522 aw963xx_get_cap_offset() warn: inconsistent indenting

Old smatch warnings:
drivers/input/misc/aw_sar/./aw963xx/aw963xx.c:557 aw963xx_get_cap_offset() warn: inconsistent indenting

vim +/fw +221 drivers/input/misc/aw_sar/aw_sar.c

e5df082e247559 shuaijie wang 2024-05-29 203 static int32_t aw_sar_load_bin_comm(struct aw_sar *p_sar)
e5df082e247559 shuaijie wang 2024-05-29 204 {
e5df082e247559 shuaijie wang 2024-05-29 205 const struct firmware *fw;
e5df082e247559 shuaijie wang 2024-05-29 206 int32_t ret;
e5df082e247559 shuaijie wang 2024-05-29 207
e5df082e247559 shuaijie wang 2024-05-29 208 ret = request_firmware(&fw, p_sar->load_bin.bin_name, p_sar->dev);
e5df082e247559 shuaijie wang 2024-05-29 209 if (ret != 0) {
e5df082e247559 shuaijie wang 2024-05-29 210 dev_err(p_sar->dev, "parse %s error!", p_sar->load_bin.bin_name);
e5df082e247559 shuaijie wang 2024-05-29 211 return ret;
e5df082e247559 shuaijie wang 2024-05-29 212 }
e5df082e247559 shuaijie wang 2024-05-29 213
e5df082e247559 shuaijie wang 2024-05-29 214 ret = aw_sar_parse_bin(fw, p_sar);
e5df082e247559 shuaijie wang 2024-05-29 215 if (ret != 0) {
e5df082e247559 shuaijie wang 2024-05-29 216 dev_err(p_sar->dev, "reg_bin %s load error!", p_sar->load_bin.bin_name);

release_firmware(fw);

e5df082e247559 shuaijie wang 2024-05-29 217 return ret;
e5df082e247559 shuaijie wang 2024-05-29 218 }
e5df082e247559 shuaijie wang 2024-05-29 219 release_firmware(fw);
e5df082e247559 shuaijie wang 2024-05-29 220
e5df082e247559 shuaijie wang 2024-05-29 @221 return 0;
e5df082e247559 shuaijie wang 2024-05-29 222 }
e5df082e247559 shuaijie wang 2024-05-29 223
e5df082e247559 shuaijie wang 2024-05-29 224 static int32_t aw_sar_parse_dts_comm(struct device *dev, struct device_node *np,
e5df082e247559 shuaijie wang 2024-05-29 225 struct aw_sar_dts_info *p_dts_info)
e5df082e247559 shuaijie wang 2024-05-29 226 {
e5df082e247559 shuaijie wang 2024-05-29 227 int32_t val;
e5df082e247559 shuaijie wang 2024-05-29 228
e5df082e247559 shuaijie wang 2024-05-29 229 val = of_property_read_u32(np, "sar-num", &p_dts_info->sar_num);
e5df082e247559 shuaijie wang 2024-05-29 230 dev_info(dev, "sar num = %d", p_dts_info->sar_num);
e5df082e247559 shuaijie wang 2024-05-29 231 if (val != 0) {
e5df082e247559 shuaijie wang 2024-05-29 232 dev_err(dev, "multiple sar failed!");
e5df082e247559 shuaijie wang 2024-05-29 233 return -EINVAL;
e5df082e247559 shuaijie wang 2024-05-29 234 }
e5df082e247559 shuaijie wang 2024-05-29 235
e5df082e247559 shuaijie wang 2024-05-29 236 p_dts_info->irq_gpio = of_get_named_gpio(np, "irq-gpio", 0);
e5df082e247559 shuaijie wang 2024-05-29 237 if (p_dts_info->irq_gpio < 0) {
e5df082e247559 shuaijie wang 2024-05-29 238 p_dts_info->irq_gpio = -1;
e5df082e247559 shuaijie wang 2024-05-29 239 dev_err(dev, "no irq gpio provided.");
e5df082e247559 shuaijie wang 2024-05-29 240 return -EINVAL;
e5df082e247559 shuaijie wang 2024-05-29 241 }
e5df082e247559 shuaijie wang 2024-05-29 242
e5df082e247559 shuaijie wang 2024-05-29 243 val = of_property_read_u32(np, "channel_use_flag", &p_dts_info->channel_use_flag);
e5df082e247559 shuaijie wang 2024-05-29 244 if (val != 0) {
e5df082e247559 shuaijie wang 2024-05-29 245 dev_err(dev, "channel_use_flag failed!");
e5df082e247559 shuaijie wang 2024-05-29 246 return -EINVAL;
e5df082e247559 shuaijie wang 2024-05-29 247 }
e5df082e247559 shuaijie wang 2024-05-29 248
e5df082e247559 shuaijie wang 2024-05-29 249 //GPIO is set as internal pull-up input
e5df082e247559 shuaijie wang 2024-05-29 250 p_dts_info->use_inter_pull_up = of_property_read_bool(np, "aw_sar,pin_set_inter_pull-up");
e5df082e247559 shuaijie wang 2024-05-29 251 p_dts_info->use_pm = of_property_read_bool(np, "aw_sar,using_pm_ops");
e5df082e247559 shuaijie wang 2024-05-29 252 p_dts_info->update_fw_flag = of_property_read_bool(np, "aw_sar,update_fw");
e5df082e247559 shuaijie wang 2024-05-29 253 p_dts_info->use_plug_cail_flag = of_property_read_bool(np, "aw_sar,use_plug_cail");
e5df082e247559 shuaijie wang 2024-05-29 254 p_dts_info->monitor_esd_flag = of_property_read_bool(np, "aw_sar,monitor_esd");
e5df082e247559 shuaijie wang 2024-05-29 255
e5df082e247559 shuaijie wang 2024-05-29 256 return 0;
e5df082e247559 shuaijie wang 2024-05-29 257 }
e5df082e247559 shuaijie wang 2024-05-29 258
e5df082e247559 shuaijie wang 2024-05-29 259 static int32_t aw_sar_parse_dts(struct aw_sar *p_sar)
e5df082e247559 shuaijie wang 2024-05-29 260 {
e5df082e247559 shuaijie wang 2024-05-29 261 int32_t ret;
e5df082e247559 shuaijie wang 2024-05-29 262
e5df082e247559 shuaijie wang 2024-05-29 263 ret = aw_sar_parse_dts_comm(p_sar->dev, p_sar->i2c->dev.of_node, &p_sar->dts_info);
e5df082e247559 shuaijie wang 2024-05-29 264
e5df082e247559 shuaijie wang 2024-05-29 265 //Special requirements of SAR chip
e5df082e247559 shuaijie wang 2024-05-29 266 if (p_sar->p_sar_para->p_platform_config->p_add_parse_dts_fn != NULL)
e5df082e247559 shuaijie wang 2024-05-29 267 ret |= p_sar->p_sar_para->p_platform_config->p_add_parse_dts_fn(p_sar);
e5df082e247559 shuaijie wang 2024-05-29 268
e5df082e247559 shuaijie wang 2024-05-29 269 return ret;
e5df082e247559 shuaijie wang 2024-05-29 270 }
e5df082e247559 shuaijie wang 2024-05-29 271
e5df082e247559 shuaijie wang 2024-05-29 272 static irqreturn_t aw_sar_irq(int32_t irq, void *data)
e5df082e247559 shuaijie wang 2024-05-29 273 {
e5df082e247559 shuaijie wang 2024-05-29 274 struct aw_sar *p_sar = (struct aw_sar *)data;
e5df082e247559 shuaijie wang 2024-05-29 275 uint32_t irq_status;
e5df082e247559 shuaijie wang 2024-05-29 276
e5df082e247559 shuaijie wang 2024-05-29 277 //step1: read clear interrupt
e5df082e247559 shuaijie wang 2024-05-29 278 if (p_sar->p_sar_para->p_platform_config->p_irq_init->rc_irq_fn != NULL)
e5df082e247559 shuaijie wang 2024-05-29 279 irq_status = p_sar->p_sar_para->p_platform_config->p_irq_init->rc_irq_fn(p_sar->i2c);
e5df082e247559 shuaijie wang 2024-05-29 280
e5df082e247559 shuaijie wang 2024-05-29 281 //step2: Read the status register for status reporting
e5df082e247559 shuaijie wang 2024-05-29 282 if (p_sar->p_sar_para->p_platform_config->p_irq_init->irq_spec_handler_fn != NULL)
e5df082e247559 shuaijie wang 2024-05-29 @283 p_sar->p_sar_para->p_platform_config->p_irq_init->irq_spec_handler_fn(irq_status,
e5df082e247559 shuaijie wang 2024-05-29 284 p_sar);

Probably if ->irq_spec_handler_fn is non-NULL then ->rc_irq_fn is also
non-NULL, but the static checker doesn't know that so it warns about
uninitialized variables.

e5df082e247559 shuaijie wang 2024-05-29 285
e5df082e247559 shuaijie wang 2024-05-29 286 //step3: The chip
e5df082e247559 shuaijie wang 2024-05-29 287
e5df082e247559 shuaijie wang 2024-05-29 288 if ((!p_sar->dts_info.monitor_esd_flag) && (p_sar->fault_flag == AW_SAR_UNHEALTHY)) {
e5df082e247559 shuaijie wang 2024-05-29 289 p_sar->fault_flag = AW_SAR_HEALTHY;
e5df082e247559 shuaijie wang 2024-05-29 290 disable_irq_nosync(p_sar->irq_init.to_irq);
e5df082e247559 shuaijie wang 2024-05-29 291 p_sar->irq_init.host_irq_stat = IRQ_DISABLE;
e5df082e247559 shuaijie wang 2024-05-29 292 //aw_sar_soft_reset(p_sar);
e5df082e247559 shuaijie wang 2024-05-29 293 schedule_delayed_work(&p_sar->update_work, msecs_to_jiffies(500));
e5df082e247559 shuaijie wang 2024-05-29 294 }
e5df082e247559 shuaijie wang 2024-05-29 295
e5df082e247559 shuaijie wang 2024-05-29 296 return IRQ_HANDLED;
e5df082e247559 shuaijie wang 2024-05-29 297 }

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

Next message: Huang Ying: "[PATCH 1/3] mm,swap: fix a theoretical underflow in readahead window calculation"
Previous message: Huang Ying: "[PATCH 0/3] mm,swap: cleanup VMA based swap readahead window calculation"
In reply to: kernel test robot: "Re: [PATCH V1 5/5] Add support for Awinic sar sensor."
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]