On Tue, 2013-02-26 at 10:19 +0800, Wei Yongjun wrote:
> From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
>
> 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 <yongjun_wei@xxxxxxxxxxxxxxxxx>
> ---
> 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
