diff options
author | Steve Lin <steven.lin1@broadcom.com> | 2016-11-14 13:25:57 -0500 |
---|---|---|
committer | Jassi Brar <jaswinder.singh@linaro.org> | 2016-12-19 09:40:19 -0500 |
commit | 9310f1ded44067b2da61fa0471ca5b7768dd28ae (patch) | |
tree | 5ca14a8d0da8ff54d17d2981ac9151a883f248fd /drivers/mailbox/bcm-pdc-mailbox.c | |
parent | 9fb0f9ac54b393ddfe49be7da7751f02bb133db6 (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/bcm-pdc-mailbox.c')
-rw-r--r-- | drivers/mailbox/bcm-pdc-mailbox.c | 6 |
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 | ||
493 | static void pdc_free_debugfs(void) | 493 | static 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 | /** |