Skip to contentSkip to conversation

No big TTY/serial patch merge for 3.6-rc1

DISCUSSION Greg KH 4 participants 8 messages Jul 26, 2012 TTY layer and serial drivers

lore
Share & tools
lore

Greg KH

author maintainer
#0 lore raw

Hi Linus,

I don't really feel comfortable sending you the tty tree at the present time to have merged for 3.6-rc1. It contains some tty changes that are still causing build problems, as Stephen has pointed out over the past week. These fixes are being resolved by Alan, but I don't feel that they have had the time to fully be tested, and given the late arrival of them (i.e. the past few days), and the lack of real amount of time testing in linux-next, I'd really like to postpone the whole merge until 3.7.

Right now, this really isn't a whole lot of patches, there are only 62 patches in the tty-next tree. I've included below the full diffstat and shortlog of them if anyone wants to see them.

There are maybe a few patches below that I think I should cherry-pick and have you pull, but that's just a handful, and are only for a few drivers, nothing in the tty core code at all.

Jiri, I know this postpones your patches from being merged, sorry about that, but this gives us a few more months to ensure that they are working properly :)

Alan, please keep sending me patches to fix these merge issues, but for now, I think it's best to wait until 3.7 for this to go to Linus.

thanks,

greg k-h

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

 .../bindings/tty/serial/nxp-lpc32xx-hsuart.txt     |   14 +
 .../devicetree/bindings/tty/serial/of-serial.txt   |    3 +
 arch/ia64/hp/sim/simserial.c                       |    2 +-
 arch/um/drivers/chan_kern.c                        |    4 +-
 arch/um/drivers/line.c                             |   32 +-
 arch/um/drivers/line.h                             |    3 +-
 drivers/bluetooth/hci_ath.c                        |    2 +-
 drivers/char/mwave/mwavedd.c                       |   16 +-
 drivers/char/pcmcia/synclink_cs.c                  |   24 +-
 drivers/isdn/gigaset/interface.c                   |    4 +-
 drivers/isdn/i4l/isdn_tty.c                        |   16 +-
 drivers/misc/ibmasm/uart.c                         |   16 +-
 drivers/mmc/card/sdio_uart.c                       |   20 +-
 drivers/net/ethernet/sgi/ioc3-eth.c                |   22 +-
 drivers/net/irda/irtty-sir.c                       |   10 +-
 drivers/net/usb/hso.c                              |   12 +-
 drivers/tty/amiserial.c                            |   20 +-
 drivers/tty/cyclades.c                             |   82 +-
 drivers/tty/hvc/hvsi_lib.c                         |    2 +-
 drivers/tty/isicom.c                               |    8 +-
 drivers/tty/moxa.c                                 |   10 +-
 drivers/tty/mxser.c                                |   20 +-
 drivers/tty/n_gsm.c                                |    8 +-
 drivers/tty/n_tty.c                                |    8 +-
 drivers/tty/pty.c                                  |  144 ++--
 drivers/tty/rocket.c                               |   18 +-
 drivers/tty/serial/8250/8250.c                     |   88 +--
 drivers/tty/serial/8250/8250.h                     |   31 +-
 drivers/tty/serial/8250/8250_acorn.c               |   22 +-
 drivers/tty/serial/8250/8250_dw.c                  |   38 +-
 drivers/tty/serial/8250/8250_gsc.c                 |   26 +-
 drivers/tty/serial/8250/8250_hp300.c               |   26 +-
 drivers/tty/serial/8250/8250_pci.c                 |  126 +--
 drivers/tty/serial/8250/8250_pnp.c                 |   28 +-
 drivers/tty/serial/8250/serial_cs.c                |   30 +-
 drivers/tty/serial/Kconfig                         |   19 +
 drivers/tty/serial/Makefile                        |    1 +
 drivers/tty/serial/amba-pl011.c                    |   34 +-
 drivers/tty/serial/bfin_uart.c                     |    2 +-
 drivers/tty/serial/crisv10.c                       |   26 +-
 drivers/tty/serial/imx.c                           |    2 +-
 drivers/tty/serial/ioc4_serial.c                   |    2 +-
 drivers/tty/serial/jsm/jsm_tty.c                   |    8 +-
 drivers/tty/serial/lpc32xx_hs.c                    |  823 ++++++++++++++++++++
 drivers/tty/serial/of_serial.c                     |   14 +-
 drivers/tty/serial/pch_uart.c                      |   59 +-
 drivers/tty/serial/pxa.c                           |   14 +
 drivers/tty/serial/samsung.c                       |   30 +-
 drivers/tty/serial/serial_core.c                   |   34 +-
 drivers/tty/synclink.c                             |   36 +-
 drivers/tty/synclink_gt.c                          |   24 +-
 drivers/tty/synclinkmp.c                           |   24 +-
 drivers/tty/tty_io.c                               |  104 +--
 drivers/tty/tty_ioctl.c                            |  100 +--
 drivers/tty/tty_ldisc.c                            |   12 +-
 drivers/tty/tty_port.c                             |   23 +-
 drivers/tty/vt/keyboard.c                          |   50 +-
 drivers/tty/vt/vt.c                                |   63 +-
 drivers/tty/vt/vt_ioctl.c                          |   47 +-
 drivers/usb/class/cdc-acm.c                        |    2 +-
 drivers/usb/serial/ark3116.c                       |    4 +-
 drivers/usb/serial/belkin_sa.c                     |    2 +-
 drivers/usb/serial/cp210x.c                        |    8 +-
 drivers/usb/serial/cypress_m8.c                    |   40 +-
 drivers/usb/serial/digi_acceleport.c               |   14 +-
 drivers/usb/serial/empeg.c                         |    2 +-
 drivers/usb/serial/f81232.c                        |    3 +-
 drivers/usb/serial/ftdi_sio.c                      |    2 +-
 drivers/usb/serial/io_edgeport.c                   |   12 +-
 drivers/usb/serial/io_ti.c                         |   12 +-
 drivers/usb/serial/ir-usb.c                        |    2 +-
 drivers/usb/serial/iuu_phoenix.c                   |   28 +-
 drivers/usb/serial/keyspan.c                       |    6 +-
 drivers/usb/serial/keyspan_pda.c                   |    4 +-
 drivers/usb/serial/kl5kusb105.c                    |   18 +-
 drivers/usb/serial/kobil_sct.c                     |   14 +-
 drivers/usb/serial/mct_u232.c                      |    4 +-
 drivers/usb/serial/metro-usb.c                     |    8 +-
 drivers/usb/serial/mos7720.c                       |   14 +-
 drivers/usb/serial/mos7840.c                       |   12 +-
 drivers/usb/serial/oti6858.c                       |   10 +-
 drivers/usb/serial/pl2303.c                        |    6 +-
 drivers/usb/serial/quatech2.c                      |    4 +-
 drivers/usb/serial/sierra.c                        |    2 +-
 drivers/usb/serial/spcp8x5.c                       |   12 +-
 drivers/usb/serial/ssu100.c                        |    4 +-
 drivers/usb/serial/ti_usb_3410_5052.c              |   10 +-
 drivers/usb/serial/usb-serial.c                    |    7 +-
 drivers/usb/serial/usb_wwan.c                      |    2 +-
 drivers/usb/serial/whiteheat.c                     |    2 +-
 include/linux/Kbuild                               |    3 -
 include/linux/cd1400.h                             |  292 -------
 include/linux/cdk.h                                |  486 ------------
 include/linux/comstats.h                           |  119 ---
 include/linux/generic_serial.h                     |   35 -
 include/linux/istallion.h                          |  123 ---
 include/linux/kbd_kern.h                           |   13 -
 include/linux/sc26198.h                            |  533 -------------
 include/linux/serial167.h                          |  157 ----
 include/linux/serial_8250.h                        |   33 +-
 include/linux/serial_core.h                        |    3 +-
 include/linux/stallion.h                           |  147 ----
 include/linux/tty.h                                |   52 +-
 include/linux/tty_driver.h                         |   12 +-
 include/net/irda/ircomm_tty.h                      |   17 +-
 net/bluetooth/rfcomm/tty.c                         |    2 +-
 net/irda/ircomm/ircomm_param.c                     |    5 -
 net/irda/ircomm/ircomm_tty.c                       |  281 ++++---
 net/irda/ircomm/ircomm_tty_attach.c                |   40 +-
 net/irda/ircomm/ircomm_tty_ioctl.c                 |   33 +-
 110 files changed, 2081 insertions(+), 3061 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/tty/serial/nxp-lpc32xx-hsuart.txt
 create mode 100644 drivers/tty/serial/lpc32xx_hs.c
 delete mode 100644 include/linux/cd1400.h
 delete mode 100644 include/linux/cdk.h
 delete mode 100644 include/linux/comstats.h
 delete mode 100644 include/linux/generic_serial.h
 delete mode 100644 include/linux/istallion.h
 delete mode 100644 include/linux/sc26198.h
 delete mode 100644 include/linux/serial167.h
 delete mode 100644 include/linux/stallion.h

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

