Re: [PATCH 2/3] clocksource/drivers/timer-ti-dm: Make timer selectable for ARCH_K3

From: Tony Lindgren <tony@atomide.com>
To: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Thomas Gleixner <tglx@linutronix.de>,
	linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, Keerthy <j-keerthy@ti.com>,
	Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>
Subject: Re: [PATCH 2/3] clocksource/drivers/timer-ti-dm: Make timer selectable for ARCH_K3
Date: Mon, 23 May 2022 10:13:36 +0300	[thread overview]
Message-ID: <Yos0IOkCd4R9a9YZ@atomide.com> (raw)
In-Reply-To: <156a53a2-c33b-7ec5-3584-41ef25cb9cdd@linaro.org>

Hi,

* Daniel Lezcano <daniel.lezcano@linaro.org> [220520 15:24]:
> On 08/04/2022 12:17, Tony Lindgren wrote:
> > diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig
> > --- a/drivers/clocksource/Kconfig
> > +++ b/drivers/clocksource/Kconfig
> > @@ -22,7 +22,7 @@ config CLKEVT_I8253
> >   config I8253_LOCK
> >   	bool
> > -config OMAP_DM_TIMER
> > +config OMAP_DM_SYSTIMER
> >   	bool
> >   	select TIMER_OF
> > @@ -56,6 +56,12 @@ config DIGICOLOR_TIMER
> >   	help
> >   	  Enables the support for the digicolor timer driver.
> > +config OMAP_DM_TIMER
> > +	tristate "OMAP dual-mode timer driver" if ARCH_K3 || COMPILE_TEST
> 
> Actually, I missed this. Could you convert to a 'bool' there is no module in
> the timer drivers ATM.

OK sure will do.

Did you notice some loadable module usage issues for timers that are not
system timers? Just wondering if there are some issues that I did not
notice.

Regards,

Tony