diff mbox

[U-Boot,v4,4/8] siemens-am33x-common: Hardcoded value instead of non-included define

Message ID 1435789184-6815-5-git-send-email-contact@paulk.fr
State Superseded
Delegated to: Tom Rini
Headers show

Commit Message

Paul Kocialkowski July 1, 2015, 10:19 p.m. UTC
The config file for the siemens-am33x-common was using OMAP_I2C_STANDARD, which
is defined in a header that is not included in the config header. In most cases,
it was being included by the code using CONFIG_SYS_OMAP24_I2C_SPEED, but it
might not always be the case.

In particular, when introducing I2C SPL support in omap-common's boot-common.c,
the header is missing and including it breaks other devices.

Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
---
 include/configs/siemens-am33x-common.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tom Rini July 14, 2015, 10:11 p.m. UTC | #1
On Thu, Jul 02, 2015 at 12:19:40AM +0200, Paul Kocialkowski wrote:

> The config file for the siemens-am33x-common was using OMAP_I2C_STANDARD, which
> is defined in a header that is not included in the config header. In most cases,
> it was being included by the code using CONFIG_SYS_OMAP24_I2C_SPEED, but it
> might not always be the case.
> 
> In particular, when introducing I2C SPL support in omap-common's boot-common.c,
> the header is missing and including it breaks other devices.
> 
> Signed-off-by: Paul Kocialkowski <contact@paulk.fr>

Reviewed-by: Tom Rini <trini@konsulko.com>
diff mbox

Patch

diff --git a/include/configs/siemens-am33x-common.h b/include/configs/siemens-am33x-common.h
index 7e75797..8b49426 100644
--- a/include/configs/siemens-am33x-common.h
+++ b/include/configs/siemens-am33x-common.h
@@ -133,7 +133,7 @@ 
 #define CONFIG_I2C
 #define CONFIG_CMD_I2C
 #define CONFIG_SYS_I2C
-#define CONFIG_SYS_OMAP24_I2C_SPEED	OMAP_I2C_STANDARD
+#define CONFIG_SYS_OMAP24_I2C_SPEED	100000
 #define CONFIG_SYS_OMAP24_I2C_SLAVE	1
 #define CONFIG_SYS_I2C_OMAP24XX