[PATCH v6 1/6] clocksource: Add support for the Mediatek SoCs
Thomas Gleixner
tglx at linutronix.de
Wed Jun 4 06:46:00 PDT 2014
On Wed, 4 Jun 2014, Gregory CLEMENT wrote:
> 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.
What about creating of_io_request_and_map() instead of open coding it ?
Thanks,
tglx
More information about the linux-arm-kernel
mailing list