public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Russell King <rmk+lkml@arm.linux.org.uk>
To: Rob Prowel <prowel@kuchera.com>
Cc: Stuart MacDonald <stuartm@connecttech.com>, linux-kernel@vger.kernel.org
Subject: Re: questions about 8250 uart support for adhoc boards
Date: Tue, 27 Feb 2007 17:38:45 +0000	[thread overview]
Message-ID: <20070227173844.GA10380@flint.arm.linux.org.uk> (raw)
In-Reply-To: <45E43F20.9060300@kuchera.com>

On Tue, Feb 27, 2007 at 09:24:32AM -0500, Rob Prowel wrote:
> Russell King wrote:
> >On Mon, Feb 26, 2007 at 12:02:57PM -0500, Rob Prowel wrote:
> >  
> >>See below for a (script) dump of that demonstrates the behavior I 
> >>describe.  Additional ports are not configurable until a driver such as 
> >>fourport is loaded.  This is in 2.6.20.1 and 2.6.17 (those versions I 
> >>have tested).
> >>    
> >
> >Check what /proc/tty/driver/serial contains, rather than relying on
> >setserial.  setserial distributed by distros hasn't kept up with the
> >changes in the serial layer.
> >
> >  
> The setserial being used is the latest one from sourceforge.  Its 
> relevant output matches the /proc entries.  The problem still remains 
> though.  Entries for ttyS4 and up seem to be read-only until I add a 
> supplemental driver such as fourport.

After experimenting here, it turns out the reason is you're trying to
configure a port with a zero base baud.  Unfortunately, it starts off
as zero.

However, when you try to change a port without specifying the base baud,
it is possible for the port to become "usable" and unless we enforce
the "must not have zero base baud" rule we have the possibility to oops
the kernel via a divide by zero.

So, the answer is to specify the base baud when using setserial to
configure new ports.

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:

  reply	other threads:[~2007-02-27 17:38 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-23 17:05 questions about 8250 uart support for adhoc boards Rob Prowel
2007-02-23 20:21 ` Russell King
2007-02-23 20:48   ` Rob Prowel
2007-02-24 20:34     ` Stuart MacDonald
2007-02-26 17:02       ` Rob Prowel
2007-02-26 18:05         ` Russell King
2007-02-27 14:24           ` Rob Prowel
2007-02-27 17:38             ` Russell King [this message]
2007-02-27 19:06               ` Stuart MacDonald
2007-02-27 20:40                 ` Rob Prowel
2007-03-08 15:16                   ` Russell King

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=20070227173844.GA10380@flint.arm.linux.org.uk \
    --to=rmk+lkml@arm.linux.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=prowel@kuchera.com \
    --cc=stuartm@connecttech.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox