On 8/4/23 17:53, Alexander Dahl wrote:
Function 'cmd_usage()' already prints one command in usage before
printing out the help text given to the U_BOOT_CMD_WITH_SUBCMDS macro.
Wrong previous output:
Usage:
cyclic cyclic demo - register cyclic demo
function
cyclic list - list c
Hello Teresa,
On Tue, 2023-07-18 at 15:35 +0200, Teresa Remmet wrote:
> Due to PCB layout constraints in PCB revisions until including
> 1549.2,
> a RAM frequency of 2 GHz can cause rare instabilities. Set the RAM
> frequency to 1.5 GHz to achieve a stable system under all conditions.
>
> Signed-
Since the Patch 55171aedda88, VisionFive2 booting has been broken [1].
VisionFive2 board requires to enable CONFIG_TIMER_EARLY but booting went
to panic from initr_dm_devices due to lack of a timer device.
- Error logs
initcall sequence fffd8d38 failed at call 402185e4
(err=-19)
W
Hello Akashi-san,
> From: AKASHI Takahiro
> Sent: Monday, August 7, 2023 12:03 PM
>
> Hi Etienne,
>
> On Thu, Aug 03, 2023 at 09:24:51AM +, Etienne CARRIERE wrote:
> > Hello Takahiro-san,
> >
> > Sorry again for this late feedback.
> > Testing the series against stm32mp135f-dk board which is
From: Frank Wunderlich
Add Bananapi R2 Pro board.
Signed-off-by: Frank Wunderlich
---
because iodomain is different to evb and now iodomain driver is sent as
patch we need to separate between EVB and R2Pro else board can be bricked.
---
v2:
- drop switch-node for now as u-boot driver works diff
Hi Jonas,
On 8/3/23 23:11, Jonas Karlman wrote:
A RK8XX PMIC is typically using i2c0 on RK356x devices. Add bootph-all
to required pinctrl nodes to simplify use of the prevent booting on
power plug-in option in SPL.
I'd say this should be added to device-specific DTSes whenever
appropriate a
On Thu, 3 Aug 2023 at 22:21, Maxim Uvarov wrote:
>
>
> On Thu, 3 Aug 2023 at 03:32, Simon Glass wrote:
>
>> Hi Maxim,
>>
>> On Wed, 2 Aug 2023 at 08:11, Maxim Uvarov
>> wrote:
>> >
>>
>> subject: U-Boot
>>
>> commit message please (throughout series)
>>
>> > Signed-off-by: Maxim Uvarov
>> > --
In case CC_OPTIMIZE_FOR_DEBUG is set, unused code will not be optimized out,
hence the reference to fpga_load will be compiled.
if DM_FPGA and SPL_FPGA are not set, the build will fail with :
aarch64-none-linux-gnu-ld.bfd: common/spl/spl_fit.o: in function
`spl_fit_upload_fpga':
u-boot/common/spl
From: Ricardo Salveti
Linux microPlatform uses an rngb device in optee-os in boot scheme
SPL -> OPTEE -> U-Boot. To make rngb available for optee-os, enable
it in SPL.
Signed-off-by: Ricardo Salveti
Co-developed-by: Oleksandr Suvorov
Signed-off-by: Oleksandr Suvorov
---
arch/arm/dts/imx6ull
Hi Frank,
On 8/8/23 11:09, Frank Wunderlich wrote:
From: Frank Wunderlich
Add Bananapi R2 Pro board.
Signed-off-by: Frank Wunderlich
---
because iodomain is different to evb and now iodomain driver is sent as
patch we need to separate between EVB and R2Pro else board can be bricked.
---
v2:
Hi
thx for first look
> Gesendet: Dienstag, 08. August 2023 um 13:18 Uhr
> Von: "Eugen Hristev"
> Hi Frank,
>
> On 8/8/23 11:09, Frank Wunderlich wrote:
> > From: Frank Wunderlich
> >
> > Add Bananapi R2 Pro board.
> >
> > Signed-off-by: Frank Wunderlich
> > ---
> > because iodomain is differe
On Thu, 3 Aug 2023 at 14:56, Ilias Apalodimas
wrote:
> Hi Maxim
>
> On Wed, Aug 02, 2023 at 08:06:56PM +0600, Maxim Uvarov wrote:
> > Signed-off-by: Maxim Uvarov
> > ---
> > lib/lwip/Makefile | 2 +
>
>
> > +
> > +#include "apps/dns/lwip-dns.h"
> > +#include "apps/ping/lwip_ping.h"
> > +#inc
On Wed, Jun 08, 2022 at 02:30:14PM -0400, Tom Rini wrote:
> When migrating CONFIG_CONS_INDEX to Kconfig, on this platform we changed
> what "board" evaluated to in the environment. This in turn meant that
> we would no longer try and find the correct fdtfile via the normal
> distro boot logic. Fi
Hi Neil,
I would like to know your opinion before I change the patches.
On Sat, Jul 15, 2023 at 05:40:53PM -0600, Simon Glass wrote:
> Hi,
>
> On Thu, 13 Jul 2023 at 23:30, AKASHI Takahiro
> wrote:
> >
> > On Tue, Jul 11, 2023 at 01:13:29PM -0600, Simon Glass wrote:
> > > +AKASHI Takahiro
> >
>
On 8/7/2023 3:50 PM, Wu, Fei wrote:
> Hi All,
>
> I am working on enabling PCIe passthrough on qemu risc-v, in order for
> the guest to access the host x86 pci resource directly, ram on guest
> won't cover this range, so if guest has 4GB ram, two ranges are created:
> 1. 2G-3G
> 2. 4G-7G
>
>
If dram_init_banksize() is called from SPL, the rom_pointer, at that
point, is not correctly initialized. This causes wrong calculation of
DRAM start and size in dram_init_banksize(). The issue became apparent
only in Falcon Mode. Added an extra condition to prevent using
rom_pointer in SPL.
Signe
This series is the second version of Bo Gan's L2 LIM series. Original
author hasn't sent v2 for almost 2 months, so I decided to take over
this series.
Background information:
JH7110 SPL runs in L2 LIM (2M in size mapped at 0x800). It
consists of 16 0x2 sized regions, each one can be use
Add a Kconfig item to allow SPL to clear stack/GD/malloc area before
using them.
Signed-off-by: Bo Gan
Signed-off-by: Shengyu Qu
---
arch/riscv/Kconfig | 8
1 file changed, 8 insertions(+)
diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
index 867cbcbe74..6771d8d919 100644
--- a/a
Add the actual support code for SPL_ZERO_MEM_BEFORE_USE and remove
existing Starfive JH7110's L2 LIM clean code, since existing code has
following issues:
1. Each hart (in the middle of a function call) overwriting its own
stack and other harts' stacks.
(data-race and data-corruption)
2.
Add Kconfig item for Starfive JH7110 to select SPL_ZERO_MEM_BEFORE_USE.
Signed-off-by: Bo Gan
Signed-off-by: Shengyu Qu
---
arch/riscv/cpu/jh7110/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/riscv/cpu/jh7110/Kconfig b/arch/riscv/cpu/jh7110/Kconfig
index 4d9581165b..2e26d0731
On 8/8/23 13:58, Elena Popa wrote:
If dram_init_banksize() is called from SPL, the rom_pointer, at that
point, is not correctly initialized. This causes wrong calculation of
DRAM start and size in dram_init_banksize(). The issue became apparent
only in Falcon Mode. Added an extra condition to pre
Adapt behaviour to Linux kernel driver.
The return value of gpio_request_by_name_nodev() was not checked before,
and thus in case 'rb-gpios' was missing in DT, rb.type was set to
ATMEL_NAND_GPIO_RB nevertheless, leading to output like this for
example (on sam9x60-curiosity with the line removed fr
Hello everyone,
this is a patch series wtih some real fixes _and_ a question or some
kind of support request in the cover letter. I would be happy if anyone
could read the cover letter carefully and answer to that one what might
be the problem I see. O:-)
I'm currently working on the sam9x60-cur
Signed-off-by: Alexander Dahl
---
drivers/mtd/nand/raw/atmel/nand-controller.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/mtd/nand/raw/atmel/nand-controller.c
b/drivers/mtd/nand/raw/atmel/nand-controller.c
index 9873d11254..2b29c8def6 100644
--- a/drivers/mtd/nand/raw/atmel/nand-
Added Mihai who tested this a lot at some point in time
Eugen
On 8/8/23 16:02, Alexander Dahl wrote:
Hello everyone,
this is a patch series wtih some real fixes _and_ a question or some
kind of support request in the cover letter. I would be happy if anyone
could read the cover letter careful
On Starfive Visionfive 2, the u-boot environment settings are saved to
on-board SPI flash. Enable relative configs by default and set offset
and size according to upstream linux dts.
Signed-off-by: Shengyu Qu
---
configs/starfive_visionfive2_defconfig | 9 +
1 file changed, 9 insertions(
Hi Alex,
Please find bellow my answer:
--
Added Mihai who tested this a lot at some point in time
Eugen
On 8/8/23 16:02, Alexander Dahl wrote:
> Hello everyone,
>
> this is a patch series wtih some real fixes _and_ a question or some
> kind of support request in th
Hi
On Tue, Aug 8, 2023 at 3:03 PM Alexander Dahl wrote:
>
> Signed-off-by: Alexander Dahl
> ---
> drivers/mtd/nand/raw/atmel/nand-controller.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/mtd/nand/raw/atmel/nand-controller.c
> b/drivers/mtd/nand/raw/atmel/nand-controller.c
>
Hi
On Tue, Aug 8, 2023 at 3:03 PM Alexander Dahl wrote:
>
> Adapt behaviour to Linux kernel driver.
>
> The return value of gpio_request_by_name_nodev() was not checked before,
> and thus in case 'rb-gpios' was missing in DT, rb.type was set to
> ATMEL_NAND_GPIO_RB nevertheless, leading to output
When Falcon Mode is enabled, SPL needs to check the value of
CONFIG_SYS_MMCSD_RAW_MODE_ARGS_SECTOR. Unfortunately, it was using the
CONFIG_VAL(SYS_MMCSD_RAW_MODE_ARGS_SECTOR) which converts it into
CONFIG_SPL_SYS_MMCSD_RAW_MODE_ARGS_SECTOR when CONFIG_SPL_BUILD is
enabled. CONFIG_SPL_SYS_MMCSD_RAW_
On Tue, Aug 08, 2023 at 04:42:15PM +0300, Elena Popa wrote:
> When Falcon Mode is enabled, SPL needs to check the value of
> CONFIG_SYS_MMCSD_RAW_MODE_ARGS_SECTOR. Unfortunately, it was using the
> CONFIG_VAL(SYS_MMCSD_RAW_MODE_ARGS_SECTOR) which converts it into
> CONFIG_SPL_SYS_MMCSD_RAW_MODE_AR
On Thu, 3 Aug 2023 at 03:32, Simon Glass wrote:
> Hi Maxim,
>
> On Wed, 2 Aug 2023 at 08:11, Maxim Uvarov wrote:
> >
> > Replace original commands: ping, tftp, dhcp and wget.
> >
> > Signed-off-by: Maxim Uvarov
> > ---
> > boot/bootmeth_efi.c | 2 +-
> > boot/bootmeth_pxe.c | 2 +-
> > cmd/n
On Thu, Jul 20, 2023 at 3:50 PM Detlev Casanova
wrote:
>
> The command can be used to show various information that can be used to
> identify the running system.
>
> Currently supported subcommands are:
> * model: A string representing the model
> * id: The id of the board
> * revision: The revisi
Hi Jonas,
On Sun, 2023-07-23 at 15:04 +, Jonas Karlman wrote:
> On 2023-07-23 16:55, Jonas Karlman wrote:
> > The Pine64 Quartz64 Model A is a single-board computer based on the
> > Rockchip RK3566 SoC. The board features USB3, SATA, PCIe, HDMI, USB2.0,
> > CSI, DSI, eDP, eMMC, SD, and an e-pa
On Tuesday, August 8, 2023 10:21:11 A.M. EDT Peter Robinson wrote:
> On Thu, Jul 20, 2023 at 3:50 PM Detlev Casanova
>
> wrote:
> > The command can be used to show various information that can be used to
> > identify the running system.
> >
> > Currently supported subcommands are:
> > * model: A
On Tue, Aug 8, 2023 at 3:27 PM Detlev Casanova
wrote:
>
> On Tuesday, August 8, 2023 10:21:11 A.M. EDT Peter Robinson wrote:
> > On Thu, Jul 20, 2023 at 3:50 PM Detlev Casanova
> >
> > wrote:
> > > The command can be used to show various information that can be used to
> > > identify the running
On 8/8/23 16:31, Peter Robinson wrote:
On Tue, Aug 8, 2023 at 3:27 PM Detlev Casanova
wrote:
On Tuesday, August 8, 2023 10:21:11 A.M. EDT Peter Robinson wrote:
On Thu, Jul 20, 2023 at 3:50 PM Detlev Casanova
wrote:
The command can be used to show various information that can be used to
ide
Hello Mihai,
Am Tue, Aug 08, 2023 at 01:40:26PM + schrieb mihai.s...@microchip.com:
> Hi Alex,
>
> Please find bellow my answer:
>
> --
>
> Added Mihai who tested this a lot at some point in time
>
> Eugen
>
> On 8/8/23 16:02, Alexander Dahl wrote:
> > Hello ev
Hello Michael,
Am Tue, Aug 08, 2023 at 03:49:45PM +0200 schrieb Michael Nazzareno Trimarchi:
> Hi
>
> On Tue, Aug 8, 2023 at 3:03 PM Alexander Dahl wrote:
> >
> > Adapt behaviour to Linux kernel driver.
> >
> > The return value of gpio_request_by_name_nodev() was not checked before,
> > and thus
When Falcon Mode is enabled, spl_start_uboot() function must be defined. If
this function returns 0 then SPL should start the kernel, if it returns 1 then
U-Boot must be started.
When spl_start_uboot() returns 1, then U-Boot must be loaded, and as far as I
can tell spl_image->os should be set
Driver is currently defining the mask and bit shifting itself,
there is no need for that as U-Boot has generic bitfield macros that help
us achieve the same result but in a cleaner way.
Signed-off-by: Robert Marko
---
drivers/net/mv88e6xxx.c | 25 +
1 file changed, 13 ins
Marvell LinkStreet switches support Clause 45 MDIO on the internal bus.
C45 read or writes require the register address to be written first to
the SMI PHY Data register, and then a special C45 Write Address Register
OP is used on the SMI PHY Register before making a C45 Read Data Register
OP and b
From: Yang Xiwen
It's already implemented in clk-mux.c, export it in the header file.
Signed-off-by: Yang Xiwen
---
include/linux/clk-provider.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h
index b8acacd49e..801404480b 10
They are found during my development for HiSilicon clock driver. Details
are in commit logs.
Signed-off-by: Yang Xiwen
---
Yang Xiwen (5):
clk: export clk_register_mux_table()
clk: call log_debug() instead to avoid console log printing
clk: also handle ENOENT in *_optional funct
From: Yang Xiwen
If the device does not specify any clocks in device tree, these
functions will return PTR_ERR(-ENOENT). This is not the intended
behavior and does not comply with linux kernel CCF. Fix that by
returning NULL under such circumstances instead.
Signed-off-by: Yang Xiwen
---
inclu
From: Yang Xiwen
it's a very common case to register a clock without a parent, such as
clk_register_fixed_rate(). Replace log_error() with log_debug() to avoid
useless console log if not debugging.
Signed-off-by: Yang Xiwen
---
drivers/clk/clk.c | 4 ++--
1 file changed, 2 insertions(+), 2 del
From: Yang Xiwen
Calling into CCF framework will cause a clock being enabled twice
instead of once (clk->enable_count becomes 2 rather than 1), thus making
it hard to disable (needs to call clk_disable() twice).
Fix that by calling clock provided ops directly.
Signed-off-by: Yang Xiwen
---
dri
From: Yang Xiwen
So that it can be used by others.
Signed-off-by: Yang Xiwen
---
drivers/clk/clk-uclass.c | 5 -
include/linux/clk-provider.h | 5 +
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c
index dc3e9d6a26..
On Mon, 7 Aug 2023 at 15:08, Tom Rini wrote:
>
> In order to reduce the number of people that are cc'd on a patch for
> simply touching arch/arm/dts/Makefile (which is a big common file) add
> an entry specifically to MAINTAINERS under the ARM entry.
>
> Signed-off-by: Tom Rini
> ---
> MAINTAINE
Hi Maxim,
On Tue, 8 Aug 2023 at 08:06, Maxim Uvarov wrote:
>
>
>
> On Thu, 3 Aug 2023 at 03:32, Simon Glass wrote:
>>
>> Hi Maxim,
>>
>> On Wed, 2 Aug 2023 at 08:11, Maxim Uvarov wrote:
>> >
>> > Replace original commands: ping, tftp, dhcp and wget.
>> >
>> > Signed-off-by: Maxim Uvarov
>> > -
Hi Chanho,
On Tue, 8 Aug 2023 at 01:35, Chanho Park wrote:
>
> Since the Patch 55171aedda88, VisionFive2 booting has been broken [1].
> VisionFive2 board requires to enable CONFIG_TIMER_EARLY but booting went
> to panic from initr_dm_devices due to lack of a timer device.
>
> - Error logs
> initc
Hi Maxim,
On Tue, 8 Aug 2023 at 04:07, Maxim Uvarov wrote:
>
>
>
>
> On Thu, 3 Aug 2023 at 22:21, Maxim Uvarov wrote:
>>
>>
>>
>> On Thu, 3 Aug 2023 at 03:32, Simon Glass wrote:
>>>
>>> Hi Maxim,
>>>
>>> On Wed, 2 Aug 2023 at 08:11, Maxim Uvarov wrote:
>>> >
>>>
>>> subject: U-Boot
>>>
>>> com
Hi Tim,
On Mon, 7 Aug 2023 at 14:03, Tim Harvey wrote:
>
> On Sun, Jul 30, 2023 at 7:50 PM Simon Glass wrote:
> >
> > Hi Tim,
> >
> > On Sun, 30 Jul 2023 at 17:42, Tim Harvey wrote:
> > >
> > >
> > > On Wed, Jul 26, 2023, 5:55 PM Simon Glass wrote:
> > >>
> > >> Hi Tim,
> > >>
> > >> On Mon, 2
Hi Maxim
[...]
> > > +static int lwip_empty_tmo(void) { return 0; };
> > > +int (*ulwip_tmo)(void) = lwip_empty_tmo;
> > > +void ulwip_set_tmo(int (*tmo)(void))
> > > +{
> > > + ulwip_tmo = tmo;
> > > +}
> > > +
> > > +static void ulwip_clear_tmo(void)
> > > +{
> > > + ulwip_tmo = lwip_em
Hi Marek,
On Mon, Jun 26, 2023 at 5:53 AM Marek Vasut wrote:
>
> Add weak default reset_cpu() implementation needed by e.g. panic().
>
> Signed-off-by: Marek Vasut
> ---
> Cc: "NXP i.MX U-Boot Team"
> Cc: Fabio Estevam
> Cc: Stefano Babic
> ---
> arch/arm/mach-imx/cpu.c | 4
> 1 file ch
Now that the driver for the Bochs VGA card emulated by QEMU is no longer
limited to x86 architectures [1], this series enables it on arm and
arm64 virtual machines to provide a graphical interface. In line with
that series this also enables console buffering and USB keyboard.
Tested with the Debia
Commit 716161663ec49 ("riscv: qemu: Enable Bochs video support") enables
a video console for QEMU RISC-V virtual machines using an emulated Bochs
VGA card. Similarly, enable it for ARM virtual machines as well.
Signed-off-by: Alper Nebi Yasak
---
arch/arm/Kconfig | 4
Commit 608b80b5b855 ("riscv: qemu: Enable PRE_CONSOLE_BUFFER") enables
buffering console messages for QEMU RISC-V virtual machines so those
printed before the video console is available will still show up on the
display. Similarly, enable it for ARM virtual machines as well.
Signed-off-by: Alper N
Commit 02be57caf730 ("riscv: qemu: Enable usb keyboard as an input
device") adds PCI xHCI support to QEMU RISC-V virtual machines and
enables using a USB keyboard as one of the input devices. Similarly,
enable those for ARM virtual machines as well.
Signed-off-by: Alper Nebi Yasak
---
arch/arm/
On Fri, Aug 04, 2023 at 02:33:36PM +0100, Abdellatif El Khlifi wrote:
> Adding support for Arm FF-A v1.0 (Arm Firmware Framework for Armv8-A) [A].
>
> FF-A specifies interfaces that enable a pair of software execution
> environments aka partitions to
> communicate with each other. A partition co
On 8/8/23 00:15, Tom Rini wrote:
To help guide developers down the right path, begin a document that
lists some best practices to follow when creating a new board port.
Signed-off-by: Tom Rini
---
doc/develop/board_best_practices.rst | 26 ++
doc/develop/index.rst
To help guide developers down the right path, begin a document that
lists some best practices to follow when creating a new board port.
Signed-off-by: Tom Rini
---
Cc: Heinrich Schuchardt
Changes in v2:
- Apply feedback from Heinrich
---
doc/develop/board_best_practices.rst | 26 +++
On 8/8/23 21:17, Fabio Estevam wrote:
Hi Marek,
On Mon, Jun 26, 2023 at 5:53 AM Marek Vasut wrote:
Add weak default reset_cpu() implementation needed by e.g. panic().
Signed-off-by: Marek Vasut
---
Cc: "NXP i.MX U-Boot Team"
Cc: Fabio Estevam
Cc: Stefano Babic
---
arch/arm/mach-imx/cpu
Hi there,
I'm working on certain feature to be added to ONIE that is running on
Broadcom's Helix4 -based network switch. Part of the task is to add FIT
signature signing to U-boot versions supported by ONIE. ONIE as part of the
build process uses uboot snapshots, and applies platform patches on to
From: Fabio Estevam
SPDX-License tag is missing and checkpatch complains about it.
Add the SPDX-License tag using the same one from imx7d-sdb.dts.
Signed-off-by: Fabio Estevam
---
arch/arm/dts/imx7d-sdb-u-boot.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/dts/imx7d-sdb-u
From: Fabio Estevam
Commit 68dcbdd594d4 ("ARM: imx: Add weak default reset_cpu()") caused
the 'reset' command in U-Boot to not cause a board reset.
Fix it by switching to the watchdog driver model via sysreset, which
is the preferred method for implementing the watchdog reset.
Signed-off-by: Fa
From: Fabio Estevam
Commit 68dcbdd594d4 ("ARM: imx: Add weak default reset_cpu()") caused
the 'reset' command in U-Boot to not cause a board reset.
Fix it by switching to the watchdog driver model via sysreset, which
is the preferred method for implementing the watchdog reset.
Signed-off-by: Fa
On Wed, Jul 26, 2023 at 10:01:21AM +0300, Dan Carpenter wrote:
> This code has a & vs && typo so it only preserves odd value error
> codes and not even value error codes.
>
> Signed-off-by: Dan Carpenter
> Reviewed-by: Simon Glass
> ---
> drivers/power/regulator/regulator-uclass.c | 2 +-
> 1
Hi Elena,
On 08/08/2023 08:58, Elena Popa wrote:
If dram_init_banksize() is called from SPL, the rom_pointer, at that
point, is not correctly initialized. This causes wrong calculation of
DRAM start and size in dram_init_banksize(). The issue became apparent
only in Falcon Mode. Added an extra c
On 8/8/23 23:40, Fabio Estevam wrote:
From: Fabio Estevam
Commit 68dcbdd594d4 ("ARM: imx: Add weak default reset_cpu()") caused
the 'reset' command in U-Boot to not cause a board reset.
Fix it by switching to the watchdog driver model via sysreset, which
is the preferred method for implementin
On 8/8/23 23:40, Fabio Estevam wrote:
From: Fabio Estevam
Commit 68dcbdd594d4 ("ARM: imx: Add weak default reset_cpu()") caused
the 'reset' command in U-Boot to not cause a board reset.
Fix it by switching to the watchdog driver model via sysreset, which
is the preferred method for implementin
On 8/8/23 23:40, Fabio Estevam wrote:
From: Fabio Estevam
SPDX-License tag is missing and checkpatch complains about it.
Add the SPDX-License tag using the same one from imx7d-sdb.dts.
Signed-off-by: Fabio Estevam
---
arch/arm/dts/imx7d-sdb-u-boot.dtsi | 2 ++
1 file changed, 2 insertions
Instead of marking malloc as initialized as soon as relocation is done,
defer it until after we call mem_malloc_init. This ensures that malloc
initialization is done before we switch away from simple_malloc, and
matches the SPL behavior.
Fixes: c9356be3074 ("dm: Split the simple malloc() implement
With CONFIG_IS_ENABLED we can eliminate some ifdefs.
Signed-off-by: Sean Anderson
Reviewed-by: Heinrich Schuchardt
---
(no changes since v1)
common/dlmalloc.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/common/dlmalloc.c b/common/dlmalloc.c
index 0f9b7262d51..
On boards with size restrictions, 1-2k can be a significant fraction of
the binary size. Add a new SPL version of SYS_MALLOC_RUNTIME_INIT and
enable it by default.
Signed-off-by: Sean Anderson
---
(no changes since v1)
Kconfig | 11 +++
1 file changed, 11 insertions(+)
diff --git a/Kc
When we enable malloc_init, there is no need to statically initialize
av_, since we are going to do it manually. This lets us move av_ to
.bss, saving around 1-2k of data (depending on the pointer size).
cALLOc must be adjusted to not access top before malloc_init.
While we're at it, rename/rewor
In my efforts to get SPL to fit into flash after some changes I made, I
noticed that av_ is one of the largest variables in SPL. As it turns
out, we can generate it at runtime, and the code is already there. This
has the potential to save 1-2k across the board, for some (very) minor
boot time incre
I compiled u-boot version 2023-07 and boot it on a RPi CM4 using a PCIe M.2
NVMe drive, but it doesn't appear to be working.
Once u-boot has started and boot successfully it recognizes the PCIe device
giving me the next output
U-Boot> pci
BusDevFun VendorId DeviceId Device Class Sub-Cl
On Tue, Aug 8, 2023 at 10:54 AM Simon Glass wrote:
>
> Hi Tim,
>
> On Mon, 7 Aug 2023 at 14:03, Tim Harvey wrote:
> >
> > On Sun, Jul 30, 2023 at 7:50 PM Simon Glass wrote:
> > >
> > > Hi Tim,
> > >
> > > On Sun, 30 Jul 2023 at 17:42, Tim Harvey wrote:
> > > >
> > > >
> > > > On Wed, Jul 26, 20
On Wed, Jul 26, 2023 at 09:54:08AM +0300, Dan Carpenter wrote:
> The VNBYTES() macro needs to have parentheses to prevent some (harmless)
> macro expansion bugs. The VNBYTES() macro is used like this:
>
> VID_TO_PIXEL(x) * VNBYTES(vid_priv->bpix)
>
> The * operation is done before the / o
On Fri, Jul 14, 2023 at 01:24:50PM +0200, Matthias Schiffer wrote:
> Both the Linux kernel and libbsd agree that strlcpy() should always
> return strlen(src) and not include the NUL termination. The incorrect
> U-Boot implementation makes it impossible to check the return value for
> truncation, a
On Fri, Jul 14, 2023 at 01:24:51PM +0200, Matthias Schiffer wrote:
> strlcat returns min(strlen(dest), count)+strlen(src). Make u16_strlcat's
> behaviour the same for consistency.
>
> Fixes: eca08ce94ceb ("lib/charset: add u16_strlcat() function")
> Signed-off-by: Matthias Schiffer
Applied to u
On Fri, Jul 14, 2023 at 05:39:32PM +0300, Ilya Lukin wrote:
> Fixes: 3db711085752 ("crc32: Use the crc.h header for crc functions")
> Signed-off-by: Ilya Lukin <4.sh...@gmail.com>
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Description: PGP signature
On Sat, Jul 22, 2023 at 12:15:21AM +0800, Bin Meng wrote:
> Some coding convention fixes for print_resetinfo().
>
> Signed-off-by: Bin Meng
> Reviewed-by: Simon Glass
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Description: PGP signature
On Tue, Jul 25, 2023 at 09:50:08AM +0300, Dan Carpenter wrote:
> This returns the wrong variable. It ends up returning NULL when it was
> suppose to return an error pointer.
>
> Signed-off-by: Dan Carpenter
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Description: PGP signature
On Tue, Jul 25, 2023 at 09:50:26AM +0300, Dan Carpenter wrote:
> The parentheses are in the wrong place so this passes the number of
> bytes to write as "sizeof(index_0) != TPM_SUCCESS" when just
> "sizeof(index_0)" was intended. (1 byte vs 4 bytes).
>
> Signed-off-by: Dan Carpenter
Applied to
On Wed, Jul 26, 2023 at 09:58:34AM +0300, Dan Carpenter wrote:
> The ec_command_inptr() function returns negative error codes or
> the number of bytes that it was able to read. The cros_ec_get_sku_id()
> function should return negative error codes. Right now it returns
> positive error codes or
On Wed, Jul 26, 2023 at 09:59:04AM +0300, Dan Carpenter wrote:
> If btrfs_read_fs_root() fails with -ENOENT, then we go to the next
> entry. Fine. But if it fails for a different reason then we need
> to clean up and return an error code. In the current code it
> doesn't clean up but instead de
On Wed, Jul 26, 2023 at 09:59:52AM +0300, Dan Carpenter wrote:
> The "oftree_count" is the number of entries which have been set in
> the oftree_list[] array. If all the entries have been initialized then
> this off by one would result in reading one element beyond the end
> of the array.
>
> Si
On Wed, Jul 26, 2023 at 10:00:33AM +0300, Dan Carpenter wrote:
> We know that "pa" is non-NULL so it's nicer to just return zero instead
> of return !pa. This has no effect on runtime behavior.
>
> Signed-off-by: Dan Carpenter
> Reviewed-by: Simon Glass
Applied to u-boot/next, thanks!
--
To
On Thu, Jul 27, 2023 at 10:12:39AM +0300, Dan Carpenter wrote:
> These lines are supposed to be indented one more tab. Otherwise it's
> confusing to read.
>
> Signed-off-by: Dan Carpenter
> Reviewed-by: Patrick Delaunay
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Description: PGP
On Thu, Jul 27, 2023 at 10:12:58AM +0300, Dan Carpenter wrote:
> The u16_strlcat() is in units of u16 not bytes. So the limit needs to
> be ARRAY_SIZE() instead of sizeof().
>
> Signed-off-by: Dan Carpenter
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Description: PGP signature
On Thu, Jul 27, 2023 at 10:16:03AM +0300, Dan Carpenter wrote:
> This line break is not done correctly. We don't want to have all those
> tabs in the printed output.
>
> Signed-off-by: Dan Carpenter
> Reviewed-by: Simon Glass
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Descriptio
On Thu, Jul 27, 2023 at 06:50:14PM +0200, Heinrich Schuchardt wrote:
> If an error message contains a function name, it should match the name of
> the function throwing the message.
>
> Fixes: 7739d93d8288 ("pci: Match region flags using a mask")
> Signed-off-by: Heinrich Schuchardt
> Reviewed-b
On Mon, Jul 31, 2023 at 05:03:59PM +0300, Dan Carpenter wrote:
> Match the "=0x" instead of just "=0".
>
> Signed-off-by: Dan Carpenter
> Reviewed-by: Heinrich.Schuchardt
> Reviewed-by: Simon Glass
> Reviewed-by: Ilias Apalodimas
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Descr
On Mon, Jul 31, 2023 at 05:08:29PM +0300, Dan Carpenter wrote:
> This should be allocating the memory for "item" instead of "menu".
> The item struct is 48 bytes instead of 96 (assuming a 64bit system)
> so this saves a little memory.
>
> Signed-off-by: Dan Carpenter
> Reviewed-by: Simon Glass
On Mon, Jul 31, 2023 at 05:08:41PM +0300, Dan Carpenter wrote:
> The x509_cert_parse() and pkcs7_parse_message() functions return error
> pointers. They don't return NULL. Update the checks accordingly.
>
> Signed-off-by: Dan Carpenter
> Reviewed-by: Simon Glass
Applied to u-boot/next, thank
On Thu, Aug 03, 2023 at 01:29:34PM +0300, Dan Carpenter wrote:
> The btrfs_decompress() function mostly (u32)-1 on error but it can
> also return -EPERM or other kernel error codes from zstd_decompress().
> The "ret" variable is an int, so we could just check for negatives.
>
> Signed-off-by: Dan
On Tue, 25 Jul 2023 17:08:44 -0400, Tom Rini wrote:
> Rather than invoking the script that will write to the mounted directory
> as a binary, source it as a script so that it has access to more than
> just two parameters. This will allow us to have the same flexibility in
> our writers that othe
On Wed, Jun 07, 2023 at 04:37:10PM -0700, William Zhang wrote:
> Add entry for Broadcom Broadband SoC HS SPI drivers
>
> Signed-off-by: William Zhang
Can you please rebase this rest of this series on top of current next
and fix the compiler warnings that show up, thanks.
--
Tom
signature.as
1 - 100 of 128 matches
Mail list logo
