[PATCH 12/17] i2c: Rename CONFIG_SYS_I2C to CONFIG_SYS_I2C_LEGACY

Heiko Schocher hs at denx.de
Mon Jul 19 06:05:09 CEST 2021


Hello Simon,

On 11.07.21 05:14, Simon Glass wrote:
> It is quite confusing that CONFIG_SYS_I2C selects the legacy I2C and
> CONFIG_DM_I2C selects the current I2C. The deadline to migrate I2C is less
> than a year away.
> 
> Also we want to have a CONFIG_I2C for U-Boot proper just like we have
> CONFIG_SPL_I2C for SPL, so we can simplify the Makefile rules.
> 
> Rename this symbol so it is clear it is going away.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
>  README                                        |  7 ++++--
>  arch/arm/cpu/armv8/fsl-layerscape/soc.c       |  2 +-
>  arch/arm/cpu/armv8/fsl-layerscape/spl.c       |  2 +-
>  .../include/asm/arch-fsl-layerscape/config.h  |  2 +-
>  arch/arm/mach-kirkwood/include/mach/config.h  |  2 +-
>  board/Arcturus/ucp1020/spl.c                  |  2 +-
>  board/compulab/common/Makefile                |  2 +-
>  board/compulab/common/eeprom.h                |  2 +-
>  board/freescale/p1_p2_rdb_pc/spl.c            |  2 +-
>  board/renesas/draak/draak.c                   |  2 +-
>  board/renesas/salvator-x/salvator-x.c         |  4 ++--
>  board/renesas/ulcb/ulcb.c                     |  2 +-
>  board/somlabs/visionsom-6ull/visionsom-6ull.c |  2 +-
>  board/tqc/tqma6/tqma6.c                       |  4 ++--
>  cmd/date.c                                    |  4 ++--
>  cmd/eeprom.c                                  |  2 +-
>  cmd/i2c.c                                     | 24 +++++++++----------
>  common/board_f.c                              |  4 ++--
>  common/stdio.c                                |  2 +-
>  drivers/i2c/Makefile                          |  2 +-
>  include/asm-generic/global_data.h             |  2 +-
>  include/config_fallbacks.h                    |  4 ++--
>  include/configs/M5208EVBE.h                   |  2 +-
>  include/configs/M52277EVB.h                   |  2 +-
>  include/configs/M5235EVB.h                    |  2 +-
>  include/configs/M5253DEMO.h                   |  2 +-
>  include/configs/M5275EVB.h                    |  2 +-
>  include/configs/M53017EVB.h                   |  2 +-
>  include/configs/M5329EVB.h                    |  2 +-
>  include/configs/M5373EVB.h                    |  2 +-
>  include/configs/M54451EVB.h                   |  2 +-
>  include/configs/M54455EVB.h                   |  2 +-
>  include/configs/MPC8349EMDS.h                 |  2 +-
>  include/configs/MPC8349EMDS_SDRAM.h           |  2 +-
>  include/configs/MPC837XERDB.h                 |  2 +-
>  include/configs/MPC8540ADS.h                  |  2 +-
>  include/configs/MPC8548CDS.h                  |  2 +-
>  include/configs/MPC8560ADS.h                  |  2 +-
>  include/configs/P1010RDB.h                    |  2 +-
>  include/configs/P2041RDB.h                    |  2 +-
>  include/configs/T102xRDB.h                    |  2 +-
>  include/configs/T104xRDB.h                    |  2 +-
>  include/configs/T208xQDS.h                    |  2 +-
>  include/configs/T208xRDB.h                    |  2 +-
>  include/configs/T4240RDB.h                    |  2 +-
>  include/configs/UCP1020.h                     |  2 +-
>  include/configs/astro_mcf5373l.h              |  2 +-
>  include/configs/bur_am335x_common.h           |  2 +-
>  include/configs/cl-som-imx7.h                 |  2 +-
>  include/configs/cm_fx6.h                      |  2 +-
>  include/configs/colibri_pxa270.h              |  2 +-
>  include/configs/corenet_ds.h                  |  2 +-
>  include/configs/db-88f6720.h                  |  2 +-
>  include/configs/db-88f6820-gp.h               |  2 +-
>  include/configs/db-mv784mp-gp.h               |  2 +-
>  include/configs/devkit3250.h                  |  2 +-
>  include/configs/ds414.h                       |  2 +-
>  include/configs/eb_cpu5282.h                  |  2 +-
>  include/configs/edminiv2.h                    |  2 +-
>  include/configs/el6x_common.h                 |  2 +-
>  include/configs/embestmx6boards.h             |  2 +-
>  include/configs/ethernut5.h                   |  2 +-
>  include/configs/flea3.h                       |  2 +-
>  include/configs/gw_ventana.h                  |  2 +-
>  include/configs/ids8313.h                     |  2 +-
>  include/configs/imx8mp_evk.h                  |  2 +-
>  include/configs/imx8mq_evk.h                  |  2 +-
>  include/configs/imx8mq_phanbell.h             |  2 +-
>  include/configs/km/km-mpc83xx.h               |  2 +-
>  include/configs/km/km_arm.h                   |  2 +-
>  include/configs/km/pg-wcom-ls102xa.h          |  2 +-
>  include/configs/kzm9g.h                       |  2 +-
>  include/configs/legoev3.h                     |  2 +-
>  include/configs/ls1012a_common.h              |  2 +-
>  include/configs/ls1021aiot.h                  |  2 +-
>  include/configs/ls1021aqds.h                  |  2 +-
>  include/configs/ls1021atsn.h                  |  2 +-
>  include/configs/ls1021atwr.h                  |  2 +-
>  include/configs/ls1028a_common.h              |  2 +-
>  include/configs/ls1043a_common.h              |  2 +-
>  include/configs/ls1046a_common.h              |  2 +-
>  include/configs/ls1088a_common.h              |  2 +-
>  include/configs/ls2080a_common.h              |  2 +-
>  include/configs/m53menlo.h                    |  2 +-
>  include/configs/maxbcm.h                      |  2 +-
>  include/configs/mx53loco.h                    |  2 +-
>  include/configs/mx6sabreauto.h                |  2 +-
>  include/configs/mx6sabresd.h                  |  2 +-
>  include/configs/nitrogen6x.h                  |  2 +-
>  include/configs/novena.h                      |  2 +-
>  include/configs/p1_p2_rdb_pc.h                |  2 +-
>  include/configs/phycore_imx8mp.h              |  2 +-
>  include/configs/pico-imx7d.h                  |  2 +-
>  include/configs/pico-imx8mq.h                 |  2 +-
>  include/configs/siemens-am33x-common.h        |  2 +-
>  include/configs/snapper9260.h                 |  2 +-
>  include/configs/sniper.h                      |  2 +-
>  include/configs/sunxi-common.h                |  2 +-
>  include/configs/t4qds.h                       |  2 +-
>  include/configs/tam3517-common.h              |  2 +-
>  include/configs/theadorable.h                 |  2 +-
>  include/configs/ti_armv7_common.h             |  2 +-
>  include/configs/ti_omap4_common.h             |  2 +-
>  include/configs/tqma6_wru4.h                  |  2 +-
>  include/configs/udoo_neo.h                    |  2 +-
>  include/configs/usbarmory.h                   |  2 +-
>  include/configs/vf610twr.h                    |  2 +-
>  include/configs/vining_2000.h                 |  2 +-
>  include/configs/warp.h                        |  2 +-
>  include/configs/work_92105.h                  |  2 +-
>  include/configs/xpress.h                      |  2 +-
>  include/i2c.h                                 |  8 +++----
>  scripts/config_whitelist.txt                  |  2 +-
>  113 files changed, 136 insertions(+), 133 deletions(-)

Reviewed-by: Heiko Schocher <hs at denx.de>

Thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de


More information about the U-Boot mailing list