afbr-s50: Remove asserts from driver

ubieda · fabiobaltieri · commit 2e2d4afa06b9 · 2025-08-04T19:56:22.000+01:00
Run-time errors can still be considered asserts if checkif is
configured as such, otherwise, prevent crashing.

Signed-off-by: Luis Ubieda &lt;luisf@croxel.com&gt;
diff --git a/drivers/sensor/broadcom/afbr_s50/afbr_s50.c b/drivers/sensor/broadcom/afbr_s50/afbr_s50.c
@@ -153,7 +153,7 @@ static status_t data_ready_callback(status_t status, argus_hnd_t *hnd)
 	 */
 	err = afbr_s50_platform_get_by_hnd(hnd, &platform);
 	CHECKIF(err) {
-		__ASSERT(false, "Failed to get platform data SQE response can't be sent");
+		LOG_ERR("Failed to get platform data SQE response can't be sent");
 		return ERROR_FAIL;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -153,7 +153,7 @@ static status_t data_ready_callback(status_t status, argus_hnd_t *hnd)`
`153`	`153`	`*/`
`154`	`154`	`err = afbr_s50_platform_get_by_hnd(hnd, &platform);`
`155`	`155`	`CHECKIF(err) {`
`156`		`- __ASSERT(false, "Failed to get platform data SQE response can't be sent");`
	`156`	`+ LOG_ERR("Failed to get platform data SQE response can't be sent");`
`157`	`157`	`return ERROR_FAIL;`
`158`	`158`	`}`
`159`	`159`