From: Heiko Schocher <hs@denx.de>
To: Simon Glass <sjg@chromium.org>,
U-Boot Mailing List <u-boot@lists.denx.de>
Cc: Tom Rini <trini@konsulko.com>
Subject: Re: [PATCH 12/17] i2c: Rename CONFIG_SYS_I2C to CONFIG_SYS_I2C_LEGACY
Date: Mon, 19 Jul 2021 06:05:09 +0200 [thread overview]
Message-ID: <d9ee7d3b-778c-cbe6-2c9a-dd225cec853f@denx.de> (raw)
In-Reply-To: <20210710211425.12.I0829cbe09db29d57b69efc8a266f8b6159a588ad@changeid>
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@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@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@denx.de
next prev parent reply other threads:[~2021-07-19 4:05 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-11 3:14 [PATCH 00/17] Makefile: Start simplifying Makefile rules (Part A) Simon Glass
2021-07-11 3:14 ` [PATCH 01/17] README: Fix hyphenation in the directory docs Simon Glass
2021-07-28 22:56 ` Tom Rini
2021-07-11 3:14 ` [PATCH 02/17] Makefile: Sort the subdirectories Simon Glass
2021-07-11 3:14 ` [PATCH 03/17] Makefile: Move phy rules into drivers/phy Simon Glass
2021-07-11 3:14 ` [PATCH 04/17] Rename SPL_POWER_SUPPORT to SPL_POWER Simon Glass
2021-07-11 22:22 ` Jaehoon Chung
2021-07-11 3:14 ` [PATCH 05/17] Rename SPL_CRYPTO_SUPPORT to SPL_CRYPTO Simon Glass
2021-07-11 3:14 ` [PATCH 06/17] Rename SPL_ETH_SUPPORT to SPL_ETH Simon Glass
2021-07-11 3:14 ` [PATCH 07/17] Rename SPL_MUSB_NEW_SUPPORT to SPL_MUSB_NEW Simon Glass
2021-07-11 3:14 ` [PATCH 08/17] Rename SPL_WATCHDOG_SUPPORT to SPL_WATCHDOG Simon Glass
2021-07-11 3:14 ` [PATCH 09/17] Rename SPL_USB_HOST_SUPPORT to SPL_USB_HOST Simon Glass
2021-07-11 3:14 ` [PATCH 10/17] Rename GPIO_SUPPORT to GPIO Simon Glass
2021-07-11 3:14 ` [PATCH 11/17] Rename DRIVERS_MISC_SUPPORT to DRIVERS_MISC Simon Glass
2021-07-11 3:14 ` [PATCH 12/17] i2c: Rename CONFIG_SYS_I2C to CONFIG_SYS_I2C_LEGACY Simon Glass
2021-07-19 4:05 ` Heiko Schocher [this message]
2021-07-11 3:14 ` [PATCH 13/17] i2c: Fix the migration warning Simon Glass
2021-07-19 4:05 ` Heiko Schocher
2021-07-11 3:14 ` [PATCH 14/17] i2c: Drop unused CONFIG_I2C Simon Glass
2021-07-19 4:06 ` Heiko Schocher
2021-07-11 3:14 ` [PATCH 15/17] i2c: Create a new Kconfig for I2C Simon Glass
2021-07-19 4:06 ` Heiko Schocher
2021-07-11 3:14 ` [PATCH 16/17] i2c: Rename SPL/TPL_I2C_SUPPORT to I2C Simon Glass
2021-07-19 4:07 ` Heiko Schocher
2021-07-11 3:14 ` [PATCH 17/17] Makefile: Move drivers/i2c/ into drivers/Makefile Simon Glass
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d9ee7d3b-778c-cbe6-2c9a-dd225cec853f@denx.de \
--to=hs@denx.de \
--cc=sjg@chromium.org \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.