From: Ben Dooks <ben.dooks-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
To: Dmitry Osipenko <digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Bob Mottram <bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
linux-kernel-81qHHgoATdFT9dQujB1mzip2UmYkHbXO@public.gmane.org,
swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org
Subject: Re: [Linux-kernel] [PATCH 3/4] ARM: tegra: enable big-endian support
Date: Fri, 09 Jan 2015 17:14:00 +0000 [thread overview]
Message-ID: <54B00C58.3070303@codethink.co.uk> (raw)
In-Reply-To: <54B0028C.8050802-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
On 09/01/15 16:32, Dmitry Osipenko wrote:
> 09.01.2015 19:27, Bob Mottram пишет:
>> On 09/01/15 15:46, Dmitry Osipenko wrote:
>>> What about suspend/cpuidle? Won't it make sense to set BE mode in reset handler?
>>
>> I did try adding:
>>
>> ARM_BE8(setend be)
>>
>> to the entry point within reset-handler.S, but to no avail. There are
>> definitely problems with suspend, which currently I can work around by
>> turning off ARCH_SUSPEND_POSSIBLE. More investigation is going to be
>> needed to get the sleep/suspend working.
>>
>
> Yeah, just noticed that it mentioned in cover letter.
I had a look through the code and it was not easy to work out what
was going on so skipped over it. We decided to leave it out of the
initial submission as we can get a workable system without suspend.
The suspend/resume code does make accesses to registers, and all
these will need to be found and have the correct endian conversion
done (stores will be in big-endian but the hardware registers stay
little endian)
--
Ben Dooks http://www.codethink.co.uk/
Senior Engineer Codethink - Providing Genius
next prev parent reply other threads:[~2015-01-09 17:14 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-09 15:27 [PATCH 0/4] ARM: tegra: Support for big-endian (version 3) Bob Mottram
[not found] ` <1420817231-8346-1-git-send-email-bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-09 15:27 ` [PATCH 1/4] ARM: tegra: Add a separate big-endian configuration Bob Mottram
2015-01-09 15:27 ` [PATCH 2/4] ARM: gpio: tegra: Maintain endianness Bob Mottram
[not found] ` <1420817231-8346-3-git-send-email-bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-27 18:41 ` Dmitry Osipenko
2015-03-11 9:43 ` Thierry Reding
[not found] ` <20150311094324.GH19577-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2015-03-11 9:46 ` Alexandre Courbot
[not found] ` <CAAVeFuLBz0UJsVEi=EYPbGd2x3Q6_JOELb4SHspR+w=tBS8mZw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-03-11 9:55 ` Thierry Reding
[not found] ` <20150311095543.GJ19577-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2015-03-11 9:58 ` Alexandre Courbot
[not found] ` <CAAVeFuLenb8V3rk6bbc2uDWo65sWZ2YLtFfuRLt2yEZ-3KwFwQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-03-11 10:03 ` [Ct173-kernel-internal] " Ben Dooks
2015-03-18 1:36 ` Linus Walleij
2015-01-09 15:27 ` [PATCH 3/4] ARM: tegra: enable big-endian support Bob Mottram
[not found] ` <1420817231-8346-4-git-send-email-bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-09 15:46 ` Dmitry Osipenko
[not found] ` <54AFF7E6.9030308-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-09 16:27 ` Bob Mottram
[not found] ` <54B0017E.2060509-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-09 16:32 ` Dmitry Osipenko
[not found] ` <54B0028C.8050802-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-09 17:14 ` Ben Dooks [this message]
[not found] ` <54B00C58.3070303-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-12 18:04 ` [Linux-kernel] " Dmitry Osipenko
[not found] ` <54B40C99.4020907-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-13 8:47 ` Ben Dooks
2015-01-13 9:05 ` Bob Mottram
2015-01-19 14:26 ` Bob Mottram
[not found] ` <54BD141F.7070803-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-19 15:08 ` Dmitry Osipenko
2015-01-09 15:27 ` [PATCH 4/4] ARM: tegra: fix big-endian issue with irq code Bob Mottram
[not found] ` <1420817231-8346-5-git-send-email-bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-27 18:42 ` Dmitry Osipenko
2015-03-11 9:48 ` Thierry Reding
2015-01-20 10:31 ` [Ct173-kernel-internal] [PATCH 0/4] ARM: tegra: Support for big-endian (version 3) Ben Dooks
[not found] ` <54BE2E77.9050909-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-20 11:36 ` Dmitry Osipenko
[not found] ` <54BE3DD2.1060204-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-20 11:45 ` Ben Dooks
[not found] ` <54BE3FC1.5010107-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-20 13:18 ` Dmitry Osipenko
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=54B00C58.3070303@codethink.co.uk \
--to=ben.dooks-4ydnlxn2s6swdatgbsphta@public.gmane.org \
--cc=bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org \
--cc=digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=linux-kernel-81qHHgoATdFT9dQujB1mzip2UmYkHbXO@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).