linux.git/drivers/tty/serial/mcf.c, branch v3.18.71

serial: Use UPF_* constants with struct uart_port flags

2014-07-10T23:06:49+00:00

Fix ASYNC_* constant usage that should be the corresponding UPF_*
constant.

CC: Grant Likely 
CC: Rob Herring 
CC: devicetree@vger.kernel.org
Signed-off-by: Peter Hurley 
Signed-off-by: Greg Kroah-Hartman

serial: treewide: Remove empty implementations of enable_ms()

2014-07-10T00:29:38+00:00

Signed-off-by: Alexander Shiyan 
Signed-off-by: Greg Kroah-Hartman

serial: Fix IGNBRK handling

2014-06-19T20:04:52+00:00

If IGNBRK is set without either BRKINT or PARMRK set, some uart
drivers send a 0x00 byte for BREAK without the TTYBREAK flag to the
line discipline, when it should send either nothing or the TTYBREAK flag
set. This happens because the read_status_mask masks out the BI
condition, which uart_insert_char() then interprets as a normal 0x00 byte.

SUS v3 is clear regarding the meaning of IGNBRK; Section 11.2.2, General
Terminal Interface - Input Modes, states:
  "If IGNBRK is set, a break condition detected on input shall be ignored;
   that is, not put on the input queue and therefore not read by any
   process."

Fix read_status_mask to include the BI bit if IGNBRK is set; the
lsr status retains the BI bit if a BREAK is recv'd, which is
subsequently ignored in uart_insert_char() when masked with the
ignore_status_mask.

Affected drivers:
8250 - all
serial_txx9
mfd
amba-pl010
amba-pl011
atmel_serial
bfin_uart
dz
ip22zilog
max310x
mxs-auart
netx-serial
pnx8xxx_uart
pxa
sb1250-duart
sccnxp
serial_ks8695
sirfsoc_uart
st-asc
vr41xx_siu
zs
sunzilog
fsl_lpuart
sunsab
ucc_uart
bcm63xx_uart
sunsu
efm32-uart
pmac_zilog
mpsc
msm_serial
m32r_sio

Unaffected drivers:
omap-serial
rp2
sa1100
imx
icom

Annotated for fixes:
altera_uart
mcf

Drivers without break detection:
21285
xilinx-uartps
altera_jtaguart
apbuart
arc-uart
clps711x
max3100
uartlite
msm_serial_hs
nwpserial
lantiq
vt8500_serial

Unknown:
samsung
mpc52xx_uart
bfin_sport_uart
cpm_uart/core

Fixes: Bugzilla #71651, '8250_core.c incorrectly handles IGNBRK flag'
Reported-by: Ivan 
Signed-off-by: Peter Hurley 
Signed-off-by: Greg Kroah-Hartman

tty: serial: mcf: drop uart_port->lock before calling tty_flip_buffer_push()

2013-08-27T23:17:34+00:00

The current driver triggers a lockdep warning for if tty_flip_buffer_push() is
called with uart_port->lock locked. This never shows up on UP kernels and comes
up only on SMP kernels.

Crash looks like this (produced with samsung.c driver):

-----
[] (unwind_backtrace+0x0/0xf8) from [] (show_stack+0x10/0x14)
[] (show_stack+0x10/0x14) from [] (dump_stack+0x6c/0xac)
[] (dump_stack+0x6c/0xac) from [] (do_raw_spin_unlock+0xc4/0xd8)
[] (do_raw_spin_unlock+0xc4/0xd8) from [] (_raw_spin_unlock_irqrestore+0xc/0)
[] (_raw_spin_unlock_irqrestore+0xc/0x38) from [] (s3c24xx_serial_rx_chars+0)
[] (s3c24xx_serial_rx_chars+0x12c/0x260) from [] (s3c64xx_serial_handle_irq+)
[] (s3c64xx_serial_handle_irq+0x48/0x60) from [] (handle_irq_event_percpu+0x)
[] (handle_irq_event_percpu+0x50/0x194) from [] (handle_irq_event+0x3c/0x5c)
[] (handle_irq_event+0x3c/0x5c) from [] (handle_fasteoi_irq+0x80/0x13c)
[] (handle_fasteoi_irq+0x80/0x13c) from [] (generic_handle_irq+0x20/0x30)
[] (generic_handle_irq+0x20/0x30) from [] (handle_IRQ+0x38/0x94)
[] (handle_IRQ+0x38/0x94) from [] (gic_handle_irq+0x34/0x68)
[] (gic_handle_irq+0x34/0x68) from [] (__irq_svc+0x40/0x70)
Exception stack(0xc04cdf70 to 0xc04cdfb8)
df60:                                     00000000 00000000 0000166e 00000000
df80: c04cc000 c050278f c050278f 00000001 c04d444c 410fc0f4 c03649b0 00000000
dfa0: 00000001 c04cdfb8 c000f758 c000f75c 60070013 ffffffff
[] (__irq_svc+0x40/0x70) from [] (arch_cpu_idle+0x28/0x30)
[] (arch_cpu_idle+0x28/0x30) from [] (cpu_startup_entry+0x5c/0x148)
[] (cpu_startup_entry+0x5c/0x148) from [] (start_kernel+0x334/0x38c)
BUG: spinlock lockup suspected on CPU#0, kworker/0:1/360
 lock: s3c24xx_serial_ports+0x1d8/0x370, .magic: dead4ead, .owner: /-1, .owner_cpu: -1
