Eric Lee / smarc-uboot

08 Jan, 2020

1 commit

  • 4ad4edfe7   cmd_ut: add a parameter prefix to the function cmd_ut_category ... Browse Code »
    12

    There is black magic in the file conftest.py that list
    all the test unit. Then, all those test unit are called
    in pytest. This call is done with the end of the name
    (for example checksum if the full name is bloblist_test_checksum).

    The result is that only test for dm are really executed.
    by pytest, all others tests are listed but never executed.

    This behaviour happens because the dm test unit only check
    the end of the name and others tests checks the full name.

    To fix this issue, I've added a prefix to the function
    cmd_ut_category, and this prefix is removed when looking
    for the unit test.

    Signed-off-by: Philippe Reynes
    Tested-by: Heinrich Schuchardt

    Philippe Reynes
     

12 Aug, 2019

2 commits

  • 3f0d68074   env: Drop the ACTION typedef ... Browse Code »

    Avoid using a typedef here which is unnecessary. Add an 'env_' prefix to
    both the enum and its members to make it clear that these are related to
    the environment.

    Add an ENV prefix to these two flags so that it is clear what they relate
    to. Also move them to env.h since they are part of the public API. Use an
    enum rather than a #define to tie them together.

    Signed-off-by: Simon Glass

    Simon Glass
     
  • dd2408cac   env: Drop the ENTRY typedef ... Browse Code »

    U-Boot is not supposed to use typedef for structs anymore. Also this name
    is the same as the ENTRY() macro used in assembler files, and 'entry'
    itself is widely used in U-Boot (>8k matches).

    Drop the typedef and rename the struct to env_entry to reduce confusion.

    Signed-off-by: Simon Glass
    Acked-by: Joe Hershberger

    Simon Glass
     

24 Apr, 2019

1 commit

09 Feb, 2019

1 commit

  • 9dfdbd9f0   hashtable: fix environment variable corruption ... Browse Code »

    Only first previously deleted entry was recognized, leading hsearch_r
    to think that there was no previously deleted entry. It then conluded
    that a free entry was found, even if there were no free entries and it
    overwrote a random entry.

    This patch makes sure all deleted or free entries are always found and
    also introduces constants for the 0 and -1 numbers. Unit tests to excersise a
    simple hash table usage and catch the corruption were added.

    To trash your environment, simply run this loop:

    setenv i 0
    while true; do
    setenv v_$i $i
    setenv v_$i
    setexpr i $i + 1
    done

    Signed-off-by: Roman Kapl

    Roman Kapl
     

07 May, 2018

1 commit

  • 83d290c56   SPDX: Convert all of our single license tags to Linux Kernel style ... Browse Code »
    56

    When U-Boot started using SPDX tags we were among the early adopters and
    there weren't a lot of other examples to borrow from. So we picked the
    area of the file that usually had a full license text and replaced it
    with an appropriate SPDX-License-Identifier: entry. Since then, the
    Linux Kernel has adopted SPDX tags and they place it as the very first
    line in a file (except where shebangs are used, then it's second line)
    and with slightly different comment styles than us.

    In part due to community overlap, in part due to better tag visibility
    and in part for other minor reasons, switch over to that style.

    This commit changes all instances where we have a single declared
    license in the tag as both the before and after are identical in tag
    contents. There's also a few places where I found we did not have a tag
    and have introduced one.

    Signed-off-by: Tom Rini

    Tom Rini
     

03 Dec, 2017

1 commit

21 May, 2015

3 commits