Re: [PATCH] clocksource/drivers/timer-ti-dm: Register clocksource/event only for am654

Re: [PATCH] clocksource/drivers/timer-ti-dm: Register clocksource/event only for am654

[H. Nikolaus Schaller]

> Am 28.08.2026 um 23:04 schrieb Andreas Kemnade <andreas@kemnade.info>:
> 
> On Fri, 28 Aug 2026 20:47:45 +0200
> "Markus Schneider-Pargmann (TI)" <msp@baylibre.com> wrote:
> 
>> Registering clocksource/clockevent for the older platforms breaks
>> booting them. The older platforms have their own implementation in early
>> boot that is used to register clocksource/clockevent in
>> timer-ti-dm-systimer.c.
>> 
>> With am654, the timer is much more complex with more dependencies so
>> doing this in early boot with the other driver is very difficult and
>> gets ugly. Instead it is implemented in the platform driver for am654.
>> However the old platforms may not register clocksource/clockevent in
>> this driver as that would break the boot.
>> 
>> Add a system_timer_capable flag that is only set for am654-timer that
>> gates the clocksource/clockevent registration.
>> 
>> Reported-by: H. Nikolaus Schaller <hns@goldelico.com>
>> Closes: https://lore.kernel.org/r/2013A99A-0502-4128-ACFA-B04886224393@goldelico.com
>> Fixes: b8eeeca55456 ("clocksource/drivers/timer-ti-dm: Add clocksource support")
>> Signed-off-by: Markus Schneider-Pargmann (TI) <msp@baylibre.com>
> 
> Tested-by: Andreas Kemnade <andreas@kemnade.info> # OMAP3 GTA04

Tested-by: H. Nikolaus Schaller <hns@goldelico.com> # OMAP3 OpenPandora

BR and thanks,
Nikolaus

PS: I would suggest that patches for omap-dm-timer are also routed to linux-omap@vger.kernel.org

Re: [PATCH] clocksource/drivers/timer-ti-dm: Register clocksource/event only for am654

[Kevin Hilman]

"H. Nikolaus Schaller" <hns@goldelico.com> writes:

>> Am 28.08.2026 um 23:04 schrieb Andreas Kemnade <andreas@kemnade.info>:
>> 
>> On Fri, 28 Aug 2026 20:47:45 +0200
>> "Markus Schneider-Pargmann (TI)" <msp@baylibre.com> wrote:
>> 
>>> Registering clocksource/clockevent for the older platforms breaks
>>> booting them. The older platforms have their own implementation in early
>>> boot that is used to register clocksource/clockevent in
>>> timer-ti-dm-systimer.c.
>>> 
>>> With am654, the timer is much more complex with more dependencies so
>>> doing this in early boot with the other driver is very difficult and
>>> gets ugly. Instead it is implemented in the platform driver for am654.
>>> However the old platforms may not register clocksource/clockevent in
>>> this driver as that would break the boot.
>>> 
>>> Add a system_timer_capable flag that is only set for am654-timer that
>>> gates the clocksource/clockevent registration.
>>> 
>>> Reported-by: H. Nikolaus Schaller <hns@goldelico.com>
>>> Closes: https://lore.kernel.org/r/2013A99A-0502-4128-ACFA-B04886224393@goldelico.com
>>> Fixes: b8eeeca55456 ("clocksource/drivers/timer-ti-dm: Add clocksource support")
>>> Signed-off-by: Markus Schneider-Pargmann (TI) <msp@baylibre.com>
>> 
>> Tested-by: Andreas Kemnade <andreas@kemnade.info> # OMAP3 GTA04
>
> Tested-by: H. Nikolaus Schaller <hns@goldelico.com> # OMAP3 OpenPandora
>

Tested-by: Kevin Hilman <khilman@baylibre.com> # OMAP3 Beagle XM