15 Nov, 2018

18 commits

  • Add a driver for gdsys IHS (Integrated Hardware Systems) FPGAs, which
    supports initialization of the FPGA, as well as information gathering.

    Reviewed-by: Simon Glass
    Signed-off-by: Mario Six

    Mario Six
     
  • This patch adds a driver for the bus associated with a IHS FPGA.

    Reviewed-by: Simon Glass
    Signed-off-by: Mario Six

    Mario Six
     
  • Makefile entries should be sorted.

    Reviewed-by: Anatolij Gustschin
    Reviewed-by: Simon Glass
    Signed-off-by: Mario Six
    Signed-off-by: Anatolij Gustschin

    Mario Six
     
  • ofnode_read_simple_addr_cells may fail and return a negative error code.
    Check for this when initializing regmaps.

    Also check if both_len is zero, since this is perfectly possible, and
    would lead to a division-by-zero further down the line.

    Reviewed-by: Anatolij Gustschin
    Reviewed-by: Simon Glass
    Signed-off-by: Mario Six

    Mario Six
     
  • Document the regmap_alloc() function.

    Reviewed-by: Anatolij Gustschin
    Reviewed-by: Simon Glass
    Signed-off-by: Mario Six

    Mario Six
     
  • The documentation in regmap.h is not in kernel-doc format. Correct this.

    Reviewed-by: Anatolij Gustschin
    Reviewed-by: Simon Glass
    Signed-off-by: Mario Six

    Mario Six
     
  • The upcoming changes to the regmap interface will contain a proper check
    for plausibility when reading/writing from/to a register map. To still
    have the current tests pass, increase the size of the memory region for
    the syscon0 device, since one of the tests reads and writes beyond this
    range.

    Reviewed-by: Anatolij Gustschin
    Reviewed-by: Simon Glass
    Signed-off-by: Mario Six

    Mario Six
     
  • This is currently out of order. Sort it.

    Signed-off-by: Bin Meng
    Reviewed-by: Simon Glass

    Bin Meng
     
  • Now that we fixed the pre-relocation driver binding for driver marked
    with DM_FLAG_PRE_RELOC flag, add a test case to cover that scenario.

    Signed-off-by: Bin Meng
    Reviewed-by: Simon Glass

    Bin Meng
     
  • Commit f2006808f099: ("dm: core: parse chosen node") added a logic
    to parse the chosen node during dm_scan_fdt_node(), but unfortunately
    it missed adding the same logic in dm_scan_fdt_live(). This mirrors
    the logic in the livetree version.

    The weird thing is that commit f2006808f099 did update the test case
    to test such logic, but even if I reset to that commit, the test case
    still fails, and I have no idea how it could pass.

    With this fix, the following 2 test cases now pass:

    Test: dm_test_bus_children: bus.c
    test/dm/bus.c:112, dm_test_bus_children(): num_devices ==
    list_count_items(&uc->dev_head): Expected 7, got 6

    Test: dm_test_fdt: test-fdt.c
    test/dm/test-fdt.c:184, dm_test_fdt(): num_devices ==
    list_count_items(&uc->dev_head): Expected 7, got 6

    Fixes: f2006808f099 ("dm: core: parse chosen node")
    Signed-off-by: Bin Meng
    Reviewed-by: Simon Glass

    Bin Meng
     
  • The pre_reloc_only parameter description currently only mentions
    drivers with the DM_FLAG_PRE_RELOC flag, but does not mention the
    special device tree properties. Correct them.

    Signed-off-by: Bin Meng
    Reviewed-by: Simon Glass

    Bin Meng
     
  • Currently the comments of several APIs (eg: dm_init_and_scan()) say:

    @pre_reloc_only: If true, bind only drivers with the DM_FLAG_PRE_RELOC
    flag. If false bind all drivers.

    The 'Pre-Relocation Support' chapter in doc/driver-model/README.txt
    documents the same that both device tree properties and driver flag
    are supported.

    However the implementation only checks these special device tree
    properties without checking the driver flag at all. This updates
    lists_bind_fdt() to consider both scenarios.

    Signed-off-by: Bin Meng
    Reviewed-by: Simon Glass
    Squashed in http://patchwork.ozlabs.org/patch/996473/ :
    Signed-off-by: Simon Glass

    Bin Meng
     
  • This adds a new API dm_ofnode_pre_reloc(), a livetree equivalent
    API of dm_fdt_pre_reloc().

    Signed-off-by: Bin Meng
    Reviewed-by: Simon Glass

    Bin Meng
     
  • The description string should not contain unnecessary characters,
    like the ending '\n' or the leading 'CPU:'.

    Signed-off-by: Bin Meng
    Reviewed-by: Simon Glass

    Bin Meng
     
  • It was observed that current output of print_cpuinfo() on QEMU
    x86 targets does not have an ending '\n', neither have a leading
    'CPU:' any more. However it used to have these before.

    It turns out commit c0434407b595 introduced a unified DM version
    of print_cpuinfo() that exposed such issue on QEMU x86.

    Fixes: c0434407b595 ("board_f: Use static print_cpuinfo if CONFIG_CPU is active")
    Signed-off-by: Bin Meng
    Reviewed-by: Simon Glass

    Bin Meng
     
  • When the DM CPU drivers are active, printing information about a CPU
    should be delegated to a matching driver.

    Hence, add a static print_cpuinfo that implements this delegation when
    DM CPU drivers are active.

    Reviewed-by: Simon Glass
    Signed-off-by: Mario Six
    Changed condition to CONFIG_IS_ENABLED(CPU):
    Signed-off-by: Simon Glass

    Mario Six
     
  • This brings in the u-boot-net PR from Joe.

    Tom Rini
     
  • Signed-off-by: Tom Rini

    Tom Rini
     

