Re: [PATCH 2/2] media: v4l2-async: Make subdev notifier cleanup conditional

From: Sakari Ailus
Date: Wed Jun 09 2021 - 08:27:43 EST


Hi Paul,

On Wed, Jun 09, 2021 at 01:54:57PM +0200, Paul Kocialkowski wrote:
> A dedicated subdev notified is registered when using the helper
> dedicated to sensors (v4l2_async_register_subdev_sensor_common),
> but this is not the case when a driver uses v4l2_async_register_subdev
> directly.

Is this a problem?

The notifier unregistration and cleanup functions should be safe to call on
a notifier that's not been initialised or registered. The same goes for
kfree with NULL argument.

--
Kind regards,

Sakari Ailus