Hi Stefan, On Thu, 7 Feb 2019 at 10:46, Stefan Agner <[email protected]> wrote: > > Use unified assembler syntax (UAL) in macros. Divided syntax is > considered depricated.
'deprecated' > This will also allow to build the kernel > using LLVM's integrated assembler. > > Signed-off-by: Stefan Agner <[email protected]> > --- > arch/arm/lib/copy_from_user.S | 2 +- > arch/arm/lib/copy_to_user.S | 2 +- > arch/arm/lib/memcpy.S | 4 ++-- > 3 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/lib/copy_from_user.S b/arch/arm/lib/copy_from_user.S > index 0d4c189c7f4f..712ca399f559 100644 > --- a/arch/arm/lib/copy_from_user.S > +++ b/arch/arm/lib/copy_from_user.S > @@ -91,7 +91,7 @@ > .endm > > .macro str1b ptr reg cond=al abort > - str\cond\()b \reg, [\ptr], #1 > + strb\cond\() \reg, [\ptr], #1 You can drop the null token here, you only need that in the middle of a word. Same below. > .endm > > .macro enter reg1 reg2 > diff --git a/arch/arm/lib/copy_to_user.S b/arch/arm/lib/copy_to_user.S > index 97a6ff4b7e3c..2e402b815e13 100644 > --- a/arch/arm/lib/copy_to_user.S > +++ b/arch/arm/lib/copy_to_user.S > @@ -49,7 +49,7 @@ > .endm > > .macro ldr1b ptr reg cond=al abort > - ldr\cond\()b \reg, [\ptr], #1 > + ldrb\cond\() \reg, [\ptr], #1 > .endm > > #ifdef CONFIG_CPU_USE_DOMAINS > diff --git a/arch/arm/lib/memcpy.S b/arch/arm/lib/memcpy.S > index 64111bd4440b..1c9e625148ca 100644 > --- a/arch/arm/lib/memcpy.S > +++ b/arch/arm/lib/memcpy.S > @@ -30,7 +30,7 @@ > .endm > > .macro ldr1b ptr reg cond=al abort > - ldr\cond\()b \reg, [\ptr], #1 > + ldrb\cond\() \reg, [\ptr], #1 > .endm > > .macro str1w ptr reg abort > @@ -42,7 +42,7 @@ > .endm > > .macro str1b ptr reg cond=al abort > - str\cond\()b \reg, [\ptr], #1 > + strb\cond\() \reg, [\ptr], #1 > .endm > > .macro enter reg1 reg2 > -- > 2.20.1 >
