All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tools: compiler.h: remove duplicate #ifndef noinline block
@ 2021-10-15  8:31 Rasmus Villemoes
  2021-10-15 11:46 ` Miguel Ojeda
  0 siblings, 1 reply; 3+ messages in thread
From: Rasmus Villemoes @ 2021-10-15  8:31 UTC (permalink / raw)
  To: Arnaldo Carvalho de Melo, Miguel Ojeda
  Cc: linux-kernel, Ard Biesheuvel, Rasmus Villemoes

The same three lines also appear a bit earlier in the same file.

Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
---
 tools/include/linux/compiler.h | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/tools/include/linux/compiler.h b/tools/include/linux/compiler.h
index ff872dc2637c..90ba44a99199 100644
--- a/tools/include/linux/compiler.h
+++ b/tools/include/linux/compiler.h
@@ -102,10 +102,6 @@
 # define __init
 #endif
 
-#ifndef noinline
-# define noinline
-#endif
-
 #include <linux/types.h>
 
 /*
-- 
2.31.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] tools: compiler.h: remove duplicate #ifndef noinline block
  2021-10-15  8:31 [PATCH] tools: compiler.h: remove duplicate #ifndef noinline block Rasmus Villemoes
@ 2021-10-15 11:46 ` Miguel Ojeda
  2022-03-07 17:59   ` Arnaldo Carvalho de Melo
  0 siblings, 1 reply; 3+ messages in thread
From: Miguel Ojeda @ 2021-10-15 11:46 UTC (permalink / raw)
  To: Rasmus Villemoes
  Cc: Arnaldo Carvalho de Melo, Miguel Ojeda, linux-kernel,
	Ard Biesheuvel

On Fri, Oct 15, 2021 at 10:31 AM Rasmus Villemoes
<linux@rasmusvillemoes.dk> wrote:
>
> The same three lines also appear a bit earlier in the same file.
>
> Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>

Looks good to me -- they also appear to be at the same level.

Reviewed-by: Miguel Ojeda <ojeda@kernel.org>

Cheers,
Miguel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] tools: compiler.h: remove duplicate #ifndef noinline block
  2021-10-15 11:46 ` Miguel Ojeda
@ 2022-03-07 17:59   ` Arnaldo Carvalho de Melo
  0 siblings, 0 replies; 3+ messages in thread
From: Arnaldo Carvalho de Melo @ 2022-03-07 17:59 UTC (permalink / raw)
  To: Miguel Ojeda
  Cc: Rasmus Villemoes, Arnaldo Carvalho de Melo, Miguel Ojeda,
	linux-kernel, Ard Biesheuvel

Em Fri, Oct 15, 2021 at 01:46:02PM +0200, Miguel Ojeda escreveu:
> On Fri, Oct 15, 2021 at 10:31 AM Rasmus Villemoes
> <linux@rasmusvillemoes.dk> wrote:
> >
> > The same three lines also appear a bit earlier in the same file.
> >
> > Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
> 
> Looks good to me -- they also appear to be at the same level.
> 
> Reviewed-by: Miguel Ojeda <ojeda@kernel.org>

Fell thru the cracks, duh.

Thanks, applied.

- Arnaldo


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-03-07 18:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-15  8:31 [PATCH] tools: compiler.h: remove duplicate #ifndef noinline block Rasmus Villemoes
2021-10-15 11:46 ` Miguel Ojeda
2022-03-07 17:59   ` Arnaldo Carvalho de Melo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.