diff options
author | Tejun Heo <htejun@gmail.com> | 2007-08-02 08:38:03 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-10-12 17:51:04 -0400 |
commit | 990e53f880be9ff93072b4cce590ec2826cee0b6 (patch) | |
tree | faa7cbedef2f1d9b8d27832ccbaa28e508d134e8 /fs/sysfs/sysfs.h | |
parent | 23dc279950a056c33a14d09cf759f5173d41abd9 (diff) |
sysfs: make sysfs_addrm_finish() return void
With the previous sysfs_add_one() update, there is only one user of
the return value of sysfs_addrm_finish() and the user can switch to
testing @sd easily. Make sysfs_addrm_finish() return void for cleaner
semantics as suggested by Satyam Sharma.
This patch doesn't introduce any noticeable behavior change.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Cc: Satyam Sharma <satyam.sharma@gmail.com>
Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'fs/sysfs/sysfs.h')
-rw-r--r-- | fs/sysfs/sysfs.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/sysfs/sysfs.h b/fs/sysfs/sysfs.h index bb3f0c999b15..043675471031 100644 --- a/fs/sysfs/sysfs.h +++ b/fs/sysfs/sysfs.h | |||
@@ -66,7 +66,7 @@ extern int sysfs_add_one(struct sysfs_addrm_cxt *acxt, | |||
66 | struct sysfs_dirent *sd); | 66 | struct sysfs_dirent *sd); |
67 | extern void sysfs_remove_one(struct sysfs_addrm_cxt *acxt, | 67 | extern void sysfs_remove_one(struct sysfs_addrm_cxt *acxt, |
68 | struct sysfs_dirent *sd); | 68 | struct sysfs_dirent *sd); |
69 | extern int sysfs_addrm_finish(struct sysfs_addrm_cxt *acxt); | 69 | extern void sysfs_addrm_finish(struct sysfs_addrm_cxt *acxt); |
70 | 70 | ||
71 | extern struct inode * sysfs_get_inode(struct sysfs_dirent *sd); | 71 | extern struct inode * sysfs_get_inode(struct sysfs_dirent *sd); |
72 | extern void sysfs_instantiate(struct dentry *dentry, struct inode *inode); | 72 | extern void sysfs_instantiate(struct dentry *dentry, struct inode *inode); |