video: add more resolutions to GC2145

video: add more resolutions to GC2145

As I mentioned in issue #91968,

It would be great if the camera drivers would support a larger number of resolutions.   For example those resolutions of the displays that they wish to display an image on.  Potentially in both landscape and portrait mode.

So far I have added  one the works in landscape mode for
ILI948x and ST7796 displays as well

as a QVGA in portrait mode, which should be good for
LI9341 and some ST7789 displays.

If this looks like a valid approach will also add in GIGA Display Adapter size, and maybe and add the duplicate items to the FORMAT_CAP list for YUYV mode.

Note: the other than add the items to the list, I just
added code to set resolution which computes the c_ratio/r_ratio chooses the smaller of the two to use.

Could probably do several of the others n the switch statement.

Signed-off-by: Kurt Eckhardt <[email protected]>

Author: KurtE

drivers/video/gc2145.c

@@ -796,6 +796,8 @@ static const struct video_format_cap fmts[] = {
 	GC2145_VIDEO_FORMAT_CAP(RESOLUTION_QVGA_W, RESOLUTION_QVGA_H, VIDEO_PIX_FMT_RGB565),
 	GC2145_VIDEO_FORMAT_CAP(RESOLUTION_VGA_W, RESOLUTION_VGA_H, VIDEO_PIX_FMT_RGB565),
 	GC2145_VIDEO_FORMAT_CAP(RESOLUTION_UXGA_W, RESOLUTION_UXGA_H, VIDEO_PIX_FMT_RGB565),
+	GC2145_VIDEO_FORMAT_CAP(480, 320, VIDEO_PIX_FMT_RGB565), /* ILI948x, ST7796 */
+	GC2145_VIDEO_FORMAT_CAP(240, 320, VIDEO_PIX_FMT_RGB565), /* ILI9341, ST7785 port */
 	GC2145_VIDEO_FORMAT_CAP(RESOLUTION_QVGA_W, RESOLUTION_QVGA_H, VIDEO_PIX_FMT_YUYV),
 	GC2145_VIDEO_FORMAT_CAP(RESOLUTION_VGA_W, RESOLUTION_VGA_H, VIDEO_PIX_FMT_YUYV),
 	GC2145_VIDEO_FORMAT_CAP(RESOLUTION_UXGA_W, RESOLUTION_UXGA_H, VIDEO_PIX_FMT_YUYV),
@@ -1036,8 +1038,19 @@ static int gc2145_set_resolution(const struct device *dev, uint32_t w, uint32_t
 		r_ratio = 1;
 		break;
 	default:
-		LOG_ERR("Unsupported resolution %d %d", w, h);
-		return -EIO;
+		if ((w > UXGA_HSIZE) || (h > UXGA_VSIZE)) {
+			LOG_ERR("Unsupported resolution %d %d", w, h);
+			return -EIO;
+		}
+		c_ratio = UXGA_HSIZE / w;
+		r_ratio = UXGA_VSIZE / h;
+		if (c_ratio < r_ratio) {
+			r_ratio = c_ratio;
+		} else {
+			c_ratio = r_ratio;
+		}
+		LOG_DBG("set resolution(%u %u): ratio: %u %u\n", w, h, c_ratio, r_ratio);
+		break;
 	};
 
 	/* Calculates the window boundaries to obtain the desired resolution */
@@ -1048,6 +1061,7 @@ static int gc2145_set_resolution(const struct device *dev, uint32_t w, uint32_t
 	win_x = ((UXGA_HSIZE - win_w) / 2);
 	win_y = ((UXGA_VSIZE - win_h) / 2);
 
+	LOG_DBG("xy: %u %u win: %u %u\n", x, y, win_w, win_h);
 	/* Set readout window first. */
 	ret = gc2145_set_window(dev, GC2145_REG_BLANK_WINDOW_BASE, win_x, win_y, win_w + 16,
 				win_h + 8);