media: vimc: move media_entity_cleanup to release callbacks

according to the docs, this function must be called during
the cleanup phase after unregistering the entity.

Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com>
Acked-by: Helen Koike <helen.koike@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>

diff --git a/drivers/media/platform/vimc/vimc-capture.c b/drivers/media/platform/vimc/vimc-capture.c
index 602f8032303173add8afc813e5e52f7d40528e3a..5f353c20e605f31deed9a136e8c850b0842e4ee2 100644 (file)
--- a/drivers/media/platform/vimc/vimc-capture.c
+++ b/drivers/media/platform/vimc/vimc-capture.c
@@ -330,6 +330,7 @@ static void vimc_cap_release(struct video_device *vdev)
        struct vimc_cap_device *vcap =
                container_of(vdev, struct vimc_cap_device, vdev);
 
+       media_entity_cleanup(vcap->ved.ent);
        vimc_pads_cleanup(vcap->ved.pads);
        kfree(vcap);
 }
@@ -340,7 +341,6 @@ void vimc_cap_rm(struct vimc_device *vimc, struct vimc_ent_device *ved)
 
        vcap = container_of(ved, struct vimc_cap_device, ved);
        vb2_queue_release(&vcap->queue);
-       media_entity_cleanup(ved->ent);
        video_unregister_device(&vcap->vdev);
 }
 

diff --git a/drivers/media/platform/vimc/vimc-common.c b/drivers/media/platform/vimc/vimc-common.c
index a3120f4f7a90e8d0834fd57ca9272455c534f9dd..999bc353fb102eb248824f9733f37c7d6ae69197 100644 (file)
--- a/drivers/media/platform/vimc/vimc-common.c
+++ b/drivers/media/platform/vimc/vimc-common.c
@@ -423,7 +423,6 @@ EXPORT_SYMBOL_GPL(vimc_ent_sd_register);
 
 void vimc_ent_sd_unregister(struct vimc_ent_device *ved, struct v4l2_subdev *sd)
 {
-       media_entity_cleanup(ved->ent);
        v4l2_device_unregister_subdev(sd);
 }
 EXPORT_SYMBOL_GPL(vimc_ent_sd_unregister);

diff --git a/drivers/media/platform/vimc/vimc-debayer.c b/drivers/media/platform/vimc/vimc-debayer.c
index feac47d79449780f06af677d1d8dbf07bf0781ca..e1bad6713cde9da23f3e754c8ec527e3bd20c5f3 100644 (file)
--- a/drivers/media/platform/vimc/vimc-debayer.c
+++ b/drivers/media/platform/vimc/vimc-debayer.c
@@ -477,6 +477,7 @@ static void vimc_deb_release(struct v4l2_subdev *sd)
        struct vimc_deb_device *vdeb =
                                container_of(sd, struct vimc_deb_device, sd);
 
+       media_entity_cleanup(vdeb->ved.ent);
        vimc_pads_cleanup(vdeb->ved.pads);
        kfree(vdeb);
 }

diff --git a/drivers/media/platform/vimc/vimc-scaler.c b/drivers/media/platform/vimc/vimc-scaler.c
index a6a3cc5be8722becd31b45ffda646e597f9860cd..1982bc089af5f892f5b32d08e0bfab66ee94b87d 100644 (file)
--- a/drivers/media/platform/vimc/vimc-scaler.c
+++ b/drivers/media/platform/vimc/vimc-scaler.c
@@ -336,6 +336,7 @@ static void vimc_sca_release(struct v4l2_subdev *sd)
        struct vimc_sca_device *vsca =
                                container_of(sd, struct vimc_sca_device, sd);
 
+       media_entity_cleanup(vsca->ved.ent);
        vimc_pads_cleanup(vsca->ved.pads);
        kfree(vsca);
 }

diff --git a/drivers/media/platform/vimc/vimc-sensor.c b/drivers/media/platform/vimc/vimc-sensor.c
index ee2306c08569215990bc8b940e5008b2ed803fae..63fe024ccea5e7a9e090577292762e759a56dbd0 100644 (file)
--- a/drivers/media/platform/vimc/vimc-sensor.c
+++ b/drivers/media/platform/vimc/vimc-sensor.c
@@ -291,6 +291,7 @@ static void vimc_sen_release(struct v4l2_subdev *sd)
 
        v4l2_ctrl_handler_free(&vsen->hdl);
        tpg_free(&vsen->tpg);
+       media_entity_cleanup(vsen->ved.ent);
        vimc_pads_cleanup(vsen->ved.pads);
        kfree(vsen);
 }