Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Fabio M. De Francesco" <fmdefrancesco@gmail.com>
To:     "Ryan C. England" <rcengland@gmail.com>
Cc:     gregkh@linuxfoundation.org, arve@android.com,
        linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev,
        "Ryan C. England" <rcengland@gmail.com>
Subject: Re: [PATCH] staging: android: ashmem: Fixed a struct coding style issue
Date:   Sun, 20 Mar 2022 07:49:24 +0100
Message-ID: <2821664.e9J7NaK4W3@leap>
In-Reply-To: <20220320033337.47118-1-rcengland@gmail.com>
References: <20220320033337.47118-1-rcengland@gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="ISO-8859-1"
Precedence: bulk

On domenica 20 marzo 2022 04:33:37 CET Ryan C. England wrote:
> Fixed a coding style issue.
> 
> Signed-off-by: Ryan C. England <rcengland@gmail.com>
> ---
>  drivers/staging/android/ashmem.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/android/ashmem.c b/drivers/staging/android/ashmem.c
> index ddbde3f8430e..f2bf7995b403 100644
> --- a/drivers/staging/android/ashmem.c
> +++ b/drivers/staging/android/ashmem.c
> @@ -377,7 +377,7 @@ ashmem_vmfile_get_unmapped_area(struct file *file, unsigned long addr,
>  
>  static int ashmem_mmap(struct file *file, struct vm_area_struct *vma)
>  {
> -	static struct file_operations vmfile_fops;
> +	const struct file_operations vmfile_fops;
>  	struct ashmem_area *asma = file->private_data;
>  	int ret = 0;
>  
> -- 
> 2.27.0
> 
Hi Ryan,

Welcome to Linux development.

You are working on an old version of the staging tree. Please rebase to
the latest version of staging-next. drivers/android/ is not anymore here.

Aside from the above, what style issue were you trying to fix? Everything 
looks fine in the line that you changed,

Why did you decide to add the "const" qualifier to vmfile_fops? Do you 
know that since then the fields of vmfile_fops are not anymore assignable?
What should we do with an empty list of file operations?

Furthermore, why did you drop the "static" memory qualifier? Do you know
why it was (I suppose correctly) "static"?

Please don't ever forget to build your code before submitting patches.

Thanks,

Fabio M. De Francesco