drivers: video: common: turn ASSERT(false) into CODE_UNREACHABLE

josuah · josuah · commit bd176c22549d · 2025-05-09T00:24:17.000Z
The assert message was not very helpful, CODE_UNREACHABLE is more
readable and requires fewer effort while proofreading.

Signed-off-by: Josuah Demangeon &lt;me@josuah.net&gt;
diff --git a/drivers/video/video_common.c b/drivers/video/video_common.c
@@ -149,7 +149,7 @@ void video_closest_frmival(const struct device *dev, enum video_endpoint_id ep,
 			video_closest_frmival_stepwise(&fie.stepwise, &desired, &tmp);
 			break;
 		default:
-			__ASSERT(false, "invalid answer from the queried video device");
+			CODE_UNREACHABLE;
 		}
 
 		tmp_nsec = video_frmival_nsec(&tmp);

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ void video_closest_frmival(const struct device *dev, enum video_endpoint_id ep,`
`149`	`149`	`video_closest_frmival_stepwise(&fie.stepwise, &desired, &tmp);`
`150`	`150`	`break;`
`151`	`151`	`default:`
`152`		`- __ASSERT(false, "invalid answer from the queried video device");`
	`152`	`+ CODE_UNREACHABLE;`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`tmp_nsec = video_frmival_nsec(&tmp);`