diff mbox series

[v2] docs: kbuild: remove description of KBUILD_LDS_MODULE

Message ID 20230222104719.16374-1-sangmoon.kim@samsung.com (mailing list archive)
State New, archived
Headers show
Series [v2] docs: kbuild: remove description of KBUILD_LDS_MODULE | expand

Commit Message

Sangmoon Kim Feb. 22, 2023, 10:47 a.m. UTC
Commit 596b0474d3d9 ("kbuild: preprocess module linker script")
removes KBUILD_LDS_MODULE, yet the variable is still mentioned in
kbuild documentation. Remove the reference to the now-nonexistent
variable.

Signed-off-by: Sangmoon Kim <sangmoon.kim@samsung.com>
---
Thank you for your review.

Changes in v2:
 - Update the commit message.

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

Comments

Nathan Chancellor Feb. 22, 2023, 4:21 p.m. UTC | #1
On Wed, Feb 22, 2023 at 07:47:19PM +0900, Sangmoon Kim wrote:
> Commit 596b0474d3d9 ("kbuild: preprocess module linker script")
> removes KBUILD_LDS_MODULE, yet the variable is still mentioned in
> kbuild documentation. Remove the reference to the now-nonexistent
> variable.
> 
> Signed-off-by: Sangmoon Kim <sangmoon.kim@samsung.com>

Reviewed-by: Nathan Chancellor <nathan@kernel.org>

> ---
> Thank you for your review.
> 
> Changes in v2:
>  - Update the commit message.
> 
>  Documentation/kbuild/makefiles.rst | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/Documentation/kbuild/makefiles.rst b/Documentation/kbuild/makefiles.rst
> index 38bc74eaa547..468e7830c1c3 100644
> --- a/Documentation/kbuild/makefiles.rst
> +++ b/Documentation/kbuild/makefiles.rst
> @@ -1250,11 +1250,6 @@ When kbuild executes, the following steps are followed (roughly):
>  
>  	The linker script with full path. Assigned by the top-level Makefile.
>  
> -    KBUILD_LDS_MODULE
> -
> -	The module linker script with full path. Assigned by the top-level
> -	Makefile and additionally by the arch Makefile.
> -
>      KBUILD_VMLINUX_OBJS
>  
>  	All object files for vmlinux. They are linked to vmlinux in the same
> -- 
> 2.17.1
>
Masahiro Yamada Feb. 25, 2023, 8:41 a.m. UTC | #2
On Wed, Feb 22, 2023 at 7:49 PM Sangmoon Kim <sangmoon.kim@samsung.com> wrote:
>
> Commit 596b0474d3d9 ("kbuild: preprocess module linker script")
> removes KBUILD_LDS_MODULE, yet the variable is still mentioned in
> kbuild documentation. Remove the reference to the now-nonexistent
> variable.
>
> Signed-off-by: Sangmoon Kim <sangmoon.kim@samsung.com>
> ---
> Thank you for your review.
>
> Changes in v2:
>  - Update the commit message.
>
>  Documentation/kbuild/makefiles.rst | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/Documentation/kbuild/makefiles.rst b/Documentation/kbuild/makefiles.rst
> index 38bc74eaa547..468e7830c1c3 100644
> --- a/Documentation/kbuild/makefiles.rst
> +++ b/Documentation/kbuild/makefiles.rst
> @@ -1250,11 +1250,6 @@ When kbuild executes, the following steps are followed (roughly):
>
>         The linker script with full path. Assigned by the top-level Makefile.
>
> -    KBUILD_LDS_MODULE
> -
> -       The module linker script with full path. Assigned by the top-level
> -       Makefile and additionally by the arch Makefile.
> -
>      KBUILD_VMLINUX_OBJS
>
>         All object files for vmlinux. They are linked to vmlinux in the same
> --
> 2.17.1
>

Applied to linux-kbuild.
Thanks.
diff mbox series

Patch

diff --git a/Documentation/kbuild/makefiles.rst b/Documentation/kbuild/makefiles.rst
index 38bc74eaa547..468e7830c1c3 100644
--- a/Documentation/kbuild/makefiles.rst
+++ b/Documentation/kbuild/makefiles.rst
@@ -1250,11 +1250,6 @@  When kbuild executes, the following steps are followed (roughly):
 
 	The linker script with full path. Assigned by the top-level Makefile.
 
-    KBUILD_LDS_MODULE
-
-	The module linker script with full path. Assigned by the top-level
-	Makefile and additionally by the arch Makefile.
-
     KBUILD_VMLINUX_OBJS
 
 	All object files for vmlinux. They are linked to vmlinux in the same