Hi Tony,
On 12.08.22 01:40, Tony Dinh wrote:
- Add distro boot to board include file and deconfig file
- Miscellaneous changes:
- Remove Gerald from maintainer list (email bounced)
- Add CONFIG_SUPPORT_PASSING_ATAGS and friends to support legacy
kernel method of booting
This is not used, so drop it.
Suggested-by: AKASHI Takahiro
Signed-off-by: Simon Glass
---
Changes in v2:
- Add new patch to drop ifname field from struct efi_disk_obj
lib/efi_loader/efi_disk.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/lib/efi_loader/efi_disk.c
We currently have an if_type (interface type) and a uclass id. These are
closely related and we don't need to have both.
Drop the if_type values and use the uclass ones instead.
Maintain the existing, subtle, one-way conversion between UCLASS_USB and
UCLASS_MASS_STORAGE for now, and add a
Hi Takahiro,
On Sun, 7 Aug 2022 at 19:05, AKASHI Takahiro wrote:
>
> My comment below is not directly related to the change, but
>
> On Sun, Aug 07, 2022 at 09:47:06AM -0600, Simon Glass wrote:
> > We currently have an if_type (interface type) and a uclass id. These are
> > closely related and
The block interface has two separate implementations, one using driver
model and one not. The latter is really only needed for SPL, where
size constraints allegedly don't allow use of driver model. Of course
we still need space for filesystems and other code, so it isn't clear
that driver model is
Use the word 'uclass' instead of 'if_type' to complete the conversion.
Signed-off-by: Simon Glass
---
Changes in v2:
- Use conv_uclass_id() instead of the confusing uclass_id_to_uclass_id()
board/st/common/stm32mp_dfu.c| 2 +-
cmd/blk_common.c | 18 +--
Use the uclass ID instead.
Signed-off-by: Simon Glass
---
(no changes since v1)
cmd/blk_common.c | 2 +-
drivers/block/blk-uclass.c | 34 +++
drivers/block/blk_legacy.c | 20 +--
include/blk.h | 41
This is defined automatically when needed, so drop it from the few
defconfig files that define it manually.
Signed-off-by: Simon Glass
---
(no changes since v1)
configs/axm_defconfig | 1 -
configs/bcm968380gerg_ram_defconfig| 1 -
This is currently missing. Add it.
Fix the code style for the function while we are here.
Suggested-by: AKASHI Takahiro
Signed-off-by: Simon Glass
---
Changes in v2:
- Add new patch to handle UCLASS_EFI_MEDIA in dev_print()
disk/part.c | 5 -
1 file changed, 4 insertions(+), 1
These are currently using a simple array lookup in one direction, and
relying on if_type being sequential.
With the move to uclass IDs this needs to change. Update the code to
prepare for the new way. This patch is intended to introduce no
functional change.
The returning of "(none)" from
We don't want boards to be able to change these. They can be handled
as dependencies of options that need them, such as SPL_MMC. There is no
point in enabling the block interface without any storage devices to
create a block device.
Hide both options from the 'menuconfig' display and deny their
This is not really needed since it does the same things as MMC. Drop it.
Signed-off-by: Simon Glass
---
(no changes since v1)
disk/part.c| 2 --
drivers/block/blk-uclass.c | 2 --
include/blk.h | 1 -
3 files changed, 5 deletions(-)
diff --git a/disk/part.c
efi_set_blk_dev_to_system_partition() assumes that 0 is an invalid
if_type. This is true now but is about to be false. Fix this bug to avoid
a test failure.
Signed-off-by: Simon Glass
---
(no changes since v1)
lib/efi_loader/efi_var_file.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
At present we use a variable with the same name as the enum. Change this
since we plan to #define the enum to uclass_id.
Signed-off-by: Simon Glass
---
(no changes since v1)
drivers/block/blk-uclass.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git
This is not really needed since it does the same things as IDE. Drop it.
Signed-off-by: Simon Glass
---
(no changes since v1)
disk/part.c| 5 -
drivers/block/blk-uclass.c | 2 --
include/blk.h | 1 -
3 files changed, 8 deletions(-)
diff --git a/disk/part.c
Rather than setting a different interface type, use a flag to indicate
that a device is ATAPI.
Signed-off-by: Simon Glass
---
(no changes since v1)
drivers/block/ide.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/block/ide.c b/drivers/block/ide.c
index
This doesn't seem to be used for anything and it isn't clear what it is.
It dates from the first U-Boot commit.
Drop it.
Signed-off-by: Simon Glass
---
(no changes since v1)
disk/part.c| 9 -
drivers/block/blk-uclass.c | 2 --
include/blk.h | 1 -
3 files
Selecting this option can be handled in the Kconfig option itself, as it
is with BLK. Update this an drop the various 'select' clauses.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/spl/Kconfig | 2 --
drivers/ata/Kconfig| 1 -
drivers/block/Kconfig | 11 +++
This option is fact really related to SPL. For U-Boot proper we always use
driver model for block devices, so CONFIG_BLK is enabled if block devices
are in use.
It is only for SPL that we have two cases:
- SPL_BLK is enabled, in which case we use driver model and blk-uclass.c
- SPL_BLK is not
We can rely on the compiler to eliminate any dead code.
Signed-off-by: Simon Glass
---
(no changes since v1)
drivers/block/blk_legacy.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/block/blk_legacy.c b/drivers/block/blk_legacy.c
index df9a3597545..bd8a17df6a9 100644
---
At present these files have an #ifdef covering the whole file. Move the
condition to the Makefile instead.
Add BLK to the condition since future patches will adjust things so that
HAVE_BLOCK_DEVICE is only for SPL, but the partition drivers are needed
in U-Boot proper too.
Signed-off-by: Simon
This condition is not needed for these commands, since BLK is enabled for
all boards which use block devices and commands are not available in SPL,
so even if SPL_BLK is not enabled, it doesn't affect commands.
Signed-off-by: Simon Glass
---
(no changes since v1)
cmd/Kconfig | 5 -
At present we use HAVE_BLOCK_DEVICE to indicate when block devices are
available.
This is a very strange option, since it partially duplicates the BLK
option used by driver model. It also covers both U-Boot proper and SPL,
even though one might have block devices and another not.
As a first step
Enable this option on all boards which support block devices. Drop the
related depencies on BLK since these are not needed anymore.
Disable BLOCK_CACHE on M5253DEMO as this causes a build error.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add SATA also
arch/arm/Kconfig| 1 -
The _SUPPORT suffix should be dropped. This happened because the rename
was applied around the same time as this new option, so did not include
renaming the new option.
The relevant commits are:
f7560376ae sata: Rename SATA_SUPPORT to SATA
73059529b2 ata: ahci-pci: Add new option
This is not needed as it is implied or selected by other options anyway.
Signed-off-by: Simon Glass
---
(no changes since v1)
arch/Kconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/Kconfig b/arch/Kconfig
index 6495e780fec..d3c5d86fb5b 100644
--- a/arch/Kconfig
+++ b/arch/Kconfig
Fix a few typos in this help text. Fix a typo in SPL_PARTITIONS while
we are here.
Signed-off-by: Simon Glass
---
Changes in v2:
- Update commit message
- Fix SPL_PARTITIONS too
disk/Kconfig | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/disk/Kconfig
Hi Heinrich,
On Mon, 8 Aug 2022 at 02:09, Heinrich Schuchardt wrote:
>
> On 8/7/22 17:46, Simon Glass wrote:
> > We don't want boards to be able to change these. They can be handled
> > as dependencies of options that need them, such as SPL_MMC. There is no
> > point in enabling the block
Hi Takahiro,
On Sun, 7 Aug 2022 at 18:49, AKASHI Takahiro wrote:
>
> On Sun, Aug 07, 2022 at 09:46:55AM -0600, Simon Glass wrote:
> > This option is fact really related to SPL. For U-Boot proper we always use
> > driver model for block devices, so CONFIG_BLK is enabled if block devices
> > are
Hi Heinrich,
On Mon, 8 Aug 2022 at 02:00, Heinrich Schuchardt wrote:
>
> On 8/7/22 17:46, Simon Glass wrote:
> > Enable this option on all boards which support block devices. Drop the
> > related depencies on BLK since these are not needed anymore.
> >
> > Disable BLOCK_CACHE on M5253DEMO as
Hi Takahiro,
On Sun, 7 Aug 2022 at 20:00, AKASHI Takahiro wrote:
>
> On Mon, Aug 08, 2022 at 10:32:16AM +0900, AKASHI Takahiro wrote:
> > My comment below is not directly related to the change, but
>
> Another comment,
>
OK I did a patch for the first comment, but can I please leave you to
On Thu, Aug 11, 2022 at 06:08:51PM -0600, Simon Glass wrote:
> Hi,
>
> On Thu, 11 Aug 2022 at 11:44, Tom Rini wrote:
> >
> > On Thu, Aug 11, 2022 at 09:59:05AM -0700, Tim Harvey wrote:
> > > On Thu, Aug 11, 2022 at 9:48 AM Tom Rini wrote:
> > > >
> > > > On Thu, Aug 11, 2022 at 09:27:44AM
On 8/12/2022 2:59 AM, Tim Harvey wrote:
In some situations the GSC firmware where the EEPROM containing the
model and DRAM configuration may not be ready by the time the SoC
is ready to talk to it over I2C.
Instead of a hard delay, poll the I2C lines to wait until they are
released to avoid
On 8/12/2022 3:02 AM, Tim Harvey wrote:
Add missing LTE_PWR# and LTE_RST gpio pinmux.
Signed-off-by: Tim Harvey
If this not diverge from Linux dts,
Acked-by: Peng Fan
---
arch/arm/dts/imx8mm-venice-gw7902.dts | 3 +++
1 file changed, 3 insertions(+)
diff --git
On 8/12/2022 2:53 AM, Tim Harvey wrote:
When booting from USB/SDP mmc_get_env_dev() returns
CONFIG_SYS_MMC_ENV_DEV as the MMC env device (while booting from MMC
will call board_mmc_get_env_dev() to get this). Configure
CONFIG_SYS_MMC_ENV_DEV for SDHC3 (devno=2) as all Gateworks Venice
boards
On Thu, Aug 11, 2022 at 5:09 PM Simon Glass wrote:
>
> Hi,
>
> On Thu, 11 Aug 2022 at 11:44, Tom Rini wrote:
> >
> > On Thu, Aug 11, 2022 at 09:59:05AM -0700, Tim Harvey wrote:
> > > On Thu, Aug 11, 2022 at 9:48 AM Tom Rini wrote:
> > > >
> > > > On Thu, Aug 11, 2022 at 09:27:44AM -0700, Tim
Hi,
On Thu, 11 Aug 2022 at 11:44, Tom Rini wrote:
>
> On Thu, Aug 11, 2022 at 09:59:05AM -0700, Tim Harvey wrote:
> > On Thu, Aug 11, 2022 at 9:48 AM Tom Rini wrote:
> > >
> > > On Thu, Aug 11, 2022 at 09:27:44AM -0700, Tim Harvey wrote:
> > >
> > > > Greetings,
> > > >
> > > > After a couple
Hi Quentin,
On Thu, 11 Aug 2022 at 09:04, Quentin Schulz
wrote:
>
> Hi Simon,
>
> On 8/11/22 16:04, Simon Glass wrote:
> > Some image types use the -n parameter to pass in the data file. Add
> > support for this, with a new property.
> >
> > Signed-off-by: Simon Glass
> > ---
> >
> >
Hi Pali,
On Thu, 11 Aug 2022 at 08:50, Pali Rohár wrote:
>
> On Thursday 11 August 2022 08:47:50 Simon Glass wrote:
> > > diff --git a/include/stdio_dev.h b/include/stdio_dev.h
> > > index 270fa2729fb2..06278366ae88 100644
> > > --- a/include/stdio_dev.h
> > > +++ b/include/stdio_dev.h
> > > @@
Hi Pali,
On Thu, 11 Aug 2022 at 08:51, Pali Rohár wrote:
>
> On Thursday 11 August 2022 08:47:53 Simon Glass wrote:
> > On Thu, 11 Aug 2022 at 06:39, Pali Rohár wrote:
> > >
> > > Changing baudrate is sensitive operation. To ensure that U-Boot messages
> >
> > is a sensitive
> >
> > > printed
Sorry forgot to mention. This patch series need to apply on top of my
previous patch series:
https://lists.denx.de/pipermail/u-boot/2022-August/491061.html
On 08/11/2022 04:17 PM, William Zhang wrote:
BCM63158 is one of the Broadcom Broadband origin DSL Gateway router SoC.
It was originally
- Add distro boot to board include file and deconfig file
- Miscellaneous changes:
- Remove Gerald from maintainer list (email bounced)
- Add CONFIG_SUPPORT_PASSING_ATAGS and friends to support legacy
kernel method of booting (e.g. OpenWrt) with appended DTB.
- Add
BCM63158 is one of the Broadcom Broadband origin DSL Gateway router SoC.
It was originally added by Philippe before Broadcom started to upstream
the support for broadband SoCs. The ARM based Broadcom Broadband SoC
family is now supported under the same ARCH_BCMBCA config. This patch
series migrate
BCM63158 carries the CONFIG_SYSRESET from the original configuration. It
provide reset_cpu function already so need to define weak version of the
dummy reset_cpu for other BCMBCA SoCs to avoid linking error.
Signed-off-by: William Zhang
---
board/broadcom/bcmbca/board.c | 2 +-
1 file
Since ARCH_BCM63158 SoC support is merged into ARCH_BCMBCA, add BCM63158
maintainer Philippe to bcmbca maintainer list.
Signed-off-by: William Zhang
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 5b219d62f6bf..d0a5b2352cc8 100644
---
As CONFIG_ARCH_BCM63158 is replaced with CONFIG_BCM63158, update the
driver Kconfig to use the new config symbol
Signed-off-by: William Zhang
---
drivers/gpio/Kconfig | 2 +-
drivers/led/Kconfig | 2 +-
drivers/mtd/nand/raw/Kconfig | 2 +-
drivers/spi/Kconfig | 2 +-
Now that BCM63158 is supported under CONFIG_ARCH_BCMBCA and
CONFIG_BCM63158, remove the original ARCH_BCM63158 support and migrate
configuration settings.
Signed-off-by: William Zhang
---
arch/arm/Kconfig | 8 +---
arch/arm/dts/Makefile| 3 --
BCM63158 is a Broadcom B53 based DSL Gateway SoC. It is part of the
BCA (Broadband Carrier Access origin) chipset family. Like other
Broadband SoC, this patch adds it under CONFIG_BCM63158 chip
config and CONFIG_ARCH_BCMBCA platform config.
This initial support includes a bare-bone implementation
On Mon, May 30, 2022 at 11:05:54AM -0400, Tom Rini wrote:
> On Sat, May 28, 2022 at 09:02:09AM +, Leo Liang wrote:
> > On Fri, May 27, 2022 at 09:30:49AM -0400, Tom Rini wrote:
> > > On Fri, May 27, 2022 at 02:36:29AM +, Leo Liang wrote:
> > >
> > > > Hi Tom,
> > > >
> > > > The
Hi Tom,
The following changes since commit cdebee1fd9fa04cc4c972f826bae19b28c253eb0:
Merge branch '2022-08-10-assorted-updates' (2022-08-10 17:49:20 -0400)
are available in the Git repository at:
https://source.denx.de/u-boot/custodians/u-boot-riscv.git
for you to fetch changes up to
For boards which requires correct MACH_TYPE, set their correct default
values directly in Kconfig.
Signed-off-by: Pali Rohár
---
arch/arm/Kconfig | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 9898c7d68e1b..f78726f7c1be 100644
---
Expression (r + (r0 - r1)) produce same result as (r - (r1 - r0)). So it
does not matter which one is called. Always call the first option and
remove second one.
Signed-off-by: Pali Rohár
---
board/nokia/rx51/lowlevel_init.S | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff
Switch from custom board specific fixup/copy code to U-Boot generic
position independent code provided by config option POSITION_INDEPENDENT.
This also slightly decrease size of u-boot.bin binary (by 52 bytes). Note
that option POSITION_INDEPENDENT increase size but not more than custom
board
Now when board starup code does not copy image to CONFIG_SYS_TEXT_BASE
address there is no need to calculate all addresses from base address at
runtime. The only address which needs to be calculated is attached kernel
image address which can be simplified at compile time without need to know
Some of config options are board specific and should be set in into their
default values automatically. So move them from defconfig file to Kconfig
definitions to ensure that possible user custom defconfig files would have
these required options also enabled.
Signed-off-by: Pali Rohár
---
Prior starting copy of kernel image to target location, invalidate also
zImage magic header. This ensures that on target location would be image
with valid header only in the case valid header was also in the source
location and copy from source to target finished successfully. Copy is
always
On 8/10/22 11:35 AM, Michal Suchánek wrote:
> On Wed, Aug 10, 2022 at 05:17:56PM +0200, Andrew Lunn wrote:
>> > > I guess you are new to the netdev list :-)
>> > >
>> > > This is one of those FAQ sort of things, discussed every
>> > > year. Anything like this is always NACKed. I don't see why
rename eeprom_init to avoid build failure when using CMD_EEPROM.
Signed-off-by: Tim Harvey
---
board/gateworks/venice/eeprom.c | 2 +-
board/gateworks/venice/eeprom.h | 2 +-
board/gateworks/venice/spl.c| 2 +-
board/gateworks/venice/venice.c | 2 +-
4 files changed, 4 insertions(+), 4
Add missing LTE_PWR# and LTE_RST gpio pinmux.
Signed-off-by: Tim Harvey
---
arch/arm/dts/imx8mm-venice-gw7902.dts | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm/dts/imx8mm-venice-gw7902.dts
b/arch/arm/dts/imx8mm-venice-gw7902.dts
index 407ab4592b4c..8e8d0d34c237 100644
---
In some situations the GSC firmware where the EEPROM containing the
model and DRAM configuration may not be ready by the time the SoC
is ready to talk to it over I2C.
Instead of a hard delay, poll the I2C lines to wait until they are
released to avoid the I2C drivers 'Arbitation lost' error
The GW7903 has a BD71847 PMIC on I2C1. Adjust the model compare strings
to add it.
Signed-off-by: Tim Harvey
Reviewed-by: Fabio Estevam
Reviewed-by: Jaehoon Chung
---
v3: add Jaehoon's Reviewed-By tag
v2: fixed typo in commit log and added Fabio's rb tag
---
board/gateworks/venice/spl.c | 9
The GW73xx-C revision and onward replaced the 5-port PCIe switch with a
4-port (dropping PCIe to one of the miniPCIe sockets) due to part
availability. This moved the PCI bus of the GbE eth1 device. Use a fixup
to adjust the dt accordingly so that local-mac-address assigned from dt
works on new
When booting from USB/SDP mmc_get_env_dev() returns
CONFIG_SYS_MMC_ENV_DEV as the MMC env device (while booting from MMC
will call board_mmc_get_env_dev() to get this). Configure
CONFIG_SYS_MMC_ENV_DEV for SDHC3 (devno=2) as all Gateworks Venice
boards use SDHC3 as eMMC so that persistent env
According to the comment block "The default {addr} parameter is one byte
(.1) which works well for memories and registers with 8 bits of address
space."
While this is true for legacy I2C a default length of -1 is being passed
for DM_I2C which results in a usage error.
Restore the documented
On Thu, Aug 11, 2022 at 09:59:05AM -0700, Tim Harvey wrote:
> On Thu, Aug 11, 2022 at 9:48 AM Tom Rini wrote:
> >
> > On Thu, Aug 11, 2022 at 09:27:44AM -0700, Tim Harvey wrote:
> >
> > > Greetings,
> > >
> > > After a couple of hours troubleshooting a bad boot image today I
> > > realized the
On Wed, Aug 03, 2022 at 11:04:45AM +0100, Martin Bonner wrote:
> I and my colleagues have a number of patches we would like to
> contribute back to the community, however for various reasons
> (principally operating inside corporate firewalls), it isn't possible
> to use `git send-email`, and I
On Thu, Aug 11, 2022 at 9:48 AM Tom Rini wrote:
>
> On Thu, Aug 11, 2022 at 09:27:44AM -0700, Tim Harvey wrote:
>
> > Greetings,
> >
> > After a couple of hours troubleshooting a bad boot image today I
> > realized the issue was that I had some 0 byte files for the lpddr4
> > training blobs that
On Thu, Aug 11, 2022 at 05:18:12PM +0300, Matwey V. Kornilov wrote:
> pcm051_rev3_defconfig config (Phytec Wega board) has been dropped in
>
> 64efd11d ("arm: Remove pcm051 board")
>
> due to expired migration deadlines. Here, pcm051_rev3_defconfig support is
> reintroduced.
>
>
On Thu, Aug 11, 2022 at 09:27:44AM -0700, Tim Harvey wrote:
> Greetings,
>
> After a couple of hours troubleshooting a bad boot image today I
> realized the issue was that I had some 0 byte files for the lpddr4
> training blobs that are part of the imx8mp binman created image.
>
> Digging in I
Greetings,
After a couple of hours troubleshooting a bad boot image today I
realized the issue was that I had some 0 byte files for the lpddr4
training blobs that are part of the imx8mp binman created image.
Digging in I found that if a blob referenced in the binman node is
missing a warning
On Wed, Aug 10, 2022 at 09:18:11PM -0600, Simon Glass wrote:
> Hi Tom,
>
> I've dropped the setuptools patch and will look at it later.
>
> https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/13113
>
>
> The following changes since commit 3dd4e916324efc825a7ee8e412f5cf1ded839021:
>
чт, 11 авг. 2022 г. в 15:53, Alex Kiernan :
>
> On Thu, Aug 11, 2022 at 1:46 PM Tom Rini wrote:
> >
> > On Thu, Aug 11, 2022 at 10:47:18AM +0300, Matwey V. Kornilov wrote:
> >
> > > Can somebody please advise me working CONFIG_DEBUG_UART _* values for
> > > the board? I think I need to obtain
Hi Simon,
On 8/11/22 16:04, Simon Glass wrote:
Add a new function to Entry to check for missing properties, since this
is likely to be come a common requirement.
Signed-off-by: Simon Glass
---
tools/binman/entry.py | 16
tools/binman/etype/fill.py | 3 +--
Hi Simon,
On 8/11/22 16:04, Simon Glass wrote:
Expand this a little to make things clearer. Also drop the invalid
entry arg.
Signed-off-by: Simon Glass
---
tools/binman/entries.rst | 28 +---
tools/binman/etype/mkimage.py | 28 +---
2
Hi Simon,
On 8/11/22 16:04, Simon Glass wrote:
Some image types use the -n parameter to pass in the data file. Add
support for this, with a new property.
Signed-off-by: Simon Glass
---
tools/binman/entries.rst | 15 +++
tools/binman/etype/mkimage.py | 26
At present the collections etype only works with entries in the same
section. This can be limiting, since in some cases the data may be inside
a subsection, e.g. if there are alignment constraints.
Add a function to find the entries in an etype and have it search
recursively. Make use of this for
On Thursday 11 August 2022 08:47:53 Simon Glass wrote:
> On Thu, 11 Aug 2022 at 06:39, Pali Rohár wrote:
> >
> > Changing baudrate is sensitive operation. To ensure that U-Boot messages
>
> is a sensitive
>
> > printed before changing baudrate are not lost, call new U-Boot console
> > flush()
On Thursday 11 August 2022 08:47:50 Simon Glass wrote:
> > diff --git a/include/stdio_dev.h b/include/stdio_dev.h
> > index 270fa2729fb2..06278366ae88 100644
> > --- a/include/stdio_dev.h
> > +++ b/include/stdio_dev.h
> > @@ -37,6 +37,10 @@ struct stdio_dev {
> > void (*putc)(struct
Hi ziheng,
On Thu, 11 Aug 2022 at 00:26, 曹子恒 wrote:
>
> Hi, Simon Glass：
> we have an issue based on this patch below, can you help us ?
>
> Patch：https://patchwork.ozlabs.org/project/uboot/patch/20220117150428.1580273-2-...@chromium.org/
>
> Our Environment：
> Qemu
On Thu, 11 Aug 2022 at 06:39, Pali Rohár wrote:
>
> In lot of cases kernel resets UART HW. To ensure that U-Boot messages
In a lot
> printed before booting kernel are not lost, call new U-Boot console flush()
booting the kernel
> function.
>
> Signed-off-by: Pali Rohár
> ---
>
Hi Pali,
On Thu, 11 Aug 2022 at 06:39, Pali Rohár wrote:
>
> On certain places it is required to flush output print buffers to ensure
> that text strings were sent to console or serial devices. For example when
> printing message that U-Boot is going to boot kernel or when U-Boot is
> going to
On Thu, 11 Aug 2022 at 06:39, Pali Rohár wrote:
>
> Changing baudrate is sensitive operation. To ensure that U-Boot messages
is a sensitive
> printed before changing baudrate are not lost, call new U-Boot console
> flush() function.
>
> Signed-off-by: Pali Rohár
> ---
> cmd/load.c
On Thu, 11 Aug 2022 at 06:39, Pali Rohár wrote:
>
> UART drivers have putc/puts functions which just put characters into HW
> transmit queue and do not wait until all data are transmitted.
>
> Implement flush callback via serial driver's pending(false) callback which
> waits until HW transmit all
Hi Pali,
On Thu, 11 Aug 2022 at 06:39, Pali Rohár wrote:
>
> Like in all other console functions, implement also serial_flush() function
> as a fallback int console flush() function.
>
> Flush support is available only when config option CONSOLE_FLUSH_SUPPORT is
> enabled. So when it is disabled
On Thu, 11 Aug 2022 at 05:09, Lee Jones wrote:
>
> Currently the default initialisation frequency is 50MHz. Although
> this does appear to be suitable for some LPDDR4 RAM chips, training at
> this low frequency has been seen to cause Column errors, leading to
> Capacity check errors on others.
>
Hi Michal,
On Thu, 11 Aug 2022 at 06:46, Michal Simek wrote:
>
> bootstage_mark() and bootstate_error() are not recording any name and in
> report it is showing as id=. That's not useful and it is better to
> show function name which calls it.
> That's why use macros with passing __func__ as
On Thu, 11 Aug 2022 at 05:09, Lee Jones wrote:
>
> Frequency changes to 400MHz are presently reported as:
>
> lpddr4_set_rate_0: change freq to 4 mhz 0, 1
>
> This is obviously wrong by 6 orders of magnitude.
>
> Ensure frequency changes are reported accurately.
>
> Signed-off-by: Lee
On Thu, 11 Aug 2022 at 06:39, Pali Rohár wrote:
>
> It flushes stdout.
>
> Signed-off-by: Pali Rohár
> ---
> arch/sandbox/cpu/os.c | 5 +
> include/os.h | 8
> 2 files changed, 13 insertions(+)
Reviewed-by: Simon Glass
On Thu, 11 Aug 2022 at 05:09, Lee Jones wrote:
>
> Functions pointed to by this op pointer can return non-zero values
> indicating an error. Ensure any error value is propagated back up the
> call-chain.
>
> Signed-off-by: Lee Jones
> Tested-by: Xavier Drudis Ferran
> Reviewed-by: Kever Yang
On Thu, Aug 11, 2022 at 08:58:48AM +0100, Lee Jones wrote:
> Currently the default initialisation frequency is 50MHz. Although
> this does appear to be suitable for some LPDDR4 RAM chips, training at
> this low frequency has been seen to cause Column errors, leading to
> Capacity check errors on
pcm051_rev3_defconfig config (Phytec Wega board) has been dropped in
64efd11d ("arm: Remove pcm051 board")
due to expired migration deadlines. Here, pcm051_rev3_defconfig support is
reintroduced.
Signed-off-by: Matwey V. Kornilov
---
Changes since v1:
- Do not add symbols to whitelist
Hi Simon,
On 8/11/22 16:04, Simon Glass wrote:
With the addition of the revision stats this now takes over a minute. Use
a parallel build to reduce it a bit (24 seconds for me).
Signed-off-by: Simon Glass
---
doc/Makefile | 1 +
1 file changed, 1 insertion(+)
diff --git a/doc/Makefile
Also control over what goes in the file passed with -n using a separate
imagename subnode. This can include a section or any other entry type.
Signed-off-by: Simon Glass
---
tools/binman/entries.rst | 18 +
tools/binman/etype/mkimage.py | 39
At present the collections etype only works with entries in the same
section. This can be limiting, since in some cases the data may be inside
a subsection, e.g. if there are alignment constraints.
Add a function to find the entries in an etype and have it search
recursively. Make use of this for
Expand this a little to make things clearer. Also drop the invalid
entry arg.
Signed-off-by: Simon Glass
---
tools/binman/entries.rst | 28 +---
tools/binman/etype/mkimage.py | 28 +---
2 files changed, 42 insertions(+), 14 deletions(-)
Some image types use the -n parameter to pass in the data file. Add
support for this, with a new property.
Signed-off-by: Simon Glass
---
tools/binman/entries.rst | 15 +++
tools/binman/etype/mkimage.py | 26 --
tools/binman/ftest.py
The testReplaceSectionSimple() test is the only one which expects failure.
It looks odd in the output and takes time to glance at it to see that all
is in fact well. Also it does not check that the right exception is
generated.
Use the more common (in binman) approach of checking for an
Since this is implemented as a section, it should really be split into
several functions, one to read the node and one to read the entries. Do
this so that it matches how Entry_section works.
Signed-off-by: Simon Glass
---
tools/binman/etype/mkimage.py | 19 +++
1 file changed,
Add a new function to Entry to check for missing properties, since this
is likely to be come a common requirement.
Signed-off-by: Simon Glass
---
tools/binman/entry.py | 16
tools/binman/etype/fill.py | 3 +--
tools/binman/ftest.py | 2 +-
3 files changed, 18
This has got out of sync and needs a line wrap. Fix it.
Signed-off-by: Simon Glass
---
tools/binman/entries.rst | 3 ++-
tools/binman/etype/pre_load.py | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/tools/binman/entries.rst b/tools/binman/entries.rst
index
1 - 100 of 124 matches
Mail list logo
