mmc: omap_hsmmc: clear status flags before starting a new command
authorFrancesco Lavra <francescolavra.fl@gmail.com>
Sat, 29 Jun 2013 06:25:12 +0000 (08:25 +0200)
committerChris Ball <cjb@laptop.org>
Sun, 25 Aug 2013 19:41:17 +0000 (15:41 -0400)
commit7472bab236bdee1173412585591329e718f4d324
tree72c62f00ba540c775227300631869eef4ee8d6ef
parent00fd041b0c5265ca45e6f60f4d92234b9f37a064
mmc: omap_hsmmc: clear status flags before starting a new command

Commit 1f6b9fa40e76fffaaa0b3bd6a0bfdcf1cdc06efa consolidated writes to
the STAT register in one location, moving them from omap_hsmmc_do_irq()
to omap_hsmmc_irq(). This move has the unwanted side effect that the
controller status flags are potentially cleared after a new command has
been started as a consequence of reading the previous status flags.
This means that if the new command changes the status flags before the
IRQ routine returns, those flags may be cleared without handling the
event which asserted them, and thus missing the event.
Move the writing of the STAT register back in omap_hsmmc_do_irq(),
before handling the status flags which generated the interrupt.

Signed-off-by: Francesco Lavra <francescolavra.fl@gmail.com>
Reviewed-and-Tested-by: Balaji T K <balajitk@ti.com>
Signed-off-by: Chris Ball <cjb@laptop.org>
drivers/mmc/host/omap_hsmmc.c