Subject: [PATCH v6 1/6] clocksource: Add support for the Mediatek SoCs

From: [email-masked] (Gregory CLEMENT)
Date: Wed, 04 Jun 2014 13:25:36 +0200

Hi Rob,

On 04/06/2014 13:18, Rob Herring wrote:

[...]

>> +
>> +       if (of_address_to_resource(node, 0, &res))
>> +               panic("Failed to parse resource\n");
> 
> panics are a bit pointless at this stage since your console is
> typically not up yet. Warning and continuing on with boot gives you a
> better shot at actually seeing the message.
> 
>> +
>> +       if (!request_mem_region(res.start, resource_size(&res), "mtk-timer"))
>> +               panic("Registers are a busy IO resource");
>> +
>> +       evt->gpt_base = ioremap(res.start, resource_size(&res));
> 
> of_iomap instead of these 3 calls.

Actually of_iomap doesn't request_mem_region, that's why for
mvebu we are trying now to replace of_iomap by request_mem_region/ioremap.

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com