From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_PASS, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3995BC04EB8 for ; Mon, 10 Dec 2018 10:50:41 +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 E89D120645 for ; Mon, 10 Dec 2018 10:50:40 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="YayMI6KO" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E89D120645 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=bootlin.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.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:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=zyDpxrWb6jnt8WF+1GlrHqSkgmyRsMIdy+4Rya88OHc=; b=YayMI6KOtn3sba WUtm8ngacT4OwPS9FoPeePlnmuNipPzvbRw6tUSdlaSuiEaoqHc2c0eOaAWascoxqXfInOLIoaxuj 3xVltnqdREgVbd7brG92gPcZeBwKxVDvjxBfuqaczzw3oklk5RBoRo42lNOZ7UxVUd8cxcbaTRLRb f1XSHWlinuLuYvISl+at8AiUM/vaZpn2C9GrsW3tpFqwZDGpQAGQLiayw54QEUI65UbUWoVf2csOm jf4AP7glJ1SS+FYmPQB1mXZh4MLaxug7r4o4GTMxHmjMYWBOTzaCtWvRhxqGBTqR6XxfRq0DnP1tj N53cP+5kvy8IZ98TQd0w==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gWJ93-00020I-3R; Mon, 10 Dec 2018 10:50:37 +0000 Received: from mail.bootlin.com ([62.4.15.54]) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gWJ90-0001zq-1g; Mon, 10 Dec 2018 10:50:35 +0000 Received: by mail.bootlin.com (Postfix, from userid 110) id B9FAB20CEB; Mon, 10 Dec 2018 11:50:22 +0100 (CET) Received: from bbrezillon (unknown [91.160.177.164]) by mail.bootlin.com (Postfix) with ESMTPSA id 2EE5820DC0; Mon, 10 Dec 2018 11:50:01 +0100 (CET) Date: Mon, 10 Dec 2018 11:50:01 +0100 From: Boris Brezillon To: Yogesh Narayan Gaur Subject: Re: [PATCH v5 1/5] spi: spi-mem: Add driver for NXP FlexSPI controller Message-ID: <20181210115001.6c7af1d7@bbrezillon> In-Reply-To: References: <1542366701-16065-1-git-send-email-yogeshnarayan.gaur@nxp.com> <1542366701-16065-2-git-send-email-yogeshnarayan.gaur@nxp.com> <20181210111909.35384eee@bbrezillon> X-Mailer: Claws Mail 3.16.0 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20181210_025034_228856_8FB6A558 X-CRM114-Status: GOOD ( 12.13 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "mark.rutland@arm.com" , "devicetree@vger.kernel.org" , "robh@kernel.org" , "linux-kernel@vger.kernel.org" , Schrempf Frieder , "linux-spi@vger.kernel.org" , "marek.vasut@gmail.com" , "broonie@kernel.org" , "linux-mtd@lists.infradead.org" , "computersforpeace@gmail.com" , "shawnguo@kernel.org" , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, 10 Dec 2018 10:43:56 +0000 Yogesh Narayan Gaur wrote: > > > Thus, in LUT preparation we have assigned only the base address. > > > Now if I have assigned ahb_buf_size to FSPI_FLSHXXCR0 register then for > > read/write data beyond limit of ahb_buf_size offset I get data corruption. > > > > Why would you do that? We have the ->adjust_op_size() exactly for this reason, > > so, if someone tries to do a spi_mem_op with data.nbytes > ahb_buf_size you > > should return an error. > > > Let me explain my implementation with example. If I have to write data of size 0x100 bytes at offset 0x1200 for CS1, I would program as below: > In func nxp_fspi_select_mem(), would set value of controller address space size, memmap_phy_size, to FSPI_FLSHA2CR0 and rest all FSPI_FLSHXXCR0 as 0. > Value of memmap_phy_size is 0x10000000 i.e. 256 MB for my LX2160ARDB target. > Then in nxp_fspi_prepare_lut(), I would prepare LUT ADDR with address length requirement 3/4 byte for NOR or 1/2/3/4 bytes for NAND flash. > Also for LUT_NXP_WRITE would program data bytes as 0. > > Then inside func nxp_fspi_do_op(), set register FSPI_IPCR0 as the address offset i.e. 0x1200 and in register FSPI_IPCR1 program the data size to write i.e. 0x100 > > If, as suggested if I tries to mark value of register FSPI_FLSHA2CR0 equal to ahb_buf_size (0x800), then access for address 0x1200 gives me wrong data. This is because as per the controller specification access to flash connected at CS1 can be performed under range of FSPI_ FLSHA1CR0 and FSPI_ FLSHA2CR0. Don't you have a way to set an offset to apply to the address accessed through the AHB? And if you don't, how will it work if your mapping is smaller than the flash size? _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel