drivers: video: sw_generator: fix off-by-one in ARRAY_SIZE() comparison

The struct video_format_caps fmts[] is an array terminated by an empty
entry which should not be used while searching or comparing the formats.
If using ARRAY_SIZE(), this terminator entry should be subtracted from
the list.

Signed-off-by: Josuah Demangeon <me@josuah.net>

Author: josuah

drivers/video/video_sw_generator.c

@@ -73,15 +73,15 @@ static int video_sw_generator_set_fmt(const struct device *dev, struct video_for
 	struct video_sw_generator_data *data = dev->data;
 	int i = 0;
 
-	for (i = 0; i < ARRAY_SIZE(fmts); ++i) {
+	for (i = 0; i < ARRAY_SIZE(fmts) - 1; ++i) {
 		if (fmt->pixelformat == fmts[i].pixelformat &&
 		    IN_RANGE(fmt->width, fmts[i].width_min, fmts[i].width_max) &&
 		    IN_RANGE(fmt->height, fmts[i].height_min, fmts[i].height_max)) {
 			break;
 		}
 	}
 
-	if (i == ARRAY_SIZE(fmts)) {
+	if (i == ARRAY_SIZE(fmts) - 1) {
 		LOG_ERR("Unsupported pixel format or resolution");
 		return -ENOTSUP;
 	}