diff options
author | Dustin Brown <dustinb@codeaurora.org> | 2018-06-27 15:22:06 -0700 |
---|---|---|
committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2018-07-16 10:50:17 -0700 |
commit | ae68001cc0014ace036b983e1dfbb8fe85008e63 (patch) | |
tree | bdc744653df7aa2f332a203e9e3ce0de5b15cf1b /cfg/src | |
parent | 71ea8e9aec265b90059b364ddcef02fc499be3f2 (diff) | |
download | qca-wfi-host-cmn-ae68001cc0014ace036b983e1dfbb8fe85008e63.tar.gz |
qcacmn: Move panic messages into QDF_DEBUG_PANIC
QDF_DEBUG_PANIC has recently gained a reason parameter. Move instance of
logging followed immediately by QDF_DEBUG_PANIC to single calls.
Change-Id: Ifc00be410621005e8494074ed00b1b7e44cc6a03
CRs-Fixed: 2271769
Diffstat (limited to 'cfg/src')
-rw-r--r-- | cfg/src/cfg.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/cfg/src/cfg.c b/cfg/src/cfg.c index 6c3cc5417..fbda51826 100644 --- a/cfg/src/cfg.c +++ b/cfg/src/cfg.c @@ -101,7 +101,8 @@ static void cfg_int_item_handler(struct cfg_value_store *store, switch (meta->fallback) { default: - QDF_DEBUG_PANIC(); + QDF_DEBUG_PANIC("Unknown fallback method %d for cfg item '%s'", + meta->fallback, meta->name); /* fall through */ case CFG_VALUE_OR_DEFAULT: /* store already contains default */ @@ -153,7 +154,8 @@ static void cfg_uint_item_handler(struct cfg_value_store *store, switch (meta->fallback) { default: - QDF_DEBUG_PANIC(); + QDF_DEBUG_PANIC("Unknown fallback method %d for cfg item '%s'", + meta->fallback, meta->name); /* fall through */ case CFG_VALUE_OR_DEFAULT: /* store already contains default */ @@ -421,10 +423,9 @@ static void cfg_store_free(struct cfg_value_store *store) qdf_spin_lock_bh(&__cfg_stores_lock); status = qdf_list_remove_node(&__cfg_stores_list, &store->node); qdf_spin_unlock_bh(&__cfg_stores_lock); - if (QDF_IS_STATUS_ERROR(status)) { - cfg_err("Failed config store list removal; status:%d", status); - QDF_DEBUG_PANIC(); - } + if (QDF_IS_STATUS_ERROR(status)) + QDF_DEBUG_PANIC("Failed config store list removal; status:%d", + status); qdf_mem_free(store->path); qdf_mem_free(store); |