CPU: 0 PID: 360 Comm: kworker/0:1 Not tainted 3.11.0-rc6-next-20130819-00003-g75485f1 #2
Workqueue: events flush_to_ldisc
[] (unwind_backtrace+0x0/0xf8) from [] (show_stack+0x10/0x14)
[] (show_stack+0x10/0x14) from [] (dump_stack+0x6c/0xac)
[] (dump_stack+0x6c/0xac) from [] (do_raw_spin_lock+0x100/0x17c)
[] (do_raw_spin_lock+0x100/0x17c) from [] (_raw_spin_lock_irqsave+0x20/0x28)
[] (_raw_spin_lock_irqsave+0x20/0x28) from [] (uart_start+0x18/0x34)
[] (uart_start+0x18/0x34) from [] (__receive_buf+0x4b4/0x738)
[] (__receive_buf+0x4b4/0x738) from [] (n_tty_receive_buf2+0x30/0x98)
[] (n_tty_receive_buf2+0x30/0x98) from [] (flush_to_ldisc+0xec/0x138)
[] (flush_to_ldisc+0xec/0x138) from [] (process_one_work+0xfc/0x348)
[] (process_one_work+0xfc/0x348) from [] (worker_thread+0x138/0x37c)
[] (worker_thread+0x138/0x37c) from [] (kthread+0xa4/0xb0)
[] (kthread+0xa4/0xb0) from [] (ret_from_fork+0x14/0x3c)
-----

Release the port lock before calling tty_flip_buffer_push() and reacquire it
after the call.

Similar stuff was already done for few other drivers in the past, like:

commit 2389b272168ceec056ca1d8a870a97fa9c26e11a
Author: Thomas Gleixner 
Date:   Tue May 29 21:53:50 2007 +0100

    [ARM] 4417/1: Serial: Fix AMBA drivers locking

Signed-off-by: Viresh Kumar 
Signed-off-by: Greg Kroah-Hartman

serial: use dev_get_platdata()

2013-08-01T00:56:15+00:00

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly.

Signed-off-by: Jingoo Han 
Signed-off-by: Greg Kroah-Hartman

serial: mcf: missing uart_unregister_driver() on error in mcf_init()

2013-05-20T18:54:56+00:00

Add the missing uart_unregister_driver() before return
from mcf_init() in the error handling case.

Signed-off-by: Wei Yongjun 
Signed-off-by: Greg Kroah-Hartman

serial: mcf: Add support RS485 in ColdFire serial driver

2013-01-16T06:56:34+00:00

Signed-off-by: Quoc-Viet Nguyen 
Signed-off-by: Greg Kroah-Hartman

TTY: switch tty_flip_buffer_push

2013-01-16T06:30:15+00:00

Now, we start converting tty buffer functions to actually use
tty_port. This will allow us to get rid of the need of tty in many
call sites. Only tty_port will needed and hence no more
tty_port_tty_get in those paths.

Now, the one where most of tty_port_tty_get gets removed:
tty_flip_buffer_push.

IOW we also closed all the races in drivers not using tty_port_tty_get
at all yet.

Also we move tty_flip_buffer_push declaration from include/linux/tty.h
to include/linux/tty_flip.h to all others while we are changing it
anyway.

Signed-off-by: Jiri Slaby 
Signed-off-by: Greg Kroah-Hartman

tty: remove use of __devexit

2012-11-21T23:27:46+00:00

CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.

Signed-off-by: Bill Pemberton 
Cc: Jiri Slaby 
Cc: Alan Cox 
Acked-by: Tobias Klauser 
Cc: Lucas Tavares 
Cc: Daniel Walker 
Cc: Bryan Huntsman 
Cc: "David S. Miller" 
Cc: Peter Korsgaard 
Cc: Tony Prisk 
Acked-by: David Brown 
Signed-off-by: Greg Kroah-Hartman

tty: remove use of __devinit

2012-11-21T23:22:52+00:00

CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.

Signed-off-by: Bill Pemberton 
Cc: Jiri Slaby 
Cc: Alan Cox 
Cc: Lucas Tavares 
Cc: "David S. Miller" 
Cc: Peter Korsgaard 
Cc: Tony Prisk 
Acked-by: Tobias Klauser 
Acked-by: Nicolas Ferre 
Signed-off-by: Greg Kroah-Hartman