Home » Mailing list archives » linux-kernel » 2008 » January » 23

Re: [PATCH 6/6] atmel_serial: Add DMA support

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Marc Pignat <marc.pignat@...>
To: Haavard Skinnemoen <hskinnemoen@...>
Cc: Andrew Victor <linux@...>, <kernel@...>, <linux-kernel@...>, Remy Bohmer <linux@...>, Chip Coldwell <coldwell@...>
Subject: Re: [PATCH 6/6] atmel_serial: Add DMA support
Date: Wednesday, January 23, 2008 - 8:30 am

Hi!

On Wednesday 23 January 2008, Haavard Skinnemoen wrote:
extract from the linux-arm-kernel 'Mailing List Etiquette':
             10. Cross-posting between linux-arm* lists and other lists. [[40]rmk]
                Please do not do this. Subscribers on other lists may not be
                subscribed to the linux-arm lists, so when they try to reply to
                such a message, they will receive a bounce. This is deemed by
                others to be rude behaviour on the part of the person who
                originally cross-posted.
I just think there is no need to warn, even if definitions are sub-optimal.

Probably for dma safety/performance, The PDC buffer start should be aligned to
cache line, and the size be a multiple of cache line size.

For me there is no need to disbable pdc support once working. PDC can be
enabled/disabled in the board setup code. -> simply remove this definition.

Regards

Marc

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH v3 0/6] atmel_serial: Cleanups, irq handler splitup &..., Haavard Skinnemoen, (Tue Jan 22, 10:50 am)
[PATCH 1/6] atmel_serial: Clean up the code, Haavard Skinnemoen, (Tue Jan 22, 10:50 am)
[PATCH 2/6] atmel_serial: Use cpu_relax() when busy-waiting, Haavard Skinnemoen, (Tue Jan 22, 10:50 am)
[PATCH 3/6] atmel_serial: Use existing console options only ..., Haavard Skinnemoen, (Tue Jan 22, 10:50 am)
[PATCH 4/6] atmel_serial: Fix bugs in probe() error path and..., Haavard Skinnemoen, (Tue Jan 22, 10:50 am)
[PATCH 5/6] atmel_serial: Split the interrupt handler, Haavard Skinnemoen, (Tue Jan 22, 10:50 am)
[PATCH 6/6] atmel_serial: Add DMA support, Haavard Skinnemoen, (Tue Jan 22, 10:50 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Marc Pignat, (Tue Jan 22, 12:52 pm)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Haavard Skinnemoen, (Wed Jan 23, 7:53 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Marc Pignat, (Wed Jan 23, 8:30 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Haavard Skinnemoen, (Wed Jan 23, 8:45 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Marc Pignat, (Wed Jan 23, 9:18 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Haavard Skinnemoen, (Wed Jan 23, 9:35 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Marc Pignat, (Wed Jan 23, 9:52 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Haavard Skinnemoen, (Wed Jan 23, 10:05 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Alan Cox, (Wed Jan 23, 11:04 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Haavard Skinnemoen, (Wed Jan 23, 11:14 am)
Re: [PATCH 6/6] atmel_serial: Add DMA support, Alan Cox, (Wed Jan 23, 12:41 pm)