1. 24 12月, 2011 4 次提交
    • W
      post/lib_powerpc/multi.c: fix stack overflow error · 7ddd4475
      Wolfgang Denk 提交于
      The code and comment disagreed: the comment claimed that r6...r31
      were copied, and consequently the arrays for "src" and "dst" were
      declared with 26 entries, but the actual code ("lmw r5,0(r3)" and
      "stmw r5,0(r4)") copied _27_ words (r5 through r31), which resulted
      in false "POST cpu Error at multi test" messages.
      
      Fix the comment and the array sizes.
      Signed-off-by: NWolfgang Denk <wd@denx.de>
      Cc: Anatolij Gustschin <agust@denx.de>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Kumar Gala <galak@kernel.crashing.org>
      Cc: Kim Phillips <kim.phillips@freescale.com>
      Cc: Andy Fleming <afleming@gmail.com>
      Acked-by: NAnatolij Gustschin <agust@denx.de>
      Tested-by: NAnatolij Gustschin <agust@denx.de>
      7ddd4475
    • W
      post/lib_powerpc/multi.c: code cleanup · 38081ff7
      Wolfgang Denk 提交于
      Clean up and document the code:
      
      - get rid of unneeded code block
      - add comment which code is generated
      Signed-off-by: NWolfgang Denk <wd@denx.de>
      Cc: Anatolij Gustschin <agust@denx.de>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Kumar Gala <galak@kernel.crashing.org>
      Cc: Kim Phillips <kim.phillips@freescale.com>
      Cc: Andy Fleming <afleming@gmail.com>
      Acked-by: NAnatolij Gustschin <agust@denx.de>
      38081ff7
    • W
      post/lib_powerpc/multi.c: make checkpatch clean · a63aec54
      Wolfgang Denk 提交于
      Signed-off-by: NWolfgang Denk <wd@denx.de>
      Cc: Anatolij Gustschin <agust@denx.de>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Kumar Gala <galak@kernel.crashing.org>
      Cc: Kim Phillips <kim.phillips@freescale.com>
      Cc: Andy Fleming <afleming@gmail.com>
      Acked-by: NAnatolij Gustschin <agust@denx.de>
      a63aec54
    • W
      PPC: fix "Warning: FOO uses hard float, BAR uses soft float". · 139e1875
      Wolfgang Denk 提交于
      It appears that with recent versions of GCC the explicit
      "-mhard-float" command line option takes precedence over the
      ``asm(".gnu_attribute 4, 2");'' in the source file, so this no longer
      helps to avoid the warnings we get when linking code that uses FP
      instructions with other code that was built using soft-float.
      
      We can remove the ".gnu_attribute" (which appears to carry no other
      information, at least so far) from the object files, but we also have
      to make sure we don't pull in the __gcc_qsub() and __gcc_qmul()
      functions from the standard libgcc, as these would again "infect" our
      linking.  We copy this code from:
      	gcc-4.2.2/gcc/config/rs6000/darwin-ldouble.c
      This old version was chosen because it was still available under a
      compatible license (GCC v2+).   The file was stripped down to the
      needed parts, and reformatted so it passes checkpatch with only one
      warning (do not add new typedefs).
      Signed-off-by: NWolfgang Denk <wd@denx.de>
      Cc: Kumar Gala <galak@kernel.crashing.org>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Andy Fleming <afleming@gmail.com>
      Cc: Kim Phillips <kim.phillips@freescale.com>
      Tested-by: NStefan Roese <sr@denx.de>
      Tested-by: NAnatolij Gustschin <agust@denx.de>
      139e1875
  2. 22 12月, 2011 5 次提交
  3. 21 12月, 2011 15 次提交
    • H
      km/common: add toolchain variable · d26982cb
      Holger Brunck 提交于
      Add a variable "toolchain" and configure the rootpath
      for the nfsargs with this variable.
      Signed-off-by: NHolger Brunck <holger.brunck@keymile.com>
      cc: Wolfgang Denk <wd@denx.de>
      d26982cb
    • L
    • A
      board/eltec/elppc/misc.c: Fix GCC 4.6 build warning · c3e17d5c
      Anatolij Gustschin 提交于
      Fix:
      misc.c: In function 'misc_init_r':
      misc.c:119:3: warning: dereferencing type-punned pointer
      will break strict-aliasing rules [-Wstrict-aliasing]
      Signed-off-by: NAnatolij Gustschin <agust@denx.de>
      c3e17d5c
    • A
      drivers/net/e1000.c: Fix GCC 4.6 build warnings · 40867a2f
      Anatolij Gustschin 提交于
      Fix:
      e1000.c: In function 'e1000_read_mac_addr':
      e1000.c:1149:2: warning: dereferencing type-punned pointer
      will break strict-aliasing rules [-Wstrict-aliasing]
      
      e1000.c:1149:2: warning: dereferencing type-punned pointer
      will break strict-aliasing rules [-Wstrict-aliasing]
      Signed-off-by: NAnatolij Gustschin <agust@denx.de>
      Cc: Kyle Moffett <Kyle.D.Moffett@boeing.com>
      Acked-by: NKyle Moffett <Kyle.D.Moffett@boeing.com>
      40867a2f
    • A
      board/esd/cpci750/mv_eth.c: Fix GCC 4.6 build warnings · 821315fe
      Anatolij Gustschin 提交于
      Fix:
      mv_eth.c: In function 'mv64360_eth_real_open':
      mv_eth.c:425:6: warning: variable 'port_status' set but not used [-Wunused-but-set-variable]
      mv_eth.c: In function 'mv64360_eth_stop':
      mv_eth.c:643:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
      mv_eth.c: In function 'mv64360_eth_xmit':
      mv_eth.c:719:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
      mv_eth.c: In function 'mv64360_eth_receive':
      mv_eth.c:804:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
      mv_eth.c: In function 'mv64360_eth_get_stats':
      mv_eth.c:903:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
      mv_eth.c: In function 'mv64360_eth_update_stat':
      mv_eth.c:931:24: warning: variable 'dummy' set but not used [-Wunused-but-set-variable]
      mv_eth.c:930:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
      mv_eth.c: In function 'mv64360_eth_print_stat':
      mv_eth.c:1012:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
      mv_eth.c: In function 'eth_clear_mib_counters':
      mv_eth.c:2069:15: warning: variable 'dummy' set but not used [-Wunused-but-set-variable
      Signed-off-by: NAnatolij Gustschin <agust@denx.de>
      Cc: Reinhard Arlt <reinhard.arlt@esd.eu>
      821315fe
    • A
      usb: s3c_udc_otg: fix GCC 4.6 build warnings · ea2d9159
      Anatolij Gustschin 提交于
      Fix:
      In file included from s3c_udc_otg.c:216:0:
      s3c_udc_otg_xfer_dma.c: In function 'complete_tx':
      s3c_udc_otg_xfer_dma.c:280:33: warning: variable 'is_short' set but not used
      s3c_udc_otg_xfer_dma.c:280:6: warning: variable 'ep_tsr' set but not used
      s3c_udc_otg_xfer_dma.c: In function 's3c_udc_irq':
      s3c_udc_otg_xfer_dma.c:469:16: warning: variable 'flags' set but not used
      s3c_udc_otg_xfer_dma.c:468:18: warning: variable 'gintmsk' set but not used
      s3c_udc_otg_xfer_dma.c: In function 's3c_queue':
      s3c_udc_otg_xfer_dma.c:582:14: warning: variable 'gintsts' set but not used
      s3c_udc_otg_xfer_dma.c:581:16: warning: variable 'flags' set but not used
      s3c_udc_otg_xfer_dma.c: In function 's3c_ep0_read':
      s3c_udc_otg_xfer_dma.c:778:6: warning: variable 'ret' set but not used
      s3c_udc_otg_xfer_dma.c: In function 's3c_udc_set_halt':
      s3c_udc_otg_xfer_dma.c:1020:16: warning: variable 'flags' set but not used
      s3c_udc_otg_xfer_dma.c: In function 's3c_ep0_setup':
      s3c_udc_otg_xfer_dma.c:1258:13: warning: initialization from incompatible
      pointer type [enabled by default]
      s3c_udc_otg_xfer_dma.c:1239:16: warning: variable 'is_in' set but not used
      s3c_udc_otg_xfer_dma.c:1239:9: warning: variable 'bytes' set but not used
      s3c_udc_otg.c: In function 'usb_gadget_register_driver':
      s3c_udc_otg.c:292:16: warning: variable 'flags' set but not used
      s3c_udc_otg.c: In function 'usb_gadget_unregister_driver':
      s3c_udc_otg.c:338:16: warning: variable 'flags' set but not used
      s3c_udc_otg.c: In function 's3c_ep_enable':
      s3c_udc_otg.c:582:16: warning: variable 'flags' set but not used
      s3c_udc_otg.c: In function 's3c_ep_disable':
      s3c_udc_otg.c:646:16: warning: variable 'flags' set but not used
      s3c_udc_otg.c: In function 's3c_dequeue':
      s3c_udc_otg.c:704:16: warning: variable 'flags' set but not used
      Signed-off-by: NAnatolij Gustschin <agust@denx.de>
      Cc: Lukasz Majewski <l.majewski@samsung.com>
      Cc: Remy Bohmer <linux@bohmer.net>
      ea2d9159
    • A
      board/samsung/universal_c210/universal.c: Fix build warning · e03492c8
      Anatolij Gustschin 提交于
      Fix:
      universal.c: In function 's5pc210_phy_control':
      universal.c:273:7: warning: 'ret' may be used uninitialized
      in this function [-Wuninitialized]
      Signed-off-by: NAnatolij Gustschin <agust@denx.de>
      Cc: Lukasz Majewski <l.majewski@samsung.com>
      Cc: Minkyu Kang <mk7.kang@samsung.com>
      Acked-by: NMinkyu Kang <mk7.kang@samsung.com>
      e03492c8
    • A
      FAT: fix some issues in FAT write support code · 8506eb8d
      Anatolij Gustschin 提交于
      Writing a file to the FAT partition didn't work while a
      test using a CF card. The test was done on mpc5200 based
      board (powerpc). There is a number of problems in FAT
      write code:
      
      Compiler warning:
      fat_write.c: In function 'file_fat_write':
      fat_write.c:326: warning: 'counter' may be used uninitialized
      in this function
      fat_write.c:326: note: 'counter' was declared here
      
      'l_filename' string is not terminated, so a file name
      with garbage at the end is used as a file name as shown
      by debug code.
      
      Return value of set_contents() is not checked properly
      so actually a file won't be written at all (as checked
      using 'fatls' after a write attempt with 'fatwrite'
      command).
      
      do_fat_write() doesn't return the number of written bytes
      if no error happened. However the return value of this
      function is used to show the number of written bytes
      in do_fat_fswrite().
      
      The patch adds some debug code and fixes above mentioned
      problems and also fixes a typo in error output.
      
      NOTE: after a successful write to the FAT partition (under
      U-Boot) the partition was checked under Linux using fsck.
      The partition needed fixing FATs:
      -bash-3.2# fsck -a /dev/sda1
      fsck 1.39 (29-May-2006)
      dosfsck 2.11, 12 Mar 2005, FAT32, LFN
      FATs differ but appear to be intact. Using first FAT.
      Performing changes.
      Signed-off-by: NAnatolij Gustschin <agust@denx.de>
      Cc: Donggeun Kim <dg77.kim@samsung.com>
      Cc: Aaron Williams <Aaron.Williams@cavium.com>
      Acked-by: NDonggeun Kim <dg77.kim@samsung.com>
      8506eb8d
    • W
      Merge branch 'master' of /home/wd/git/u-boot/custodians · e2733421
      Wolfgang Denk 提交于
      * 'master' of /home/wd/git/u-boot/custodians:
        common/cmd_pxe.c: Fix GCC 4.6 build warnings
        AVR32: fix timer_init() function
      e2733421
    • W
      Merge branch 'agust@denx.de' of git://git.denx.de/u-boot-staging · c89fd4f0
      Wolfgang Denk 提交于
      * 'agust@denx.de' of git://git.denx.de/u-boot-staging:
        common/cmd_pxe.c: Fix GCC 4.6 build warnings
        AVR32: fix timer_init() function
      c89fd4f0
    • A
      drivers/net/e1000_spi.c: Fix build warnings · deb7282f
      Anatolij Gustschin 提交于
      Fix:
      e1000_spi.c: In function 'spi_free_slave':
      e1000_spi.c:115: warning: unused variable 'hw'
      e1000_spi.c: In function 'do_e1000_spi':
      e1000_spi.c:472: warning: 'checksum' may be used uninitialized in this function
      e1000_spi.c:472: note: 'checksum' was declared here
      Signed-off-by: NAnatolij Gustschin <agust@denx.de>
      Cc: Kyle Moffett <Kyle.D.Moffett@boeing.com>
      Acked-by: NKyle Moffett <Kyle.D.Moffett@boeing.com>
      deb7282f
    • S
      net: Fix arguments in tftpput command · 38bd80b4
      Simon Glass 提交于
      The switch to strict_strtoul() was not done correctly - this fixes
      the compile error when CONFIG_CMD_TFTPPUT is active.
      
      There appears to very little use of strict_strtoul() in commands, so
      it might be easier to switch to simple_strtoul(). If not, we should
      decide the best way to use strict_strtoul(). For now I have added a
      friendly message which will of course increase code size.
      Signed-off-by: NSimon Glass <sjg@chromium.org>
      38bd80b4
    • S
      AVR32: fix timer_init() function · bf0b6313
      Sven Schnelle 提交于
      timer_init() now returns an int (the error code) instead of void.
      This makes compilation fail with:
      
      interrupts.c:111: error: conflicting types for 'timer_init'
      /home/svens/u-boot/u-boot/include/common.h:246: error: previous
      declaration of 'timer_init' was here
      make[1]: *** [interrupts.o] Error 1
      Signed-off-by: NSven Schnelle <svens@stackframe.org>
      Acked-by: NAndreas Bießmann <andreas.devel@googlemail.com>
      Signed-off-by: NAnatolij Gustschin <agust@denx.de>
      bf0b6313
    • H
      common/cmd_pxe.c: Fix GCC 4.6 build warnings · 43d4a5e6
      Heiko Schocher 提交于
      Fix:
      cmd_pxe.c: In function 'parse_pxefile_top':
      cmd_pxe.c:941:5: warning: 'err' may be used uninitialized in this
        function [-Wuninitialized]
      cmd_pxe.c:921:6: note: 'err' was declared here
      Signed-off-by: NHeiko Schocher <hs@denx.de>
      Cc: Jason Hobbs <jason.hobbs@calxeda.com>
      Acked-by: NJason Hobbs <jason.hobbs@calxeda.com>
      43d4a5e6
    • S
      AVR32: fix timer_init() function · c0e6723e
      Sven Schnelle 提交于
      timer_init() now returns an int (the error code) instead of void.
      This makes compilation fail with:
      
      interrupts.c:111: error: conflicting types for 'timer_init'
      /home/svens/u-boot/u-boot/include/common.h:246: error: previous
      declaration of 'timer_init' was here
      make[1]: *** [interrupts.o] Error 1
      Signed-off-by: NSven Schnelle <svens@stackframe.org>
      Acked-by: NAndreas Bießmann <andreas.devel@googlemail.com>
      Signed-off-by: NAnatolij Gustschin <agust@denx.de>
      c0e6723e
  4. 20 12月, 2011 10 次提交
  5. 19 12月, 2011 4 次提交
  6. 18 12月, 2011 2 次提交
    • K
      tools/setlocalversion: Update from the Linux Kernel · a7b39185
      Kyle Moffett 提交于
      Several bugfixes have occurred upstream since this script was imported
      into U-Boot.  In particular, the script currently in U-Boot does not
      describe commit f8bbb4da correctly,
      resulting in a version of "2011.09" instead of "2011.09-01460-gf8bbb4da".
      
      With that commit checked out, the command "git name-rev --tags HEAD"
      gives this result:
        HEAD tags/v2011.12-rc1~30^2
      
      Then the "changes" regex does not match because of the trailing '^2':
        grep -E '^HEAD[[:space:]]+(.*~[0-9]*|undefined)$'
      
      The new version of tools/setlocalversion in the kernel correctly handles
      those situations by using different plumbing commands.
      
      The version from the kernel is not directly usable as it does not append
      the full GIT version that U-Boot expects unless the right config options
      are set (CONFIG_LOCALVERSION_AUTO=y).  Other than a few minor changes
      for Kconfig, the imported version is very similar to Linux v3.2-rc4.
      Signed-off-by: NKyle Moffett <Kyle.D.Moffett@boeing.com>
      Cc: Wolfgang Denk <wd@denx.de>
      Cc: Mike Frysinger <vapier@gentoo.org>
      Acked-by: NMike Frysinger <vapier@gentoo.org>
      a7b39185
    • W
      Merge branch 'master' of git://git.denx.de/u-boot-usb · 8b0affec
      Wolfgang Denk 提交于
      * 'master' of git://git.denx.de/u-boot-usb:
        USB: Use (get|put)_unaligned for accessing wMaxPacketSize
        usb:gadget:s5p Enable the USB Gadget framework at Exynos4210 (C210 Universal)
        README: add documentation for CONFIG_USB_ULPI*
        USB: ULPI: increase error case verbosity
        USB: ULPI: clean a mixup of return types
        USB: ULPI: switch argument type from u8 to unsigned
      8b0affec