Assigning the value of a variable to itself should be avoided.
Addresses-Coverity-ID: 451089 ("Evaluation order violation")
Fixes: 180b7118bed8 ("efi_loader: fix device-path for USB devices")
Signed-off-by: Heinrich Schuchardt
---
lib/efi_loader/efi_device_path.c | 2 +-
1 file changed, 1
Hi Francis,
On Thu, Mar 30, 2023 at 3:14 PM Francis Laniel
wrote:
>
> Hi.
>
> Le jeudi 30 mars 2023, 21:17:33 WEST Tony Dinh a écrit :
> > Hi Francis,
> >
> > On Thu, Mar 30, 2023 at 12:50 PM Francis Laniel
> >
> > wrote:
> > > This commit set CONFIG_HUSH_PARSER_2021 as the default to trigger
;
> Merge tag 'u-boot-at91-2023.07-a' of
> https://source.denx.de/u-boot/custodians/u-boot-at91 into next (2023-03-28
> 11:21:29 -0400)
>
> are available in the Git repository at:
>
> https://gitlab.denx.de/u-boot/custodians/u-boot-imx.git
> tags/u-boot-im
On Fri, Mar 31, 2023 at 03:36:15PM -0500, Judith Mendez wrote:
> List some common boot modes and their corresponding switch
> settings for AM62 SK.
>
> Signed-off-by: Judith Mendez
Thanks for reworking.
Reviewed-by: Tom Rini
--
Tom
signature.asc
Description: PGP signature
List some common boot modes and their corresponding switch
settings for AM62 SK.
Signed-off-by: Judith Mendez
---
Changes from v1: Change table format from ascii-art to list-table
doc/board/ti/am62x_sk.rst | 36
1 file changed, 36 insertions(+)
diff --git
On Fri, Mar 31, 2023 at 01:52:44PM -0500, Andrew Davis wrote:
> On 3/31/23 1:36 PM, Tom Rini wrote:
> > On Fri, Mar 31, 2023 at 01:23:23PM -0500, Andrew Davis wrote:
> > > On 3/31/23 1:11 PM, Tom Rini wrote:
> > > > On Fri, Mar 31, 2023 at 12:19:14PM -0500, Andrew Davis wrote:
> > > > > On 3/31/23
On 3/31/23 1:36 PM, Tom Rini wrote:
On Fri, Mar 31, 2023 at 01:23:23PM -0500, Andrew Davis wrote:
On 3/31/23 1:11 PM, Tom Rini wrote:
On Fri, Mar 31, 2023 at 12:19:14PM -0500, Andrew Davis wrote:
On 3/31/23 12:04 PM, Nishanth Menon wrote:
On 16:06-20230330, Ryan Eatmon wrote:
The oe-core
On Fri, Mar 31, 2023 at 01:23:23PM -0500, Andrew Davis wrote:
> On 3/31/23 1:11 PM, Tom Rini wrote:
> > On Fri, Mar 31, 2023 at 12:19:14PM -0500, Andrew Davis wrote:
> > > On 3/31/23 12:04 PM, Nishanth Menon wrote:
> > > > On 16:06-20230330, Ryan Eatmon wrote:
> > > > > The oe-core class for
On 3/31/23 1:11 PM, Tom Rini wrote:
On Fri, Mar 31, 2023 at 12:19:14PM -0500, Andrew Davis wrote:
On 3/31/23 12:04 PM, Nishanth Menon wrote:
On 16:06-20230330, Ryan Eatmon wrote:
The oe-core class for assembling the fitImage includes the vendor
sub-directory (with the / changed to _) in the
On Fri, Mar 31, 2023 at 12:19:14PM -0500, Andrew Davis wrote:
> On 3/31/23 12:04 PM, Nishanth Menon wrote:
> > On 16:06-20230330, Ryan Eatmon wrote:
> > > The oe-core class for assembling the fitImage includes the vendor
> > > sub-directory (with the / changed to _) in the config sections of
> > >
On Fri, Mar 31, 2023 at 10:25:56AM +1300, Simon Glass wrote:
> The current EFI implementation has a strange quirk where it watches
> loaded files and uses the last-loaded file to determine the device that
> is being booted from.
>
> This is confusing with bootstd, where multiple options may
On Fri, Mar 31, 2023 at 10:26:01AM +1300, Simon Glass wrote:
> This is needed to enable the boot command used to start standard boot.
> Enable it by default. This brings in quite a few features, mostly in
> common with DISTRO_DEFAULTS
>
> Exclude boards which have what looks like a custom boot
On Fri, Mar 31, 2023 at 09:39:53AM +0100, Peter Robinson wrote:
> Hi Simon,
>
> > > > > > > I took a look at Simon's v3 series to fix the rk3399 bootstd
> > > > > > > migration,
> > > > > > > and it changed too much for everything else. I took about half of
> > > > > > > that
> > > > > > >
On 3/31/23 12:04 PM, Nishanth Menon wrote:
On 16:06-20230330, Ryan Eatmon wrote:
The oe-core class for assembling the fitImage includes the vendor
sub-directory (with the / changed to _) in the config sections of
the fitImage. Our env var settings for chosing which section to
boot from needs
On 16:06-20230330, Ryan Eatmon wrote:
> The oe-core class for assembling the fitImage includes the vendor
> sub-directory (with the / changed to _) in the config sections of
> the fitImage. Our env var settings for chosing which section to
> boot from needs to be updated to agree with the
On Fri, Mar 31, 2023 at 10:29:39AM +1300, Simon Glass wrote:
> Hi Tom,
>
> https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15825
>
>
> The following changes since commit 565681e596381b5b6db59cf06931e698f2d829e5:
>
> imx6sx-udoo-neo-basic: Introduce the u-boot.dtsi (2023-03-28
From: Stefan Herbrechtsmeier
This reverts commit e09784728689de7949d4cdd559a9590e0bfcc702. The
commit wrongly divides the dummy bytes by dummy bus width to calculate
the dummy bytes. The framework already converts the dummy cycles to the
number of bytes and the controller use the SPI flash
From: Stefan Herbrechtsmeier
The u-boot hush shell doesn’t support the unset command to clear a
variable and therefore an empty value ("c=") should be a valid value
for the set_local_var function to clear the variable. This partial
reverts commit aa722529635c ("common: cli_hush: avoid dead
From: Manuel Traut
The board is similar to "Kontron SL i.MX8M Mini" SoM. There are two variants.
Snowflake_v2 has an external SDCard Interface. LEIG is the same, but without
display and without external SDCard Interface. Both variants support
only 1GiB of main memory.
[ bigeasy: porting and a
On Mon, Mar 20, 2023 at 09:06:30PM +0200, Svyatoslav Ryhel wrote:
> Remove need of dts binding for button keyboard since it reuses
> gpio-keys binding. Select gpio-keys driver if button keyboard
> is selected since button keyboard can not operate on its own.
>
> Tested-by: Svyatoslav Ryhel #
On Mon, Mar 20, 2023 at 09:01:43PM +0200, Svyatoslav Ryhel wrote:
> This patch introduses config which allows interrupt run of usb
> mass storage with any key. This is especially useful on devices
> with limited input capabilities like tablets and smatphones which
> have only gpio keys in direct
On Mon, Mar 20, 2023 at 08:29:57AM +1300, Simon Glass wrote:
> Reduce the duplicated code slightly by using a helper function to handle
> the common code.
>
> This reduces the code size very slightly.
>
> Signed-off-by: Simon Glass
Applied to u-boot/next, thanks!
--
Tom
signature.asc
On Sun, Mar 19, 2023 at 04:13:02AM +0100, Marek Vasut wrote:
> The SYS_TIMER_COUNTS_DOWN description contains a typo,
> s@rathe@@ , fix it.
>
> Signed-off-by: Marek Vasut
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Description: PGP signature
On Fri, Mar 17, 2023 at 09:12:22PM +0100, Rasmus Villemoes wrote:
> The API is more convenient to use if one doesn't have to know upfront
> which gpio controller has a line with the name one is searching for,
> and arrange to look that device up somehow. Or implement this loop
> oneself.
>
>
On Mon, Feb 27, 2023 at 08:56:31PM +0100, Marek Vasut wrote:
> In case fitImage support is disabled, and image_locate_script() is
> passed a fitImage, then the 'data' variable is used uninitialized.
> Drop into the default: branch of the switch-case statement and do
> not return the uninitialized
On Fri, Mar 10, 2023 at 11:07:15AM -0800, stcar...@linux.microsoft.com wrote:
> From: Stephen Carlson
>
> Adds a test for the new pci_mps command to ensure that it can set the
> Maximum Payload Size (MPS) of all devices to 256 bytes in the sandbox
> environment. Enables the pci_mps command in
On Fri, Mar 10, 2023 at 11:07:14AM -0800, stcar...@linux.microsoft.com wrote:
> From: Stephen Carlson
>
> Reports the sandbox swapcase PCI Express device to support a 256 byte
> Maximum Payload Size for MPS tuning tests.
>
> Signed-off-by: Stephen Carlson
> Reviewed-by: Simon Glass
Applied
On Fri, Mar 10, 2023 at 11:07:13AM -0800, stcar...@linux.microsoft.com wrote:
> From: Stephen Carlson
>
> Enable tuning of the PCI Express MPS (Maximum Payload Size) of
> each device. The Maximum Read Request Size is not altered.
>
> The SAFE method uses the largest MPS value supported by all
On Fri, Mar 10, 2023 at 11:58:03AM +0100, Rasmus Villemoes wrote:
> There can certainly be a lot more elements in the "revisions" (and
> "names") arrays than there are gpios used to form the trinary number
> we're searching for; we simply don't know the array size up-front.
>
> Nor do we need
On Mon, Mar 06, 2023 at 02:27:21PM +0100, Rasmus Villemoes wrote:
> Instead of calling env_get(from) up to three times, just do it once,
> computing the value we will put into 'to' and error out if that is
> NULL (i.e. no 'from' variable and no default provided).
>
> No functional change.
>
>
On Wed, Mar 22, 2023 at 09:46:02AM +0100, Stefan Herbrechtsmeier wrote:
> From: Stefan Herbrechtsmeier
>
> Ensure that a freshly written fat file with a lower case filename which
> fits into the upper case 8.3 short filename is not mangeled with a tilde
> and number.
>
> Signed-off-by: Stefan
On Mon, Mar 20, 2023 at 11:23:14AM +0300, Evgeny Bachinin wrote:
> Fix warnings after adding printf-like attribute format for
> run_commandf():
> warning: too many arguments for format [-Wformat-extra-args]
>
> Signed-off-by: Evgeny Bachinin
> Reviewed-by: Simon Glass
> Reviewed-by: Simon
On Mon, Mar 20, 2023 at 11:23:13AM +0300, Evgeny Bachinin wrote:
> Fix warnings both for 32bit and 64bit architecture after adding
> printf-like attribute format for run_commandf():
> warning: format ‘%x’ expects argument of type ‘unsigned int’, but
> argument 2 has type ‘ulong {aka long
On Fri, Mar 03, 2023 at 11:31:22PM +0100, Heinrich Schuchardt wrote:
> Kconfig settings that are related to the API for standalone applications
> should be in the API sub-menu and not on the top level.
>
> CONFIG_STANDALONE_LOAD_ADDR is only relevant if standalone example
> applications are
On Tue, Feb 28, 2023 at 07:22:21AM +0100, Marek Vasut wrote:
> Sort the defaults list in BUILD_TARGET Kconfig option. No functional change.
>
> Signed-off-by: Marek Vasut
> Reviewed-by: Simon Glass
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Description: PGP signature
On Thu, Feb 23, 2023 at 07:03:47AM +0100, Heiko Schocher wrote:
> disable CONFIG_LEGACY_IMAGE_FORMAT so prevent of booting
> not signed fitimages.
>
> Signed-off-by: Heiko Schocher
Applied to u-boot/next, thanks!
--
Tom
signature.asc
Description: PGP signature
On Fri, Mar 17, 2023 at 01:15:12PM +0100, Corentin GUILLEVIC wrote:
> The deletion process handles special case for symlinks whose target are
> small enough that it fits in struct ext2_inode.b.symlink. So no block had
> been allocated. But the check of file type wrongly considered regular
> files
On Mon, Mar 20, 2023 at 11:23:12AM +0300, Evgeny Bachinin wrote:
> As run_commandf() is variadic version of run_command() and just a wrapper,
> hence apply similar run_command's test-cases.
>
> Let's avoid warning about empty string passing:
> warning: zero-length gnu_printf format string
On Fri, Mar 17, 2023 at 01:04:13PM +0100, Stefan Herbrechtsmeier wrote:
> From: Stefan Herbrechtsmeier
>
> Do not mangle lower or mixed case filenames which fit into the upper
> case 8.3 short filename. This ensures FAT standard compatible short
> filenames (SFN) to support systems without long
On Mon, Mar 20, 2023 at 11:23:11AM +0300, Evgeny Bachinin wrote:
> * vsnprintf() can truncate cmd, hence it makes no sense to launch such
> command (it's broken). Moreover, it's better to signalize to the caller
> about such case (for facilitating debugging or bug hunting).
>
> * Fix kernel-doc
On Mon, Mar 20, 2023 at 10:23:44AM +1300, Chris Packham wrote:
> Adding support for Analog Devices MAX313XX series RTCs.
>
> This is ported from the Linux driver and adapted for use in u-boot.
> Notable differences are
> - handling of tm_year and tm_mon differ
> - clock source support is omitted
On Mon, Mar 20, 2023 at 10:23:43AM +1300, Chris Packham wrote:
> The find_closest() macro can be used to find an element in a sorted
> array that is closest to an input value. Bring in this macro from
> Linux v6.3-rc1-2-g8ca09d5fa354.
>
> Signed-off-by: Chris Packham
> Reviewed-by: Simon Glass
On Fri, Mar 17, 2023 at 03:22:41PM +0100, thomas.per...@bootlin.com wrote:
> From: Thomas Perrot
>
> These operations are required by dm_rtc_read and
> dm_bootcount_get helpers.
>
> Signed-off-by: Thomas Perrot
> Reviewed-by: Simon Glass
Applied to u-boot/next, thanks!
--
Tom
On 2023-03-16 11:33:47 [+0100], Frieder Schrempf wrote:
> > diff --git a/arch/arm/dts/imx8mm-mt-snowflake-v2.dts
> > b/arch/arm/dts/imx8mm-mt-snowflake-v2.dts
> > new file mode 100644
> > index 0..49761b22afcf0
> > --- /dev/null
> > +++ b/arch/arm/dts/imx8mm-mt-snowflake-v2.dts
> > @@
Hi,
I changed from v2020.10 to v2022.10 and suddenly I get a bunch of "Bootstage
space exhasuted" messages during bootm.
reloc pc : [] is memcpy()
It evens triggers a reset. Older version works just fine:
Sign value: unavailable
Timestamp:unavailable
Verifying Hash
On Friday 31 March 2023 18:35:00 Minda Chen wrote:
> On 2023/3/30 1:27, Pali Rohár wrote:
> > Hello!
> >
> > On Wednesday 29 March 2023 18:01:41 Minda Chen wrote:
> >> + /* PCIe PCI Standard Configuration Identification Settings. */
> >> + value = (PCI_CLASS_BRIDGE_PCI_NORMAL <<
Hi Bryan,
On 28/03/23 12:10 am, Bryan Brattlof wrote:
> Update the uboot dtsi to enable DMA and CPSW at the uboot level
>
> Signed-off-by: Bryan Brattlof
> ---
> arch/arm/dts/k3-am62a7-r5-sk.dts | 8
> arch/arm/dts/k3-am62a7-sk-u-boot.dtsi | 26 +-
> 2
Implement a ARM SMCCC based driver that allow to use
a secure watchdog on the platform.
Signed-off-by: Lionel Debieve
---
drivers/watchdog/Kconfig | 8 +++
drivers/watchdog/Makefile | 1 +
drivers/watchdog/arm_smc_wdt.c | 116 +
3 files changed,
---
drivers/mmc/sdhci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
index c6b250b9a1..4e57e12732 100644
--- a/drivers/mmc/sdhci.c
+++ b/drivers/mmc/sdhci.c
@@ -882,7 +882,7 @@ int sdhci_setup_cfg(struct mmc_config *cfg, struct
Hi Simon
On 31/03/23 02:01, Simon Glass wrote:
Hi Neha,
On Fri, 24 Mar 2023 at 22:28, Neha Malcom Francis wrote:
Hi Simon,
Before I roll out the entire series that works for packaging K3
bootloader images, wanted to get some reviews and comments regarding the
implementation of the signing
On 2023/3/30 1:27, Pali Rohár wrote:
> Hello!
>
> On Wednesday 29 March 2023 18:01:41 Minda Chen wrote:
>> From: Mason Huo
>>
>> Add pcie driver for StarFive JH7110, the driver depends on
>> starfive gpio, pinctrl, clk and reset driver to do init.
>>
>> Several devices are tested:
>> a) M.2
On 3/31/23 00:55, Heinrich Schuchardt wrote:
Am 30. März 2023 23:32:22 MESZ schrieb Simon Glass :
At present it is not possible to read from some CDROM drives since the
FAT sector size does not match the media's block size. Add a conversion
option for this, so that reading is possible.
This
On 3/31/23 09:33, Ilias Apalodimas wrote:
On Fri, 31 Mar 2023 at 09:51, Heinrich Schuchardt wrote:
Am 31. März 2023 03:16:13 MESZ schrieb Simon Glass :
Hi Heinrich,
On Fri, 31 Mar 2023 at 13:05, Heinrich Schuchardt wrote:
Am 31. März 2023 01:49:35 MESZ schrieb Simon Glass :
Hi
> Date: Fri, 31 Mar 2023 00:31:17 +0200
> From: Heinrich Schuchardt
>
> Am 30. März 2023 23:32:03 MESZ schrieb Simon Glass :
> >Some files have an associated address. Show this with the 'qfw list'
> >command so that it is possible to dump the data.
> >
> >Signed-off-by: Simon Glass
> >---
> >
>
Enable USB command, USB drivers, PHY and regulators, for USB host
operations.
Signed-off-by: Eugen Hristev
---
configs/rock5b-rk3588_defconfig | 18 +-
1 file changed, 17 insertions(+), 1 deletion(-)
diff --git a/configs/rock5b-rk3588_defconfig b/configs/rock5b-rk3588_defconfig
Add initial support for the rk3588 PHY variant.
The driver now looks for phy-supply and enables/disables the vbus
accordingly.
The lookup for the host-port reg inside the struct now does a do {} while()
instead of a while() {} in order to allow a first check for reg == 0.
Co-developed-by: Frank
Add USB 2.0 host nodes and PHYs.
Co-developed-by: William Wu
Signed-off-by: William Wu
Signed-off-by: Eugen Hristev
---
Changes in v2,v3:
- none
arch/arm/dts/rk3588-rock-5b-u-boot.dtsi | 169
1 file changed, 169 insertions(+)
diff --git
Simplify the subsystem by renaming `dev_pdata` to just `plat`.
No functional change, just trivial renaming.
Suggested-by: Simon Glass
Signed-off-by: Eugen Hristev
Reviewed-by: Simon Glass
---
Changes in v3,v4:
- none
Changes in v2:
- this is a new patch
drivers/power/regulator/fan53555.c
Some devices share a regulator supply, when the first one will request
regulator disable, the second device will have it's supply cut off before
graciously shutting down. Hence there will be timeouts and other failed
operations.
Implement a reference counter mechanism similar with what is done in
From: Peter Hoyes
The vexpress64 board family now relies on OF_CONTROL and
OF_HAS_PRIOR_STAGE, so platform-specific configuration requirements
are minimal.
The vexpress_aemv8a_semi_defconfig file defines many flags that are not
needed for a minimal boot, such as flash memory configuration.
From: Peter Hoyes
The Arm EBBR (Embedded Base Boot Requirements) require that the time
and basic networking EFI interfaces are available and working, so long
as the hardware has an RTC and network interface.
Arm FVPs typically have a memory-mapped PL031 RTC and a VIRTIO_NET
device defined in
From: Peter Hoyes
BASE_FVP now typically uses a devicetree provided by a prior boot stage
(typically Arm TF-A), so imply this option by default when
TARGET_VEXPRESS64_BASE_FVP is selected.
OF_HAS_PRIOR_STAGE selects OF_BOARD so this change is minor, but aligns
TARGET_VEXPRESS64_BASE_FVP with
> The Linux kernel device tree already specifies the device to be used for
> boot console output with a stdout-path property under /chosen.
> Commit 36b661dc919da ("Merge branch 'next'") re-added the console
> setting that commit bede82f750752 ("configs: imx8mn_bsh_smm_s2: remove
> console from
(2023-03-28 11:21:29 -0400)
are available in the Git repository at:
https://gitlab.denx.de/u-boot/custodians/u-boot-imx.git
tags/u-boot-imx-next-20230331
for you to fetch changes up to f216580b642c7244aeb3619aeceecb331e025ad3:
clk: imx8mp: add pwm clocks support (2023-03-30 13:53:55
Hi Simon,
> > > > > > I took a look at Simon's v3 series to fix the rk3399 bootstd
> > > > > > migration,
> > > > > > and it changed too much for everything else. I took about half of
> > > > > > that
> > > > > > series and then reworked a few things. Now only rk3399 platforms
> > > > > >
Hi Abdellatif,
On Tue, Mar 28, 2023 at 05:11:56PM +0100, Abdellatif El Khlifi wrote:
> Add MM communication support using FF-A transport
>
> This feature allows accessing MM partitions services through
> EFI MM communication protocol. MM partitions such as StandAlonneMM
> or smm-gateway secure
Hi Lionel,
On 3/31/23 09:14, Lionel Debieve wrote:
Implement a ARM SMCCC based driver that allow to use
a secure watchdog on the platform.
Signed-off-by: Lionel Debieve
---
drivers/watchdog/Kconfig | 8 +++
drivers/watchdog/Makefile | 1 +
drivers/watchdog/arm_smc_wdt.c |
On Fri, 31 Mar 2023 at 09:51, Heinrich Schuchardt wrote:
>
>
>
> Am 31. März 2023 03:16:13 MESZ schrieb Simon Glass :
> >Hi Heinrich,
> >
> >On Fri, 31 Mar 2023 at 13:05, Heinrich Schuchardt wrote:
> >>
> >>
> >>
> >> Am 31. März 2023 01:49:35 MESZ schrieb Simon Glass :
> >> >Hi Heinrich,
> >> >
Am 31. März 2023 03:16:13 MESZ schrieb Simon Glass :
>Hi Heinrich,
>
>On Fri, 31 Mar 2023 at 13:05, Heinrich Schuchardt wrote:
>>
>>
>>
>> Am 31. März 2023 01:49:35 MESZ schrieb Simon Glass :
>> >Hi Heinrich,
>> >
>> >On Fri, 31 Mar 2023 at 11:48, Heinrich Schuchardt
>> >wrote:
>> >>
>> >>
Am 31. März 2023 03:16:06 MESZ schrieb Simon Glass :
>Hi Heinrich,
>
>On Fri, 31 Mar 2023 at 13:27, Heinrich Schuchardt wrote:
>>
>>
>>
>> Am 31. März 2023 01:48:57 MESZ schrieb Simon Glass :
>> >Hi Heinrich,
>> >
>> >On Fri, 31 Mar 2023 at 11:38, Heinrich Schuchardt
>> >wrote:
>> >>
>> >>
From: Sinthu Raja
This resyncs the dts file of j721e-sk with the currently in-tree K3
platforms. Of note are that the main-navss/mcu-navss nodes were renamed
to main_navss / mcu_navss and so the u-boot.dtsi file needed to be
updated to match.
Also, disable the HBMC which is not supported in the
On Fri, Mar 31, 2023 at 5:36 AM Simon Glass wrote:
>
> Hi,
>
> I notice that in image.h we have IH_ARCH_RISCV but no mention of
> IH_ARCH_RISCV64. Should we not have two separate image types, as we do
> with ARM and x86? Otherwise, how would a loader know the word size of
> the target machine?
I
72 matches
Mail list logo