All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] move libgcc.h to include/linux
@ 2017-11-22 10:47 Christoph Hellwig
  2017-11-22 16:50 ` Palmer Dabbelt
  0 siblings, 1 reply; 4+ messages in thread
From: Christoph Hellwig @ 2017-11-22 10:47 UTC (permalink / raw)
  To: torvalds; +Cc: palmer, linux-kernel

Introducing a new include/lib directory just for this file totally
messes up tab completion for include/linux, which is highly annoying.

Move it to include/linux where we have headers for all kinds of other
lib/ code as well.

Signed-off-by: Christoph Hellwig <hch@lst.de>
---
 include/{lib => linux}/libgcc.h | 0
 lib/ashldi3.c                   | 2 +-
 lib/ashrdi3.c                   | 2 +-
 lib/cmpdi2.c                    | 2 +-
 lib/lshrdi3.c                   | 2 +-
 lib/muldi3.c                    | 2 +-
 lib/ucmpdi2.c                   | 2 +-
 7 files changed, 6 insertions(+), 6 deletions(-)
 rename include/{lib => linux}/libgcc.h (100%)

diff --git a/include/lib/libgcc.h b/include/linux/libgcc.h
similarity index 100%
rename from include/lib/libgcc.h
rename to include/linux/libgcc.h
diff --git a/lib/ashldi3.c b/lib/ashldi3.c
index 1b6087db95a5..3ffc46e3bb6c 100644
--- a/lib/ashldi3.c
+++ b/lib/ashldi3.c
@@ -16,7 +16,7 @@
 
 #include <linux/export.h>
 
-#include <lib/libgcc.h>
+#include <linux/libgcc.h>
 
 long long notrace __ashldi3(long long u, word_type b)
 {
diff --git a/lib/ashrdi3.c b/lib/ashrdi3.c
index 2e67c97ac65a..ea054550f0e8 100644
--- a/lib/ashrdi3.c
+++ b/lib/ashrdi3.c
@@ -16,7 +16,7 @@
 
 #include <linux/export.h>
 
-#include <lib/libgcc.h>
+#include <linux/libgcc.h>
 
 long long notrace __ashrdi3(long long u, word_type b)
 {
diff --git a/lib/cmpdi2.c b/lib/cmpdi2.c
index 6d7ebf6c2b86..2250da7e503e 100644
--- a/lib/cmpdi2.c
+++ b/lib/cmpdi2.c
@@ -16,7 +16,7 @@
 
 #include <linux/export.h>
 
-#include <lib/libgcc.h>
+#include <linux/libgcc.h>
 
 word_type notrace __cmpdi2(long long a, long long b)
 {
diff --git a/lib/lshrdi3.c b/lib/lshrdi3.c
index 8e845f4bb65f..99cfa5721f2d 100644
--- a/lib/lshrdi3.c
+++ b/lib/lshrdi3.c
@@ -17,7 +17,7 @@
  */
 
 #include <linux/module.h>
-#include <lib/libgcc.h>
+#include <linux/libgcc.h>
 
 long long notrace __lshrdi3(long long u, word_type b)
 {
diff --git a/lib/muldi3.c b/lib/muldi3.c
index 88938543e10a..54c8b3123376 100644
--- a/lib/muldi3.c
+++ b/lib/muldi3.c
@@ -15,7 +15,7 @@
  */
 
 #include <linux/export.h>
-#include <lib/libgcc.h>
+#include <linux/libgcc.h>
 
 #define W_TYPE_SIZE 32
 
diff --git a/lib/ucmpdi2.c b/lib/ucmpdi2.c
index 49a53505c8e3..25ca2d4c1e19 100644
--- a/lib/ucmpdi2.c
+++ b/lib/ucmpdi2.c
@@ -15,7 +15,7 @@
  */
 
 #include <linux/module.h>
-#include <lib/libgcc.h>
+#include <linux/libgcc.h>
 
 word_type __ucmpdi2(unsigned long long a, unsigned long long b)
 {
-- 
2.14.2

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

* Re: [PATCH] move libgcc.h to include/linux
  2017-11-22 10:47 [PATCH] move libgcc.h to include/linux Christoph Hellwig
@ 2017-11-22 16:50 ` Palmer Dabbelt
  2017-11-23  8:03   ` Christoph Hellwig
  0 siblings, 1 reply; 4+ messages in thread
From: Palmer Dabbelt @ 2017-11-22 16:50 UTC (permalink / raw)
  To: hch; +Cc: Linus Torvalds, linux-kernel

On Wed, 22 Nov 2017 02:47:28 PST (-0800), hch@lst.de wrote:
> Introducing a new include/lib directory just for this file totally
> messes up tab completion for include/linux, which is highly annoying.

Sorry about that.

> Move it to include/linux where we have headers for all kinds of other
> lib/ code as well.

That looks good to me.  I'm kind of new to this, do you want me to take this 
into my tree?

Reviewed-by: Palmer Dabbelt <palmer@sifive.com>

>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
>  include/{lib => linux}/libgcc.h | 0
>  lib/ashldi3.c                   | 2 +-
>  lib/ashrdi3.c                   | 2 +-
>  lib/cmpdi2.c                    | 2 +-
>  lib/lshrdi3.c                   | 2 +-
>  lib/muldi3.c                    | 2 +-
>  lib/ucmpdi2.c                   | 2 +-
>  7 files changed, 6 insertions(+), 6 deletions(-)
>  rename include/{lib => linux}/libgcc.h (100%)
>
> diff --git a/include/lib/libgcc.h b/include/linux/libgcc.h
> similarity index 100%
> rename from include/lib/libgcc.h
> rename to include/linux/libgcc.h
> diff --git a/lib/ashldi3.c b/lib/ashldi3.c
> index 1b6087db95a5..3ffc46e3bb6c 100644
> --- a/lib/ashldi3.c
> +++ b/lib/ashldi3.c
> @@ -16,7 +16,7 @@
>
>  #include <linux/export.h>
>
> -#include <lib/libgcc.h>
> +#include <linux/libgcc.h>
>
>  long long notrace __ashldi3(long long u, word_type b)
>  {
> diff --git a/lib/ashrdi3.c b/lib/ashrdi3.c
> index 2e67c97ac65a..ea054550f0e8 100644
> --- a/lib/ashrdi3.c
> +++ b/lib/ashrdi3.c
> @@ -16,7 +16,7 @@
>
>  #include <linux/export.h>
>
> -#include <lib/libgcc.h>
> +#include <linux/libgcc.h>
>
>  long long notrace __ashrdi3(long long u, word_type b)
>  {
> diff --git a/lib/cmpdi2.c b/lib/cmpdi2.c
> index 6d7ebf6c2b86..2250da7e503e 100644
> --- a/lib/cmpdi2.c
> +++ b/lib/cmpdi2.c
> @@ -16,7 +16,7 @@
>
>  #include <linux/export.h>
>
> -#include <lib/libgcc.h>
> +#include <linux/libgcc.h>
>
>  word_type notrace __cmpdi2(long long a, long long b)
>  {
> diff --git a/lib/lshrdi3.c b/lib/lshrdi3.c
> index 8e845f4bb65f..99cfa5721f2d 100644
> --- a/lib/lshrdi3.c
> +++ b/lib/lshrdi3.c
> @@ -17,7 +17,7 @@
>   */
>
>  #include <linux/module.h>
> -#include <lib/libgcc.h>
> +#include <linux/libgcc.h>
>
>  long long notrace __lshrdi3(long long u, word_type b)
>  {
> diff --git a/lib/muldi3.c b/lib/muldi3.c
> index 88938543e10a..54c8b3123376 100644
> --- a/lib/muldi3.c
> +++ b/lib/muldi3.c
> @@ -15,7 +15,7 @@
>   */
>
>  #include <linux/export.h>
> -#include <lib/libgcc.h>
> +#include <linux/libgcc.h>
>
>  #define W_TYPE_SIZE 32
>
> diff --git a/lib/ucmpdi2.c b/lib/ucmpdi2.c
> index 49a53505c8e3..25ca2d4c1e19 100644
> --- a/lib/ucmpdi2.c
> +++ b/lib/ucmpdi2.c
> @@ -15,7 +15,7 @@
>   */
>
>  #include <linux/module.h>
> -#include <lib/libgcc.h>
> +#include <linux/libgcc.h>
>
>  word_type __ucmpdi2(unsigned long long a, unsigned long long b)
>  {

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

* Re: [PATCH] move libgcc.h to include/linux
  2017-11-22 16:50 ` Palmer Dabbelt