Alan Cox (16):
tty: note race we need to fix
tty: localise the lock
usb: fix sillies in the metro USB driver
8250: use the 8250 register interface not the legacy one 8250: propogate the bugs field
8250: add support for ASIX devices with a FIFO bug
tty: revert incorrectly applied lock patch
tty: move the termios object into the tty
f81232: correct stubbed termios handler
usb, kobil: Sort out some bogus tty handling
tty: Fix up PPC fallout from the termios move 8250: three way resolve of the 8250 diffs
vt: fix the keyboard/led locking
tty: Move the handling of the tty release logic
pch_uart: Fix missing break for 16 byte fifo pcmcia,synclink_cs: fix termios port I missed

Chao Xie (1):
serial: pxa: add spin lock for console write

Christopher Brannon (1):
tty: keyboard.c: Remove locking from vt_get_leds.

Corbin (1):
serial_core: Update buffer overrun statistics.

Dan Carpenter (1):
tty: double unlock on error in ptmx_open()

Darren Hart (1):
pch_uart: Add eg20t_port lock field, avoid recursive spinlocks

Gabor Juhos (1):
tty: of_serial: add no-loopback-test property

Jiri Slaby (23):
TTY: cyclades, add local pointer for card
TTY: ircomm, add tty_port
TTY: ircomm, use close times from tty_port
TTY: ircomm, use open counts from tty_port
TTY: ircomm, use flags from tty_port
TTY: ircomm, revamp locking
TTY: ircomm, use tty from tty_port
TTY: ircomm, define local tty_port
TTY: ircomm, define carrier routines
TTY: ircomm, use tty_port_close_end helper
TTY: ircomm, use tty_port_close_start helper
TTY: um/line, add tty_port
TTY: um/line, use tty from tty_port
PTY: remove one empty ops->remove
PTY: merge pty_install implementations
PTY: add tty_port
TTY: vt, remove con_schedule_flip
TTY: provide drivers with tty_port_install
TTY: vt, add ->install
TTY: usb-serial, use tty_port_install
TTY: centralize fail paths in tty_register_driver
TTY: add ports array to tty_driver
TTY: add tty_port_register_device helper

