Re: [PATCH 1/3] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan
Date: Wed Apr 20 2016 - 15:35:07 EST


I messed up with the subject prefix, but this is v11, adds typecheck()
to patch 2.

2016-04-20 Gustavo Padovan <gustavo@xxxxxxxxxxx>:

> From: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
>
> struct sync_merge_data already have documentation on top of the
> struct definition. No need to duplicate it.
>
> Signed-off-by: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> ---
> drivers/staging/android/uapi/sync.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/staging/android/uapi/sync.h b/drivers/staging/android/uapi/sync.h
> index a0cf357..4467c76 100644
> --- a/drivers/staging/android/uapi/sync.h
> +++ b/drivers/staging/android/uapi/sync.h
> @@ -21,9 +21,9 @@
> * @fence: returns the fd of the new fence to userspace
> */
> struct sync_merge_data {
> - __s32 fd2; /* fd of second fence */
> - char name[32]; /* name of new fence */
> - __s32 fence; /* fd on newly created fence */
> + __s32 fd2;
> + char name[32];
> + __s32 fence;
> };
>
> /**
> --
> 2.5.5
>

Gustavo