diff options
author | David Howells <dhowells@redhat.com> | 2013-04-11 22:02:22 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2013-05-01 17:29:41 -0400 |
commit | cc87e0fff1e639c4c7832075b8cdd89ab30d2a1f (patch) | |
tree | ade79fe46c172a2de9c09682b8af9b6860b8726b /drivers | |
parent | c4558a26ff661b5299942ce2b735f3cab4aed1e5 (diff) |
rtl8192u: Don't need to save device proc dir PDE
Don't need to save the PDE of a directory created under /proc/net/rtl8192/ as
we can use proc subtree deletion to get rid of it and all its children.
Signed-off-by: David Howells <dhowells@redhat.com>
Acked-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
cc: Jerry Chuang <jerry-chuang@realtek.com>
cc: linux-wireless@vger.kernel.org
cc: devel@driverdev.osuosl.org
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/staging/rtl8192u/r8192U.h | 1 | ||||
-rw-r--r-- | drivers/staging/rtl8192u/r8192U_core.c | 18 |
2 files changed, 6 insertions, 13 deletions
diff --git a/drivers/staging/rtl8192u/r8192U.h b/drivers/staging/rtl8192u/r8192U.h index e538e026b512..bedeb330ad4f 100644 --- a/drivers/staging/rtl8192u/r8192U.h +++ b/drivers/staging/rtl8192u/r8192U.h | |||
@@ -946,7 +946,6 @@ typedef struct r8192_priv { | |||
946 | /*stats*/ | 946 | /*stats*/ |
947 | struct Stats stats; | 947 | struct Stats stats; |
948 | struct iw_statistics wstats; | 948 | struct iw_statistics wstats; |
949 | struct proc_dir_entry *dir_dev; | ||
950 | 949 | ||
951 | /*RX stuff*/ | 950 | /*RX stuff*/ |
952 | // u32 *rxring; | 951 | // u32 *rxring; |
diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u/r8192U_core.c index d81d7d55f25e..27ba2a3111d2 100644 --- a/drivers/staging/rtl8192u/r8192U_core.c +++ b/drivers/staging/rtl8192u/r8192U_core.c | |||
@@ -669,20 +669,19 @@ static const struct rtl8192_proc_file rtl8192_proc_files[] = { | |||
669 | void rtl8192_proc_init_one(struct net_device *dev) | 669 | void rtl8192_proc_init_one(struct net_device *dev) |
670 | { | 670 | { |
671 | const struct rtl8192_proc_file *f; | 671 | const struct rtl8192_proc_file *f; |
672 | struct r8192_priv *priv = (struct r8192_priv *)ieee80211_priv(dev); | 672 | struct proc_dir_entry *dir; |
673 | 673 | ||
674 | if (rtl8192_proc) { | 674 | if (rtl8192_proc) { |
675 | priv->dir_dev = proc_mkdir_data(dev->name, 0, rtl8192_proc, dev); | 675 | dir = proc_mkdir_data(dev->name, 0, rtl8192_proc, dev); |
676 | if (!priv->dir_dev) { | 676 | if (!dir) { |
677 | RT_TRACE(COMP_ERR, "Unable to initialize /proc/net/rtl8192/%s\n", | 677 | RT_TRACE(COMP_ERR, "Unable to initialize /proc/net/rtl8192/%s\n", |
678 | dev->name); | 678 | dev->name); |
679 | return; | 679 | return; |
680 | } | 680 | } |
681 | 681 | ||
682 | for (f = rtl8192_proc_files; f->name[0]; f++) { | 682 | for (f = rtl8192_proc_files; f->name[0]; f++) { |
683 | if (!proc_create_data(f->name, S_IFREG | S_IRUGO, | 683 | if (!proc_create_data(f->name, S_IFREG | S_IRUGO, dir, |
684 | priv->dir_dev, | 684 | &rtl8192_proc_fops, f->show)) { |
685 | rtl8192_proc_fops, f->show)) { | ||
686 | RT_TRACE(COMP_ERR, "Unable to initialize " | 685 | RT_TRACE(COMP_ERR, "Unable to initialize " |
687 | "/proc/net/rtl8192/%s/%s\n", | 686 | "/proc/net/rtl8192/%s/%s\n", |
688 | dev->name, f->name); | 687 | dev->name, f->name); |
@@ -694,12 +693,7 @@ void rtl8192_proc_init_one(struct net_device *dev) | |||
694 | 693 | ||
695 | void rtl8192_proc_remove_one(struct net_device *dev) | 694 | void rtl8192_proc_remove_one(struct net_device *dev) |
696 | { | 695 | { |
697 | struct r8192_priv *priv = (struct r8192_priv *)ieee80211_priv(dev); | 696 | remove_proc_subtree(dev->name, rtl8192_proc); |
698 | |||
699 | if (priv->dir_dev) { | ||
700 | remove_proc_subtree(dev->name, rtl8192_proc); | ||
701 | priv->dir_dev = NULL; | ||
702 | } | ||
703 | } | 697 | } |
704 | 698 | ||
705 | /**************************************************************************** | 699 | /**************************************************************************** |