Russell King - ARM Linux wrote: > > 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 <jassisinghb...@gmail.com> > > Cc: Russell King <rmk+ker...@arm.linux.org.uk> > > Cc: Vinod Koul <vinod.k...@intel.com> > > Acked-by: Linus Walleij <linus.wall...@linaro.org> > > Signed-off-by: Boojin Kim <boojin....@samsung.com> > > Signed-off-by: Kukjin Kim <kgene....@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. > Yes, right.
> 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. > OK. > That can be the subject of a follow-up patch if this is already finalized. Yes, this has been already sent to arm-soc for this merge window. So we need another one for address comments from you. Boojin, could you please do that? Thanks. Best regards, Kgene. -- Kukjin Kim <kgene....@samsung.com>, Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html