Commit 873760fb authored by Cornelia Huck's avatar Cornelia Huck Committed by Greg Kroah-Hartman

debugfs: Remove misleading comments.

Just mention which error will be returned if debugfs is disabled. Callers
should be able to figure out themselves what they need to check.
Signed-off-by: default avatarCornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 66f54963
...@@ -212,9 +212,7 @@ static int debugfs_create_by_name(const char *name, mode_t mode, ...@@ -212,9 +212,7 @@ static int debugfs_create_by_name(const char *name, mode_t mode,
* you are responsible here.) If an error occurs, %NULL will be returned. * you are responsible here.) If an error occurs, %NULL will be returned.
* *
* If debugfs is not enabled in the kernel, the value -%ENODEV will be * If debugfs is not enabled in the kernel, the value -%ENODEV will be
* returned. It is not wise to check for this value, but rather, check for * returned.
* %NULL or !%NULL instead as to eliminate the need for #ifdef in the calling
* code.
*/ */
struct dentry *debugfs_create_file(const char *name, mode_t mode, struct dentry *debugfs_create_file(const char *name, mode_t mode,
struct dentry *parent, void *data, struct dentry *parent, void *data,
...@@ -264,9 +262,7 @@ EXPORT_SYMBOL_GPL(debugfs_create_file); ...@@ -264,9 +262,7 @@ EXPORT_SYMBOL_GPL(debugfs_create_file);
* you are responsible here.) If an error occurs, %NULL will be returned. * you are responsible here.) If an error occurs, %NULL will be returned.
* *
* If debugfs is not enabled in the kernel, the value -%ENODEV will be * If debugfs is not enabled in the kernel, the value -%ENODEV will be
* returned. It is not wise to check for this value, but rather, check for * returned.
* %NULL or !%NULL instead as to eliminate the need for #ifdef in the calling
* code.
*/ */
struct dentry *debugfs_create_dir(const char *name, struct dentry *parent) struct dentry *debugfs_create_dir(const char *name, struct dentry *parent)
{ {
...@@ -297,9 +293,7 @@ EXPORT_SYMBOL_GPL(debugfs_create_dir); ...@@ -297,9 +293,7 @@ EXPORT_SYMBOL_GPL(debugfs_create_dir);
* returned. * returned.
* *
* If debugfs is not enabled in the kernel, the value -%ENODEV will be * If debugfs is not enabled in the kernel, the value -%ENODEV will be
* returned. It is not wise to check for this value, but rather, check for * returned.
* %NULL or !%NULL instead as to eliminate the need for #ifdef in the calling
* code.
*/ */
struct dentry *debugfs_create_symlink(const char *name, struct dentry *parent, struct dentry *debugfs_create_symlink(const char *name, struct dentry *parent,
const char *target) const char *target)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment