Re: [U-Boot] [PATCH 01/21] sunxi: Do not build i2c support when we've no i2c controllers

2015-04-25 Thread Ian Campbell
On Fri, 2015-04-24 at 15:48 +0200, Hans de Goede wrote:
 This fixes the following errors being printed during boot:
 
 Error, wrong i2c adapter 0 max 0 possible
 Error, wrong i2c adapter 0 max 0 possible
 
 Signed-off-by: Hans de Goede hdego...@redhat.com

Acked-by: Ian Campbell i...@hellion.org.uk


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 01/21] sunxi: Do not build i2c support when we've no i2c controllers

2015-04-25 Thread Hans de Goede

Hi,

On 25-04-15 01:22, Simon Glass wrote:

Hi Hans,

On 24 April 2015 at 07:48, Hans de Goede hdego...@redhat.com wrote:

This fixes the following errors being printed during boot:

Error, wrong i2c adapter 0 max 0 possible
Error, wrong i2c adapter 0 max 0 possible

Signed-off-by: Hans de Goede hdego...@redhat.com


Where does the error come from?


Somewhere in the i2c code I presume, I did not investigate, I started
by trying to simply disable building the i2c stuff as it makes no sense
to have i2c support build-in when there are no controllers, and that
indeed fixed things.

Regards,

Hans



Reviewed-by: Simon Glass s...@chromium.org


---
  include/configs/sunxi-common.h | 7 +++
  1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h
index 3e49aba..f97e626 100644
--- a/include/configs/sunxi-common.h
+++ b/include/configs/sunxi-common.h
@@ -261,14 +261,15 @@
  #define CONFIG_SPL_I2C_SUPPORT
  #endif

-#define CONFIG_SYS_I2C
  #if defined CONFIG_I2C0_ENABLE || defined CONFIG_I2C1_ENABLE || \
  defined CONFIG_I2C2_ENABLE || defined CONFIG_I2C3_ENABLE || \
  defined CONFIG_I2C4_ENABLE
+#define CONFIG_SYS_I2C
  #define CONFIG_SYS_I2C_MVTWSI
-#endif
  #define CONFIG_SYS_I2C_SPEED   40
  #define CONFIG_SYS_I2C_SLAVE   0x7f
+#define CONFIG_CMD_I2C
+#endif

  #if defined CONFIG_VIDEO_LCD_PANEL_I2C  !(defined CONFIG_SPL_BUILD)
  #define CONFIG_SYS_I2C_SOFT
@@ -288,8 +289,6 @@ extern int soft_i2c_gpio_scl;
  #define CONFIG_VIDEO_LCD_I2C_BUS   -1 /* NA, but necessary to compile */
  #endif

-#define CONFIG_CMD_I2C
-
  /* PMU */
  #if defined CONFIG_AXP152_POWER || defined CONFIG_AXP209_POWER || defined 
CONFIG_AXP221_POWER
  #define CONFIG_SPL_POWER_SUPPORT
--
2.3.5



Regards,
Simon


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 01/21] sunxi: Do not build i2c support when we've no i2c controllers

2015-04-24 Thread Simon Glass
Hi Hans,

On 24 April 2015 at 07:48, Hans de Goede hdego...@redhat.com wrote:
 This fixes the following errors being printed during boot:

 Error, wrong i2c adapter 0 max 0 possible
 Error, wrong i2c adapter 0 max 0 possible

 Signed-off-by: Hans de Goede hdego...@redhat.com

Where does the error come from?

Reviewed-by: Simon Glass s...@chromium.org

 ---
  include/configs/sunxi-common.h | 7 +++
  1 file changed, 3 insertions(+), 4 deletions(-)

 diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h
 index 3e49aba..f97e626 100644
 --- a/include/configs/sunxi-common.h
 +++ b/include/configs/sunxi-common.h
 @@ -261,14 +261,15 @@
  #define CONFIG_SPL_I2C_SUPPORT
  #endif

 -#define CONFIG_SYS_I2C
  #if defined CONFIG_I2C0_ENABLE || defined CONFIG_I2C1_ENABLE || \
  defined CONFIG_I2C2_ENABLE || defined CONFIG_I2C3_ENABLE || \
  defined CONFIG_I2C4_ENABLE
 +#define CONFIG_SYS_I2C
  #define CONFIG_SYS_I2C_MVTWSI
 -#endif
  #define CONFIG_SYS_I2C_SPEED   40
  #define CONFIG_SYS_I2C_SLAVE   0x7f
 +#define CONFIG_CMD_I2C
 +#endif

  #if defined CONFIG_VIDEO_LCD_PANEL_I2C  !(defined CONFIG_SPL_BUILD)
  #define CONFIG_SYS_I2C_SOFT
 @@ -288,8 +289,6 @@ extern int soft_i2c_gpio_scl;
  #define CONFIG_VIDEO_LCD_I2C_BUS   -1 /* NA, but necessary to compile */
  #endif

 -#define CONFIG_CMD_I2C
 -
  /* PMU */
  #if defined CONFIG_AXP152_POWER || defined CONFIG_AXP209_POWER || defined 
 CONFIG_AXP221_POWER
  #define CONFIG_SPL_POWER_SUPPORT
 --
 2.3.5


Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 01/21] sunxi: Do not build i2c support when we've no i2c controllers

2015-04-24 Thread Hans de Goede
This fixes the following errors being printed during boot:

Error, wrong i2c adapter 0 max 0 possible
Error, wrong i2c adapter 0 max 0 possible

Signed-off-by: Hans de Goede hdego...@redhat.com
---
 include/configs/sunxi-common.h | 7 +++
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h
index 3e49aba..f97e626 100644
--- a/include/configs/sunxi-common.h
+++ b/include/configs/sunxi-common.h
@@ -261,14 +261,15 @@
 #define CONFIG_SPL_I2C_SUPPORT
 #endif
 
-#define CONFIG_SYS_I2C
 #if defined CONFIG_I2C0_ENABLE || defined CONFIG_I2C1_ENABLE || \
 defined CONFIG_I2C2_ENABLE || defined CONFIG_I2C3_ENABLE || \
 defined CONFIG_I2C4_ENABLE
+#define CONFIG_SYS_I2C
 #define CONFIG_SYS_I2C_MVTWSI
-#endif
 #define CONFIG_SYS_I2C_SPEED   40
 #define CONFIG_SYS_I2C_SLAVE   0x7f
+#define CONFIG_CMD_I2C
+#endif
 
 #if defined CONFIG_VIDEO_LCD_PANEL_I2C  !(defined CONFIG_SPL_BUILD)
 #define CONFIG_SYS_I2C_SOFT
@@ -288,8 +289,6 @@ extern int soft_i2c_gpio_scl;
 #define CONFIG_VIDEO_LCD_I2C_BUS   -1 /* NA, but necessary to compile */
 #endif
 
-#define CONFIG_CMD_I2C
-
 /* PMU */
 #if defined CONFIG_AXP152_POWER || defined CONFIG_AXP209_POWER || defined 
CONFIG_AXP221_POWER
 #define CONFIG_SPL_POWER_SUPPORT
-- 
2.3.5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot