Re: [EXT] Re: [PATCH v4 3/4] clocksource/drivers/timer-mediatek: Convert timer-mediatek to a loadable module

[Daniel Lezcano]

On 3/6/26 06:46, Zhipeng Wang wrote:
> Hi Daniel,
>
> Thank you for sharing your RFC patch. I think the approach is very
> interesting and addresses a real problem in the timer subsystem.
>
> For the MediaTek and IMX timer driver, I see a few considerations:
>
> 1. Current approach: My current patch uses the existing pattern
>     (similar to renesas-ostm.c) with #ifdef MODULE. It's simple and
>     works, but I agree it's not ideal.
> 2. Your approach: Your TIMER_OF_DECLARE_PDEV approach would be
>     cleaner and allow us to use devm_ functions, which would be a
>     nice improvement.
> 3. MediaTek and IMX timer specifics: The IMX timer doesn't require
>     early initialization on modern platforms (we have arch timers on
>     ARM64), so it would fit well with your TIMER_OF_DECLARE_PLATFORM_DRIVER
>     macro (if you go with the two-macro approach).
>
> I'm very interested in your approach. May I ask what timeline you're
> considering for your patch series?
> The reason I ask is that we're working towards GKI support and I want
> to make sure I choose the right approach.

Well, if you are willing to collaborate, we can start right now to think 
about it and propose an unified solution