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. Thank for this. I suggest submitting directly as there is no need to make a coherent series of these. -- Ben Dooks http://www.codethink.co.uk/ Senior Engineer Codethink - Providing Genius -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
