drivers: i2c_rtio: max32: fix i2c_configure to return 0 on success

ubieda · kartben · commit a6c76c4dd92f · 2025-04-22T04:33:42.000+02:00
To comply with the API definition. This was caught by running i2c_ram
test.

Signed-off-by: Luis Ubieda &lt;luisf@croxel.com&gt;
diff --git a/drivers/i2c/i2c_max32.c b/drivers/i2c/i2c_max32.c
@@ -100,7 +100,7 @@ static int api_configure(const struct device *dev, uint32_t dev_cfg)
 		return -ENOTSUP;
 	}
 
-	return ret;
+	return ((ret > 0) ? 0 : -EIO);
 }
 
 #ifdef CONFIG_I2C_TARGET
diff --git a/drivers/i2c/i2c_max32_rtio.c b/drivers/i2c/i2c_max32_rtio.c
@@ -97,7 +97,7 @@ static int max32_do_configure(const struct device *dev, uint32_t dev_cfg)
 		return -ENOTSUP;
 	}
 
-	return ret;
+	return ((ret > 0) ? 0 : -EIO);
 }
 
 static void max32_complete(const struct device *dev, int status);

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ static int api_configure(const struct device *dev, uint32_t dev_cfg)`
`100`	`100`	`return -ENOTSUP;`
`101`	`101`	`}`
`102`	`102`
`103`		`- return ret;`
	`103`	`+ return ((ret > 0) ? 0 : -EIO);`
`104`	`104`	`}`
`105`	`105`
`106`	`106`	`#ifdef CONFIG_I2C_TARGET`
Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ static int max32_do_configure(const struct device *dev, uint32_t dev_cfg)`
`97`	`97`	`return -ENOTSUP;`
`98`	`98`	`}`
`99`	`99`
`100`		`- return ret;`
	`100`	`+ return ((ret > 0) ? 0 : -EIO);`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`static void max32_complete(const struct device *dev, int status);`