From 8b3560a1196030ad5d9f07bc2cf27add8431171c Mon Sep 17 00:00:00 2001 From: Gerard Marull-Paretas Date: Thu, 23 Dec 2021 17:23:14 +0100 Subject: [PATCH] drivers: clock_control: use device_is_ready device_usable_check() can return 0 or -ENODEV, so using device_is_ready is equivalent if we return -ENODEV when the device is not ready. Signed-off-by: Gerard Marull-Paretas --- include/drivers/clock_control.h | 30 ++++++++++-------------------- 1 file changed, 10 insertions(+), 20 deletions(-) diff --git a/include/drivers/clock_control.h b/include/drivers/clock_control.h index e63ecc234de..b64b3d17089 100644 --- a/include/drivers/clock_control.h +++ b/include/drivers/clock_control.h @@ -117,10 +117,8 @@ struct clock_control_driver_api { static inline int clock_control_on(const struct device *dev, clock_control_subsys_t sys) { - int ret = device_usable_check(dev); - - if (ret != 0) { - return ret; + if (!device_is_ready(dev)) { + return -ENODEV; } const struct clock_control_driver_api *api = @@ -142,10 +140,8 @@ static inline int clock_control_on(const struct device *dev, static inline int clock_control_off(const struct device *dev, clock_control_subsys_t sys) { - int ret = device_usable_check(dev); - - if (ret != 0) { - return ret; + if (!device_is_ready(dev)) { + return -ENODEV; } const struct clock_control_driver_api *api = @@ -183,10 +179,8 @@ static inline int clock_control_async_on(const struct device *dev, return -ENOSYS; } - int ret = device_usable_check(dev); - - if (ret != 0) { - return ret; + if (!device_is_ready(dev)) { + return -ENODEV; } return api->async_on(dev, sys, cb, user_data); @@ -228,10 +222,8 @@ static inline int clock_control_get_rate(const struct device *dev, clock_control_subsys_t sys, uint32_t *rate) { - int ret = device_usable_check(dev); - - if (ret != 0) { - return ret; + if (!device_is_ready(dev)) { + return -ENODEV; } const struct clock_control_driver_api *api = @@ -264,10 +256,8 @@ static inline int clock_control_set_rate(const struct device *dev, clock_control_subsys_t sys, clock_control_subsys_rate_t rate) { - int ret = device_usable_check(dev); - - if (ret != 0) { - return ret; + if (!device_is_ready(dev)) { + return -ENODEV; } const struct clock_control_driver_api *api =