[PATCH-for-5.2 1/3] hw/clock: Remove unused clock_init*() functions

Philippe Mathieu-Daudé posted 3 patches 5 years, 6 months ago
[PATCH-for-5.2 1/3] hw/clock: Remove unused clock_init*() functions
Posted by Philippe Mathieu-Daudé 5 years, 6 months ago
clock_init*() inlined funtions are simple wrappers around
clock_set*() and are not used. Remove them in favor of clock_set*().

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
 include/hw/clock.h | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/include/hw/clock.h b/include/hw/clock.h
index f822a94220..468fed0996 100644
--- a/include/hw/clock.h
+++ b/include/hw/clock.h
@@ -209,17 +209,4 @@ static inline bool clock_is_enabled(const Clock *clk)
     return clock_get(clk) != 0;
 }
 
-static inline void clock_init(Clock *clk, uint64_t value)
-{
-    clock_set(clk, value);
-}
-static inline void clock_init_hz(Clock *clk, uint64_t value)
-{
-    clock_set_hz(clk, value);
-}
-static inline void clock_init_ns(Clock *clk, uint64_t value)
-{
-    clock_set_ns(clk, value);
-}
-
 #endif /* QEMU_HW_CLOCK_H */
-- 
2.21.3


Re: [PATCH-for-5.2 1/3] hw/clock: Remove unused clock_init*() functions
Posted by Richard Henderson 5 years, 6 months ago
On 8/6/20 5:38 AM, Philippe Mathieu-Daudé wrote:
> clock_init*() inlined funtions are simple wrappers around
> clock_set*() and are not used. Remove them in favor of clock_set*().
> 
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>  include/hw/clock.h | 13 -------------
>  1 file changed, 13 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


r~