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

From: wangshuaijie
Date: Wed Jun 05 2024 - 04:42:40 EST

Next message: Dmitry Baryshkov: "Re: [PATCH v2 11/14] dt-bindings: mfd: pm8008: rework binding"
Previous message: Phil Chang: "[PATCH] hrtimer: check hrtimer with a NULL function"
Next in thread: wangshuaijie: "Re: [PATCH V1 5/5] Add support for Awinic sar sensor."
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Rob Herring,

On Fri, 31 May 2024 11:12:51 +0300, dan.carpenter@xxxxxxxxxx wrote:
>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
>

The patch for version v2 will fix these issues.

>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);
>

The patch for version v2 will fix this issue.

>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.
>

The patch for version v2 will fix this issue.

>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
--

Thanks,

Wang Shuaijie

Next message: Dmitry Baryshkov: "Re: [PATCH v2 11/14] dt-bindings: mfd: pm8008: rework binding"
Previous message: Phil Chang: "[PATCH] hrtimer: check hrtimer with a NULL function"
Next in thread: wangshuaijie: "Re: [PATCH V1 5/5] Add support for Awinic sar sensor."
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]