Re: [PATCH 1/2] PPC4xx: Generelizing drivers/dma/ppc4xx/adma.c

On 9/17/2010 6:42 PM, tmarri@apm.com wrote:

From: Tirumala Marri<redacted>

This patch generalizes the existing drver/dma/ppc4xx/adma.c, so that
common code can be shared between different similar DMA engine
drivers in other SoCs.

Signed-off-by: Tirumala R Marri<redacted>
---
  drivers/dma/ppc4xx/adma.c        | 4370 +++-----------------------------------
  drivers/dma/ppc4xx/adma.h        |  116 +-
  drivers/dma/ppc4xx/ppc4xx-adma.h | 4020 +++++++++++++++++++++++++++++++++++
  3 files changed, 4357 insertions(+), 4149 deletions(-)
  create mode 100644 drivers/dma/ppc4xx/ppc4xx-adma.h

Will both versions of this driver exist in the same kernel build?  For 
example the iop-adma driver supports iop13xx and iop3xx, but we select 
the archtitecture at build time?  Or, as I assume in this case, will the 
two (maybe more?) ppc4xx adma drivers all be built in the same image, 
more like ioatdma?

In the latter case I would recommend a file structure like:

drivers/dma/ppc4xx/adma.c
drivers/dma/ppc4xx/adma_440spe.c
drivers/dma/ppc4xx/adma_460ex.c

With patches to move the chipset specific pieces to their own file. 
Minimizing the code churn in adma.c, or at least showing a progression 
of what is unique and needs to be moved.

This would be similar to how ioatdma is structured and compiles a single 
driver to cover the three major hardware revisions.

--
Dan