Re: [PATCH 1/2] kbuild: rename multi-used-* to multi-obj-*

[Masahiro Yamada]

On Sat, Mar 6, 2021 at 3:11 PM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote:
>
> I think multi-obj-* is clearer, and more consisten with real-obj-*.
>
> Rename as follows:
>
>   multi-used-y  ->  multi-obj-y
>   multi-used-m  ->  multi-obj-m
>   multi-used    ->  multi-obj-ym
>
> Signed-off-by: Masahiro Yamada <masahiroy@xxxxxxxxxx>

Fixed the typo "consisten" to "consistent",
and applied to linux-kbuild.




> ---
>
>  scripts/Makefile.build |  6 +++---
>  scripts/Makefile.lib   | 10 +++++-----
>  2 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/scripts/Makefile.build b/scripts/Makefile.build
> index 1b6094a13034..56cf8eb475cf 100644
> --- a/scripts/Makefile.build
> +++ b/scripts/Makefile.build
> @@ -444,11 +444,11 @@ quiet_cmd_link_multi-m = LD [M]  $@
>        cmd_link_multi-m = $(LD) $(ld_flags) -r -o $@ $(filter %.o,$^)
>  endif
>
> -$(multi-used-m): FORCE
> +$(multi-obj-m): FORCE
>         $(call if_changed,link_multi-m)
> -$(call multi_depend, $(multi-used-m), .o, -objs -y -m)
> +$(call multi_depend, $(multi-obj-m), .o, -objs -y -m)
>
> -targets += $(multi-used-m)
> +targets += $(multi-obj-m)
>  targets := $(filter-out $(PHONY), $(targets))
>
>  # Add intermediate targets:
> diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib
> index eee59184de64..e60be0bddda2 100644
> --- a/scripts/Makefile.lib
> +++ b/scripts/Makefile.lib
> @@ -48,9 +48,9 @@ endif
>  suffix-search = $(foreach s,$(2),$($(1:.o=$s)))
>  # If $(foo-objs), $(foo-y), $(foo-m), or $(foo-) exists, foo.o is a composite object
>  multi-search = $(sort $(foreach m,$(1), $(if $(strip $(call suffix-search,$(m),$(2) -)), $(m))))
> -multi-used-y := $(call multi-search,$(obj-y),-objs -y)
> -multi-used-m := $(call multi-search,$(obj-m),-objs -y -m)
> -multi-used   := $(multi-used-y) $(multi-used-m)
> +multi-obj-y := $(call multi-search,$(obj-y),-objs -y)
> +multi-obj-m := $(call multi-search,$(obj-m),-objs -y -m)
> +multi-obj-ym := $(multi-obj-y) $(multi-obj-m)
>
>  # Replace multi-part objects by their individual parts,
>  # including built-in.a from subdirectories
> @@ -92,12 +92,12 @@ obj-m               := $(addprefix $(obj)/,$(obj-m))
>  lib-y          := $(addprefix $(obj)/,$(lib-y))
>  real-obj-y     := $(addprefix $(obj)/,$(real-obj-y))
>  real-obj-m     := $(addprefix $(obj)/,$(real-obj-m))
> -multi-used-m   := $(addprefix $(obj)/,$(multi-used-m))
> +multi-obj-m    := $(addprefix $(obj)/, $(multi-obj-m))
>  subdir-ym      := $(addprefix $(obj)/,$(subdir-ym))
>
>  # Finds the multi-part object the current object will be linked into.
>  # If the object belongs to two or more multi-part objects, list them all.
> -modname-multi = $(sort $(foreach m,$(multi-used),\
> +modname-multi = $(sort $(foreach m,$(multi-obj-ym),\
>                 $(if $(filter $*.o, $($(m:.o=-objs)) $($(m:.o=-y)) $($(m:.o=-m))),$(m:.o=))))
>
>  __modname = $(if $(modname-multi),$(modname-multi),$(basetarget))
> --
> 2.27.0
>


-- 
Best Regards
Masahiro Yamada