diff options
| author | Ovidiu Panait | 2022-10-12 08:36:54 +0300 |
|---|---|---|
| committer | Michal Simek | 2022-11-22 15:02:07 +0100 |
| commit | 1e766a04c723e003c001c0f1a4f301aef026a75e (patch) | |
| tree | 7c8d9a613b2ffb556b23e5f56eaba5f25a8454c4 | |
| parent | ba74bcf3e07b10ffebf42e72a656b420215b5a2e (diff) | |
timer-uclass: add timer_get_ops() macro
Align timer uclass with the other subsystems and provide a timer_get_ops()
convenience macro.
Using this instead of the generic device_get_ops() also prevents
-Wdiscarded-qualifiers warnings when used with non-const variables.
Signed-off-by: Ovidiu Panait <ovpanait@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Link: https://lore.kernel.org/r/20221012053656.1492457-1-ovpanait@gmail.com
Signed-off-by: Michal Simek <michal.simek@amd.com>
| -rw-r--r-- | drivers/timer/timer-uclass.c | 2 | ||||
| -rw-r--r-- | include/timer.h | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/drivers/timer/timer-uclass.c b/drivers/timer/timer-uclass.c index cbc36476987..bdc77b38223 100644 --- a/drivers/timer/timer-uclass.c +++ b/drivers/timer/timer-uclass.c @@ -32,7 +32,7 @@ DECLARE_GLOBAL_DATA_PTR; int notrace timer_get_count(struct udevice *dev, u64 *count) { - const struct timer_ops *ops = device_get_ops(dev); + struct timer_ops *ops = timer_get_ops(dev); if (!ops->get_count) return -ENOSYS; diff --git a/include/timer.h b/include/timer.h index a044cb034ed..d33a26e28fe 100644 --- a/include/timer.h +++ b/include/timer.h @@ -6,6 +6,8 @@ #ifndef _TIMER_H_ #define _TIMER_H_ +#define timer_get_ops(dev) ((struct timer_ops *)(dev)->driver->ops) + /** * dm_timer_init() - initialize a timer for time keeping. On success * initializes gd->timer so that lib/timer can use it for future |