[media] omap3isp: get entity ID using media_entity_id()
authorJavier Martinez Canillas <javier@osg.samsung.com>
Wed, 19 Aug 2015 15:35:20 +0000 (12:35 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Mon, 11 Jan 2016 14:18:41 +0000 (12:18 -0200)
Accessing media_entity ID should now use media_entity_id() macro to
obtain the entity ID, as a next patch will remove the .id field from
struct media_entity .

So, get rid of it, otherwise the omap3isp driver will fail to build.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/platform/omap3isp/isp.c
drivers/media/platform/omap3isp/ispccdc.c
drivers/media/platform/omap3isp/ispvideo.c

index 56e683b19a73e38c1fa60f4b4b12c92f8cd62393..e08183f9d0f78ea874607b19749e50bc0e369173 100644 (file)
@@ -975,6 +975,7 @@ static int isp_pipeline_disable(struct isp_pipeline *pipe)
        struct v4l2_subdev *subdev;
        int failure = 0;
        int ret;
+       u32 id;
 
        /*
         * We need to stop all the modules after CCDC first or they'll
@@ -1027,8 +1028,10 @@ static int isp_pipeline_disable(struct isp_pipeline *pipe)
                if (ret) {
                        dev_info(isp->dev, "Unable to stop %s\n", subdev->name);
                        isp->stop_failure = true;
-                       if (subdev == &isp->isp_prev.subdev)
-                               isp->crashed |= 1U << subdev->entity.id;
+                       if (subdev == &isp->isp_prev.subdev) {
+                               id = media_entity_id(&subdev->entity);
+                               isp->crashed |= 1U << id;
+                       }
                        failure = -ETIMEDOUT;
                }
        }
index 3b10304b580bd818177f17a3b8904d3705142d7a..d96e3be5e252f0d87aea3719d818b565e572cd0f 100644 (file)
@@ -1608,7 +1608,7 @@ static int ccdc_isr_buffer(struct isp_ccdc_device *ccdc)
        /* Wait for the CCDC to become idle. */
        if (ccdc_sbl_wait_idle(ccdc, 1000)) {
                dev_info(isp->dev, "CCDC won't become idle!\n");
-               isp->crashed |= 1U << ccdc->subdev.entity.id;
+               isp->crashed |= 1U << media_entity_id(&ccdc->subdev.entity);
                omap3isp_pipeline_cancel_stream(pipe);
                return 0;
        }
index 963cb9318873d0c8f073c3fcffbe24fff1762004..0e129075e99ffa328c44ecc63bbe116776bd4f00 100644 (file)
@@ -235,7 +235,7 @@ static int isp_video_get_graph_data(struct isp_video *video,
        while ((entity = media_entity_graph_walk_next(&graph))) {
                struct isp_video *__video;
 
-               pipe->entities |= 1 << entity->id;
+               pipe->entities |= 1 << media_entity_id(entity);
 
                if (far_end != NULL)
                        continue;
@@ -893,6 +893,7 @@ static int isp_video_check_external_subdevs(struct isp_video *video,
        struct v4l2_ext_control ctrl;
        unsigned int i;
        int ret;
+       u32 id;
 
        /* Memory-to-memory pipelines have no external subdev. */
        if (pipe->input != NULL)
@@ -900,7 +901,7 @@ static int isp_video_check_external_subdevs(struct isp_video *video,
 
        for (i = 0; i < ARRAY_SIZE(ents); i++) {
                /* Is the entity part of the pipeline? */
-               if (!(pipe->entities & (1 << ents[i]->id)))
+               if (!(pipe->entities & (1 << media_entity_id(ents[i]))))
                        continue;
 
                /* ISP entities have always sink pad == 0. Find source. */
@@ -952,7 +953,8 @@ static int isp_video_check_external_subdevs(struct isp_video *video,
 
        pipe->external_rate = ctrl.value64;
 
-       if (pipe->entities & (1 << isp->isp_ccdc.subdev.entity.id)) {
+       id = media_entity_id(&isp->isp_ccdc.subdev.entity);
+       if (pipe->entities & (1 << id)) {
                unsigned int rate = UINT_MAX;
                /*
                 * Check that maximum allowed CCDC pixel rate isn't