All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ralf Baechle <ralf@linux-mips.org>
To: Alan Cox <alan@linux.intel.com>
Cc: Yong Wang <yong.y.wang@linux.intel.com>,
	Greg Kroah-Hartman <gregkh@suse.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	Manuel Lauss <manuel.lauss@googlemail.com>,
	linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org,
	Masayuki Ohtake <masa-korg@dsn.okisemi.com>,
	"Khor, Andrew Chih Howe" <andrew.chih.howe.khor@intel.com>
Subject: Re: One question about 8250 UART driver
Date: Tue, 17 Aug 2010 14:24:27 +0100	[thread overview]
Message-ID: <20100817132427.GA17861@linux-mips.org> (raw)
In-Reply-To: <20100817091622.5aa3dcf2@linux.intel.com>

On Tue, Aug 17, 2010 at 09:16:22AM +0100, Alan Cox wrote:

> > We have a PCI based UART controller that is compatible with the
> > existing 8250 serial driver in most aspects. However, one exception
> > is that our UART controller can take advantage of the onboard DMA
> > contoller to achieve higher throughput. Could you please share your
> > insights about what is the proper way to add such DMA support into
> > existing 8250 driver?
> 
> I'd say the same as I said to the other folks who asked this - create a
> new driver. All the DMA based 8250 devices have differing DMA engines
> and it'll become unmanagable in the existing driver.
> 
> Once we can see which bits of the existing code are useful for the DMA
> 8250-style devices we can then create an 8250-lib.c which contains the
> bits that are useful to 8250 and to non standard 8250-like devices.

An existing example is drivers/serial/ioc3_serial.c.  IOC3 is an SGI
custom chip which is only being used in SGI systems.  It has a PCI
master and 486-style slave bus to which a SuperIO chip is connected.
The IOC3 can either be configured to use the chip in legacy mode; then
the standard 8250 driver will work fine.  It also support a DMA mode
with receive and transmit rings which make it similar to a network chip.

  Ralf

      reply	other threads:[~2010-08-17 13:25 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-17  1:37 One question about 8250 UART driver Yong Wang
2010-08-17  8:16 ` Alan Cox
2010-08-17 13:24   ` Ralf Baechle [this message]

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=20100817132427.GA17861@linux-mips.org \
    --to=ralf@linux-mips.org \
    --cc=akpm@linux-foundation.org \
    --cc=alan@linux.intel.com \
    --cc=andrew.chih.howe.khor@intel.com \
    --cc=gregkh@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    --cc=manuel.lauss@googlemail.com \
    --cc=masa-korg@dsn.okisemi.com \
    --cc=yong.y.wang@linux.intel.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.