From patchwork Thu Jun 29 07:16:04 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Abhishek Sahu X-Patchwork-Id: 782089 X-Patchwork-Delegate: boris.brezillon@free-electrons.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3wysHk45Vpz9s0g for ; Thu, 29 Jun 2017 17:49:22 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="C2e6B7Z1"; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=infradead.org header.i=@infradead.org header.b="j11pFLS0"; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=codeaurora.org header.i=@codeaurora.org header.b="LzYhQEe2"; dkim=neutral (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="Bejfsyul"; dkim-atps=neutral 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:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: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=xPvxzfddunoSJGY6iNSuXU6TdSKqc35T/0B4NIflrlU=; b=C2e6B7Z1DqzUtTCoXim1/+NAhu ikiVibUF3vUaKUze+HMvWUugKjZIpms9KrRRfxxKCnUG4OXRz9gHy2S43tLiYjtdeUC+LAZKxD6qH OzNu6PcAMx4zTPz6azoes5F0BHSgFhwV5p/YxHNu2C9OnG6pCLBBm0F7G1xAvcymdDrKacrBUyOz3 mywgnXKya/TgWuSgtKgcW7z+L5DJqeYUfc6cNROhRl534JHJmoKPVVOun+WQcJvSOtmfzwZOEqbE/ ttYnuiiZ5d8K39gDczkOZ3ul0Nc1v4iJAHcSVdEuscduAIFS8RGGDRB8dBN1MZOFsHcQGvfaGHrfV HZLQJT9w==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dQUCP-0006ac-HK; Thu, 29 Jun 2017 07:49:13 +0000 Received: from merlin.infradead.org ([2001:8b0:10b:1231::1]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1dQUCN-0006Wu-L0 for linux-mtd@bombadil.infradead.org; Thu, 29 Jun 2017 07:49:12 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=merlin.20170209; h=References:In-Reply-To:Message-Id:Date: Subject:Cc:To:From:Sender:Reply-To:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=M1D/wgehD4qMmZkb6cYz/+YxicxPZdP6qmqZuaANPMc=; b=j11pFLS0nu/Me/tsOOcWwhi59 /oOR7PGAoTEawuokSaZUzSzWmSpfeEAI+YGNtZ9d5ldwgoM+2s41hXz/uIwR68wvHSOAw6RGnvvXm vQxJsYYv4wOyISNecb7JQRux/e+6VQ0ipWz9oUYSLC/Z9x1wbcU8mNgyl4rMUstJYIraFtC5pUBYK Np/y3vKsNGwCnKbkL6CPXCUzdF59k+Z9Tk/vm1QPRGNElSTpORWtv2MV1n2AbbeADgF9U/PIlc1QR HlRRL7eAy3cNzSRAIH9UXBqAoWt4tJwTG8qebtuiXeVlqGDAo7/7JOIe9qIfjpR2xaWfC3mhIFqq3 j06OR29pw==; Received: from smtp.codeaurora.org ([198.145.29.96]) by merlin.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1dQThw-0008Vq-5O for linux-mtd@lists.infradead.org; Thu, 29 Jun 2017 07:17:44 +0000 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id F2C4460FEE; Thu, 29 Jun 2017 07:17:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1498720648; bh=6gK3kC1z2Cyh4lILFntRr6LLiWOdHaUpS5CUqFc6MTA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=LzYhQEe27D7Wai/7D+No4nT+BLgh6rfkyKEnUJqrBghmnqn2AtOyYMt+ueIKagppH 9Yp6ySAF7ixyHOoFlJCcTeO+HklyqOmAg9EznK++nUQPvkThqixukik25Y0nTF8t75 7ojq24AmSCveDICQoKJWMGLGY8CvM0NtMlaT8vN4= X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on pdx-caf-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.8 required=2.0 tests=ALL_TRUSTED,BAYES_00, DKIM_SIGNED, T_DKIM_INVALID autolearn=no autolearn_force=no version=3.4.0 Received: from absahu-linux.qualcomm.com (blr-c-bdr-fw-01_globalnat_allzones-outside.qualcomm.com [103.229.19.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: absahu@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id E322660DDD; Thu, 29 Jun 2017 07:17:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1498720646; bh=6gK3kC1z2Cyh4lILFntRr6LLiWOdHaUpS5CUqFc6MTA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=BejfsyulkzhTHjYFiQCLVtqQI5To2d6tgq9ks/AAdNuisaIQ62UG3FJbksXmUqqa+ 7laJMrZj+fiRoC+DvPeeJXN13Ewj8ajiQOCDsi/egfWvdD+X0mQbzcO6cb0olNydGl b3DXWFiC1WMBXnPJfCsBDQ+MgIrihqJLSZnCwMpQ= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org E322660DDD Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=absahu@codeaurora.org From: Abhishek Sahu To: dwmw2@infradead.org, computersforpeace@gmail.com, boris.brezillon@free-electrons.com, marek.vasut@gmail.com, richard@nod.at, cyrille.pitchen@wedev4u.fr, robh+dt@kernel.org, mark.rutland@arm.com Subject: [PATCH 12/14] qcom: mtd: nand: change register offset defines with enums Date: Thu, 29 Jun 2017 12:46:04 +0530 Message-Id: <1498720566-20782-13-git-send-email-absahu@codeaurora.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1498720566-20782-1-git-send-email-absahu@codeaurora.org> References: <1498720566-20782-1-git-send-email-absahu@codeaurora.org> X-Spam-Note: CRM114 invocation failed X-Spam-Score: -2.0 (--) X-Spam-Report: SpamAssassin version 3.4.1 on merlin.infradead.org summary: Content analysis details: (-2.0 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/, no trust [198.145.29.96 listed in list.dnswl.org] -0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay domain -0.0 SPF_PASS SPF: sender matches SPF record -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, architt@codeaurora.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Abhishek Sahu , linux-mtd@lists.infradead.org, andy.gross@linaro.org, sricharan@codeaurora.org MIME-Version: 1.0 Sender: "linux-mtd" Errors-To: linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org The current driver defines the register offset with preprocessor macro which is defined crossponding to NAND controller version 1.4.0. This patch changes these macro with enumeration. It also adds mapping array which contains controller register offsets for each register offset enumeration. This mapping array will be referenced before each register read and writes, where the register offset enumeration is being replaced with actual register offsets. Signed-off-by: Abhishek Sahu --- drivers/mtd/nand/qcom_nandc.c | 136 +++++++++++++++++++++++++++--------------- 1 file changed, 89 insertions(+), 47 deletions(-) diff --git a/drivers/mtd/nand/qcom_nandc.c b/drivers/mtd/nand/qcom_nandc.c index 6d749b6..24750e6 100644 --- a/drivers/mtd/nand/qcom_nandc.c +++ b/drivers/mtd/nand/qcom_nandc.c @@ -24,43 +24,6 @@ #include #include -/* NANDc reg offsets */ -#define NAND_FLASH_CMD 0x00 -#define NAND_ADDR0 0x04 -#define NAND_ADDR1 0x08 -#define NAND_FLASH_CHIP_SELECT 0x0c -#define NAND_EXEC_CMD 0x10 -#define NAND_FLASH_STATUS 0x14 -#define NAND_BUFFER_STATUS 0x18 -#define NAND_DEV0_CFG0 0x20 -#define NAND_DEV0_CFG1 0x24 -#define NAND_DEV0_ECC_CFG 0x28 -#define NAND_DEV1_ECC_CFG 0x2c -#define NAND_DEV1_CFG0 0x30 -#define NAND_DEV1_CFG1 0x34 -#define NAND_READ_ID 0x40 -#define NAND_READ_STATUS 0x44 -#define NAND_DEV_CMD0 0xa0 -#define NAND_DEV_CMD1 0xa4 -#define NAND_DEV_CMD2 0xa8 -#define NAND_DEV_CMD_VLD 0xac -#define SFLASHC_BURST_CFG 0xe0 -#define NAND_ERASED_CW_DETECT_CFG 0xe8 -#define NAND_ERASED_CW_DETECT_STATUS 0xec -#define NAND_EBI2_ECC_BUF_CFG 0xf0 -#define FLASH_BUF_ACC 0x100 - -#define NAND_CTRL 0xf00 -#define NAND_VERSION 0xf08 -#define NAND_READ_LOCATION_0 0xf20 -#define NAND_READ_LOCATION_1 0xf24 -#define NAND_READ_LOCATION_2 0xf28 -#define NAND_READ_LOCATION_3 0xf2c - -/* dummy register offsets, used by write_reg_dma */ -#define NAND_DEV_CMD1_RESTORE 0xdead -#define NAND_DEV_CMD_VLD_RESTORE 0xbeef - /* NAND_FLASH_CMD bits */ #define PAGE_ACC BIT(4) #define LAST_PAGE BIT(5) @@ -204,6 +167,44 @@ #define QPIC_PER_CW_MAX_CMD_SGL (32) #define QPIC_PER_CW_MAX_DATA_SGL (8) +/* NANDc reg offsets enumeration */ +enum { + NAND_FLASH_CMD, + NAND_ADDR0, + NAND_ADDR1, + NAND_FLASH_CHIP_SELECT, + NAND_EXEC_CMD, + NAND_FLASH_STATUS, + NAND_BUFFER_STATUS, + NAND_DEV0_CFG0, + NAND_DEV0_CFG1, + NAND_DEV0_ECC_CFG, + NAND_DEV1_ECC_CFG, + NAND_DEV1_CFG0, + NAND_DEV1_CFG1, + NAND_READ_ID, + NAND_READ_STATUS, + NAND_DEV_CMD0, + NAND_DEV_CMD1, + NAND_DEV_CMD2, + NAND_DEV_CMD_VLD, + SFLASHC_BURST_CFG, + NAND_ERASED_CW_DETECT_CFG, + NAND_ERASED_CW_DETECT_STATUS, + NAND_EBI2_ECC_BUF_CFG, + FLASH_BUF_ACC, + NAND_CTRL, + NAND_VERSION, + NAND_READ_LOCATION_0, + NAND_READ_LOCATION_1, + NAND_READ_LOCATION_2, + NAND_READ_LOCATION_3, + + /* dummy register offsets, used by write_reg_dma */ + NAND_DEV_CMD1_RESTORE, + NAND_DEV_CMD_VLD_RESTORE, +}; + /* * This data type corresponds to the BAM transaction which will be used for all * NAND transfers. @@ -326,6 +327,7 @@ struct nandc_regs { * bam dma * @max_cwperpage: maximum qpic codeword required. calcualted * from all nand device pagesize + * @regs_offsets: register offset mapping array */ struct qcom_nand_controller { struct nand_hw_control controller; @@ -371,6 +373,7 @@ struct qcom_nand_controller { u32 cmd1, vld; u32 ecc_modes; + const u32 *regs_offsets; }; /* @@ -434,6 +437,40 @@ struct qcom_nand_driver_data { bool dma_bam_enabled; }; +/* Mapping table which contains the actual register offsets */ +static const u32 regs_offsets[] = { + [NAND_FLASH_CMD] = 0x00, + [NAND_ADDR0] = 0x04, + [NAND_ADDR1] = 0x08, + [NAND_FLASH_CHIP_SELECT] = 0x0c, + [NAND_EXEC_CMD] = 0x10, + [NAND_FLASH_STATUS] = 0x14, + [NAND_BUFFER_STATUS] = 0x18, + [NAND_DEV0_CFG0] = 0x20, + [NAND_DEV0_CFG1] = 0x24, + [NAND_DEV0_ECC_CFG] = 0x28, + [NAND_DEV1_ECC_CFG] = 0x2c, + [NAND_DEV1_CFG0] = 0x30, + [NAND_DEV1_CFG1] = 0x34, + [NAND_READ_ID] = 0x40, + [NAND_READ_STATUS] = 0x44, + [NAND_DEV_CMD0] = 0xa0, + [NAND_DEV_CMD1] = 0xa4, + [NAND_DEV_CMD2] = 0xa8, + [NAND_DEV_CMD_VLD] = 0xac, + [SFLASHC_BURST_CFG] = 0xe0, + [NAND_ERASED_CW_DETECT_CFG] = 0xe8, + [NAND_ERASED_CW_DETECT_STATUS] = 0xec, + [NAND_EBI2_ECC_BUF_CFG] = 0xf0, + [FLASH_BUF_ACC] = 0x100, + [NAND_CTRL] = 0xf00, + [NAND_VERSION] = 0xf08, + [NAND_READ_LOCATION_0] = 0xf20, + [NAND_READ_LOCATION_1] = 0xf24, + [NAND_READ_LOCATION_2] = 0xf28, + [NAND_READ_LOCATION_3] = 0xf2c, +}; + /* Frees the BAM transaction memory */ static void free_bam_transaction(struct qcom_nand_controller *nandc) { @@ -516,13 +553,13 @@ static inline struct qcom_nand_host *to_qcom_nand_host(struct nand_chip *chip) static inline u32 nandc_read(struct qcom_nand_controller *nandc, int offset) { - return ioread32(nandc->base + offset); + return ioread32(nandc->base + nandc->regs_offsets[offset]); } static inline void nandc_write(struct qcom_nand_controller *nandc, int offset, u32 val) { - iowrite32(val, nandc->base + offset); + iowrite32(val, nandc->base + nandc->regs_offsets[offset]); } static __le32 *offset_to_nandc_reg(struct nandc_regs *regs, int offset) @@ -903,11 +940,12 @@ static int read_reg_dma(struct qcom_nand_controller *nandc, int first, nandc->reg_read_pos += num_regs; if (nandc->dma_bam_enabled) - return prep_dma_desc_command(nandc, true, first, vaddr, + return prep_dma_desc_command(nandc, true, + nandc->regs_offsets[first], vaddr, num_regs, flags); - return prep_dma_desc(nandc, true, first, vaddr, num_regs * sizeof(u32), - flow_control); + return prep_dma_desc(nandc, true, nandc->regs_offsets[first], vaddr, + num_regs * sizeof(u32), flow_control); } /* @@ -946,11 +984,12 @@ static int write_reg_dma(struct qcom_nand_controller *nandc, int first, first = NAND_DEV_CMD_VLD; if (nandc->dma_bam_enabled) - return prep_dma_desc_command(nandc, false, first, vaddr, + return prep_dma_desc_command(nandc, false, + nandc->regs_offsets[first], vaddr, num_regs, flags); - return prep_dma_desc(nandc, false, first, vaddr, num_regs * sizeof(u32), - flow_control); + return prep_dma_desc(nandc, false, nandc->regs_offsets[first], vaddr, + num_regs * sizeof(u32), flow_control); } /* @@ -968,7 +1007,8 @@ static int read_data_dma(struct qcom_nand_controller *nandc, int reg_off, return prep_dma_desc_data_bam(nandc, true, reg_off, vaddr, size, flags); - return prep_dma_desc(nandc, true, reg_off, vaddr, size, false); + return prep_dma_desc(nandc, true, nandc->regs_offsets[FLASH_BUF_ACC] + + reg_off - FLASH_BUF_ACC, vaddr, size, false); } /* @@ -986,7 +1026,8 @@ static int write_data_dma(struct qcom_nand_controller *nandc, int reg_off, return prep_dma_desc_data_bam(nandc, false, reg_off, vaddr, size, flags); - return prep_dma_desc(nandc, false, reg_off, vaddr, size, false); + return prep_dma_desc(nandc, false, nandc->regs_offsets[FLASH_BUF_ACC] + + reg_off - FLASH_BUF_ACC, vaddr, size, false); } /* @@ -2791,6 +2832,7 @@ static int qcom_nandc_probe(struct platform_device *pdev) nandc->ecc_modes = driver_data->ecc_modes; nandc->dma_bam_enabled = driver_data->dma_bam_enabled; + nandc->regs_offsets = regs_offsets; res = platform_get_resource(pdev, IORESOURCE_MEM, 0); nandc->base = devm_ioremap_resource(dev, res);