[PATCH v2 0/8] stdio: Clean up stdio.c

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH v2 0/8] stdio: Clean up stdio.c

Simon Glass-3
This file is quite old and has some crufty code. This series converts a
few things to Kconfig and updates the file.

Changes in v2:
- Rebase to master

Simon Glass (8):
  moveconfig: Skip binary and ELF files
  Convert CONFIG_SPLASH_SCREEN et al to Kconfig
  Convert CONFIG_SYS_DEVICE_NULLDEV to Kconfig
  stdio: Tidy up use of CONFIG_SYS_DEVICE_NULLDEV
  stdio: Drop #ifdefs in the header file
  stdio: Update to use compiler for Kconfig checks
  stdio: Drop brackets around &devs.list
  stdio: Tidy up the coding style

 README                                  |  42 ----
 common/Kconfig                          |  17 ++
 common/stdio.c                          | 251 +++++++++++-------------
 configs/M5249EVB_defconfig              |   1 +
 configs/apalis_imx6_defconfig           |   2 +
 configs/aristainetos2_defconfig         |   2 +
 configs/aristainetos2b_defconfig        |   2 +
 configs/aristainetos2bcsl_defconfig     |   2 +
 configs/aristainetos2c_defconfig        |   2 +
 configs/cgtqmx6eval_defconfig           |   2 +
 configs/cm_fx6_defconfig                |   2 +
 configs/colibri-imx6ull_defconfig       |   2 +
 configs/colibri_imx6_defconfig          |   2 +
 configs/colibri_imx7_defconfig          |   2 +
 configs/colibri_imx7_emmc_defconfig     |   2 +
 configs/colibri_pxa270_defconfig        |   1 +
 configs/colibri_vf_defconfig            |   1 +
 configs/dms-ba16-1g_defconfig           |   2 +
 configs/dms-ba16_defconfig              |   2 +
 configs/eb_cpu5282_defconfig            |   1 +
 configs/eb_cpu5282_internal_defconfig   |   1 +
 configs/gwventana_emmc_defconfig        |   1 +
 configs/gwventana_gw5904_defconfig      |   1 +
 configs/gwventana_nand_defconfig        |   1 +
 configs/imx6dl_icore_nand_defconfig     |   2 +
 configs/imx6q_icore_nand_defconfig      |   2 +
 configs/imx6qdl_icore_mmc_defconfig     |   2 +
 configs/imx6qdl_icore_nand_defconfig    |   2 +
 configs/imxrt1050-evk_defconfig         |   2 +
 configs/khadas-vim3_defconfig           |   2 +
 configs/khadas-vim3l_defconfig          |   2 +
 configs/libretech-ac_defconfig          |   2 +
 configs/libretech-cc_defconfig          |   2 +
 configs/libretech-s905d-pc_defconfig    |   2 +
 configs/libretech-s912-pc_defconfig     |   2 +
 configs/m53menlo_defconfig              |   4 +
 configs/marsboard_defconfig             |   2 +
 configs/mx23evk_defconfig               |   1 +
 configs/mx28evk_auart_console_defconfig |   1 +
 configs/mx28evk_defconfig               |   1 +
 configs/mx28evk_nand_defconfig          |   1 +
 configs/mx28evk_spi_defconfig           |   1 +
 configs/mx51evk_defconfig               |   1 +
 configs/mx53loco_defconfig              |   1 +
 configs/mx6cuboxi_defconfig             |   2 +
 configs/mx6qsabrelite_defconfig         |   2 +
 configs/mx6sabreauto_defconfig          |   2 +
 configs/mx6sabresd_defconfig            |   2 +
 configs/mx6sxsabresd_defconfig          |   2 +
 configs/mx6ul_14x14_evk_defconfig       |   2 +
 configs/mx6ul_9x9_evk_defconfig         |   2 +
 configs/mx7dsabresd_defconfig           |   2 +
 configs/mx7dsabresd_qspi_defconfig      |   2 +
 configs/nitrogen6dl2g_defconfig         |   2 +
 configs/nitrogen6dl_defconfig           |   2 +
 configs/nitrogen6q2g_defconfig          |   2 +
 configs/nitrogen6q_defconfig            |   2 +
 configs/nitrogen6s1g_defconfig          |   2 +
 configs/nitrogen6s_defconfig            |   2 +
 configs/nokia_rx51_defconfig            |   1 +
 configs/novena_defconfig                |   1 +
 configs/odroid-c2_defconfig             |   2 +
 configs/odroid-c4_defconfig             |   2 +
 configs/opos6uldev_defconfig            |   3 +
 configs/pico-dwarf-imx6ul_defconfig     |   2 +
 configs/pico-dwarf-imx7d_defconfig      |   2 +
 configs/pico-hobbit-imx6ul_defconfig    |   2 +
 configs/pico-hobbit-imx7d_defconfig     |   2 +
 configs/pico-imx6_defconfig             |   2 +
 configs/pico-imx6ul_defconfig           |   2 +
 configs/pico-imx7d_bl33_defconfig       |   2 +
 configs/pico-imx7d_defconfig            |   2 +
 configs/pico-nymph-imx7d_defconfig      |   2 +
 configs/pico-pi-imx6ul_defconfig        |   2 +
 configs/pico-pi-imx7d_defconfig         |   2 +
 configs/pxm2_defconfig                  |   2 +
 configs/riotboard_defconfig             |   2 +
 configs/riotboard_spl_defconfig         |   2 +
 configs/rut_defconfig                   |   2 +
 configs/s5p4418_nanopi2_defconfig       |   3 +
 configs/sandbox64_defconfig             |   1 +
 configs/sandbox_defconfig               |   1 +
 configs/sandbox_flattree_defconfig      |   1 +
 configs/sandbox_spl_defconfig           |   1 +
 configs/sei510_defconfig                |   2 +
 configs/sei610_defconfig                |   2 +
 configs/stm32f746-disco_defconfig       |   2 +
 configs/stm32f769-disco_defconfig       |   2 +
 configs/tools-only_defconfig            |   1 +
 configs/wandboard_defconfig             |   2 +
 doc/README.silent                       |   2 +-
 drivers/video/Kconfig                   |  85 ++++++++
 include/configs/M52277EVB.h             |   1 -
 include/configs/M5249EVB.h              |   2 -
 include/configs/advantech_dms-ba16.h    |   2 -
 include/configs/apalis_imx6.h           |   2 -
 include/configs/aristainetos2.h         |   2 -
 include/configs/cgtqmx6eval.h           |   2 -
 include/configs/cm_fx6.h                |   2 -
 include/configs/colibri-imx6ull.h       |   2 -
 include/configs/colibri_imx6.h          |   2 -
 include/configs/colibri_imx7.h          |   2 -
 include/configs/colibri_pxa270.h        |   2 -
 include/configs/colibri_vf.h            |   1 -
 include/configs/eb_cpu5282.h            |   1 -
 include/configs/embestmx6boards.h       |   2 -
 include/configs/gw_ventana.h            |   1 -
 include/configs/imx6-engicam.h          |   2 -
 include/configs/imxrt1050-evk.h         |   2 -
 include/configs/m53menlo.h              |   4 -
 include/configs/meson64.h               |   2 -
 include/configs/mx23evk.h               |   1 -
 include/configs/mx28evk.h               |   1 -
 include/configs/mx51evk.h               |   1 -
 include/configs/mx53loco.h              |   1 -
 include/configs/mx6cuboxi.h             |   2 -
 include/configs/mx6sabre_common.h       |   2 -
 include/configs/mx6sxsabresd.h          |   2 -
 include/configs/mx6ul_14x14_evk.h       |   2 -
 include/configs/mx7dsabresd.h           |   2 -
 include/configs/nitrogen6x.h            |   2 -
 include/configs/nokia_rx51.h            |   1 -
 include/configs/novena.h                |   1 -
 include/configs/opos6uldev.h            |   3 -
 include/configs/pico-imx6.h             |   2 -
 include/configs/pico-imx6ul.h           |   2 -
 include/configs/pico-imx7d.h            |   2 -
 include/configs/pxm2.h                  |   2 -
 include/configs/rut.h                   |   2 -
 include/configs/s5p4418_nanopi2.h       |   4 -
 include/configs/sandbox.h               |   1 -
 include/configs/stm32f746-disco.h       |   2 -
 include/configs/wandboard.h             |   2 -
 include/stdio_dev.h                     |  49 ++---
 scripts/config_whitelist.txt            |   5 -
 tools/moveconfig.py                     |   3 +-
 136 files changed, 397 insertions(+), 289 deletions(-)

