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

e `sf protect` subcommand (which is the main goal of this series). I'm sending it now to make sure it's headed in the right direction. Thanks, Liam Beguin [ 1 ] https://patchwork.ozlabs.org/patch/623061/ Liam Beguin (3): test/py: README: fix typo test/py: README: add HOSTNAME to PYTHONPATH

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

Add basic tests for the spi_flash subsystem. Signed-off-by: Liam Beguin --- test/py/tests/test_sf.py | 233 +++ 1 file changed, 233 insertions(+) diff --git a/test/py/tests/test_sf.py b/test/py/tests/test_sf.py new file mode 100644 index

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

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. Signed-off-by: Liam Beguin --- test/py/README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/py/README.md b/test/py

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

fix a minor typo causing vim (and possibly other) to get confused with coloring. Signed-off-by: Liam Beguin --- test/py/README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/py/README.md b/test/py/README.md index eefac377567a..000afce93c4a 100644 --- a/test/py

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

Hi, On Tue, 27 Feb 2018 at 11:29 Stephen Warren wrote: > > 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 ma

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

Hi Michal, On 27 February 2018 at 03:51, Michal Simek wrote: > Hi Liam, > > On 27.2.2018 05:17, Liam Beguin wrote: >> Add basic tests for the spi_flash subsystem. > > Good to see this patch. > FYI: We have created qspi tests too but didn't push it out because of &g

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

On 1 March 2018 at 01:59, Michal Simek wrote: > On 1.3.2018 05:01, Liam Beguin wrote: >> Hi Michal, >> >> On 27 February 2018 at 03:51, Michal Simek wrote: >>> Hi Liam, >>> >>> On 27.2.2018 05:17, Liam Beguin wrote: >>>> Add basic tests

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

On 1 March 2018 at 16:23, Stephen Warren wrote: > 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/. Will fix &

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

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 pytes

[U-Boot] [PATCH v2 1/7] spi: spi_flash: do not fail silently on bad user input

Make sure the user is notified instead of silently returning an error. Signed-off-by: Liam Beguin --- drivers/mtd/spi/spi_flash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c index 294d9f9d79c6..2e61685d3ea4 100644

[U-Boot] [PATCH v2 0/8] add inital SF tests

stently instead of " - use sf_read() in test_sf_read() - rename crc variables - add speed parameter with optional random range - allow `sf read` to write at 0x00 Thanks, Liam Beguin [ 1 ] https://patchwork.ozlabs.org/patch/623061/ [ 2 ] https://lists.denx.de/pipermail/u-boot/2018-March/321688

[U-Boot] [PATCH v2 5/7] test/py: do not import pytest multiple times

Signed-off-by: Liam Beguin --- test/py/u_boot_utils.py | 1 - 1 file changed, 1 deletion(-) diff --git a/test/py/u_boot_utils.py b/test/py/u_boot_utils.py index 9acb92ddc448..64584494e463 100644 --- a/test/py/u_boot_utils.py +++ b/test/py/u_boot_utils.py @@ -11,7 +11,6 @@ import os.path import

[U-Boot] [PATCH v2 2/7] cmd: sf: fix map_physmem check

Make sure 0x00 is a valid address to read to. If `addr` is 0x00 then map_physmem() will return 0 which should be a valid address. Signed-off-by: Liam Beguin --- cmd/sf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/sf.c b/cmd/sf.c index f971eec781cc..e7ff9a646208

[U-Boot] [PATCH v2 3/7] test/py: README: fix typo

Fix a minor typo causing vim (and possibly other) to get confused with coloring. Signed-off-by: Liam Beguin --- test/py/README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/py/README.md b/test/py/README.md index eefac377567a..000afce93c4a 100644 --- a/test/py

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

Add a generic function which can be used to compute the CRC32 value of a region of RAM. Signed-off-by: Liam Beguin --- test/py/u_boot_utils.py | 23 +++ 1 file changed, 23 insertions(+) diff --git a/test/py/u_boot_utils.py b/test/py/u_boot_utils.py index 64584494e463

[U-Boot] [PATCH v2 4/7] test/py: README: add HOSTNAME to PYTHONPATH

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. Signed-off-by: Liam Beguin --- test/py/README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/py/README.md b/test/py

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

Add basic tests for the spi_flash subsystem. Signed-off-by: Liam Beguin --- test/py/tests/test_sf.py | 223 +++ 1 file changed, 223 insertions(+) create mode 100644 test/py/tests/test_sf.py diff --git a/test/py/tests/test_sf.py b/test/py/tests

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

Hi everyone, Any new comments on this iteration? Thanks, Liam On Sun, 4 Mar 2018 at 23:22 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

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

Hi Stephen, On Tue, 13 Mar 2018 at 17:22 Stephen Warren wrote: > 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_fla

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

Hi Stephen, On 13 March 2018 at 17:41, Stephen Warren wrote: > 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): > > ..

[U-Boot] [PATCH v3 4/7] test/py: README: add HOSTNAME to PYTHONPATH

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. Signed-off-by: Liam Beguin --- test/py/README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/py/README.md b/test/py

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

Add a generic function which can be used to compute the CRC32 value of a region of RAM. Signed-off-by: Liam Beguin --- test/py/u_boot_utils.py | 23 +++ 1 file changed, 23 insertions(+) diff --git a/test/py/u_boot_utils.py b/test/py/u_boot_utils.py index 64584494e463

[U-Boot] [PATCH v3 3/7] test/py: README: fix typo

Fix a minor typo causing vim (and possibly other) to get confused with coloring. Signed-off-by: Liam Beguin --- test/py/README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/py/README.md b/test/py/README.md index eefac377567a..000afce93c4a 100644 --- a/test/py

[U-Boot] [PATCH v3 5/7] test/py: do not import pytest multiple times

Signed-off-by: Liam Beguin --- test/py/u_boot_utils.py | 1 - 1 file changed, 1 deletion(-) diff --git a/test/py/u_boot_utils.py b/test/py/u_boot_utils.py index 9acb92ddc448..64584494e463 100644 --- a/test/py/u_boot_utils.py +++ b/test/py/u_boot_utils.py @@ -11,7 +11,6 @@ import os.path import

[U-Boot] [PATCH v3 2/7] cmd: sf: fix map_physmem check

Make sure 0x00 is a valid address to read to. If `addr` is 0x00 then map_physmem() will return 0 which should be a valid address. Signed-off-by: Liam Beguin --- cmd/sf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/sf.c b/cmd/sf.c index f971eec781cc..e7ff9a646208

[U-Boot] [PATCH v3 1/7] spi: spi_flash: do not fail silently on bad user input

Make sure the user is notified instead of silently returning an error. Signed-off-by: Liam Beguin --- drivers/mtd/spi/spi_flash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c index 294d9f9d79c6..2e61685d3ea4 100644

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

Add basic tests for the spi_flash subsystem. Signed-off-by: Liam Beguin --- test/py/tests/test_sf.py | 217 +++ 1 file changed, 217 insertions(+) create mode 100644 test/py/tests/test_sf.py diff --git a/test/py/tests/test_sf.py b/test/py/tests

[U-Boot] [PATCH v3 0/7] add inital SF tests

eturn a dict of parameter - use assert instead of pytest.fail - remove verbose from sf_prepare() - update documentation - improve readability - use ' consistently instead of " - use sf_read() in test_sf_read() - rename crc variables - add speed parameter with optional random range

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

Hi Michal, On Wed, 14 Mar 2018 at 10:35 Michal Simek wrote: > On 14.3.2018 03:03, Liam Beguin wrote: > > Add basic tests for the spi_flash subsystem. > > > > Signed-off-by: Liam Beguin > > --- > > test/py/tests/test_sf.py | 217 > +++

