Re: [Linux-kernel] [PATCH 3/4] ARM: tegra: enable big-endian support

[Dmitry Osipenko <digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

09.01.2015 20:14, Ben Dooks пишет:
> 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)
>
I fixed cpuidle and suspend for tegra2, it involved fixing reset handler, sleep 
and l2 cache resume asm code. Also fixed tegra timer and i2c drivers in order to 
get booting. Unfortunately I don't have tegra3 hw nor support for it in QEMU, 
however cpuidle should work I suppose. I'll post patches later.

https://dl.dropboxusercontent.com/u/3913472/IMG_20150112_202034.jpg


-- 
Dmitry