Messages in this thread | | | Date | Mon, 2 May 2016 18:11:51 -0300 | From | Gustavo Padovan <> | Subject | Re: [PATCH 01/12] staging/android: remove redundant comments on sync_merge_data |
| |
2016-05-02 Pavel Machek <pavel@ucw.cz>:
> On Wed 2016-04-27 13:27:08, Gustavo Padovan wrote: > > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > > > 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@collabora.co.uk> > > Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> > > > @@ -33,8 +33,8 @@ struct sync_merge_data { > > /** > > * struct sync_fence_info - detailed fence information > > * @obj_name: name of parent sync_timeline > > - * @driver_name: name of driver implementing the parent > > - * @status: status of the fence 0:active 1:signaled <0:error > > +* @driver_name: name of driver implementing the parent > > +* @status: status of the fence 0:active 1:signaled <0:error > > The whitespace (or mail client configuration?) looks wrong here.
this has been fixed in v2 already and Greg pulled everything into staging next already.
Gustavo
| |