[PATCH v4] twd: Don't set CLOCK_EVT_FEAT_C3STOP unconditionally
Mason
slash.tmp at free.fr
Thu Oct 8 11:25:07 PDT 2015
On 08/10/2015 19:57, Mark Rutland wrote:
>> 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.
Hmmm, I think I'll do it in a follow-up patch, once Felipe's
revert hits mainline.
Regards.
More information about the linux-arm-kernel
mailing list