u-boot.git/test/cmd/mbr.c, branch master Unnamed repository; edit this file 'description' to name the repository. test: Drop suites.h 2025-02-12T02:12:36+00:00 Simon Glass sjg@chromium.org 2025-02-07T18:30:56+00:00 32aba887e3d7cbcdf8fd98c107aaa79f3bd44a16 This file is empty now. Remove it and its uses. Signed-off-by: Simon Glass <sjg@chromium.org> 
This file is empty now. Remove it and its uses.

Signed-off-by: Simon Glass <sjg@chromium.org>
test: Drop the function for running mbr tests 2025-01-24T20:34:40+00:00 Simon Glass sjg@chromium.org 2025-01-20T21:25:44+00:00 1ade3afa2da2242ad1006dc393a3bb3242226d4c Use the new suite-runner to run these tests instead. Signed-off-by: Simon Glass <sjg@chromium.org> 
Use the new suite-runner to run these tests instead.

Signed-off-by: Simon Glass <sjg@chromium.org>
test: Drop the _test suffix on linker lists 2025-01-24T20:34:40+00:00 Simon Glass sjg@chromium.org 2025-01-20T21:25:32+00:00 4ba3ab4901c564725c37e927e2ba29595712cd9d Most test suites have a _test suffix. This is not necessary as there is also a ut_ prefix. Drop the suffix so that (with future work) the suite name can be used as the linker-list name. Remove the suffix from the pytest regex as well, moving it to the top of the file, as it is a constant. Signed-off-by: Simon Glass <sjg@chromium.org> 
Most test suites have a _test suffix. This is not necessary as there is
also a ut_ prefix.

Drop the suffix so that (with future work) the suite name can be used as
the linker-list name.

Remove the suffix from the pytest regex as well, moving it to the top of
the file, as it is a constant.

Signed-off-by: Simon Glass <sjg@chromium.org>
test: use %zd for size_t in mbr_test_run() 2024-11-15T00:14:06+00:00 Heinrich Schuchardt heinrich.schuchardt@canonical.com 2024-11-03T22:48:11+00:00 5430a5f67e5f73b0411d550b5dd6aa446a3b0299 For printing size_t we must use %zd and not %ld to avoid a -Wformat error on 32-bit systems. Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> 
For printing size_t we must use %zd and not %ld to avoid
a -Wformat error on 32-bit systems.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
test: cmd/mbr: pass correct buffer size to init_write_buffers 2024-11-15T00:14:06+00:00 Heinrich Schuchardt heinrich.schuchardt@canonical.com 2024-11-02T15:41:30+00:00 d8c5bcbd224fddd983f0796886409ede880d11ac We want to completely initialize the mbr and embr buffers. This requires passing the buffer size and not the size of a pointer to the buffer. Addresses-Coverity-ID: 510454 Wrong sizeof argument Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> 
We want to completely initialize the mbr and embr buffers. This requires
passing the buffer size and not the size of a pointer to the buffer.

Addresses-Coverity-ID: 510454 Wrong sizeof argument
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
test: mbr: Drop a duplicate test 2024-09-18T19:01:00+00:00 Simon Glass sjg@chromium.org 2024-09-01T22:26:34+00:00 017b441b2e3c879b20bcac496369f1213c4bdbcd The test currently runs twice as it is declared twice. Unwind this. Signed-off-by: Simon Glass <sjg@chromium.org> 
The test currently runs twice as it is declared twice. Unwind this.

Signed-off-by: Simon Glass <sjg@chromium.org>
test: mbr: Use RAM for the buffers 2024-09-18T19:01:00+00:00 Simon Glass sjg@chromium.org 2024-09-01T22:26:33+00:00 6cfc777b9695003c4303a9e61194fe2c64af40e3 The normal approach with sandbox is to use a fixed memory address in the RAM, to avoid needing to create a map for transient local variables. Update this test to use this approach. Signed-off-by: Simon Glass <sjg@chromium.org> 
The normal approach with sandbox is to use a fixed memory address in the
RAM, to avoid needing to create a map for transient local variables.

Update this test to use this approach.

Signed-off-by: Simon Glass <sjg@chromium.org>
test: mbr: Use a constant for the block size 2024-09-18T19:01:00+00:00 Simon Glass sjg@chromium.org 2024-09-01T22:26:32+00:00 e9d899591c7b1cf9661a22a66e0f1c41f37eaeb9 It isn't that important to factor out constants in tests, but in this case we have 0x200 and 512 used. The commands don't use the constant as they use a block count ('1'). It doesn't create more code to use a constant, so create one. Signed-off-by: Simon Glass <sjg@chromium.org> 
It isn't that important to factor out constants in tests, but in this
case we have 0x200 and 512 used. The commands don't use the constant
as they use a block count ('1'). It doesn't create more code to use a
constant, so create one.

Signed-off-by: Simon Glass <sjg@chromium.org>
test: mbr: Unmap the buffers after use 2024-09-18T19:01:00+00:00 Simon Glass sjg@chromium.org 2024-09-01T22:26:31+00:00 705cc13ce7a81111b93a11cfff1763e11586f33e This tests maps some local variables into sandbox's address space. Make sure to unmap them afterwards. Note that the normal approach with sandbox is to use a fixed memory address in the RAM, to avoid needing to create a map for transient local variables. Signed-off-by: Simon Glass <sjg@chromium.org> Fixes: 04291ee0aba ("cmd: mbr: Allow 4 MBR partitions without need...") 
This tests maps some local variables into sandbox's address space. Make
sure to unmap them afterwards.

Note that the normal approach with sandbox is to use a fixed memory
address in the RAM, to avoid needing to create a map for transient local
variables.

Signed-off-by: Simon Glass <sjg@chromium.org>
Fixes: 04291ee0aba ("cmd: mbr: Allow 4 MBR partitions without need...")
test: mbr: Adjust test to drop 0x 2024-09-18T19:01:00+00:00 Simon Glass sjg@chromium.org 2024-09-01T22:26:22+00:00 e7474ac6895a340d81d6ee2706e1dd227331adf7 U-Boot commands typically don't need 0x to specify hex, since they use hex by default. Adding 0x in this test is confusing since it suggests that it is necessary. Drop it from the file. Also use the %#x construct to get the 0x when needed. Signed-off-by: Simon Glass <sjg@chromium.org> 
U-Boot commands typically don't need 0x to specify hex, since they use
hex by default. Adding 0x in this test is confusing since it suggests
that it is necessary. Drop it from the file.

Also use the %#x construct to get the 0x when needed.

Signed-off-by: Simon Glass <sjg@chromium.org>