diff mbox

kbuild: remove old information in headers_install.txt document

Message ID 1417494533-24461-1-git-send-email-yamada.m@jp.panasonic.com (mailing list archive)
State New, archived
Headers show

Commit Message

Masahiro Yamada Dec. 2, 2014, 4:28 a.m. UTC
The arch header directories "include/asm-*" were moved long before.

Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
---

 Documentation/kbuild/headers_install.txt | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comments

Jonathan Corbet Dec. 2, 2014, 1:39 p.m. UTC | #1
On Tue,  2 Dec 2014 13:28:53 +0900
Masahiro Yamada <yamada.m@jp.panasonic.com> wrote:

> diff --git a/Documentation/kbuild/headers_install.txt b/Documentation/kbuild/headers_install.txt
> index 951eb9f..0d8fe5e 100644
> --- a/Documentation/kbuild/headers_install.txt
> +++ b/Documentation/kbuild/headers_install.txt
> @@ -28,10 +28,7 @@ optional arguments:
>  
>  ARCH indicates which architecture to produce headers for, and defaults to the
>  current architecture.  The linux/asm directory of the exported kernel headers
> -is platform-specific, to see a complete list of supported architectures use
> -the command:
> -
> -  ls -d include/asm-* | sed 's/.*-//'
> +is platform-specific.
>  

This change makes sense as far as it goes, but might it not be better to
tell readers the way to get the list of supported architectures in the
new scheme of things...?

Thanks,

jon
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Yann Droneaud Dec. 5, 2014, 2:23 p.m. UTC | #2
Hi,

Le mardi 02 décembre 2014 à 13:28 +0900, Masahiro Yamada a écrit :
> The arch header directories "include/asm-*" were moved long before.
> 
> Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> ---
> 
>  Documentation/kbuild/headers_install.txt | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/Documentation/kbuild/headers_install.txt b/Documentation/kbuild/headers_install.txt
> index 951eb9f..0d8fe5e 100644
> --- a/Documentation/kbuild/headers_install.txt
> +++ b/Documentation/kbuild/headers_install.txt
> @@ -28,10 +28,7 @@ optional arguments:
>  
>  ARCH indicates which architecture to produce headers for, and defaults to the
>  current architecture.  The linux/asm directory of the exported kernel headers
> -is platform-specific, to see a complete list of supported architectures use
> -the command:
> -
> -  ls -d include/asm-* | sed 's/.*-//'
> +is platform-specific.
>  
>  INSTALL_HDR_PATH indicates where to install the headers.  It defaults to
>  "./usr/include".

Back in september, I've already sent a patch[1] to address this,
following a discussion with Sam Ravnborg[2]. See the mail thread[3].

Regards.

[1] http://lkml.kernel.org/r/b5181cf7604baa454c11f7aa92d07dd05349ce46.1410712841.git.ydroneaud@opteya.com
[2] http://lkml.kernel.org/r/20140713212608.GB16496@ravnborg.org
[3] http://lkml.kernel.org/g/cover.1410712841.git.ydroneaud@opteya.com
Masahiro Yamada Dec. 11, 2014, 1:41 a.m. UTC | #3
Hi Yann,






On Fri, 05 Dec 2014 15:23:48 +0100
Yann Droneaud <ydroneaud@opteya.com> wrote:

> Hi,
> 
> Le mardi 02 decembre 2014 a 13:28 +0900, Masahiro Yamada a ecrit :
> > The arch header directories "include/asm-*" were moved long before.
> > 
> > Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> > ---
> > 
> >  Documentation/kbuild/headers_install.txt | 5 +----
> >  1 file changed, 1 insertion(+), 4 deletions(-)
> > 
> > diff --git a/Documentation/kbuild/headers_install.txt b/Documentation/kbuild/headers_install.txt
> > index 951eb9f..0d8fe5e 100644
> > --- a/Documentation/kbuild/headers_install.txt
> > +++ b/Documentation/kbuild/headers_install.txt
> > @@ -28,10 +28,7 @@ optional arguments:
> >  
> >  ARCH indicates which architecture to produce headers for, and defaults to the
> >  current architecture.  The linux/asm directory of the exported kernel headers
> > -is platform-specific, to see a complete list of supported architectures use
> > -the command:
> > -
> > -  ls -d include/asm-* | sed 's/.*-//'
> > +is platform-specific.
> >  
> >  INSTALL_HDR_PATH indicates where to install the headers.  It defaults to
> >  "./usr/include".
> 
> Back in september, I've already sent a patch[1] to address this,
> following a discussion with Sam Ravnborg[2]. See the mail thread[3].
> 
> Regards.
> 
> [1] http://lkml.kernel.org/r/b5181cf7604baa454c11f7aa92d07dd05349ce46.1410712841.git.ydroneaud@opteya.com
> [2] http://lkml.kernel.org/r/20140713212608.GB16496@ravnborg.org
> [3] http://lkml.kernel.org/g/cover.1410712841.git.ydroneaud@opteya.com
> 

Sorry, I missed your patch.

You are the first, so I agree yours should go in.



Best Regards
Masahiro Yamada

--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Documentation/kbuild/headers_install.txt b/Documentation/kbuild/headers_install.txt
index 951eb9f..0d8fe5e 100644
--- a/Documentation/kbuild/headers_install.txt
+++ b/Documentation/kbuild/headers_install.txt
@@ -28,10 +28,7 @@  optional arguments:
 
 ARCH indicates which architecture to produce headers for, and defaults to the
 current architecture.  The linux/asm directory of the exported kernel headers
-is platform-specific, to see a complete list of supported architectures use
-the command:
-
-  ls -d include/asm-* | sed 's/.*-//'
+is platform-specific.
 
 INSTALL_HDR_PATH indicates where to install the headers.  It defaults to
 "./usr/include".