Hi,

I just sent the message & patch below to linux-usb.

Groeten,
Bart
----- Forwarded message from Bart Hartgers <bart.hartg...@gmail.com> -----

Date: Wed, 26 Oct 2011 13:29:42 +0200
From: Bart Hartgers <bart.hartg...@gmail.com>
To: linux-...@vger.kernel.org, GregKH <gre...@suse.de>
Cc: ael <law_ence....@ntlworld.com>
Subject: [PATCH] ark3116 initialisation fix
User-Agent: Mutt/1.5.21 (2010-09-15)

Hi,

This patch for the usb serial ark3116 driver fixes an initialisation
ordering bug that gets triggered on hotplug when using at least recent
debian/ubuntu userspace. Without it, ark3116 serial cables don't work.

Patch is against 3.1.

Groeten,
Bart

Signed-off-by: Bart Hartgers <bart.hartg...@gmail.com>
Tested-by: law_ence....@ntlworld.com
---
Move set_termios below dma setup to prevent hardware crash with recent 
userspace.

diff -ur linux-3.1.orig//drivers/usb/serial/ark3116.c 
linux-3.1/drivers/usb/serial/ark3116.c
--- linux-3.1.orig//drivers/usb/serial/ark3116.c        2011-10-24 
09:10:05.000000000 +0200
+++ linux-3.1/drivers/usb/serial/ark3116.c      2011-10-26 09:16:41.776874595 
+0200
@@ -42,7 +42,7 @@
  * Version information
  */
 
-#define DRIVER_VERSION "v0.6"
+#define DRIVER_VERSION "v0.7"
 #define DRIVER_AUTHOR "Bart Hartgers <bart.hartgers+ark3...@gmail.com>"
 #define DRIVER_DESC "USB ARK3116 serial/IrDA driver"
 #define DRIVER_DEV_DESC "ARK3116 RS232/IrDA"
@@ -380,10 +380,6 @@
                goto err_out;
        }
 
-       /* setup termios */
-       if (tty)
-               ark3116_set_termios(tty, port, NULL);
-
        /* remove any data still left: also clears error state */
        ark3116_read_reg(serial, UART_RX, buf);
 
@@ -406,6 +402,10 @@
        /* enable DMA */
        ark3116_write_reg(port->serial, UART_FCR, UART_FCR_DMA_SELECT);
 
+       /* setup termios */
+       if (tty)
+               ark3116_set_termios(tty, port, NULL);
+
 err_out:
        kfree(buf);
        return result;

----- End forwarded message -----



-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20111026113431.GA1955@blackbox

Reply via email to