Re: [U-Boot] [PATCH] travis: give every job a name

2018-08-02 Thread Stephen Warren
On 07/30/2018 10:19 AM, Stephen Warren wrote: From: Stephen Warren Travis CI now supports giving jobs an explicit name. Do this for all jobs. This allows more direct control over jobs names than the previous automatic or implicit naming based on the environment variables or script text. Does

Re: [U-Boot] [PATCH] efi_loader: don't allocate unusable RAM

2018-08-02 Thread Stephen Warren
On 08/01/2018 11:57 PM, Heinrich Schuchardt wrote: On 08/01/2018 06:17 PM, Stephen Warren wrote: On 08/01/2018 12:07 AM, Heinrich Schuchardt wrote: On 07/31/2018 09:44 PM, Stephen Warren wrote: From: Stephen Warren Some boards define a maximum usable RAM top that's more restrictive tha

Re: [U-Boot] [PATCH] ARM: tegra: align carveout size

2018-08-02 Thread Stephen Warren
TK1 tests all look good again. Tom -Original Message- From: Tom Rini Sent: Wednesday, August 1, 2018 4:47 PM To: Stephen Warren Cc: Tom Warren ; u-boot@lists.denx.de; Simon Glass ; Stephen Warren ; Mark Kettenis ; Alexander Graf Subject: Re: [U-Boot] [PATCH] ARM: tegra: align car

Re: [U-Boot] [PATCH] ARM: tegra: align carveout size

2018-08-01 Thread Stephen Warren
On 07/31/2018 12:38 PM, Stephen Warren wrote: From: Stephen Warren Align the size of the carveout region to 2M. This ensures that the size can be accurately represented by an LPAE page table that uses sections. This solves a bug (hang at boot time soon after printing the DRAM size) that only

Re: [U-Boot] [PATCH] efi_loader: don't allocate unusable RAM

2018-08-01 Thread Stephen Warren
On 08/01/2018 12:07 AM, Heinrich Schuchardt wrote: On 07/31/2018 09:44 PM, Stephen Warren wrote: From: Stephen Warren Some boards define a maximum usable RAM top that's more restrictive than the ranges defined by U-Boot's memory bank definitions[1]. In this case, the unusable RAM is

Re: [U-Boot] [PATCH] efi_loader: don't allocate unusable RAM

2018-07-31 Thread Stephen Warren
On 07/31/2018 02:15 PM, Alexander Graf wrote: Am 31.07.2018 um 22:04 schrieb Stephen Warren : On 07/31/2018 02:03 PM, Alexander Graf wrote: Am 31.07.2018 um 21:44 schrieb Stephen Warren : From: Stephen Warren Some boards define a maximum usable RAM top that's more restrictive tha

Re: [U-Boot] [PATCH] efi_loader: don't allocate unusable RAM

2018-07-31 Thread Stephen Warren
On 07/31/2018 02:03 PM, Alexander Graf wrote: Am 31.07.2018 um 21:44 schrieb Stephen Warren : From: Stephen Warren Some boards define a maximum usable RAM top that's more restrictive than the ranges defined by U-Boot's memory bank definitions[1]. In this case, the unusable RAM is

[U-Boot] [PATCH] efi_loader: don't allocate unusable RAM

2018-07-31 Thread Stephen Warren
From: Stephen Warren Some boards define a maximum usable RAM top that's more restrictive than the ranges defined by U-Boot's memory bank definitions[1]. In this case, the unusable RAM isn't mapped in the page tables, and so the EFI code must not attempt to allocate RAM from out

[U-Boot] Regression due to "efi_loader: efi_allocate_pages is too restrictive"

2018-07-31 Thread Stephen Warren
Commit aa909462d018 "efi_loader: efi_allocate_pages is too restrictive" causes a failure (synchronous abort) in the command "dhcp zImage" on board p2371-2180 (Jetson TX1). Reverting this commit alone solves the problem. According to the exception dump, elr/pc is in efi_allocate_pages(), which m

[U-Boot] [PATCH] ARM: tegra: avoid more operations in non-secure world

2018-07-31 Thread Stephen Warren
From: Stephen Warren A secure monitor that runs before U-Boot, and hence causes U-Boot to run in non-secure world, must implement a few operations that U-Boot otherwise implements when running in secure world. Fix U-Boot to skip these operations when running in non-secure world. In particular

[U-Boot] [PATCH] ARM: tegra: align carveout size

2018-07-31 Thread Stephen Warren
From: Stephen Warren Align the size of the carveout region to 2M. This ensures that the size can be accurately represented by an LPAE page table that uses sections. This solves a bug (hang at boot time soon after printing the DRAM size) that only shows up when the following two commits are

Re: [U-Boot] Latest u-boot-spi.git master branch build breaks

