Date:	Mon, 19 Jan 2015 01:40:15 +0100
From:	Alexandre Belloni <alexandre.belloni@...e-electrons.com>
To:	Lee Jones <lee.jones@...aro.org>
Cc:	Boris Brezillon <boris.brezillon@...e-electrons.com>,
	Nicolas Ferre <nicolas.ferre@...el.com>,
	Jean-Christophe Plagniol-Villard <plagnioj@...osoft.com>,
	Andrew Victor <linux@...im.org.za>,
	Samuel Ortiz <sameo@...ux.intel.com>,
	Rob Herring <robh+dt@...nel.org>,
	Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kumar Gala <galak@...eaurora.org>, devicetree@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
	Arnd Bergmann <arnd@...db.de>,
	Jean-Jacques Hiblot <jjhiblot@...phandler.com>
Subject: Re: [PATCH v5 00/11] memory: add Atmel EBI (External Bus Interface)
 driver

Hi Lee,

On 18/01/2015 at 12:51:17 +0000, Lee Jones wrote :
> Sorry for the delay, I have been on vacation.
> 
> I'm not sure why why those patches are preventing the application of
> the remainder of the set.  They are only documentation entries and I
> have Acked them already?
> 

We had some series depending on the new syscon headers introduced by
those patches.

Indeed, at the time I wrote that email, I forgot you acked them. As
Felipe Balbi wanted to ensure that the dependency would be handled
correctly, we asked Nicolas to take those patches (as I explained in one
of my mails).

> > On 03/12/2014 at 21:44:04 +0100, Boris Brezillon wrote :
> > > The EBI (External Bus Interface) is used to access external peripherals
> > > (NOR, SRAM, NAND, and other specific devices like ethernet controllers).
> > > Each device is assigned a CS line and an address range and can have its
> > > own configuration (timings, access mode, bus width, ...).
> > > This driver provides a generic DT binding to configure a device according
> > > to its requirements.
> > > For specific device controllers (like the NAND one) the SMC timings
> > > should be configured by the controller driver through the matrix and
> > > smc syscon regmaps.
> > > 
> > > The first 4 patches introduce 2 syscon devices needed to configure the
> > > EBI bus, patch 5 and 6 adds support for the EBI bus, and the remaining
> > > patches declares the EBI related nodes in sama5d3 dts[i] files.
> > > 
> > > Changes since v4:
> > > - fix inconsistencies in SMC and MATRIX registers definition
> > > - add missing compatible strings for at91sam9rl SoC
> > > - fix DT bindings documentation
> > > - replace "atmel,generic-dev" property by "atmel,specialized-logic"
> > > 
> > > Changes since v3:
> > > - added AT91_MATRIX_USBPUCR_PUON definition
> > > - removed useless macros (those directly returning SoC specific register
> > >   offsets)
> > > - use syscon_regmap_lookup_by_phandle instead of of_parse_phandle +
> > >   syscon_node_to_regmap
> > > - drop AT91_EBICSA_REGFIELD and AT91_MULTI_EBICSA_REGFIELD macros
> > > 
> > > Changes since v2:
> > > - minor fixes int DT bindings doc
> > > - fix SMC macros
> > > - make use of SMC macros defined in include/linux/mfd/syscon/atmel-smc.h
> > > 
> > > Changes since v1:
> > > - almost everything :-)
> > > 
> > > Boris Brezillon (11):
> > >   mfd: syscon: Add atmel-matrix registers definition
> > >   mfd: syscon: Add Atmel Matrix bus DT binding documentation
> > >   mfd: syscon: Add atmel-smc registers definition
> > >   mfd: syscon: Add Atmel SMC binding doc
> > >   memory: add Atmel EBI (External Bus Interface) driver
> > >   memory: atmel-ebi: add DT bindings documentation
> > >   ARM: at91: select ATMEL_EBI when compiling a kernel for at91sam9 or
> > >     sama5d3
> > >   ARM: at91/dt: add HSMC (Static Memory Controller) node in sama5d3 dtsi
> > >   ARM: at91/dt: add matrix node in sama5d3 dtsi
> > >   ARM: at91/dt: add EBI (External Bus Interface) node in sama5d3 dtsi
> > >   ARM: at91/dt: add NOR definition in sama5d3xcm dtsi
> > > 
> > >  .../bindings/memory-controllers/atmel-ebi.txt      | 155 ++++++
> > >  .../devicetree/bindings/mfd/atmel-matrix.txt       |  24 +
> > >  .../devicetree/bindings/mfd/atmel-smc.txt          |  19 +
> > >  arch/arm/boot/dts/sama5d3.dtsi                     |  93 ++++
> > >  arch/arm/boot/dts/sama5d3xcm.dtsi                  |  33 ++
> > >  arch/arm/mach-at91/Kconfig                         |   2 +
> > >  drivers/memory/Kconfig                             |  11 +
> > >  drivers/memory/Makefile                            |   1 +
> > >  drivers/memory/atmel-ebi.c                         | 579 +++++++++++++++++++++
> > >  include/linux/mfd/syscon/atmel-matrix.h            | 117 +++++
> > >  include/linux/mfd/syscon/atmel-smc.h               | 173 ++++++
> > >  11 files changed, 1207 insertions(+)
> > >  create mode 100644 Documentation/devicetree/bindings/memory-controllers/atmel-ebi.txt
> > >  create mode 100644 Documentation/devicetree/bindings/mfd/atmel-matrix.txt
> > >  create mode 100644 Documentation/devicetree/bindings/mfd/atmel-smc.txt
> > >  create mode 100644 drivers/memory/atmel-ebi.c
> > >  create mode 100644 include/linux/mfd/syscon/atmel-matrix.h
> > >  create mode 100644 include/linux/mfd/syscon/atmel-smc.h
> > > 
> > 
> 
> -- 
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists