diff options
author | Cornelia Huck <cornelia.huck@de.ibm.com> | 2007-02-14 01:57:47 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-02-16 18:19:17 -0500 |
commit | 873760fbf4d1c8c477a50386438303b6b89b6566 (patch) | |
tree | e5ad4f6df5806b9597be59e3fb6277be265ac9fe /fs | |
parent | 66f5496393dcc9f9d05c46f00ed93d5040d6035b (diff) |
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: Cornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/debugfs/inode.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/fs/debugfs/inode.c b/fs/debugfs/inode.c index 9c51a9f630a1..7b324cfebcb1 100644 --- a/fs/debugfs/inode.c +++ b/fs/debugfs/inode.c | |||
@@ -212,9 +212,7 @@ static int debugfs_create_by_name(const char *name, mode_t mode, | |||
212 | * you are responsible here.) If an error occurs, %NULL will be returned. | 212 | * you are responsible here.) If an error occurs, %NULL will be returned. |
213 | * | 213 | * |
214 | * If debugfs is not enabled in the kernel, the value -%ENODEV will be | 214 | * If debugfs is not enabled in the kernel, the value -%ENODEV will be |
215 | * returned. It is not wise to check for this value, but rather, check for | 215 | * returned. |
216 | * %NULL or !%NULL instead as to eliminate the need for #ifdef in the calling | ||
217 | * code. | ||
218 | */ | 216 | */ |
219 | struct dentry *debugfs_create_file(const char *name, mode_t mode, | 217 | struct dentry *debugfs_create_file(const char *name, mode_t mode, |
220 | struct dentry *parent, void *data, | 218 | struct dentry *parent, void *data, |
@@ -264,9 +262,7 @@ EXPORT_SYMBOL_GPL(debugfs_create_file); | |||
264 | * you are responsible here.) If an error occurs, %NULL will be returned. | 262 | * you are responsible here.) If an error occurs, %NULL will be returned. |
265 | * | 263 | * |
266 | * If debugfs is not enabled in the kernel, the value -%ENODEV will be | 264 | * If debugfs is not enabled in the kernel, the value -%ENODEV will be |
267 | * returned. It is not wise to check for this value, but rather, check for | 265 | * returned. |
268 | * %NULL or !%NULL instead as to eliminate the need for #ifdef in the calling | ||
269 | * code. | ||
270 | */ | 266 | */ |
271 | struct dentry *debugfs_create_dir(const char *name, struct dentry *parent) | 267 | struct dentry *debugfs_create_dir(const char *name, struct dentry *parent) |
272 | { | 268 | { |
@@ -297,9 +293,7 @@ EXPORT_SYMBOL_GPL(debugfs_create_dir); | |||
297 | * returned. | 293 | * returned. |
298 | * | 294 | * |
299 | * If debugfs is not enabled in the kernel, the value -%ENODEV will be | 295 | * If debugfs is not enabled in the kernel, the value -%ENODEV will be |
300 | * returned. It is not wise to check for this value, but rather, check for | 296 | * returned. |
301 | * %NULL or !%NULL instead as to eliminate the need for #ifdef in the calling | ||
302 | * code. | ||
303 | */ | 297 | */ |
304 | struct dentry *debugfs_create_symlink(const char *name, struct dentry *parent, | 298 | struct dentry *debugfs_create_symlink(const char *name, struct dentry *parent, |
305 | const char *target) | 299 | const char *target) |