Re: [PATCH 00/58] serial/sysrq: Cleanup ifdeffery
Hi Christophe, On 12/13/19 5:47 AM, Christophe Leroy wrote: > Le 13/12/2019 à 01:05, Dmitry Safonov a écrit : [..] > > powerpc patchwork didn't get the full series, see > https://patchwork.ozlabs.org/project/linuxppc-dev/list/?series=148198 Yes, I was under impression that architecture mail-lists want related patches. But now I see that from the patchwork point of view it's better to have the whole series in inbox. > Can't find them on linux-serial patchwork either > (https://patches.linaro.org/project/linux-serial/list/) I'm not sure - maybe the frequency of checking is low? I see all patches in linux-serial ml: https://marc.info/?l=linux-serial=1=201912=2 > It is impossible to review/test powerpc bits without the first patches > of the series, where can the entire series be found ? Sorry for the inconvenience. I can resend without Cc'ing all people just to ppc mail-list if that works for you. Or you can clone it directly from my github: https://github.com/0x7f454c46/linux/tree/sysrq-serial-seq-v1 Thanks, Dmitry
Re: [PATCH 00/58] serial/sysrq: Cleanup ifdeffery
Le 13/12/2019 à 01:05, Dmitry Safonov a écrit : The original purpose of the patches set was to add a way to enable sysrq on a uart where currently it can be constantly either on or off (CONFIG_MAGIC_SYSRQ_SERIAL), see the last patch: "serial/sysrq: Add MAGIC_SYSRQ_SERIAL_SEQUENCE" But to do that, I had to add uart_try_toggle_sysrq() and I didn't want to bloat serial_core.h even more. So, I did cleanup by removing SUPPORT_SYSRQ resulting in a nice diff-stat and lesser ifdeffery. Most patches are one-liners, I decided to keep them separated per-driver to let reviewers easier follow the purpose. Cc: Greg Kroah-Hartman Cc: Jiri Slaby Cc: Vasiliy Khoruzhick Cc: linux-ser...@vger.kernel.org Dmitry Safonov (58): sysrq: Remove sysrq_handler_registered serial: Move sysrq members above serial_core: Un-ifdef sysrq SUPPORT_SYSRQ tty/serial: Migrate aspeed_vuart to use has_sysrq tty/serial: Migrate 8250_fsl to use has_sysrq tty/serial: Migrate bcm63xx_uart to use has_sysrq tty/serial: Migrate 8250_omap to use has_sysrq tty/serial: Migrate 8250_port to use has_sysrq tty/serial: Migrate amba-pl01* to use has_sysrq tty/serial: Migrate apbuart to use has_sysrq tty/serial: Migrate arc_uart to use has_sysrq tty/serial: Migrate atmel_serial to use has_sysrq tty/serial: Migrate clps711x to use has_sysrq tty/serial: Migrate cpm_uart to use has_sysrq tty/serial: Migrate dz to use has_sysrq tty/serial: Migrate efm32-uart to use has_sysrq tty/serial: Migrate fsl_linflexuart to use has_sysrq tty/serial: Migrate fsl_lpuart to use has_sysrq tty/serial: Migrate imx to use has_sysrq tty/serial: Migrate ip22zilog to use has_sysrq tty/serial: Migrate meson_uart to use has_sysrq tty/serial: Migrate milbeaut_usio to use has_sysrq tty/serial: Migrate mpc52xx_uart to use has_sysrq tty/serial: Don't zero port->sysrq tty/serial: Migrate msm_serial to use has_sysrq tty/serial: Migrate mux to use has_sysrq tty/serial: Migrate mxs-auart to use has_sysrq tty/serial: Migrate omap-serial to use has_sysrq tty/serial: Migrate pch_uart to use has_sysrq tty/serial: Don't check port->sysrq tty/serial: Migrate pmac_zilog to use has_sysrq tty/serial: Migrate pnx8xxx_uart to use has_sysrq serial/f81534: Don't check port->sysrq tty/serial: Migrate pxa to use has_sysrq tty/serial: Migrate qcom_geni_serial to use has_sysrq tty/serial: Migrate sa1100 to use has_sysrq tty/serial: Migrate samsung_tty to use has_sysrq tty/serial: Migrate sb1250-duart to use has_sysrq tty/serial: Migrate sccnxp to use has_sysrq tty/serial: Migrate serial_txx9 to use has_sysrq tty/serial: Migrate sh-sci to use has_sysrq tty/serial: Migrate sprd_serial to use has_sysrq tty/serial: Migrate st-asc to use has_sysrq tty/serial: Migrate stm32-usart to use has_sysrq tty/serial: Migrate sunhv to use has_sysrq tty/serial: Migrate sunsab to use has_sysrq tty/serial: Migrate sunsu to use has_sysrq tty/serial: Migrate sunzilog to use has_sysrq serial/ucc_uart: Remove ifdef SUPPORT_SYSRQ tty/serial: Migrate vr41xx_siu to use has_sysrq tty/serial: Migrate vt8500_serial to use has_sysrq tty/serial: Migrate xilinx_uartps to use has_sysrq tty/serial: Migrate zs to use has_sysrq serial_core: Remove SUPPORT_SYSRQ ifdeffery usb/serial: Don't handle break when CONFIG_MAGIC_SYSRQ is disabled serial_core: Move sysrq functions from header file sysctl/sysrq: Remove __sysrq_enabled copy serial/sysrq: Add MAGIC_SYSRQ_SERIAL_SEQUENCE powerpc patchwork didn't get the full series, see https://patchwork.ozlabs.org/project/linuxppc-dev/list/?series=148198 Can't find them on linux-serial patchwork either (https://patches.linaro.org/project/linux-serial/list/) It is impossible to review/test powerpc bits without the first patches of the series, where can the entire series be found ? Christophe arch/powerpc/kernel/legacy_serial.c | 4 +- drivers/tty/serial/8250/8250_aspeed_vuart.c | 5 +- drivers/tty/serial/8250/8250_fsl.c | 4 - drivers/tty/serial/8250/8250_of.c | 4 +- drivers/tty/serial/8250/8250_omap.c | 5 +- drivers/tty/serial/8250/8250_port.c | 5 +- drivers/tty/serial/amba-pl010.c | 5 +- drivers/tty/serial/amba-pl011.c | 6 +- drivers/tty/serial/apbuart.c| 5 +- drivers/tty/serial/arc_uart.c | 5 +- drivers/tty/serial/atmel_serial.c | 9 +- drivers/tty/serial/bcm63xx_uart.c | 5 +- drivers/tty/serial/clps711x.c | 5 +- drivers/tty/serial/cpm_uart/cpm_uart_core.c | 9 +- drivers/tty/serial/dz.c | 5 +- drivers/tty/serial/efm32-uart.c | 5 +- drivers/tty/serial/fsl_linflexuart.c| 8 +- drivers/tty/serial/fsl_lpuart.c | 9 +- drivers/tty/serial/imx.c| 7
[PATCH 00/58] serial/sysrq: Cleanup ifdeffery
The original purpose of the patches set was to add a way to enable sysrq on a uart where currently it can be constantly either on or off (CONFIG_MAGIC_SYSRQ_SERIAL), see the last patch: "serial/sysrq: Add MAGIC_SYSRQ_SERIAL_SEQUENCE" But to do that, I had to add uart_try_toggle_sysrq() and I didn't want to bloat serial_core.h even more. So, I did cleanup by removing SUPPORT_SYSRQ resulting in a nice diff-stat and lesser ifdeffery. Most patches are one-liners, I decided to keep them separated per-driver to let reviewers easier follow the purpose. Cc: Greg Kroah-Hartman Cc: Jiri Slaby Cc: Vasiliy Khoruzhick Cc: linux-ser...@vger.kernel.org Dmitry Safonov (58): sysrq: Remove sysrq_handler_registered serial: Move sysrq members above serial_core: Un-ifdef sysrq SUPPORT_SYSRQ tty/serial: Migrate aspeed_vuart to use has_sysrq tty/serial: Migrate 8250_fsl to use has_sysrq tty/serial: Migrate bcm63xx_uart to use has_sysrq tty/serial: Migrate 8250_omap to use has_sysrq tty/serial: Migrate 8250_port to use has_sysrq tty/serial: Migrate amba-pl01* to use has_sysrq tty/serial: Migrate apbuart to use has_sysrq tty/serial: Migrate arc_uart to use has_sysrq tty/serial: Migrate atmel_serial to use has_sysrq tty/serial: Migrate clps711x to use has_sysrq tty/serial: Migrate cpm_uart to use has_sysrq tty/serial: Migrate dz to use has_sysrq tty/serial: Migrate efm32-uart to use has_sysrq tty/serial: Migrate fsl_linflexuart to use has_sysrq tty/serial: Migrate fsl_lpuart to use has_sysrq tty/serial: Migrate imx to use has_sysrq tty/serial: Migrate ip22zilog to use has_sysrq tty/serial: Migrate meson_uart to use has_sysrq tty/serial: Migrate milbeaut_usio to use has_sysrq tty/serial: Migrate mpc52xx_uart to use has_sysrq tty/serial: Don't zero port->sysrq tty/serial: Migrate msm_serial to use has_sysrq tty/serial: Migrate mux to use has_sysrq tty/serial: Migrate mxs-auart to use has_sysrq tty/serial: Migrate omap-serial to use has_sysrq tty/serial: Migrate pch_uart to use has_sysrq tty/serial: Don't check port->sysrq tty/serial: Migrate pmac_zilog to use has_sysrq tty/serial: Migrate pnx8xxx_uart to use has_sysrq serial/f81534: Don't check port->sysrq tty/serial: Migrate pxa to use has_sysrq tty/serial: Migrate qcom_geni_serial to use has_sysrq tty/serial: Migrate sa1100 to use has_sysrq tty/serial: Migrate samsung_tty to use has_sysrq tty/serial: Migrate sb1250-duart to use has_sysrq tty/serial: Migrate sccnxp to use has_sysrq tty/serial: Migrate serial_txx9 to use has_sysrq tty/serial: Migrate sh-sci to use has_sysrq tty/serial: Migrate sprd_serial to use has_sysrq tty/serial: Migrate st-asc to use has_sysrq tty/serial: Migrate stm32-usart to use has_sysrq tty/serial: Migrate sunhv to use has_sysrq tty/serial: Migrate sunsab to use has_sysrq tty/serial: Migrate sunsu to use has_sysrq tty/serial: Migrate sunzilog to use has_sysrq serial/ucc_uart: Remove ifdef SUPPORT_SYSRQ tty/serial: Migrate vr41xx_siu to use has_sysrq tty/serial: Migrate vt8500_serial to use has_sysrq tty/serial: Migrate xilinx_uartps to use has_sysrq tty/serial: Migrate zs to use has_sysrq serial_core: Remove SUPPORT_SYSRQ ifdeffery usb/serial: Don't handle break when CONFIG_MAGIC_SYSRQ is disabled serial_core: Move sysrq functions from header file sysctl/sysrq: Remove __sysrq_enabled copy serial/sysrq: Add MAGIC_SYSRQ_SERIAL_SEQUENCE arch/powerpc/kernel/legacy_serial.c | 4 +- drivers/tty/serial/8250/8250_aspeed_vuart.c | 5 +- drivers/tty/serial/8250/8250_fsl.c | 4 - drivers/tty/serial/8250/8250_of.c | 4 +- drivers/tty/serial/8250/8250_omap.c | 5 +- drivers/tty/serial/8250/8250_port.c | 5 +- drivers/tty/serial/amba-pl010.c | 5 +- drivers/tty/serial/amba-pl011.c | 6 +- drivers/tty/serial/apbuart.c| 5 +- drivers/tty/serial/arc_uart.c | 5 +- drivers/tty/serial/atmel_serial.c | 9 +- drivers/tty/serial/bcm63xx_uart.c | 5 +- drivers/tty/serial/clps711x.c | 5 +- drivers/tty/serial/cpm_uart/cpm_uart_core.c | 9 +- drivers/tty/serial/dz.c | 5 +- drivers/tty/serial/efm32-uart.c | 5 +- drivers/tty/serial/fsl_linflexuart.c| 8 +- drivers/tty/serial/fsl_lpuart.c | 9 +- drivers/tty/serial/imx.c| 7 +- drivers/tty/serial/ip22zilog.c | 7 +- drivers/tty/serial/meson_uart.c | 5 +- drivers/tty/serial/milbeaut_usio.c | 5 +- drivers/tty/serial/mpc52xx_uart.c | 11 +- drivers/tty/serial/msm_serial.c | 5 +- drivers/tty/serial/mux.c| 5 +- drivers/tty/serial/mxs-auart.c | 5 +- drivers/tty/serial/omap-serial.c| 5 +- drivers/tty/serial/pch_uart.c | 12 +-
