Re: [PATCH] ionic: fix simple_open.cocci warnings

[Shannon Nelson <snelson@pensando.io>]

On 6/21/19 2:27 AM, kbuild test robot wrote:
> From: kbuild test robot <lkp@intel.com>
>
> drivers/net/ethernet/pensando/ionic/ionic_debugfs.c:12:11-20: WARNING opportunity for simple_open, see also structure on line 56
>
>   Remove an open coded simple_open() function
>   and replace file operations references to the function
>   with simple_open() instead.
>
> Generated by: scripts/coccinelle/api/simple_open.cocci
>
> Fixes: fb4f52d18a75 ("ionic: Add hardware init and device commands")
> CC: Shannon Nelson <snelson@pensando.io>
> Signed-off-by: kbuild test robot <lkp@intel.com>

Thanks, I'll roll this into the next verison of the patchset.
sln

> ---
>
> url:    https://github.com/0day-ci/linux/commits/Shannon-Nelson/ionic-Add-basic-framework-for-IONIC-Network-device-driver/20190621-110046
>
>   ionic_debugfs.c |    8 +-------
>   1 file changed, 1 insertion(+), 7 deletions(-)
>
> --- a/drivers/net/ethernet/pensando/ionic/ionic_debugfs.c
> +++ b/drivers/net/ethernet/pensando/ionic/ionic_debugfs.c
> @@ -9,12 +9,6 @@
>   
>   #ifdef CONFIG_DEBUG_FS
>   
> -static int blob_open(struct inode *inode, struct file *filp)
> -{
> -	filp->private_data = inode->i_private;
> -	return 0;
> -}
> -
>   static ssize_t blob_read(struct file *filp, char __user *buffer,
>   			 size_t count, loff_t *ppos)
>   {
> @@ -53,7 +47,7 @@ static ssize_t blob_write(struct file *f
>   
>   static const struct file_operations blob_fops = {
>   	.owner = THIS_MODULE,
> -	.open = blob_open,
> +	.open = simple_open,
>   	.read = blob_read,
>   	.write = blob_write,
>   };