BMC/Intel-BMC/linux.git/drivers/usb/serial/ch341.c, branch dev-4.6

USB: serial: fix compare_const_fl.cocci warnings

2016-02-28T13:35:58+00:00

Move constants to the right of binary operators where it increases readability. Generated by: scripts/coccinelle/misc/compare_const_fl.cocci Signed-off-by: Mathieu OTHACEHE [johan: drop some chunks and fix others, amend commit message ] Signed-off-by: Johan Hovold

Merge tag 'usb-serial-4.1-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/johan/usb-serial into usb-next

2015-04-03T16:59:34+00:00

Johan writes: USB-serial updates for v4.1-rc1 Fix up the f81232 driver, which up to this point has mostly been a placeholder without a proper implementation. Included is also a minor clean up of ch341. Signed-off-by: Johan Hovold

USB: ch341: set tty baud speed according to tty struct

2015-03-02T08:31:02+00:00

The ch341_set_baudrate() function initialize the device baud speed according to the value on priv->baud_rate. By default the ch341_open() set it to a hardcoded value (DEFAULT_BAUD_RATE 9600). Unfortunately, the tty_struct is not initialized with the same default value. (usually 56700) This means that the tty_struct and the device baud rate generator are not synchronized after opening the port. Fixup is done by calling ch341_set_termios() if tty exist. Remove unnecessary variable priv->baud_rate setup as it's already done by ch341_port_probe(). Remove unnecessary call to ch341_set_{handshake,baudrate}() in ch341_open() as there already called in ch341_configure() and ch341_set_termios() Signed-off-by: Nicolas PLANEL Signed-off-by: Johan Hovold

USB: ch341: remove redundant close from open error path

2015-02-26T17:13:02+00:00

Remove redundant call to ch341_close from error path when submission of the interrupt urb fails in open. Signed-off-by: Johan Hovold

USB: serial: add missing newlines to dev_ messages.

2014-03-12T19:44:50+00:00

Add missing newlines to dev_ messages. Also make some messages less verbose where appropriate. Signed-off-by: Johan Hovold Signed-off-by: Greg Kroah-Hartman

usb: delete non-required instances of include

2014-01-08T23:01:39+00:00

None of these files are actually using any __init type directives and hence don't need to include . Most are just a left over from __devinit and __cpuinit removal, or simply due to code getting copied from one driver to the next. Signed-off-by: Paul Gortmaker Signed-off-by: Greg Kroah-Hartman

USB: ch341: clean up interrupt handler

2014-01-03T20:42:24+00:00

Clean up interrupt completion handler somewhat. Signed-off-by: Johan Hovold Signed-off-by: Greg Kroah-Hartman

USB: ch341: switch to generic TIOCMIWAIT implementation

2014-01-03T20:42:24+00:00

Switch to the generic TIOCMIWAIT implementation which does not suffer from the races involved when using the deprecated sleep_on functions. Acked-by: Arnd Bergmann Signed-off-by: Johan Hovold Signed-off-by: Greg Kroah-Hartman

USB: ch341: only wake up MSR queue on changes

2014-01-03T20:42:24+00:00

Only wake up MSR wait queue on actual modem-status changes. Signed-off-by: Johan Hovold Signed-off-by: Greg Kroah-Hartman

USB: ch341: fix ignored TIOCMIWAIT mask

2014-01-03T20:42:24+00:00

Make sure the TIOCMIWAIT mask is always honoured. The CH341 interrupt status has a multiple-status changed flag which indicates that multiple status changes has occurred since last interrupt event. Unfortunately, if the final status is the same, there appears to be no way to determine which signal(s) has changed (an even number of times). This means that the multiple-status flag should not be used in TIOCMIWAIT as it leads to the signal mask argument being ignored (e.g. TIOCMIWAIT could return if DSR changes twice, even though the user only cares about carrier changes). Signed-off-by: Johan Hovold Signed-off-by: Greg Kroah-Hartman