Skip to content

Commit

Permalink
staging: vc04_services: no need to save the log debufs dentries
Browse files Browse the repository at this point in the history
The log entry dentries are only set, never referenced, so no need to
keep them around.  Remove the pointer from struct
vchiq_debugfs_log_entry as it is not needed anymore and get rid of the
separate vchiq_debugfs_create_log_entries() function as it is only used
in one place.

Suggested-by: Eric Anholt <eric@anholt.net>
Cc: Stefan Wahren <stefan.wahren@i2se.com>
Cc: Kees Cook <keescook@chromium.org>
Cc: Dan Carpenter <dan.carpenter@oracle.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Keerthi Reddy <keerthigd4990@gmail.com>
Cc: linux-rpi-kernel@lists.infradead.org
Cc: linux-arm-kernel@lists.infradead.org
Reviewed-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Greg Kroah-Hartman committed Jun 1, 2018
1 parent 54f1569 commit 3b93c0f
Showing 1 changed file with 10 additions and 19 deletions.
29 changes: 10 additions & 19 deletions drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ struct dentry *vchiq_dbg_clients;
struct vchiq_debugfs_log_entry {
const char *name;
void *plevel;
struct dentry *dir;
};

static struct vchiq_debugfs_log_entry vchiq_debugfs_log_entries[] = {
Expand Down Expand Up @@ -144,23 +143,6 @@ static const struct file_operations debugfs_log_fops = {
.release = single_release,
};

/* create an entry under <debugfs>/vchiq/log for each log category */
static void vchiq_debugfs_create_log_entries(struct dentry *top)
{
struct dentry *dir;
size_t i;

dir = debugfs_create_dir("log", vchiq_dbg_dir);

for (i = 0; i < n_log_entries; i++) {
dir = debugfs_create_file(vchiq_debugfs_log_entries[i].name,
0644, dir,
vchiq_debugfs_log_entries[i].plevel,
&debugfs_log_fops);
vchiq_debugfs_log_entries[i].dir = dir;
}
}

static int debugfs_usecount_show(struct seq_file *f, void *offset)
{
VCHIQ_INSTANCE_T instance = f->private;
Expand Down Expand Up @@ -268,10 +250,19 @@ void vchiq_debugfs_remove_instance(VCHIQ_INSTANCE_T instance)

void vchiq_debugfs_init(void)
{
struct dentry *dir;
int i;

vchiq_dbg_dir = debugfs_create_dir("vchiq", NULL);
vchiq_dbg_clients = debugfs_create_dir("clients", vchiq_dbg_dir);

vchiq_debugfs_create_log_entries(vchiq_dbg_dir);
/* create an entry under <debugfs>/vchiq/log for each log category */
dir = debugfs_create_dir("log", vchiq_dbg_dir);

for (i = 0; i < n_log_entries; i++)
debugfs_create_file(vchiq_debugfs_log_entries[i].name, 0644,
dir, vchiq_debugfs_log_entries[i].plevel,
&debugfs_log_fops);
}

/* remove all the debugfs entries */
Expand Down

0 comments on commit 3b93c0f

Please sign in to comment.