public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Heinz-Ado Arnolds <Ado.Arnolds@dhm-systems.de>
To: Marcelo Tosatti <marcelo@conectiva.com.br>,
	Linus Torvalds <torvalds@transmeta.com>,
	alan@lxorguk.ukuu.org.uk
Cc: linux-kernel@vger.kernel.org,
	Rogier Wolff <R.E.Wolff@BitWizard.nl>,
	hylafax-users@hylafax.org, hylafax-devel@hylafax.org
Subject: Re: sx driver, DCD-HylaFAX problem solved
Date: Fri, 21 Dec 2001 10:13:42 +0100	[thread overview]
Message-ID: <3C22FD46.11CC921A@web-systems.net> (raw)
In-Reply-To: <200112210008.BAA20609@cave.bitwizard.nl>

Rogier Wolff wrote:
> 
...
> Please Email Marcello, Linus and linux-kernel with this patch, and
> state that you are submitting this patch on my request for inclusion
> in the standard kernel.
> 
>                         Roger.

Hi Marcelo, hi Linus, hi Alan,

I'm sending you this patch to the specialix sx driver on the request
of Roger E. Wolf. It fixes a long outstanding problem with hangup when
a DCD change is detected by the driver. The CLOCAL in c_cflag was
ignored. The problem shows up in connection with HylaFAX and gettys.

Please insert this patch into the next available kernel releases.
The patch is to linux-2.4.16, but this section of the sx driver
didn't change at least since 2.2.19.

Thanks a lot for your attention.

Ado

-------------------------------------------------------------------------

--- linux/drivers/char/sx.c.~1~	Fri Nov  9 23:01:21 2001
+++ linux/drivers/char/sx.c	Thu Dec 20 17:22:53 2001
@@ -1160,7 +1160,8 @@
 				/* DCD went UP */
 				if( (~(port->gs.flags & ASYNC_NORMAL_ACTIVE) || 
 						 ~(port->gs.flags & ASYNC_CALLOUT_ACTIVE)) &&
-						(sx_read_channel_byte(port, hi_hstat) != HS_IDLE_CLOSED)) {
+						(sx_read_channel_byte(port, hi_hstat) != HS_IDLE_CLOSED) &&
+						!(port->gs.tty->termios->c_cflag & CLOCAL) ) {
 					/* Are we blocking in open?*/
 					sx_dprintk (SX_DEBUG_MODEMSIGNALS, "DCD active, unblocking
open\n");
 					wake_up_interruptible(&port->gs.open_wait);
@@ -1170,7 +1171,8 @@
 			} else {
 				/* DCD went down! */
 				if (!((port->gs.flags & ASYNC_CALLOUT_ACTIVE) &&
-				      (port->gs.flags & ASYNC_CALLOUT_NOHUP))) {
+				      (port->gs.flags & ASYNC_CALLOUT_NOHUP)) &&
+				    !(port->gs.tty->termios->c_cflag & CLOCAL) ) {
 					sx_dprintk (SX_DEBUG_MODEMSIGNALS, "DCD dropped. hanging
up....\n");
 					tty_hangup (port->gs.tty);
 				} else {

-- 
------------------------------------------------------------------------
  Heinz-Ado Arnolds                        Ado.Arnolds@web-systems.net
  Websystems GmbH                              +49 2234 1840-0 (voice)
  Max-Planck-Strasse 2, 50858 Koeln, Germany   +49 2234 1840-40  (fax)

       reply	other threads:[~2001-12-21  9:14 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200112210008.BAA20609@cave.bitwizard.nl>
2001-12-21  9:13 ` Heinz-Ado Arnolds [this message]
2001-12-21  9:16   ` sx driver, DCD-HylaFAX problem solved Heinz-Ado Arnolds

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=3C22FD46.11CC921A@web-systems.net \
    --to=ado.arnolds@dhm-systems.de \
    --cc=R.E.Wolff@BitWizard.nl \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=hylafax-devel@hylafax.org \
    --cc=hylafax-users@hylafax.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcelo@conectiva.com.br \
    --cc=torvalds@transmeta.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