diff options
| author | Jakub Czapiga <jacz@semihalf.com> | 2022-01-10 13:36:47 +0000 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-01-14 14:29:29 +0000 |
| commit | c08b6a7037e4dd21f08a2fc8d40db51c7b4e051b (patch) | |
| tree | bcab84b0853151766b0fe4a3c324e962fb05bb4c /tests/acpi | |
| parent | 63ec2ac97aab17a7e5c8e929c96a188bcd59b9c0 (diff) | |
tests: Fix tests code and comments style
This patch applies clang-format settings to most of tests files. Some
files were fixed "by-hand" to exclude some lines, which whould be less
readable after automatic style fixing.
Moreover, some comments (mostly in tests/lib/edid-test.c) were adjusted
to match coreboot coding style guidelines.
Change-Id: I69f25a7b6d8265800c731754e2fbb2255f482134
Signed-off-by: Jakub Czapiga <jacz@semihalf.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/60970
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
Diffstat (limited to 'tests/acpi')
| -rw-r--r-- | tests/acpi/acpigen-test.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/tests/acpi/acpigen-test.c b/tests/acpi/acpigen-test.c index 6bd1d02689..156b544095 100644 --- a/tests/acpi/acpigen-test.c +++ b/tests/acpi/acpigen-test.c @@ -13,7 +13,7 @@ static u32 decode_package_length(const char *ptr) { const u8 *aml = (u8 *)ptr; const u32 offset = (aml[0] == EXT_OP_PREFIX ? 2 : 1); - u32 byte_zero_mask = 0x3F; /* Bits [0:5] */ + u32 byte_zero_mask = 0x3F; /* Bits [0:5] */ u32 byte_count = aml[offset] >> 6; u32 package_length = 0; @@ -106,7 +106,7 @@ static void test_acpigen_nested_ifs(void **state) for (int i = 0; i < nesting_level; ++i) assert_int_equal(decode_package_length(block_start[i]), - block_end[i] - block_start[i] - 1); + block_end[i] - block_start[i] - 1); } static void test_acpigen_write_package(void **state) @@ -203,14 +203,14 @@ static void test_acpigen_scope_with_contents(void **state) int main(void) { const struct CMUnitTest tests[] = { - cmocka_unit_test_setup_teardown(test_acpigen_single_if, - setup_acpigen, teardown_acpigen), - cmocka_unit_test_setup_teardown(test_acpigen_nested_ifs, - setup_acpigen, teardown_acpigen), - cmocka_unit_test_setup_teardown(test_acpigen_write_package, - setup_acpigen, teardown_acpigen), - cmocka_unit_test_setup_teardown(test_acpigen_scope_with_contents, - setup_acpigen, teardown_acpigen), + cmocka_unit_test_setup_teardown(test_acpigen_single_if, setup_acpigen, + teardown_acpigen), + cmocka_unit_test_setup_teardown(test_acpigen_nested_ifs, setup_acpigen, + teardown_acpigen), + cmocka_unit_test_setup_teardown(test_acpigen_write_package, setup_acpigen, + teardown_acpigen), + cmocka_unit_test_setup_teardown(test_acpigen_scope_with_contents, setup_acpigen, + teardown_acpigen), }; return cb_run_group_tests(tests, NULL, NULL); |