linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: linux-omap@vger.kernel.org, linux-serial@vger.kernel.org
Subject: omap-serial: transmission of x-char with DMA (and other issues)
Date: Fri, 13 Apr 2012 11:41:11 +0100	[thread overview]
Message-ID: <20120413104111.GF25053@n2100.arm.linux.org.uk> (raw)

Can someone tell me how this works with the current omap-serial driver
please?  It looks to me like this has been broken when DMA support was
added to the driver.

Moreover, please look at the probe function error paths.  They seem to
be lacking any kind of realistic cleanup, so are a potential memory leak.

Then there's the issue of fiddling with the xmit buffer so that it's
using coherent memory in the startup and shutdown functions (why?  when
other serial drivers cope just fine without doing this).  If we want to
use DMA coherent memory there, there should be a clean way to do this,
rather than going behind the upper layers.

             reply	other threads:[~2012-04-13 10:41 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-13 10:41 Russell King - ARM Linux [this message]
2012-04-16 11:09 ` omap-serial: transmission of x-char with DMA (and other issues) Raja, Govindraj
2012-04-16 11:19   ` Russell King - ARM Linux

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=20120413104111.GF25053@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).