--
2.28.0.236.gb10cc79966-goog

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH v2 0/8] stdio: Clean up stdio.c

Simon Glass-3
+Tom Rini too

On Tue, 11 Aug 2020 at 11:22, Simon Glass <[hidden email]> wrote:

>
> This file is quite old and has some crufty code. This series converts a
> few things to Kconfig and updates the file.
>
> Changes in v2:
> - Rebase to master
>
> Simon Glass (8):
>   moveconfig: Skip binary and ELF files
>   Convert CONFIG_SPLASH_SCREEN et al to Kconfig
>   Convert CONFIG_SYS_DEVICE_NULLDEV to Kconfig
>   stdio: Tidy up use of CONFIG_SYS_DEVICE_NULLDEV
>   stdio: Drop #ifdefs in the header file
>   stdio: Update to use compiler for Kconfig checks
>   stdio: Drop brackets around &devs.list
>   stdio: Tidy up the coding style
>
>  README                                  |  42 ----
>  common/Kconfig                          |  17 ++
>  common/stdio.c                          | 251 +++++++++++-------------
>  configs/M5249EVB_defconfig              |   1 +
>  configs/apalis_imx6_defconfig           |   2 +
>  configs/aristainetos2_defconfig         |   2 +
>  configs/aristainetos2b_defconfig        |   2 +
>  configs/aristainetos2bcsl_defconfig     |   2 +
>  configs/aristainetos2c_defconfig        |   2 +
>  configs/cgtqmx6eval_defconfig           |   2 +
>  configs/cm_fx6_defconfig                |   2 +
>  configs/colibri-imx6ull_defconfig       |   2 +
>  configs/colibri_imx6_defconfig          |   2 +
>  configs/colibri_imx7_defconfig          |   2 +
>  configs/colibri_imx7_emmc_defconfig     |   2 +
>  configs/colibri_pxa270_defconfig        |   1 +
>  configs/colibri_vf_defconfig            |   1 +
>  configs/dms-ba16-1g_defconfig           |   2 +
>  configs/dms-ba16_defconfig              |   2 +
>  configs/eb_cpu5282_defconfig            |   1 +
>  configs/eb_cpu5282_internal_defconfig   |   1 +
>  configs/gwventana_emmc_defconfig        |   1 +
>  configs/gwventana_gw5904_defconfig      |   1 +
>  configs/gwventana_nand_defconfig        |   1 +
>  configs/imx6dl_icore_nand_defconfig     |   2 +
>  configs/imx6q_icore_nand_defconfig      |   2 +
>  configs/imx6qdl_icore_mmc_defconfig     |   2 +
>  configs/imx6qdl_icore_nand_defconfig    |   2 +
>  configs/imxrt1050-evk_defconfig         |   2 +
>  configs/khadas-vim3_defconfig           |   2 +
>  configs/khadas-vim3l_defconfig          |   2 +
>  configs/libretech-ac_defconfig          |   2 +
>  configs/libretech-cc_defconfig          |   2 +
>  configs/libretech-s905d-pc_defconfig    |   2 +
>  configs/libretech-s912-pc_defconfig     |   2 +
>  configs/m53menlo_defconfig              |   4 +
>  configs/marsboard_defconfig             |   2 +
>  configs/mx23evk_defconfig               |   1 +
>  configs/mx28evk_auart_console_defconfig |   1 +
>  configs/mx28evk_defconfig               |   1 +
>  configs/mx28evk_nand_defconfig          |   1 +
>  configs/mx28evk_spi_defconfig           |   1 +
>  configs/mx51evk_defconfig               |   1 +
>  configs/mx53loco_defconfig              |   1 +
>  configs/mx6cuboxi_defconfig             |   2 +
>  configs/mx6qsabrelite_defconfig         |   2 +
>  configs/mx6sabreauto_defconfig          |   2 +
>  configs/mx6sabresd_defconfig            |   2 +
>  configs/mx6sxsabresd_defconfig          |   2 +
>  configs/mx6ul_14x14_evk_defconfig       |   2 +
>  configs/mx6ul_9x9_evk_defconfig         |   2 +
>  configs/mx7dsabresd_defconfig           |   2 +
>  configs/mx7dsabresd_qspi_defconfig      |   2 +
>  configs/nitrogen6dl2g_defconfig         |   2 +
>  configs/nitrogen6dl_defconfig           |   2 +
>  configs/nitrogen6q2g_defconfig          |   2 +
>  configs/nitrogen6q_defconfig            |   2 +
>  configs/nitrogen6s1g_defconfig          |   2 +
>  configs/nitrogen6s_defconfig            |   2 +
>  configs/nokia_rx51_defconfig            |   1 +
>  configs/novena_defconfig                |   1 +
>  configs/odroid-c2_defconfig             |   2 +
>  configs/odroid-c4_defconfig             |   2 +
>  configs/opos6uldev_defconfig            |   3 +
>  configs/pico-dwarf-imx6ul_defconfig     |   2 +
>  configs/pico-dwarf-imx7d_defconfig      |   2 +
>  configs/pico-hobbit-imx6ul_defconfig    |   2 +
>  configs/pico-hobbit-imx7d_defconfig     |   2 +
>  configs/pico-imx6_defconfig             |   2 +
>  configs/pico-imx6ul_defconfig           |   2 +
>  configs/pico-imx7d_bl33_defconfig       |   2 +
>  configs/pico-imx7d_defconfig            |   2 +
>  configs/pico-nymph-imx7d_defconfig      |   2 +
>  configs/pico-pi-imx6ul_defconfig        |   2 +
>  configs/pico-pi-imx7d_defconfig         |   2 +
>  configs/pxm2_defconfig                  |   2 +
>  configs/riotboard_defconfig             |   2 +
>  configs/riotboard_spl_defconfig         |   2 +
>  configs/rut_defconfig                   |   2 +
>  configs/s5p4418_nanopi2_defconfig       |   3 +
>  configs/sandbox64_defconfig             |   1 +
>  configs/sandbox_defconfig               |   1 +
>  configs/sandbox_flattree_defconfig      |   1 +
>  configs/sandbox_spl_defconfig           |   1 +
>  configs/sei510_defconfig                |   2 +
>  configs/sei610_defconfig                |   2 +
>  configs/stm32f746-disco_defconfig       |   2 +
>  configs/stm32f769-disco_defconfig       |   2 +
>  configs/tools-only_defconfig            |   1 +
>  configs/wandboard_defconfig             |   2 +
>  doc/README.silent                       |   2 +-
>  drivers/video/Kconfig                   |  85 ++++++++
>  include/configs/M52277EVB.h             |   1 -
>  include/configs/M5249EVB.h              |   2 -
>  include/configs/advantech_dms-ba16.h    |   2 -
>  include/configs/apalis_imx6.h           |   2 -
>  include/configs/aristainetos2.h         |   2 -
>  include/configs/cgtqmx6eval.h           |   2 -
>  include/configs/cm_fx6.h                |   2 -
>  include/configs/colibri-imx6ull.h       |   2 -
>  include/configs/colibri_imx6.h          |   2 -
>  include/configs/colibri_imx7.h          |   2 -
>  include/configs/colibri_pxa270.h        |   2 -
>  include/configs/colibri_vf.h            |   1 -
>  include/configs/eb_cpu5282.h            |   1 -
>  include/configs/embestmx6boards.h       |   2 -
>  include/configs/gw_ventana.h            |   1 -
>  include/configs/imx6-engicam.h          |   2 -
>  include/configs/imxrt1050-evk.h         |   2 -
>  include/configs/m53menlo.h              |   4 -
>  include/configs/meson64.h               |   2 -
>  include/configs/mx23evk.h               |   1 -
>  include/configs/mx28evk.h               |   1 -
>  include/configs/mx51evk.h               |   1 -
>  include/configs/mx53loco.h              |   1 -
>  include/configs/mx6cuboxi.h             |   2 -
>  include/configs/mx6sabre_common.h       |   2 -
>  include/configs/mx6sxsabresd.h          |   2 -
>  include/configs/mx6ul_14x14_evk.h       |   2 -
>  include/configs/mx7dsabresd.h           |   2 -
>  include/configs/nitrogen6x.h            |   2 -
>  include/configs/nokia_rx51.h            |   1 -
>  include/configs/novena.h                |   1 -
>  include/configs/opos6uldev.h            |   3 -
>  include/configs/pico-imx6.h             |   2 -
>  include/configs/pico-imx6ul.h           |   2 -
>  include/configs/pico-imx7d.h            |   2 -
>  include/configs/pxm2.h                  |   2 -
>  include/configs/rut.h                   |   2 -
>  include/configs/s5p4418_nanopi2.h       |   4 -
>  include/configs/sandbox.h               |   1 -
>  include/configs/stm32f746-disco.h       |   2 -
>  include/configs/wandboard.h             |   2 -
>  include/stdio_dev.h                     |  49 ++---
>  scripts/config_whitelist.txt            |   5 -
>  tools/moveconfig.py                     |   3 +-
>  136 files changed, 397 insertions(+), 289 deletions(-)
>
> --
> 2.28.0.236.gb10cc79966-goog
>