mmc: rmobile: add a compiler barrier
authorYannick Gicquel <yannick.gicquel@iot.bzh>
Mon, 4 Apr 2016 09:49:59 +0000 (11:49 +0200)
committerNobuhiro Iwamatsu <iwamatsu@nigauri.org>
Wed, 17 Aug 2016 01:25:35 +0000 (10:25 +0900)
Building w/ GCC v5.2, the SD card access is broken due to invalid data
in the response command reconstructed at the end of
sh_sdhci_get_response().

Add a memory barrier between the two main steps of this function to
ensure the resp[] table content is consistent before bits reordering.

This fix has been tested Ok on Porter board rev1.0 using v2016.03
release.

Signed-off-by: Yannick Gicquel <yannick.gicquel@iot.bzh>
drivers/mmc/sh_sdhi.c

index 30e538c..dc4c686 100644 (file)
@@ -430,6 +430,9 @@ static void sh_sdhi_get_response(struct sh_sdhi_host *host, struct mmc_cmd *cmd)
        }
 
        p2 = (unsigned long *)cmd->response;
+
+       barrier();
+
 #if defined(__BIG_ENDIAN_BITFIELD)
        for (i = 0; i < cnt; i++) {
                *p2++ = ((*p1 >> 16) & 0x0000ffff) |