- Nov 14, 2018
-
-
A newly created property is currently not freed if a name could not be allocated. This patch fixes the resulting memory leak in the error patch. Reported-by: Coverity (CID: 184085) Fixes: e369e58d ("core: Add functions to set properties in live-tree") Signed-off-by:
Mario Six <mario.six@gdsys.cc>
-
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 <sjg@chromium.org> Signed-off-by:
-
This patch adds a driver for the bus associated with a IHS FPGA. Reviewed-by:
-
Makefile entries should be sorted. Reviewed-by:
Anatolij Gustschin <agust@denx.de> Reviewed-by:
-
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:
-
Document the regmap_alloc() function. Reviewed-by:
-
The documentation in regmap.h is not in kernel-doc format. Correct this. Reviewed-by:
-
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:
-
This is currently out of order. Sort it. Signed-off-by:
Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: -
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 <bmeng.cn@gmail.com>
Reviewed-by: -
Commit f2006808: ("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 f2006808 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: f2006808 ("dm: core: parse chosen node") Signed-off-by:
Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: -
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 <bmeng.cn@gmail.com>
Reviewed-by: -
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 <bmeng.cn@gmail.com>
Reviewed-by: -
This adds a new API dm_ofnode_pre_reloc(), a livetree equivalent API of dm_fdt_pre_reloc(). Signed-off-by:
Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: -
The description string should not contain unnecessary characters, like the ending '\n' or the leading 'CPU:'. Signed-off-by:
Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: -
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 c0434407 introduced a unified DM version of print_cpuinfo() that exposed such issue on QEMU x86. Fixes: c0434407 ("board_f: Use static print_cpuinfo if CONFIG_CPU is active") Signed-off-by:
Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: -
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:
-
Tom Rini authored
This brings in the u-boot-net PR from Joe.
-
Tom Rini authored
Signed-off-by:Tom Rini <trini@konsulko.com>
-
Commit fb01e07a accidentally broke initialisation of GPIO descriptor flags from device tree: currently the active low flag from gpio-specifier is always ignored. Fix it. Signed-off-by:
-
Tom Rini authored
Now that CMD_UBI does not select CMD_MTDPARTS we need to make platforms that had been enabling it turn it on by hand. This exposed that we had not yet migrated CMD_MTDPARTS fully, so do so now. Fixes: 86dfa556 ("cmd: ubi: Remove useless call to mtdparts_init()") Signed-off-by:
-
git://git.denx.de/u-boot-spiTom Rini authored
-
Tom Rini authored
Rsync all defconfig files using moveconfig.py Signed-off-by:
-
- Nov 13, 2018
-
-
Commit 9c5b0097 ("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 4048a5c5 ("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: 4048a5c5 ("mtd: declare MTD_PARTITIONS symbol in Kconfig") Signed-off-by:
Boris Brezillon <boris.brezillon@bootlin.com> Reviewed-by:
Lukasz Majewski <lukma@denx.de> Reviewed-by:
Jagan Teki <jagan@openedev.com>
-
Commit c58fb2cd ("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: c58fb2cd ("cmd: ubi: clean the partition handling") Reported-by:
Stefan Roese <sr@denx.de> Signed-off-by:
-
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 <jagan@amarulasolutions.com> Signed-off-by:
-
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:
-
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: 5db66b3a ("cmd: mtd: add 'mtd' command") Reported-by:
-
- Nov 12, 2018
-
-
-
The make macro to check if the binary exceeds the board size limit is taken straight from the root Makefile. Without this and e.g. enabled EFI Vybrid fails booting as the regular size limit check does not take the final u-boot.imx binary size into account which is bigger due to alignment as well as IMX header stuff. Signed-off-by:
Marcel Ziswiler <marcel.ziswiler@toradex.com> Reviewed-by:
Fabio Estevam <festevam@gmail.com>
-
Drop SPI support saving precious 4 Kb on boards with tough size restrictions. Signed-off-by:
Marcel Ziswiler <marcel@ziswiler.com>
-
Unset CONFIG_CMDLINE_EDITING saving precious 4 Kb on boards with tough size restrictions. Signed-off-by: -
Unset CONFIG_EFI_UNICODE_CAPITALIZATION on boards with tough size restrictions. This is analogous to commit a90bf07a ("efi_loader: unset CONFIG_EFI_UNICODE_CAPITALIZATION"). Signed-off-by:
Alexander Graf <agraf@suse.de>
-
- Nov 10, 2018
-
-
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 <marex@denx.de> Cc: Heiko Stuebner <heiko@sntech.de> Cc: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
-
- Nov 09, 2018
-
-
Build ymodem only on need to shrink spl image size. Signed-off-by:Jun Nie <jun.nie@linaro.org>
-
common/common_fit.c is including <spl.h>, but not actually using it. The inclusion will cuase compile error on platforms using CONFIG_OF_SEPARATE and not SPL. Signed-off-by:
Lars Povlsen <lars.povlsen@microsemi.com> Reviewed-by:
Jean-Jacques Hiblot <jjhiblot@ti.com>
-
The commit 484fdf5b ("dm: Add support for all targets which requires MANUAL_RELOC") introduces subtle typo, i.e. missed semicolon. Fixes: 484fdf5b ("dm: Add support for all targets which requires MANUAL_RELOC") Cc: Michal Simek <michal.simek@xilinx.com> Signed-off-by:
Andy Shevchenko <andriy.shevchenko@linux.intel.com> Reviewed-by:
Michal Simek <michal.simek@xilinx.com>
-
Tiny printf does not support %.*s and %lX. Since tiny printf should be very common in SPL, replace these by %32s (for printing image name) and %lx. Signed-off-by:
Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com> Reviewed-by:
-
Add mailing list for stm32mp architecture and board. Signed-off-by:Patrick Delaunay <patrick.delaunay@st.com>
-
Testing has shown that a line-break is missing in one debug line in fit_find_config_node(). Signed-off-by:
-
