[PATCH] fix-glamo-idleclock-around-suspend.patch
Werner Almesberger
werner at openmoko.org
Wed Sep 3 15:10:47 CEST 2008
Andy Green wrote:
> + /* we are going to do more commands to override this in
> + * mmc_suspend_host(), so we need to change sd_idleclk for the
> + * duration as well
> + */
Hmm, isn't this a race between the card's internal controller doing
whatever it does and the glamo's PLLs being shut down, i.e., when the
clock is being stopped for real ?
I think it would be good if someone could test this by turning off
TCLK right after mmc_suspend_host returns. If there is still
corruption, add a suitable usleep or msleep. Then measure the time
the system spends in mmc_suspend_host and add that to the sleep.
It's also not so nice that there's per-device state in a single
global variable. But okay, one Glamo per system is bad enough, who
would want to have more anyway ? ;-)
- Werner
More information about the openmoko-kernel
mailing list