Hello Valentin,

Am 13.11.2015 um 15:59 schrieb Valentin Longchamp:
From: Holger Brunck <holger.bru...@keymile.com>

On mgcoge3ne we also want to start the test application if the testpin
is asserted. But we don't have a full POST test support yet. So simply
add a function to read the testpin value.

Signed-off-by: Holger Brunck <holger.bru...@keymile.com>
Signed-off-by: Valentin Longchamp <valentin.longch...@keymile.com>
---

  board/keymile/common/common.c | 5 ++++-
  board/keymile/common/common.h | 2 ++
  board/keymile/km82xx/km82xx.c | 9 +++++++++
  3 files changed, 15 insertions(+), 1 deletion(-)

same here ... gpio support would be great.

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

bye,
Heiko

diff --git a/board/keymile/common/common.c b/board/keymile/common/common.c
index b9aff1a..8de129d 100644
--- a/board/keymile/common/common.c
+++ b/board/keymile/common/common.c
@@ -354,8 +354,11 @@ static int do_checktestboot(cmd_tbl_t *cmdtp, int flag, 
int argc,

  #if defined(CONFIG_POST)
        testpin = post_hotkeys_pressed();
-       s = getenv("test_bank");
  #endif
+#if defined(CONFIG_MGCOGE3NE)
+       testpin = get_testpin();
+#endif
+       s = getenv("test_bank");
        /* when test_bank is not set, act as if testpin is not asserted */
        testboot = (testpin != 0) && (s);
        if (verbose) {
diff --git a/board/keymile/common/common.h b/board/keymile/common/common.h
index dcfefc4..c74f569 100644
--- a/board/keymile/common/common.h
+++ b/board/keymile/common/common.h
@@ -134,6 +134,8 @@ int wait_for_fpga_config(void);
  int fpga_reset(void);
  int toggle_eeprom_spi_bus(void);

+int get_testpin(void);
+
  int set_km_env(void);
  int fdt_set_node_and_value(void *blob,
                        char *nodename,
diff --git a/board/keymile/km82xx/km82xx.c b/board/keymile/km82xx/km82xx.c
index 03cd10c..cc9a7c9 100644
--- a/board/keymile/km82xx/km82xx.c
+++ b/board/keymile/km82xx/km82xx.c
@@ -473,3 +473,12 @@ int ft_board_setup(void *blob, bd_t *bd)
        return 0;
  }
  #endif /* defined(CONFIG_OF_BOARD_SETUP) && defined(CONFIG_OF_LIBFDT) */
+
+#if defined(CONFIG_MGCOGE3NE)
+int get_testpin(void)
+{
+       /* Testpin is Port C pin 29 - enable = low */
+       int testpin = !get_pin(0x00000004, 2);
+       return testpin;
+}
+#endif


--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to