From b0b320b7dd7ab46343834d5c960807551438394b Mon Sep 17 00:00:00 2001 From: Flavio Ceolin Date: Wed, 11 Mar 2020 11:23:12 -0700 Subject: [PATCH] drivers: i2c: Remove invalid syscalls i2c_slave_register/unregister must not be syscalls since it provides callbacks that will run in supervisor mode. Nonetheless, verification functions were missing which means that these functions never worked from usermode. Signed-off-by: Flavio Ceolin --- include/drivers/i2c.h | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/include/drivers/i2c.h b/include/drivers/i2c.h index 8c51f12a75e..d572a895a5a 100644 --- a/include/drivers/i2c.h +++ b/include/drivers/i2c.h @@ -275,11 +275,8 @@ static inline int z_impl_i2c_transfer(struct device *dev, * @retval -EIO General input / output error. * @retval -ENOTSUP If slave mode is not supported */ -__syscall int i2c_slave_register(struct device *dev, - struct i2c_slave_config *cfg); - -static inline int z_impl_i2c_slave_register(struct device *dev, - struct i2c_slave_config *cfg) +static inline int i2c_slave_register(struct device *dev, + struct i2c_slave_config *cfg) { const struct i2c_driver_api *api = (const struct i2c_driver_api *)dev->driver_api; @@ -306,11 +303,8 @@ static inline int z_impl_i2c_slave_register(struct device *dev, * @retval -EINVAL If parameters are invalid * @retval -ENOTSUP If slave mode is not supported */ -__syscall int i2c_slave_unregister(struct device *dev, - struct i2c_slave_config *cfg); - -static inline int z_impl_i2c_slave_unregister(struct device *dev, - struct i2c_slave_config *cfg) +static inline int i2c_slave_unregister(struct device *dev, + struct i2c_slave_config *cfg) { const struct i2c_driver_api *api = (const struct i2c_driver_api *)dev->driver_api;