Skip to content
Snippets Groups Projects
  1. Feb 07, 2014
  2. Feb 04, 2014
  3. Jan 27, 2014
    • Alexey Brodkin's avatar
      board_r - fixup functions table after relocation · 7395398a
      Alexey Brodkin authored
      
      This is only required for "PIC" relocation and doesn't apply to modern
      "PIE" relocation which does data relocation as well as code.
      
      "init_sequence_r" is just an array that consists of compile-time
      adresses of init functions. Since this is basically an array of integers
      (pointers to "void" to be more precise) it won't be modified during
      relocation - it will be just copied to new location as it is.
      
      As a consequence on execution after relocation "initcall_run_list" will
      be jumping to pre-relocation addresses. As long as we don't overwrite
      pre-relocation memory area init calls are executed correctly. But still
      it is dangerous because after relocation we don't expect initially used
      memory to stay untouched.
      
      Cc: Tom Rini <trini@ti.com>
      Cc: Masahiro Yamada <yamada.m@jp.panasonic.com>
      Cc: Doug Anderson <dianders@chromium.org>
      Cc: Thomas Langer <thomas.langer@lantiq.com>
      Cc: Albert ARIBAUD <albert.u.boot@aribaud.net>
      Acked-by: default avatarSimon Glass <sjg@chromium.org>
      Signed-off-by: default avatarAlexey Brodkin <abrodkin@synopsys.com>
      7395398a
  4. Jan 24, 2014
  5. Jan 20, 2014
  6. Jan 14, 2014
    • Heiko Schocher's avatar
      common, env: optimize boottime · 1b610271
      Heiko Schocher authored
      
      when creating the hashtable, for each environmentvariable
      getenv(ENV_CALLBACK_VAR) and getenv(ENV_FLAGS_VAR) is called,
      which costs at this point a lot of time. So call this two
      getenv() calls only once.
      
      Boottime on the ids8313 board without this patch:
      
      2013-12-19 13:38:22,894:  NAND:  128 MiB
      2013-12-19 13:38:27,659:  In:    serial
      (~4.8 sec)
      
      Bootime with this patch on the ids8313 board:
      
      2013-12-19 13:40:25,332:  NAND:  128 MiB
      2013-12-19 13:40:25,546:  In:    serial
      (~0.2 sec)
      
      Signed-off-by: default avatarHeiko Schocher <hs@denx.de>
      Cc: Tom Rini <trini@ti.com>
      Cc: Joe Hershberger <joe.hershberger@ni.com>
      Cc: Wolfgang Denk <wd@denx.de>
      1b610271
    • Simon Glass's avatar
      bootm: Reinstate special case for standalone images · c5cbe1e2
      Simon Glass authored
      
      For standalone images, bootm had a special case where the OS boot function
      was NULL but did actually exist. It was just called manually.
      
      This was removed by commit 35fc84fa which checks for the non-existence of
      this function before the special case is examined.
      
      There is no obvious reason why standalone is handled with a special case.
      Adjust the code so that standalone has a normal OS boot function. We still
      need a special case for when the function returns, but at least we can
      avoid the main problem.
      
      This is intended to fix the reported:
      
          ERROR: booting os 'U-Boot' (17) is not supported
      
      but needs testing.
      
      Signed-off-by: default avatarSimon Glass <sjg@chromium.org>
      c5cbe1e2
    • miao.yan@windriver.com's avatar
      common/image.c: move VxWorks header string out of CONFIG_CMD_ELF · 68b15e83
      miao.yan@windriver.com authored
      
      Otherwise, when booting VxWorks kernel, the incorrect message will
      be seen:
      
          ARM Unknown OS Kernel Image (uncompressed)
      
      Signed-off-by: default avatarMiao Yan <miao.yan@windriver.com>
      68b15e83
    • Andrew Gabbasov's avatar
      command.c: Fix auto-completion for the full commands list case · 9b438946
      Andrew Gabbasov authored
      
      Compiling of full list of commands does not advance the counter,
      so it always results in an empty list.
      This seems to be (inadvertently?) introduced by commit
      6c7c946c.
      
      Signed-off-by: default avatarAndrew Gabbasov <andrew_gabbasov@mentor.com>
      9b438946
  7. Jan 13, 2014
  8. Jan 11, 2014
  9. Jan 09, 2014
  10. Dec 16, 2013
  11. Dec 13, 2013
  12. Dec 05, 2013
    • Kuo-Jung Su's avatar
      cmd_eeprom: bug fix for i2c read/write · 6ca6d080
      Kuo-Jung Su authored
      
      The local pointer of address (i.e., addr) only gets
      referenced under SPI mode, and it won't be appropriate
      to pass only 1-byte addr[1] to i2c_read/i2c_write while
      CONFIG_SYS_I2C_EEPROM_ADDR_LEN > 1.
      
      1. In U-boot's I2C model, the address would be re-assembled
         to a byte string in MSB order inside I2C controller drivers.
      
      2. The 'CONFIG_SYS_I2C_EEPROM_ADDR_OVERFLOW' option which could
         be found at soft_i2c.c is always turned on in cmd_eeprom.c,
         the addr[0] always contains the device address with overflowed
         MSB address bits.
      
      Signed-off-by: default avatarKuo-Jung Su <dantesu@faraday-tech.com>
      Cc: Alexey Brodkin <abrodkin@synopsys.com>
      Cc: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
      cc: Peter Tyser <ptyser@xes-inc.com>
      Cc: Heiko Schocher <hs@denx.de>
      Cc: Wolfgang Denk <wd@denx.de>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Mischa Jonker <mjonker@synopsys.com>
      6ca6d080
  13. Nov 25, 2013
    • Masahiro Yamada's avatar
      common: Delete unnecessary rules. · 017ce20f
      Masahiro Yamada authored
      
      The directory tools/ is always built before common/.
      So when envcrc tool is necessary in common/Makefile,
      it already exists.
      
      Signed-off-by: default avatarMasahiro Yamada <yamada.m@jp.panasonic.com>
      017ce20f
    • Tom Rini's avatar
      hash.c: Correct non-hash subcommand crc32 addr-save support · 4b756b01
      Tom Rini authored
      
      In the case of not having CONFIG_CMD_HASH but having CONFIG_CMD_CRC32
      enabled (and not CONFIG_CRC32_VERIFY), we end up in this part of the
      code path on hash_command().  However, we will only have exactly 3 args
      here, and 3 > 3 is false, and we will not try and store the hash at the
      address given as arg #3.  The next problem however is that we've been
      moving argv around so the third value is now in argv[0] not argv[3].
      
      Confirmed on AM335x Beaglebone White.
      
      Signed-off-by: default avatarTom Rini <trini@ti.com>
      4b756b01
  14. Nov 22, 2013
Loading