]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
media: omap3isp: configure entity functions
authorHans Verkuil <hverkuil+cisco@kernel.org>
Wed, 30 Apr 2025 07:12:27 +0000 (09:12 +0200)
committerHans Verkuil <hverkuil+cisco@kernel.org>
Tue, 6 Jan 2026 07:22:46 +0000 (08:22 +0100)
For the various subdevices, set the function field.

This fixes v4l2-compliance errors:

$ v4l2-compliance -M0
v4l2-compliance 1.33.0-5410, 32 bits, 64-bit time_t
v4l2-compliance SHA: c12c89c5bd70 2025-10-05 09:58:42

Compliance test for omap3isp device /dev/media0:

Media Driver Info:
Driver name      : omap3isp
Model            : TI OMAP3 ISP
Serial           :
Bus info         : platform:480bc000.isp
Media version    : 6.17.0
Hardware revision: 0x000000f0 (240)
Driver version   : 6.17.0

Required ioctls:
test MEDIA_IOC_DEVICE_INFO: OK
test invalid ioctls: OK

Allow for multiple opens:
test second /dev/media0 open: OK
test MEDIA_IOC_DEVICE_INFO: OK
test for unlimited opens: OK

Media Controller ioctls:
fail: v4l2-test-media.cpp(108): function == MEDIA_ENT_F_V4L2_SUBDEV_UNKNOWN
fail: v4l2-test-media.cpp(196): checkFunction(ent.function, true)
test MEDIA_IOC_G_TOPOLOGY: FAIL
fail: v4l2-test-media.cpp(398): num_data_links != num_links
test MEDIA_IOC_ENUM_ENTITIES/LINKS: FAIL
test MEDIA_IOC_SETUP_LINK: OK

Total for omap3isp device /dev/media0: 8, Succeeded: 6, Failed: 2, Warnings: 0

Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
drivers/media/platform/ti/omap3isp/ispccdc.c
drivers/media/platform/ti/omap3isp/ispccp2.c
drivers/media/platform/ti/omap3isp/ispcsi2.c
drivers/media/platform/ti/omap3isp/isppreview.c
drivers/media/platform/ti/omap3isp/ispresizer.c
drivers/media/platform/ti/omap3isp/ispstat.c

index 55ee14e8b4491887aa1fec7653be5be6424536ed..9dbf06ac058d603b9e707b521e8e90209625019c 100644 (file)
@@ -2675,6 +2675,7 @@ static int ccdc_init_entities(struct isp_ccdc_device *ccdc)
        pads[CCDC_PAD_SOURCE_OF].flags = MEDIA_PAD_FL_SOURCE;
 
        me->ops = &ccdc_media_ops;
+       me->function = MEDIA_ENT_F_PROC_VIDEO_PIXEL_ENC_CONV;
        ret = media_entity_pads_init(me, CCDC_PADS_NUM, pads);
        if (ret < 0)
                return ret;
index 1204ee221c9e42f26c1bbea01df94cfa4b8ead3f..f4147c79639cc348ec292c587c2cf01043a77a62 100644 (file)
@@ -1086,6 +1086,7 @@ static int ccp2_init_entities(struct isp_ccp2_device *ccp2)
        pads[CCP2_PAD_SOURCE].flags = MEDIA_PAD_FL_SOURCE;
 
        me->ops = &ccp2_media_ops;
+       me->function = MEDIA_ENT_F_VID_IF_BRIDGE;
        ret = media_entity_pads_init(me, CCP2_PADS_NUM, pads);
        if (ret < 0)
                return ret;
index ae574e1b65284d61a86a0d03f052b6e026789341..f227042b61b68e5c07d82c9aa6c2161342939980 100644 (file)
@@ -1251,6 +1251,7 @@ static int csi2_init_entities(struct isp_csi2_device *csi2)
                                    | MEDIA_PAD_FL_MUST_CONNECT;
 
        me->ops = &csi2_media_ops;
+       me->function = MEDIA_ENT_F_VID_IF_BRIDGE;
        ret = media_entity_pads_init(me, CSI2_PADS_NUM, pads);
        if (ret < 0)
                return ret;
index e383a57654de8129cb97fc2592af1c106ce1debe..26f7167d1f4f10dd50675099cc8d1ff6f05f9734 100644 (file)
@@ -2294,6 +2294,7 @@ static int preview_init_entities(struct isp_prev_device *prev)
        pads[PREV_PAD_SOURCE].flags = MEDIA_PAD_FL_SOURCE;
 
        me->ops = &preview_media_ops;
+       me->function = MEDIA_ENT_F_PROC_VIDEO_PIXEL_FORMATTER;
        ret = media_entity_pads_init(me, PREV_PADS_NUM, pads);
        if (ret < 0)
                goto error_handler_free;
index 87d821b02e5cc062e5f875d47761b935e472383a..5dff48489394b063be00752307d4a534357be1d3 100644 (file)
@@ -1738,6 +1738,7 @@ static int resizer_init_entities(struct isp_res_device *res)
        pads[RESZ_PAD_SOURCE].flags = MEDIA_PAD_FL_SOURCE;
 
        me->ops = &resizer_media_ops;
+       me->function = MEDIA_ENT_F_PROC_VIDEO_SCALER;
        ret = media_entity_pads_init(me, RESZ_PADS_NUM, pads);
        if (ret < 0)
                return ret;
index 07bd62a93d9902249862edb0e1ec0d248ccfcf3c..64bc71d830c47c97c49f0ec1238568301a4eeb82 100644 (file)
@@ -1037,6 +1037,7 @@ static int isp_stat_init_entities(struct ispstat *stat, const char *name,
 
        stat->pad.flags = MEDIA_PAD_FL_SINK | MEDIA_PAD_FL_MUST_CONNECT;
        me->ops = NULL;
+       me->function = MEDIA_ENT_F_PROC_VIDEO_STATISTICS;
 
        return media_entity_pads_init(me, 1, &stat->pad);
 }