devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Boris Brezillon <boris.brezillon@free-electrons.com>
To: Jean-Jacques Hiblot <jjhiblot@traphandler.com>
Cc: Samuel Ortiz <sameo@linux.intel.com>,
	Lee Jones <lee.jones@linaro.org>,
	Nicolas Ferre <nicolas.ferre@atmel.com>,
	Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>,
	Alexandre Belloni <alexandre.belloni@free-electrons.com>,
	Andrew Victor <linux@maxim.org.za>,
	Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Kumar Gala <galak@codeaurora.org>,
	devicetree <devicetree@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 06/11] memory: atmel-ebi: add DT bindings documentation
Date: Wed, 5 Nov 2014 17:39:57 +0100	[thread overview]
Message-ID: <20141105173957.1726d369@bbrezillon> (raw)
In-Reply-To: <CACh+v5N8dXaH3ZNGr=MKXbQRU7nOMPLvxT0rEsZrbSKj20b_Dg@mail.gmail.com>

On Wed, 5 Nov 2014 17:22:57 +0100
Jean-Jacques Hiblot <jjhiblot@traphandler.com> wrote:

> Hi Boris,
> 
> 2014-11-05 17:01 GMT+01:00 Boris Brezillon <boris.brezillon@free-electrons.com>:
> > Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> > ---
> >  .../bindings/memory-controllers/atmel-ebi.txt      | 153 +++++++++++++++++++++
> >  1 file changed, 153 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/memory-controllers/atmel-ebi.txt
> >
> > diff --git a/Documentation/devicetree/bindings/memory-controllers/atmel-ebi.txt b/Documentation/devicetree/bindings/memory-controllers/atmel-ebi.txt
> > new file mode 100644
> > index 0000000..dc2c34f
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/memory-controllers/atmel-ebi.txt
> > @@ -0,0 +1,153 @@
> > +* Device tree bindings for Atmel EBI
> > +
> > +The External Bus Interface (EBI) controller is a bus where you can connect
> > +asynchronous (NAND, NOR, SRAM, ....) and synchronous memories (SDR/DDR SDRAMs).
> > +The EBI provides a glue-less interface to asynchronous memories though the SMC
> > +(Static Memory Controller).
> > +Synchronous memories (and some asynchronous memories like NANDs) can be
> > +attached to specialized controllers which are responsible for configuring the
> > +bus appropriately according to the connected device.
> > +In the other hand, the bus interface can be automated for simple asynchronous
> > +devices.

[...]

> > +
> > +Optional child cs node properties:
> > +- atmel,generic-dev            boolean property specifying if the device is
> > +                               a generic device.
> > +                               The following properties are only parsed if
> > +                               this property is present.
> > +                               Specialized devices are attached to specialized
> > +                               controllers which should configure the bus
> > +                               appropriately.
> 
> What do you mean by specialized devices ? Can you give an example ?

The ones I have in mind are NAND chips: the NAND controller can
automatically discover the required timings and configure the SMC
accordingly. In that case there's no need to specify timings in the DT,
because they will/should be dynalically configured by the NAND
controller driver.

But more generally, the datasheet describe 2 modes for some CS ports:
1) the generic mode, which bind the device to the generic SMC engine
2) the specialized mode which binds it to a specific HW block

This mode can be configured in CCFG_EBICSA, and the specialized mode
supported by each CS depends on each SoC.
For example, for the at91sam9x5:
CS1 => DDR2SDR controller
CS3 => NAND Flash Controller/Logic

-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

  reply	other threads:[~2014-11-05 16:39 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-05 16:01 [PATCH v2 00/11] memory: add Atmel EBI (External Bus Interface) driver Boris Brezillon
2014-11-05 16:01 ` [PATCH v2 02/11] mfd: syscon: Add Atmel Matrix bus DT binding documentation Boris Brezillon
2014-11-10  9:55   ` Lee Jones
2014-11-05 16:01 ` [PATCH v2 04/11] mfd: syscon: Add Atmel SMC binding doc Boris Brezillon
2014-11-10  9:55   ` Lee Jones
2014-11-05 16:01 ` [PATCH v2 05/11] memory: add Atmel EBI (External Bus Interface) driver Boris Brezillon
2014-11-05 16:53   ` Jean-Jacques Hiblot
2014-11-05 16:59     ` Boris Brezillon
2014-11-05 17:05       ` Jean-Jacques Hiblot
2014-11-05 16:01 ` [PATCH v2 06/11] memory: atmel-ebi: add DT bindings documentation Boris Brezillon
2014-11-05 16:22   ` Jean-Jacques Hiblot
2014-11-05 16:39     ` Boris Brezillon [this message]
     [not found]   ` <1415203287-21517-7-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-11-07 13:55     ` Alexandre Belloni
2014-11-07 15:21   ` Rob Herring
2014-11-07 15:49     ` Boris Brezillon
2014-11-07 16:19       ` Tomeu Vizoso
2014-11-07 19:39       ` Rob Herring
2014-11-05 16:01 ` [PATCH v2 07/11] ARM: at91: select ATMEL_EBI when compiling a kernel for at91sam9 or sama5d3 Boris Brezillon
     [not found] ` <1415203287-21517-1-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-11-05 16:01   ` [PATCH v2 01/11] mfd: syscon: Add atmel-matrix registers definition Boris Brezillon
     [not found]     ` <1415203287-21517-2-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-11-07 13:45       ` Alexandre Belloni
     [not found]         ` <20141107134510.GC4068-m++hUPXGwpdeoWH0uzbU5w@public.gmane.org>
2014-11-07 14:14           ` Boris Brezillon
2014-11-10  9:47     ` Lee Jones
2014-11-05 16:01   ` [PATCH v2 03/11] mfd: syscon: Add atmel-smc " Boris Brezillon
     [not found]     ` <1415203287-21517-4-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-11-10  9:57       ` Lee Jones
2014-12-01  9:38         ` Boris Brezillon
2014-12-01  9:43           ` Nicolas Ferre
2014-11-05 16:01   ` [PATCH v2 08/11] ARM: at91/dt: add HSMC (Static Memory Controller) node in sama5d3 dtsi Boris Brezillon
2014-11-05 16:01   ` [PATCH v2 09/11] ARM: at91/dt: add matrix " Boris Brezillon
2014-11-05 16:01   ` [PATCH v2 10/11] ARM: at91/dt: add EBI (External Bus Interface) " Boris Brezillon
2014-11-05 16:01   ` [PATCH v2 11/11] ARM: at91/dt: add NOR definition in sama5d3xcm dtsi Boris Brezillon
2014-11-07 13:41   ` [PATCH v2 00/11] memory: add Atmel EBI (External Bus Interface) driver Alexandre Belloni
     [not found]     ` <20141107134112.GB4068-m++hUPXGwpdeoWH0uzbU5w@public.gmane.org>
2014-11-07 14:19       ` Boris Brezillon

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=20141105173957.1726d369@bbrezillon \
    --to=boris.brezillon@free-electrons.com \
    --cc=alexandre.belloni@free-electrons.com \
    --cc=devicetree@vger.kernel.org \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=jjhiblot@traphandler.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@maxim.org.za \
    --cc=mark.rutland@arm.com \
    --cc=nicolas.ferre@atmel.com \
    --cc=pawel.moll@arm.com \
    --cc=plagnioj@jcrosoft.com \
    --cc=robh+dt@kernel.org \
    --cc=sameo@linux.intel.com \
    /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).