- Dec 15, 2024
-
-
Most Rockchip device tree related bindings are converted to YAML and available in the U-boot /dts/upstream/Bindings/ directory. Remove all redundant U-boot entries. Signed-off-by:
Johan Jonker <jbx6244@gmail.com>
-
Although it has historically been different, the current standard spelling of the neutral singular possessive pronoun is "its". Signed-off-by:
J. Neuschäfer <j.ne@posteo.net>
-
Cosmetic change. Signed-off-by:
J. Neuschäfer <j.ne@posteo.net>
-
- Dec 12, 2024
-
-
-
https://source.denx.de/u-boot/custodians/u-boot-microblazeTom Rini authored
AMD/Xilinx changes for v2025.01-rc5 - Fix reset issue for SOM
-
The boot_fdt_add_mem_rsv_regions() function can be called twice, e.g. first time during the board init (as a part of LMB init), and then when booting the OS with 'booti' command: lmb_add_region_flags lmb_reserve_flags boot_fdt_reserve_region boot_fdt_add_mem_rsv_regions ^ | +-----------------------+ | (1) | (2) lmb_reserve_common image_setup_linux lmb_init ... initr_lmb do_booti board_init_r 'booti' That consequently leads to the attempt of reserving the same memory areas (described in the 'reserved-memory' dts node) in LMB. The lmb_add_region_flags() returns -EEXIST error code in such cases, but boot_fdt_reserve_region() handles all negative error codes as a failure to reserve fdt memory region, printing corresponding error messages, which are essentially harmless, but misleading. For example, this is the output of 'booti' command on E850-96 board: => booti $loadaddr - $fdtaddr ... ERROR: reserving fdt memory region failed (addr=bab00000 size=5500000 flags=2) ERROR: reserving fdt memory region failed (addr=f0000000 size=200000 flags=4) ... Starting kernel ... The mentioned false positive error messages are observed starting with commit 1d9aa4a2 ("lmb: Fix the allocation of overlapping memory areas with !LMB_NONE"), which removes the check for the already added memory regions in lmb_add_region_flags(), making it return -1 for !LMB_NONE cases. Another commit 827dee58 ("fdt: lmb: add reserved regions as no-overwrite") changes flags used for reserving memory in boot_fdt_add_mem_rsv_regions() from LMB_NONE to LMB_NOOVERWRITE. So together with the patch mentioned earlier, it makes lmb_add_region_flags() return -1 when called from boot_fdt_reserve_region(). Since then, the different patch was implemented, returning -EEXIST error code in described cases, which is: lmb: Return -EEXIST in lmb_add_region_flags() if region already added Handle -EEXIST error code as a normal (successful) case in lmb_reserve_flags() and don't print any messages. Fixes: 1d9aa4a2 ("lmb: Fix the allocation of overlapping memory areas with !LMB_NONE") Signed-off-by:
Sam Protsenko <semen.protsenko@linaro.org> Reviewed-by:
Ilias Apalodimas <ilias.apalodimas@linaro.org> Tested-by:
Michal Simek <michal.simek@amd.com>
-
An attempt to add the already added LMB region using lmb_add_region_flags() ends up in lmb_addrs_overlap() check, which eventually leads to either returning 0 if 'flags' is LMB_NONE, or -1 otherwise. It makes it impossible for the user of this function to catch the case when the region is already added and differentiate it from regular errors. That in turn may lead to incorrect error handling in the caller code, like reporting misleading errors or interrupting the normal code path where it could be treated as the normal case. An example is boot_fdt_reserve_region() function, which might be called twice (e.g. during board startup in initr_lmb(), and then during 'booti' command booting the OS), thus trying to reserve exactly the same memory regions described in the device tree twice, which produces an error message on second call. Return -EEXIST error code in case when the added region exists and it's not LMB_NONE; for LMB_NONE return 0, to conform to unit tests (specifically test_alloc_addr() in test/lib/lmb.c) and the preferred behavior described in commit 1d9aa4a2 ("lmb: Fix the allocation of overlapping memory areas with !LMB_NONE"). The change of lmb_add_region_flags() return values is described in the table below: Return case Pre-1d9 1d9 New ----------------------------------------------------------- Added successfully 0 0 0 Failed to add -1 -1 -1 Already added, flags == LMB_NONE 0 0 0 Already added, flags != LMB_NONE 0 -1 -EEXIST Rework all affected functions and their documentation. Also fix the corresponding unit test which checks reserving the same region with the same flags to account for the changed return value. No functional change is intended (by this patch itself). Fixes: 1d9aa4a2 ("lmb: Fix the allocation of overlapping memory areas with !LMB_NONE") Signed-off-by:
Sam Protsenko <semen.protsenko@linaro.org> Reviewed-by:
Ilias Apalodimas <ilias.apalodimas@linaro.org> Tested-by:
Michal Simek <michal.simek@amd.com>
-
CONFIG_USB_XHCI_DWC3 is not required for AM62x as the XHCI driver is registered through the dwc3-generic driver. CONFIG_USB_XHCI_DWC3 causes problems by hijacking the USB controller even if it is not set for Host mode in device tree. 'dm tree' output after 'usb start' is fixed from simple_bus 5 [ + ] dwc3-am62 | |-- dwc3-usb@f900000 usb_gadget 0 [ ] dwc3-generic-periphe | | |-- usb@31000000 usb 0 [ + ] xhci-dwc3 | | `-- usb@31000000 usb_hub 0 [ + ] usb_hub | | `-- usb_hub simple_bus 6 [ + ] dwc3-am62 | |-- dwc3-usb@f910000 usb 1 [ + ] dwc3-generic-host | | |-- usb@31100000 usb_hub 1 [ + ] usb_hub | | | `-- usb_hub usb 1 [ + ] xhci-dwc3 | | `-- usb@31100000 usb_hub 2 [ + ] usb_hub | | `-- usb_hub [notice that 'xhci-dwc3' and 'usb_hub' drivers are probed for both USB instances although the first instance is supposed to be 'peripheral' only] to simple_bus 5 [ ] dwc3-am62 | |-- dwc3-usb@f900000 usb_gadget 0 [ ] dwc3-generic-periphe | | `-- usb@31000000 simple_bus 6 [ + ] dwc3-am62 | |-- dwc3-usb@f910000 usb 1 [ + ] dwc3-generic-host | | `-- usb@31100000 usb_hub 0 [ + ] usb_hub | | `-- usb_hub Fixes: dfc2dff5 ("configs: am62x_evm_*: Enable USB and DFU support") Signed-off-by:
Roger Quadros <rogerq@kernel.org> Reviewed-by:
Siddharth Vadapalli <s-vadapalli@ti.com>
-
- Dec 11, 2024
-
-
-
https://source.denx.de/u-boot/custodians/u-boot-raspberrypiTom Rini authored
Updates for RPi for 2025.01: - Enable capsule updates - Pass CMA through from firmware DT
-
Since RPI works well using EFI and has no size limitations with regards to U-Boot, add the needed structures to support capsule updates. While at it update the most commonly used defconfigs and include capsule support and U-Boot commands needed by EFI Tested-by:
Sughosh Ganu <sughosh.ganu@linaro.org> Signed-off-by:
Ilias Apalodimas <ilias.apalodimas@linaro.org>
-
Michal Simek authored
In Kria SOM configuration DP is under reset and access to DP is causing hang that's why call reset at probe to avoid this situation. Signed-off-by:
Michal Simek <michal.simek@amd.com> Link: https://lore.kernel.org/r/0504474a91a9839828aecd37f8855fd154cdf2e1.1733742708.git.michal.simek@amd.com
-
Peter Robinson authored
For a lot of usecases, such as display, camera, media the Raspberry Pi needs a lot more CMA than distros configure as default so we should pass this parameter through so things work as expected. Fix a spelling mistake while we're at it. Signed-off-by:
Peter Robinson <pbrobinson@gmail.com> Reviewed-by:
Simon Glass <sjg@chromium.org> Acked-by:
Matthias Brugger <mbrugger@suse.com>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Comment changes only, no functional change. Reviewed-by:
Jaehoon Chung <jh80.chung@samsung.com> Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Kconfig strings and comment changes only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Marek Vasut authored
The correct spelling is R-Car, including the dash, update the usage. Comment change only, no functional change. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
As the "peer-hub" property is optional, don't error out just skip the bind function. Fixes: 57e30b09 ("usb: onboard-hub: Bail out if peer hub is already probed") Signed-off-by:
Venkatesh Yadav Abbarapu <venkatesh.abbarapu@amd.com> Acked-by:
Michal Simek <michal.simek@amd.com> Reviewed-by:
Patrice Chotard <patrice.chotard@foss.st.com> Tested-by:
Patrice Chotard <patrice.chotard@foss.st.com>
-
- Dec 09, 2024
-
-
Tom Rini authored
Signed-off-by:
Tom Rini <trini@konsulko.com>
-
- Dec 08, 2024
-
-
Marek Vasut authored
The U-Boot is currently not capable of handling ethernet-phy-ieee802.3-c45 PHYs correctly, and also does not handle MDIO bus wide reset-gpios property. Until proper C45 PHY support lands in U-Boot, disable AVB1/AVB2 interfaces. This only disables the two MACs with 88Q2110/88Q2112 100/1000BASE-T1 PHYs on ethenet sub-board, the main board AVB0 ethernet is unaffected. Signed-off-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Paul Barker authored
In rzg2l_pinconf_set(), there are no new variables defined in the case statement for PIN_CONFIG_INPUT_ENABLE so no additional scope is needed. Signed-off-by:
Paul Barker <paul.barker.ct@bp.renesas.com> Reviewed-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Paul Barker authored
On the RZ/G2L SoC family, the direction of the Ethernet TXC/TX_CLK signal is selectable to support an Ethernet PHY operating in either MII or RGMII mode. By default, the signal is configured as an input and MII mode is supported. The ETH_MODE register can be modified to configure this signal as an output to support RGMII mode. As this signal is be default an input, and can optionally be switched to an output, it maps neatly onto an `output-enable` property in the device tree. Signed-off-by:
Paul Barker <paul.barker.ct@bp.renesas.com> Reviewed-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Paul Barker authored
The Ethenet interfaces on the Renesas RZ/G2L SoC family can operate at multiple power supply voltages: 3.3V (default value), 2.5V and 1.8V. rzg2l_pinconf_set() is extended to support the 2.5V setting, with a check to ensure this is only used on Ethernet interfaces as it is not supported on the SD & QSPI interfaces. While we're modifying rzg2l_pinconf_set(), drop the unnecessary default value for pwr_reg as it is set in every branch of the following if condition. Signed-off-by:
Paul Barker <paul.barker.ct@bp.renesas.com> Reviewed-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Paul Barker authored
We can call dev_read_u32_default() instead of calling fdt_getprop() then fdt32_to_cpu(). Signed-off-by:
Paul Barker <paul.barker.ct@bp.renesas.com> Reviewed-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
Paul Barker authored
In the RZ/G2L family, core clocks are always on and can't be disabled. However, drivers which are shared with other SoCs may call clk_enable() or clk_enable_bulk() for a clock referenced in the device tree which happens to be a core clock on the RZ/G2L. To avoid the need for conditionals in these drivers, simply ignore attempts to enable a core clock. Signed-off-by:
Paul Barker <paul.barker.ct@bp.renesas.com> Reviewed-by:
Marek Vasut <marek.vasut+renesas@mailbox.org>
-
- Dec 06, 2024
-
-
When MbedTLS TLS features were added MBEDTLS_HAVE_TIME was defined as part of enabling https:// support. However that pointed to the wrong function which could crash if it received a NULL pointer. Looking closer that function is not really needed, as it only seems to increase the RNG entropy by using 4b of the current time and date. The reason that was enabled is that lwIP was unconditionally requiring it, although it's configurable and can be turned off. Since lwIP doesn't use that field anywhere else, make it conditional and disable it from our config. Fixes: commit a564f509 ("mbedtls: Enable TLS 1.2 support") Reported-by:
Heinrich Schuchardt <heinrich.schuchardt@canonical.com> Signed-off-by:
Ilias Apalodimas <ilias.apalodimas@linaro.org> Reviewed-by:
Heinrich Schuchardt <xypron.glpk@gmx.de> Acked-by:
Jerome Forissier <jerome.forissier@linaro.org>
-
Add a missing comma in the update_info structure declaration. This fixes the following build error when building with EFI_RUNTIME_UPDATE_CAPSULE or EFI_CAPSULE_ON_DISK: board/emulation/qemu-arm/qemu-arm.c:52:9: error: request for member ‘images’ in something not a structure or union Fixes: cccea188 ("efi_loader: add the number of image entries in efi_capsule_update_info") Signed-off-by:
Vincent Stehlé <vincent.stehle@arm.com> Cc: Masahisa Kojima <masahisa.kojima@linaro.org> Cc: Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi> Cc: Tom Rini <trini@konsulko.com>
-
Building SPL fails with MBEDTLS enabled. Currently we don't need it there. Signed-off-by:
Heinrich Schuchardt <heinrich.schuchardt@canonical.com> Acked-by:
Jerome Forissier <jerome.forissier@linaro.org>
-
Building qemu_arm64_defconfig with CONFIGS_DM_RTC=n and CONFIGS_RTC_PL031=y leads to a build failure. Adjust the vexpress64 configuration to avoid circular dependency. Signed-off-by:
Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
-
There are platforms which set the value of ram_top based on certain restrictions that the platform might have in accessing memory above ram_top, even when the memory region is in the same DRAM bank. So, even though the LMB allocator works as expected, when trying to allocate memory above ram_top, prohibit this by marking all memory above ram_top as reserved, even if the said memory region is from the same bank. Signed-off-by:
Sughosh Ganu <sughosh.ganu@linaro.org> Tested-by:
Andreas Schwab <schwab@suse.de>
-