* sound/soc/codecs/aw88399.c:1700 aw88399_request_firmware_file() warn: '&cont' from request_firmware() not released on lines: 1681,1690,1700.
@ 2023-11-08 19:09 kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2023-11-08 19:09 UTC (permalink / raw)
To: oe-kbuild; +Cc: lkp, Dan Carpenter
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Weidong Wang <wangweidong.a@awinic.com>
CC: Mark Brown <broonie@kernel.org>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 90450a06162e6c71ab813ea22a83196fe7cff4bc
commit: 8ade6cc7e26175529474690fdbc1965901ec2024 ASoC: codecs: Add aw88399 amplifier driver
date: 2 weeks ago
:::::: branch date: 71 minutes ago
:::::: commit date: 2 weeks ago
config: i386-randconfig-141-20231108 (https://download.01.org/0day-ci/archive/20231109/202311090211.W4fxFXHT-lkp@intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce: (https://download.01.org/0day-ci/archive/20231109/202311090211.W4fxFXHT-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>
| Reported-by: Dan Carpenter <error27@gmail.com>
| Closes: https://lore.kernel.org/r/202311090211.W4fxFXHT-lkp@intel.com/
New smatch warnings:
sound/soc/codecs/aw88399.c:1700 aw88399_request_firmware_file() warn: '&cont' from request_firmware() not released on lines: 1681,1690,1700.
Old smatch warnings:
sound/soc/codecs/aw88399.c:441 aw_dev_set_vcalb() error: uninitialized symbol 'vsense_select'.
sound/soc/codecs/aw88399.c:963 aw_dev_check_sram() error: uninitialized symbol 'reg_val'.
vim +1700 sound/soc/codecs/aw88399.c
8ade6cc7e26175 Weidong Wang 2023-10-25 1660
8ade6cc7e26175 Weidong Wang 2023-10-25 1661 static int aw88399_request_firmware_file(struct aw88399 *aw88399)
8ade6cc7e26175 Weidong Wang 2023-10-25 1662 {
8ade6cc7e26175 Weidong Wang 2023-10-25 1663 const struct firmware *cont = NULL;
8ade6cc7e26175 Weidong Wang 2023-10-25 1664 int ret;
8ade6cc7e26175 Weidong Wang 2023-10-25 1665
8ade6cc7e26175 Weidong Wang 2023-10-25 1666 aw88399->aw_pa->fw_status = AW88399_DEV_FW_FAILED;
8ade6cc7e26175 Weidong Wang 2023-10-25 1667
8ade6cc7e26175 Weidong Wang 2023-10-25 1668 ret = request_firmware(&cont, AW88399_ACF_FILE, aw88399->aw_pa->dev);
8ade6cc7e26175 Weidong Wang 2023-10-25 1669 if (ret) {
8ade6cc7e26175 Weidong Wang 2023-10-25 1670 dev_err(aw88399->aw_pa->dev, "request [%s] failed!", AW88399_ACF_FILE);
8ade6cc7e26175 Weidong Wang 2023-10-25 1671 return ret;
8ade6cc7e26175 Weidong Wang 2023-10-25 1672 }
8ade6cc7e26175 Weidong Wang 2023-10-25 1673
8ade6cc7e26175 Weidong Wang 2023-10-25 1674 dev_dbg(aw88399->aw_pa->dev, "loaded %s - size: %zu\n",
8ade6cc7e26175 Weidong Wang 2023-10-25 1675 AW88399_ACF_FILE, cont ? cont->size : 0);
8ade6cc7e26175 Weidong Wang 2023-10-25 1676
8ade6cc7e26175 Weidong Wang 2023-10-25 1677 aw88399->aw_cfg = devm_kzalloc(aw88399->aw_pa->dev,
8ade6cc7e26175 Weidong Wang 2023-10-25 1678 struct_size(aw88399->aw_cfg, data, cont->size), GFP_KERNEL);
8ade6cc7e26175 Weidong Wang 2023-10-25 1679 if (!aw88399->aw_cfg) {
8ade6cc7e26175 Weidong Wang 2023-10-25 1680 release_firmware(cont);
8ade6cc7e26175 Weidong Wang 2023-10-25 1681 return -ENOMEM;
8ade6cc7e26175 Weidong Wang 2023-10-25 1682 }
8ade6cc7e26175 Weidong Wang 2023-10-25 1683 aw88399->aw_cfg->len = (int)cont->size;
8ade6cc7e26175 Weidong Wang 2023-10-25 1684 memcpy(aw88399->aw_cfg->data, cont->data, cont->size);
8ade6cc7e26175 Weidong Wang 2023-10-25 1685 release_firmware(cont);
8ade6cc7e26175 Weidong Wang 2023-10-25 1686
8ade6cc7e26175 Weidong Wang 2023-10-25 1687 ret = aw88395_dev_load_acf_check(aw88399->aw_pa, aw88399->aw_cfg);
8ade6cc7e26175 Weidong Wang 2023-10-25 1688 if (ret) {
8ade6cc7e26175 Weidong Wang 2023-10-25 1689 dev_err(aw88399->aw_pa->dev, "load [%s] failed!", AW88399_ACF_FILE);
8ade6cc7e26175 Weidong Wang 2023-10-25 1690 return ret;
8ade6cc7e26175 Weidong Wang 2023-10-25 1691 }
8ade6cc7e26175 Weidong Wang 2023-10-25 1692
8ade6cc7e26175 Weidong Wang 2023-10-25 1693 mutex_lock(&aw88399->lock);
8ade6cc7e26175 Weidong Wang 2023-10-25 1694 /* aw device init */
8ade6cc7e26175 Weidong Wang 2023-10-25 1695 ret = aw88399_dev_init(aw88399, aw88399->aw_cfg);
8ade6cc7e26175 Weidong Wang 2023-10-25 1696 if (ret)
8ade6cc7e26175 Weidong Wang 2023-10-25 1697 dev_err(aw88399->aw_pa->dev, "dev init failed");
8ade6cc7e26175 Weidong Wang 2023-10-25 1698 mutex_unlock(&aw88399->lock);
8ade6cc7e26175 Weidong Wang 2023-10-25 1699
8ade6cc7e26175 Weidong Wang 2023-10-25 @1700 return ret;
8ade6cc7e26175 Weidong Wang 2023-10-25 1701 }
8ade6cc7e26175 Weidong Wang 2023-10-25 1702
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 2+ messages in thread
* sound/soc/codecs/aw88399.c:1700 aw88399_request_firmware_file() warn: '&cont' from request_firmware() not released on lines: 1681,1690,1700.
@ 2023-11-09 19:30 kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2023-11-09 19:30 UTC (permalink / raw)
To: oe-kbuild; +Cc: lkp, Dan Carpenter
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Weidong Wang <wangweidong.a@awinic.com>
CC: Mark Brown <broonie@kernel.org>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 6bc986ab839c844e78a2333a02e55f02c9e57935
commit: 8ade6cc7e26175529474690fdbc1965901ec2024 ASoC: codecs: Add aw88399 amplifier driver
date: 2 weeks ago
:::::: branch date: 22 hours ago
:::::: commit date: 2 weeks ago
config: i386-randconfig-141-20231108 (https://download.01.org/0day-ci/archive/20231110/202311100355.TAxEyTmH-lkp@intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce: (https://download.01.org/0day-ci/archive/20231110/202311100355.TAxEyTmH-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>
| Reported-by: Dan Carpenter <error27@gmail.com>
| Closes: https://lore.kernel.org/r/202311100355.TAxEyTmH-lkp@intel.com/
New smatch warnings:
sound/soc/codecs/aw88399.c:1700 aw88399_request_firmware_file() warn: '&cont' from request_firmware() not released on lines: 1681,1690,1700.
Old smatch warnings:
sound/soc/codecs/aw88399.c:441 aw_dev_set_vcalb() error: uninitialized symbol 'vsense_select'.
sound/soc/codecs/aw88399.c:963 aw_dev_check_sram() error: uninitialized symbol 'reg_val'.
vim +1700 sound/soc/codecs/aw88399.c
8ade6cc7e26175 Weidong Wang 2023-10-25 1660
8ade6cc7e26175 Weidong Wang 2023-10-25 1661 static int aw88399_request_firmware_file(struct aw88399 *aw88399)
8ade6cc7e26175 Weidong Wang 2023-10-25 1662 {
8ade6cc7e26175 Weidong Wang 2023-10-25 1663 const struct firmware *cont = NULL;
8ade6cc7e26175 Weidong Wang 2023-10-25 1664 int ret;
8ade6cc7e26175 Weidong Wang 2023-10-25 1665
8ade6cc7e26175 Weidong Wang 2023-10-25 1666 aw88399->aw_pa->fw_status = AW88399_DEV_FW_FAILED;
8ade6cc7e26175 Weidong Wang 2023-10-25 1667
8ade6cc7e26175 Weidong Wang 2023-10-25 1668 ret = request_firmware(&cont, AW88399_ACF_FILE, aw88399->aw_pa->dev);
8ade6cc7e26175 Weidong Wang 2023-10-25 1669 if (ret) {
8ade6cc7e26175 Weidong Wang 2023-10-25 1670 dev_err(aw88399->aw_pa->dev, "request [%s] failed!", AW88399_ACF_FILE);
8ade6cc7e26175 Weidong Wang 2023-10-25 1671 return ret;
8ade6cc7e26175 Weidong Wang 2023-10-25 1672 }
8ade6cc7e26175 Weidong Wang 2023-10-25 1673
8ade6cc7e26175 Weidong Wang 2023-10-25 1674 dev_dbg(aw88399->aw_pa->dev, "loaded %s - size: %zu\n",
8ade6cc7e26175 Weidong Wang 2023-10-25 1675 AW88399_ACF_FILE, cont ? cont->size : 0);
8ade6cc7e26175 Weidong Wang 2023-10-25 1676
8ade6cc7e26175 Weidong Wang 2023-10-25 1677 aw88399->aw_cfg = devm_kzalloc(aw88399->aw_pa->dev,
8ade6cc7e26175 Weidong Wang 2023-10-25 1678 struct_size(aw88399->aw_cfg, data, cont->size), GFP_KERNEL);
8ade6cc7e26175 Weidong Wang 2023-10-25 1679 if (!aw88399->aw_cfg) {
8ade6cc7e26175 Weidong Wang 2023-10-25 1680 release_firmware(cont);
8ade6cc7e26175 Weidong Wang 2023-10-25 1681 return -ENOMEM;
8ade6cc7e26175 Weidong Wang 2023-10-25 1682 }
8ade6cc7e26175 Weidong Wang 2023-10-25 1683 aw88399->aw_cfg->len = (int)cont->size;
8ade6cc7e26175 Weidong Wang 2023-10-25 1684 memcpy(aw88399->aw_cfg->data, cont->data, cont->size);
8ade6cc7e26175 Weidong Wang 2023-10-25 1685 release_firmware(cont);
8ade6cc7e26175 Weidong Wang 2023-10-25 1686
8ade6cc7e26175 Weidong Wang 2023-10-25 1687 ret = aw88395_dev_load_acf_check(aw88399->aw_pa, aw88399->aw_cfg);
8ade6cc7e26175 Weidong Wang 2023-10-25 1688 if (ret) {
8ade6cc7e26175 Weidong Wang 2023-10-25 1689 dev_err(aw88399->aw_pa->dev, "load [%s] failed!", AW88399_ACF_FILE);
8ade6cc7e26175 Weidong Wang 2023-10-25 1690 return ret;
8ade6cc7e26175 Weidong Wang 2023-10-25 1691 }
8ade6cc7e26175 Weidong Wang 2023-10-25 1692
8ade6cc7e26175 Weidong Wang 2023-10-25 1693 mutex_lock(&aw88399->lock);
8ade6cc7e26175 Weidong Wang 2023-10-25 1694 /* aw device init */
8ade6cc7e26175 Weidong Wang 2023-10-25 1695 ret = aw88399_dev_init(aw88399, aw88399->aw_cfg);
8ade6cc7e26175 Weidong Wang 2023-10-25 1696 if (ret)
8ade6cc7e26175 Weidong Wang 2023-10-25 1697 dev_err(aw88399->aw_pa->dev, "dev init failed");
8ade6cc7e26175 Weidong Wang 2023-10-25 1698 mutex_unlock(&aw88399->lock);
8ade6cc7e26175 Weidong Wang 2023-10-25 1699
8ade6cc7e26175 Weidong Wang 2023-10-25 @1700 return ret;
8ade6cc7e26175 Weidong Wang 2023-10-25 1701 }
8ade6cc7e26175 Weidong Wang 2023-10-25 1702
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-11-09 19:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-08 19:09 sound/soc/codecs/aw88399.c:1700 aw88399_request_firmware_file() warn: '&cont' from request_firmware() not released on lines: 1681,1690,1700 kernel test robot
-- strict thread matches above, loose matches on Subject: below --
2023-11-09 19:30 kernel test robot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.