Hi Sakari, Thank you for the patch. On Wed, Mar 09, 2022 at 07:10:06PM +0200, Sakari Ailus wrote: > Document that drivers must first initialise a media device before > registering it, and clean up once the media device is unregistered. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > --- > Documentation/driver-api/media/mc-core.rst | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > diff --git a/Documentation/driver-api/media/mc-core.rst b/Documentation/driver-api/media/mc-core.rst > index 57b5bbba944e..02481a2513b9 100644 > --- a/Documentation/driver-api/media/mc-core.rst > +++ b/Documentation/driver-api/media/mc-core.rst > @@ -42,9 +42,16 @@ Allocation of the structure is handled by the media device driver, usually by > embedding the :c:type:`media_device` instance in a larger driver-specific > structure. > > -Drivers register media device instances by calling > -:c:func:`__media_device_register()` via the macro ``media_device_register()`` > -and unregistered by calling :c:func:`media_device_unregister()`. > +Drivers initialise media device instances by calling > +:c:func:`media_device_init()`. After initialising a media device instance, it is > +registered by calling :c:func:`__media_device_register()` via the macro > +``media_device_register()`` and unregistered by calling > +:c:func:`media_device_unregister()`. An initialised media device must be > +eventually cleaned up by calling :c:func:`media_device_cleanup()`. > + > +Note that it is not allowed to unregister a media device instance that was not > +previously registered, or clean up a media device instance that was not > +previously initialised. Could we fix this, to simplify drivers ? This patch is fine as it documents the current behaviour, so Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> but it will need to be updated. Bonus points if the update can go in a v2 ;-) > Entities > ^^^^^^^^ -- Regards, Laurent Pinchart