KeyYoung Park (1):
serial: samsung: protect NULL dereference of clock name

Kyoungil Kim (2):
serial: samsung: Remove NULL checking for baud clock
serial: samsung: Fixed wrong comparison for baudclk_rate

Laurent Pinchart (2):
serial: sh-sci: Fix probe error paths
serial: sh-sci: Make probe fail for ports that exceed the maximum count

Linus Walleij (1):
serial/amba-pl011: fix ages old copy-paste errors

Paul Bolle (2):
delete seven tty headers
Delete generic_serial.h

Rabin Vincent (1):
vt: fix race in vt_waitactive()

Roland Stigge (3):
serial/8250: Add LPC3220 standard UART type serial/of-serial: Add LPC3220 standard UART compatible string
serial: Add driver for LPC32xx High Speed UARTs

Shachar Shemesh (1):
tty ldisc: Close/Reopen race prevention should check the proper flag

Shawn Bohrer (1):
8250_pci: Remove duplicate struct pciserial_board

Tomoya MORINAGA (2):
pch_uart: Fix rx error interrupt setting issue
pch_uart: Fix parity setting issue

Uwe Kleine-König (1):
serial/imx: make devdata member point to const data

--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

#1 lore raw
Quoted 3 lines · Greg KH

Jiri, I know this postpones your patches from being merged, sorry about that, but this gives us a few more months to ensure that they are working properly :)

Fine with me.

When should I send you 3.7 material I have in my local queue -- now or after 3.6-rc1 is out as usual?

thanks,

Signature · 2 lines
js
suse labs
#4 lore raw
In reply to Greg KH
Quoted 3 lines · Greg KH

I don't really feel comfortable sending you the tty tree at the present time to have merged for 3.6-rc1.

Good. This is what I like to see. If it's not ready to be merged, let's not merge it. I don't think anybody will mind horribly.

Linus

