aboutsummaryrefslogtreecommitdiffstats
path: root/tools/testing/selftests/arm64
diff options
context:
space:
mode:
authorMark Brown <broonie@kernel.org>2021-09-29 16:19:23 +0100
committerWill Deacon <will@kernel.org>2021-09-29 16:33:04 +0100
commit4caf339c037c5e70d0ea789af093e123c2c6fc80 (patch)
treea854fe4e73a2b081bf4e5168d26baf7dae06170e /tools/testing/selftests/arm64
parentselftests: arm64: Fix printf() format mismatch in vec-syscfg (diff)
downloadlinux-dev-4caf339c037c5e70d0ea789af093e123c2c6fc80.tar.xz
linux-dev-4caf339c037c5e70d0ea789af093e123c2c6fc80.zip
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')
-rw-r--r--tools/testing/selftests/arm64/fp/vec-syscfg.c6
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;
}