[PATCH 4/4] Staging: android: Fix struct definitioni warning from static to const

Dan Carpenter dan.carpenter at oracle.com
Sat Jan 21 08:08:38 UTC 2012


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/attachments/20120121/7ae3e4a6/attachment.asc>


More information about the devel mailing list