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

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

19.01.2015 17:26, Bob Mottram пишет:
> On 12/01/15 18:04, Dmitry Osipenko wrote:
>> 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
>
> Nice. Do you have a patch for those fixes?
>
Sorry for delay. I have patches locally, need to prepare them. I think some arch 
code needs to be cleaned first, so it might be some more work. Planning to send 
this week.


-- 
Dmitry