[PATCH v3 0/3] Add DMA and device tree support to the flash controller FLCTL

Bastian Hecht hechtb at gmail.com
Wed Nov 7 06:57:35 EST 2012


Hello Artem,

I hope all aiaiai warnings except the adress space issue are fixed. As
I don't see how to fix the address space conversion I wonder if you
can accept the patchset as it is now.

cheers,

 Bastian


2012/10/19 Bastian Hecht <hechtb at googlemail.com>:
> changelog v3:   - due to shame on me by letting aiaiai complain, I've added
>                   a third patch (1/3) that cleans up some parts of the driver
>                   and added a correct error path to probe() in patch 3 (formerly 2/2).
>                   I found no way to fix this though:
> +drivers/mtd/nand/sh_flctl.c:181:25: warning: cast removes address space of expression [sparse]
>
> changelog v2:   - cosmetic fixes in patch 1
>                 - added dmas and dma-names field in patch 2
>
> This mini-series consists of 2 separately posted patch-sets that shrunk
> to one patch each. It adds DMA support for the data part (not ECC) as
> well as device tree support to the FLCTL flash controller.
> As the 2nd patch is based on the 1st, I've decided to
> collect them here to avoid any confusion.
>
> The first set contained
> [PATCH 1/2] mtd: sh_flctl: Setup and release DMA channels
> [PATCH 2/2] mtd: sh_flctl: Use DMA for data fifo FLTDFIFO when available
> and was merged after 2 reviews to
> [PATCH] mtd: sh_flctl: Add DMA capabilty (same patch as in here)
>
> The second set contained
> [PATCH 1/3] mtd: sh_flctl: Probe SNAND_E flag from NAND chip
> [PATCH 2/3] mtd: sh_flctl: Add device tree support
> [PATCH 3/3] mtd: sh_flctl: Add sh7372 device tree config
> and is merged here to
> mtd: sh_flctl: Add device tree support
>
> Patch 1 can be omitted as SNAND_E is handled in the FLCTL source anyway
> correctly in set_cmd_regs() and the flag can be removed from existing board
> codes without this patch.
> Patch 2 without patch 3 may be confusing so I merged them too. Documentation
> is added as well.
>
> I've added linux-arm-kernel at lists.infradead.org as recipient for the
> device tree part.
>
> The patchset is based on
> l2-mtd  git://git.infradead.org/users/dedekind/l2-mtd.git
> with reverted commit e26c113b4130aefa1d8446602bb5b05cfd646bfe.
>
>
> Bastian Hecht (2):
>   mtd: sh_flctl: Add DMA capabilty
>   mtd: sh_flctl: Add device tree support
>
>  .../devicetree/bindings/mtd/flctl-nand.txt         |   37 +++
>  drivers/mtd/nand/sh_flctl.c                        |  266 +++++++++++++++++++-
>  include/linux/mtd/sh_flctl.h                       |   12 +
>  3 files changed, 305 insertions(+), 10 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/mtd/flctl-nand.txt
>
> --
> 1.7.5.4
>



More information about the linux-arm-kernel mailing list