linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] kbuild: remove wrong documentation about mandatory-y
@ 2020-02-19  1:15 Masahiro Yamada
  2020-02-20 16:28 ` Masahiro Yamada
  0 siblings, 1 reply; 2+ messages in thread
From: Masahiro Yamada @ 2020-02-19  1:15 UTC (permalink / raw)
  To: linux-kbuild
  Cc: Masahiro Yamada, Jonathan Corbet, Michal Marek, linux-doc, linux-kernel

This sentence does not make sense in the section about mandatory-y.

This seems to be a copy-paste mistake of commit fcc8487d477a ("uapi:
export all headers under uapi directories").

The correct description would be "The convention is to list one
mandatory-y per line ...".

I just removed it instead of fixing it. If such information is needed,
it could be commented in include/asm-generic/Kbuild and
include/uapi/asm-generic/Kbuild.

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
---

 Documentation/kbuild/makefiles.rst | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/Documentation/kbuild/makefiles.rst b/Documentation/kbuild/makefiles.rst
index 0e0eb2c8da7d..4018ad7c7a11 100644
--- a/Documentation/kbuild/makefiles.rst
+++ b/Documentation/kbuild/makefiles.rst
@@ -1379,9 +1379,6 @@ See subsequent chapter for the syntax of the Kbuild file.
 	in arch/$(ARCH)/include/(uapi/)/asm, Kbuild will automatically generate
 	a wrapper of the asm-generic one.
 
-	The convention is to list one subdir per line and
-	preferably in alphabetic order.
-
 8 Kbuild Variables
 ==================
 
-- 
2.17.1


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

* Re: [PATCH] kbuild: remove wrong documentation about mandatory-y
  2020-02-19  1:15 [PATCH] kbuild: remove wrong documentation about mandatory-y Masahiro Yamada
@ 2020-02-20 16:28 ` Masahiro Yamada
  0 siblings, 0 replies; 2+ messages in thread
From: Masahiro Yamada @ 2020-02-20 16:28 UTC (permalink / raw)
  To: Linux Kbuild mailing list
  Cc: Jonathan Corbet, Michal Marek, open list:DOCUMENTATION,
	Linux Kernel Mailing List

On Wed, Feb 19, 2020 at 10:15 AM Masahiro Yamada <masahiroy@kernel.org> wrote:
>
> This sentence does not make sense in the section about mandatory-y.
>
> This seems to be a copy-paste mistake of commit fcc8487d477a ("uapi:
> export all headers under uapi directories").
>
> The correct description would be "The convention is to list one
> mandatory-y per line ...".
>
> I just removed it instead of fixing it. If such information is needed,
> it could be commented in include/asm-generic/Kbuild and
> include/uapi/asm-generic/Kbuild.
>
> Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>


Applied to linux-kbuild.

> ---
>
>  Documentation/kbuild/makefiles.rst | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/Documentation/kbuild/makefiles.rst b/Documentation/kbuild/makefiles.rst
> index 0e0eb2c8da7d..4018ad7c7a11 100644
> --- a/Documentation/kbuild/makefiles.rst
> +++ b/Documentation/kbuild/makefiles.rst
> @@ -1379,9 +1379,6 @@ See subsequent chapter for the syntax of the Kbuild file.
>         in arch/$(ARCH)/include/(uapi/)/asm, Kbuild will automatically generate
>         a wrapper of the asm-generic one.
>
> -       The convention is to list one subdir per line and
> -       preferably in alphabetic order.
> -
>  8 Kbuild Variables
>  ==================
>
> --
> 2.17.1
>


-- 
Best Regards
Masahiro Yamada

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

end of thread, other threads:[~2020-02-20 16:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-19  1:15 [PATCH] kbuild: remove wrong documentation about mandatory-y Masahiro Yamada
2020-02-20 16:28 ` Masahiro Yamada

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).