Re: [PATCH] clocksource: timer-atmel-pit: enable mck

From: Nicolas Ferre
Date: Tue Jul 05 2016 - 03:48:09 EST


Le 04/07/2016 22:24, Boris Brezillon a écrit :
> On Mon, 4 Jul 2016 18:00:09 +0200
> Alexandre Belloni <alexandre.belloni@xxxxxxxxxxxxxxxxxx> wrote:
>
>> mck is needed to get the PIT working. Explicitly prepare_enable it instead
>> of assuming it is enabled.
>>
>> This solves an issue were the system is freezing when the ETM/ETB drivers
>
> ^ where
>
>> are enabled.
>>
>> Reported-by: Olivier Schonken <olivier.schonken@xxxxxxxxx>
>> Signed-off-by: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxxxxxxxxx>
>
> Reviewed-by: Boris Brezillon <boris.brezillon@xxxxxxxxxxxxxxxxxx>

Acked-by: Nicolas Ferre <nicolas.ferre@xxxxxxxxx>

>> ---
>> drivers/clocksource/timer-atmel-pit.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/clocksource/timer-atmel-pit.c b/drivers/clocksource/timer-atmel-pit.c
>> index d911c5dca8f1..fcc97711cd79 100644
>> --- a/drivers/clocksource/timer-atmel-pit.c
>> +++ b/drivers/clocksource/timer-atmel-pit.c
>> @@ -248,6 +248,9 @@ static void __init at91sam926x_pit_dt_init(struct device_node *node)
>> if (IS_ERR(data->mck))
>> panic(pr_fmt("Unable to get mck clk\n"));
>>
>> + if (clk_prepare_enable(data->mck))
>> + panic(pr_fmt("Unable to enable mck\n"));
>> +
>> /* Get the interrupts property */
>> data->irq = irq_of_parse_and_map(node, 0);
>> if (!data->irq)
>
>


--
Nicolas Ferre