author maintainer
#2 lore raw
In reply to Jiri Slaby
Quoted 9 lines · Jiri Slaby
Quoted 3 lines · Greg KH

Jiri, I know this postpones your patches from being merged, sorry about that, but this gives us a few more months to ensure that they are working properly :)

Fine with me.

When should I send you 3.7 material I have in my local queue -- now or after 3.6-rc1 is out as usual?

After 3.6-rc1 is out is good, as I'll just be ignoring any new stuff until then, with the exception of patches to fix build errors in the tty-next tree :)

thanks,

greg k-h

#3 lore raw
Quoted 3 lines · Greg KH

I'll just be ignoring any new stuff
until then, with the exception of patches to fix build errors in the tty-next tree :)

Hehe, OK :).

Signature · 2 lines
js
suse labs
#5 lore raw
In reply to Greg KH
Quoted 1 line · On Thu, 26 Jul 2012 12:08:14 -0700 Greg KH <[email protected]> wrote:

tty: Move the handling of the tty release logic

Can we lose that one specifically. I've chased down Ian Abbotts problem and replicated it and that is the offending patch not the lock localise (which still needs to be kept out as it depends upon this one)

I have it fixed but it's not had enough testing for -rc1 and moving the termios data has enough spectacular hits all drivers fallout for 3.6

Alan

author maintainer
#6 lore raw
Quoted 11 lines · Alan Cox
Quoted 1 line · On Thu, 26 Jul 2012 12:08:14 -0700 Greg KH <[email protected]> wrote:

tty: Move the handling of the tty release logic

Can we lose that one specifically. I've chased down Ian Abbotts problem and replicated it and that is the offending patch not the lock localise (which still needs to be kept out as it depends upon this one)

I have it fixed but it's not had enough testing for -rc1 and moving the termios data has enough spectacular hits all drivers fallout for 3.6

Yes, I can go revert it. I'm working on building up a small tty-linus branch at the moment for just the "must haves" for 3.6. Right now it's just 10 patches listed below. Any objection to me sending these after a round trip through linux-next?

thanks,

greg k-h

 .../devicetree/bindings/tty/serial/of-serial.txt   |    1 +
 drivers/tty/serial/8250/8250.c                     |    8 +++
 drivers/tty/serial/of_serial.c                     |    1 +
 drivers/tty/serial/pch_uart.c                      |   59 +++++++++++++-------
 drivers/tty/serial/samsung.c                       |    4 +-
 drivers/tty/serial/serial_core.c                   |    6 +-
 drivers/tty/tty_ldisc.c                            |    2 +-
 drivers/tty/vt/vt_ioctl.c                          |   47 +++++++++++-----
 include/linux/serial_core.h                        |    3 +-
 9 files changed, 93 insertions(+), 38 deletions(-)

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

Alan Cox (1):
pch_uart: Fix missing break for 16 byte fifo

Corbin (1):
serial_core: Update buffer overrun statistics.

Darren Hart (1):
pch_uart: Add eg20t_port lock field, avoid recursive spinlocks

Kyoungil Kim (1):
serial: samsung: Fixed wrong comparison for baudclk_rate

Rabin Vincent (1):
vt: fix race in vt_waitactive()

Roland Stigge (2):
serial/8250: Add LPC3220 standard UART type serial/of-serial: Add LPC3220 standard UART compatible string

Shachar Shemesh (1):
tty ldisc: Close/Reopen race prevention should check the proper flag

Tomoya MORINAGA (2):
pch_uart: Fix rx error interrupt setting issue
pch_uart: Fix parity setting issue

#7 lore raw
Quoted 10 lines · On Thu, 26 Jul 2012 14:28:07 -0700 Greg KH <[email protected]> wrote:
Quoted 9 lines · Alan Cox
Quoted 1 line · On Thu, 26 Jul 2012 12:08:14 -0700 Greg KH <[email protected]> wrote:

tty: Move the handling of the tty release logic

Can we lose that one specifically. I've chased down Ian Abbotts problem and replicated it and that is the offending patch not the lock localise (which still needs to be kept out as it depends upon this one)

Quoted 7 lines
Quoted 2 lines

I have it fixed but it's not had enough testing for -rc1 and moving the termios data has enough spectacular hits all drivers fallout for 3.6

Yes, I can go revert it. I'm working on building up a small tty-linus branch at the moment for just the "must haves" for 3.6. Right now it's just 10 patches listed below. Any objection to me sending these after a round trip through linux-next?

All look good to me