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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel




[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [CentOS ARM]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]     [Photos]