devfs_mk_symlink simplification
Subsystems (3)
PATCH 1/1 devfs_mk_symlink simplification
All devfs_mk_symlink arguments except the from and to strings are unused. Bring the prototype in shape.
arch/um/drivers/line.c +1 -1
@@ -438,7 +438,7 @@ from = line_driver->symlink_from; to = line_driver->symlink_to; - err =devfs_mk_symlink (NULL , from,0 , to,NULL ,NULL ); + err =devfs_mk_symlink (from, to);if (err)printk ("Symlink creation from /dev/%s to /dev/%s " "returned %d\n" , from, to, err);
arch/um/drivers/mmapper_kern.c +1 -2
@@ -127,8 +127,7 @@devfs_register (NULL ,"mmapper" ,DEVFS_FL_DEFAULT ,30 ,0 ,S_IFCHR |S_IRUGO |S_IWUGO , &mmapper_fops,NULL ); -devfs_mk_symlink (NULL ,"mmapper0" ,DEVFS_FL_DEFAULT ,"mmapper" , -NULL ,NULL ); +devfs_mk_symlink ("mmapper0" ,"mmapper" );return (0 ); }
drivers/md/dm-ioctl.c +1 -4
@@ -1089,8 +1089,6 @@ .owner = THIS_MODULE, }; -static devfs_handle_t _ctl_handle; -static struct miscdevice _dm_misc = { .minor =MISC_DYNAMIC_MINOR , .name =DM_NAME ,
@@ -1115,8 +1113,7 @@return r; } - r =devfs_mk_symlink (NULL ,DM_DIR "/control" ,DEVFS_FL_DEFAULT , -"../misc/" DM_NAME , &_ctl_handle,NULL ); + r =devfs_mk_symlink (DM_DIR "/control" ,"../misc/" DM_NAME );if (r) {DMERR ("devfs_mk_symlink failed for control device" ); goto failed;
drivers/media/radio/miropcm20-rds.c +2 -2
@@ -125,8 +125,8 @@if (error)return error; - error =devfs_mk_symlink (NULL ,"v4l/rds/radiotext" ,0 , -"../misc/radiotext" ,NULL ,NULL ); + error =devfs_mk_symlink ("v4l/rds/radiotext" , +"../misc/radiotext" );if (error)misc_deregister (&rds_miscdev);
fs/devfs/base.c +16 -25
71 lines not on this page · raw message
fs/devfs/util.c +1 -2
@@ -87,7 +87,6 @@int devfs_register_tape (devfs_handle_t de) {int pos; -devfs_handle_t slave;char name[32 ], dest[64 ];static unsigned int tape_counter;int n = tape_counter++;
@@ -96,7 +95,7 @@if (pos <0 )return -1 ;strncpy (dest + pos,"../" ,3 );sprintf (name,"tapes/tape%u" , n); -devfs_mk_symlink (NULL , name,DEVFS_FL_DEFAULT , dest + pos, &slave,NULL ); +devfs_mk_symlink (name, dest + pos);return n; }/* End Function devfs_register_tape */ EXPORT_SYMBOL (devfs_register_tape);
fs/partitions/check.c +3 -6
@@ -185,7 +185,7 @@ {#ifdef CONFIG_DEVFS_FS int pos =0 ; -devfs_handle_t dir, slave; +devfs_handle_t dir;char dirname[64 ], symlink[16 ];if (dev->flags &GENHD_FL_DEVFS ) {
@@ -205,8 +205,7 @@ } dev->number =devfs_alloc_unique_number (&disc_numspace);sprintf (symlink,"discs/disc%d" , dev->number); -devfs_mk_symlink (NULL , symlink,DEVFS_FL_DEFAULT , - dirname + pos, &slave,NULL ); +devfs_mk_symlink (symlink, dirname + pos); dev->disk_de =devfs_register (dir,"disc" ,0 , dev->major, dev->first_minor,S_IFBLK |S_IRUSR |S_IWUSR , dev->fops,NULL );
@@ -222,7 +221,6 @@sprintf (vname,"cdroms/cdrom%d" , dev->number);if (dev->de) {int pos; -devfs_handle_t slave;char rname[64 ]; dev->disk_de = devfs_register(dev->de, "cd", DEVFS_FL_DEFAULT,
@@ -233,8 +231,7 @@ pos =devfs_generate_path (dev->disk_de, rname+3 ,sizeof (rname)-3 );if (pos >=0 ) {strncpy (rname + pos,"../" ,3 ); -devfs_mk_symlink (NULL , vname,DEVFS_FL_DEFAULT , - rname + pos, &slave,NULL ); +devfs_mk_symlink (vname, rname + pos); } }else { dev->disk_de = devfs_register (NULL, vname, DEVFS_FL_DEFAULT,
include/linux/devfs_fs_kernel.h +2 -6
@@ -41,9 +41,7 @@ unsigned int major, unsigned int minor, umode_t mode, void *ops, void *info);extern void devfs_unregister (devfs_handle_t de); -extern int devfs_mk_symlink (devfs_handle_t dir,const char *name, -unsigned int flags,const char *link, -devfs_handle_t *handle,void *info); +extern int devfs_mk_symlink (const char *name,const char *link);extern devfs_handle_t devfs_mk_dir (devfs_handle_t dir,const char *name,void *info);extern int devfs_generate_path (devfs_handle_t de,char *path,int buflen);
@@ -78,9 +76,7 @@ {return ; } -static inline int devfs_mk_symlink (devfs_handle_t dir,const char *name, -unsigned int flags,const char *link, -devfs_handle_t *handle,void *info) +static inline int devfs_mk_symlink (const char *name,const char *link) {return 0 ; }
Hello!
Quoted 2 lines · Christoph Hellwig
All
devfs_mk_symlinkarguments except the from and to strings are unused. Bring the prototype in shape.
This patch is needed in UML subtree now:
Bye,
Oleg
===== arch/um/drivers/ubd_kern.c 1.33 vs edited =====
arch/um/drivers/ubd_kern.c +1 -1
@@ -577,7 +577,7 @@sprintf (link,"ubd/disc%d" , n);/* and we link it to /dev/ubd/discN */ -devfs_mk_symlink (NULL , link,DEVFS_FL_DEFAULT , real,NULL ,NULL ); +devfs_mk_symlink (link, real);if (fake_major !=MAJOR_NR ) {/* /dev/fakeN style names */ sprintf (real,"fake%d" ,n);
Hello!
Quoted 4 lines · Oleg Drokin
Quoted 2 lines · Christoph Hellwig
All
devfs_mk_symlinkarguments except the from and to strings are unused. Bring the prototype in shape.This patch is needed in UML subtree now:
Argh, this patch is only relevant to patched Jeff's tree, everybody else please ignore it.
Bye,
Oleg