Skip to contentSkip to conversation

ARM: S3C2412: DMA: Remove I2S FIFO address

PATCH New Ben Dooks 2 participants 2 messages Apr 28, 2010 1 message not in the archive

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

Ben Dooks

author
#1 lore raw

PATCH 1/1 ARM: S3C2412: DMA: Remove I2S FIFO address

Quoted 8 lines · JassiBrar

From: Jassi Brar <[email protected]>

The S3C DMA API doesn't make use of hw_addr.to/from and also the FIFO addresses are provided from the I2S drivers. So these fields are redundant.
This patch removes the hw_addr.to/from fields for I2S and the inclusion of header, paving way for the header to be moved closer to the I2S controller drivers.

Ok, if this works for -rc series then it may be something that could be merged before we get to the next merge window.

Quoted 34 lines
---
 arch/arm/mach-s3c2412/dma.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)
arch/arm/mach-s3c2412/dma.c +0 -3
index e880524..7abecfc 100644
@@ -30,7 +30,6 @@
 #include <mach/regs-mem.h>
 #include <mach/regs-lcd.h>
 #include <mach/regs-sdi.h>
-#include <plat/regs-s3c2412-iis.h>
 #include <plat/regs-iis.h>
 #include <plat/regs-spi.h>
 
@@ -119,13 +118,11 @@ static struct s3c24xx_dma_map __initdata s3c2412_dma_mappings[] = {
 		.name		= "i2s-sdi",
 		.channels	= MAP(S3C2412_DMAREQSEL_I2SRX),
 		.channels_rx	= MAP(S3C2412_DMAREQSEL_I2SRX),
-		.hw_addr.from	= S3C2410_PA_IIS + S3C2412_IISRXD,
 	},
 	[DMACH_I2S_OUT] = {
 		.name		= "i2s-sdo",
 		.channels	= MAP(S3C2412_DMAREQSEL_I2STX),
 		.channels_rx	= MAP(S3C2412_DMAREQSEL_I2STX),
-		.hw_addr.to	= S3C2410_PA_IIS + S3C2412_IISTXD,
 	},
 	[DMACH_USB_EP1] = {
 		.name		= "usb-ep1",
Signature · 1 line
1.6.2.5
Signature · 5 lines
-- 
Ben

Q:      What's a light-year?
A:      One-third less calories than a regular year.
#2 lore raw

PATCH 1/1 ARM: S3C2412: DMA: Remove I2S FIFO address

Quoted 10 lines · Ben Dooks
Quoted 8 lines · JassiBrar

From: Jassi Brar <[email protected]>

The S3C DMA API doesn't make use of hw_addr.to/from and also the FIFO addresses are provided from the I2S drivers. So these fields are redundant.
This patch removes the hw_addr.to/from fields for I2S and the inclusion of header, paving way for the header to be moved closer to the I2S controller drivers.

Quoted 2 lines

Ok, if this works for -rc series then it may be something that could be merged before we get to the next merge window.

I believe this has never had any effect and can be safely merged. Just try a quick
grep hw_addr -r arch/arm/*