All of lore.kernel.org
 help / color / mirror / Atom feed
From: daniel.lezcano@linaro.org (Daniel Lezcano)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 4/7] clocksource: dw_apb_timer_of: make IO endian agnostic
Date: Thu, 26 Mar 2015 11:58:22 +0100	[thread overview]
Message-ID: <5513E64E.1060802@linaro.org> (raw)
In-Reply-To: <5513E5CC.3040807@codethink.co.uk>

On 03/26/2015 11:56 AM, Ben Dooks wrote:
> 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@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.

Ok, thanks.

   -- Daniel


-- 
  <http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

  reply	other threads:[~2015-03-26 10:58 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1427282610-10350-1-git-send-email-ben.dooks@codethink.co.uk>
2015-03-25 11:23 ` [PATCH 2/7] ARM: socfpga: enable big endian for secondary core(s) Ben Dooks
2015-03-25 11:23 ` [PATCH 3/7] ARM: socfpga: support big endian for socfpga Ben Dooks
2015-03-25 11:23 ` [PATCH 4/7] clocksource: dw_apb_timer_of: make IO endian agnostic Ben Dooks
2015-03-26  9:43   ` Daniel Lezcano
2015-03-26 10:56     ` Ben Dooks
2015-03-26 10:58       ` Daniel Lezcano [this message]
2015-03-25 11:23 ` [PATCH 5/7] mmc: host: dw_mmc make IO accessors " Ben Dooks
2015-03-25 11:23   ` Ben Dooks
2015-03-25 11:23 ` [PATCH 6/7] mmc: host: dw_mmc: change idmac descriptor files to __le32 Ben Dooks
2015-03-25 11:23   ` Ben Dooks
2015-03-25 11:23 ` [PATCH 7/7] mmc: host: dw_mmc: fix fifo ordering in big endian Ben Dooks
2015-03-25 11:23   ` Ben Dooks

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5513E64E.1060802@linaro.org \
    --to=daniel.lezcano@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.