remoteproc: debugfs: Remove state entry which is duplicated is sysfs
Since there is now an always available state file in sysfs with the same function as this one in debugfs, remove the redundant entry. Signed-off-by: Matt Redfearn <matt.redfearn@imgtec.com> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
This commit is contained in:
parent
2aefbef041
commit
fd50d8c061
@ -59,75 +59,6 @@ static const struct file_operations trace_rproc_ops = {
|
|||||||
.llseek = generic_file_llseek,
|
.llseek = generic_file_llseek,
|
||||||
};
|
};
|
||||||
|
|
||||||
/*
|
|
||||||
* A state-to-string lookup table, for exposing a human readable state
|
|
||||||
* via debugfs. Always keep in sync with enum rproc_state
|
|
||||||
*/
|
|
||||||
static const char * const rproc_state_string[] = {
|
|
||||||
"offline",
|
|
||||||
"suspended",
|
|
||||||
"running",
|
|
||||||
"crashed",
|
|
||||||
"invalid",
|
|
||||||
};
|
|
||||||
|
|
||||||
/* expose the state of the remote processor via debugfs */
|
|
||||||
static ssize_t rproc_state_read(struct file *filp, char __user *userbuf,
|
|
||||||
size_t count, loff_t *ppos)
|
|
||||||
{
|
|
||||||
struct rproc *rproc = filp->private_data;
|
|
||||||
unsigned int state;
|
|
||||||
char buf[30];
|
|
||||||
int i;
|
|
||||||
|
|
||||||
state = rproc->state > RPROC_LAST ? RPROC_LAST : rproc->state;
|
|
||||||
|
|
||||||
i = scnprintf(buf, 30, "%.28s (%d)\n", rproc_state_string[state],
|
|
||||||
rproc->state);
|
|
||||||
|
|
||||||
return simple_read_from_buffer(userbuf, count, ppos, buf, i);
|
|
||||||
}
|
|
||||||
|
|
||||||
static ssize_t rproc_state_write(struct file *filp, const char __user *userbuf,
|
|
||||||
size_t count, loff_t *ppos)
|
|
||||||
{
|
|
||||||
struct rproc *rproc = filp->private_data;
|
|
||||||
char buf[10];
|
|
||||||
int ret;
|
|
||||||
|
|
||||||
if (count > sizeof(buf) || count <= 0)
|
|
||||||
return -EINVAL;
|
|
||||||
|
|
||||||
ret = copy_from_user(buf, userbuf, count);
|
|
||||||
if (ret)
|
|
||||||
return -EFAULT;
|
|
||||||
|
|
||||||
if (buf[count - 1] == '\n')
|
|
||||||
buf[count - 1] = '\0';
|
|
||||||
|
|
||||||
if (!strncmp(buf, "start", count)) {
|
|
||||||
ret = rproc_boot(rproc);
|
|
||||||
if (ret) {
|
|
||||||
dev_err(&rproc->dev, "Boot failed: %d\n", ret);
|
|
||||||
return ret;
|
|
||||||
}
|
|
||||||
} else if (!strncmp(buf, "stop", count)) {
|
|
||||||
rproc_shutdown(rproc);
|
|
||||||
} else {
|
|
||||||
dev_err(&rproc->dev, "Unrecognised option: %s\n", buf);
|
|
||||||
return -EINVAL;
|
|
||||||
}
|
|
||||||
|
|
||||||
return count;
|
|
||||||
}
|
|
||||||
|
|
||||||
static const struct file_operations rproc_state_ops = {
|
|
||||||
.read = rproc_state_read,
|
|
||||||
.write = rproc_state_write,
|
|
||||||
.open = simple_open,
|
|
||||||
.llseek = generic_file_llseek,
|
|
||||||
};
|
|
||||||
|
|
||||||
/* expose the name of the remote processor via debugfs */
|
/* expose the name of the remote processor via debugfs */
|
||||||
static ssize_t rproc_name_read(struct file *filp, char __user *userbuf,
|
static ssize_t rproc_name_read(struct file *filp, char __user *userbuf,
|
||||||
size_t count, loff_t *ppos)
|
size_t count, loff_t *ppos)
|
||||||
@ -265,8 +196,6 @@ void rproc_create_debug_dir(struct rproc *rproc)
|
|||||||
|
|
||||||
debugfs_create_file("name", 0400, rproc->dbg_dir,
|
debugfs_create_file("name", 0400, rproc->dbg_dir,
|
||||||
rproc, &rproc_name_ops);
|
rproc, &rproc_name_ops);
|
||||||
debugfs_create_file("state", 0400, rproc->dbg_dir,
|
|
||||||
rproc, &rproc_state_ops);
|
|
||||||
debugfs_create_file("recovery", 0400, rproc->dbg_dir,
|
debugfs_create_file("recovery", 0400, rproc->dbg_dir,
|
||||||
rproc, &rproc_recovery_ops);
|
rproc, &rproc_recovery_ops);
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user