linux-spi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sourav Poddar <sourav.poddar@ti.com>
To: <computersforpeace@gmail.com>
Cc: devicetree@vger.kernel.org, balbi@ti.com,
	linux-spi@vger.kernel.org, broonie@kernel.org,
	linux-mtd@lists.infradead.org, bcousson@baylibre.com,
	Sourav Poddar <sourav.poddar@ti.com>,
	linux-omap@vger.kernel.org
Subject: Re: [PATCH 00/17] Add memory mapped support for ti qspi, m25p80 serial flash.
Date: Thu, 5 Dec 2013 14:39:54 +0530	[thread overview]
Message-ID: <52A042E2.509@ti.com> (raw)
In-Reply-To: <1385451313-1875-1-git-send-email-sourav.poddar@ti.com>

Hi Brian,
On Tuesday 26 November 2013 01:04 PM, Sourav Poddar wrote:
> The patch series aims to add memory mapped support for TI qspi
> contoller and also add support for the same in serial flash driver(m25p80).
>
> These feature was added as a part of the series[1]:
> The series[1] add support for quad and memory mapped feature in spi and
> flash driver. Based on Brian Norris suggestion on splitting the quad and
> memory mapped feature, I had sent a seperate series for quad support which
> has beed accepted by Brian.
>
> The $subject patch series takes care of the memory mapped part.
>
> Overview of memory mapped support in qspi:
>   TI qspi controller supports memory mapped read support, switching to which
>   requires  register modifications(qspi registers, control module register for
>   DRA, qspi register for am43x). Once switched, the flash data can be available
>   at a soc specific memory mapped address. For operations in normal mode, we need
>   to keep memory mapped disable.
>
>   Flow of the patch is to keep the memory mapped enable by default in the driver.
>   Whenever Normal spi mode operations are desired, we switch to configuration mode
>   and at the end of it memory mapped is again enabled. There are few function pointers
>   added to fecilitate easy communication between the flash and spi controllers.
>
> Major changes in the $subject patch based on discussion in [1]:
>
> - Feature has been broken down into small meaningfl patches.
> - memcpy is done in flash driver and spi framework is bypassed.
> - configuration of memory mapped spi controller regsiters is done
>    through function calls from flash side.
>
> [1]: http://lists.infradead.org/pipermail/linux-mtd/2013-October/049058.html
>
>
> Tested on DRA7 board with spansion S25FL256S flash along with some
> internal irq crossbar patches.
> Tested on AM437x epos evm with macronix MX66l51235l flash.
>
> Testing done:
> -Erase the whole flash.
> -write to the flash
> -read back the flash
> - compare the data
>
> Sourav Poddar (17):
>    spi/spi.h: Add get_buf/put_buf support in spi master..
>    spi/spi.h: Add configure from slave support.
>    spi/qspi: parse register by name.
>    spi/qspi: parse control module register.
>    spi/qspi: parse memory map register.
>    spi/qspi: Add support to switc to memory mapped operation.
>    spi/qspi: configure set up register for memory map..
>    spi/qspi: Add api for get_buf/put_buf.
>    spi/spi.h: Add "mem" variable in spi master.
>    spi/spi.h: Add mmap flag in spi master.
>    spi/qspi: set "mmap" flag to true.
>    spi/qspi: convert driver to enable/disable memory mapped..
>    drivers: mtd: m25p80: Add api to configure master register .
>    drivers: mtd: m25p80: Adapt driver to support memory mapped read.
>    arm: dts: dra7: Add qspi device.
>    arm: dts: am43x-epos: Add qspi device.
>    Documentation: bindings: ti-qspi: update binding information.
>
>   Documentation/devicetree/bindings/spi/ti_qspi.txt |    2 +
>   arch/arm/boot/dts/am4372.dtsi                     |   12 ++
>   arch/arm/boot/dts/am43x-epos-evm.dts              |   28 +++++
>   arch/arm/boot/dts/dra7-evm.dts                    |   32 ++++++
>   arch/arm/boot/dts/dra7.dtsi                       |   13 +++
>   drivers/mtd/devices/m25p80.c                      |   30 ++++++
>   drivers/spi/spi-ti-qspi.c                         |  115 ++++++++++++++++++++-
>   include/linux/spi/spi.h                           |   16 +++
>   8 files changed, 246 insertions(+), 2 deletions(-)
>
Gentle ping on this series on mtd side...

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

      parent reply	other threads:[~2013-12-05  9:09 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-26  7:34 [PATCH 00/17] Add memory mapped support for ti qspi, m25p80 serial flash Sourav Poddar
2013-11-26  7:34 ` [PATCH 03/17] spi/qspi: parse register by name Sourav Poddar
2013-11-26 10:30   ` Mark Brown
     [not found]     ` <20131126103029.GY14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-26 11:17       ` Sourav Poddar
2013-11-26 12:24         ` Mark Brown
2013-11-28  4:45           ` Sourav Poddar
     [not found]             ` <5296CA5A.90304-l0cyMroinI0@public.gmane.org>
2013-11-28 10:54               ` Mark Brown
     [not found]                 ` <20131128105423.GL14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-28 10:58                   ` Sourav Poddar
     [not found]                     ` <529721BF.4010902-l0cyMroinI0@public.gmane.org>
2013-11-28 11:11                       ` Mark Brown
     [not found]                         ` <20131128111152.GN14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-28 11:19                           ` Sourav Poddar
2013-11-26  7:35 ` [PATCH 07/17] spi/qspi: configure set up register for memory map Sourav Poddar
     [not found] ` <1385451313-1875-1-git-send-email-sourav.poddar-l0cyMroinI0@public.gmane.org>
2013-11-26  7:34   ` [PATCH 01/17] spi/spi.h: Add get_buf/put_buf support in spi master Sourav Poddar
     [not found]     ` <1385451313-1875-2-git-send-email-sourav.poddar-l0cyMroinI0@public.gmane.org>
2013-11-26 12:38       ` Mark Brown
2013-11-26  7:34   ` [PATCH 02/17] spi/spi.h: Add configure from slave support Sourav Poddar
     [not found]     ` <1385451313-1875-3-git-send-email-sourav.poddar-l0cyMroinI0@public.gmane.org>
2013-11-26 10:29       ` Mark Brown
     [not found]         ` <20131126102951.GX14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-26 11:21           ` Sourav Poddar
     [not found]             ` <52948451.3060706-l0cyMroinI0@public.gmane.org>
2013-11-26 12:21               ` Mark Brown
2013-11-28  4:47                 ` Sourav Poddar
     [not found]                   ` <5296CAEA.3030002-l0cyMroinI0@public.gmane.org>
2013-11-28 10:55                     ` Mark Brown
2013-11-26  7:35   ` [PATCH 04/17] spi/qspi: parse control module register Sourav Poddar
2013-11-26 10:31     ` Mark Brown
     [not found]       ` <20131126103104.GZ14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-26 11:18         ` Sourav Poddar
     [not found]           ` <529483A2.3080000-l0cyMroinI0@public.gmane.org>
2013-11-26 12:25             ` Mark Brown
     [not found]               ` <20131126122512.GD14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-26 12:38                 ` Sourav Poddar
2013-11-26 12:55                   ` Mark Brown
2013-11-26 13:00                     ` Sourav Poddar
2013-11-26 13:19                       ` Mark Brown
     [not found]                         ` <20131126131927.GN14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-26 13:20                           ` Sourav Poddar
2013-11-26  7:35   ` [PATCH 05/17] spi/qspi: parse memory map register Sourav Poddar
2013-11-26  7:35   ` [PATCH 06/17] spi/qspi: Add support to switc to memory mapped operation Sourav Poddar
2013-11-26 12:57     ` Mark Brown
2013-11-28  4:39       ` Sourav Poddar
2013-11-26  7:35   ` [PATCH 08/17] spi/qspi: Add api for get_buf/put_buf Sourav Poddar
2013-11-26  7:35   ` [PATCH 09/17] spi/spi.h: Add "mem" variable in spi master Sourav Poddar
2013-11-26 13:01     ` Mark Brown
     [not found]       ` <20131126130119.GH14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-26 13:05         ` Sourav Poddar
     [not found]           ` <52949C9F.10008-l0cyMroinI0@public.gmane.org>
2013-11-26 13:20             ` Mark Brown
     [not found]               ` <20131126132051.GO14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-28  4:38                 ` Sourav Poddar
2013-11-26  7:35   ` [PATCH 12/17] spi/qspi: convert driver to enable/disable memory mapped Sourav Poddar
2013-11-26 13:11     ` Mark Brown
2013-11-28  5:24       ` Sourav Poddar
2013-11-28 10:49         ` Mark Brown
     [not found]           ` <20131128104922.GK14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-28 11:08             ` Sourav Poddar
     [not found]               ` <5297242B.3040807-l0cyMroinI0@public.gmane.org>
2013-11-28 11:59                 ` Mark Brown
2013-11-28 12:02                   ` Sourav Poddar
2013-11-26  7:35   ` [PATCH 13/17] drivers: mtd: m25p80: Add api to configure master register Sourav Poddar
2013-11-26  7:35   ` [PATCH 14/17] drivers: mtd: m25p80: Adapt driver to support memory mapped read Sourav Poddar
2013-11-26  7:35   ` [PATCH 15/17] arm: dts: dra7: Add qspi device Sourav Poddar
2013-11-26  7:35   ` [PATCH 17/17] Documentation: bindings: ti-qspi: update binding information Sourav Poddar
2013-11-26  7:35 ` [PATCH 10/17] spi/spi.h: Add mmap flag in spi master Sourav Poddar
2013-11-26  7:35 ` [PATCH 11/17] spi/qspi: set "mmap" flag to true Sourav Poddar
     [not found]   ` <1385451313-1875-12-git-send-email-sourav.poddar-l0cyMroinI0@public.gmane.org>
2013-11-26 13:10     ` Mark Brown
     [not found]       ` <20131126131029.GI14725-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-11-26 13:11         ` Sourav Poddar
2013-11-26  7:35 ` [PATCH 16/17] arm: dts: am43x-epos: Add qspi device Sourav Poddar
2013-12-05  9:09 ` Sourav Poddar [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=52A042E2.509@ti.com \
    --to=sourav.poddar@ti.com \
    --cc=balbi@ti.com \
    --cc=bcousson@baylibre.com \
    --cc=broonie@kernel.org \
    --cc=computersforpeace@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-spi@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).