summaryrefslogtreecommitdiffstats
path: root/drivers/mailbox
diff options
context:
space:
mode:
authorSteve Lin <steven.lin1@broadcom.com>2016-11-14 13:25:57 -0500
committerJassi Brar <jaswinder.singh@linaro.org>2016-12-19 09:40:19 -0500
commit9310f1ded44067b2da61fa0471ca5b7768dd28ae (patch)
tree5ca14a8d0da8ff54d17d2981ac9151a883f248fd /drivers/mailbox
parent9fb0f9ac54b393ddfe49be7da7751f02bb133db6 (diff)
mailbox: bcm-pdc: PDC driver leaves debugfs files after removal
Minor fix to ensure that debugfs stats pseudo-files are removed when driver module is unloaded. Previously, the call to debugfs_remove_recursive() was never being called since the directory was not empty, and a seg fault would occur if another process tried to access these leftover files. Signed-off-by: Steve Lin <steven.lin1@broadcom.com> Signed-off-by: Rob Rice <rob.rice@broadcom.com> Reviewed-by: Andy Gospodarek <gospo@broadcom.com> Signed-off-by: Jassi Brar <jaswinder.singh@linaro.org>
Diffstat (limited to 'drivers/mailbox')
-rw-r--r--drivers/mailbox/bcm-pdc-mailbox.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/mailbox/bcm-pdc-mailbox.c b/drivers/mailbox/bcm-pdc-mailbox.c
index 3b4ebbe81964..c9434a756bf3 100644
--- a/drivers/mailbox/bcm-pdc-mailbox.c
+++ b/drivers/mailbox/bcm-pdc-mailbox.c
@@ -492,10 +492,8 @@ static void pdc_setup_debugfs(struct pdc_state *pdcs)
492 492
493static void pdc_free_debugfs(void) 493static void pdc_free_debugfs(void)
494{ 494{
495 if (debugfs_dir && simple_empty(debugfs_dir)) { 495 debugfs_remove_recursive(debugfs_dir);
496 debugfs_remove_recursive(debugfs_dir); 496 debugfs_dir = NULL;
497 debugfs_dir = NULL;
498 }
499} 497}
500 498
501/** 499/**