10.06.2019 11:11, Daniel Lezcano пишет:
>
> Hi Dmitry,
>
>
> On 09/06/2019 21:27, Dmitry Osipenko wrote:
>> It was left unnoticed by accident, which means that the code could be
>> cleaned up a tad more.
>>
>> Signed-off-by: Dmitry Osipenko <digetx@xxxxxxxxx>
>> ---
>> drivers/clocksource/timer-tegra.c | 40 +++++++++++++++++++------------
>> 1 file changed, 25 insertions(+), 15 deletions(-)
>>
>> diff --git a/drivers/clocksource/timer-tegra.c b/drivers/clocksource/timer-tegra.c
>> index 9406855781ff..6da169de47f9 100644
>> --- a/drivers/clocksource/timer-tegra.c
>> +++ b/drivers/clocksource/timer-tegra.c
>> @@ -216,6 +216,19 @@ static inline unsigned int tegra_irq_idx_for_cpu(int cpu, bool tegra20)
>> return TIMER10_IRQ_IDX + cpu;
>> }
>>
>> +static inline unsigned long tegra_rate_for_timer(struct timer_of *to,
>> + bool tegra20)
>> +{
>> + /*
>> + * TIMER1-9 are fixed to 1MHz, TIMER10-13 are running off the
>> + * parent clock.
>> + */
>> + if (tegra20)
>> + return 1000000;
>
> Mind to take the opportunity to convert the literal value to a constant?
Sure!
>> +
>> + return to->of_clk.rate;
>> +}
>> +
>> static int __init tegra_init_timer(struct device_node *np, bool tegra20,
>> int rating)
>> {
>> @@ -268,30 +281,27 @@ static int __init tegra_init_timer(struct device_node *np, bool tegra20,
>>
>> for_each_possible_cpu(cpu) {
>> struct timer_of *cpu_to = per_cpu_ptr(&tegra_to, cpu);
>> + unsigned long flags = IRQF_TIMER | IRQF_NOBALANCING;
>> + unsigned long rate = tegra_rate_for_timer(&tegra_to, tegra20);
Oh, this actually shall be (to, tegra20). Will correct this in v2!
>> unsigned int base = tegra_base_for_cpu(cpu, tegra20);
>> unsigned int idx = tegra_irq_idx_for_cpu(cpu, tegra20);
>> + unsigned int irq = irq_of_parse_and_map(np, idx);
>>
>> - /*
>> - * TIMER1-9 are fixed to 1MHz, TIMER10-13 are running off the
>> - * parent clock.
>> - */
>> - if (tegra20)
>> - cpu_to->of_clk.rate = 1000000;
I also spotted that there is a bug here that I introduced in a previous
patch. The of_clk.rate is initialized only for the first per-cpu
clocksource and then we need to replicate it for the rest of CPU's in a
case of T210. I'll add an explicit fixup for this.
