Attachment 308360 Details for Bug 445211 – first test patch

[patch] first test patch

test-20080604.patch (text/plain), 7.43 KB, created by Aristeu Rozanski on 2008-06-04 15:49:33 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aristeu Rozanski

Created: 2008-06-04 15:49:33 UTC

Size: 7.43 KB

patch

obsolete

>--- linus-2.6.orig/drivers/serial/8250.c	2008-05-06 14:46:11.000000000 -0400
>+++ linus-2.6/drivers/serial/8250.c	2008-06-04 11:46:29.000000000 -0400
>@@ -1401,7 +1401,7 @@ static unsigned int check_modem_status(s
> 		if (status & UART_MSR_TERI)
> 			up->port.icount.rng++;
> 		if (status & UART_MSR_DDSR)
>-			up->port.icount.dsr++;
>+			uart_handle_dsr_change(&up->port, status & UART_MSR_DSR);
> 		if (status & UART_MSR_DDCD)
> 			uart_handle_dcd_change(&up->port, status & UART_MSR_DCD);
> 		if (status & UART_MSR_DCTS)
>@@ -1729,10 +1729,16 @@ static inline void wait_for_xmitr(struct
> 	/* Wait up to 1s for flow control if necessary */
> 	if (up->port.flags & UPF_CONS_FLOW) {
> 		unsigned int tmout;
>+		struct uart_info *info = up->port.info;
>+
> 		for (tmout = 1000000; tmout; tmout--) {
> 			unsigned int msr = serial_in(up, UART_MSR);
> 			up->msr_saved_flags |= msr & MSR_SAVE_FLAGS;
>-			if (msr & UART_MSR_CTS)
>+			if ((info->flags & UIF_CTS_FLOW) &&
>+			    (msr & UART_MSR_CTS))
>+				break;
>+ 			else if ((info->flags & UIF_DSR_FLOW) &&
>+				 (msr & UART_MSR_DSR))
> 				break;
> 			udelay(1);
> 			touch_nmi_watchdog();
>--- linus-2.6.orig/drivers/serial/serial_core.c	2008-05-06 14:46:11.000000000 -0400
>+++ linus-2.6/drivers/serial/serial_core.c	2008-06-04 11:46:29.000000000 -0400
>@@ -193,6 +193,13 @@ static int uart_startup(struct uart_stat
> 			spin_unlock_irq(&port->lock);
> 		}
> 
>+		if (info->flags & UIF_DSR_FLOW) {
>+			spin_lock_irq(&port->lock);
>+			if (!(port->ops->get_mctrl(port) & TIOCM_DSR))
>+				info->tty->hw_stopped = 1;
>+			spin_unlock_irq(&port->lock);
>+		}
>+
> 		info->flags |= UIF_INITIALIZED;
> 
> 		clear_bit(TTY_IO_ERROR, &info->tty->flags);
>@@ -447,6 +454,11 @@ uart_change_speed(struct uart_state *sta
> 	else
> 		state->info->flags &= ~UIF_CTS_FLOW;
> 
>+	if (termios->c_cflag & CDTRDSR)
>+		state->info->flags |= UIF_DSR_FLOW;
>+	else
>+		state->info->flags &= ~UIF_DSR_FLOW;
>+
> 	if (termios->c_cflag & CLOCAL)
> 		state->info->flags &= ~UIF_CHECK_CD;
> 	else
>@@ -607,6 +619,8 @@ static void uart_throttle(struct tty_str
> 
> 	if (tty->termios->c_cflag & CRTSCTS)
> 		uart_clear_mctrl(state->port, TIOCM_RTS);
>+	if (tty->termios->c_cflag & CDTRDSR)
>+		uart_clear_mctrl(state->port, TIOCM_DTR);
> }
> 
> static void uart_unthrottle(struct tty_struct *tty)
>@@ -623,6 +637,8 @@ static void uart_unthrottle(struct tty_s
> 
> 	if (tty->termios->c_cflag & CRTSCTS)
> 		uart_set_mctrl(port, TIOCM_RTS);
>+	if (tty->termios->c_cflag & CDTRDSR)
>+		uart_set_mctrl(port, TIOCM_DTR);
> }
> 
> static int uart_get_info(struct uart_state *state,
>@@ -1198,6 +1214,9 @@ static void uart_set_termios(struct tty_
> 		if (!(cflag & CRTSCTS) ||
> 		    !test_bit(TTY_THROTTLED, &tty->flags))
> 			mask |= TIOCM_RTS;
>+		if (!(cflag & CDTRDSR) ||
>+		    !test_bit(TTY_THROTTLED, &tty->flags))
>+			mask &= ~TIOCM_DTR;
> 		uart_set_mctrl(state->port, mask);
> 	}
> 
>@@ -1218,6 +1237,24 @@ static void uart_set_termios(struct tty_
> 		}
> 		spin_unlock_irqrestore(&state->port->lock, flags);
> 	}
>+
>+	/* Handle turning off CDTRDSR */
>+	if ((old_termios->c_cflag & CDTRDSR) && !(cflag & CDTRDSR)) {
>+		spin_lock_irqsave(&state->port->lock, flags);
>+		tty->hw_stopped = 0;
>+		__uart_start(tty);
>+		spin_unlock_irqrestore(&state->port->lock, flags);
>+	}
>+
>+	if (!(old_termios->c_cflag & CDTRDSR) && (cflag & CDTRDSR)) {
>+		spin_lock_irqsave(&state->port->lock, flags);
>+		if (!(state->port->ops->get_mctrl(state->port) & TIOCM_DSR)) {
>+			tty->hw_stopped = 1;
>+			state->port->ops->stop_tx(state->port);
>+		}
>+		spin_unlock_irqrestore(&state->port->lock, flags);
>+	}
>+
> #if 0
> 	/*
> 	 * No need to wake up processes in open wait, since they
>@@ -1497,7 +1534,8 @@ uart_block_til_ready(struct file *filp, 
> 		 * not set RTS here - we want to make sure we catch
> 		 * the data from the modem.
> 		 */
>-		if (info->tty->termios->c_cflag & CBAUD)
>+		if (info->tty->termios->c_cflag & CBAUD &&
>+		    !(info->tty->termios->c_cflag & CDTRDSR))
> 			uart_set_mctrl(port, TIOCM_DTR);
> 
> 		/*
>@@ -1942,6 +1980,8 @@ uart_set_options(struct uart_port *port,
> 
> 	if (flow == 'r')
> 		termios.c_cflag |= CRTSCTS;
>+	if (flow == 'd')
>+		termios.c_cflag |= CDTRDSR;
> 
> 	/*
> 	 * some uarts on other side don't support no flow control.
>--- linus-2.6.orig/include/asm-x86/termbits.h	2008-05-06 14:46:11.000000000 -0400
>+++ linus-2.6/include/asm-x86/termbits.h	2008-05-06 14:46:11.000000000 -0400
>@@ -157,6 +157,7 @@ struct ktermios {
> #define  B3500000 0010016
> #define  B4000000 0010017
> #define CIBAUD	  002003600000	/* input baud rate */
>+#define CDTRDSR	  004000000000	/* dtrdsr flow control */
> #define CMSPAR	  010000000000	/* mark or space (stick) parity */
> #define CRTSCTS	  020000000000	/* flow control */
> 
>--- linus-2.6.orig/include/linux/serial_core.h	2008-05-06 14:46:11.000000000 -0400
>+++ linus-2.6/include/linux/serial_core.h	2008-06-04 11:46:29.000000000 -0400
>@@ -350,6 +350,7 @@ struct uart_info {
>  * Definitions for info->flags.  These are _private_ to serial_core, and
>  * are specific to this structure.  They may be queried by low level drivers.
>  */
>+#define UIF_DSR_FLOW		((__force uif_t) (1 << 22))
> #define UIF_CHECK_CD		((__force uif_t) (1 << 25))
> #define UIF_CTS_FLOW		((__force uif_t) (1 << 26))
> #define UIF_NORMAL_ACTIVE	((__force uif_t) (1 << 29))
>@@ -508,34 +509,48 @@ uart_handle_dcd_change(struct uart_port 
> }
> 
> /**
>- *	uart_handle_cts_change - handle a change of clear-to-send state
>+ *	uart_handle_flow_control_change - handle a change of CTS or DSR
>  *	@port: uart_port structure for the open port
>- *	@status: new clear to send status, nonzero if active
>+ *	@status: new CTS/DTR status, nonzero if active
>  */
> static inline void
>-uart_handle_cts_change(struct uart_port *port, unsigned int status)
>+uart_handle_flow_control_change(struct uart_port *port, unsigned int status)
> {
> 	struct uart_info *info = port->info;
> 	struct tty_struct *tty = info->tty;
> 
>-	port->icount.cts++;
>-
>-	if (info->flags & UIF_CTS_FLOW) {
>-		if (tty->hw_stopped) {
>-			if (status) {
>-				tty->hw_stopped = 0;
>-				port->ops->start_tx(port);
>-				uart_write_wakeup(port);
>-			}
>-		} else {
>-			if (!status) {
>-				tty->hw_stopped = 1;
>-				port->ops->stop_tx(port);
>-			}
>+	if (tty->hw_stopped) {
>+		if (status) {
>+			tty->hw_stopped = 0;
>+			port->ops->start_tx(port);
>+			uart_write_wakeup(port);
>+		}
>+	} else {
>+		if (!status) {
>+			tty->hw_stopped = 1;
>+			port->ops->stop_tx(port);
> 		}
> 	}
> }
> 
>+static inline void
>+uart_handle_cts_change(struct uart_port *port, unsigned int status)
>+{
>+	struct uart_info *info = port->info;
>+	port->icount.cts++;
>+	if (info->flags & UIF_CTS_FLOW)
>+		uart_handle_flow_control_change(port, status);
>+}
>+
>+static inline void
>+uart_handle_dsr_change(struct uart_port *port, unsigned int status)
>+{
>+	struct uart_info *info = port->info;
>+	port->icount.dsr++;
>+	if (info->flags & UIF_DSR_FLOW)
>+		uart_handle_flow_control_change(port, status);
>+}
>+
> #include <linux/tty_flip.h>
> 
> static inline void
>@@ -559,7 +574,7 @@ uart_insert_char(struct uart_port *port,
>  *	UART_ENABLE_MS - determine if port should enable modem status irqs
>  */
> #define UART_ENABLE_MS(port,cflag)	((port)->flags & UPF_HARDPPS_CD || \
>-					 (cflag) & CRTSCTS || \
>+					 (cflag) & (CRTSCTS | CDTRDSR) || \
> 					 !((cflag) & CLOCAL))
> 
> #endif
>--- linus-2.6.orig/include/linux/termios.h	2008-05-05 18:12:20.000000000 -0400
>+++ linus-2.6/include/linux/termios.h	2008-05-06 14:47:14.000000000 -0400
>@@ -4,4 +4,9 @@
> #include <linux/types.h>
> #include <asm/termios.h>
> 
>+#ifndef CDTRDSR
>+#warning This architecture should implement CDTRDSR
>+#define CDTRDSR 0 /* remove this when all architectures have a definition */
>+#endif
>+
> #endif

Actions: View | Diff

Attachments on bug 445211: 308359 | 308360 | 309219 | 309849 | 315299 | 315300 | 315301