14 Nov, 2018

4 commits


13 Nov, 2018

5 commits

  • Commit 9c5b00973bce ("Convert CONFIG_MTD_PARTITIONS et al to Kconfig")
    introduced a publicly visible Kconfig entry for the
    CONFIG_MTD_PARTITIONS option, while the rework on MTD partitioning
    was in progress, and we somehow did not notice that the same Kconfig
    entry was added by commit 4048a5c519a8 ("mtd: declare MTD_PARTITIONS
    symbol in Kconfig"), but this time as an invisible entry (this can
    only be selected by other options).

    Keep the non-visible version of this symbol, since MTD_PARTITIONS is
    not something the user should be able to enable/disable directly.

    Fixes: 4048a5c519a8 ("mtd: declare MTD_PARTITIONS symbol in Kconfig")
    Signed-off-by: Boris Brezillon
    Reviewed-by: Lukasz Majewski
    Reviewed-by: Jagan Teki

    Boris Brezillon
     
  • Commit c58fb2cdb3e4 ("cmd: ubi: clean the partition handling")
    introduced a call to mtd_probe_devices() in the ubi_attach() path
    and this function takes care of parsing mtdparts/mtdids and
    creating/registering the associated mtd partitions.

    The mtdparts_init() call in the ubi_detach() path is not only
    unnecessary but can sometimes print error messages even when things
    work properly (that's the case with SPI NAND devices that have not
    been probed with 'mtd list'), which is misleading.

    Remove this call to mtdparts_init() and drop the dependency on
    CMD_MTDPARTS.

    Fixes: c58fb2cdb3e4 ("cmd: ubi: clean the partition handling")
    Reported-by: Stefan Roese
    Signed-off-by: Boris Brezillon
    Tested-by: Stefan Roese
    Reviewed-by: Lukasz Majewski
    Reviewed-by: Jagan Teki

    Boris Brezillon
     
  • gwventana configs are relying on CMD_UBI to select CMD_MTDPARTS,
    which is then making {MTDIDS,MTDPARTS}_DEFAULT options available.

    We are about to remove the 'select CMD_MTDPARTS' statement in the
    CMD_UBI entry, but if we do that without first making sure
    {MTDIDS,MTDPARTS}_DEFAULT are visible, we end up with a build
    failure when building gwventana configs.

    Address that by adding a depends on MTD_PARTITIONS to
    {MTDIDS,MTDPARTS}_DEFAULT which does the trick since CMD_UBI selects
    MTD_UBI which in turn selects MTD_PARTITIONS.

    We also get rid of the depends on CMD_MTD, since CMD_MTD also selects
    MTD_PARTITIONS.

    Reported-by: Jagan Teki
    Signed-off-by: Boris Brezillon
    Reviewed-by: Lukasz Majewski
    Reviewed-by: Jagan Teki

    Boris Brezillon
     
  • dfu_fill_entity_nand() uses find_dev_and_part() and mtdparts_init()
    which are provided by cmd/mtdparts.c.

    Add the dependency to avoid build failures when CMD_MTDPARTS is not
    selected.

    Reported-by: Jagan Teki
    Fixes: 6828e602b722d ("dfu: Migrate to Kconfig")
    Signed-off-by: Boris Brezillon
    Reviewed-by: Lukasz Majewski
    Reviewed-by: Jagan Teki

    Boris Brezillon
     
  • U-boot provides a mean to define default values for mtdids and mtdparts
    when they're not defined in the environment. Patch mtd_probe_devices()
    to use those default values when env_get("mtdparts") or
    env_get("mtdids") return NULL.

    This implementation is based on the logic found in cmd/mtdparts.c.

    Fixes: 5db66b3aee6f ("cmd: mtd: add 'mtd' command")
    Reported-by: Stefan Roese
    Signed-off-by: Boris Brezillon
    Tested-by: Stefan Roese
    Reviewed-by: Lukasz Majewski
    Reviewed-by: Jagan Teki

    Boris Brezillon
     

12 Nov, 2018

5 commits


10 Nov, 2018

2 commits

  • This patch adds check for command response CRC failure. The driver
    is currently ignoring CRC check failure on command resposes which
    have CRC atteched to it, which can be potentially dangerous. Even
    more grueling problem happens when the command response is followed
    by data transfer though, as in that case, the dwmci_data_transfer()
    function will spin until it reaches the 240s timeout.

    Signed-off-by: Marek Vasut
    Cc: Heiko Stuebner
    Cc: Philipp Tomsich

    Marek Vasut
     
  • Build ymodem only on need to shrink spl image size.

    Signed-off-by: Jun Nie

    Jun Nie
     

09 Nov, 2018

5 commits


08 Nov, 2018

1 commit