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 -- 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
