On 05/04/2015 07:04 AM, fu.wei@linaro.org wrote:

+       gsi = map_generic_timer_interrupt(irq, flags);

...

+       res[2].start = res[2].end = gsi;

Wouldn't it make more sense to do this:

        res[2].start = res[2].end = irq;

and then the driver should call map_generic_timer_interrupt()?  That way, the resource array contains all physical values.

--
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.