Skip to contentSkip to conversation

mfd: db8500-prcmu: remove check for CONFIG_DBX500_PRCMU_DEBUG

PATCH Applied Paul Bolle 2 participants 2 messages May 22, 2014 – May 23, 2014

Subsystems (2)
Multifunction devices (MFD) Arm/nomadik/Ux500 architectures
lore
Share & tools
lore

Paul Bolle

author
#0 lore raw

PATCH 1/1 mfd: db8500-prcmu: remove check for CONFIG_DBX500_PRCMU_DEBUG

A check for CONFIG_DBX500_PRCMU_DEBUG was added in v3.6. But there's no Kconfig symbol DBX500_PRCMU_DEBUG. So remove this check.

Signed-off-by Paul Bolle <[email protected]>

---
Untested. Note that db8500_prcmu_debug_dump() is nowhere to be found in the tree.

This check has been in the tree for less than two years. Am I being impatient here?

 drivers/mfd/db8500-prcmu.c | 3 ---
 1 file changed, 3 deletions(-)
drivers/mfd/db8500-prcmu.c +0 -3
index 7694e0700d34..c248bde367ab 100644
@@ -2303,9 +2303,6 @@ int prcmu_ac_wake_req(void)
 
 	if (!wait_for_completion_timeout(&mb0_transfer.ac_wake_work,
 			msecs_to_jiffies(5000))) {
-#if defined(CONFIG_DBX500_PRCMU_DEBUG)
-		db8500_prcmu_debug_dump(__func__, true, true);
-#endif
 		pr_crit("prcmu: %s timed out (5 s) waiting for a reply.\n",
 			__func__);
 		ret = -EFAULT;
Signature · 1 line
1.9.0
Applied
#1 lore raw
Quoted 8 lines

A check for CONFIG_DBX500_PRCMU_DEBUG was added in v3.6. But there's no Kconfig symbol DBX500_PRCMU_DEBUG. So remove this check.

Signed-off-by Paul Bolle <[email protected]>

---
Untested. Note that db8500_prcmu_debug_dump() is nowhere to be found in the tree.

Quoted 5 lines

This check has been in the tree for less than two years. Am I being impatient here?

 drivers/mfd/db8500-prcmu.c | 3 ---
 1 file changed, 3 deletions(-)

Applied, thanks.

Quoted 14 lines
drivers/mfd/db8500-prcmu.c +0 -3
index 7694e0700d34..c248bde367ab 100644
@@ -2303,9 +2303,6 @@ int prcmu_ac_wake_req(void)
 
 	if (!wait_for_completion_timeout(&mb0_transfer.ac_wake_work,
 			msecs_to_jiffies(5000))) {
-#if defined(CONFIG_DBX500_PRCMU_DEBUG)
-		db8500_prcmu_debug_dump(__func__, true, true);
-#endif
 		pr_crit("prcmu: %s timed out (5 s) waiting for a reply.\n",
 			__func__);
 		ret = -EFAULT;
Signature · 4 lines
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog