Re: [PATCH 0/4] v4l: async: fixes for v4l2_async_notifier_unregister()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/31/2017 12:31 AM, Niklas Söderlund wrote:
> Hi,
> 
> This series is based on top of media-tree and some patches where
> previously part of the series '[PATCH v5 0/4] v4l2-async: add
> subnotifier registration for subdevices'. Hans suggested the cleanups
> could be broken out to a separate series, so this is this series :-)
> 
> The aim of this series is to cleanup and document some of the odd things
> that happens in v4l2_async_notifier_unregister(). The purpose for this
> in the short term is to make it easier to implement subnotifiers which
> both I and Sakari are trying to address, this feature is blocking other
> drivers such as the Renesas R-Car CSI-2 receiver driver. And in the long
> run (I hope) to make it easier to get rid of the need to do re-probing
> at all in v4l2_async_notifier_unregister() :-)
> 
> Niklas Söderlund (4):
>   v4l: async: fix unbind error in v4l2_async_notifier_unregister()
>   v4l: async: abort if memory allocation fails when unregistering
>     notifiers
>   v4l: async: do not hold list_lock when re-probing devices
>   v4l: async: add comment about re-probing to
>     v4l2_async_notifier_unregister()
> 
>  drivers/media/v4l2-core/v4l2-async.c | 49 ++++++++++++++++++++----------------
>  1 file changed, 28 insertions(+), 21 deletions(-)
> 

Looks good to me, but I'd like Sakari's Ack as well. He's still on vacation
for another week, so it won't be merged before next week at the earliest.

Regards,

	Hans



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux