2021-03-20 18:30:07

by Bhaskar Chowdhury

[permalink] [raw]
Subject: [PATCH] IA64: Trivial spelling fixes


s/seralize/serialize/ .....three different places

Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
arch/ia64/kernel/pal.S | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/ia64/kernel/pal.S b/arch/ia64/kernel/pal.S
index d3e22c018b68..06d01a070aae 100644
--- a/arch/ia64/kernel/pal.S
+++ b/arch/ia64/kernel/pal.S
@@ -86,7 +86,7 @@ GLOBAL_ENTRY(ia64_pal_call_static)
mov ar.pfs = loc1
mov rp = loc0
;;
- srlz.d // seralize restoration of psr.l
+ srlz.d // serialize restoration of psr.l
br.ret.sptk.many b0
END(ia64_pal_call_static)
EXPORT_SYMBOL(ia64_pal_call_static)
@@ -194,7 +194,7 @@ GLOBAL_ENTRY(ia64_pal_call_phys_static)
mov rp = loc0
;;
mov ar.rsc=loc4 // restore RSE configuration
- srlz.d // seralize restoration of psr.l
+ srlz.d // serialize restoration of psr.l
br.ret.sptk.many b0
END(ia64_pal_call_phys_static)
EXPORT_SYMBOL(ia64_pal_call_phys_static)
@@ -252,7 +252,7 @@ GLOBAL_ENTRY(ia64_pal_call_phys_stacked)
mov rp = loc0
;;
mov ar.rsc=loc4 // restore RSE configuration
- srlz.d // seralize restoration of psr.l
+ srlz.d // serialize restoration of psr.l
br.ret.sptk.many b0
END(ia64_pal_call_phys_stacked)
EXPORT_SYMBOL(ia64_pal_call_phys_stacked)
--
2.26.2


Subject: Re: [PATCH] IA64: Trivial spelling fixes

Hi Bhaskar!

On 3/20/21 7:23 PM, Bhaskar Chowdhury wrote:
>
> s/seralize/serialize/ .....three different places
>
> Signed-off-by: Bhaskar Chowdhury <[email protected]>
> ---
> arch/ia64/kernel/pal.S | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/ia64/kernel/pal.S b/arch/ia64/kernel/pal.S
> index d3e22c018b68..06d01a070aae 100644
> --- a/arch/ia64/kernel/pal.S
> +++ b/arch/ia64/kernel/pal.S
> @@ -86,7 +86,7 @@ GLOBAL_ENTRY(ia64_pal_call_static)
> mov ar.pfs = loc1
> mov rp = loc0
> ;;
> - srlz.d // seralize restoration of psr.l
> + srlz.d // serialize restoration of psr.l
> br.ret.sptk.many b0
> END(ia64_pal_call_static)
> EXPORT_SYMBOL(ia64_pal_call_static)
> @@ -194,7 +194,7 @@ GLOBAL_ENTRY(ia64_pal_call_phys_static)
> mov rp = loc0
> ;;
> mov ar.rsc=loc4 // restore RSE configuration
> - srlz.d // seralize restoration of psr.l
> + srlz.d // serialize restoration of psr.l
> br.ret.sptk.many b0
> END(ia64_pal_call_phys_static)
> EXPORT_SYMBOL(ia64_pal_call_phys_static)
> @@ -252,7 +252,7 @@ GLOBAL_ENTRY(ia64_pal_call_phys_stacked)
> mov rp = loc0
> ;;
> mov ar.rsc=loc4 // restore RSE configuration
> - srlz.d // seralize restoration of psr.l
> + srlz.d // serialize restoration of psr.l
> br.ret.sptk.many b0
> END(ia64_pal_call_phys_stacked)
> EXPORT_SYMBOL(ia64_pal_call_phys_stacked)
> --
> 2.26.2

Thanks for fixing this. Btw, we usually use all-lowercase letters for architecture
names in the Linux kernel, so it should probably be "ia64: Trivial spelling fixes".

And the easiest way to get those fixes into the kernel would be through Andrew Morton's
tree.

Adrian

--
.''`. John Paul Adrian Glaubitz
: :' : Debian Developer - [email protected]
`. `' Freie Universitaet Berlin - [email protected]
`- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913

2021-03-20 19:12:14

by Bhaskar Chowdhury

[permalink] [raw]
Subject: Re: [PATCH] IA64: Trivial spelling fixes

On 20:01 Sat 20 Mar 2021, John Paul Adrian Glaubitz wrote:
>Hi Bhaskar!
>
>On 3/20/21 7:23 PM, Bhaskar Chowdhury wrote:
>>
>> s/seralize/serialize/ .....three different places
>>
>> Signed-off-by: Bhaskar Chowdhury <[email protected]>
>> ---
>> arch/ia64/kernel/pal.S | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/ia64/kernel/pal.S b/arch/ia64/kernel/pal.S
>> index d3e22c018b68..06d01a070aae 100644
>> --- a/arch/ia64/kernel/pal.S
>> +++ b/arch/ia64/kernel/pal.S
>> @@ -86,7 +86,7 @@ GLOBAL_ENTRY(ia64_pal_call_static)
>> mov ar.pfs = loc1
>> mov rp = loc0
>> ;;
>> - srlz.d // seralize restoration of psr.l
>> + srlz.d // serialize restoration of psr.l
>> br.ret.sptk.many b0
>> END(ia64_pal_call_static)
>> EXPORT_SYMBOL(ia64_pal_call_static)
>> @@ -194,7 +194,7 @@ GLOBAL_ENTRY(ia64_pal_call_phys_static)
>> mov rp = loc0
>> ;;
>> mov ar.rsc=loc4 // restore RSE configuration
>> - srlz.d // seralize restoration of psr.l
>> + srlz.d // serialize restoration of psr.l
>> br.ret.sptk.many b0
>> END(ia64_pal_call_phys_static)
>> EXPORT_SYMBOL(ia64_pal_call_phys_static)
>> @@ -252,7 +252,7 @@ GLOBAL_ENTRY(ia64_pal_call_phys_stacked)
>> mov rp = loc0
>> ;;
>> mov ar.rsc=loc4 // restore RSE configuration
>> - srlz.d // seralize restoration of psr.l
>> + srlz.d // serialize restoration of psr.l
>> br.ret.sptk.many b0
>> END(ia64_pal_call_phys_stacked)
>> EXPORT_SYMBOL(ia64_pal_call_phys_stacked)
>> --
>> 2.26.2
>
>Thanks for fixing this. Btw, we usually use all-lowercase letters for architecture
>names in the Linux kernel, so it should probably be "ia64: Trivial spelling fixes".
>
>And the easiest way to get those fixes into the kernel would be through Andrew Morton's
>tree.
>

I did send it to Andrew to pick up . And I took a cue from the earlier commit
messages for the architecture ...like this :

4d5a31977cc6 [IA64] reformat pal.S to fit in 80 columns, fix typos
c12fb1885787 [IA64] remove unused PAL_CALL_IC_OFF
acb15c85de57 [IA64] Do not assume output registers be reservered.

....so followed the conventions :)

>Adrian
>
>--
> .''`. John Paul Adrian Glaubitz
>: :' : Debian Developer - [email protected]
>`. `' Freie Universitaet Berlin - [email protected]
> `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
>


Attachments:
(No filename) (2.37 kB)
signature.asc (499.00 B)
Download all attachments