All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH]: Remove duplicate s3c_dma_has_circular() definition for S3C64xx.
@ 2009-11-03 22:35 Maurus Cuelenaere
  2009-11-03 22:39 ` Ben Dooks
  0 siblings, 1 reply; 5+ messages in thread
From: Maurus Cuelenaere @ 2009-11-03 22:35 UTC (permalink / raw)
  To: linux-arm-kernel

This patch removes the duplicated s3c_dma_has_circular() definition and so fixes
compilation for S3C64xx.

Signed-off-by: Maurus Cuelenaere <mcuelenaere@gmail.com>

---
 arch/arm/mach-s3c6400/include/mach/dma.h |    8 --------
 1 files changed, 0 insertions(+), 8 deletions(-)

diff --git a/arch/arm/mach-s3c6400/include/mach/dma.h
b/arch/arm/mach-s3c6400/include/mach/dma.h
index 004edab..b7e5494 100644
--- a/arch/arm/mach-s3c6400/include/mach/dma.h
+++ b/arch/arm/mach-s3c6400/include/mach/dma.h
@@ -58,14 +58,6 @@ enum dma_ch {
 	DMACH_MAX		/* the end */
 };

-static __inline__ int s3c_dma_has_circular(void)
-{
-	/* we will be supporting ciruclar buffers as soon as we have DMA
-	 * engine support.
-	 */
-	return 1;
-}
-
 #define S3C2410_DMAF_CIRCULAR		(1 << 0)

 static inline bool s3c_dma_has_circular(void)
-- 
1.6.3.3

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH]: Remove duplicate s3c_dma_has_circular() definition for S3C64xx.
  2009-11-03 22:35 [PATCH]: Remove duplicate s3c_dma_has_circular() definition for S3C64xx Maurus Cuelenaere
@ 2009-11-03 22:39 ` Ben Dooks
  2009-11-03 22:43   ` Maurus Cuelenaere
  0 siblings, 1 reply; 5+ messages in thread
From: Ben Dooks @ 2009-11-03 22:39 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Nov 03, 2009 at 11:35:30PM +0100, Maurus Cuelenaere wrote:
> This patch removes the duplicated s3c_dma_has_circular() definition and so fixes
> compilation for S3C64xx.

removing the wrong one, we've submitted the necessary circular buffer
updates to allow the audio to work without causing it to lose buffers.

-- 
Ben

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH]: Remove duplicate s3c_dma_has_circular() definition for S3C64xx.
  2009-11-03 22:39 ` Ben Dooks
@ 2009-11-03 22:43   ` Maurus Cuelenaere
  2009-11-03 23:06     ` Ben Dooks
  0 siblings, 1 reply; 5+ messages in thread
From: Maurus Cuelenaere @ 2009-11-03 22:43 UTC (permalink / raw)
  To: linux-arm-kernel

Op 03-11-09 23:39, Ben Dooks schreef:
> On Tue, Nov 03, 2009 at 11:35:30PM +0100, Maurus Cuelenaere wrote:
>    
>> This patch removes the duplicated s3c_dma_has_circular() definition and so fixes
>> compilation for S3C64xx.
>>      
> removing the wrong one, we've submitted the necessary circular buffer
> updates to allow the audio to work without causing it to lose buffers.
>    

Ah, should I resubmit a patch which removes the other one?

Regards,
Maurus Cuelenaere

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH]: Remove duplicate s3c_dma_has_circular() definition for S3C64xx.
  2009-11-03 22:43   ` Maurus Cuelenaere
@ 2009-11-03 23:06     ` Ben Dooks
  2009-11-09  5:57       ` jassi brar
  0 siblings, 1 reply; 5+ messages in thread
From: Ben Dooks @ 2009-11-03 23:06 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Nov 03, 2009 at 11:43:17PM +0100, Maurus Cuelenaere wrote:
> Op 03-11-09 23:39, Ben Dooks schreef:
>> On Tue, Nov 03, 2009 at 11:35:30PM +0100, Maurus Cuelenaere wrote:
>>    
>>> This patch removes the duplicated s3c_dma_has_circular() definition and so fixes
>>> compilation for S3C64xx.
>>>      
>> removing the wrong one, we've submitted the necessary circular buffer
>> updates to allow the audio to work without causing it to lose buffers.
>>    
>
> Ah, should I resubmit a patch which removes the other one?

yes.

-- 
Ben

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH]: Remove duplicate s3c_dma_has_circular() definition for S3C64xx.
  2009-11-03 23:06     ` Ben Dooks
@ 2009-11-09  5:57       ` jassi brar
  0 siblings, 0 replies; 5+ messages in thread
From: jassi brar @ 2009-11-09  5:57 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Nov 4, 2009 at 8:06 AM, Ben Dooks <ben-linux@fluff.org> wrote:
> On Tue, Nov 03, 2009 at 11:43:17PM +0100, Maurus Cuelenaere wrote:
>> Op 03-11-09 23:39, Ben Dooks schreef:
>>> On Tue, Nov 03, 2009 at 11:35:30PM +0100, Maurus Cuelenaere wrote:
>>>
>>>> This patch removes the duplicated s3c_dma_has_circular() definition and so fixes
>>>> compilation for S3C64xx.
>>>>
>>> removing the wrong one, we've submitted the necessary circular buffer
>>> updates to allow the audio to work without causing it to lose buffers.
>>>
>>
>> Ah, should I resubmit a patch which removes the other one?
>
> yes.
but, then we will have 'bool' version for s3c2410 and 'int' type for s3c64xx.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2009-11-09  5:57 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-03 22:35 [PATCH]: Remove duplicate s3c_dma_has_circular() definition for S3C64xx Maurus Cuelenaere
2009-11-03 22:39 ` Ben Dooks
2009-11-03 22:43   ` Maurus Cuelenaere
2009-11-03 23:06     ` Ben Dooks
2009-11-09  5:57       ` jassi brar

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.