25 Sep, 2014

36 commits

  • This option specifies the default Device Tree used for the run-time
    configuration of U-Boot.

    Signed-off-by: Masahiro Yamada
    Cc: Simon Glass
    Cc: Stephen Warren
    Cc: Minkyu Kang
    Cc: Michal Simek

    Masahiro Yamada
     
  • This commit moves:
    CONFIG_OF_CONTROL
    CONFIG_OF_SEPARATE
    CONFIG_OF_EMBED
    CONFIG_OF_HOSTFILE

    Because these options are currently not supported for SPL,
    the "Device Tree Control" menu does not appear in the SPL
    configuration.

    Note:
    zynq-common.h should be adjusted so as not to change the
    default value of CONFIG_SPL_FAT_LOAD_PAYLOAD_NAME.

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass
    Cc: Stephen Warren
    Cc: Minkyu Kang
    Acked-by: Michal Simek

    Masahiro Yamada
     
  • This tools is unnecessary since commit f6c8f38ec601
    (tools/genboardscfg.py: improve performance more with Kconfiglib).

    Signed-off-by: Masahiro Yamada

    Masahiro Yamada
     
  • The mentioned binutils port got removed while the patch was
    pending. As Ian pointed out there is another port providing
    the binutils for arm now. Update the instructions accordingly.

    Cc: ian@FreeBSD.org
    Cc: Tom Rini
    Signed-off-by: Jeroen Hofstee

    Jeroen Hofstee
     
  • The libc headers on FreeBSD and likely related projects as well contain an
    header file, cdefs.h which provides similiar functionality as linux/compiler.h.
    It provides compiler independent defines like __weak __packed, to allow
    compiling with multiple compilers which might have a different syntax for such
    extension.

    Since that header file is included in multiple standard headers, like stddef.h
    and stdarg.h, multiple definitions of those defines will be present if both are
    included. When compiling u-boot the compiler will warn about it hundreds of
    times since e.g. common.h will include both files indirectly.

    commit 7ea50d52849fe8ffa5b5b74c979b60b1045d6fc9 "compiler_gcc: do not redefine
    __gnu_attributes" prevented such redefinitions, but this was undone by commit
    fb8ffd7cfc68b3dc44e182356a207d784cb30b34 "compiler*.h: sync
    include/linux/compiler*.h with Linux 3.16".

    Add the checks back where necessary to prevent such warnings.

    As the original patch this checkpatch warning is ignored:
    "WARNING: Adding new packed members is to be done with care"

    Cc: Masahiro Yamada
    Cc: Tom Rini
    Signed-off-by: Jeroen Hofstee
    Acked-by: Masahiro Yamada

    Jeroen Hofstee
     
  • [1] Move driver/core/, driver/input/ and drivers/input/ entries
    from the top Makefile to drivers/Makefile

    [2] Remove the conditional by CONFIG_DM in drivers/core/Makefile
    because the whole drivers/core directory is already selected
    by CONFIG_DM in the upper level

    [3] Likewise for CONFIG_DM_DEMO in drivers/demo/Makefile

    [4] Simplify common/Makefile - both CONFIG_DDR_SPD and
    CONFIG_SPD_EEPROM are boolean macros so they can directly
    select objects

    Signed-off-by: Masahiro Yamada
    Acked-by: Marek Vasut

    Masahiro Yamada
     
  • Now MIN, MAX, MIN3, MAX are not used.
    Going forward, use min, max, min3, max3.

    Signed-off-by: Masahiro Yamada

    Masahiro Yamada
     
  • The macro MIN, MAX is defined as the aliase of min, max,
    respectively.

    Signed-off-by: Masahiro Yamada

    Masahiro Yamada
     
  • Since clang has a different definition for uninitialized_var
    it will complain that it is redefined in include/compiler.h.
    Since these are already defined in linux/compiler.h just remove
    this instance.

    Cc: Masahiro Yamada
    Cc: Tom Rini
    Signed-off-by: Jeroen Hofstee

    Jeroen Hofstee
     
  • The arg_off() and arg_off_size() update the 'current NAND
    device' variable (dev). This is then used when assigning the
    (nand_info_t*)nand value. Place the assignment after the
    arg_off(_size) calls to prevent using incorrect (nand_info_t*)
    nand value.

    Signed-off-by: Rostislav Lisovy

    Rostislav Lisovy
     
  • The section name and the C variable name seem to be opposite.

    Signed-off-by: Masahiro Yamada
    Cc: Marek Vasut
    Acked-by: Marek Vasut

    Masahiro Yamada
     
  • Now config_cmd_defaults.h is empty so it can be deleted safely.

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass

    Masahiro Yamada
     
  • Since CONFIG_CMD_IMPORTENV is defined in config_cmd_defaults.h,
    it should be enabled for all the boards except bf506f-ezkit
    that undefs it explicitely.

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass
    Tested-by: Simon Glass

    Masahiro Yamada
     
  • Since CONFIG_CMD_GO is defined in config_cmd_defaults.h
    (and no board undefs it its own header), it can be moved to
    Kconfig with the default value "y".

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass
    Tested-by: Simon Glass

    Masahiro Yamada
     
  • Since CONFIG_CMD_EXPORTENV is defined in config_cmd_defaults.h,
    it should be enabled for all the boards except bf506f-ezkit
    that undefs it explicitely.

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass
    Tested-by: Simon Glass

    Masahiro Yamada
     
  • Since CONFIG_CMD_CRC32 is defined in config_cmd_defaults.h,
    it is enabled for all the boards except the ones undefining it
    explicitly:
    kwb
    tseries_mmc
    tseries_nand
    tseries_spi
    vct_platinum_onenand_small
    vct_platinum_small
    vct_platinumavc_onenand_small
    vct_platinumavc_small
    vct_premium_onenand_small
    vct_premium_small

    The default value of this config option should be "y" and
    "# CONFIG_CMD_CRC32 is not set" should be added for those exceptions.

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass

    Masahiro Yamada
     
  • CONFIG_CMD_BOOTM is defined in config_cmd_defaults.h
    which is forcebly included from each board.
    So, the default value of "config CMD_BOOTM" should be "y".

    For some boards undefining it (bf506f-ezkit, controlcenterd_TRAILBLA,
    controlcenterd_TRAILBLAZER_DEVELOP, controlcenterd_TRAILBLAZER),
    "# CONFIG_CMD_BOOTM is not set" should be added to their defconfig.

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass

    Masahiro Yamada
     
  • If this option is enabled, the objects under lib/ directory
    are compiled with speed optimization, not size optimization.
    (Currently, only used by some Blackfin boards.)

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass
    Tested-by: Simon Glass

    Masahiro Yamada
     
  • This would be useful to start moving various config options.

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass
    Tested-by: Simon Glass

    Masahiro Yamada
     
  • Scsi disks need to be probed before we try to access them, otherwise all
    accesses fail with: ** Bad device size - scsi 0 **.

    Reported-by: Karsten Merker
    Signed-off-by: Hans de Goede
    Reviewed-by: Stephen Warren
    Tested-by: Karsten Merker

    Hans de Goede
     
  • Since commit ddaf5c8f3030050fcd356a1e49e3ee8f8f52c6d4
    (patman: RunPipe() should not pipe stdout/stderr unless asked),
    Patman spits lots of "Invalid MAINTAINERS address: '-'"
    error messages for patches with global changes.
    It takes too long for Patman to process them.

    Anyway, "M: -" does not carry any important information.
    Rather, it is just like a place holder in case of assigning
    a new board maintainer. Let's comment out.

    This commit can be reproduced by the following command:

    find . -name MAINTAINERS | xargs sed -i -e '/^M:[[:blank:]]*-$/s/^/#/'

    Signed-off-by: Masahiro Yamada

    Masahiro Yamada
     
  • The "S: Orphan" in MAINTAINERS means that the maintainer in the
    "M:" field is unreachable (i.e. the email address is not working).
    (Refer to the definition of "Orphan" adopted in U-Boot
    in the log of commit 31f1b654b2f395b69faa5d0d3c1eb0803923bd3b,
    "boards.cfg: move boards with invalid emails to Orphan")

    For patch files adding global changes, scripts/get_maintainer.pl
    adds bunch of such invalid email addresses, which results in
    tons of annoying bounce emails.

    This commit can be reproduced by the following command:

    find . -name MAINTAINERS | xargs sed -i -e '
    /^M:[[:blank:]]/ {
    N
    /S:[[:blank:]]Orphan/s/^/#/
    }
    '

    Signed-off-by: Masahiro Yamada
    Acked-by: Simon Glass

    Masahiro Yamada
     
  • We are still keeping invalid email addressed in MAINTAINERS
    because they carry information.

    The problem is that scripts/get_maintainer.pl adds emails in the
    "M:" field including invalid ones.

    We want to comment out invalid email addresses in MAINTAINERS
    to prevent scripts/get_maintainer.pl from picking them up.
    On the other hand, we want to collect them for boards.cfg
    to know the last known maintainer of each board.

    This commit adjusts tools/genboardscfg.py to parse also
    the commented "M:" fields, which is useful for the next commit.

    Signed-off-by: Masahiro Yamada

    Masahiro Yamada
     
  • This performs a command, then repeats it, and checks that the repeat
    happens.

    Signed-off-by: Simon Glass

    Simon Glass
     
  • These cause U-Boot to print a list of available commands. It doesn't break
    the test, but it is best to remove them from the output.

    Signed-off-by: Simon Glass

    Simon Glass
     
  • This is not supported properly on sandbox, and interferes with running
    tests, since when a test script is piped in, some commands will call
    ctrlc() which will drop characters from the test script.

    Signed-off-by: Simon Glass

    Simon Glass
     
  • This was lost sometime in the Kbuild conversion. Add it back.

    Check that the trace test now passes:

    $ ./test/trace/test-trace.sh
    Simple trace test / sanity check using sandbox

    /tmp/filemHKPGw
    Build sandbox
    O=sandbox FTRACE=1
    GEN /home/sjg/c/src/third_party/u-boot/files/sandbox/Makefile
    Configuring for sandbox board...
    Check results
    Test passed

    Signed-off-by: Simon Glass

    Simon Glass
     
  • Cleanup disk/part.c

    Signed-off-by: Pavel Machek

    Pavel Machek
     
  • Each CPU of PowerPC has its default linker script under the CPU
    directory, except mpc8xx.

    Every mpc8xx board has its own linker script under the board
    directory, resulting in lots of duplication of linker scripts.

    I notice eight mpc8xx boards have the same linker script.
    We can decrease the number of linker scripts by putting a single
    default linker script, arch/powerpc/cpu/mpc8xx/u-boot.lds.

    Signed-off-by: Masahiro Yamada
    Cc: Wolfgang Denk
    Acked-by: Stefan Roese

    Masahiro Yamada
     
  • Fix the following build error in case CONFIG_E1000_NO_NVM is enabled:
    CC drivers/net/e1000.o
    drivers/net/e1000.c: In function ‘e1000_initialize’:
    drivers/net/e1000.c:5365:5: error: ‘struct e1000_hw’ has no
    member named ‘eeprom_semaphore_present’
    make[1]: *** [drivers/net/e1000.o] Error 1
    make: *** [drivers/net] Error 2
    Acked-by: Marek Vasut

    Marcel Ziswiler
     
  • When building the U-Boot tools for non-ELF platforms (such as Blackfin
    FLAT), since commit 79fc0c5f498c3982aa4740c273ab1a9255063d9c
    ("tools/env: cross-compile fw_printenv without setting HOSTCC"), the
    build fails because it tries to strip a FLAT binary, which does not
    make sense.

    This commit solves this by changing the stripping logic in
    tools/env/Makefile to be similar to the one in tools/Makefile. This
    logic continues to apply strip to the final binary, but does not abort
    the build if it fails, and does the stripping in place on the final
    binary. This allows the logic to work fine if stripping doesn't work,
    as it leaves the final binary untouched.

    Signed-off-by: Thomas Petazzoni
    Cc: Masahiro Yamada
    Cc: Sonic Zhang
    Reviewed-by: Masahiro Yamada
    Reviewed-by: Sonic Zhang

    Thomas Petazzoni
     
  • - add capability to "fastboot flash" with sparse format images

    Signed-off-by: Steve Rae
    Acked-by: Lukasz Majewski

    Steve Rae
     
  • - update static function
    - additional debugging statements
    - update "fastboot command" information
    - add missing include file
    - update spelling

    Signed-off-by: Steve Rae

    Steve Rae
     
  • - implement 'fastboot flash' for eMMC devices

    Signed-off-by: Steve Rae
    Acked-by: Lukasz Majewski
    Reviewed-by: Marek Vasut

    Steve Rae
     
  • - add support for 'fastboot flash' command for eMMC devices

    Signed-off-by: Steve Rae

    Steve Rae
     
  • Enable the Extend address to support SPI flash more than 16MB.

    Signed-off-by: Hou Zhiqiang
    Reviewed-by: Jagannadha Sutradharudu Teki

    Zhiqiang Hou
     

