2017-09-04 19:15:30

by Jérémy Lefaure

[permalink] [raw]
Subject: [RESEND PATCH] c6x: remove unused KTHREAD_SIZE definition

KTHREAD_SIZE has never been used since it has been defined for c6x arch.
Let's remove this useless definition.

Signed-off-by: Jérémy Lefaure <[email protected]>
Acked-by: Mark Salter <[email protected]>
---
Hi,
I sent this patch in June. It was acked by Mark Salter but I don't see
it in the last kernel release so I resend it today.

arch/c6x/kernel/asm-offsets.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/c6x/kernel/asm-offsets.c b/arch/c6x/kernel/asm-offsets.c
index 60f1e437745d..f8ededbfb63f 100644
--- a/arch/c6x/kernel/asm-offsets.c
+++ b/arch/c6x/kernel/asm-offsets.c
@@ -106,7 +106,6 @@ void foo(void)
/* These would be unneccessary if we ran asm files
* through the preprocessor.
*/
- DEFINE(KTHREAD_SIZE, THREAD_SIZE);
DEFINE(KTHREAD_SHIFT, THREAD_SHIFT);
DEFINE(KTHREAD_START_SP, THREAD_START_SP);
DEFINE(ENOSYS_, ENOSYS);
--
2.14.1


2017-09-06 15:41:21

by Mark Salter

[permalink] [raw]
Subject: Re: [RESEND PATCH] c6x: remove unused KTHREAD_SIZE definition

On Mon, 2017-09-04 at 15:14 -0400, Jérémy Lefaure wrote:
> KTHREAD_SIZE has never been used since it has been defined for c6x arch.
> Let's remove this useless definition.
>
> Signed-off-by: Jérémy Lefaure <[email protected]>
> Acked-by: Mark Salter <[email protected]>
> ---
> Hi,
> I sent this patch in June. It was acked by Mark Salter but I don't see
> it in the last kernel release so I resend it today.

Thanks. I pulled it into c6x tree.

>
> arch/c6x/kernel/asm-offsets.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/c6x/kernel/asm-offsets.c b/arch/c6x/kernel/asm-offsets.c
> index 60f1e437745d..f8ededbfb63f 100644
> --- a/arch/c6x/kernel/asm-offsets.c
> +++ b/arch/c6x/kernel/asm-offsets.c
> @@ -106,7 +106,6 @@ void foo(void)
> /* These would be unneccessary if we ran asm files
> * through the preprocessor.
> */
> - DEFINE(KTHREAD_SIZE, THREAD_SIZE);
> DEFINE(KTHREAD_SHIFT, THREAD_SHIFT);
> DEFINE(KTHREAD_START_SP, THREAD_START_SP);
> DEFINE(ENOSYS_, ENOSYS);