Commit 572cc345 authored Jun 29, 2022 by Stefan Agner Committed by Greg Kroah-Hartman Jul 02, 2022

crypto: arm/ghash-ce - define fpu before fpu registers are referenced



commit 7548bf8c17d84607c106bd45d81834afd95a2edb upstream

Building ARMv7 with Clang's integrated assembler leads to errors such
as:
arch/arm/crypto/ghash-ce-core.S:34:11: error: register name expected
 t3l .req d16
          ^

Since no FPU has selected yet Clang considers d16 not a valid register.
Moving the FPU directive on-top allows Clang to parse the registers and
allows to successfully build this file with Clang's integrated assembler.

Signed-off-by: Stefan Agner <stefan@agner.ch>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Tested-by: Nick Desaulniers <ndesaulniers@google.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 3bf992f9

arch/arm/crypto/ghash-ce-core.S

+3 −2

Original line number	Diff line number	Diff line
		@@ -8,6 +8,9 @@
		#include <linux/linkage.h>
		#include <asm/assembler.h>

		.arch armv8-a
		.fpu crypto-neon-fp-armv8

		SHASH .req q0
		T1 .req q1
		XL .req q2
		@@ -88,8 +91,6 @@
		T3_H .req d17

		.text
		.arch armv8-a
		.fpu crypto-neon-fp-armv8

		.macro __pmull_p64, rd, rn, rm, b1, b2, b3, b4
		vmull.p64 \rd, \rn, \rm