diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2018-03-19 05:01:26 -0400 |
---|---|---|
committer | Masahiro Yamada <yamada.masahiro@socionext.com> | 2018-03-25 13:01:25 -0400 |
commit | fe852ac200311bcbbb0a8c84096e4cd44ea623d9 (patch) | |
tree | 185dcea25b35ff7856f108d042e5b028389c61f0 | |
parent | c96a294eb6972742ed9d55f8d4afa7e0051c0942 (diff) |
kbuild: simplify modname calculation
modname can be calculated much more simply. If modname-multi is
empty, it is a single-used object. So, modname = $(basetarget).
Otherwise, modname = $(modname-multi).
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Reviewed-by: Cao jin <caoj.fnst@cn.fujitsu.com>
-rw-r--r-- | scripts/Makefile.build | 12 | ||||
-rw-r--r-- | scripts/Makefile.lib | 7 |
2 files changed, 1 insertions, 18 deletions
diff --git a/scripts/Makefile.build b/scripts/Makefile.build index 7cd2f4a6c2ac..f5858f609210 100644 --- a/scripts/Makefile.build +++ b/scripts/Makefile.build | |||
@@ -131,17 +131,7 @@ $(real-objs-m:.o=.lst): quiet_modtag := [M] | |||
131 | 131 | ||
132 | $(obj-m) : quiet_modtag := [M] | 132 | $(obj-m) : quiet_modtag := [M] |
133 | 133 | ||
134 | # Default for not multi-part modules | 134 | modname = $(if $(modname-multi),$(modname-multi),$(basetarget)) |
135 | modname = $(basetarget) | ||
136 | |||
137 | $(multi-objs-m) : modname = $(modname-multi) | ||
138 | $(multi-objs-m:.o=.i) : modname = $(modname-multi) | ||
139 | $(multi-objs-m:.o=.s) : modname = $(modname-multi) | ||
140 | $(multi-objs-m:.o=.lst) : modname = $(modname-multi) | ||
141 | $(multi-objs-y) : modname = $(modname-multi) | ||
142 | $(multi-objs-y:.o=.i) : modname = $(modname-multi) | ||
143 | $(multi-objs-y:.o=.s) : modname = $(modname-multi) | ||
144 | $(multi-objs-y:.o=.lst) : modname = $(modname-multi) | ||
145 | 135 | ||
146 | quiet_cmd_cc_s_c = CC $(quiet_modtag) $@ | 136 | quiet_cmd_cc_s_c = CC $(quiet_modtag) $@ |
147 | cmd_cc_s_c = $(CC) $(c_flags) $(DISABLE_LTO) -fverbose-asm -S -o $@ $< | 137 | cmd_cc_s_c = $(CC) $(c_flags) $(DISABLE_LTO) -fverbose-asm -S -o $@ $< |
diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index f855c2353b42..b5fa80a07c26 100644 --- a/scripts/Makefile.lib +++ b/scripts/Makefile.lib | |||
@@ -47,11 +47,6 @@ multi-used-m := $(sort $(foreach m,$(obj-m), $(if $(strip $($(m:.o=-objs)) $($(m | |||
47 | multi-used := $(multi-used-y) $(multi-used-m) | 47 | multi-used := $(multi-used-y) $(multi-used-m) |
48 | single-used-m := $(sort $(filter-out $(multi-used-m),$(obj-m))) | 48 | single-used-m := $(sort $(filter-out $(multi-used-m),$(obj-m))) |
49 | 49 | ||
50 | # Build list of the parts of our composite objects, our composite | ||
51 | # objects depend on those (obviously) | ||
52 | multi-objs-y := $(foreach m, $(multi-used-y), $($(m:.o=-objs)) $($(m:.o=-y))) | ||
53 | multi-objs-m := $(foreach m, $(multi-used-m), $($(m:.o=-objs)) $($(m:.o=-y)) $($(m:.o=-m))) | ||
54 | |||
55 | # $(subdir-obj-y) is the list of objects in $(obj-y) which uses dir/ to | 50 | # $(subdir-obj-y) is the list of objects in $(obj-y) which uses dir/ to |
56 | # tell kbuild to descend | 51 | # tell kbuild to descend |
57 | subdir-obj-y := $(filter %/built-in.a, $(obj-y)) | 52 | subdir-obj-y := $(filter %/built-in.a, $(obj-y)) |
@@ -80,8 +75,6 @@ real-objs-m := $(addprefix $(obj)/,$(real-objs-m)) | |||
80 | single-used-m := $(addprefix $(obj)/,$(single-used-m)) | 75 | single-used-m := $(addprefix $(obj)/,$(single-used-m)) |
81 | multi-used-y := $(addprefix $(obj)/,$(multi-used-y)) | 76 | multi-used-y := $(addprefix $(obj)/,$(multi-used-y)) |
82 | multi-used-m := $(addprefix $(obj)/,$(multi-used-m)) | 77 | multi-used-m := $(addprefix $(obj)/,$(multi-used-m)) |
83 | multi-objs-y := $(addprefix $(obj)/,$(multi-objs-y)) | ||
84 | multi-objs-m := $(addprefix $(obj)/,$(multi-objs-m)) | ||
85 | subdir-ym := $(addprefix $(obj)/,$(subdir-ym)) | 78 | subdir-ym := $(addprefix $(obj)/,$(subdir-ym)) |
86 | 79 | ||
87 | # These flags are needed for modversions and compiling, so we define them here | 80 | # These flags are needed for modversions and compiling, so we define them here |