From owner-cvs-all@FreeBSD.ORG  Sat Sep 27 04:14:00 2003
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id DD2EE16A4B3; Sat, 27 Sep 2003 04:14:00 -0700 (PDT)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 6F84343F93; Sat, 27 Sep 2003 04:14:00 -0700 (PDT)
	(envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.12.9/8.12.9) with ESMTP id h8RBE0XJ018832;
	Sat, 27 Sep 2003 04:14:00 -0700 (PDT)
	(envelope-from bde@repoman.freebsd.org)
Received: (from bde@localhost)
	by repoman.freebsd.org (8.12.9/8.12.9/Submit) id h8RBE0Gs018831;
	Sat, 27 Sep 2003 04:14:00 -0700 (PDT)
	(envelope-from bde)
Message-Id: <200309271114.h8RBE0Gs018831@repoman.freebsd.org>
From: Bruce Evans <bde@FreeBSD.org>
Date: Sat, 27 Sep 2003 04:14:00 -0700 (PDT)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/dev/sio sio.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sat, 27 Sep 2003 11:14:01 -0000

bde         2003/09/27 04:14:00 PDT

  FreeBSD src repository

  Modified files:
    sys/dev/sio          sio.c 
  Log:
  Cleaned up and fixed setting of speeds in comparam():
  - Removed conversion of a zero input speed to the output speed.  This
    has been done better in ttioctl() since rev.1.108 of kern/tty.c
    almost 5 years ago.  comparam() did the conversion incompletely for
    the case where the output speed is also zero.  It had complications
    to avoid using zero speeds, but would still have used a zero input
    speed for setting watermarks if kern/tty.c had passed one.
  - Never permit the input speed to be different from the output speed.
    There was no validity check on the input speed for the case of a zero
    output speed.  Then we didn't change the physical speeds, but we used
    the unvalidated input speed for setting watermarks and didn't return
    an error, so ttioctl() stored the unvalidated input speed in the tty
    struct where it could cause problems later.
  - Removed complications that were to avoid using a divisor of 0.  The
    divisor is now always valid if the speed is accepted.
  
  Revision  Changes    Path
  1.413     +20 -31    src/sys/dev/sio/sio.c