summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYuanyuan Liu <yuanliu@codeaurora.org>2018-01-05 15:55:23 -0800
committerGerrit - the friendly Code Review server <code-review@localhost>2018-01-25 10:23:36 -0800
commit8d7bc77acdc2675054c659934a4321de3f785292 (patch)
tree69f83337cc35880d189f73a71395762597bb75c5
parent717bb7a529f71a3074aa29388f9e4a365308f8dc (diff)
icnss: Remove sending uevent after FW ready
FW ready indication doesn't mean it is safe to access HW register. FW ready only means that WLAN FW is initialized. The HW register is safe to access only after FW ack the wlan enable mode request. Hence removing uevent in FW ready indication handler. WLAN driver will responsible for setting FW ready state once wlan enable mode request is return success. CRs-Fixed: 2168628 Change-Id: I769d848efb9dd427da0d71494090fcbf9514cfb2 Signed-off-by: Yuanyuan Liu <yuanliu@codeaurora.org>
-rw-r--r--drivers/soc/qcom/icnss.c4
-rw-r--r--include/soc/qcom/icnss.h3
2 files changed, 2 insertions, 5 deletions
diff --git a/drivers/soc/qcom/icnss.c b/drivers/soc/qcom/icnss.c
index 7f71824d9548..122f758888a6 100644
--- a/drivers/soc/qcom/icnss.c
+++ b/drivers/soc/qcom/icnss.c
@@ -1,4 +1,4 @@
-/* Copyright (c) 2015-2017, The Linux Foundation. All rights reserved.
+/* Copyright (c) 2015-2018, The Linux Foundation. All rights reserved.
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 2 and
@@ -2235,8 +2235,6 @@ static int icnss_driver_event_fw_ready_ind(void *data)
set_bit(ICNSS_FW_READY, &penv->state);
- icnss_call_driver_uevent(penv, ICNSS_UEVENT_FW_READY, NULL);
-
icnss_pr_info("WLAN FW is ready: 0x%lx\n", penv->state);
icnss_hw_power_off(penv);
diff --git a/include/soc/qcom/icnss.h b/include/soc/qcom/icnss.h
index 4fff429dc0b2..ae8834d3fe54 100644
--- a/include/soc/qcom/icnss.h
+++ b/include/soc/qcom/icnss.h
@@ -1,4 +1,4 @@
-/* Copyright (c) 2015-2017, The Linux Foundation. All rights reserved.
+/* Copyright (c) 2015-2018, The Linux Foundation. All rights reserved.
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 2 and
@@ -23,7 +23,6 @@
#endif
enum icnss_uevent {
- ICNSS_UEVENT_FW_READY,
ICNSS_UEVENT_FW_CRASHED,
ICNSS_UEVENT_FW_DOWN,
};