[PATCH v4] twd: Don't set CLOCK_EVT_FEAT_C3STOP unconditionally

Mark Rutland mark.rutland at arm.com
Thu Oct 8 10:57:34 PDT 2015


> Is it possible that twd_local_timer_common_register() would be
> called more than once?
> 
> twd_local_timer_register() guards against multiple invocations,
> but twd_local_timer_of_register() doesn't (but maybe there is
> some guarantee from OF, or it's invalid to have multiple
> "arm,cortex-a9-twd-timer" strings in the DT).

It does appear that were you to have have two or more
"arm,cortex-a9-twd-timer" nodes in the DT, things would go wrong. As far
as I am aware, that would result in multiple calls to
twd_local_timer_of_register.

I don't think it would ever be valid to have two nodes for this
particular device, but it's probably worth having the same check as
twd_local_timer_register does, with a warning that there was more than
one such node logged to the console.

Thanks,
Mark



More information about the linux-arm-kernel mailing list