selftests: arm64: Remove bogus error check on writing to files
authorMark Brown <broonie@kernel.org>
Wed, 29 Sep 2021 15:19:23 +0000 (16:19 +0100)
committerWill Deacon <will@kernel.org>
Wed, 29 Sep 2021 15:33:04 +0000 (16:33 +0100)
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>
tools/testing/selftests/arm64/fp/vec-syscfg.c

index b2de002..d48d3ee 100644 (file)
@@ -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;
 }