From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 05/11] memory: add Atmel EBI (External Bus Interface) driver
Date: Tue, 02 Dec 2014 10:41:57 +0100 [thread overview]
Message-ID: <3316464.yrAZ5QMKbg@wuerfel> (raw)
In-Reply-To: <1724175.Jqk8gQW3pz@ws-stein>
On Tuesday 02 December 2014 10:18:45 Alexander Stein wrote:
> On Monday 01 December 2014 17:17:30, Arnd Bergmann wrote:
> > On Monday 01 December 2014 11:50:06 Boris Brezillon wrote:
> > >
> > > I don't have a strong opinion regarding where this driver should
> > > live (I even considered putting it in drivers/bus) :-).
> > > But note that there are other "external memory interface" drivers
> > > in drivers/memory too: TI AEMIF and Marvell DEVBUS ;-).
> > >
> > > Does anyone else think this driver should go in drivers/bus ?
> > >
> >
> > I think drivers/memory is better.
>
> I don't mind if it's drivers/bus or drivers/memory. But I'm
> wondering: When is a driver a bus driver when it is a memory
> driver? See imx-weim and atmel-ebi. Apparently both add support
> for devices attached on a bus which can be accessed through memory
> addresses.
I think drivers/memory should be used for external buses with an
SRAM or DRAM interface that need to configure the timings.
drivers/bus is more a collection for random other buses, usually
internal to a chip, but it's less well-defined.
Arnd
WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
To: Alexander Stein
<alexander.stein-93q1YBGzJSMe9JSWTWOYM3xStJ4P+DSV@public.gmane.org>
Cc: Boris Brezillon
<boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
Nicolas Ferre
<nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>,
Jean-Christophe Plagniol-Villard
<plagnioj-sclMFOaUSTBWk0Htik3J/w@public.gmane.org>,
Alexandre Belloni
<alexandre.belloni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
Andrew Victor <linux-PelNFVqkFnVyf+4FbqDuWQ@public.gmane.org>,
Samuel Ortiz <sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>,
Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Ian Campbell
<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Jean-Jacques Hiblot
<jjhiblot-dLKeG7h1OhBDOHtkgc7UlQ@public.gmane.org>
Subject: Re: [PATCH v3 05/11] memory: add Atmel EBI (External Bus Interface) driver
Date: Tue, 02 Dec 2014 10:41:57 +0100 [thread overview]
Message-ID: <3316464.yrAZ5QMKbg@wuerfel> (raw)
In-Reply-To: <1724175.Jqk8gQW3pz@ws-stein>
On Tuesday 02 December 2014 10:18:45 Alexander Stein wrote:
> On Monday 01 December 2014 17:17:30, Arnd Bergmann wrote:
> > On Monday 01 December 2014 11:50:06 Boris Brezillon wrote:
> > >
> > > I don't have a strong opinion regarding where this driver should
> > > live (I even considered putting it in drivers/bus) :-).
> > > But note that there are other "external memory interface" drivers
> > > in drivers/memory too: TI AEMIF and Marvell DEVBUS ;-).
> > >
> > > Does anyone else think this driver should go in drivers/bus ?
> > >
> >
> > I think drivers/memory is better.
>
> I don't mind if it's drivers/bus or drivers/memory. But I'm
> wondering: When is a driver a bus driver when it is a memory
> driver? See imx-weim and atmel-ebi. Apparently both add support
> for devices attached on a bus which can be accessed through memory
> addresses.
I think drivers/memory should be used for external buses with an
SRAM or DRAM interface that need to configure the timings.
drivers/bus is more a collection for random other buses, usually
internal to a chip, but it's less well-defined.
Arnd
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd@arndb.de>
To: Alexander Stein <alexander.stein@systec-electronic.com>
Cc: Boris Brezillon <boris.brezillon@free-electrons.com>,
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>,
Samuel Ortiz <sameo@linux.intel.com>,
Lee Jones <lee.jones@linaro.org>,
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@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
Jean-Jacques Hiblot <jjhiblot@traphandler.com>
Subject: Re: [PATCH v3 05/11] memory: add Atmel EBI (External Bus Interface) driver
Date: Tue, 02 Dec 2014 10:41:57 +0100 [thread overview]
Message-ID: <3316464.yrAZ5QMKbg@wuerfel> (raw)
In-Reply-To: <1724175.Jqk8gQW3pz@ws-stein>
On Tuesday 02 December 2014 10:18:45 Alexander Stein wrote:
> On Monday 01 December 2014 17:17:30, Arnd Bergmann wrote:
> > On Monday 01 December 2014 11:50:06 Boris Brezillon wrote:
> > >
> > > I don't have a strong opinion regarding where this driver should
> > > live (I even considered putting it in drivers/bus) :-).
> > > But note that there are other "external memory interface" drivers
> > > in drivers/memory too: TI AEMIF and Marvell DEVBUS ;-).
> > >
> > > Does anyone else think this driver should go in drivers/bus ?
> > >
> >
> > I think drivers/memory is better.
>
> I don't mind if it's drivers/bus or drivers/memory. But I'm
> wondering: When is a driver a bus driver when it is a memory
> driver? See imx-weim and atmel-ebi. Apparently both add support
> for devices attached on a bus which can be accessed through memory
> addresses.
I think drivers/memory should be used for external buses with an
SRAM or DRAM interface that need to configure the timings.
drivers/bus is more a collection for random other buses, usually
internal to a chip, but it's less well-defined.
Arnd
next prev parent reply other threads:[~2014-12-02 9:41 UTC|newest]
Thread overview: 103+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-01 10:27 [PATCH v3 00/11] memory: add Atmel EBI (External Bus Interface) driver Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` [PATCH v3 01/11] mfd: syscon: Add atmel-matrix registers definition Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-03 15:57 ` Nicolas Ferre
2014-12-03 15:57 ` Nicolas Ferre
2014-12-03 15:57 ` Nicolas Ferre
2014-12-01 10:27 ` [PATCH v3 02/11] mfd: syscon: Add Atmel Matrix bus DT binding documentation Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-03 14:32 ` Nicolas Ferre
2014-12-03 14:32 ` Nicolas Ferre
2014-12-03 14:32 ` Nicolas Ferre
2014-12-03 14:52 ` Boris Brezillon
2014-12-03 14:52 ` Boris Brezillon
2014-12-03 14:52 ` Boris Brezillon
2014-12-01 10:27 ` [PATCH v3 03/11] mfd: syscon: Add atmel-smc registers definition Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` [PATCH v3 04/11] mfd: syscon: Add Atmel SMC binding doc Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` [PATCH v3 05/11] memory: add Atmel EBI (External Bus Interface) driver Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:40 ` Alexander Stein
2014-12-01 10:40 ` Alexander Stein
2014-12-01 10:40 ` Alexander Stein
2014-12-01 10:50 ` Boris Brezillon
2014-12-01 10:50 ` Boris Brezillon
2014-12-01 10:50 ` Boris Brezillon
2014-12-01 16:17 ` Arnd Bergmann
2014-12-01 16:17 ` Arnd Bergmann
2014-12-02 9:18 ` Alexander Stein
2014-12-02 9:18 ` Alexander Stein
2014-12-02 9:41 ` Arnd Bergmann [this message]
2014-12-02 9:41 ` Arnd Bergmann
2014-12-02 9:41 ` Arnd Bergmann
2014-12-01 16:26 ` Arnd Bergmann
2014-12-01 16:26 ` Arnd Bergmann
2014-12-01 16:26 ` Arnd Bergmann
2014-12-01 18:29 ` Boris Brezillon
2014-12-01 18:29 ` Boris Brezillon
2014-12-01 18:29 ` Boris Brezillon
2014-12-01 19:43 ` Arnd Bergmann
2014-12-01 19:43 ` Arnd Bergmann
2014-12-01 19:43 ` Arnd Bergmann
2014-12-01 20:28 ` Boris Brezillon
2014-12-01 20:28 ` Boris Brezillon
2014-12-01 20:28 ` Boris Brezillon
2014-12-01 21:28 ` Arnd Bergmann
2014-12-01 21:28 ` Arnd Bergmann
2014-12-09 20:53 ` Alexander Stein
2014-12-09 20:53 ` Alexander Stein
2014-12-09 20:53 ` Alexander Stein
2014-12-15 10:22 ` Boris Brezillon
2014-12-15 10:22 ` Boris Brezillon
2014-12-15 10:22 ` Boris Brezillon
2014-12-15 10:29 ` Alexander Stein
2014-12-15 10:29 ` Alexander Stein
2014-12-15 10:29 ` Alexander Stein
2014-12-15 10:43 ` Boris Brezillon
2014-12-15 10:43 ` Boris Brezillon
2014-12-15 10:43 ` Boris Brezillon
2014-12-01 10:27 ` [PATCH v3 06/11] memory: atmel-ebi: add DT bindings documentation Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-03 14:56 ` Nicolas Ferre
2014-12-03 14:56 ` Nicolas Ferre
2014-12-03 14:56 ` Nicolas Ferre
2014-12-03 15:15 ` Nicolas Ferre
2014-12-03 15:15 ` Nicolas Ferre
2014-12-03 15:15 ` Nicolas Ferre
2014-12-03 15:34 ` Boris Brezillon
2014-12-03 15:34 ` Boris Brezillon
2014-12-03 15:34 ` Boris Brezillon
2014-12-03 15:38 ` Boris Brezillon
2014-12-03 15:38 ` Boris Brezillon
2014-12-03 15:38 ` Boris Brezillon
2014-12-03 19:28 ` Boris Brezillon
2014-12-03 19:28 ` Boris Brezillon
2014-12-03 19:28 ` Boris Brezillon
2014-12-01 10:27 ` [PATCH v3 07/11] ARM: at91: select ATMEL_EBI when compiling a kernel for at91sam9 or sama5d3 Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-03 16:21 ` Nicolas Ferre
2014-12-03 16:21 ` Nicolas Ferre
2014-12-03 16:21 ` Nicolas Ferre
2014-12-01 10:27 ` [PATCH v3 08/11] ARM: at91/dt: add HSMC (Static Memory Controller) node in sama5d3 dtsi Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` [PATCH v3 09/11] ARM: at91/dt: add matrix " Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` [PATCH v3 10/11] ARM: at91/dt: add EBI (External Bus Interface) " Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-01 10:27 ` [PATCH v3 11/11] ARM: at91/dt: add NOR definition in sama5d3xcm dtsi Boris Brezillon
2014-12-01 10:27 ` Boris Brezillon
2014-12-02 8:47 ` [PATCH v4 05/11] memory: add Atmel EBI (External Bus Interface) driver Boris Brezillon
2014-12-02 8:47 ` Boris Brezillon
2014-12-02 8:47 ` Boris Brezillon
2014-12-03 15:24 ` Nicolas Ferre
2014-12-03 15:24 ` Nicolas Ferre
2014-12-03 15:24 ` Nicolas Ferre
2014-12-03 11:14 ` [PATCH v4 01/11] mfd: syscon: Add atmel-matrix registers definition Boris Brezillon
2014-12-03 11:14 ` Boris Brezillon
2014-12-03 11:14 ` 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=3316464.yrAZ5QMKbg@wuerfel \
--to=arnd@arndb.de \
--cc=linux-arm-kernel@lists.infradead.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.