On Tue, Nov 08, 2016 at 07:51:33PM +0200, Mikko Perttunen wrote: [...] > @@ -86,7 +88,17 @@ static struct host1x_syncpt *host1x_syncpt_alloc(struct host1x *host, > else > sp->client_managed = false; > > + mutex_unlock(&host->syncpt_mutex); > return sp; > + > +err_alloc_name: It's better to use labels that describe what they do, rather than when they get called. > + host1x_syncpt_base_free(sp->base); > + sp->base = NULL; > +err_alloc_base: > + sp = NULL; This is useless because the variable is no longer used after this and goes out of scope. I fixed up these two issues and applied. Thanks, Thierry
Attachment:
signature.asc
Description: PGP signature
