Re: [PATCH] [media] v4l2-async: failing functions shouldn't have side effects

2017-01-30 Thread Sakari Ailus
Heippa! On Fri, Jan 27, 2017 at 12:32:56PM +0200, Tuukka Toivonen wrote: > v4l2-async had several functions doing some operations and then > not undoing the operations in a failure situation. For example, > v4l2_async_test_notify() moved a subdev into notifier's done list > even if registering the

[PATCH] [media] v4l2-async: failing functions shouldn't have side effects

2017-01-27 Thread Tuukka Toivonen
v4l2-async had several functions doing some operations and then not undoing the operations in a failure situation. For example, v4l2_async_test_notify() moved a subdev into notifier's done list even if registering the subdev (v4l2_device_register_subdev) failed. If the subdev was allocated and v4l2