diff options
| author | Mark Brown <broonie@kernel.org> | 2021-09-29 16:19:23 +0100 |
|---|---|---|
| committer | Will Deacon <will@kernel.org> | 2021-09-29 16:33:04 +0100 |
| commit | 4caf339c037c5e70d0ea789af093e123c2c6fc80 (patch) | |
| tree | a854fe4e73a2b081bf4e5168d26baf7dae06170e /tools/testing/selftests/arm64/fp | |
| parent | ff944c44b782d34dc95ea9dcf3a5b62a0bdbac18 (diff) | |
selftests: arm64: Remove bogus error check on writing to files
Due to some refactoring with the error handling we ended up mangling things
so we never actually set ret and therefore shouldn't be checking it.
Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20210929151925.9601-3-broonie@kernel.org
Signed-off-by: Will Deacon <will@kernel.org>
Diffstat (limited to 'tools/testing/selftests/arm64/fp')
| -rw-r--r-- | tools/testing/selftests/arm64/fp/vec-syscfg.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/tools/testing/selftests/arm64/fp/vec-syscfg.c b/tools/testing/selftests/arm64/fp/vec-syscfg.c index b2de002ee325..d48d3ee1bc36 100644 --- a/tools/testing/selftests/arm64/fp/vec-syscfg.c +++ b/tools/testing/selftests/arm64/fp/vec-syscfg.c @@ -180,7 +180,6 @@ static int file_read_integer(const char *name, int *val) static int file_write_integer(const char *name, int val) { FILE *f; - int ret; f = fopen(name, "w"); if (!f) { @@ -192,11 +191,6 @@ static int file_write_integer(const char *name, int val) fprintf(f, "%d", val); fclose(f); - if (ret < 0) { - ksft_test_result_fail("Error writing %d to %s\n", - val, name); - return -1; - } return 0; } |