History

USB: serial: add sysrq break-handler dummy

Add inline sysrq break-handler dummy to allow the compiler to eliminate further code when either console or sysrq support isn't enabled and to clearly mark the two sysrq functions as belonging together.

Johan Hovold committed 6 years ago

Commit 4b5cf2b1 parent 4fbfbdb2 files · +7 −3Browse files

Unified · Split

2 files
  1. drivers/usb/serial/generic.c +2 −2
  2. include/linux/usb/serial.h +5 −1
drivers/usb/serial/generic.c+2−2
@@ -585,11 +585,10 @@ int usb_serial_handle_sysrq_char(struct usb_serial_port *port, unsigned int ch) · 584 unmodified lines
585585 return 0;
586586 }
587587 EXPORT_SYMBOL_GPL(usb_serial_handle_sysrq_char);
588−#endif
589588
590589 int usb_serial_handle_break(struct usb_serial_port *port)
591590 {
592− if (!port->port.console || !IS_ENABLED(CONFIG_MAGIC_SYSRQ))
591+ if (!port->port.console)
593592 return 0;
594593
595594 if (!port->sysrq) {
@@ -600,6 +599,7 @@ int usb_serial_handle_break(struct usb_serial_port *port) · 4 unmodified lines
600599 return 0;
601600 }
602601 EXPORT_SYMBOL_GPL(usb_serial_handle_break);
602+#endif
603603
604604 /**
605605 * usb_serial_handle_dcd_change - handle a change of carrier detect state
include/linux/usb/serial.h+5−1
@@ -369,14 +369,18 @@ extern int usb_serial_generic_prepare_write_buffer(struct usb_serial_port *port, · 368 unmodified lines
369369 #if defined(CONFIG_USB_SERIAL_CONSOLE) && defined(CONFIG_MAGIC_SYSRQ)
370370 extern int usb_serial_handle_sysrq_char(struct usb_serial_port *port,
371371 unsigned int ch);
372+extern int usb_serial_handle_break(struct usb_serial_port *port);
372373 #else
373374 static inline int usb_serial_handle_sysrq_char(struct usb_serial_port *port, unsigned int ch)
374375 {
375376 return 0;
376377 }
378+static inline int usb_serial_handle_break(struct usb_serial_port *port)
379+{
380+ return 0;
381+}
377382 #endif
378383
379−extern int usb_serial_handle_break(struct usb_serial_port *port);
380384 extern void usb_serial_handle_dcd_change(struct usb_serial_port *usb_port,
381385 struct tty_struct *tty,
382386 unsigned int status);