drivers: video: removed supported resolution

Removed CID for supported resolution as this information is covered by formats.

Signed-off-by: Krystian Balicki <kristos_b@wp.pl>

Author: kristosb

drivers/video/video_arducam_mega.c

@@ -20,10 +20,6 @@
 
 LOG_MODULE_REGISTER(mega_camera, CONFIG_VIDEO_LOG_LEVEL);
 
-/* Info default settings */
-#define SUPPORT_RESOLUTION_5M 7894
-#define SUPPORT_RESOLUTION_3M 7638
-
 /* Configure camera contrast level */
 enum mega_contrast_level {
 	MEGA_CONTRAST_LEVEL_NEGATIVE_3 = 6,
@@ -236,7 +232,6 @@ struct arducam_mega_ctrls {
 	};
 	struct video_ctrl focus_auto;
 	/* Read only registers */
-	struct video_ctrl support_resolution;
 	struct video_ctrl linkfreq;
 };
 
@@ -792,8 +787,6 @@ static int arducam_mega_check_connection(const struct device *dev)
 			2592, 1944, VIDEO_PIX_FMT_JPEG);
 		fmts[26] = (struct video_format_cap)ARDUCAM_MEGA_VIDEO_FORMAT_CAP(
 			2592, 1944, VIDEO_PIX_FMT_YUYV);
-
-		drv_data->ctrls.support_resolution.val = SUPPORT_RESOLUTION_5M;
 		drv_data->features |= MEGA_HAS_FOCUS | MEGA_HAS_COLORFX;
 		break;
 	case ARDUCAM_SENSOR_3MP_1:
@@ -804,7 +797,6 @@ static int arducam_mega_check_connection(const struct device *dev)
 			2048, 1536, VIDEO_PIX_FMT_JPEG);
 		fmts[26] = (struct video_format_cap)ARDUCAM_MEGA_VIDEO_FORMAT_CAP(
 			2048, 1536, VIDEO_PIX_FMT_YUYV);
-		drv_data->ctrls.support_resolution.val = SUPPORT_RESOLUTION_3M;
 		drv_data->features |= MEGA_HAS_SHARPNESS | MEGA_HAS_COLORFX;
 		break;
 	case ARDUCAM_SENSOR_5MP_2:
@@ -814,7 +806,6 @@ static int arducam_mega_check_connection(const struct device *dev)
 			2592, 1936, VIDEO_PIX_FMT_JPEG);
 		fmts[26] = (struct video_format_cap)ARDUCAM_MEGA_VIDEO_FORMAT_CAP(
 			2592, 1936, VIDEO_PIX_FMT_YUYV);
-		drv_data->ctrls.support_resolution.val = SUPPORT_RESOLUTION_5M;
 		drv_data->features |= MEGA_HAS_FOCUS | MEGA_HAS_COLORFX;
 		break;
 	default:
@@ -1241,13 +1232,6 @@ static int arducam_mega_init_controls(const struct device *dev)
 		}
 	}
 	/* Read only controls */
-	ret = video_init_ctrl(
-		&ctrls->support_resolution, dev, VIDEO_CID_ARDUCAM_SUPP_RES,
-		(struct video_ctrl_range){.min = 0, .max = 65535, .step = 1, .def = 0});
-	ctrls->support_resolution.flags |= VIDEO_CTRL_FLAG_READ_ONLY;
-	if (ret < 0) {
-		return ret;
-	}
 	ret = video_init_int_menu_ctrl(
 		&ctrls->linkfreq, dev, VIDEO_CID_LINK_FREQ, ARDUCAM_MEGA_640_480_LINK_FREQ_ID,
 		arducam_mega_link_frequency, ARRAY_SIZE(arducam_mega_link_frequency));

include/zephyr/drivers/video/arducam_mega.h

@@ -15,9 +15,6 @@ extern "C" {
 #define VIDEO_CID_ARDUCAM_RESET    (VIDEO_CID_PRIVATE_BASE + 1)
 #define VIDEO_CID_ARDUCAM_LOWPOWER (VIDEO_CID_PRIVATE_BASE + 2)
 
-/* Read only registers */
-#define VIDEO_CID_ARDUCAM_SUPP_RES    (VIDEO_CID_PRIVATE_BASE + 3)
-
 #ifdef __cplusplus
 }
 #endif