2014-04-20 12:34:18

by Terence Eden

[permalink] [raw]
Subject: [PATCH] Fixed spelling errors within comments

From: Terence Eden <[email protected]>

This is my first patch submission. Please let me know if I have made a mistake anywhere.

Signed-off-by: Terence Eden <[email protected]>
---
arch/arc/kernel/entry.S | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arc/kernel/entry.S b/arch/arc/kernel/entry.S
index 819dd5f..38ef91a 100644
--- a/arch/arc/kernel/entry.S
+++ b/arch/arc/kernel/entry.S
@@ -156,7 +156,7 @@ ARCFP_DATA int1_saved_reg
int1_saved_reg:
.zero 4

-/* Each Interrupt level needs it's own scratch */
+/* Each Interrupt level needs its own scratch */
#ifdef CONFIG_ARC_COMPACT_IRQ_LEVELS

ARCFP_DATA int2_saved_reg
@@ -473,7 +473,7 @@ trap_with_param:
lr r0, [efa]
mov r1, sp

- ; Now that we have read EFA, its safe to do "fake" rtie
+ ; Now that we have read EFA, it is safe to do "fake" rtie
; and get out of CPU exception mode
FAKE_RET_FROM_EXCPN r11

@@ -676,9 +676,9 @@ not_exception:
brne r9, event_IRQ2, 149f

;------------------------------------------------------------------
- ; if L2 IRQ interrupted a L1 ISR, we'd disbaled preemption earlier
- ; so that sched doesnt move to new task, causing L1 to be delayed
- ; undeterministically. Now that we've achieved that, lets reset
+ ; if L2 IRQ interrupted a L1 ISR, we'd disabled preemption earlier
+ ; so that sched doesn't move to new task, causing L1 to be delayed
+ ; undeterministically. Now that we've achieved that, let's reset
; things to what they were, before returning from L2 context
;----------------------------------------------------------------

@@ -734,7 +734,7 @@ ENTRY(ret_from_fork)
; put last task in scheduler queue
bl @schedule_tail

- ; If kernel thread, jump to it's entry-point
+ ; If kernel thread, jump to its entry-point
ld r9, [sp, PT_status32]
brne r9, 0, 1f

--
1.7.9.5


2014-04-20 17:00:06

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH] Fixed spelling errors within comments

On 04/20/2014 05:33 AM, Terence Eden wrote:
> From: Terence Eden <[email protected]>
>
> This is my first patch submission. Please let me know if I have made a mistake anywhere.
>
> Signed-off-by: Terence Eden <[email protected]>

trivial's email address is [email protected] (cc-ed).

> ---
> arch/arc/kernel/entry.S | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arc/kernel/entry.S b/arch/arc/kernel/entry.S
> index 819dd5f..38ef91a 100644
> --- a/arch/arc/kernel/entry.S
> +++ b/arch/arc/kernel/entry.S
> @@ -156,7 +156,7 @@ ARCFP_DATA int1_saved_reg
> int1_saved_reg:
> .zero 4
>
> -/* Each Interrupt level needs it's own scratch */
> +/* Each Interrupt level needs its own scratch */
> #ifdef CONFIG_ARC_COMPACT_IRQ_LEVELS
>
> ARCFP_DATA int2_saved_reg
> @@ -473,7 +473,7 @@ trap_with_param:
> lr r0, [efa]
> mov r1, sp
>
> - ; Now that we have read EFA, its safe to do "fake" rtie
> + ; Now that we have read EFA, it is safe to do "fake" rtie
> ; and get out of CPU exception mode
> FAKE_RET_FROM_EXCPN r11
>
> @@ -676,9 +676,9 @@ not_exception:
> brne r9, event_IRQ2, 149f
>
> ;------------------------------------------------------------------
> - ; if L2 IRQ interrupted a L1 ISR, we'd disbaled preemption earlier
> - ; so that sched doesnt move to new task, causing L1 to be delayed
> - ; undeterministically. Now that we've achieved that, lets reset
> + ; if L2 IRQ interrupted a L1 ISR, we'd disabled preemption earlier

I would write: an L1 ISR,

anyway:
Acked-by: Randy Dunlap <[email protected]>


> + ; so that sched doesn't move to new task, causing L1 to be delayed
> + ; undeterministically. Now that we've achieved that, let's reset
> ; things to what they were, before returning from L2 context
> ;----------------------------------------------------------------
>
> @@ -734,7 +734,7 @@ ENTRY(ret_from_fork)
> ; put last task in scheduler queue
> bl @schedule_tail
>
> - ; If kernel thread, jump to it's entry-point
> + ; If kernel thread, jump to its entry-point
> ld r9, [sp, PT_status32]
> brne r9, 0, 1f
>
>


--
~Randy

2014-04-21 05:24:45

by Vineet Gupta

[permalink] [raw]
Subject: Re: [PATCH] Fixed spelling errors within comments

Hi Terence,

Thanks for the fixes.

On Sunday 20 April 2014 10:30 PM, Randy Dunlap wrote:
> On 04/20/2014 05:33 AM, Terence Eden wrote:
>> > From: Terence Eden <[email protected]>
>> >
>> > This is my first patch submission. Please let me know if I have made a mistake anywhere.
>> >
>> ...
> I would write: an L1 ISR,
>
> anyway:
> Acked-by: Randy Dunlap <[email protected]>

Applied to ARC for-next with Randy's ACK.

-Vineet