24 Sep, 2014

4 commits

  • Signed-off-by: Stefan Roese
    Cc: Jagannadha Sutradharudu Teki

    Acked-by: Prafulla Wadaskar
    Tested-by: Luka Perkov
    Reviewed-by: Jagannadha Sutradharudu Teki

    Stefan Roese
     
  • Signed-off-by: Stefan Roese
    Acked-by: Prafulla Wadaskar
    Tested-by: Luka Perkov
    Reviewed-by: Jagannadha Sutradharudu Teki

    Stefan Roese
     
  • This patch introduces the clrsetbits_le32() accessor functions in the
    kirkwood SPI driver. Note that it also includes a fix:

    - writel(~KWSPI_CSN_ACT | KWSPI_SMEMRDY, &spireg->ctrl);
    + writel(KWSPI_SMEMRDY, &spireg->ctrl);

    Here the bit KWSPI_CSN_ACT (0x1) should have been cleared. Instead
    0xfffffffe is written into this control register. This is the main
    reason to use the clrsetbits() functions now. As they make clearing
    bits much less error prone.

    Additionally KWSPI_IRQUNMASK is not used in spi_cs_activate() and
    spi_cs_deactivate() any more. Its the wrong macro but has the same
    value as the correct one (KWSPI_CSN_ACT).

    This is in preparation for use of this driver on the Marvell Armada XP
    platform as well.

    Signed-off-by: Stefan Roese
    Acked-by: Prafulla Wadaskar
    Tested-by: Luka Perkov
    Reviewed-by: Jagannadha Sutradharudu Teki

    Stefan Roese
     
  • Signed-off-by: Stefan Roese
    Tested-by: Luka Perkov
    Reviewed-by: Jagannadha Sutradharudu Teki

    Stefan Roese