All of lore.kernel.org
 help / color / mirror / Atom feed
From: Timur Tabi <timur@FREESCALE.COM>
To: Sebastian Siewior <linuxppc-embedded@ml.breakpoint.cc>
Cc: linuxppc-embedded@ozlabs.org, Zhang Wei <wei.zhang@FREESCALE.COM>
Subject: Re: Using the loop feature of the DMA controller on MPC8544 DS
Date: Wed, 12 Mar 2008 10:58:21 -0500	[thread overview]
Message-ID: <47D7FD9D.2040202@freescale.com> (raw)
In-Reply-To: <20080312123437.GA1589@Chamillionaire.breakpoint.cc>

Sebastian Siewior wrote:
> Hello,
> 
> I have here a MPC8544 DS board and I tried to utilise the dma
> controller. For the transfers I need set the [S|D]AHE bit in the Mode
> Register. This seems to be implemented but is currently unused in the
> driver. 

Which driver?  Zhang's DMA driver?

I haven't looked at Zhang's driver, but I just spotted this code:

case FSL_DMA_IP_85XX:
	new_fsl_chan->toggle_ext_start = fsl_chan_toggle_ext_start;
	new_fsl_chan->toggle_ext_pause = fsl_chan_toggle_ext_pause;
case FSL_DMA_IP_83XX:
	new_fsl_chan->set_src_loop_size = fsl_chan_set_src_loop_size;
	new_fsl_chan->set_dest_loop_size = fsl_chan_set_dest_loop_size;

(Don't let the missing "break" from FSL_DMA_IP_85XX fool you)

So it looks like there already is a way to set the SAHE and DAHE bits.  I guess
this is what you mean by "seems to be implemented but is currently unused"?

> I haven't found a way how to set this bit except globally for
> the whole channel.

Well, the SAHE bit is part of the MR register, so it can only apply to a whole
channel.  This is a hardware limitation.

> Is it possible to solve this nicely except dedicating one channel for
> such an operation?

No.

-- 
Timur Tabi
Linux kernel developer at Freescale

  reply	other threads:[~2008-03-12 16:31 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-12 12:34 Using the loop feature of the DMA controller on MPC8544 DS Sebastian Siewior
2008-03-12 15:58 ` Timur Tabi [this message]
2008-03-12 17:20   ` Sebastian Siewior
2008-03-12 18:47     ` Timur Tabi
2008-03-12 20:44       ` Sebastian Siewior
2008-03-12 20:49         ` Timur Tabi
2008-03-12 21:15           ` Sebastian Siewior
2008-03-12 21:16             ` Timur Tabi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=47D7FD9D.2040202@freescale.com \
    --to=timur@freescale.com \
    --cc=linuxppc-embedded@ml.breakpoint.cc \
    --cc=linuxppc-embedded@ozlabs.org \
    --cc=wei.zhang@FREESCALE.COM \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.