qcacld-3.0: Add if check before calling cds_force_assert_target()

If CNSS function pld_force_collect_target_dump() returns 0, this means
that FW is already asserted and should not call cds_force_assert_target().
Add an if check for pld_force_collect_target_dump() to only call
cds_force_assert_target() when it is not 0.

Change-Id: Ib784c1a406b93327942164a975994065149aed90
CRs-Fixed: 2799461
This commit is contained in:
Alan Chen 2020-10-16 12:39:22 -07:00 • committed by snandini
commit 0c8a010b06
2 changed files with 3 additions and 2 deletions

View file

@ -1947,7 +1947,8 @@ static void cds_trigger_recovery_handler(const char *func, const uint32_t line)
cds_set_recovery_in_progress(true);
cds_set_assert_target_in_progress(true);
cds_force_assert_target(qdf);
if (pld_collect_rddm(qdf->dev))
cds_force_assert_target(qdf);
cds_set_assert_target_in_progress(false);
/*

View file

@ -625,7 +625,7 @@ static inline int pld_pcie_idle_shutdown(struct device *dev)
static inline int pld_pcie_force_assert_target(struct device *dev)
{
return cnss_force_collect_rddm(dev);
return cnss_force_fw_assert(dev);
}
static inline int pld_pcie_get_user_msi_assignment(struct device *dev,