Re: [PATCH 4/4] Staging: android: Fix struct definitioni warningfrom static to const

[Dan Carpenter]

On Sat, Jan 21, 2012 at 02:17:49AM +0000, Kashyap Gada wrote:
> ---
>  drivers/staging/android/ashmem.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/staging/android/ashmem.c b/drivers/staging/android/ashmem.c
> index 4a00174a5..9f1f27e 100644
> --- a/drivers/staging/android/ashmem.c
> +++ b/drivers/staging/android/ashmem.c
> @@ -680,7 +680,7 @@ static long ashmem_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
>  	return ret;
>  }
>  
> -const struct file_operations ashmem_fops = {
> +static const struct file_operations ashmem_fops = {
>  	.owner = THIS_MODULE,
>  	.open = ashmem_open,
>  	.release = ashmem_release,

No.  We're not going to merge the broken patch which removes the
static.

Plus your patch numbering is totally bogus.
[PATCH 2/2]
[PATCH 3/3]
[PATCH 4/4]
[PATCH 5/5]

The second number is supposed to say how many patches there are in
the series.  If these were bug fixes, we'd go out of our way to work
with you, but you're just sending random whitespace fixes so they
have to be pretty much perfect.  (In other words, please slow down
and work more carefully or else focus on fixing bugs instead of
whitespace).

regards,
dan carpenter

Attachment: signature.asc
Description: Digital signature