2018-07-31 Thread Stephen Warren
On 07/31/2018 10:34 AM, Jagan Teki wrote: On Tue, Jul 31, 2018 at 9:51 PM, Stephen Warren wrote: Jagan, The following Tegra/sandbox builds fail with the latest u-boot-spi.git master branch: sandbox: drivers/mtd/spi/spi_flash_ids.c:113:2: error: expected ‘}’ before ‘{’ token

[U-Boot] Latest u-boot-spi.git master branch build breaks

2018-07-31 Thread Stephen Warren
Jagan, The following Tegra/sandbox builds fail with the latest u-boot-spi.git master branch: sandbox: drivers/mtd/spi/spi_flash_ids.c:113:2: error: expected ‘}’ before ‘{’ token {"s25fl256s_256k", INFO(0x010219, 0x4d00, 256 * 1024, 128, RD_FULL | WR_QPP) }, ^ plutux: medcom-wide: har

[U-Boot] [PATCH] travis: give every job a name

2018-07-30 Thread Stephen Warren
From: Stephen Warren Travis CI now supports giving jobs an explicit name. Do this for all jobs. This allows more direct control over jobs names than the previous automatic or implicit naming based on the environment variables or script text. Signed-off-by: Stephen Warren --- .travis.yml | 231

Re: [U-Boot] [PATCH 1/2] Revert "tegra: Introduce SRAM repair on tegra124"

2018-07-26 Thread Stephen Warren
On 06/22/2018 01:02 PM, Stephen Warren wrote: From: Stephen Warren This reverts commit 701b7b1d2cf657d435d2bd6caf43d0247d37220d. It will be immediately replaced by a different implementation that is more complete and runs are more targetted times. Tom, can we please apply this series, and

Re: [U-Boot] [PATCH] tegra: Indicate that binman makes all three output files

2018-07-26 Thread Stephen Warren
On 07/23/2018 09:27 AM, Stephen Warren wrote: On 07/20/2018 07:49 PM, Simon Glass wrote: Use GNU make pattern rules to indicate that a single run of binman produces all three Tegra output files. The avoids make running binman three times (perhaps in parallel) and those instances inteferring

Re: [U-Boot] [PATCH] tegra: Indicate that binman makes all three output files

2018-07-23 Thread Stephen Warren
/patch/944611/ for the bug report. Tested-by: Stephen Warren (This make feature is very odd; specifying n targets without a % means they're separate rules, but specifying n targets with a % means it's a single rule that generates n targets. Nice inconsistency on m

Re: [U-Boot] [PATCH] binman: ensure temp filenames don't collide

2018-07-19 Thread Stephen Warren
On 07/19/2018 08:17 PM, Simon Glass wrote: > Hi Stephen, > > On 19 July 2018 at 15:14, Stephen Warren wrote: >> On 07/18/2018 07:32 PM, Simon Glass wrote: >>> Hi Stephen, >>> >>> On 16 July 2018 at 16:51, Stephen Warren wrote: >>>> From: St

Re: [U-Boot] [PATCH] binman: ensure temp filenames don't collide

2018-07-19 Thread Stephen Warren
On 07/18/2018 07:32 PM, Simon Glass wrote: > Hi Stephen, > > On 16 July 2018 at 16:51, Stephen Warren wrote: >> From: Stephen Warren >> >> The U-Boot Makefile can invoke binman multiple times in parallel. This >> is problematic because binman uses a static hard

Re: [U-Boot] Please pull u-boot-dm

2018-07-17 Thread Stephen Warren
On 07/16/2018 10:40 PM, Simon Glass wrote: Hi Stephen, On 12 July 2018 at 15:26, Stephen Warren wrote: On 07/12/2018 12:17 PM, Stephen Warren wrote: On 07/12/2018 09:52 AM, Stephen Warren wrote: On 07/11/2018 06:12 PM, Simon Glass wrote: Hi Stephen, On 11 July 2018 at 16:01, Stephen

[U-Boot] [PATCH] binman: ensure temp filenames don't collide

2018-07-16 Thread Stephen Warren
From: Stephen Warren The U-Boot Makefile can invoke binman multiple times in parallel. This is problematic because binman uses a static hard-coded temporary file name. If two instances of binman use that filename at the same time, one writing one reading, they may silently read the wrong content

Re: [U-Boot] Please pull u-boot-dm

2018-07-12 Thread Stephen Warren
On 07/12/2018 12:17 PM, Stephen Warren wrote: On 07/12/2018 09:52 AM, Stephen Warren wrote: On 07/11/2018 06:12 PM, Simon Glass wrote: Hi Stephen, On 11 July 2018 at 16:01, Stephen Warren wrote: On 07/10/2018 02:24 PM, Simon Glass wrote: Hi Stephen, On 10 July 2018 at 13:53, Stephen

Re: [U-Boot] Please pull u-boot-dm

2018-07-12 Thread Stephen Warren
On 07/12/2018 09:52 AM, Stephen Warren wrote: On 07/11/2018 06:12 PM, Simon Glass wrote: Hi Stephen, On 11 July 2018 at 16:01, Stephen Warren wrote: On 07/10/2018 02:24 PM, Simon Glass wrote: Hi Stephen, On 10 July 2018 at 13:53, Stephen Warren wrote: On 07/10/2018 12:47 PM, Tom Rini

Re: [U-Boot] Please pull u-boot-dm

2018-07-12 Thread Stephen Warren
On 07/11/2018 06:12 PM, Simon Glass wrote: Hi Stephen, On 11 July 2018 at 16:01, Stephen Warren wrote: On 07/10/2018 02:24 PM, Simon Glass wrote: Hi Stephen, On 10 July 2018 at 13:53, Stephen Warren wrote: On 07/10/2018 12:47 PM, Tom Rini wrote: On Mon, Jul 09, 2018 at 01:53:43PM

Re: [U-Boot] Please pull u-boot-dm

2018-07-11 Thread Stephen Warren
On 07/10/2018 02:24 PM, Simon Glass wrote: Hi Stephen, On 10 July 2018 at 13:53, Stephen Warren wrote: On 07/10/2018 12:47 PM, Tom Rini wrote: On Mon, Jul 09, 2018 at 01:53:43PM -0600, Simon Glass wrote: Hi Tom. Here are some test-coverage and DM core enhancements. Also it adds a way to

Re: [U-Boot] [PATCH] arm: tegra: Restore host1x/dc dm-pre-reloc properties

2018-07-11 Thread Stephen Warren
On 07/11/2018 09:19 AM, Nicolas Chauvet wrote: Since commit f2faffecbd, tegra: Convert to use binman the dm-pre-reloc properties are removed. That commit hash is ambiguous; can we add a few more characters (e.g. "f2faffecb016")? Thanks for tracking this down.

Re: [U-Boot] Please pull u-boot-dm

2018-07-10 Thread Stephen Warren
On 07/10/2018 12:47 PM, Tom Rini wrote: On Mon, Jul 09, 2018 at 01:53:43PM -0600, Simon Glass wrote: Hi Tom. Here are some test-coverage and DM core enhancements. Also it adds a way to access the binman definition from U-Boot. The following changes since commit 8c5d4fd0ec222701598a27b26ab726

Re: [U-Boot] [PATCH] tegra: p2771: Add CONFIG_EFI_LOADER_BOUNCE_BUFFER

2018-07-09 Thread Stephen Warren
On 07/06/2018 04:12 AM, Peter Robinson wrote: The Jetson TX2 needs EFI loader bounce buffer enabled otherwise grub doesn't see the storage when it loads. I suspect the need for this option is imposed by the SoC HW not the board design? If so, shouldn't the SoC Kconfig symbol select this EFI b

Re: [U-Boot] [PATCH v5 3/5] test/py: remove hacks for non-zero RAM base address in tests

2018-06-29 Thread Stephen Warren
On 06/29/2018 09:25 AM, Stephen Warren wrote: On 06/29/2018 01:41 AM, Quentin Schulz wrote: Some functions have different behaviour when the given address is 0 (assumed to be NULL by the function). find_ram_base() does not return 0 anymore so it's safe to remove those offsets. Patche

Re: [U-Boot] [PATCH v5 3/5] test/py: remove hacks for non-zero RAM base address in tests

2018-06-29 Thread Stephen Warren
On 06/29/2018 01:41 AM, Quentin Schulz wrote: Some functions have different behaviour when the given address is 0 (assumed to be NULL by the function). find_ram_base() does not return 0 anymore so it's safe to remove those offsets. Patches 1-3, Reviewed-by: Stephen Warren I assum

Re: [U-Boot] [U-Boot, v4, 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-28 Thread Stephen Warren
Schulz wrote: Hi all, On Wed, Jun 13, 2018 at 01:02:06PM -0600, Stephen Warren wrote: On 06/13/2018 12:53 PM, Quentin Schulz wrote: Hi Tom, On Wed, Jun 13, 2018 at 11:43:32AM -0400, Tom Rini wrote: On Mon, Jun 04, 2018 at 11:47:30AM +0200, Quentin Schulz wrote: This tests that the importing

Re: [U-Boot] [U-Boot, v4, 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-27 Thread Stephen Warren
left untouched. Two other tests are made to test that size cannot be '-' if the checksum protection is enabled. Signed-off-by: Quentin Schulz Reviewed-by: Simon Glass Reviewed-by: Stephen Warren This seems to not be working? https://travis-ci.org/trini/u-boot/jobs/391504525 I ju

Re: [U-Boot] [U-Boot, v4, 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-26 Thread Stephen Warren
On 06/26/2018 06:41 AM, Quentin Schulz wrote: Hi all, On Wed, Jun 13, 2018 at 01:02:06PM -0600, Stephen Warren wrote: On 06/13/2018 12:53 PM, Quentin Schulz wrote: Hi Tom, On Wed, Jun 13, 2018 at 11:43:32AM -0400, Tom Rini wrote: On Mon, Jun 04, 2018 at 11:47:30AM +0200, Quentin Schulz

Re: [U-Boot] Sharing code between U-Boot and its secure monitor

2018-06-25 Thread Stephen Warren
On 06/24/2018 12:17 AM, Alexander Graf wrote: Am 22.06.2018 um 21:24 schrieb Stephen Warren : I am writing more secure monitor (PSCI) support for Jetson TK1. In particular, this code will support resume from the LP0 system suspend state, which entails signing a block of code for the boot

[U-Boot] Sharing code between U-Boot and its secure monitor

2018-06-22 Thread Stephen Warren
I am writing more secure monitor (PSCI) support for Jetson TK1. In particular, this code will support resume from the LP0 system suspend state, which entails signing a block of code for the boot ROM to validate during resume. The signing code uses lib/aes.c. This code is linked into U-Boot in t

[U-Boot] [PATCH 3/4] ARM: PSCI: Enable the PSCI node

2018-06-22 Thread Stephen Warren
From: Stephen Warren When fixing up the DT to report PSCI support, explicitly enable the node. DTs may ship with the node disabled in case a PSCI implementation is not present, and expect any PSCI implementation to enable the node if they are actually present. Signed-off-by: Stephen Warren

[U-Boot] [PATCH 4/4] ARM: tegra: avoid using secure carveout RAM

2018-06-22 Thread Stephen Warren
From: Stephen Warren If a secure carveout exists, U-Boot cannot use that memory. Fix carveout_size() to reflect this, and hence transitively fix usable_ram_size_below_4g() and board_get_usable_ram_top(). This change ensures that when U-Boot copies the secure monitor code to install it, the copy

[U-Boot] [PATCH 1/4] ARM: define MON_MODE

2018-06-22 Thread Stephen Warren
From: Stephen Warren Add a MON_MODE define for ARM's monitor mode. This can be used later by a secure monitor to avoid hard-coding mode IDs. Signed-off-by: Stephen Warren --- arch/arm/include/asm/proc-armv/ptrace.h | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/include/asm

[U-Boot] [PATCH 2/4] ARM: PSCI: Support PSCI v0.2

2018-06-22 Thread Stephen Warren
From: Stephen Warren Enhance the PSCI DT editing code to allow setting a PSCI v0.2 compatible value in the DT. The CONFIG_ option is added to the whitelist to match the existing PSCI_1_0 option. While not adding new options to Kconfig isn't ideal, I figure it's better to keep relat

[U-Boot] [PATCH 1/2] Revert "tegra: Introduce SRAM repair on tegra124"

2018-06-22 Thread Stephen Warren
From: Stephen Warren This reverts commit 701b7b1d2cf657d435d2bd6caf43d0247d37220d. It will be immediately replaced by a different implementation that is more complete and runs are more targetted times. Signed-off-by: Stephen Warren --- arch/arm/include/asm/arch-tegra124/flow.h | 12

[U-Boot] [PATCH 2/2] ARM: tegra: implement RAM repair

2018-06-22 Thread Stephen Warren
executing code, and RAM repair must complete before the CPU boots. Note that this commit is the result of squashing a numbmer of commits in NVIDIA's downstream L4T branch, hence the multiple signoffs below. Signed-off-by: Bibek Basu Signed-off-by: Sandipan Patra Signed-off-by: Stephen W

Re: [U-Boot] [PATCH v1 3/3] cmd: usb gadget: Add a command to bind a USB gadget driver to a UDC port

2018-06-14 Thread Stephen Warren
On 06/14/2018 09:11 AM, Simon Glass wrote: On 14 June 2018 at 09:02, Jean-Jacques Hiblot wrote: On 13/06/2018 03:29, Simon Glass wrote: Hi Jean-Jacques, On 12 June 2018 at 03:31, Jean-Jacques Hiblot wrote: Hi, On 08/06/2018 23:59, Simon Glass wrote: Hi, On 7 June 2018 at 01:39, Lu

Re: [U-Boot] [U-Boot, v4, 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-13 Thread Stephen Warren
other tests are made to test that size cannot be '-' if the checksum protection is enabled. Signed-off-by: Quentin Schulz Reviewed-by: Simon Glass Reviewed-by: Stephen Warren This seems to not be working? https://travis-ci.org/trini/u-boot/jobs/391504525 I just rebased on top o

Re: [U-Boot] [PATCH v4 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-07 Thread Stephen Warren
exported env (but is passed as parameter), after importing exported env, foo shall be empty, Any variable not passed as parameter should be left untouched. Two other tests are made to test that size cannot be '-' if the checksum protection is enabled. Reviewed-by: Stephen Warren

Re: [U-Boot] [PATCH] dlmalloc: ensure gd is set for early free

2018-05-23 Thread Stephen Warren
On 05/23/2018 01:07 PM, Eugeniu Rosca wrote: This fix seems to be a twin of v2015.01 commit 854d2b9753e4 ("dlmalloc: ensure gd is set for early alloc"). Here is a gdb backtrace to make them look even more similar (sandbox build): ... Interestingly, this issue appears on a very old v2015.04 U-bo

Re: [U-Boot] [PATCH v2 2/2] test/py: add test for whitelist of variables while importing environment

2018-05-18 Thread Stephen Warren
(but is in the whitelisted_vars), after importing exported env, foo shall be empty, Any variable not in whitelisted_vars should be left untouched. Acked-by: Stephen Warren diff --git a/test/py/tests/test_env.py b/test/py/tests/test_env.py +def test_env_import_whitelist(state_test_env

Re: [U-Boot] [PATCH v2 1/2] cmd: nvedit: add whitelist option for env import

2018-05-18 Thread Stephen Warren
On 05/18/2018 08:44 AM, Quentin Schulz wrote: While the `env export` can take as parameters variables to be exported, `env import` does not have such a mechanism of variable selection. Let's add a `-w` option that asks `env import` to look for the `whitelisted_vars` env variable for a space-sepa

Re: [U-Boot] [PATCH 2/2] test: Fix test_vboot to call openssl without redirection

2018-05-16 Thread Stephen Warren
On 05/16/2018 01:10 AM, Simon Glass wrote: The redirection seems to cause this test to fail now. It isn't needed, so drop it. I guess I have no particular objection to this, but I will point out that the test is working just fine as-is right now, so it might be worth investigating more re: wh

Re: [U-Boot] [PATCH 3/6] tegra: psci: save context id in cpu_on command

2018-04-18 Thread Stephen Warren
On 04/18/2018 07:58 AM, Patrick DELAUNAY wrote: Hi Stephen, From: Stephen Warren [mailto:swar...@wwwdotorg.org] Subject: Re: [U-Boot] [PATCH 3/6] tegra: psci: save context id in cpu_on command Importance: High On 04/16/2018 02:15 AM, Patrick Delaunay wrote: Replace the psci_save_target_pc

Re: [U-Boot] [PATCHv1 14/14] reset: remove request and free functions

2018-04-16 Thread Stephen Warren
On 04/16/2018 12:43 PM, Simon Glass wrote: +Stephen for comment Hi Dinh, On 14 April 2018 at 12:51, Dinh Nguyen wrote: The request and free reset functions are not really used for any useful purpose but for debugging. We can safely remove them. The API is set to line up with clocks. I think

Re: [U-Boot] [PATCH v2 1/3] arm: psci: save context id for cpu_on PSCI command

2018-04-16 Thread Stephen Warren
0) => should be removed when all platform migrate to the new API - psci_save(): new API to use by ARMv7 platform with PSCI, save pc (= entry_point_address) and context_id Reviewed-by: Stephen Warren ___ U-Boot mailing list U-Boot@lists.denx

Re: [U-Boot] [PATCH 3/6] tegra: psci: save context id in cpu_on command

2018-04-16 Thread Stephen Warren
to find it in the list archives I have, and assuming the dependency is applied first, Reviewed-by: Stephen Warren diff --git a/arch/arm/mach-tegra/psci.S b/arch/arm/mach-tegra/psci.S @@ -90,7 +90,8 @@ ENTRY(psci_cpu_on) mov r4, r1 mov r0, r1 mov r1

Re: [U-Boot] [PATCH v4 7/7] test/py: add spi_flash tests

2018-03-14 Thread Stephen Warren
On 03/14/2018 05:15 PM, Liam Beguin wrote: Add basic tests for the spi_flash subsystem. Signed-off-by: Liam Beguin Reviewed-by: Stephen Warren It's useful if you put a brief description of what changed between patch versions below the --- line so people know whether they care abou

Re: [U-Boot] [PATCH v3 7/7] test/py: add spi_flash tests

2018-03-14 Thread Stephen Warren
On 03/13/2018 08:03 PM, Liam Beguin wrote: Add basic tests for the spi_flash subsystem. Reviewed-by: Stephen Warren ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 1/2] test/py: add MMC/SD block read test

2018-03-13 Thread Stephen Warren
On 02/20/2018 12:51 PM, Stephen Warren wrote: From: Stephen Warren Add a standalone MMC block read test. This allows direct testing of MMC access rather than relying on doing so as a side-effect of e.g. DFU or UMS testing, which may not be enabled on all platforms. I assume you were only

Re: [U-Boot] [PATCH v2 7/7] test/py: add spi_flash tests

2018-03-13 Thread Stephen Warren
On 03/04/2018 09:22 PM, Liam Beguin wrote: Add basic tests for the spi_flash subsystem. Looks good. A few small issues: +def sf_prepare(u_boot_console, env__sf_config): ... +speed = env__sf_config.get('speed', 0) +if isinstance(speed, list) and len(speed) == 2: +sf_params['s

Re: [U-Boot] [PATCH v2 6/7] test/py: add generic CRC32 function

2018-03-13 Thread Stephen Warren
On 03/04/2018 09:22 PM, Liam Beguin wrote: Add a generic function which can be used to compute the CRC32 value of a region of RAM. Patches 1-6, Reviewed-by: Stephen Warren ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u

Re: [U-Boot] [PATCH 3/3] test/py: add spi_flash tests

2018-03-13 Thread Stephen Warren
On 03/03/2018 09:32 AM, Liam Beguin wrote: On 1 March 2018 at 16:56, Stephen Warren wrote: On 02/26/2018 09:17 PM, Liam Beguin wrote: Add basic tests for the spi_flash subsystem. diff --git a/test/py/tests/test_sf.py b/test/py/tests/test_sf.py +import re +import pytest +import

Re: [U-Boot] [PATCH 3/3] test/py: add spi_flash tests

2018-03-01 Thread Stephen Warren
On 02/26/2018 09:17 PM, Liam Beguin wrote: Add basic tests for the spi_flash subsystem. diff --git a/test/py/tests/test_sf.py b/test/py/tests/test_sf.py +import re +import pytest +import u_boot_utils + + Nit: Double blank line. The same issue exists in many other places too. +""" +Note:

Re: [U-Boot] [PATCH 2/3] test/py: README: add HOSTNAME to PYTHONPATH

2018-03-01 Thread Stephen Warren
On 02/26/2018 09:17 PM, Liam Beguin wrote: As opposed to PATH, HOSTNAME is not appended to PYTHONPATH automatically. Lets add it to the examples to make it more obvious to new users. Reviewed-by: Stephen Warren ___ U-Boot mailing list U-Boot

Re: [U-Boot] [PATCH 1/3] test/py: README: fix typo

2018-03-01 Thread Stephen Warren
On 02/26/2018 09:17 PM, Liam Beguin wrote: fix a minor typo causing vim (and possibly other) to get confused with coloring. Nit: s/fix/Fix/. Reviewed-by: Stephen Warren ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u

Re: [U-Boot] [PATCH 0/3] add inital SF tests

2018-03-01 Thread Stephen Warren
On 02/27/2018 04:52 AM, Michal Simek wrote: On 27.2.2018 05:17, Liam Beguin wrote: Hi all, This is the inital step to adding tests for the SF subsystem plus very minor fixes. It is based on work I found on the mailing list[1]. For now, it doesn't do much but I plan on adding code to reset the f

Re: [U-Boot] [PATCH 3/3] test/py: add spi_flash tests

2018-03-01 Thread Stephen Warren
On 03/01/2018 04:48 AM, Liam Beguin wrote: On 1 March 2018 at 01:59, Michal Simek wrote: ... Also these randomized data/lengths were able to find out more issues than fixed one. I was playing with that randomized test order and I see that not all current tests are self contained and this will

Re: [U-Boot] [PATCH 0/3] add inital SF tests

2018-02-27 Thread Stephen Warren
Liam Beguin wrote at Monday, February 26, 2018 9:18 PM: > Hi all, > > This is the inital step to adding tests for the SF subsystem plus very > minor fixes. It is based on work I found on the mailing list[1]. > For now, it doesn't do much but I plan on adding code to reset the flash > to its initia

Re: [U-Boot] Support of kernels > 16 MiB on Raspberry Pi

2018-02-23 Thread Stephen Warren
On 02/22/2018 12:58 PM, Alexander Kurtz wrote: Hi! I am using U-Boot to boot Debian Buster arm64 (standard kernel with a dracut-based initramfs) via an extlinux.conf on my Raspberry Pi 3. Recently, the Debian kernel grew beyond 16 MiB: 4.13: /boot/vmlinuz-4.13.0-1-arm64 == 16448000 bytes

[U-Boot] [PATCH 2/2] test/py: highlight warnings in the log summary

2018-02-20 Thread Stephen Warren
From: Stephen Warren Currently, if a test emits a warning message but otherwise passes, there's no indication of this in the log summary, which can lead to warnings being missed. Enhance the test logic to explicitly mention warnings in otherwise passing tests, and not to collapse th

[U-Boot] [PATCH 1/2] test/py: add MMC/SD block read test

2018-02-20 Thread Stephen Warren
From: Stephen Warren Add a standalone MMC block read test. This allows direct testing of MMC access rather than relying on doing so as a side-effect of e.g. DFU or UMS testing, which may not be enabled on all platforms. Signed-off-by: Stephen Warren --- test/py/tests/test_mmc_rd.py | 129

Re: [U-Boot] How to test new bootloaders on Jetson TX1?

2018-02-15 Thread Stephen Warren
On 02/14/2018 06:51 PM, Andreas Färber wrote: Hello, I would like to test the latest version of U-Boot on the Jetson TX1. Unfortunately U-Boot is lacking a README that would explain how to do that: > ... Here is some consolidated background on U-Boot on TX1: In all cases, U-Boot uses a built

Re: [U-Boot] [PATCH] fdt: Fixup only valid memory banks

2018-01-30 Thread Stephen Warren
amp; size[i] == 0) + break; + + banks = i; Do we want to move the for loop above the check for if (!banks)? Should we put braces around the multi-line body of the for loop? Aside from that, Acked-by: Stephen Warren + len = fdt_pack_reg(blob, tmp, start,

Re: [U-Boot] [PATCH] mmc: fix to assign to corret clock value when clock is enabling

2018-01-23 Thread Stephen Warren
On 01/22/2018 10:04 PM, Jaehoon Chung wrote: When clock is enabling, it's assigned to 0 as mmc->clock. Then it can't initialize any card. Fix to assign to correct clock value as mmc->cfg->f_min or f_max. Tested-by: Stephen Warren Thanks! Fixes: 9546eb92cb6 ("mmc:

[U-Boot] u-boot-mmc fails on almost every Tegra board

2018-01-22 Thread Stephen Warren
Jaehoon, The latest commit in u-boot-mmc.git master branch fails on almost every Tegra board. The MMC device experiences an error during initialization and hence isn't available: U-Boot 2018.01-05974-gb9b4f146c9 (Jan 22 2018 - 09:18:42 -0700), Build: jenkins-u-boot-denx_uboot_mmc-master-buil

Re: [U-Boot] Pull request, u-boot-tegra/master

2018-01-16 Thread Stephen Warren
On 01/15/2018 06:53 AM, Tom Rini wrote: On Fri, Jan 12, 2018 at 10:37:06AM -0700, Tom Warren wrote: Tom, Please pull u-boot-tegra/master into U-Boot/master. Thanks! All Tegra builds are OK, and Stephen's automated test system reports that all tests pass. The following changes since commit 37

Re: [U-Boot] [PATCH] Makefile: ensure DTB doesn't overflow into initial stack

2018-01-10 Thread Stephen Warren
On 01/10/2018 04:04 PM, Simon Glass wrote: On 9 January 2018 at 11:52, Stephen Warren wrote: With CONFIG_SYS_INIT_SP_BSS_OFFSET enabled, the initial (pre-relocation) stack is placed some distance after bss_start. The control DTB is appended to the U-Boot binary at bss_start. If the DTB is too

Re: [U-Boot] [PATCH] Makefile: ensure DTB doesn't overflow into initial stack

2018-01-10 Thread Stephen Warren
On 01/10/2018 02:56 PM, Tom Warren wrote: Stephen Warren wrote at Tuesday, January 9, 2018 12:52 PM: > With CONFIG_SYS_INIT_SP_BSS_OFFSET enabled, the initial (pre-relocation) > stack is placed some distance after bss_start. The control DTB is > appended to the U-Boot binary at bss_star

Re: [U-Boot] [PATCH 3/4] ARMv8: Allow dynamic early stack pointer

2018-01-09 Thread Stephen Warren
On 01/08/2018 03:11 PM, Simon Glass wrote: Hi Stephen, On 8 January 2018 at 11:34, Stephen Warren wrote: On 01/07/2018 08:40 PM, Simon Glass wrote: Hi Stephen, On 19 December 2017 at 18:30, Stephen Warren wrote: From: Stephen Warren U-Boot typically uses a hard-coded value for the

[U-Boot] [PATCH] Makefile: ensure DTB doesn't overflow into initial stack

2018-01-09 Thread Stephen Warren
From: Stephen Warren With CONFIG_SYS_INIT_SP_BSS_OFFSET enabled, the initial (pre-relocation) stack is placed some distance after bss_start. The control DTB is appended to the U-Boot binary at bss_start. If the DTB is too large, or the SP BSS offset too small, then the initial stack could

[U-Boot] [PATCH 2/2] ARM: Tegra: p2771-0000: use calculcate env var feature

2018-01-08 Thread Stephen Warren
From: Stephen Warren Request that all environment variables containing hard-coded address be calculated at boot time instead. Signed-off-by: Stephen Warren --- include/configs/p2771-.h | 20 1 file changed, 20 insertions(+) diff --git a/include/configs/p2771-.h b

[U-Boot] [PATCH 1/2] ARM: Tegra186: calculate load addresses at boot

2018-01-08 Thread Stephen Warren
From: Stephen Warren In the presence of potentially fragemented memory, we cannot hard-code addresses into environment variables such as kernel_addr_r. Instead, we must calculate those addresses at run-time based on available memory locations. Implement the code to perform such runtime

Re: [U-Boot] [PATCH 3/4] ARMv8: Allow dynamic early stack pointer

2018-01-08 Thread Stephen Warren
On 01/07/2018 08:40 PM, Simon Glass wrote: Hi Stephen, On 19 December 2017 at 18:30, Stephen Warren wrote: From: Stephen Warren U-Boot typically uses a hard-coded value for the stack pointer before relocation. Implement option SYS_INIT_SP_BSS_OFFSET to instead calculate the initial SP at

Re: [U-Boot] [PATCH 2/2] ARM: tegra: use LINUX_KERNEL_IMAGE_HEADER

2018-01-08 Thread Stephen Warren
On 01/07/2018 09:50 PM, Simon Glass wrote: Hi Stephen, On 2 January 2018 at 16:54, Stephen Warren wrote: From: Stephen Warren Enable CONFIG_LINUX_KERNEL_IMAGE_HEADER for all 64-bit Tegra boards. cboot (the boot SW that runs before U-Boot) will eventually use this information. How does U

[U-Boot] [PATCH] ARM: bootm: don't assume sp is in DRAM bank 0

2018-01-05 Thread Stephen Warren
From: Stephen Warren arch_lmb_reserve() currently assumes that the stack pointer is within DRAM bank 0. This is not necessarily true. Enhance the code to search through DRAM banks until the bank that does contain SP is found, and then reserve the tail of that bank. Fixes: 2d1916e48bd8 (&quo

[U-Boot] [PATCH] ARM: Tegra186: don't map memory not in RAM banks

2018-01-04 Thread Stephen Warren
From: Stephen Warren Tegra186 currently restricts its DRAM usage to entries in the /memory node in the DTB passed to it. However, the MMU configuration always maps the entire first 2GB of RAM. This could allow the CPU to speculatively access RAM that isn't part of the in-use banks. This

Re: [U-Boot] [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL

2018-01-03 Thread Stephen Warren
On 01/03/2018 03:43 PM, Tom Warren wrote: Stephen, -Original Message- From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass Sent: Thursday, December 28, 2017 8:14 PM To: Stephen Warren Cc: Tom Warren ; Tom Rini ; U-Boot Mailing List ; Stephen Warren Subject: Re

[U-Boot] [PATCH 3/3] ARM: tegra: p2771-000: increase max DRAM bank count

2018-01-03 Thread Stephen Warren
From: Stephen Warren On this platform, there may be up to 1024 unusable chunks of memory. Increase CONFIG_NR_DRAM_BANKS so that U-Boot can remember all the banks required to represent such fragmented memory. Signed-off-by: Stephen Warren --- include/configs/p2771-.h | 3 +++ 1 file

[U-Boot] [PATCH 2/3] ARM: Tegra186: search for best RAM bank

2018-01-03 Thread Stephen Warren
From: Stephen Warren In the future, the list of DRAM regions passed to U-Boot in the DTB may be quite long and fragmented. Due to this, U-Boot must search through the regions to find the best region to relocate into, rather than relying on the current assumption that the top of bank 0 is a

[U-Boot] [PATCH 1/3] ARM: Tegra186: mem parsing fixes from downstream

2018-01-03 Thread Stephen Warren
From: Stephen Warren Apply a few small fixes for the DTB /memory node parsing from NVIDIA's downstream U-Boot: - Allow arbitrary number of DRAM banks. - Correctly calculate the number of DRAM banks. - Clip PCIe memory in the same way as U-Boot CPU memory use. Signed-off-by: Stephen W

[U-Boot] [PATCH V2 1/2] ARMv8: add optional Linux kernel image header

2018-01-03 Thread Stephen Warren
From: Stephen Warren Allow placing a Linux kernel image header at the start of the U-Boot binary. This is useful since the image header reports the amount of memory (BSS and similar) that U-Boot needs to use, but that isn't part of the binary size. This can be used by the code that loads U

[U-Boot] [PATCH V2 2/2] ARM: tegra: use LINUX_KERNEL_IMAGE_HEADER

2018-01-03 Thread Stephen Warren
From: Stephen Warren Enable CONFIG_LINUX_KERNEL_IMAGE_HEADER for all 64-bit Tegra boards. cboot (the boot SW that runs before U-Boot) will eventually use this information. Signed-off-by: Stephen Warren --- v2: No change. --- arch/arm/mach-tegra/Kconfig | 6 ++ 1 file changed, 6 insertions

Re: [U-Boot] [PATCH 1/2] ARMv8: add optional Linux kernel image header

2018-01-03 Thread Stephen Warren
On 01/03/2018 10:56 AM, Tom Rini wrote: On Wed, Jan 03, 2018 at 10:42:56AM -0700, Stephen Warren wrote: On 01/03/2018 06:45 AM, Tom Rini wrote: On Tue, Jan 02, 2018 at 04:54:21PM -0700, Stephen Warren wrote: From: Stephen Warren Allow placing a Linux kernel image header at the start of the

Re: [U-Boot] [PATCH 1/2] ARMv8: add optional Linux kernel image header

2018-01-03 Thread Stephen Warren
On 01/03/2018 06:45 AM, Tom Rini wrote: On Tue, Jan 02, 2018 at 04:54:21PM -0700, Stephen Warren wrote: From: Stephen Warren Allow placing a Linux kernel image header at the start of the U-Boot binary. This is useful since the image header reports the amount of memory (BSS and similar) that

[U-Boot] [PATCH 1/2] ARMv8: add optional Linux kernel image header

2018-01-02 Thread Stephen Warren
From: Stephen Warren Allow placing a Linux kernel image header at the start of the U-Boot binary. This is useful since the image header reports the amount of memory (BSS and similar) that U-Boot needs to use, but that isn't part of the binary size. This can be used by the code that loads U

[U-Boot] [PATCH 2/2] ARM: tegra: use LINUX_KERNEL_IMAGE_HEADER

2018-01-02 Thread Stephen Warren
From: Stephen Warren Enable CONFIG_LINUX_KERNEL_IMAGE_HEADER for all 64-bit Tegra boards. cboot (the boot SW that runs before U-Boot) will eventually use this information. Signed-off-by: Stephen Warren --- arch/arm/mach-tegra/Kconfig | 6 ++ 1 file changed, 6 insertions(+) diff --git a

Re: [U-Boot] [PATCH v4] drivers: core: Add translation in live tree case

2017-12-20 Thread Stephen Warren
this situation by applying bus translations in the live tree case as well. Signed-off-by: Mario Six --- v3 -> v4: * Fixed issue on 64-bit platforms Tested-by: Stephen Warren ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinf

[U-Boot] [PATCH 4/4] ARM: tegra: use CONFIG_SYS_INIT_SP_BSS_OFFSET

2017-12-19 Thread Stephen Warren
From: Stephen Warren Enable CONFIG_SYS_INIT_SP_BSS_OFFSET for all 64-bit Tegra boards. Place the stack/... 512KiB from the end of the U-Boot binary. This should be plenty to accommodate the current DTBs (max 64 KiB), early malloc region (6KiB), stack usage, and plenty of slack, while still not

[U-Boot] [PATCH 3/4] ARMv8: Allow dynamic early stack pointer

2017-12-19 Thread Stephen Warren
From: Stephen Warren U-Boot typically uses a hard-coded value for the stack pointer before relocation. Implement option SYS_INIT_SP_BSS_OFFSET to instead calculate the initial SP at run-time. This is useful to avoid hard-coding addresses into U-Boot, so that can be loaded and executed at

[U-Boot] [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL

2017-12-19 Thread Stephen Warren
From: Stephen Warren 64-bit Tegra don't use SPL, and soon won't define CONFIG_SPL_TEXT_BASE when building. Fix the binman .dts file so that it doesn't use undefined values. Signed-off-by: Stephen Warren --- arch/arm/dts/tegra-u-boot.dtsi | 15 +-- 1 file change

[U-Boot] [PATCH 2/4] ARM: tegra: remove SPL config for non-SPL SoCs

2017-12-19 Thread Stephen Warren
From: Stephen Warren No 64-bit Tegra uses SPL. Remove various unused definitions from config headers. Signed-off-by: Stephen Warren --- include/configs/tegra-common.h| 2 ++ include/configs/tegra186-common.h | 5 - include/configs/tegra210-common.h | 5 - 3 files changed, 2

Re: [U-Boot] [PATCH v3] drivers: core: Add translation in live tree case

2017-12-18 Thread Stephen Warren
On 12/18/2017 03:33 PM, Stephen Warren wrote: On 12/18/2017 03:29 PM, Stephen Warren wrote: On 12/18/2017 01:34 AM, Mario Six wrote: The function dev_read_addr calls ofnode_get_addr_index in the live tree case, which does not apply bus translations to the address read from the device tree

Re: [U-Boot] [PATCH v3] drivers: core: Add translation in live tree case

2017-12-18 Thread Stephen Warren
On 12/18/2017 03:29 PM, Stephen Warren wrote: On 12/18/2017 01:34 AM, Mario Six wrote: The function dev_read_addr calls ofnode_get_addr_index in the live tree case, which does not apply bus translations to the address read from the device tree. This results in illegal addresses on boards that

Re: [U-Boot] [PATCH v3] drivers: core: Add translation in live tree case

2017-12-18 Thread Stephen Warren
this situation by applying bus translations in the live tree case as well. Tested-by: Stephen Warren ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

<    1   2   3   4   5   6   7   8   9   10   >