Re: [PATCH v2 4/4] clocksource/drivers/imx-tpm: Support building imx-tpm driver as module

[Daniel Lezcano]

On 1/19/26 11:38, Daniel Baluta wrote:
> On Mon, Jan 19, 2026 at 11:56 AM Zhipeng Wang <zhipeng.wang_1@xxxxxxx> wrote:
> > From: Jindong Yue <jindong.yue@xxxxxxx>
> >
> > Change defconfig as tristate type and add platform driver
> > to support building it as a module.
> >
> > Signed-off-by: Jindong Yue <jindong.yue@xxxxxxx>
> > Signed-off-by: Zhipeng Wang <zhipeng.wang_1@xxxxxxx>
>
> Reviewed-by: Daniel Baluta <daniel.baluta@xxxxxxx>
>
> So you can now select the driver Y/M/n even if no COMPILE_TEST present.
>
> I wonder why we needed the `if COMPILE_TEST` in the first place. But anyhow,
> this change looks good.

The logic is to keep the timer option silent and let the platform's 
Kconfig to select the timer driver when enabling the platform.

Then we wanted to be able to compile-test those drivers even if the 
platform is not selected.

Now we want to choose between module or compiled-in.

I'm not sure if we really want to manually select the timer driver or 
switch to module.


--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog