Hi Marri,

On Friday 23 July 2010 02:57:18 tmarri@amcc.com wrote:

From: Tirumala Marri <redacted>

  This patch will add ADMA support for DMA engine and HW offload for
  XOR/ADG (RAID-5/6) functionalities.
  1. It supports memcpy, xor, GF(2) based RAID-6.
  2. It supports interrupt based DMA completions.
  3. Also supports memcpy in RAID-1 case.

  Kernel version: 2.6.35-rc5

  Testing:
    Created RAID-5/6 arrays usign mdadm.
    And ran raw IO and filesystem IO to the RAID array.
    Chunk size 4k,64k was tested.
    RAID rebuild , disk fail, resync tested.

  File names:
    This code is similar to ppc440spe . So I named the files as
    drivers/dma/ppc4xx/adma1.c and drivers/dma/ppc4xx/adma1.h

As you describe above, a lot of the code seems to be copied from 
drivers/dma/ppc4xx/adma.c/h. Wouldn't it make more sense to factor out the 
common code instead of duplicating it?
 
Thanks.

Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office@denx.de