[PATCH v2 1/2] DMA: PL330: Merge PL330 driver into drivers/dma/

Russell King - ARM Linux linux at arm.linux.org.uk
Thu Jan 5 09:54:25 EST 2012


On Mon, Dec 12, 2011 at 10:51:08AM +0900, Boojin Kim wrote:
> Currently there were two part of DMAC PL330 driver for
> support old styled s3c-pl330 which has been merged into
> drivers/dma/pl330.c driver. Actually, there is no reason
> to separate them now.
> 
> Basically this patch merges arch/arm/common/pl330.c into
> drivers/dma/pl330.c driver and removes useless exported
> symbol, externed function and so on.
> 
> The newer pl330 driver tested on SMDKV310 and SMDK4212 boards
> 
> Cc: Jassi Brar <jassisinghbrar at gmail.com>
> Cc: Russell King <rmk+kernel at arm.linux.org.uk>
> Cc: Vinod Koul <vinod.koul at intel.com>
> Acked-by: Linus Walleij <linus.walleij at linaro.org>
> Signed-off-by: Boojin Kim <boojin.kim at samsung.com>
> Signed-off-by: Kukjin Kim <kgene.kim at samsung.com>
> 
> Change since V1:
> Modify the order of definitions/declarations according to linux coding rules.
> ---
>  arch/arm/common/Kconfig               |    3 -
>  arch/arm/common/Makefile              |    1 -
>  arch/arm/common/pl330.c               | 1971 ---------------------------------
>  arch/arm/include/asm/hardware/pl330.h |  333 +++---
>  drivers/dma/Kconfig                   |    1 -
>  drivers/dma/pl330.c                   | 1892 +++++++++++++++++++++++++++++++
>  6 files changed, 2074 insertions(+), 2127 deletions(-)
>  delete mode 100644 arch/arm/common/pl330.c

Now that the PL330 code is entirely out of arch/arm, the header should no
longer be in asm/hardware/pl330.h.

Definitions private to the driver should be in drivers/dma/pl330.c or a
header file co-located.  Other definitions for interfaces to that driver
(eg, platform data) should be in include/linux/amba.

That can be the subject of a follow-up patch if this is already finalized.



More information about the linux-arm-kernel mailing list