On Tue, 2013-02-26 at 10:19 +0800, Wei Yongjun wrote: > From: Wei Yongjun <[email protected]> > > This removes an open coded simple_open() function and > replaces file operations references to the function > with simple_open() instead. > > Signed-off-by: Wei Yongjun <[email protected]> > --- > drivers/firmware/efivars.c | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/drivers/firmware/efivars.c b/drivers/firmware/efivars.c > index ef5070d..7c3477e 100644 > --- a/drivers/firmware/efivars.c > +++ b/drivers/firmware/efivars.c > @@ -647,12 +647,6 @@ efivar_unregister(struct efivar_entry *var) > kobject_put(&var->kobj); > } > > -static int efivarfs_file_open(struct inode *inode, struct file *file) > -{ > - file->private_data = inode->i_private; > - return 0; > -} > - > static int efi_status_to_err(efi_status_t status) > { > int err; > @@ -870,7 +864,7 @@ static struct super_block *efivarfs_sb; > static const struct inode_operations efivarfs_dir_inode_operations; > > static const struct file_operations efivarfs_file_operations = { > - .open = efivarfs_file_open, > + .open = simple_open, > .read = efivarfs_file_read, > .write = efivarfs_file_write, > .llseek = no_llseek, >
Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
