* drivers/net/wireless/intel/iwlwifi/fw/pnvm.c:332 iwl_pnvm_load() error: uninitialized symbol 'len'.
@ 2021-08-30 10:22 Dan Carpenter
0 siblings, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2021-08-30 10:22 UTC (permalink / raw)
To: kbuild, Luca Coelho; +Cc: lkp, kbuild-all, linux-kernel
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 64b4fc45bea6f4faa843d2f97ff51665280efee1
commit: 9dad325f9d57508b154f0bebbc341a8528e5729c iwlwifi: support loading the reduced power table from UEFI
date: 10 weeks ago
config: arc-randconfig-m031-20210827 (attached as .config)
compiler: arceb-elf-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
drivers/net/wireless/intel/iwlwifi/fw/pnvm.c:332 iwl_pnvm_load() error: uninitialized symbol 'len'.
vim +/len +332 drivers/net/wireless/intel/iwlwifi/fw/pnvm.c
cdda18fbbefafc Luca Coelho 2021-02-10 256 int iwl_pnvm_load(struct iwl_trans *trans,
cdda18fbbefafc Luca Coelho 2021-02-10 257 struct iwl_notif_wait_data *notif_wait)
cdda18fbbefafc Luca Coelho 2021-02-10 258 {
cdda18fbbefafc Luca Coelho 2021-02-10 259 u8 *data;
cdda18fbbefafc Luca Coelho 2021-02-10 260 size_t len;
84c3c9952afbf7 Luca Coelho 2021-06-21 261 struct pnvm_sku_package *package;
cdda18fbbefafc Luca Coelho 2021-02-10 262 struct iwl_notification_wait pnvm_wait;
cdda18fbbefafc Luca Coelho 2021-02-10 263 static const u16 ntf_cmds[] = { WIDE_ID(REGULATORY_AND_NVM_GROUP,
cdda18fbbefafc Luca Coelho 2021-02-10 264 PNVM_INIT_COMPLETE_NTFY) };
cdda18fbbefafc Luca Coelho 2021-02-10 265 int ret;
cdda18fbbefafc Luca Coelho 2021-02-10 266
cdda18fbbefafc Luca Coelho 2021-02-10 267 /* if the SKU_ID is empty, there's nothing to do */
cdda18fbbefafc Luca Coelho 2021-02-10 268 if (!trans->sku_id[0] && !trans->sku_id[1] && !trans->sku_id[2])
cdda18fbbefafc Luca Coelho 2021-02-10 269 return 0;
cdda18fbbefafc Luca Coelho 2021-02-10 270
cdda18fbbefafc Luca Coelho 2021-02-10 271 /*
cdda18fbbefafc Luca Coelho 2021-02-10 272 * If we already loaded (or tried to load) it before, we just
cdda18fbbefafc Luca Coelho 2021-02-10 273 * need to set it again.
cdda18fbbefafc Luca Coelho 2021-02-10 274 */
cdda18fbbefafc Luca Coelho 2021-02-10 275 if (trans->pnvm_loaded) {
cdda18fbbefafc Luca Coelho 2021-02-10 276 ret = iwl_trans_set_pnvm(trans, NULL, 0);
cdda18fbbefafc Luca Coelho 2021-02-10 277 if (ret)
cdda18fbbefafc Luca Coelho 2021-02-10 278 return ret;
cdda18fbbefafc Luca Coelho 2021-02-10 279 goto skip_parse;
^^^^^^^^^^^^^^^^
"len" not initialized here.
cdda18fbbefafc Luca Coelho 2021-02-10 280 }
cdda18fbbefafc Luca Coelho 2021-02-10 281
a1a6a4cf49eceb Luca Coelho 2021-02-11 282 /* First attempt to get the PNVM from BIOS */
84c3c9952afbf7 Luca Coelho 2021-06-21 283 package = iwl_uefi_get_pnvm(trans, &len);
84c3c9952afbf7 Luca Coelho 2021-06-21 284 if (!IS_ERR_OR_NULL(package)) {
84c3c9952afbf7 Luca Coelho 2021-06-21 285 data = kmemdup(package->data, len, GFP_KERNEL);
84c3c9952afbf7 Luca Coelho 2021-06-21 286
84c3c9952afbf7 Luca Coelho 2021-06-21 287 /* free package regardless of whether kmemdup succeeded */
84c3c9952afbf7 Luca Coelho 2021-06-21 288 kfree(package);
84c3c9952afbf7 Luca Coelho 2021-06-21 289
84c3c9952afbf7 Luca Coelho 2021-06-21 290 if (data) {
84c3c9952afbf7 Luca Coelho 2021-06-21 291 /* we need only the data size */
84c3c9952afbf7 Luca Coelho 2021-06-21 292 len -= sizeof(*package);
a1a6a4cf49eceb Luca Coelho 2021-02-11 293 goto parse;
84c3c9952afbf7 Luca Coelho 2021-06-21 294 }
84c3c9952afbf7 Luca Coelho 2021-06-21 295 }
a1a6a4cf49eceb Luca Coelho 2021-02-11 296
a1a6a4cf49eceb Luca Coelho 2021-02-11 297 /* If it's not available, try from the filesystem */
cdda18fbbefafc Luca Coelho 2021-02-10 298 ret = iwl_pnvm_get_from_fs(trans, &data, &len);
cdda18fbbefafc Luca Coelho 2021-02-10 299 if (ret) {
82a08d0cd7b503 Johannes Berg 2021-01-15 300 /*
82a08d0cd7b503 Johannes Berg 2021-01-15 301 * Pretend we've loaded it - at least we've tried and
82a08d0cd7b503 Johannes Berg 2021-01-15 302 * couldn't load it at all, so there's no point in
82a08d0cd7b503 Johannes Berg 2021-01-15 303 * trying again over and over.
82a08d0cd7b503 Johannes Berg 2021-01-15 304 */
82a08d0cd7b503 Johannes Berg 2021-01-15 305 trans->pnvm_loaded = true;
6972592850c00e Luca Coelho 2020-10-08 306
cdda18fbbefafc Luca Coelho 2021-02-10 307 goto skip_parse;
1c58bed4b7f755 Johannes Berg 2021-01-15 308 }
b3e4c0f34c1752 Luca Coelho 2020-10-08 309
a1a6a4cf49eceb Luca Coelho 2021-02-11 310 parse:
cdda18fbbefafc Luca Coelho 2021-02-10 311 iwl_pnvm_parse(trans, data, len);
cdda18fbbefafc Luca Coelho 2021-02-10 312
cdda18fbbefafc Luca Coelho 2021-02-10 313 kfree(data);
cdda18fbbefafc Luca Coelho 2021-02-10 314
cdda18fbbefafc Luca Coelho 2021-02-10 315 skip_parse:
9dad325f9d5750 Luca Coelho 2021-06-21 316 data = NULL;
9dad325f9d5750 Luca Coelho 2021-06-21 317 /* now try to get the reduce power table, if not loaded yet */
9dad325f9d5750 Luca Coelho 2021-06-21 318 if (!trans->reduce_power_loaded) {
9dad325f9d5750 Luca Coelho 2021-06-21 319 data = iwl_uefi_get_reduced_power(trans, &len);
Possibly initialized here? I looked at this and wasn't certain.
9dad325f9d5750 Luca Coelho 2021-06-21 320 if (IS_ERR_OR_NULL(data)) {
9dad325f9d5750 Luca Coelho 2021-06-21 321 /*
9dad325f9d5750 Luca Coelho 2021-06-21 322 * Pretend we've loaded it - at least we've tried and
9dad325f9d5750 Luca Coelho 2021-06-21 323 * couldn't load it at all, so there's no point in
9dad325f9d5750 Luca Coelho 2021-06-21 324 * trying again over and over.
9dad325f9d5750 Luca Coelho 2021-06-21 325 */
9dad325f9d5750 Luca Coelho 2021-06-21 326 trans->reduce_power_loaded = true;
9dad325f9d5750 Luca Coelho 2021-06-21 327
9dad325f9d5750 Luca Coelho 2021-06-21 328 goto skip_reduce_power;
9dad325f9d5750 Luca Coelho 2021-06-21 329 }
9dad325f9d5750 Luca Coelho 2021-06-21 330 }
9dad325f9d5750 Luca Coelho 2021-06-21 331
9dad325f9d5750 Luca Coelho 2021-06-21 @332 ret = iwl_trans_set_reduce_power(trans, data, len);
^^^
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 3+ messages in thread
* drivers/net/wireless/intel/iwlwifi/fw/pnvm.c:332 iwl_pnvm_load() error: uninitialized symbol 'len'.
@ 2022-02-25 8:25 Dan Carpenter
0 siblings, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2022-02-25 8:25 UTC (permalink / raw)
To: kbuild, Luca Coelho; +Cc: lkp, kbuild-all, linux-kernel
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: d8152cfe2f21d6930c680311b03b169899c8d2a0
commit: 9dad325f9d57508b154f0bebbc341a8528e5729c iwlwifi: support loading the reduced power table from UEFI
config: ia64-randconfig-m031-20220220 (https://download.01.org/0day-ci/archive/20220225/202202250921.59KWuV9B-lkp@intel.com/config)
compiler: ia64-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
drivers/net/wireless/intel/iwlwifi/fw/pnvm.c:332 iwl_pnvm_load() error: uninitialized symbol 'len'.
vim +/len +332 drivers/net/wireless/intel/iwlwifi/fw/pnvm.c
cdda18fbbefafc Luca Coelho 2021-02-10 256 int iwl_pnvm_load(struct iwl_trans *trans,
cdda18fbbefafc Luca Coelho 2021-02-10 257 struct iwl_notif_wait_data *notif_wait)
cdda18fbbefafc Luca Coelho 2021-02-10 258 {
cdda18fbbefafc Luca Coelho 2021-02-10 259 u8 *data;
cdda18fbbefafc Luca Coelho 2021-02-10 260 size_t len;
84c3c9952afbf7 Luca Coelho 2021-06-21 261 struct pnvm_sku_package *package;
cdda18fbbefafc Luca Coelho 2021-02-10 262 struct iwl_notification_wait pnvm_wait;
cdda18fbbefafc Luca Coelho 2021-02-10 263 static const u16 ntf_cmds[] = { WIDE_ID(REGULATORY_AND_NVM_GROUP,
cdda18fbbefafc Luca Coelho 2021-02-10 264 PNVM_INIT_COMPLETE_NTFY) };
cdda18fbbefafc Luca Coelho 2021-02-10 265 int ret;
cdda18fbbefafc Luca Coelho 2021-02-10 266
cdda18fbbefafc Luca Coelho 2021-02-10 267 /* if the SKU_ID is empty, there's nothing to do */
cdda18fbbefafc Luca Coelho 2021-02-10 268 if (!trans->sku_id[0] && !trans->sku_id[1] && !trans->sku_id[2])
cdda18fbbefafc Luca Coelho 2021-02-10 269 return 0;
cdda18fbbefafc Luca Coelho 2021-02-10 270
cdda18fbbefafc Luca Coelho 2021-02-10 271 /*
cdda18fbbefafc Luca Coelho 2021-02-10 272 * If we already loaded (or tried to load) it before, we just
cdda18fbbefafc Luca Coelho 2021-02-10 273 * need to set it again.
cdda18fbbefafc Luca Coelho 2021-02-10 274 */
cdda18fbbefafc Luca Coelho 2021-02-10 275 if (trans->pnvm_loaded) {
cdda18fbbefafc Luca Coelho 2021-02-10 276 ret = iwl_trans_set_pnvm(trans, NULL, 0);
cdda18fbbefafc Luca Coelho 2021-02-10 277 if (ret)
cdda18fbbefafc Luca Coelho 2021-02-10 278 return ret;
cdda18fbbefafc Luca Coelho 2021-02-10 279 goto skip_parse;
"len" is not set on this path
cdda18fbbefafc Luca Coelho 2021-02-10 280 }
cdda18fbbefafc Luca Coelho 2021-02-10 281
a1a6a4cf49eceb Luca Coelho 2021-02-11 282 /* First attempt to get the PNVM from BIOS */
84c3c9952afbf7 Luca Coelho 2021-06-21 283 package = iwl_uefi_get_pnvm(trans, &len);
84c3c9952afbf7 Luca Coelho 2021-06-21 284 if (!IS_ERR_OR_NULL(package)) {
84c3c9952afbf7 Luca Coelho 2021-06-21 285 data = kmemdup(package->data, len, GFP_KERNEL);
84c3c9952afbf7 Luca Coelho 2021-06-21 286
84c3c9952afbf7 Luca Coelho 2021-06-21 287 /* free package regardless of whether kmemdup succeeded */
84c3c9952afbf7 Luca Coelho 2021-06-21 288 kfree(package);
84c3c9952afbf7 Luca Coelho 2021-06-21 289
84c3c9952afbf7 Luca Coelho 2021-06-21 290 if (data) {
84c3c9952afbf7 Luca Coelho 2021-06-21 291 /* we need only the data size */
84c3c9952afbf7 Luca Coelho 2021-06-21 292 len -= sizeof(*package);
a1a6a4cf49eceb Luca Coelho 2021-02-11 293 goto parse;
84c3c9952afbf7 Luca Coelho 2021-06-21 294 }
84c3c9952afbf7 Luca Coelho 2021-06-21 295 }
a1a6a4cf49eceb Luca Coelho 2021-02-11 296
a1a6a4cf49eceb Luca Coelho 2021-02-11 297 /* If it's not available, try from the filesystem */
cdda18fbbefafc Luca Coelho 2021-02-10 298 ret = iwl_pnvm_get_from_fs(trans, &data, &len);
cdda18fbbefafc Luca Coelho 2021-02-10 299 if (ret) {
82a08d0cd7b503 Johannes Berg 2021-01-15 300 /*
82a08d0cd7b503 Johannes Berg 2021-01-15 301 * Pretend we've loaded it - at least we've tried and
82a08d0cd7b503 Johannes Berg 2021-01-15 302 * couldn't load it at all, so there's no point in
82a08d0cd7b503 Johannes Berg 2021-01-15 303 * trying again over and over.
82a08d0cd7b503 Johannes Berg 2021-01-15 304 */
82a08d0cd7b503 Johannes Berg 2021-01-15 305 trans->pnvm_loaded = true;
6972592850c00e Luca Coelho 2020-10-08 306
cdda18fbbefafc Luca Coelho 2021-02-10 307 goto skip_parse;
1c58bed4b7f755 Johannes Berg 2021-01-15 308 }
b3e4c0f34c1752 Luca Coelho 2020-10-08 309
a1a6a4cf49eceb Luca Coelho 2021-02-11 310 parse:
cdda18fbbefafc Luca Coelho 2021-02-10 311 iwl_pnvm_parse(trans, data, len);
cdda18fbbefafc Luca Coelho 2021-02-10 312
cdda18fbbefafc Luca Coelho 2021-02-10 313 kfree(data);
cdda18fbbefafc Luca Coelho 2021-02-10 314
cdda18fbbefafc Luca Coelho 2021-02-10 315 skip_parse:
9dad325f9d5750 Luca Coelho 2021-06-21 316 data = NULL;
9dad325f9d5750 Luca Coelho 2021-06-21 317 /* now try to get the reduce power table, if not loaded yet */
9dad325f9d5750 Luca Coelho 2021-06-21 318 if (!trans->reduce_power_loaded) {
Can trans->reduce_power_loaded be false when trans->pnvm_loaded is true?
9dad325f9d5750 Luca Coelho 2021-06-21 319 data = iwl_uefi_get_reduced_power(trans, &len);
9dad325f9d5750 Luca Coelho 2021-06-21 320 if (IS_ERR_OR_NULL(data)) {
9dad325f9d5750 Luca Coelho 2021-06-21 321 /*
9dad325f9d5750 Luca Coelho 2021-06-21 322 * Pretend we've loaded it - at least we've tried and
9dad325f9d5750 Luca Coelho 2021-06-21 323 * couldn't load it at all, so there's no point in
9dad325f9d5750 Luca Coelho 2021-06-21 324 * trying again over and over.
9dad325f9d5750 Luca Coelho 2021-06-21 325 */
9dad325f9d5750 Luca Coelho 2021-06-21 326 trans->reduce_power_loaded = true;
9dad325f9d5750 Luca Coelho 2021-06-21 327
9dad325f9d5750 Luca Coelho 2021-06-21 328 goto skip_reduce_power;
9dad325f9d5750 Luca Coelho 2021-06-21 329 }
9dad325f9d5750 Luca Coelho 2021-06-21 330 }
9dad325f9d5750 Luca Coelho 2021-06-21 331
9dad325f9d5750 Luca Coelho 2021-06-21 @332 ret = iwl_trans_set_reduce_power(trans, data, len);
^^^
This is the warning if those conditions are possible
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 3+ messages in thread
* drivers/net/wireless/intel/iwlwifi/fw/pnvm.c:332 iwl_pnvm_load() error: uninitialized symbol 'len'.
@ 2021-07-07 7:46 Dan Carpenter
0 siblings, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2021-07-07 7:46 UTC (permalink / raw)
To: kbuild, Luca Coelho; +Cc: lkp, kbuild-all, linux-kernel
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 77d34a4683b053108ecd466cc7c4193b45805528
commit: 9dad325f9d57508b154f0bebbc341a8528e5729c iwlwifi: support loading the reduced power table from UEFI
config: x86_64-randconfig-m001-20210706 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
drivers/net/wireless/intel/iwlwifi/fw/pnvm.c:332 iwl_pnvm_load() error: uninitialized symbol 'len'.
vim +/len +332 drivers/net/wireless/intel/iwlwifi/fw/pnvm.c
cdda18fbbefafc6 Luca Coelho 2021-02-10 256 int iwl_pnvm_load(struct iwl_trans *trans,
cdda18fbbefafc6 Luca Coelho 2021-02-10 257 struct iwl_notif_wait_data *notif_wait)
cdda18fbbefafc6 Luca Coelho 2021-02-10 258 {
cdda18fbbefafc6 Luca Coelho 2021-02-10 259 u8 *data;
cdda18fbbefafc6 Luca Coelho 2021-02-10 260 size_t len;
84c3c9952afbf7d Luca Coelho 2021-06-21 261 struct pnvm_sku_package *package;
cdda18fbbefafc6 Luca Coelho 2021-02-10 262 struct iwl_notification_wait pnvm_wait;
cdda18fbbefafc6 Luca Coelho 2021-02-10 263 static const u16 ntf_cmds[] = { WIDE_ID(REGULATORY_AND_NVM_GROUP,
cdda18fbbefafc6 Luca Coelho 2021-02-10 264 PNVM_INIT_COMPLETE_NTFY) };
cdda18fbbefafc6 Luca Coelho 2021-02-10 265 int ret;
cdda18fbbefafc6 Luca Coelho 2021-02-10 266
cdda18fbbefafc6 Luca Coelho 2021-02-10 267 /* if the SKU_ID is empty, there's nothing to do */
cdda18fbbefafc6 Luca Coelho 2021-02-10 268 if (!trans->sku_id[0] && !trans->sku_id[1] && !trans->sku_id[2])
cdda18fbbefafc6 Luca Coelho 2021-02-10 269 return 0;
cdda18fbbefafc6 Luca Coelho 2021-02-10 270
cdda18fbbefafc6 Luca Coelho 2021-02-10 271 /*
cdda18fbbefafc6 Luca Coelho 2021-02-10 272 * If we already loaded (or tried to load) it before, we just
cdda18fbbefafc6 Luca Coelho 2021-02-10 273 * need to set it again.
cdda18fbbefafc6 Luca Coelho 2021-02-10 274 */
cdda18fbbefafc6 Luca Coelho 2021-02-10 275 if (trans->pnvm_loaded) {
cdda18fbbefafc6 Luca Coelho 2021-02-10 276 ret = iwl_trans_set_pnvm(trans, NULL, 0);
cdda18fbbefafc6 Luca Coelho 2021-02-10 277 if (ret)
cdda18fbbefafc6 Luca Coelho 2021-02-10 278 return ret;
cdda18fbbefafc6 Luca Coelho 2021-02-10 279 goto skip_parse;
cdda18fbbefafc6 Luca Coelho 2021-02-10 280 }
cdda18fbbefafc6 Luca Coelho 2021-02-10 281
a1a6a4cf49eceb3 Luca Coelho 2021-02-11 282 /* First attempt to get the PNVM from BIOS */
84c3c9952afbf7d Luca Coelho 2021-06-21 283 package = iwl_uefi_get_pnvm(trans, &len);
Let's say this fails
84c3c9952afbf7d Luca Coelho 2021-06-21 284 if (!IS_ERR_OR_NULL(package)) {
84c3c9952afbf7d Luca Coelho 2021-06-21 285 data = kmemdup(package->data, len, GFP_KERNEL);
84c3c9952afbf7d Luca Coelho 2021-06-21 286
84c3c9952afbf7d Luca Coelho 2021-06-21 287 /* free package regardless of whether kmemdup succeeded */
84c3c9952afbf7d Luca Coelho 2021-06-21 288 kfree(package);
84c3c9952afbf7d Luca Coelho 2021-06-21 289
84c3c9952afbf7d Luca Coelho 2021-06-21 290 if (data) {
84c3c9952afbf7d Luca Coelho 2021-06-21 291 /* we need only the data size */
84c3c9952afbf7d Luca Coelho 2021-06-21 292 len -= sizeof(*package);
a1a6a4cf49eceb3 Luca Coelho 2021-02-11 293 goto parse;
84c3c9952afbf7d Luca Coelho 2021-06-21 294 }
84c3c9952afbf7d Luca Coelho 2021-06-21 295 }
a1a6a4cf49eceb3 Luca Coelho 2021-02-11 296
a1a6a4cf49eceb3 Luca Coelho 2021-02-11 297 /* If it's not available, try from the filesystem */
cdda18fbbefafc6 Luca Coelho 2021-02-10 298 ret = iwl_pnvm_get_from_fs(trans, &data, &len);
Lets say this fails as well
cdda18fbbefafc6 Luca Coelho 2021-02-10 299 if (ret) {
82a08d0cd7b503b Johannes Berg 2021-01-15 300 /*
82a08d0cd7b503b Johannes Berg 2021-01-15 301 * Pretend we've loaded it - at least we've tried and
82a08d0cd7b503b Johannes Berg 2021-01-15 302 * couldn't load it at all, so there's no point in
82a08d0cd7b503b Johannes Berg 2021-01-15 303 * trying again over and over.
82a08d0cd7b503b Johannes Berg 2021-01-15 304 */
82a08d0cd7b503b Johannes Berg 2021-01-15 305 trans->pnvm_loaded = true;
6972592850c00e5 Luca Coelho 2020-10-08 306
cdda18fbbefafc6 Luca Coelho 2021-02-10 307 goto skip_parse;
^^^^^^^^^^^^^^^^
1c58bed4b7f7551 Johannes Berg 2021-01-15 308 }
b3e4c0f34c1752c Luca Coelho 2020-10-08 309
a1a6a4cf49eceb3 Luca Coelho 2021-02-11 310 parse:
cdda18fbbefafc6 Luca Coelho 2021-02-10 311 iwl_pnvm_parse(trans, data, len);
cdda18fbbefafc6 Luca Coelho 2021-02-10 312
cdda18fbbefafc6 Luca Coelho 2021-02-10 313 kfree(data);
cdda18fbbefafc6 Luca Coelho 2021-02-10 314
cdda18fbbefafc6 Luca Coelho 2021-02-10 315 skip_parse:
9dad325f9d57508 Luca Coelho 2021-06-21 316 data = NULL;
9dad325f9d57508 Luca Coelho 2021-06-21 317 /* now try to get the reduce power table, if not loaded yet */
9dad325f9d57508 Luca Coelho 2021-06-21 318 if (!trans->reduce_power_loaded) {
^^^^^^^^^^^^^^^^^^^^^^^^^^
This might make a difference but I Smatch wouldn't be able to figure it
out.
9dad325f9d57508 Luca Coelho 2021-06-21 319 data = iwl_uefi_get_reduced_power(trans, &len);
9dad325f9d57508 Luca Coelho 2021-06-21 320 if (IS_ERR_OR_NULL(data)) {
9dad325f9d57508 Luca Coelho 2021-06-21 321 /*
9dad325f9d57508 Luca Coelho 2021-06-21 322 * Pretend we've loaded it - at least we've tried and
9dad325f9d57508 Luca Coelho 2021-06-21 323 * couldn't load it at all, so there's no point in
9dad325f9d57508 Luca Coelho 2021-06-21 324 * trying again over and over.
9dad325f9d57508 Luca Coelho 2021-06-21 325 */
9dad325f9d57508 Luca Coelho 2021-06-21 326 trans->reduce_power_loaded = true;
9dad325f9d57508 Luca Coelho 2021-06-21 327
9dad325f9d57508 Luca Coelho 2021-06-21 328 goto skip_reduce_power;
9dad325f9d57508 Luca Coelho 2021-06-21 329 }
9dad325f9d57508 Luca Coelho 2021-06-21 330 }
9dad325f9d57508 Luca Coelho 2021-06-21 331
9dad325f9d57508 Luca Coelho 2021-06-21 @332 ret = iwl_trans_set_reduce_power(trans, data, len);
^^^
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-02-25 8:25 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-30 10:22 drivers/net/wireless/intel/iwlwifi/fw/pnvm.c:332 iwl_pnvm_load() error: uninitialized symbol 'len' Dan Carpenter
-- strict thread matches above, loose matches on Subject: below --
2022-02-25 8:25 Dan Carpenter
2021-07-07 7:46 Dan Carpenter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).