All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ulf Samuelsson" <ulf@atmel.com>
To: "Haavard Skinnemoen" <hskinnemoen@atmel.com>,
	"Andrew Victor" <linux@maxim.org.za>
Cc: "Russell King" <linux@arm.linux.org.uk>,
	"Haavard Skinnemoen" <hskinnemoen@atmel.com>,
	<kernel@avr32linux.org>, <linux-kernel@vger.kernel.org>,
	"Remy Bohmer" <linux@bohmer.net>,
	<linux-arm-kernel@lists.arm.linux.org.uk>
Subject: Re: [PATCH v2 0/6] atmel_serial: Cleanups, irq handler splitup & DMA
Date: Wed, 19 Dec 2007 23:13:53 +0100	[thread overview]
Message-ID: <024201c8429e$6df3f670$3ba6fea9@atmel.com> (raw)
In-Reply-To: 1198077130-9021-1-git-send-email-hskinnemoen@atmel.com



| The following patchset cleans up the atmel_serial driver a bit,
| moves a significant portion of the interrupt handler into a tasklet,
| and adds DMA support. This is the result of a combined effort by Chip
| Coldwell, Remy Bohmer and me. The patches should apply cleanly onto
| Linus' latest git tree.
| 
| With DMA, I see transfer rates around 92 kbps when transferring a big
| file using ZModem (both directions are roughly the same.)
| 

The start and stop bits will use 20 % of the bit rate so the
teoretical max throughput at 115,200 BAUD is 115,200 * 0,8 = 92160 bps.

What would be interesting is to figure out if we can get reliable
transmission at higher frequencies - 230/460/921 kbps.

| Note that break and error handling doesn't work too well with DMA
| enabled. This is a common problem with all the efforts I've seen
| adding DMA support to this driver (including my own). The PDC error
| handling also accesses icount without locking. I'm tempted to just
| ignore the problem for now and hopefully come up with a solution
| later.

Have told the guys responsible for the USART IP block we
need to improve the H/W for error control for the last 3-4 years.
Why not at 120000 gates/sq mm, add error counters???

Best Regards
Ulf Samuelsson

      parent reply	other threads:[~2007-12-20  0:22 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-19 15:12 [PATCH v2 0/6] atmel_serial: Cleanups, irq handler splitup & DMA Haavard Skinnemoen
2007-12-19 15:12 ` [PATCH v2 1/6] atmel_serial: Clean up the code Haavard Skinnemoen
2007-12-19 15:12   ` [PATCH v2 2/6] atmel_serial: Use cpu_relax() when busy-waiting Haavard Skinnemoen
2007-12-19 15:12     ` [PATCH v2 3/6] atmel_serial: Use existing console options only if BRG is running Haavard Skinnemoen
2007-12-19 15:12       ` [PATCH v2 4/6] atmel_serial: Fix bugs in probe() error path and remove() Haavard Skinnemoen
2007-12-19 15:12         ` [PATCH v2 5/6] atmel_serial: Split the interrupt handler Haavard Skinnemoen
2007-12-19 15:12           ` [PATCH v2 6/6] atmel_serial: Add DMA support Haavard Skinnemoen
2007-12-19 15:57 ` [PATCH v2 0/6] atmel_serial: Cleanups, irq handler splitup & DMA Remy Bohmer
2007-12-19 16:09   ` Haavard Skinnemoen
2007-12-19 16:40     ` Remy Bohmer
2007-12-19 16:47       ` Haavard Skinnemoen
2007-12-19 16:59         ` Remy Bohmer
2007-12-19 17:19           ` Haavard Skinnemoen
2007-12-19 20:32             ` Remy Bohmer
2007-12-22 15:56               ` Haavard Skinnemoen
2007-12-19 22:13 ` Ulf Samuelsson [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='024201c8429e$6df3f670$3ba6fea9@atmel.com' \
    --to=ulf@atmel.com \
    --cc=hskinnemoen@atmel.com \
    --cc=kernel@avr32linux.org \
    --cc=linux-arm-kernel@lists.arm.linux.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=linux@bohmer.net \
    --cc=linux@maxim.org.za \
    /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.