[U-Boot] [PATCH v4 1/7] spi: spi_flash: do not fail silently on bad user input

Make sure the user is notified instead of silently returning an error. Signed-off-by: Liam Beguin Reviewed-by: Stephen Warren --- drivers/mtd/spi/spi_flash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c index

[U-Boot] [PATCH v4 0/7] add inital SF tests

" - use sf_read() in test_sf_read() - rename crc variables - add speed parameter with optional random range - allow `sf read` to write at 0x00 Thanks, Liam Beguin [ 1 ] https://patchwork.ozlabs.org/patch/623061/ [ 2 ] https://lists.denx.de/pipermail/u-boot/2018-March/321688.html Liam Beg

[U-Boot] [PATCH v4 2/7] cmd: sf: fix map_physmem check

Make sure 0x00 is a valid address to read to. If `addr` is 0x00 then map_physmem() will return 0 which should be a valid address. Signed-off-by: Liam Beguin Reviewed-by: Stephen Warren --- cmd/sf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/sf.c b/cmd/sf.c index

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

Add a generic function which can be used to compute the CRC32 value of a region of RAM. Signed-off-by: Liam Beguin Reviewed-by: Stephen Warren --- test/py/u_boot_utils.py | 23 +++ 1 file changed, 23 insertions(+) diff --git a/test/py/u_boot_utils.py b/test/py

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

Add basic tests for the spi_flash subsystem. Signed-off-by: Liam Beguin Reviewed-by: Stephen Warren --- test/py/tests/test_sf.py | 218 +++ 1 file changed, 218 insertions(+) create mode 100644 test/py/tests/test_sf.py diff --git a/test/py/tests

[U-Boot] [PATCH v4 5/7] test/py: do not import pytest multiple times

Signed-off-by: Liam Beguin Reviewed-by: Stephen Warren --- test/py/u_boot_utils.py | 1 - 1 file changed, 1 deletion(-) diff --git a/test/py/u_boot_utils.py b/test/py/u_boot_utils.py index 9acb92ddc448..64584494e463 100644 --- a/test/py/u_boot_utils.py +++ b/test/py/u_boot_utils.py @@ -11,7

[U-Boot] [PATCH v4 3/7] test/py: README: fix typo

Fix a minor typo causing vim (and possibly other) to get confused with coloring. Signed-off-by: Liam Beguin Reviewed-by: Stephen Warren --- test/py/README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/py/README.md b/test/py/README.md index eefac377567a

[U-Boot] [PATCH v4 4/7] test/py: README: add HOSTNAME to PYTHONPATH

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. Signed-off-by: Liam Beguin Reviewed-by: Stephen Warren --- test/py/README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test

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

Hi, On Wed, 14 Mar 2018 at 19:27 Stephen Warren wrote: > 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 des

[U-Boot] UBI/UBIFS complete integrity check

a _single bit_ of flash may have been corrupted. My question is, does UBI/UBIFS have this kind of complete integrity check built-in? If not, can I take advantage of these CRC-32, to do something equivalent to my md5sum check from U-Boot. Thanks, Liam Beguin Xiphos Systems Corp. ht

Re: [U-Boot] UBI/UBIFS complete integrity check

OR Memory content to RAM, calculate crc32/sha1 and compare with some stored value (e.g. in u-boot envs). This all should be done with u-boot prompt. This was my backup plan. I should have enough RAM to do it. Thanks, Liam Beguin Xiphos Systems Corp. http://xiphos.com ___

Re: [U-Boot] UBI/UBIFS complete integrity check

Hi, Thanks for taking the time to answer. On 11/05/2017 03:37 AM, Ladislav Michl wrote: On Tue, Oct 31, 2017 at 11:01:21AM -0400, Liam Beguin wrote: Hi everyone, I'm currently using a UBIFS root file system (stored on SPI-NOR flash) and would like to perform a full integrity check b

