Re: [PATCH 1/1] platform/x86: surface3-wmi: Balance locking on error path

2016-12-15 Thread Benjamin Tissoires
On Dec 15 2016 or thereabouts, Andy Shevchenko wrote: > There is a possibility that lock will be left acquired. > Consolidate error path under out_free_unlock label. > > Reported-by: kbuild test robot > Cc: Benjamin Tissoires > Signed-off-by: Andy Shevchenko With Darren's remarks, the patch is

Re: [PATCH 1/1] platform/x86: surface3-wmi: Balance locking on error path

2016-12-14 Thread Darren Hart
On Thu, Dec 15, 2016 at 03:24:58AM +0200, Andy Shevchenko wrote: > There is a possibility that lock will be left acquired. > Consolidate error path under out_free_unlock label. > > Reported-by: kbuild test robot > Cc: Benjamin Tissoires > Signed-off-by: Andy Shevchenko > --- > drivers/platform

[PATCH 1/1] platform/x86: surface3-wmi: Balance locking on error path

2016-12-14 Thread Andy Shevchenko
There is a possibility that lock will be left acquired. Consolidate error path under out_free_unlock label. Reported-by: kbuild test robot Cc: Benjamin Tissoires Signed-off-by: Andy Shevchenko --- drivers/platform/x86/surface3-wmi.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletion

[PATCH 1/1] platform/x86: surface3-wmi: Balance locking on error path

2016-12-14 Thread Andy Shevchenko
There is a possibility that lock will be left acquired. Consolidate error path under out_free_unlock label. Reported-by: kbuild test robot Cc: Benjamin Tissoires Signed-off-by: Andy Shevchenko --- drivers/platform/x86/surface3-wmi.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletion