Inter-revision diff: cover letter

Comparing v11 (message) to v8 (message)

--- v11
+++ v8
@@ -82,37 +82,6 @@
  Part 6:
  - remove since it has become obsolete;
 
-Changes in v9:
- A new patch (part 3/6) is added to this series according the
-   feedback of Andy Shevchenko.
- Part 2/6:
- - keep style of the comments;
- - use is_slave_direction() instead of manual checks;
- - remove redundant else branches of the conditions;
- - make mpc_dma_device_control() return -ENXIO for unknown command;
- Part 6/6:
- - change according the new part 3/6;
- - fix style issues;
-
-Changes in v10:
- Part 2/6:
- - don't use direction field of dma_slave_config in mpc_dma_device_control()
-    but store settings in mpc_dma_chan for both DMA_DEV_TO_MEM and
-    DMA_MEM_TO_DEV cases; then retrieve the needed values in
-    mpc_dma_prep_slave_sg();
- - fix style issue and put 2014 instead of 2013;
- Part 3/6:
- - fix mpc_dma_probe() error path and mpc_dma_remove(): manually free IRQs and
-    dispose IRQ mappings before devm_* takes care of other resources;
- Part 6/6:
- - change according the new part 3/6;
- - fix style issue;
-
-Changes in v11:
- Part 5/6:
- - remake device tree binding document according the recommendations of
-    Gerhard Sittig, Mark Rutland and Arnd Bergmann;
-
 > known issues:
 > - it's yet to get confirmed whether MPC8308 can use slave support or
 >    whether the DMA controller's driver shall actively reject it, the
@@ -122,23 +91,32 @@
     do increment the peripheral "port's" address is easy with
     this implementation; but which options of the common API
     should be used for specifying such transfers?
+2014/02/13 Gerhard Sittig <gsi@denx.de>:
+> - The MPC512x DMA completely lacks a binding document, so one
+>    should get added.
+> - The MPC8308 hardware is similar and can re-use the MPC512x
+>    binding, which should be stated.
+> - The Linux implementation currently has no OF based channel
+>    lookup support, so '#dma-cells' is "a future feature".  I guess
+>    the binding can and should already discuss the feature,
+>    regardless of whether all implementations support it.
 
 
-Alexander Popov (6):
+Alexander Popov (3):
   dma: mpc512x: reorder mpc8308 specific instructions
   dma: mpc512x: add support for peripheral transfers
-  dma: mpc512x: fix freeing resources in mpc_dma_probe() and
-    mpc_dma_remove()
   dma: of: Add common xlate function for matching by channel id
+
+Gerhard Sittig (2):
   dma: mpc512x: add device tree binding document
   dma: mpc512x: register for device tree channel lookup
 
- .../devicetree/bindings/dma/mpc512x-dma.txt        |  51 +++
+ .../devicetree/bindings/dma/mpc512x-dma.txt        |  55 ++++
  arch/powerpc/boot/dts/mpc5121.dtsi                 |   1 +
- drivers/dma/mpc512x_dma.c                          | 345 ++++++++++++++++++---
+ drivers/dma/mpc512x_dma.c                          | 298 +++++++++++++++++++--
  drivers/dma/of-dma.c                               |  35 +++
  include/linux/of_dma.h                             |   4 +
- 5 files changed, 398 insertions(+), 38 deletions(-)
+ 5 files changed, 368 insertions(+), 25 deletions(-)
  create mode 100644 Documentation/devicetree/bindings/dma/mpc512x-dma.txt
 
 -- 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help