Re: [U-Boot] UBI/UBIFS complete integrity check

On 11/06/2017 12:57 PM, Ladislav Michl wrote: On Mon, Nov 06, 2017 at 12:31:32PM -0500, Liam Beguin wrote: Hi, Thanks for taking the time to answer. On 11/05/2017 03:37 AM, Ladislav Michl wrote: On Tue, Oct 31, 2017 at 11:01:21AM -0400, Liam Beguin wrote: Hi everyone, I'm currently

Re: [U-Boot] [PATCH v2 1/2] spi: zynqmp_qspi: Add support for ZynqMP qspi driver

Hi Siva, I've been working on a patch to add support for io-mode [1]. It's based on xilinx/master which is closer to v1 (no include file). I haven't had much time to test it further but I can load a bitstream from a ubifs partition with it. Let me know if you have comments. On Tue, 10 Apr 2018 at

Re: [U-Boot] [PATCH v2 1/2] spi: zynqmp_qspi: Add support for ZynqMP qspi driver

tree from NGC guy John Moon. I will be sending that patch > soon may be with v3 which I am planning to send once I got response from > Jagan on my v2 queries. > > > > Regards, > > Siva > > > > *From:* Liam Beguin [mailto:liambeg...@gmail.com] > *Sent:* Tues

[U-Boot] ZynqMP qspi

Hi, I'm testing a new Xilinx zynqmp dev board and was not able to probe the qspi with the latest mainline U-Boot. I see that there is a 'zynqmp_qspi' driver in the Xilinx tree [1] but nothing in mainline. After a little digging, I found a thread on the list [2] (and [3]) and was wondering in what

Re: [U-Boot] ZynqMP qspi

Resending with proper CC since the email came back. On 24/09/17 09:36 PM, Liam Beguin wrote: > Hi, > > I'm testing a new Xilinx zynqmp dev board and was not able to probe the > qspi with the latest mainline U-Boot. I see that there is a 'zynqmp_qspi' > driver in t

Re: [U-Boot] ZynqMP qspi

Hi, On 26/09/17 01:28 AM, Siva Durga Prasad Paladugu wrote: > Hi, > >> -Original Message- >> From: Jagan Teki [mailto:jagannadh.t...@gmail.com] >> Sent: Tuesday, September 26, 2017 10:17 AM >> To: Liam Beguin >> Cc: u-boot@lists.denx.de; Michal S

Re: [U-Boot] ZynqMP qspi

Hi Siva, On 26/09/17 01:28 AM, Siva Durga Prasad Paladugu wrote: > Hi, > ... > > I am planning to send the new series in 1-2 weeks, probably you can wait till > then or > you can try with the patch[2] and see if you are able to get it working with > those( those > are known to be working when

Re: [U-Boot] ZynqMP qspi

Hi Siva, On 23/10/17 01:59 AM, Siva Durga Prasad Paladugu wrote: > Hi Liam, > >> -Original Message----- >> From: Liam Beguin [mailto:liambeg...@gmail.com] >> Sent: Saturday, October 21, 2017 11:33 PM >> To: Siva Durga Prasad Paladugu ; Jagan Teki >> >&

Re: [U-Boot] [U-Boot, v2, 1/2] spi: zynqmp_qspi: Add support for ZynqMP qspi drivers

.w 100 10 device 0 offset 0x100, size 0x10 SF: 16 bytes @ 0x100 Read: OK 0100: ffff ffff ffff 0110: I hope this helps! Thanks, Liam Beguin Xiphos Systems Corp. http://xiphos.ca ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

[PATCH 1/1] arm64: zynqmp: allow overriding board name

There is no need to use zynqmp name as SYS_BOARD for all boards. The patch is adding an option to change it. Signed-off-by: Liam Beguin --- arch/arm/mach-zynqmp/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-zynqmp/Kconfig b/arch/arm/mach-zynqmp/Kconfig index