Hi All, After the introduction of [1] v4l2_subdev_init_finalize() sets the HAS_EVENTS flag if a control handler is set, and subdev_do_ioctl() uses v4l2_ctrl_subdev_subscribe_event() and v4l2_event_subdev_unsubscribe() as defaults if the subdev doesn't have .(un)subscribe. Then Let's drop the HAS_EVENTS flag and event handlers. As suggested by LPinchart imx415 imx415 sets the HAS_EVENTS flags but not the event operations, all is already done from [1] then we can drop the HAS_EVENTS. base-commit: 0f9eef17a2638151c56663eb909fe72a7dd4c698 [1] https://git.linuxtv.org/sailus/media_tree.git/commit/?h=devel&id=0f9eef17a2638151c56663eb909fe72a7dd4c698 Tommaso Merciai (2): media: i2c: Drop HAS_EVENTS and event handlers media: i2c: imx415: Drop HAS_EVENTS flag drivers/media/i2c/alvium-csi2.c | 5 +---- drivers/media/i2c/ds90ub953.c | 5 +---- drivers/media/i2c/ds90ub960.c | 5 +---- drivers/media/i2c/gc0308.c | 4 ---- drivers/media/i2c/gc05a2.c | 10 +--------- drivers/media/i2c/gc08a3.c | 10 +--------- drivers/media/i2c/gc2145.c | 10 +--------- drivers/media/i2c/imx219.c | 10 +--------- drivers/media/i2c/imx283.c | 10 +--------- drivers/media/i2c/imx290.c | 10 +--------- drivers/media/i2c/imx415.c | 3 +-- drivers/media/i2c/max96714.c | 6 +----- drivers/media/i2c/max96717.c | 6 +----- drivers/media/i2c/ov01a10.c | 6 +----- drivers/media/i2c/ov64a40.c | 10 +--------- drivers/media/i2c/ov8858.c | 9 +-------- drivers/media/i2c/thp7312.c | 5 +---- 17 files changed, 16 insertions(+), 108 deletions(-) -- 2.34.1
