RE: [PATCH] dax: use devm_add_action_or_reset()

2016-07-06 Thread Sajjan, Vikas C
01.org; Linux ACPI <linux-a...@vger.kernel.org>; Knippers, Linda <linda.knipp...@hpe.com> Subject: Re: [PATCH] dax: use devm_add_action_or_reset() Thanks, applied. Thank you, Dan. On Mon, Jul 4, 2016 at 11:59 PM, Johannes Thumshirn <jthumsh...@suse.de> wrote: > On Tue, Ju

Re: [PATCH] dax: use devm_add_action_or_reset()

2016-07-06 Thread Dan Williams
Thanks, applied. On Mon, Jul 4, 2016 at 11:59 PM, Johannes Thumshirn wrote: > On Tue, Jul 05, 2016 at 11:20:07AM +0530, Vikas C Sajjan wrote: >> If devm_add_action() fails, we are explicitly calling the cleanup to free >> the resources allocated. Use the helper

Re: [PATCH] dax: use devm_add_action_or_reset()

2016-07-05 Thread Johannes Thumshirn
On Tue, Jul 05, 2016 at 11:20:07AM +0530, Vikas C Sajjan wrote: > If devm_add_action() fails, we are explicitly calling the cleanup to free > the resources allocated. Use the helper devm_add_action_or_reset() > and return directly in case of error, since the cleanup function > has been already