spi: mtk-snfi: preserve dma_mapping_error() error codes
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 29 Apr 2022 17:41:37 +0000 (20:41 +0300)
committerMark Brown <broonie@kernel.org>
Tue, 3 May 2022 13:00:38 +0000 (14:00 +0100)
Return -ENOMEM of there is a dma mapping error.  Do not return success.

Fixes: 764f1b748164 ("spi: add driver for MTK SPI NAND Flash Interface")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Chuanhong Guo <gch981213@gmail.com>
Link: https://lore.kernel.org/r/YmwjUcTKyQNrrn2g@kili
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-mtk-snfi.c

index 2c556e3..d66bf97 100644 (file)
@@ -903,7 +903,8 @@ static int mtk_snand_read_page_cache(struct mtk_snand *snf,
        nfi_write32(snf, NFI_CON, (snf->nfi_cfg.nsectors << CON_SEC_NUM_S));
 
        buf_dma = dma_map_single(snf->dev, buf, dma_len, DMA_FROM_DEVICE);
-       if (dma_mapping_error(snf->dev, buf_dma)) {
+       ret = dma_mapping_error(snf->dev, buf_dma);
+       if (ret) {
                dev_err(snf->dev, "DMA mapping failed.\n");
                goto cleanup;
        }
@@ -1092,7 +1093,8 @@ static int mtk_snand_write_page_cache(struct mtk_snand *snf,
 
        nfi_write32(snf, NFI_CON, (snf->nfi_cfg.nsectors << CON_SEC_NUM_S));
        buf_dma = dma_map_single(snf->dev, snf->buf, dma_len, DMA_TO_DEVICE);
-       if (dma_mapping_error(snf->dev, buf_dma)) {
+       ret = dma_mapping_error(snf->dev, buf_dma);
+       if (ret) {
                dev_err(snf->dev, "DMA mapping failed.\n");
                goto cleanup;
        }