linux-kernel.vger.kernel.org archive mirror
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).