[PATCH 4/7] clocksource: dw_apb_timer_of: make IO endian agnostic

Ben Dooks ben.dooks at codethink.co.uk
Thu Mar 26 03:56:12 PDT 2015


On 26/03/15 09:43, Daniel Lezcano wrote:
> On 03/25/2015 12:23 PM, Ben Dooks wrote:
>> The dw_apb_timer_of timer is using __raw_readl to access the timer
>> register, which is causing issues when the system is running in
>> big endian mode. Fix this by using readl_relaxed() which should
>> account for the endian settings.
>>
>> This fixes issues where the time jumps around in the dmesg output
>> due to returnling __le32 values.
>>
>> For an example, these two console lines show time running backwards:
>>
>> [   49.882572] CPU1: failed to come online
>> [   43.282457] Brought up 1 CPUs
>>
>> Signed-off-by: Ben Dooks <ben.dooks at codethink.co.uk>
> 
> Hi Ben,
> 
> Through which tree this patch is supposed to go ?

I think yours is probably the best one for it, it doesn't depend
on anything else in the series.

-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius



More information about the linux-arm-kernel mailing list