[U-Boot] [PATCH] arm: omap3: Fix tao3530/omap3_ha SPL boot hangup (GPIO clocks not enabled)

2014-02-14 Thread Stefan Roese
Patch f33b9bd3
[arm: omap3: Enable clocks for peripherals only if they are used]
breaks SPL booting on tao3530. Since some gpio input's are
read to detect the board revision. But with this patch above, the
clocks to the GPIO subsystems are not enabled per default any more.
The GPIO banks need to be configured specifically now.

Signed-off-by: Stefan Roese s...@denx.de
Cc: Tom Rini tr...@ti.com
Cc: Michael Trimarchi mich...@amarulasolutions.com
---
 include/configs/tao3530.h | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/include/configs/tao3530.h b/include/configs/tao3530.h
index 9abfe82..9c04c23 100644
--- a/include/configs/tao3530.h
+++ b/include/configs/tao3530.h
@@ -83,6 +83,13 @@
 #define CONFIG_OMAP_HSMMC
 #define CONFIG_DOS_PARTITION
 
+/* GPIO banks */
+#define CONFIG_OMAP3_GPIO_2/* GPIO32 ..63  is in GPIO bank 2 */
+#define CONFIG_OMAP3_GPIO_3/* GPIO64 ..95  is in GPIO bank 3 */
+#define CONFIG_OMAP3_GPIO_4/* GPIO96 ..127 is in GPIO bank 4 */
+#define CONFIG_OMAP3_GPIO_5/* GPIO128..159 is in GPIO bank 5 */
+#define CONFIG_OMAP3_GPIO_6/* GPIO160..191 is in GPIO bank 6 */
+
 /* commands to include */
 #include config_cmd_default.h
 
-- 
1.8.5.4

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


Re: [U-Boot] [PATCH] arm: omap3: Fix tao3530/omap3_ha SPL boot hangup (GPIO clocks not enabled)

2014-02-14 Thread Stefano Babic
Hi Stefan,

On 14/02/2014 09:47, Stefan Roese wrote:
 Patch f33b9bd3
 [arm: omap3: Enable clocks for peripherals only if they are used]
 breaks SPL booting on tao3530. Since some gpio input's are
 read to detect the board revision. But with this patch above, the
 clocks to the GPIO subsystems are not enabled per default any more.
 The GPIO banks need to be configured specifically now.
 
 Signed-off-by: Stefan Roese s...@denx.de
 Cc: Tom Rini tr...@ti.com
 Cc: Michael Trimarchi mich...@amarulasolutions.com
 ---
  include/configs/tao3530.h | 7 +++
  1 file changed, 7 insertions(+)
 
 diff --git a/include/configs/tao3530.h b/include/configs/tao3530.h
 index 9abfe82..9c04c23 100644
 --- a/include/configs/tao3530.h
 +++ b/include/configs/tao3530.h
 @@ -83,6 +83,13 @@
  #define CONFIG_OMAP_HSMMC
  #define CONFIG_DOS_PARTITION
  
 +/* GPIO banks */
 +#define CONFIG_OMAP3_GPIO_2  /* GPIO32 ..63  is in GPIO bank 2 */
 +#define CONFIG_OMAP3_GPIO_3  /* GPIO64 ..95  is in GPIO bank 3 */
 +#define CONFIG_OMAP3_GPIO_4  /* GPIO96 ..127 is in GPIO bank 4 */
 +#define CONFIG_OMAP3_GPIO_5  /* GPIO128..159 is in GPIO bank 5 */
 +#define CONFIG_OMAP3_GPIO_6  /* GPIO160..191 is in GPIO bank 6 */
 +
  /* commands to include */
  #include config_cmd_default.h
  
 

I confirm the issue, I will send a similar patch for the mcx board.

Reviewed-by: Stefano Babic sba...@denx.de

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot