Re: [PATCH v5 00/11] memory: add Atmel EBI (External Bus Interface) driver

From: Alexandre Belloni
Date: Sun Jan 18 2015 - 19:40:27 EST


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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/