memory: renesas-rpc-if: Clear HS bit during hardware initialization
authorCong Dang <cong.dang.xn@renesas.com>
Wed, 12 Oct 2022 14:01:51 +0000 (16:01 +0200)
committerKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Tue, 18 Oct 2022 17:02:44 +0000 (13:02 -0400)
According to the datasheet, HS bit should be specified to 1 when using
DMA transfer. As DMA transfer is not supported, it should be cleared to
0.

Previously, the driver relied on the HS bit being cleared by prior
firmware but this is not always the case.

Fix this by ensuring the bit is cleared during hardware initialization.

Fixes: ca7d8b980b67 ("memory: add Renesas RPC-IF driver")
Signed-off-by: Cong Dang <cong.dang.xn@renesas.com>
Signed-off-by: Hai Pham <hai.pham.ud@renesas.com>
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Link: https://lore.kernel.org/r/08d9fb10b3051decebf871267a6e2e7cb2d4faf9.1665583089.git.geert+renesas@glider.be
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
drivers/memory/renesas-rpc-if.c

index 4316988..61c288d 100644 (file)
@@ -317,6 +317,9 @@ int rpcif_hw_init(struct rpcif *rpc, bool hyperflash)
        regmap_update_bits(rpc->regmap, RPCIF_PHYCNT, RPCIF_PHYCNT_PHYMEM_MASK,
                           RPCIF_PHYCNT_PHYMEM(hyperflash ? 3 : 0));
 
+       /* DMA Transfer is not supported */
+       regmap_update_bits(rpc->regmap, RPCIF_PHYCNT, RPCIF_PHYCNT_HS, 0);
+
        if (rpc->type == RPCIF_RCAR_GEN3)
                regmap_update_bits(rpc->regmap, RPCIF_PHYCNT,
                                   RPCIF_PHYCNT_STRTIM(7), RPCIF_PHYCNT_STRTIM(7));