From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stas Sergeev <stssppnn@yahoo.com>
Subject: Re: I have a patch for dosemu-1.0.2.1 to enhance rs232 "break" support
Date: Sat, 24 Aug 2002 01:58:41 +0400
Sender: linux-msdos-owner@vger.kernel.org
Message-ID: <3D66B011.5060006@yahoo.com>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Return-path: <linux-msdos-owner@vger.kernel.org>
List-Id: <linux-msdos.vger.kernel.org>
Content-Type: text/plain; charset="us-ascii"; format="flowed"
To: linux-msdos@vger.kernel.org
Cc: Randy Thelen <rthelen@madscientistroom.org>

Hello.

Mark Rejhon wrote:
> Randy Thelen wrote:
>> Hi Mark! 
>> My name is Randy Thelen and I've hacked on 
>> src/base/serial/ser_ports.c
>> for dosemu-1.0.2.1.
Please consider hacking >=1.1.3.2 as the serial
code was changed a bit.

> + * This function updates the msr and mcr with respect to the hardware.
Are you sure we need this for mcr?
And for msr we have a modstat_engine()...
So please provide some more explanations as
currently I don't see a point (yes, as the
serial code is no longer maintained, you'll
have to provide some more explanations than
expected for makeing a maintainer to understand).

> +  if ((com[num].LSR & 1) == 0) sleep(1);
What is this?

>  put_lcr(int num, int val)
> {
> +  int changed;
I think you don't need "changed" here, "val"
must work.

> +    control = 0;  /* UNUSED for TIOCSBRK and TIOCCBRK */
> +    if (val & UART_LCR_SBC)
> +      ioctl(com[num].fd, TIOCSBRK, &control);
Will the ioctl(com[num].fd, TIOCSBRK) work?
I think this ioctl doesn't require an arg at all.