Hi Heinrich,
>-Original Message-
>From: Heinrich Schuchardt
>Sent: 12 November 2020 12:49
>To: Pragnesh Patel ; Simon Glass
>
>Cc: atish.pa...@wdc.com; palmerdabb...@google.com; u-boot@lists.denx.de;
>bmeng...@gmail.com; Paul Walmsley ( Sifive) ;
>anup.pa...@wdc.com; Sagar Kadam ;
Hi Heinrich,
>-Original Message-
>From: Heinrich Schuchardt
>Sent: 12 November 2020 12:44
>To: Simon Glass
>Cc: Pragnesh Patel ; u-boot@lists.denx.de;
>Heinrich Schuchardt
>Subject: [PATCH 1/1] trace: avoid infinite recursion
>
>[External Email] Do not click links or attachments unless
On 11/12/20 8:09 AM, Heinrich Schuchardt wrote:
> On 11/12/20 7:34 AM, Pragnesh Patel wrote:
>> Hi Heinrich,
>>
>>> -Original Message-
>>> From: Heinrich Schuchardt
>>> Sent: 11 November 2020 19:18
>>> To: Pragnesh Patel
>>> Cc: atish.pa...@wdc.com; palmerdabb...@google.com;
When tracing functions is enabled this adds calls to
__cyg_profile_func_enter() and __cyg_profile_func_exit() to the traced
functions.
__cyg_profile_func_exit() and __cyg_profile_func_exit() invoke
timer_get_us() to record the entry and exit time.
If timer_get_us() or any function used to
On 11/12/20 7:34 AM, Pragnesh Patel wrote:
> Hi Heinrich,
>
>> -Original Message-
>> From: Heinrich Schuchardt
>> Sent: 11 November 2020 19:18
>> To: Pragnesh Patel
>> Cc: atish.pa...@wdc.com; palmerdabb...@google.com; u-boot@lists.denx.de;
>> bmeng...@gmail.com; Paul Walmsley ( Sifive)
Hi Heinrich,
>-Original Message-
>From: Heinrich Schuchardt
>Sent: 11 November 2020 19:18
>To: Pragnesh Patel
>Cc: atish.pa...@wdc.com; palmerdabb...@google.com; u-boot@lists.denx.de;
>bmeng...@gmail.com; Paul Walmsley ( Sifive) ;
>anup.pa...@wdc.com; Sagar Kadam ;
>r...@andestech.com;
Hello Fabio,
Am 12.11.2020 um 03:11 schrieb Fabio Estevam:
Hi,
I am trying to load SPL and u-boot-dtb.img via USB using the
imx_usb_loader tool.
Running the latest mainline U-Boot on a imx6q-sabresd I get:
$ sudo ./imx_usb SPL
On the console:
U-Boot SPL 2021.01-rc2-5-g832bfad7451e (Nov
On Wed, Nov 11, 2020 at 04:25:53PM +0800, Leo Liang wrote:
> Hi Atish and Heinrich,
>
> On Tue, Oct 13, 2020 at 12:23:31PM -0700, Atish Patra wrote:
> > RV32 should use PE32 format instead of PE32+ as the efi header format.
> > This requires following changes
> > 1. A different header magic value
Hi,
I am trying to load SPL and u-boot-dtb.img via USB using the
imx_usb_loader tool.
Running the latest mainline U-Boot on a imx6q-sabresd I get:
$ sudo ./imx_usb SPL
On the console:
U-Boot SPL 2021.01-rc2-5-g832bfad7451e (Nov 11 2020 - 23:00:05 -0300)
Trying to boot from USB SDP
SDP:
Hi,
On 11/11/20 7:25 PM, 김호연지기 wrote:
> On Wed, Nov 11, 2020 at 7:07 PM Jorge Ramirez-Ortiz, Gmail
> wrote:
>>
>> On 11/11/20, Jaehoon Chung wrote:
>>> On 11/10/20 11:28 PM, Hoyeonjiki Kim wrote:
The function mmc_offset_try_partition searches MMC partition to save the
environment data
Add a handler for SIGILL, SIGBUS, SIGSEGV.
When an exception occurs print the program counter and the loaded
UEFI binaries and reset the system if CONFIG_SANDBOX_CRASH_RESET=y
or exit to the OS otherwise.
Signed-off-by: Heinrich Schuchardt
---
v2:
add a customizing switch
set
Implement the commands
* exception undefined - execute an illegal instruction
* exception sigsegv - cause a segment violation
Here is a possible output:
=> exception undefined
Illegal instruction
pc = 0x55eb8d0a7575, pc_reloc = 0x57575
Resetting ...
Signed-off-by: Heinrich
Provide a unit test that causes an illegal instruction to occur.
The test can be run with the following commands:
=> setenv efi_selftest exception
=> bootefi selftest
This might be the output:
Executing 'exception'
EFI application triggers exception.
Illegal instruction
Test that an exception SIGILL is answered by a reset on the sandbox if
CONFIG_SANDBOX_CRASH_RESET=y or by exiting to the OS otherwise.
Signed-off-by: Heinrich Schuchardt
---
v2:
new patch
---
arch/Kconfig | 1 +
test/py/tests/test_sandbox_exit.py | 24
Currently if an exception SIGILL, SIGBUS, SIGSEGV occurs the sandbox
stops execution. This does not match the behavior on other architectures.
Instead print the current program counter and if any the involved UEFI
binaries. Then depending on a customizing system either reset the system
or exit to
Hi Heinrich,
On Wed, 11 Nov 2020 at 13:47, Heinrich Schuchardt wrote:
>
> On 11.11.20 15:32, Simon Glass wrote:
> > Hi Heinrich,
> >
> > On Tue, 10 Nov 2020 at 16:09, Heinrich Schuchardt
> > wrote:
> >>
> >> Add a handler for SIGILL, SIGBUS, SIGSEGV.
> >>
> >> When an exception occurs print
On 11.11.20 15:32, Simon Glass wrote:
> Hi Heinrich,
>
> On Tue, 10 Nov 2020 at 16:09, Heinrich Schuchardt wrote:
>>
>> Add a handler for SIGILL, SIGBUS, SIGSEGV.
>>
>> When an exception occurs print the program counter and the loaded
>> UEFI binaries and reset the system.
>>
>> Signed-off-by:
The AT91 architecture now has two possible timer blocks, the old PIT timer
and the new PIT64B.
The timer.c file has an old non DM driver that works for platforms
that do not use the ATMEL_PIT_TIMER DM-based driver.
Update the Makefile to select this old driver in case neither of the
From: Nicolas Ferre
SAMA7G5 uses CCF driver under drivers/clk/at91/ and not the custom older
at91 clock.c driver. Remove it from the compilation list and adapt cpu.c
arch_cpu_init() to avoid calling at91_clock_init() which is wrong
anyway.
Signed-off-by: Nicolas Ferre
Reviewed-by: Claudiu
On 23.10.2020 19:05, Tom Rini wrote:
Hello Tom,
The at91/atmel related devicetrees should be fixed now in 2021.01-rc2.
Do you plan to update the DTC for 2021.01 final or for a future release ?
Thanks,
Eugen
On Fri, Oct 16, 2020 at 03:42:50PM +0100, Andre Przywara wrote:
> This is a combination of upstream libfdt commits to fix warnings about
> comparing signed and unsigned integers:
> ==
> scripts/dtc/libfdt/fdt.c: In function ‘fdt_offset_ptr’:
> scripts/dtc/libfdt/fdt.c:137:18: warning:
On Wed, Nov 11, 2020 at 11:07:11AM +, eugen.hris...@microchip.com wrote:
> On 23.10.2020 19:05, Tom Rini wrote:
>
> Hello Tom,
>
> The at91/atmel related devicetrees should be fixed now in 2021.01-rc2.
>
> Do you plan to update the DTC for 2021.01 final or for a future release ?
I think
Hi Simon,
On Wed, Nov 11, 2020 at 07:42:31AM -0700, Simon Glass wrote:
> Hi Ilias,
>
> On Wed, 11 Nov 2020 at 02:18, Ilias Apalodimas
> wrote:
> >
> > Since U-boot EFI implementation is getting richer it makes sense to
> > add support for EFI_TCG2_PROTOCOL taking advantage of any hardware TPM
On Mon, 9 Nov 2020 at 07:46, Michal Simek wrote:
>
> There is no need for GD to be used and priv variable is unused.
>
> Signed-off-by: Michal Simek
> ---
>
> drivers/tpm/tpm2_tis_spi.c | 3 ---
> 1 file changed, 3 deletions(-)
Reviewed-by: Simon Glass
Hi Ilias,
On Wed, 11 Nov 2020 at 02:18, Ilias Apalodimas
wrote:
>
> Since U-boot EFI implementation is getting richer it makes sense to
> add support for EFI_TCG2_PROTOCOL taking advantage of any hardware TPM
> available on the device.
>
> This is the initial implementation of the protocol which
On Wed, 11 Nov 2020 at 02:18, Ilias Apalodimas
wrote:
>
> A following patch introduces EFI_TCG2_PROTOCOL.
> Add the required TPMv2 headers to support it.
>
> Signed-off-by: Ilias Apalodimas
> Reviewed-by: Heinrich Schuchardt
> ---
> * changes since v3:
> - Add TPM2 prefix on PT_GROUP, PT_FIXED
While assigning the sequence number to subsystem instances by reading the
aliases property, only DT nodes names are compared and not the complete
path. This causes a problem when there are two DT nodes with same name but
have different paths.
Fix it by comparing the phandles of DT nodes after
+Heinrich Schuchardt
On Fri, 6 Nov 2020 at 10:55, Patrick Delaunay wrote:
>
> Use the debug uart functions to output the traces before
> the log initialization (when CONFIG_LOG is not activated)
> as it is done in puts/putc function in console.c.
>
> This patch allows to display the first U-Boot
On Thu, 5 Nov 2020 at 14:58, Ilias Apalodimas
wrote:
>
> For implementing the EFI_TCG2_PROTOCOL we need the count field returned by
> the TPM when reading capabilities via tpm2_get_capability().
>
> Adjust the implementation of the 'tpm2 get_capability' command accordingly.
>
> Suggested-by:
Hi Heinrich,
On Wed, 4 Nov 2020 at 17:29, Heinrich Schuchardt wrote:
>
> The tests in test/lib/test_print.c fail without CONFIG_CONSOLE_RECORD=y.
>
> Add a build dependency.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> test/lib/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Hi Patrick,
On Fri, 6 Nov 2020 at 10:55, Patrick Delaunay wrote:
>
> Update _log function to drop any traces when log is yet initialized:
> vsnprintf is no more executed in this case.
>
> This patch allows to reduce the cost for the dropped early debug trace.
>
> Signed-off-by: Patrick Delaunay
On Tue, 10 Nov 2020 at 00:49, Ilias Apalodimas
wrote:
>
> A following patch introduces EFI_TCG2_PROTOCOL.
> Add the required TPMv2 headers to support it.
>
> Signed-off-by: Ilias Apalodimas
> ---
> changes since v2:
> - Added description and pointers to TCG specs
> - updated copyright info
>
On Fri, 6 Nov 2020 at 10:55, Patrick Delaunay wrote:
>
> Reduce the log overhead when the traces are filtered,
> by moving the vsnprintf call from _log() to log_dispatch().
>
> This patch avoids the printf treatment when LOG features is
> activated, but trace is filtered, for example when
>
Hi Alper,
On Mon, 9 Nov 2020 at 17:55, Alper Nebi Yasak wrote:
>
> On 09/11/2020 22:37, Simon Glass wrote:
> > Hi Heinrich,
> >
> > On Mon, 9 Nov 2020 at 12:34, Heinrich Schuchardt wrote:
> >>
> >> On 10/30/20 6:25 PM, Alper Nebi Yasak wrote:
> >>> The cros_ec_keyb driver currently uses
Hi,
On Mon, 9 Nov 2020 at 14:25, Heinrich Schuchardt wrote:
>
> On 11/9/20 10:13 PM, Alper Nebi Yasak wrote:
> > On 09/11/2020 23:34, Heinrich Schuchardt wrote:
> >> With commit 690079767803 ("cros_ec: Support keyboard scanning with
> >> EC_CMD_GET_NEXT_EVENT") check_for_keys() tries to read
On Tue, 10 Nov 2020 at 16:09, Heinrich Schuchardt wrote:
>
> Provide a unit test that causes an illegal instruction to occur.
>
> The test can be run with the following commands:
>
> => setenv efi_selftest exception
> => bootefi selftest
>
> This might be the output:
>
> Executing
On Wed, 4 Nov 2020 at 16:29, Heinrich Schuchardt wrote:
>
> Compiling with CONFIG_CMD_CPU=y and CONFIG_CPU=n fails with:
>
> aarch64-linux-gnu-ld.bfd: cmd/built-in.o: in function `print_cpu_list':
> /home/user/u-boot/cmd/cpu.c:34: undefined reference to `cpu_get_desc'
> aarch64-linux-gnu-ld.bfd:
Hi Heinrich,
On Tue, 10 Nov 2020 at 16:09, Heinrich Schuchardt wrote:
>
> Add a handler for SIGILL, SIGBUS, SIGSEGV.
>
> When an exception occurs print the program counter and the loaded
> UEFI binaries and reset the system.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> arch/sandbox/cpu/os.c
On Tue, 10 Nov 2020 at 07:00, Alper Nebi Yasak wrote:
>
> Since commit 690079767803 ("cros_ec: Support keyboard scanning with
> EC_CMD_GET_NEXT_EVENT") the cros-ec-keyb driver has started using this
> command, but the sandbox EC emulator does not recognize it and
> continuously prints:
>
> **
On Tue, 10 Nov 2020 at 16:09, Heinrich Schuchardt wrote:
>
> Implement the commands
>
> * exception undefined - execute an illegal instruction
> * exception sigsegv - cause a segment violation
>
> Here is a possible output:
>
> => exception undefined
> Illegal instruction
> pc =
Hi Heinrich,
On Mon, 9 Nov 2020 at 00:42, Heinrich Schuchardt wrote:
>
> On 09.11.20 05:08, Simon Glass wrote:
> > This test cannot work unless CONFIG_BLOBLIST is enabled. Update it to add
> > that condition.
> >
> > Reported-by: Kever Yang
> > Signed-off-by: Simon Glass
> > ---
> >
> >
On 11/11/20 12:56 PM, Pragnesh Patel wrote:
Hi Heinrich,
-Original Message-
From: Heinrich Schuchardt
Sent: 11 November 2020 16:51
To: Pragnesh Patel ; u-boot@lists.denx.de
Cc: atish.pa...@wdc.com; palmerdabb...@google.com; bmeng...@gmail.com;
Paul Walmsley ( Sifive) ;
Lock dcache before clearing INIT_RAM.
More importantly, invalidate dcache contents before using it as RAM.
Signed-off-by: Joakim Tjernlund
---
Something odd happend, on a stable mpc8321 board, small unrelated code
changes made the board unbootable. Debugging with an emulator
it looked like
So far we used the separate mksunxiboot tool for generating a bootable
image for Allwinner SPLs, probably just for historical reasons.
Use the mkimage framework to generate a so called eGON image the
Allwinner BROM expects.
The new image type is called "sunxi_egon", to differentiate it
from the
Hi,
digging this out after two years [1] ;-)
I fixed the minor comments from v1 and rebased it, then repeated
the tests. Apart from two trivial fixes there were no changes, so
I kept the R-b: tags. For a changelog see below.
==
So far creating a bootable SPL image for Allwinner
Switch the SPL boot image generation from using mksunxiboot to the new
sunxi_egon format of mkimage.
Verified to create identical results for all 152 Allwinner boards.
Signed-off-by: Andre Przywara
Reviewed-by: Simon Glass
---
scripts/Makefile.spl | 8
1 file changed, 4
Hi,
I got a nanopc-t4 amongst others which shipped with:
DDR Version 1.15 20181010
Channel 0: LPDDR3, 933MHz
Bus Width=32 Col=10 Bank=8 Row=15/15 CS=2 Die Bus-Width=32 Size=2048MB
..
I have since upgraded to more recent u-boot versions:
U-Boot TPL 2020.07 (Sep 27 2020 - 12:34:15)
Channel 0:
Hi Heinrich,
>-Original Message-
>From: Heinrich Schuchardt
>Sent: 11 November 2020 16:51
>To: Pragnesh Patel ; u-boot@lists.denx.de
>Cc: atish.pa...@wdc.com; palmerdabb...@google.com; bmeng...@gmail.com;
>Paul Walmsley ( Sifive) ; anup.pa...@wdc.com;
>Sagar Kadam ; r...@andestech.com;
On 11.11.20 11:14, Pragnesh Patel wrote:
> Add timer_get_us() which is useful for tracing.
> For S-mode U-Boot, CSR_TIMEH and CSR_TIME will provide
> a timer ticks and For M-mode U-Boot, mtime register will
> provide the same.
>
> Signed-off-by: Pragnesh Patel
The default implementation of
If CONFIG_FIT_CIPHER=y and CONFIG_FIT_SIGNATURE=n then mkimage and
dumpimage will fail to link:
/usr/bin/ld: tools/common/image-cipher.o: in function `fit_image_decrypt_data':
image-cipher.c:(.text+0x9a): undefined reference to `image_get_host_blob'
/usr/bin/ld:
When CONFIG_FIT_CIPHER=y and CONFIG_FIT_SIGNATURE=n is there is no
implementation of image_get_host_blob for mkimage or dumpimage:
/usr/bin/ld: tools/common/image-cipher.o: in function `fit_image_decrypt_data':
image-cipher.c:(.text+0x9a): undefined reference to `image_get_host_blob'
The
Hello,
I was learning about signed FIT today and was unable to convince mkimage
to produce a signature node in my control device tree. It turns out that
Debian's packaged mkimage doesn't enable FIT_SIGNATURE, so the options I
was passing were silently ignored.
There's an existing Debian bug for
When CONFIG_FIT_SIGNATURE=n the signing options are not available. If a
user is careful they will notice this when looking at the help output.
If they are not careful they will waste several hours wondering what
they got wrong, as mkimage will silently ignore the signing related
options.
Make it
On Wed, Nov 11, 2020 at 7:07 PM Jorge Ramirez-Ortiz, Gmail
wrote:
>
> On 11/11/20, Jaehoon Chung wrote:
> > On 11/10/20 11:28 PM, Hoyeonjiki Kim wrote:
> > > The function mmc_offset_try_partition searches MMC partition to save the
> > > environment data by name. However, it only compares the
Add timer_get_us() which is useful for tracing.
For S-mode U-Boot, CSR_TIMEH and CSR_TIME will provide
a timer ticks and For M-mode U-Boot, mtime register will
provide the same.
Signed-off-by: Pragnesh Patel
---
Changes in v3:
- Added gd->arch.plmt in global data
- For timer_get_us(), use
This series add a support of tracing for RISC-V arch.
This series is also available here [1] for testing.
[1] https://github.com/pragnesh26992/u-boot/tree/trace
How to test this patch:
1) Enable tracing in "configs/sifive_fu540_defconfig"
CONFIG_TRACE=y
CONFIG_TRACE_BUFFER_SIZE=0x0100
On 11/11/20, Jaehoon Chung wrote:
> On 11/10/20 11:28 PM, Hoyeonjiki Kim wrote:
> > The function mmc_offset_try_partition searches MMC partition to save the
> > environment data by name. However, it only compares the first word-size
> > bytes (size of 'const char *'), which may make the function
Since U-boot EFI implementation is getting richer it makes sense to
add support for EFI_TCG2_PROTOCOL taking advantage of any hardware TPM
available on the device.
This is the initial implementation of the protocol which only adds
support for GetCapability(). It's limited in the newer and safer
A following patch introduces EFI_TCG2_PROTOCOL.
Add the required TPMv2 headers to support it.
Signed-off-by: Ilias Apalodimas
Reviewed-by: Heinrich Schuchardt
---
* changes since v3:
- Add TPM2 prefix on PT_GROUP, PT_FIXED
* changes since v2:
- Added description and pointers to TCG specs
-
Hi Rick,
>-Original Message-
>From: Rick Chen
>Sent: 10 November 2020 13:17
>To: Pragnesh Patel
>Cc: U-Boot Mailing List ; Atish Patra
>; Bin Meng ; Paul Walmsley (
>Sifive) ; Anup Patel ; Sagar
>Kadam ; Simon Glass ; Sean
>Anderson ; palmerdabb...@google.com; rick
>; Alan Kao
Hi Atish and Heinrich,
On Tue, Oct 13, 2020 at 12:23:31PM -0700, Atish Patra wrote:
> RV32 should use PE32 format instead of PE32+ as the efi header format.
> This requires following changes
> 1. A different header magic value
> 2. An additional parameter known as BaseOfData. Currently, it is set
On 11/10/20 11:28 PM, Hoyeonjiki Kim wrote:
> The function mmc_offset_try_partition searches MMC partition to save the
> environment data by name. However, it only compares the first word-size
> bytes (size of 'const char *'), which may make the function to find
> unintended partition.
>
>
Hi Bin
On Tuesday, November 10, 2020 5:55 PM, Ran Wang wrote:
>
> Hi Bin,
>
> > > record_transfer_result():comp_code: 1 csw org fbb4ec90,
> > > adjusted to fbb4fffe
> > > record_transfer_result():comp_code: 1
> > > record_transfer_result():comp_code: 1
> > >
63 matches
Mail list logo