@ 2017-11-23  8:03   ` Christoph Hellwig
  2017-11-27 17:01     ` Palmer Dabbelt
  0 siblings, 1 reply; 4+ messages in thread
From: Christoph Hellwig @ 2017-11-23  8:03 UTC (permalink / raw)
  To: Palmer Dabbelt; +Cc: hch, Linus Torvalds, linux-kernel

On Wed, Nov 22, 2017 at 08:50:25AM -0800, Palmer Dabbelt wrote:
> On Wed, 22 Nov 2017 02:47:28 PST (-0800), hch@lst.de wrote:
>> Introducing a new include/lib directory just for this file totally
>> messes up tab completion for include/linux, which is highly annoying.
>
> Sorry about that.
>
>> Move it to include/linux where we have headers for all kinds of other
>> lib/ code as well.
>
> That looks good to me.  I'm kind of new to this, do you want me to take 
> this into my tree?
>
> Reviewed-by: Palmer Dabbelt <palmer@sifive.com>

I sent it to Linus as I'd expect him to just apply it directly.  He
hasn't picked it up yesterday, so maybe you should just send it along
with your next RISC-V updates.

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

* Re: [PATCH] move libgcc.h to include/linux
  2017-11-23  8:03   ` Christoph Hellwig
@ 2017-11-27 17:01     ` Palmer Dabbelt
  0 siblings, 0 replies; 4+ messages in thread
From: Palmer Dabbelt @ 2017-11-27 17:01 UTC (permalink / raw)
  To: hch; +Cc: Linus Torvalds, linux-kernel

On Thu, 23 Nov 2017 00:03:18 PST (-0800), hch@lst.de wrote:
> On Wed, Nov 22, 2017 at 08:50:25AM -0800, Palmer Dabbelt wrote:
>> On Wed, 22 Nov 2017 02:47:28 PST (-0800), hch@lst.de wrote:
>>> Introducing a new include/lib directory just for this file totally
>>> messes up tab completion for include/linux, which is highly annoying.
>>
>> Sorry about that.
>>
>>> Move it to include/linux where we have headers for all kinds of other
>>> lib/ code as well.
>>
>> That looks good to me.  I'm kind of new to this, do you want me to take
>> this into my tree?
>>
>> Reviewed-by: Palmer Dabbelt <palmer@sifive.com>
>
> I sent it to Linus as I'd expect him to just apply it directly.  He
> hasn't picked it up yesterday, so maybe you should just send it along
> with your next RISC-V updates.

Doesn't look like it made 4.15-rc1, so I'll include it with my next pull 
request.

Thanks!

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

end of thread, other threads:[~2017-11-27 17:01 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-22 10:47 [PATCH] move libgcc.h to include/linux Christoph Hellwig
2017-11-22 16:50 ` Palmer Dabbelt
2017-11-23  8:03   ` Christoph Hellwig
2017-11-27 17:01     ` Palmer Dabbelt

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.