[PATCH 1/2] PPC4xx: Generelizing drivers/dma/ppc4xx/adma.c
dan.j.williams at intel.com
Fri Sep 24 06:22:03 EST 2010
On 9/17/2010 6:42 PM, tmarri at apm.com wrote:
> From: Tirumala Marri<tmarri at apm.com>
> 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<tmarri at apm.com>
> 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:
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.
More information about the Linuxppc-dev