so
mentions a comma as separator.
Signed-off-by: Vincent Stehlé
Cc: Tom Rini
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Maxim Uvarov
---
cmd/nvedit.c | 2 +-
cmd/nvedit_efi.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/cmd/nvedit.c b/cmd/nvedit.c
in
synquacer_developerbox_defconfig
xilinx_versal_virt_defconfig
xilinx_zynq_virt_defconfig
xilinx_zynqmp_kria_defconfig
Best regards,
Vincent.
Vincent Stehlé (5):
fwu: developerbox: fix dfu alt buffer clearing
board: st: common: fix dfu alt buffer clearing
arm64: versal: fix dfu alt buffer clearing
xilinx
uld indeed work fine for an array, in the present case we
end up clearing only the size of a pointer.
Fix this by specifying the explicit size `DFU_ALT_BUF_LEN' instead.
Fixes: c67fecd2125b ("ARM: zynq: Enable capsule update for qspi and mmc")
Signed-off-by: Vincent Stehlé
Cc: M
uld indeed work fine for an array, in the present case we
end up clearing only the size of a pointer.
Fix this by specifying the explicit size `DFU_ALT_BUF_LEN' instead.
Fixes: ec2933e543df ("board: stm32mp1: move set_dfu_alt_info in st common
directory")
Signed-off-by: Vincent Ste
uld indeed work fine for an array, in the present case we
end up clearing only the size of a pointer.
Fix this by specifying the explicit size `DFU_ALT_BUF_LEN' instead.
Fixes: 6b403ca4dcf4 ("fwu: DeveloperBox: add support for FWU")
Signed-off-by: Vincent Stehlé
Cc: Masahisa Kojima
uld indeed work fine for an array, in the present case we
end up clearing only the size of a pointer.
Fix this by specifying the explicit size `DFU_ALT_BUF_LEN' instead.
Fixes: 064c8978b44f ("arm64: versal: Enable capsule update (SD)")
Signed-off-by: Vincent Stehlé
Cc: Michal Simek
uld indeed work fine for an array, in the present case we
end up clearing only the size of a pointer.
Fix this by specifying the explicit size `DFU_ALT_BUF_LEN' instead.
Fixes: b86f43de0be0 ("xilinx: zynqmp: Add support for runtime dfu_alt_info
setup")
Signed-off-by: Vincent Stehlé
r 0x0
Fix this by verifying the number of input arguments instead.
Fixes: e3cf80fbe02d ("cmd: Add support for optee commands")
Signed-off-by: Vincent Stehlé
Cc: Jens Wiklander
Cc: Ilias Apalodimas
Cc: Tom Rini
Cc: Jerome Forissier
Cc: Venkatesh Yadav Abbarapu
---
cmd/optee.c | 2 +-
1 fil
The new_packagelist() function of the HII Protocols implementation is
calling malloc() without checking its return code; fix this.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Tom Rini
---
lib/efi_loader/efi_hii.c | 3 +++
1 file changed, 3 insertions
ments in the dp_msging() function to fix this
and obtain the following expected string instead:
IPv4(5.6.7.8,TCP,DHCP,1.2.3.4,9.10.11.12,255.255.255.0)
Fixes: aaf63429a112 ("efi_loader: add IPv4() to device path to text protocol")
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuch
Fixes: 4782ac80b02f ("Add AHCI support to u-boot")
Signed-off-by: Vincent Stehlé
Cc: Tom Rini
Cc: Jason Jin
---
drivers/ata/ahci.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c
index 8058d5ff1c3..e593e228685 100644
--- a/drivers/ata/ahci.c
++
On Wed, Mar 12, 2025 at 11:12:35AM -0300, Fabio Estevam wrote:
> Hi Vincent,
>
> On Mon, Mar 10, 2025 at 9:36 AM Vincent Stehlé wrote:
>
> > --- a/arch/arm/mach-imx/imx8m/soc.c
> > +++ b/arch/arm/mach-imx/imx8m/soc.c
> > @@ -1270,8 +1270,9 @@ static int ft_ad
the only in-tree i.MX8M system where both CONFIG_OPTEE and
CONFIG_OF_SYSTEM_SETUP are defined (see the imx8mm-cl-iot-gate*
defconfigs).
Deal with an existing optee node gracefully at runtime to fix this issue.
Signed-off-by: Vincent Stehlé
Cc: Stefano Babic
Cc: Fabio Estevam
Cc: Tom Rini
Cc
Add more EFI protocols GUIDs to the translation table used by
uuid_guid_get_str().
Signed-off-by: Vincent Stehlé
Cc: Tom Rini
---
Hi,
While the EFI_LOADED_IMAGE_DEVICE_PATH_PROTOCOL_GUID, the
EFI_HII_CONFIG_ACCESS_PROTOCOL_GUID and the EFI_LOAD_FILE_PROTOCOL_GUID are
actually used by the
Allow the ZyncMP RTC driver to be enabled on Versal architectures.
Also, require DM_RTC since the driver uses the RTC driver model.
Signed-off-by: Vincent Fazio
---
Changes in v3:
- Revert a typo in the help text
Changes in v2:
- Add DM_RTC dependency (Michal)
---
drivers/rtc/Kconfig | 2
> -Original Message-
> From: Vincent Fazio
> Sent: Tuesday, January 14, 2025 7:31 AM
> To: michal.si...@amd.com; u-boot@lists.denx.de
> Cc: tr...@konsulko.com; vfa...@gmail.com; Vincent Fazio inc.com>
> Subject: [PATCH v2] rtc: zynqmp: allow on Versal archite
Allow the ZyncMP RTC driver to be enabled on Versal architectures.
Also, require DM_RTC since the driver uses the RTC driver model.
Signed-off-by: Vincent Fazio
---
Changes in v2:
- Add DM_RTC dependency (Michal)
---
drivers/rtc/Kconfig | 4 ++--
1 file changed, 2 insertions(+), 2 deletions
Allow the ZyncMP RTC driver to be enabled on Versal architectures.
Signed-off-by: Vincent Fazio
---
drivers/rtc/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
index 9c2d1398247..ecdcfe1f240 100644
--- a/drivers/rtc/Kconfig
: cccea18813c4 ("efi_loader: add the number of image entries in
efi_capsule_update_info")
Signed-off-by: Vincent Stehlé
Cc: Masahisa Kojima
Cc: Tuomas Tynkkynen
Cc: Tom Rini
---
board/emulation/qemu-arm/qemu-arm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board
ks.
BTW, I though recent Rasmus series [1] was related, but it also works
without, so I dunno if I actually tested it or not.
Thanks for the hints
[1] https://patchwork.ozlabs.org/project/uboot/list/?series=423625
--
Vincent Legoll
type: 0fc63daf-8483-4772-8e79-3d69d8477de4
(linux)
guid: d74fd9eb-00df-4bc8-94db-ddd4fade2040
=> mmc dev 0
switch to partitions #0, OK
mmc0(part 0) is current device
=> mmc dev 1 3
mmc_select_hwpart() - 2 - ERROR EMEDIUMTYPE because MMCPART_NOAVAILABLE
switch to partitions #3, ERROR
--
Vincent Legoll
omething wrong, but I can't see it.
I'd like some hints about how to get further.
Thanks
--
Vincent Legoll
DDR 9fffbe1e78 cym 24/02/04-10:09:20,fwver: v1.16
LPDDR4X, 2112MHz
channel[0] BW=16 Col=10 Bk=8 CS0 Row=17 CS1 Row=17 CS=2 Die BW=8 Size=4096MB
channel[1] BW=16 Col=10 Bk=8 CS0 Row
h more efficient. It also avoids
> allocating things 'in space' in conflict with the loaded images.
Hi Simon,
Thank you for this series.
I did test it on top of v2024.10-rc4 with AArch64 simulators (FVP & Qemu), with
OS boots and ACS-IR, and I see no regression. Same wi
On Tue, Jul 02, 2024 at 03:30:49PM +0200, Caleb Connolly wrote:
Hi Caleb,
Thanks for re-spinning this series; it looks very good.
My comments below.
Best regards,
Vincent.
> Add a tool that can generate GUIDs that match those generated internally
> by U-Boot for capsule update fw_images
The scan_part() function uses a struct uuid to store the little-endian
partition type GUID, but this structure should be used only to contain a
big-endian UUID. Use an efi_guid_t instead and use guidcmp() for the
comparison.
Suggested-by: Heinrich Schuchardt
Signed-off-by: Vincent Stehlé
Cc
Move the guidcmp() and guidcpy() functions to efi.h, near the definition of
the efi_guid_t type those functions deal with.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Tom Rini
---
include/efi.h| 10 ++
include/efi_loader.h | 10 --
2
know and I will respin.
There is no difference in the sandbox binaries before/after this
series on Arm and on PC, and all the tests I have run on the sandbox
are unchanged.
Best regards,
Vincent.
[1]
https://patchwork.ozlabs.org/project/uboot/patch/20240627170629.2696427-1-vincent.ste...@arm.com/
On Thu, Jun 27, 2024 at 09:28:04PM +0200, Heinrich Schuchardt wrote:
>
Hi Heinrich,
Thanks for your review.
My comments below.
Best regards,
Vincent.
>
> Am 27. Juni 2024 19:06:29 MESZ schrieb "Vincent Stehlé"
> :
> >The scan_part() function uses a struct uui
The scan_part() function uses a struct uuid to store the little-endian
partition type GUID, but this structure should be used only to contain a
big-endian UUID. Use an efi_guid_t instead.
Signed-off-by: Vincent Stehlé
Cc: Simon Glass
Cc: Tom Rini
---
boot/bootmeth_cros.c | 4 ++--
1 file
7d8852c60d
fd5db83c-12f3-a46b-80a9-e3007c7ff56e
ffd97379-0956-fa94-c003-8bfcf5cc097b
- Also, a few minor modifications here and there.
Signed-off-by: Vincent Stehlé
Cc: Caleb Connolly
Cc: Tom Rini
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Simon Glass
Cc: Mario Six
Cc: Alper Nebi
On Fri, Jun 21, 2024 at 07:11:28PM +0200, Caleb Connolly wrote:
> On Fri, 21 Jun 2024, 19:06 Vincent Stehlé, wrote:
>
> > On Fri, Jun 21, 2024 at 01:00:51PM +0200, Heinrich Schuchardt wrote:
> > (..)
> > > The current specification is in RFC 9562, 4.1, "Variant
at for both
UUIDs and GUIDs, which uuid -d understands, and which has no notion of
endianness. (Only the structures in memory have an endianness.)
This means we do not need an output format parameter.
genguid is calling the new gen_uuid_v5() function, which outputs a big-endian
struct uuid. Therefore, genguid should print it with 'STD format, not 'GUID.
Best regards,
Vincent.
>
> Best regards
>
> Heinrich
On Wed, Jun 19, 2024 at 10:15:32PM +0300, Ilias Apalodimas wrote:
> Allô Vincent,
Hi Ilias :)
>
> Thanks for testing!
>
> On Wed, 19 Jun 2024 at 17:02, Vincent Stehlé wrote:
> >
> > On Fri, May 31, 2024 at 03:50:34PM +0200, Caleb Connolly wrote:
> > > As m
On Fri, May 31, 2024 at 03:50:40PM +0200, Caleb Connolly wrote:
> Add a tool that can generate GUIDs that match those generated internally
> by U-Boot for capsule update fw_images.
Hi Caleb,
Thanks for working on this.
I think there is a leftover "dtb" option; see below.
Best
1)
content: A8:F6:AE:40:D8:A7:08:F0:3E:05:A2:2F:94:EC:A9:EC
(not decipherable: truncated SHA-1 message digest only)
Best regards,
Vincent.
[1] https://uuid.ramsey.dev/en/stable/rfc4122/version5.html
>
> The well-known salt GUID can be specific to the arc
The README.trace has been moved and converted to rst in commit dce26c7d56ed
("doc: move README.trace to HTML documentation"); fix all the remaining
references to this file.
Signed-off-by: Vincent Stehlé
Cc: Tom Rini
Cc: Simon Glass
Cc: Heinrich Schuchardt
---
c
7;
instead, which we reallocate when needed.
This fixes the following error:
String too short (500 chars)
While at it, fix a few typos in strings and comments.
Signed-off-by: Vincent Stehlé
Cc: Tom Rini
Cc: Simon Glass
Cc: Michal Simek
---
Hi,
This can be verified using the `test_trace.py&
Stefan
> -Original Message-
> From: Stefan Wahren
> Sent: Tuesday, May 16, 2023 12:54 AM
> To: Vincent Fazio ; Nuno Gonçalves
> ; u-boot@lists.denx.de; pbrobin...@gmail.com
> Subject: Re: [External] - Re: Issues with bcm2835-host: let firmware manage
> the clock div
Stefan
> -Original Message-
> From: Stefan Wahren
> Sent: Monday, May 15, 2023 11:35 AM
> To: Vincent Fazio ; Nuno Gonçalves
> ; u-boot@lists.denx.de; pbrobin...@gmail.com
> Subject: Re: [External] - Re: Issues with bcm2835-host: let firmware manage
> the clock di
All
> -Original Message-
> From: Stefan Wahren
> Sent: Sunday, May 14, 2023 1:55 PM
> To: Nuno Gonçalves ; u-boot@lists.denx.de; Vincent
> Fazio ; pbrobin...@gmail.com
> Subject: [External] - Re: Issues with bcm2835-host: let firmware manage the
> clock divisor
again from the above
address or is there another way of loading the two MAC addresses from an
EEPROM and putting them in the environment at u-boot start-up? Any
pointers or advice would be welcome. Thanks!
Kind regards,
Vincent
--
National Institute for Subatomic Physics Nikhef
Departm
-- - --
Scanning global bootmeth 'efi_mgr':
Scanning bootdev 'virtio-blk#30.bootdev':
No more bootdevs
--- -- - --
(0 bootflows, 0 valid)
The messages seem to indicate that virtio #31 / 1 was not even considere
boot when I invert the two virtio disks.
Best regards,
Vincent.
[1]: https://lists.denx.de/pipermail/u-boot/2023-April/514527.html
#35).
Hi Mathew,
Thank you for this patch; it works for me and is much simpler that what I had in
mind.
- Your patch repairs efi with virtio, for Qemu arm and aarch64.
- It does not break efi on USB and NVMe (on Qemu).
Best regards,
Vincent.
. Sadly, I am
not able to confirm if this is a proper fix as I am hitting other boot
issues in the case of multiple boot devices of the same class it seems...
Vincent.
[1]: https://buildroot.org
---
boot/bootmeth_efi.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --
Fix a couple of links so that they are rendered correctly with sphinx.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
---
doc/develop/uefi/fwu_updates.rst | 3 ++-
doc/develop/uefi/uefi.rst| 4 ++--
2 files changed, 4 insertions(+), 3 deletions(-)
diff
Add a test for the case when the HII database protocol
set_keyboard_layout() function is called with a NULL key_guid argument.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
---
lib/efi_selftest/efi_selftest_hii.c | 12
1 file changed, 12 insertions
As per the EFI specification, the HII database protocol function
set_keyboard_layout() must return EFI_INVALID_PARAMETER when it is called
with a NULL key_guid argument. Modify the function accordingly to improve
conformance.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
Cc: Ilias
Hi,
The following couple of patches make UEFI HII set_keyboard_layout() more
conforming in the case of invalid input parameter and add a selftest.
This is sent in this order to avoid breaking `bootefi selftest' in the middle
but feel free to apply in any order if preferred.
Best regards,
Vi
Add a couple of tests for the cases when the HII database protocol
get_package_list_handle() function is called with invalid parameters.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
---
Hi,
Thanks Heinrich for your review and feedbacks; here is v2 of this patch
On Thu, Dec 22, 2022 at 10:45:22AM +0100, Heinrich Schuchardt wrote:
Hi Heinrich,
Happy new year, and thank you for the reviews.
More comments below.
> On 12/13/22 22:39, Vincent Stehlé wrote:
> > Add a test for the case when the HII database protocol
> > get_package_list_handl
The EFI Conformance Profile Table entry for EBBR appears in v2.1.0 of the
EBBR specification[1]. Update naming accordingly.
While at it, update the EBBR version referenced in the documentation.
[1]: https://github.com/ARM-software/ebbr/releases/tag/v2.1.0
Signed-off-by: Vincent Stehlé
Cc
Add a test for the case when the HII database protocol
get_package_list_handle() function is called with an invalid package list
handle.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
---
lib/efi_selftest/efi_selftest_hii.c | 10 ++
1 file changed, 10
When the HII protocol function get_package_list_handle() is called with an
invalid package list handle, it returns EFI_NOT_FOUND but this is not in
its list of possible status codes as per the EFI specification.
Return EFI_INVALID_PARAMETER instead to fix conformance.
Signed-off-by: Vincent
Hi,
The following couple of patches fixes a small UEFI HII conformance issue and
adds a selftest demonstrating the issue.
This is sent in this order to avoid breaking `bootefi selftest' in the middle
but feel free to apply in any order if preferred.
Best regards,
Vincent.
Vincent Steh
al size
probing").
Add the same workaround to the UEFI authenticated capsules tests to repair
them.
This can be tested with sandbox_defconfig, sandbox64_defconfig or
sandbox_flattree_defconfig, plus CONFIG_EFI_CAPSULE_AUTHENTICATE=y.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuch
CONFIG_EFI_CAPSULE_AUTHENTICATE=y.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
---
lib/efi_loader/efi_firmware.c | 115 +++---
test/py/tests/test_efi_capsule/conftest.py| 21 +++-
...py => test_capsule_firmware_signed_fit.py} | 41 ---
..
GUID typo introduced by commit 2e9c3c6965ba ("test:
capsule: Modify the capsule tests to use GUID values for sandbox").
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
---
test/py/tests/test_efi_capsule/conftest.py | 4 ++--
.../tests/test_efi_capsule/test_capsul
w_get_image_info'
./lib/efi_loader/efi_firmware.c:395: warning: Function parameter or member
'package_version_name' not described in 'efi_firmware_raw_get_image_info'
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
---
lib/efi_loader/efi_firmware.c | 10 +---
The firmware management protocol can be used to manage device firmware.
U-Boot can be configured to provide an implementation.
Document the related functions in the API section.
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
---
doc/api/efi.rst | 6 ++
1 file changed, 6 insertions
Frederick
> -Original Message-
> From: U-Boot On Behalf Of Frederik Lotter
> Sent: Monday, December 6, 2021 12:55 AM
> To: u-boot@lists.denx.de
> Subject: [External] - Raspberry Pi Boot 1b / Uboot boot failure with large
> initramfs
>
> Hi,
>
> I am facing intermittend boot failures re
: commit 47a25e81d35c ("Revert "efi_capsule: Move signature from DTB to
.rodata"")
Signed-off-by: Vincent Stehlé
Cc: Simon Glass
---
board/emulation/common/Makefile | 1 -
1 file changed, 1 deletion(-)
diff --git a/board/emulation/common/Makefile b/board/emulation/common/Makefile
Ivan,
> -Original Message-
> From: Ivan T. Ivanov
> Sent: Wednesday, November 10, 2021 4:01 AM
> To: Vincent Fazio
> Cc: Mario Lombardo ; u-boot@lists.denx.de
> Subject: Re: [External] - Slow MMC IO with Raspberry CM3+ Module
>
> On 11-03 20:57, Vincent Fazio
in 206978 ms (114.3 KiB/s)
>
> The configuration of by build is attached.
>
> Is there anyone else experiencing this kind of issues?
>
> Thank you very much.
>
> Mario
>
>
> CAUTION: This email originated from outside of the organization. Do
> not click links or open attachments unless you recognize the sender
> and know the content is safe.
-Vincent
- Pine64 ROCKPro64 with U-Boot "near" v2021.07[1]
- Lenovo Leez P710 with U-Boot v2021.07[2]
- Compulab IOT-GATE-iMX8 with U-Boot "near" v2021.10-rc3[3]
Feel free to add (or not):
Tested-by: Vincent Stehlé
Best regards,
Vincent Stehlé
System Architect - Arm
[1]:
https
patchwork series [2].
The series may not be necessary with newer firmware.
Any suggestions ?
Do you have a USB keyboard plugged in?
[1] https://github.com/raspberrypi/firmware/issues/1619
[2] https://patchwork.ozlabs.org/project/uboot/list/?series=262375
--
Vincent Fazio
Embedded Software
g | 2 ++
configs/rpi_defconfig| 2 ++
12 files changed, 44 insertions(+), 8 deletions(-)
--
Vincent Fazio
Embedded Software Engineer - ATS
Extreme Engineering Solutions, Inc
http://www.xes-inc.com
://github.com/raspberrypi/linux/commit/08532d242d7702ae0add95096aa49c5e96e066e2
Signed-off-by: Vincent Fazio
---
arch/arm/mach-bcm283x/include/mach/mbox.h | 16 +++
arch/arm/mach-bcm283x/include/mach/msg.h | 10 +
arch/arm/mach-bcm283x/msg.c | 28
drivers/mmc
returns zero.
Signed-off-by: Vincent Fazio
---
arch/arm/mach-bcm283x/include/mach/mbox.h | 2 ++
arch/arm/mach-bcm283x/msg.c | 19 ++-
2 files changed, 20 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-bcm283x/include/mach/mbox.h
b/arch/arm/mach-bcm283x
zero.
Additionally, let the firmware manage SDCDIV when possible.
Vincent Fazio (2):
arm: rpi: fallback to max clock rate for MMC clock
mmc: bcm2835-host: let firmware manage the clock divisor
arch/arm/mach-bcm283x/include/mach/mbox.h | 18
arch/arm/mach-bcm283x/include/mach/msg.h
sion of this patch.
Hi Arnaud,
You are right, this patch solves the issue I am seeing on the ROCKPro64.
Thanks for the pointer.
Best regards,
Vincent Stehlé
System Architect - Arm
.
Signed-off-by: Vincent Stehlé
---
THIS IS A WORKAROUND
PLEASE DO NOT APPLY!
drivers/clk/rockchip/clk_rk3399.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/clk/rockchip/clk_rk3399.c
b/drivers/clk/rockchip/clk_rk3399.c
(see the patch following this e-mail).
I don't know enough about the RK3399 clock tree to fully understand what is
going on here, and what would be a proper fix. Hopefully others will be able to
continue from there.
Best regards,
Vincent Stehlé
System Architect - Arm
Vincent Stehlé (1):
Just a gentle ping.
If this patchset has any problems, please let me know. I am willing to
modify it. Thank you.
On Thu, Jul 8, 2021 at 9:08 AM Vincent Chen wrote:
>
> In SiFive unmatched board, the reset of the USB hub, PCIe-USB bridge, and
> ULPI rely on the power-cycling. However,
Ensure USB hub, PCIe-USB bridge, and ULPI device to be reset
even if the rebooting is without power-cycling.
Signed-off-by: Vincent Chen
---
board/sifive/unmatched/spl.c | 36
1 file changed, 36 insertions(+)
diff --git a/board/sifive/unmatched/spl.c b
Create a new function spl_reset_device_by_gpio to reset the device
whose reset pin is connected to the GPIO. Then, using this function
to initialize GEMGXL.
Signed-off-by: Vincent Chen
---
board/sifive/unmatched/spl.c | 58 +---
1 file changed, 39
, because the reset pint of these four devices incluing GEMGXL
connects to the GPIO, the 1st patch creates a new wrapper,
spl_reset_device_by_gpio(), to address the GPIO operation during the reset.
Vincent Chen (2):
board: sifive: unmatched: refine GEMGXL initialized function in SPL
board: sifive
UpdateCapsule() must return EFI_INVALID_PARAMETER in a number of cases,
listed by the UEFI specification and tested by the SCT. Add a common
function to do that.
This fixes SCT UpdateCapsule_Conf failures.
Reviewed-by: Grant Likely
Signed-off-by: Vincent Stehlé
Cc: Heinrich Schuchardt
Cc
d reset after 2 second and test will be resumed after
reboot.resetting ...
Writing sandbox state
qemu: uncaught target signal 11 (Segmentation fault) - core dumped
With your patch I do not hit this segfault anymore.
FWIW, feel free to add (or not):
Tested-by: Vincent Stehlé
Best
The pwm_sifive_set_config() and pwm_sifive_set_enable() cannot work
properly due to the wrong implementations. It will cause the u-boot
PWM command to not work as expected. The bugs will be resolved in this
patch.
Signed-off-by: Vincent Chen
---
drivers/pwm/pwm-sifive.c | 21
QueryVariableInfo() must return EFI_INVALID_PARAMETER when an invalid
combination of attribute bits is supplied.
This fixes three SCT QueryVariableInfo_Conf failures.
Signed-off-by: Vincent Stehlé
Reviewed-by: Grant Likely
Cc: Heinrich Schuchardt
Cc: Alexander Graf
Changes since v1
The pwm_sifive_set_config() and pwm_sifive_set_enable() cannot work
properly due to the wrong implementations. It will cause the u-boot
PWM command to not work as expected. The bugs will be resolved in this
patch.
Signed-off-by: Vincent Chen
---
drivers/pwm/pwm-sifive.c | 21
s the following `bootefi' error:
Missing RNG device for EFI_RNG_PROTOCOL
Signed-off-by: Vincent Stehlé
Cc: Simon Glass
---
arch/sandbox/dts/sandbox.dtsi | 4
1 file changed, 4 insertions(+)
diff --git a/arch/sandbox/dts/sandbox.dtsi b/arch/sandbox/dts/sandbox.dtsi
index dc933f3bfc7
VIRTIO_ID_MAX_NUM is the largest device ID plus 1. Therefore a device id
cannot be greater or equal to VIRTIO_ID_MAX_NUM. Fix the comparison
accordingly.
Fixes: 8fb49b4c7a82 ("dm: Add a new uclass driver for VirtIO transport devices")
Signed-off-by: Vincent Stehlé
Cc: Simon Glass
Cc
I put my log here https://pastebin.com/5aPjntnb , but i use standard
DT-less u-boot.img.
Maybe it could be easier to talk in u-boot IRC?
2018-03-06 15:55 GMT+01:00 Jagan Teki :
> Hi Michael, Vincent,
>
> On Tue, Mar 6, 2018 at 7:57 PM, Vincent Prince
> wrote:
> > Hi all,
>
Hi all,
Jagan , do you confirm your mx6_usb_sdp_spl.conf contains the 3rd line?
mx6_spl_sdp
hid,uboot_header,1024,0x91,0x1000,512M,0x0090,0x4
u-boot-dtb.img:jump header2
BR,
Vincent
2018-03-06 15:18 GMT+01:00 Michael Nazzareno Trimarchi <
mich...@amarulasolutions.com>:
s to start upstreaming early, with a small feature set
> only (just serial + clocks + pinctrl, as for the H6). Additional
> features (PMIC) might then add small incompatibilities (like this one
> here), until we are reasonably confident about the DT.
> Does
Hi,
I think you can also create an ext4 image with extra space in it. You can
use bmap tool if size is a concern.
Regards,
Vincent
2018-01-12 19:22 GMT+01:00 Adam Lee :
> I was hoping to get this done in the bootloader otherwise I have to change
> the rootfs ;)
>
> On Fri, Jan 12,
Hi all,
It's my first patch contribution, do I need to do anything more ?
Best regards,
Vincent
2017-10-26 13:52 GMT+02:00 Stefan Agner :
>
>
> On 26.10.2017 13:25, Lukasz Majewski wrote:
> > Hi Vincent,
> >
> >> This prevents board resets when calling sdp
This prevents board resets when calling sdp command on boards which have a
watchdog.
Signed-off-by: Vincent Prince
---
drivers/usb/gadget/f_sdp.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/usb/gadget/f_sdp.c b/drivers/usb/gadget/f_sdp.c
index 0fae66b..c3eba6d 100644
--- a
This prevents board resets when calling sdp command on boards which have a
watchdog.
Signed-off-by: Vincent Prince
---
drivers/usb/gadget/f_sdp.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/usb/gadget/f_sdp.c b/drivers/usb/gadget/f_sdp.c
index 0fae66b..c3eba6d 100644
--- a
Hello,
Thank you for your answers. I found a solution. The problem came from the
environment variables. I just used the wrong argument with the command
"setenv". Know I have "eth0:on" everywhere and I reach to do all the run update
again.
Vincent
> Hi Vi
ARP Retry count exceeded; starting again
I tried with three different APF27 board that were working previously.
Therefore, it seems that it is a problem of the u-boot or a configuration
done later that turns off ethernet.
Does someone have any idea to solve this problem?
Thanks
Vincent
--
Sent
> SIMPLEFB is also used by other platforms, but most platforms also
> won't use it.
>
> Adding an imply VIDEO_DT_SIMPLEFB to VIDEO_DE2 would make everyone
> happy I guess.
You meant adding a "select VIDEO_DT_SIMPLEFB" to V
Hi Stefan,
It was probably a Windows bug as it is almost working now.
I was wondering why sdp command is expecting a jump ?
Is it necessary to jump after USB transfer, or we should download
anything we want and jump if asked ?
Regards,
Vincent
2017-09-08 18:37 GMT+02:00 Stefan Agner :
>
e following:
#vid:pid, config_file
0x15a2:0x0061, mx6_usb_rom.conf, 0x0525:0xb4a4, mx6_usb_sdp_spl.conf
0x0525:0xa4a5, mx6_usb_uboot.conf
Am I missing something,
Thanks,
Vincent
2017-09-01 22:09 GMT+02:00 Stefan Agner :
>
>
> On September 1, 2017 12:25:44 PM PDT, Fabio Estevam
> wrote
Hello,
> * Renamed sunxi_[eg]mac to sun[47]_mac to be more clear and allowing preparing
> to merge common parts into sunxi_common
Wouldn't those better renamed to sun[47]i_mac instead, for consistency ?
--
Vincent Legoll
___
U-Boot mai
Hi Fabio, Stefan,
Indeed i saw it this morning with great enthusiasm,
I'm currently in vacation, I'll test it in two weeks when I'll back to work.
Thanks,
Vincent
2017-08-05 14:22 GMT+02:00 Fabio Estevam :
> Hi Vincent,
>
> On Thu, Jul 27, 2017 at 6:05 AM,
following:
- https://lists.denx.de/pipermail/u-boot/2015-May/215573.html
- https://lists.denx.de/pipermail/u-boot/2015-June/215606.html
Hi everyone,
I'd like to know if something is going on regarding this topic ?
I managed to flash a yocto BSP with SPL with USB+serial with Fabio help,
but I fo
For completness, here is the original post url :
https://patchwork.ozlabs.org/patch/245097/
https://lists.denx.de/pipermail/u-boot/2013-May/154752.html
2017-07-13 15:11 GMT+02:00 Vincent Prince :
> Hi all,
>
> We had the same issue on our custom board, and this patch fixed it.
&
1 - 100 of 247 matches
Mail list logo
