> +static void qmp_debugfs_create(struct qmp *qmp) > +{ > + const struct qmp_debugfs_entry *entry; > + int i; > + > + qmp->debugfs_root = debugfs_create_dir("qcom_aoss", NULL); > + > + for (i = 0; i < ARRAY_SIZE(qmp->debugfs_files); i++) { > + entry = &qmp_debugfs_entries[i]; > + > + qmp->debugfs_files[i] = debugfs_create_file(entry->name, 0220, group write? Generally you only want root able to do writes. However, thanks for reworking this to just support some well defined commands, rather than binary blobs from user space. Andrew