Thread (2 messages) 2 messages, 2 authors, 2009-01-30

Re: [PATCH RFC] sh: maple: Add support for SEGA Dreamcast VMU and clean up maple bus driver (1/3)

From: Adrian McMenamin <hidden>
Date: 2009-01-29 16:52:58
Also in: linux-sh, lkml

Possibly related (same subject, not in this thread)

On Thu, January 29, 2009 4:43 pm, Adrian McMenamin wrote:
On Thu, January 29, 2009 3:13 am, Paul Mundt wrote:
quoted
On Wed, Jan 28, 2009 at 11:57:49PM +0000, Adrian McMenamin wrote:
quoted
Rework the maple bus driver to support block reads and writes, while
also cleaning up the code and aiming for a consistent namespace for
functions.

-	mutex_init(&mq->mutex);
+	mq->recvbuf->buf = (void *)P2SEGADDR(&((mq->recvbuf->bufx)[0]));
Please do not abuse P2SEGADDR in this fashion. No new code should be
using P2SEGADDR anyways. maple is particularly abusive in this case
since
it bounces back and forth between P2SEGADDR and PHYSADDR to try and
ignore the fact cache flushing has to be handled.

You may also wish to consider ioremap/ioremap_nocache().
Can I just use dma_cache_sync before each dma run and then use the P1
address space?
After the dma run I mean - the cache is flushed before the run - but not
after.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help