On Sun, Jun 12, 2016 at 7:48 AM, Sudip Mukherjee
<sudipm.mukher...@gmail.com> wrote:
> If devm_add_action() fails we are explicitly calling the cleanup to free
> the resources allocated. Lets use the helper devm_add_action_or_reset()
> and return directly in case of error, as we know that the cleanup
> function has been already called by the helper if there was any error.
>
> Signed-off-by: Sudip Mukherjee <sudip.mukher...@codethink.co.uk>

Reviewed-by: Dan Williams <dan.j.willi...@intel.com>

> ---
>
> There is no maintainer mentioned for this driver, so not sure who should
> this patch go through.

*raises hand*, I'll make sure MAINTAINERS gets updated, for this one
it seems Andrew has already taken it.  I notice that drivers/nvdimm/
could use the same cleanup.  I'll take a look unless you beat me to
it.

Reply via email to