[U-Boot,RESEND,v3,12/19] configs: am335x_pdu001: remove CONFIG_DM_I2C_COMPAT

Message ID 1544190655-4405-13-git-send-email-jjhiblot@ti.com
State Awaiting Upstream
Delegated to: Heiko Schocher
Headers show
Series
  • DM_I2C_COMPAT removal for all ti platforms
Related show

Commit Message

Jean-Jacques Hiblot Dec. 7, 2018, 1:50 p.m.
Remove the last call to the non-DM I2C API.
Also remove the #undef CONFIG_DM_I2C_COMPAT because it is not defined
in the common header file anymore.

Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
Reviewed-by: Felix Brack <fb@ltec.ch>
Tested-by: Felix Brack <fb@ltec.ch>

---

Changes in v3: None
Changes in v2:
- remove the remaining non-DM I2C API call for pdu001 board.

 board/eets/pdu001/board.c | 1 -
 include/configs/pdu001.h  | 6 ------
 2 files changed, 7 deletions(-)

Comments

Heiko Schocher Dec. 10, 2018, 5:21 a.m. | #1
Hello Jean-Jacques,

Am 07.12.2018 um 14:50 schrieb Jean-Jacques Hiblot:
> Remove the last call to the non-DM I2C API.
> Also remove the #undef CONFIG_DM_I2C_COMPAT because it is not defined
> in the common header file anymore.
> 
> Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
> Reviewed-by: Felix Brack <fb@ltec.ch>
> Tested-by: Felix Brack <fb@ltec.ch>
> 
> ---
> 
> Changes in v3: None
> Changes in v2:
> - remove the remaining non-DM I2C API call for pdu001 board.
> 
>   board/eets/pdu001/board.c | 1 -
>   include/configs/pdu001.h  | 6 ------
>   2 files changed, 7 deletions(-)

Thanks for resending!

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

bye,
Heiko Schocher

Patch

diff --git a/board/eets/pdu001/board.c b/board/eets/pdu001/board.c
index b4b8081..b857a5a 100644
--- a/board/eets/pdu001/board.c
+++ b/board/eets/pdu001/board.c
@@ -209,7 +209,6 @@  void am33xx_spl_board_init(void)
 const struct dpll_params *get_dpll_ddr_params(void)
 {
 	enable_i2c0_pin_mux();
-	i2c_init(CONFIG_SYS_OMAP24_I2C_SPEED, CONFIG_SYS_OMAP24_I2C_SLAVE);
 
 	return &dpll_ddr;
 }
diff --git a/include/configs/pdu001.h b/include/configs/pdu001.h
index 7b809e2..e4c2872 100644
--- a/include/configs/pdu001.h
+++ b/include/configs/pdu001.h
@@ -12,12 +12,6 @@ 
 
 #include <configs/ti_am335x_common.h>
 
-/* No more need for I2C legacy compatibility for this board.
- * CONFIG_DM_I2C_COMPAT is defined in ti_armv7_common.h. See the comment there
- * for the right moment to delete the following line.
- */
-#undef CONFIG_DM_I2C_COMPAT
-
 /* Using 32K of volatile storage for environment */
 #define CONFIG_ENV_SIZE		0x4000