| Message ID | 20220124171748.2195875-1-broonie@kernel.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 3673d4b9cf68164678c6bb8a380bfb9eebb49432 |
| Headers | show |
| Series | kselftest/arm64: Remove local ARRAY_SIZE() definitions | expand |
On 1/24/22 10:17 AM, Mark Brown wrote: > An ARRAY_SIZE() has been added to kselftest.h so remove the local versions > in some of the arm64 selftests. > Thank you for the patch removing duplicate defines. > Signed-off-by: Mark Brown <broonie@kernel.org> > --- > tools/testing/selftests/arm64/abi/syscall-abi.c | 1 - > tools/testing/selftests/arm64/fp/sve-ptrace.c | 2 -- > 2 files changed, 3 deletions(-) > > diff --git a/tools/testing/selftests/arm64/abi/syscall-abi.c b/tools/testing/selftests/arm64/abi/syscall-abi.c > index d8eeeafb50dc..1e13b7523918 100644 > --- a/tools/testing/selftests/arm64/abi/syscall-abi.c > +++ b/tools/testing/selftests/arm64/abi/syscall-abi.c > @@ -18,7 +18,6 @@ > > #include "../../kselftest.h" > > -#define ARRAY_SIZE(a) (sizeof(a) / sizeof(a[0])) > #define NUM_VL ((SVE_VQ_MAX - SVE_VQ_MIN) + 1) > > extern void do_syscall(int sve_vl); > diff --git a/tools/testing/selftests/arm64/fp/sve-ptrace.c b/tools/testing/selftests/arm64/fp/sve-ptrace.c > index af798b9d232c..90ba1d6a6781 100644 > --- a/tools/testing/selftests/arm64/fp/sve-ptrace.c > +++ b/tools/testing/selftests/arm64/fp/sve-ptrace.c > @@ -21,8 +21,6 @@ > > #include "../../kselftest.h" > > -#define ARRAY_SIZE(a) (sizeof(a) / sizeof(a[0])) > - > /* <linux/elf.h> and <sys/auxv.h> don't like each other, so: */ > #ifndef NT_ARM_SVE > #define NT_ARM_SVE 0x405 > Assuming this is going through ARM tree? Reviewed-by: Shuah Khan <skhan@linuxfoundation.org> thanks, -- Shuah
On Mon, Jan 24, 2022 at 05:17:48PM +0000, Mark Brown wrote: > An ARRAY_SIZE() has been added to kselftest.h so remove the local versions > in some of the arm64 selftests. > > Signed-off-by: Mark Brown <broonie@kernel.org> Acked-by: Catalin Marinas <catalin.marinas@arm.com>
On Mon, 24 Jan 2022 17:17:48 +0000, Mark Brown wrote: > An ARRAY_SIZE() has been added to kselftest.h so remove the local versions > in some of the arm64 selftests. > > Applied to arm64 (for-next/kselftest), thanks! [1/1] kselftest/arm64: Remove local ARRAY_SIZE() definitions https://git.kernel.org/arm64/c/3673d4b9cf68 Cheers,
diff --git a/tools/testing/selftests/arm64/abi/syscall-abi.c b/tools/testing/selftests/arm64/abi/syscall-abi.c index d8eeeafb50dc..1e13b7523918 100644 --- a/tools/testing/selftests/arm64/abi/syscall-abi.c +++ b/tools/testing/selftests/arm64/abi/syscall-abi.c @@ -18,7 +18,6 @@ #include "../../kselftest.h" -#define ARRAY_SIZE(a) (sizeof(a) / sizeof(a[0])) #define NUM_VL ((SVE_VQ_MAX - SVE_VQ_MIN) + 1) extern void do_syscall(int sve_vl); diff --git a/tools/testing/selftests/arm64/fp/sve-ptrace.c b/tools/testing/selftests/arm64/fp/sve-ptrace.c index af798b9d232c..90ba1d6a6781 100644 --- a/tools/testing/selftests/arm64/fp/sve-ptrace.c +++ b/tools/testing/selftests/arm64/fp/sve-ptrace.c @@ -21,8 +21,6 @@ #include "../../kselftest.h" -#define ARRAY_SIZE(a) (sizeof(a) / sizeof(a[0])) - /* <linux/elf.h> and <sys/auxv.h> don't like each other, so: */ #ifndef NT_ARM_SVE #define NT_ARM_SVE 0x405
An ARRAY_SIZE() has been added to kselftest.h so remove the local versions in some of the arm64 selftests. Signed-off-by: Mark Brown <broonie@kernel.org> --- tools/testing/selftests/arm64/abi/syscall-abi.c | 1 - tools/testing/selftests/arm64/fp/sve-ptrace.c | 2 -- 2 files changed, 3 deletions(-)