From patchwork Sun Nov 10 05:21:01 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chuanhong Guo X-Patchwork-Id: 11236145 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 5BA31139A for ; Sun, 10 Nov 2019 05:22:29 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 38476214E0 for ; Sun, 10 Nov 2019 05:22:29 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="hPo6k9zA"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="QrI0mXi/" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 38476214E0 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-mediatek-bounces+patchwork-linux-mediatek=patchwork.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-Id:Date:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=kgFcTK2vApBTyKXmhr3Doyxko+G3XpKFtnDmExriQkg=; b=hPo6k9zAR26kpX qb8DkeQKQqtc6RWVSuZNhgBPhFgzIjXQrBdBxqn8L+pbzPYa4NxBnW2nsO+w3JNTvc3UPcSnzlDEq aqIAZi+7jbiDzNlc0BYAet6QfYDbOEHZrSicVu/Q0OCVIIt1cs2+172iDv3SGiI+p74PaqT8IdzVv ryQAjbb82zrQ2+VSCNmLd4JCj6o9FfFTFhG6UHkB0ZJGBSm0kXHbGQ4/pyH+Dioab10hPH7v5koYV uNb5jCE0G9E2AEoR4REaBjZvwycXEQfcgRd0e7I515lfsu+sZTMMsca0u/JrGVJbzPbr7h+xO7vuw Qc3364u4eiEiNAF2Fuyw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1iTfgB-00062F-D0; Sun, 10 Nov 2019 05:22:27 +0000 Received: from mail-pg1-x544.google.com ([2607:f8b0:4864:20::544]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1iTffN-0005GI-Ao; Sun, 10 Nov 2019 05:21:38 +0000 Received: by mail-pg1-x544.google.com with SMTP id q22so6859523pgk.2; Sat, 09 Nov 2019 21:21:36 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=4ITuc3/EbXNp+U9B5Okc4zxW+jSvy8I6mUAG/x+04NA=; b=QrI0mXi/pUYaShNhrdkwb/zDqfns0lptxfNNJ2ClmqLWNmb3NK+udb3X4OiBZWARIT BgudGtWl4Le6Eq36QZ/852P4+8Sl/fp3BQGYUrl7ZfMfTPXgDX+cs0JLw7+WJg+AZpIr 2YKY+grJxf73XYofMvypHJT/tOI37BN/PupM56IbZ23ISPuTtwgn1NGysCpOHJXLFRjD 9A6oKuySSnrhy094rQP/v/E47xRWEoZmut2fAlCYecspBbONQ3gYoeX1euAOVXgVsYRd fNr+CpR4xcnp7Zkm+p05c3xtTMuf4hnh82TNmhePrRPpPoAlLW7iUaxLpfAC8zmsAUMO IsDQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=4ITuc3/EbXNp+U9B5Okc4zxW+jSvy8I6mUAG/x+04NA=; b=iwIq9N2JOckvCHZZtTJQNIXBRYUsNCwtmXlrJO7dKd/gDlPknbYvSSvCxtlvFq6D/j WDLphNPpEZIa7K3Q0eINfCKEpcYX2uCgBDu3zV+JceDQp1RwGOqCJwdQlO9NYB7YMJ0Q ZeU6OriLjWHE+6dgFsfsNU4hnKH2QBWzJTIty61LhOkSe+oUT/NXlG20wm7qYIhqMcwT 9eoxHfxyZjyfX8yGJVCSJqhLlmH0O3zkyZlNlRZb0pJOSJOGHAHAN6Rhai280rmFMsbs 2eNjo0SHS4faX69Fvl+/ScPnGAH+PE5/HGsUY/qNaE0B07tm5J7kKnJELxiSfAxqwG9i cKTw== X-Gm-Message-State: APjAAAUShz8c2QH7jLmg7PGBwwd9QXwEKqlm4P+oXel8lP/prbeaO4QP 65+FxtsUE7dFykpzuuF8TrLhMFkX X-Google-Smtp-Source: APXvYqz38qmGlTu6irHIzXwq6E1qIqAZdTkRlEXf5i+owvXl66qNobg3mB7Q/Hg+mtvZ8+5CBJaPkA== X-Received: by 2002:a63:e249:: with SMTP id y9mr21232293pgj.383.1573363296186; Sat, 09 Nov 2019 21:21:36 -0800 (PST) Received: from localhost.localdomain ([2001:19f0:7001:2668:5400:1ff:fe62:2bbd]) by smtp.gmail.com with ESMTPSA id y2sm10728754pfe.126.2019.11.09.21.21.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 09 Nov 2019 21:21:35 -0800 (PST) From: Chuanhong Guo To: linux-mtd@lists.infradead.org Subject: [PATCH 2/2] mtd: mtk-quadspi: misuse 1_1_2 read mode for custom read opcode Date: Sun, 10 Nov 2019 13:21:01 +0800 Message-Id: <20191110052104.5502-2-gch981213@gmail.com> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20191110052104.5502-1-gch981213@gmail.com> References: <20191110052104.5502-1-gch981213@gmail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20191109_212137_418218_05F291F6 X-CRM114-Status: GOOD ( 14.09 ) X-Spam-Score: 0.1 (/) X-Spam-Report: SpamAssassin version 3.4.2 on bombadil.infradead.org summary: Content analysis details: (0.1 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at https://www.dnswl.org/, no trust [2607:f8b0:4864:20:0:0:0:544 listed in] [list.dnswl.org] 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider (gch981213[at]gmail.com) -0.0 SPF_PASS SPF: sender matches SPF record 0.2 FREEMAIL_ENVFROM_END_DIGIT Envelope-from freemail username ends in digit (gch981213[at]gmail.com) -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature -0.1 DKIM_VALID_EF Message has a valid DKIM or DK signature from envelope-from domain 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Vignesh Raghavendra , Tudor Ambarus , Richard Weinberger , linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, Miquel Raynal , Matthias Brugger , Chuanhong Guo , linux-arm-kernel@lists.infradead.org Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+patchwork-linux-mediatek=patchwork.kernel.org@lists.infradead.org 1_1_1 reading mode on this controller only support 0x03 and 0x0b as opcode, but spi-nor framework uses nor->read for SFDP reading as well. Add a check for opcode and if it's not supported, misuse 1_1_2 reading and extract corresponding bits from returned data. Signed-off-by: Chuanhong Guo --- drivers/mtd/spi-nor/mtk-quadspi.c | 78 ++++++++++++++++++++++++++++++- 1 file changed, 76 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/spi-nor/mtk-quadspi.c b/drivers/mtd/spi-nor/mtk-quadspi.c index ac0e531ce80c..46bf27c0e6e8 100644 --- a/drivers/mtd/spi-nor/mtk-quadspi.c +++ b/drivers/mtd/spi-nor/mtk-quadspi.c @@ -357,8 +357,8 @@ static ssize_t mtk_nor_read_dma_bounce(struct mtk_nor *mtk_nor, loff_t from, return length; } -static ssize_t mtk_nor_read(struct spi_nor *nor, loff_t from, size_t length, - u_char *buffer) +static ssize_t mtk_nor_flash_read(struct spi_nor *nor, loff_t from, + size_t length, u_char *buffer) { struct mtk_nor *mtk_nor = nor->priv; @@ -372,6 +372,80 @@ static ssize_t mtk_nor_read(struct spi_nor *nor, loff_t from, size_t length, return mtk_nor_read_dma(mtk_nor, from, length, buffer); } +static ssize_t mtk_nor_generic_read(struct spi_nor *nor, loff_t from, + size_t length, u_char *buffer) +{ + struct mtk_nor *mtk_nor = nor->priv; + ssize_t nor_unaligned_len = from % MTK_NOR_DMA_ALIGN; + loff_t read_from = from & ~(MTK_NOR_DMA_ALIGN - 1); + ssize_t read_len; + u_char *buf, *bouncebuf, tmp; + size_t mem_unaligned_len, i; + dma_addr_t dma_addr; + int ret; + + if (length > MTK_NOR_MAX_BBUF_READ / 2) + length = MTK_NOR_MAX_BBUF_READ / 2; + read_len = ((length + nor_unaligned_len) * 2 + MTK_NOR_DMA_ALIGN) & + ~(MTK_NOR_DMA_ALIGN - 1); + + buf = kmalloc(read_len + MTK_NOR_DMA_ALIGN, GFP_KERNEL); + if (!buf) + return -ENOMEM; + + mem_unaligned_len = (u32)buf % MTK_NOR_DMA_ALIGN; + bouncebuf = (buf + MTK_NOR_DMA_ALIGN) - mem_unaligned_len; + + writeb(nor->read_opcode, mtk_nor->base + MTK_NOR_PRGDATA3_REG); + writeb(MTK_NOR_DUAL_READ_EN, mtk_nor->base + MTK_NOR_DUAL_REG); + mtk_nor_set_addr_width(mtk_nor); + + dma_addr = dma_map_single(mtk_nor->dev, bouncebuf, read_len, + DMA_FROM_DEVICE); + ret = dma_mapping_error(mtk_nor->dev, dma_addr); + if (ret) { + dev_err(mtk_nor->dev, "failed to map dma buffer."); + goto err; + } + + writel(read_from, mtk_nor->base + MTK_NOR_FDMA_FADR_REG); + writel(dma_addr, mtk_nor->base + MTK_NOR_FDMA_DADR_REG); + writel((u32)dma_addr + read_len, + mtk_nor->base + MTK_NOR_FDMA_END_DADR_REG); + ret = mtk_nor_dma_exec(mtk_nor); + dma_unmap_single(mtk_nor->dev, dma_addr, read_len, DMA_FROM_DEVICE); + + if (ret) + goto err; + + /* extract bits from DO line */ + for (i = 0; i < length; i++) { + tmp = bouncebuf[(i + nor_unaligned_len) * 2]; + buffer[i] = (tmp & BIT(7)) | ((tmp & BIT(5)) << 1) | + ((tmp & BIT(3)) << 2) | ((tmp & BIT(1)) << 3); + tmp = bouncebuf[(i + nor_unaligned_len) * 2 + 1]; + buffer[i] |= (tmp & BIT(7)) >> 4 | ((tmp & BIT(5)) >> 3) | + ((tmp & BIT(3)) >> 2) | ((tmp & BIT(1)) >> 1); + } + ret = length; +err: + kfree(buf); + return ret; +} + +static ssize_t mtk_nor_read(struct spi_nor *nor, loff_t from, size_t length, + u_char *buffer) +{ + if ((nor->read_proto != SNOR_PROTO_1_1_1) || + (nor->read_opcode == SPINOR_OP_READ) || + (nor->read_opcode == SPINOR_OP_READ_FAST)) + return mtk_nor_flash_read(nor, from, length, buffer); + else if (nor->read_dummy == 8) + return mtk_nor_generic_read(nor, from, length, buffer); + else + return -EOPNOTSUPP; +} + static int mtk_nor_write_single_byte(struct mtk_nor *mtk_nor, int addr, int length, u8 *data) {