Re: [PATCH] ARC: migrate to the generic rule for built-in DTB

On Tue, Dec 24, 2024 at 7:06 AM Vineet Gupta wrote: > > On 12/21/24 16:15, Masahiro Yamada wrote: > > Commit 654102df2ac2 ("kbuild: add generic support for built-in boot > > DTBs") introduced generic support for built-in DTBs. > > > > Select GENERIC_BUILTIN

[PATCH] ARC: migrate to the generic rule for built-in DTB

N_DTB_NAME. Now, "nsim_700" is the default value for CONFIG_BUILTIN_DTB_NAME, rather than a fallback in case it is empty. Signed-off-by: Masahiro Yamada --- arch/arc/Kconfig | 7 --- arch/arc/Makefile | 3 --- arch/a

Re: [PATCH 04/15] kbuild: add generic support for built-in boot DTBs

On Thu, Sep 5, 2024 at 11:17 PM Rob Herring wrote: > > On Thu, Sep 05, 2024 at 08:47:40AM +0900, Masahiro Yamada wrote: > > Some architectures embed boot DTBs in vmlinux. A potential issue for > > these architectures is a race condition during parallel builds because > > K

Re: [PATCH 04/15] kbuild: add generic support for built-in boot DTBs

On Thu, Sep 5, 2024 at 11:17 PM Rob Herring wrote: > > On Thu, Sep 05, 2024 at 08:47:40AM +0900, Masahiro Yamada wrote: > > Some architectures embed boot DTBs in vmlinux. A potential issue for > > these architectures is a race condition during parallel builds because > > K

Re: [PATCH 14/15] kbuild: rename CONFIG_GENERIC_BUILTIN_DTB to CONFIG_BUILTIN_DTB

On Thu, Sep 5, 2024 at 11:38 PM Rob Herring wrote: > > On Thu, Sep 05, 2024 at 08:47:50AM +0900, Masahiro Yamada wrote: > > Now that all architectures have migrated to the generic built-in > > DTB support, the GENERIC_ prefix is no longer necessary. > > > >

[PATCH 14/15] kbuild: rename CONFIG_GENERIC_BUILTIN_DTB to CONFIG_BUILTIN_DTB

Now that all architectures have migrated to the generic built-in DTB support, the GENERIC_ prefix is no longer necessary. Signed-off-by: Masahiro Yamada --- Makefile | 2 +- arch/arc/Kconfig | 2 +- arch/loongarch/Kconfig | 1

[PATCH 15/15] kbuild: use .init.rodata section unconditionally for cmd_wrap_S_dtb

Boot DTBs are now wrapped and compiled in scripts/Makefile.vmlinux. The cmd_wrap_S_dtb rule in scripts/Makefile.dtbs is now only used for generic purposes, so the .init.rodata section should be used unconditionally. Signed-off-by: Masahiro Yamada --- scripts/Makefile.dtbs | 4 +--- 1 file

[PATCH 11/15] nios2: migrate to the generic rule for built-in DTB

Select GENERIC_BUILTIN_DTB when built-in DTB support is enabled. To keep consistency across architectures, this commit also renames CONFIG_NIOS2_DTB_SOURCE_BOOL to CONFIG_BUILTIN_DTB, and CONFIG_NIOS2_DTB_SOURCE to CONFIG_BUILTIN_DTB_NAME. Signed-off-by: Masahiro Yamada --- arch/nios2/Kbuild

[PATCH 13/15] microblaze: migrate to the generic rule for built-in DTB

by include/asm-generic/vmlinux.lds.h. For safety, arch/microblaze/boot/dts/system.dtb is still placed in the __fdt_blob section, but removing the MicroBlaze-specific section should be considered. Signed-off-by: Masahiro Yamada --- arch/microblaze/Kbuild| 1 - arch/microblaze

[PATCH 10/15] xtensa: migrate to the generic rule for built-in DTB

Select GENERIC_BUILTIN_DTB to use the generic rule to support built-in DTB. To keep consistency across architectures, this commit also renames CONFIG_BUILTIN_DTB_SOURCE to CONFIG_BUILTIN_DTB_NAME. Signed-off-by: Masahiro Yamada --- arch/xtensa/Kbuild | 2 +- arch

[PATCH 12/15] sh: migrate to the generic rule for built-in DTB

Select GENERIC_BUILTIN_DTB when built-in DTB support is enabled. To keep consistency across architectures, this commit also renames CONFIG_USE_BUILTIN_DTB to CONFIG_BUILTIN_DTB, and CONFIG_BUILTIN_DTB_SOURCE to CONFIG_BUILTIN_DTB_NAME. Signed-off-by: Masahiro Yamada --- arch/sh/Kbuild

[PATCH 08/15] ARC: migrate to the generic rule for built-in DTB

Select GENERIC_BUILTIN_DTB to use the generic rule to support built-in DTB. To keep consistency across architectures, this commit also renames CONFIG_ARC_BUILTIN_DTB_NAME to CONFIG_BUILTIN_DTB_NAME. Signed-off-by: Masahiro Yamada --- arch/arc/Kconfig | 7

[PATCH 09/15] openrisc: migrate to the generic rule for built-in DTB

Select GENERIC_BUILTIN_DTB to use the generic rule to support built-in DTB. To keep consistency across architectures, this commit also renames CONFIG_OPENRISC_BUILTIN_DTB_NAME to CONFIG_BUILTIN_DTB_NAME. Signed-off-by: Masahiro Yamada --- arch/openrisc/Kbuild | 1 - arch

[PATCH 06/15] riscv: migrate to the generic rule for built-in DTB

Select GENERIC_BUILTIN_DTB when built-in DTB support is enabled. To keep consistency across architectures, this commit also renames CONFIG_BUILTIN_DTB_SOURCE to CONFIG_BUILTIN_DTB_NAME. Signed-off-by: Masahiro Yamada --- arch/riscv/Kbuild | 1 - arch/riscv/Kconfig

[PATCH 07/15] LoongArch: migrate to the generic rule for built-in DTB

Select GENERIC_BUILTIN_DTB when built-in DTB support is enabled. Signed-off-by: Masahiro Yamada --- arch/loongarch/Kbuild| 1 - arch/loongarch/Kconfig | 1 + arch/loongarch/boot/dts/Makefile | 2 -- 3 files changed, 1 insertion(+), 3 deletions(-) diff --git a/arch

[PATCH 05/15] MIPS: migrate to generic rule for built-in DTBs

Select GENERIC_BUILTIN_DTB and BUILTIN_DTB_ALL when built-in DTB support is enabled. DTBs compiled under arch/mips/boot/dts/ will be wrapped and compiled in scripts/Makefile.vmlinux. Signed-off-by: Masahiro Yamada --- arch/mips/Kconfig | 2 ++ arch/mips/Makefile

[PATCH 04/15] kbuild: add generic support for built-in boot DTBs

e to arch/*/boot/dts/) - CONFIG_BUILTIN_DTB_ALL If this is enabled, all DTB files compiled under arch/*/boot/dts/ are embedded into vmlinux. Only used by MIPS. Signed-off-by: Masahiro Yamada --- Makefile | 7 ++- drivers/of/Kconfig | 6 ++ scripts/M

[PATCH 03/15] kbuild: move non-boot builtin DTBs to .init.rodata section

odata under the arch/ directory, and .init.rodata section otherwise. This will be refactored later. Signed-off-by: Masahiro Yamada --- drivers/of/unittest.c | 2 +- scripts/Makefile.dtbs | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/of/unittest.c b/drivers

[PATCH 02/15] kbuild: split device tree build rules into scripts/Makefile.dtbs

scripts/Makefile.lib is included not only from scripts/Makefile.build but also from scripts/Makefile.{modfinal,package,vmlinux,vmlinux_o}, where DT build rules are not required. Split the DT build rules out to scripts/Makefile.dtbs, and include it only when necessary. Signed-off-by: Masahiro

[PATCH 00/15] kbuild: refactor DTB build rules, introduce a generic built-in boot DTB support

01 and 02 are kbuild cleanups. 03 and 04 parepare to wrap boot DTBs in scripts/Makefile.vmlinux. My plan is to 05-13 to each arch ML in the next cycle, but they are included in this patch set so that reviewers can understand what will happen in the future. Masahiro Yamada (15): kbuild: add

[PATCH 01/15] kbuild: add intermediate targets for Flex/Bison in scripts/Makefile.host

Flex and Bison are used only for host programs. Move their intermediate target processing from scripts/Makefile.build to scripts/Makefile.host. Signed-off-by: Masahiro Yamada --- scripts/Makefile.build | 35 --- scripts/Makefile.host | 5 + 2 files changed

Re: [PATCH] ARC: update the help message for CONFIG_ARC_BUILTIN_DTB_NAME

On Tue, Sep 3, 2024 at 9:39 AM Vineet Gupta wrote: > > > > On 8/31/24 23:50, Masahiro Yamada wrote: > > On Sun, Sep 1, 2024 at 8:02 AM Vineet Gupta wrote: > >> > >> > >> On 8/31/24 03:13, Masahiro Yamada wrote: > >>> On Sat, Aug 31, 202

Re: [PATCH] ARC: update the help message for CONFIG_ARC_BUILTIN_DTB_NAME

On Sun, Sep 1, 2024 at 8:02 AM Vineet Gupta wrote: > > > > On 8/31/24 03:13, Masahiro Yamada wrote: > > On Sat, Aug 31, 2024 at 7:10 PM Masahiro Yamada > > wrote: > >> Commit abe11ddea1d7 ("ARC: [plat-arcfpga]: Enabling DeviceTree for > >> Angel

Re: [PATCH] ARC: update the help message for CONFIG_ARC_BUILTIN_DTB_NAME

On Sat, Aug 31, 2024 at 7:10 PM Masahiro Yamada wrote: > > Commit abe11ddea1d7 ("ARC: [plat-arcfpga]: Enabling DeviceTree for > Angel4 board") changed the default built-in DTB from "skeleton" to > "angel4". > > Commit fd1557923b2e ("ARC: [plat

[PATCH] ARC: update the help message for CONFIG_ARC_BUILTIN_DTB_NAME

Commit abe11ddea1d7 ("ARC: [plat-arcfpga]: Enabling DeviceTree for Angel4 board") changed the default built-in DTB from "skeleton" to "angel4". Commit fd1557923b2e ("ARC: [plat_arcfpga]->[plat_sim]") changed it from "angel4" to "nsim_70

Re: [PATCH 01/17] syscalls: add generic scripts/syscall.tbl

n readv sys_readv 66 common writev sys_writev -- Best Regards Masahiro Yamada ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc

Re: [PATCH 03/17] um: don't generate asm/bpf_perf_event.h

* SPDX-License-Identifier: GPL-2.0 */ > + > +#include > -- > 2.39.2 > I guess this is a step backward. Technically, kernel-space asm/*.h files are allowed to wrap UAPI . There is no reason why we ban generic-y for doing this, whereas check-in source is allowed. -- Best Regards

Re: [PATCH 02/17] csky: drop asm/gpio.h wrapper

On Thu, Jul 4, 2024 at 11:36 PM Arnd Bergmann wrote: > > From: Arnd Bergmann > > The asm/gpio.h header is gone now that all architectures just use > gpiolib, and so the redirect is no longer valid. > > Signed-off-by: Arnd Bergmann > --- Reviewed-by: Masahiro Yamada

Re: [PATCH] kbuild: builddeb: always make modules_install, to install modules.builtin*

. What I meant in my previous reply was to remove "if is_enabled CONFIG_MODULES; then" Anyway, I did it by myself in a follow-up patch. https://lore.kernel.org/linux-kbuild/20230625181623.2473308-1-masahi...@kernel.org/T/#u -- Best Regards Masahiro Yamada

[PATCH] kbuild: make modules_install copy modules.builtin(.modinfo)

Link: https://lore.kernel.org/lkml/36a4014c73a52af27d930d3ca31d362b60f4461c.1686356364.git.j...@joshtriplett.org/ Reported-by: Josh Triplett Signed-off-by: Masahiro Yamada --- Makefile | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/Makefile b/Ma

[PATCH v9 08/11] kbuild: implement CONFIG_TRIM_UNUSED_KSYMS without recursion

mization where the chain reaction of trimming comes into play because: - CONFIG_LTO_CLANG cannot remove any unused symbols - CONFIG_LD_DEAD_CODE_DATA_ELIMINATION is enabled only for vmlinux, but not modules If deeper trimming is required, we need to revisit this, but I guess that is unlikely

[PATCH v9 11/11] linux/export.h: rename 'sec' argument to 'license'

Now, EXPORT_SYMBOL() is populated in two stages. In the first stage, all of EXPORT_SYMBOL/EXPORT_SYMBOL_GPL go into the same section, '.export_symbol'. 'sec' does not make sense any more. Rename it to 'license'. Signed-off-by: Masahiro Yamada Reviewed-by: Nick

[PATCH v9 09/11] modpost: merge two similar section mismatch warnings

are similar. Merge them. Signed-off-by: Masahiro Yamada Reviewed-by: Nick Desaulniers --- scripts/mod/modpost.c | 18 +++--- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 50d950e904a4..0adda14451fa 100644 --- a/scri

[PATCH v9 07/11] modpost: use null string instead of NULL pointer for default namespace

is saves 1 byte memory allocated for "", but loses the readability. In kernel-space, we strive to save memory, but modpost is a userspace tool used to build the kernel. On modern systems, such small piece of memory is not a big deal. Handle the namespace string as is. Signed-off-by:

[PATCH v9 10/11] modpost: show offset from symbol for section mismatch warnings

Currently, modpost only shows the symbol names and section names, so it repeats the same message if there are multiple relocations in the same symbol. It is common the relocation spans across multiple instructions. It is better to show the offset from the symbol. Signed-off-by: Masahiro Yamada

[PATCH v9 05/11] modpost: check static EXPORT_SYMBOL* by modpost again

d foo(void) {} EXPORT_SYMBOL(foo); foo2.c: void foo(void) {} Then, modpost can catch it correctly. MODPOST Module.symvers ERROR: modpost: vmlinux: local symbol 'foo' was exported Signed-off-by: Masahiro Yamada Reviewed-by: Nick Desaulniers --- (no changes since v6) Changes

[PATCH v9 02/11] modpost: pass struct module pointer to check_section_mismatch()

The next commit will use it. Signed-off-by: Masahiro Yamada Reviewed-by: Nick Desaulniers --- scripts/mod/modpost.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 8decf04633bc..403ba4d923f5

[PATCH v9 06/11] modpost: squash sym_update_namespace() into sym_add_exported()

Pass a set of the name, license, and namespace to sym_add_exported(). sym_update_namespace() is unneeded. Signed-off-by: Masahiro Yamada Reviewed-by: Nick Desaulniers --- scripts/mod/modpost.c | 27 --- 1 file changed, 4 insertions(+), 23 deletions(-) diff --git a

[PATCH v9 04/11] ia64,export.h: replace EXPORT_DATA_SYMBOL* with EXPORT_SYMBOL*

With the previous refactoring, you can always use EXPORT_SYMBOL*. Replace two instances in ia64, then remove EXPORT_DATA_SYMBOL*. Signed-off-by: Masahiro Yamada Reviewed-by: Nick Desaulniers --- arch/ia64/kernel/head.S | 2 +- arch/ia64/kernel/ivt.S | 2 +- include/asm-generic

[PATCH v9 03/11] kbuild: generate KSYMTAB entries by modpost

in one-pass algorithm (by a later commit) When CONFIG_TRIM_UNUSED_KSYMS is enabled, Kbuild recursively traverses the directory tree to determine which EXPORT_SYMBOL to trim. If an EXPORT_SYMBOL turns out to be unused by anyone, Kbuild begins the second traverse, where some source files are

[PATCH v9 01/11] ARC: define ASM_NL and __ALIGN(_STR) outside #ifdef __ASSEMBLY__ guard

ASM_NL (also __ALIGN and __ALIGN_STR) out of the #ifdef. Fixes: 9df62f054406 ("arch: use ASM_NL instead of ';' for assembler new line character in the macro") Fixes: 8d92e992a785 ("ARC: define __ALIGN_STR and __ALIGN symbols for ARC") Signed-off-by: Masahiro Yamada ---

[PATCH v9 00/11] Unify and , remove EXPORT_DATA_SYMBOL(), faster TRIM_UNUSED_KSYMS

/CAK7LNARBiOywrMLbR=9n35sk19u0qm3xcpy7d1wqv-eyb4w...@mail.gmail.com/T/#t v4: https://lore.kernel.org/linux-kbuild/CAK7LNASDzy9RERN6+q6WgR4ROYZQue=sbqgbcoyuvepbyht...@mail.gmail.com/T/#t v3: https://lore.kernel.org/all/20220928063947.299333-1-masahi...@kernel.org/ Masahiro Yamada (11): ARC: define ASM_NL

Re: [PATCH] kbuild: drop support for CONFIG_CC_OPTIMIZE_FOR_PERFORMANCE_O3

. Provides stats on terms of increase in file size. > > > > Might be worth cleaning up the rest of the kernel of instances of -O3, > too. -O3 used to build lz4 and mips vdso, for instance. Might be a bit > of a digression, though This patch foc

Re: [PATCH] kbuild: drop support for CONFIG_CC_OPTIMIZE_FOR_PERFORMANCE_O3

vdk_hs38_smp_defconfig > @@ -4,7 +4,6 @@ CONFIG_HIGH_RES_TIMERS=y > CONFIG_IKCONFIG=y > CONFIG_IKCONFIG_PROC=y > CONFIG_BLK_DEV_INITRD=y > -CONFIG_CC_OPTIMIZE_FOR_PERFORMANCE_O3=y > CONFIG_EMBEDDED=y > CONFIG_PERF_EVENTS=y > # CONFIG_VM_EVENT_COUNTERS is not set &

[PATCH 2/2] kbuild: use more subdir- for visiting subdirectories while cleaning

take advantage of the parallel option (-j) for "make clean". I also cleaned up the comments. The "archdep" target does not exist. Signed-off-by: Masahiro Yamada --- Documentation/kbuild/makefiles.rst | 17 ++--- arch/alpha/Kbuild | 3 +++ arc

Re: [PATCH 4/7] arc: replace cc-option-yn uses with cc-option

+= $(call cc-option,$(tune-mcpu)) > >> > >> ? > > > > Yes, you'll need to pull up the source; the diff doesn't provide > > enough context. tune-mcpu is used in the body of the else branch > > hinted at by the diff. PTAL > > Ah, fair enough. The warning is a little unconventional but oh well :) > > Reviewed-by: Nathan Chancellor > > >> > >> If $(tune-mcpu) is empty or invalid, nothing will be added to cflags-y. > >> > >>>else > >>># The flag provided by 'CONFIG_ARC_TUNE_MCPU' option isn't known by > >>> this compiler > >>> > > > > > > Applied to linux-kbuild. -- Best Regards Masahiro Yamada ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc

[PATCH 2/3] trace: refactor TRACE_IRQFLAGS_SUPPORT in Kconfig

Make architectures select TRACE_IRQFLAGS_SUPPORT instead of having many defines. Signed-off-by: Masahiro Yamada --- arch/Kconfig | 3 +++ arch/arc/Kconfig | 4 +--- arch/arm/Kconfig | 5 + arch/arm64/Kconfig| 4 +--- arch/csky/Kconfig

Re: [PATCH 1/5] kbuild: require all architectures to have arch/$(SRCARCH)/Kbuild

On Wed, May 12, 2021 at 5:00 PM Masahiro Yamada wrote: > > arch/$(SRCARCH)/Kbuild is useful for Makefile cleanups because you can > use the obj-y syntax. > > Add an empty file if it is missing in arch/$(SRCARCH)/. > > Signed-off-by: Masahiro Yamada > --- Applied to linu

[PATCH 1/5] kbuild: require all architectures to have arch/$(SRCARCH)/Kbuild

arch/$(SRCARCH)/Kbuild is useful for Makefile cleanups because you can use the obj-y syntax. Add an empty file if it is missing in arch/$(SRCARCH)/. Signed-off-by: Masahiro Yamada --- Makefile | 2 +- arch/alpha/Kbuild | 1 + arch/arc/Makefile | 3 --- arch/arm

Re: [PATCH] kbuild: use ?= to assign CROSS_COMPILE by arch-Makefile

On Mon, Apr 12, 2021 at 5:15 PM Masahiro Yamada wrote: > > On Mon, Apr 12, 2021 at 4:44 PM Geert Uytterhoeven > wrote: > > > > Hi Yamada-san, > > > > On Sun, Apr 11, 2021 at 3:56 PM Masahiro Yamada > > wrote: > > > Use ?= operator to let arch/*/

Re: [PATCH] kbuild: use ?= to assign CROSS_COMPILE by arch-Makefile

On Mon, Apr 12, 2021 at 4:44 PM Geert Uytterhoeven wrote: > > Hi Yamada-san, > > On Sun, Apr 11, 2021 at 3:56 PM Masahiro Yamada wrote: > > Use ?= operator to let arch/*/Makefile to assign CROSS_COMPILE only > > when CROSS_COMPILE is undefined. > > > > This al

[PATCH] kbuild: use ?= to assign CROSS_COMPILE by arch-Makefile

been made empty via an environment variable as in 'export CROSS_COMPILE='. With this commit, arch-Makefle will respect the user's environment set-up, which seems to be a more correct behavior. Signed-off-by: Masahiro Yamada --- arch/arc/Makefile| 4 +--- arch/h8300/Makefile

Re: [PATCH 0/6] ARC: build: fix various issues in arc boot Makefile

On Wed, Nov 25, 2020 at 1:40 PM Vineet Gupta wrote: > > Hi Masahiro San, > > On 11/21/20 11:36 AM, Masahiro Yamada wrote: > > > > > > Masahiro Yamada (6): > >ARC: build: remove non-existing bootpImage from KBUILD_IMAGE > >ARC: build: add uIma

[PATCH 3/6] ARC: build: add boot_targets to PHONY

targets as PHONY to fix this. Signed-off-by: Masahiro Yamada --- arch/arc/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arc/Makefile b/arch/arc/Makefile index 61a41123ad4c..cf9da9aea12a 100644 --- a/arch/arc/Makefile +++ b/arch/arc/Makefile @@ -104,6 +104,7 @@ boot

[PATCH 4/6] ARC: build: move symlink creation to arch/arc/Makefile to avoid race

since $(obj)/uImage is a symlink. The symlink should be created in a phony target rule. I used $(kecho) instead of echo to suppress the message 'Image arch/arc/boot/uImage is ready' when the -s option is given. Signed-off-by: Masahiro Yamada --- arch/arc/Makefile | 13

[PATCH 6/6] ARC: build: use $(READELF) instead of hard-coded readelf

The top Makefile defines READELF as the readelf in the cross-toolchains. Use it rather than the host readelf. Signed-off-by: Masahiro Yamada --- arch/arc/boot/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arc/boot/Makefile b/arch/arc/boot/Makefile index

[PATCH 2/6] ARC: build: add uImage.lzma to the top-level target

able to be set. Signed-off-by: Masahiro Yamada --- arch/arc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arc/Makefile b/arch/arc/Makefile index acf99420e161..61a41123ad4c 100644 --- a/arch/arc/Makefile +++ b/arch/arc/Makefile @@ -102,7 +102,7 @@ libs-y

[PATCH 5/6] ARC: build: remove unneeded extra-y

Adding vmlinux.* to extra-y has no point because we expect they are built on demand while building uImage.* Add them to 'targets' is enough to include the corresponding .cmd file. Signed-off-by: Masahiro Yamada --- arch/arc/boot/Makefile | 7 +++ 1 file changed, 3 insert

[PATCH 0/6] ARC: build: fix various issues in arc boot Makefile

Masahiro Yamada (6): ARC: build: remove non-existing bootpImage from KBUILD_IMAGE ARC: build: add uImage.lzma to the top-level target ARC: build: add boot_targets to PHONY ARC: build: move symlink creation to arch/arc/Makefile to avoid race ARC: build: remove unneeded extra-y ARC

[PATCH 1/6] ARC: build: remove non-existing bootpImage from KBUILD_IMAGE

eaningless KBUILD_IMAGE assignment so it will fallback to the default vmlinux. With this change, you can build the deb package. I removed the 'bootpImage' target as well. At best, it provides 'make bootpImage' as an alias of 'make vmlinux', but I do not see much sense in d

Re: [PATCH] arc: remove #ifndef CONFIG_AS_CFI_SIGNAL_FRAME

On Mon, Apr 13, 2020 at 2:38 PM Vineet Gupta wrote: > > On 4/12/20 7:05 PM, Masahiro Yamada wrote: > > CONFIG_AS_CFI_SIGNAL_FRAME is never defined for ARC. > > > > Suggested-by: Nick Desaulniers > > Where, how ? I was working on various cleanups of

[PATCH] arc: remove #ifndef CONFIG_AS_CFI_SIGNAL_FRAME

CONFIG_AS_CFI_SIGNAL_FRAME is never defined for ARC. Suggested-by: Nick Desaulniers Signed-off-by: Masahiro Yamada --- arch/arc/kernel/unwind.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/arc/kernel/unwind.c b/arch/arc/kernel/unwind.c index 27ea64b1fa33..f87758a6851b 100644

[PATCH] arc: ptrace: hard-code "arc" instead of UTS_MACHINE

ARC uses the UTS_MACHINE defined in the top Makefile as follows: UTS_MACHINE := $(ARCH) We know it is "arc" when we are building the kernel for ARC. Hard-code user_regset_view::name, like many other architectures do. Signed-off-by: Masahiro Yamada --- arch/arc/kernel/Ma

Re: [PATCH] ARC: [plat-hsdk]: fix USB regression

h submitter should take care of affected defconfigs when (s)he drops select/imply. Also, this kind of mistake should be caught in the review process. -- Best Regards Masahiro Yamada ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org htt

Re: [PATCH] initramfs: restore default compression behaviour

On Mon, Mar 16, 2020 at 8:22 PM Eugeniy Paltsev wrote: > > Hi Masahiro, > > >From: Masahiro Yamada > >Sent: Wednesday, March 11, 2020 23:12 > >To: Eugeniy Paltsev > >Cc: Linux Kernel Mailing List; arcml; Vineet Gupta; Alexey Brodkin > >Subject: Re: [PATCH]

Re: [PATCH v2] initramfs: restore default compression behavior

; kernel, > + on those architectures that support this. However, not compressing > the > + initramfs may lead to slightly higher memory consumption during a > + short time at boot, while both the cpio image and the unpacked > + filesystem image will be present in memory simultaneously > + > endchoice > -- > 2.21.1 > -- Best Regards Masahiro Yamada ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc

Re: [PATCH] initramfs: restore default compression behaviour

default INITRAMFS_COMPRESSION_GZIP if RD_GZIP > depends on INITRAMFS_SOURCE != "" > help > This option allows you to decide by which algorithm the builtin > -- > 2.21.1 > -- Best Regards Masahiro Yamada ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc

Re: [PATCH 1/2] asm-generic: Make msi.h a mandatory include/asm header

On Thu, Oct 24, 2019 at 7:13 PM Michal Simek wrote: > > msi.h is generic for all architectures expect of x86 which has own version. Maybe a typo? "except" Anyway, the code looks good to me. Reviewed-by: Masahiro Yamada > Enabling MSI by including msi.h to architec

Re: [PATCH 2/3] kbuild, arc: add CONFIG_CC_OPTIMIZE_FOR_PERFORMANCE_O3 for ARC

On Sat, Aug 31, 2019 at 1:43 AM Vineet Gupta wrote: > > On 8/20/19 10:09 AM, Masahiro Yamada wrote: > > arch/arc/Makefile overrides -O2 with -O3. This is the only user of > > ARCH_CFLAGS. There is no user of ARCH_CPPFLAGS or ARCH_AFLAGS. > > My plan is to remove A

Re: [PATCH 2/3] kbuild, arc: add CONFIG_CC_OPTIMIZE_FOR_PERFORMANCE_O3 for ARC

Hi Vineet, On Wed, Aug 21, 2019 at 2:10 AM Masahiro Yamada wrote: > > arch/arc/Makefile overrides -O2 with -O3. This is the only user of > ARCH_CFLAGS. There is no user of ARCH_CPPFLAGS or ARCH_AFLAGS. > My plan is to remove ARCH_{CPP,A,C}FLAGS after refactoring the ARC > Makefile

[PATCH 3/3] kbuild: remove ARCH_{CPP,A,C}FLAGS

Signed-off-by: Masahiro Yamada --- Documentation/kbuild/makefiles.rst | 7 --- Makefile | 14 -- 2 files changed, 4 insertions(+), 17 deletions(-) diff --git a/Documentation/kbuild/makefiles.rst b/Documentation/kbuild/makefiles.rst index 36

[PATCH 2/3] kbuild,arc: add CONFIG_CC_OPTIMIZE_FOR_PERFORMANCE_O3 for ARC

it. Enabling it will be useful for compile-testing. This commit allows allmodconfig (, which defaults to -O2) to enable it. Add CONFIG_CC_OPTIMIZE_FOR_PERFORMANCE_O3=y to all the defconfig files in arch/arc/configs/ in order to keep the current config settings. Signed-off-by: Masahiro Yamada

[PATCH 1/3] init/Kconfig: rework help of CONFIG_CC_OPTIMIZE_FOR_SIZE

ord the help text to make it appropriate for the choice menu. Signed-off-by: Masahiro Yamada --- init/Kconfig | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/init/Kconfig b/init/Kconfig index bf971b5c707d..149efd82447f 100644 --- a/init/Kconfig +++ b/init/Kconfig @@ -1219,

Re: [TRIVIAL PATCH] of: per-file dtc preprocessor flags

On Sun, Jul 21, 2019 at 12:38 AM Eugeniy Paltsev wrote: > > Hi Masahiro, > > On Sat, 2019-07-20 at 11:40 +0900, Masahiro Yamada wrote: > > On Sat, Jul 20, 2019 at 4:28 AM Eugeniy Paltsev > > wrote: > > > As of today we are able to pass custom flags to

Re: [TRIVIAL PATCH] of: per-file dtc preprocessor flags

) \ > -undef -D__DTS__ > > +dtc_cpp_flags += $(DTC_CPP_FLAGS_$(basetarget)) > + > # Useful for describing the dependency of composite objects > # Usage: > # $(call multi_depend, multi_used_targets, suffix_t

[PATCH] fixup! kbuild: remove obj and src from the top Makefile

Merging today's kbuild tree would break arc, um, parisc. I just noticed it now. I will fix it soon for tomorrow's linux-next. If needed, this might be useful for today's linux-next. Signed-off-by: Masahiro Yamada --- arch/arc/Makefile| 2 +- arch/parisc/Makefile | 12 +++

Re: [PATCH 00/15] kbuild: refactor headers_install and support compile-test of UAPI headers

On Tue, Jun 4, 2019 at 7:15 PM Masahiro Yamada wrote: > > > Multiple people have suggested to compile-test UAPI headers. > > Currently, Kbuild provides simple sanity checks by headers_check > but they are not enough to catch bugs. > > The most recent patch I know is Davi

Re: [PATCH] ARC: build: Try to guess CROSS_COMPILE with cc-cross-prefix

Hi. On Tue, Jun 4, 2019 at 2:49 AM Alexey Brodkin wrote: > > Hi Vineet, > > > -Original Message- > > From: Vineet Gupta > > Sent: Monday, June 3, 2019 7:25 PM > > To: Alexey Brodkin ; > > linux-snps-arc@lists.infradead.org > > Cc:

[PATCH 1/2] linux/bits.h: make BIT(), GENMASK(), and friends available in assembly

alternative of BIT(). For [2], it is pretty easy to change BIT() etc. for use in assembly. This allows to replace _BUTUL() in kernel-space headers with BIT(). Signed-off-by: Masahiro Yamada --- include/linux/bits.h | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git

[PATCH 2/2] arch: replace _BITUL() in kernel-space headers with BIT()

Now that BIT() can be used from assembly code, we can safely replace _BITUL() with equivalent BIT(). UAPI headers are still required to use _BITUL(), but there is no more reason to use it in kernel headers. BIT() is shorter. Signed-off-by: Masahiro Yamada --- arch/arc/include/asm/pgtable.h

[PATCH v2] kbuild: use more portable 'command -v' for cc-cross-prefix

t confuse people. The stderr would not be polluted anyway, but it will provide extra safety, and is easy to understand. Tested on Make 3.81, 3.82, 4.0, 4.1, 4.2, 4.2.1 [1] http://pubs.opengroup.org/onlinepubs/9699919799/utilities/command.html Fixes: bd55f96fa9fc ("kbuild: refactor cc-cr

Re: [PATCH 00/15] kbuild: refactor headers_install and support compile-test of UAPI headers

On Tue, Jun 4, 2019 at 7:15 PM Masahiro Yamada wrote: > > > Multiple people have suggested to compile-test UAPI headers. > > Currently, Kbuild provides simple sanity checks by headers_check > but they are not enough to catch bugs. > > The most recent patch I know is Davi

Re: [PATCH] kbuild: use more portable 'command -v' for cc-cross-prefix

On Tue, Jun 4, 2019 at 6:01 PM David Laight wrote: > > From: Masahiro Yamada > > Sent: 04 June 2019 04:31 > ... > > > > > You could use: > > > > > $(shell sh -c "command -v $(c)gcc") > > > > > or maybe: > > > &

[PATCH 05/15] kbuild: add CONFIG_HEADERS_INSTALL and loosen the dependency of samples

e.o CC [M] samples/trace_events/trace-events-sample.o CC [M] samples/trace_printk/trace-printk.o AR samples/vfio-mdev/built-in.a AR samples/built-in.a Signed-off-by: Masahiro Yamada --- Makefile | 8 arch/arc/configs/tb10x_defconfig

[PATCH 00/15] kbuild: refactor headers_install and support compile-test of UAPI headers

de to $(INSTALL_HDR_PATH)/include. [3] Support compile-test of UAPI headers This is implemented in usr/include/Makefile Jani Nikula (1): kbuild: add support for ensuring headers are self-contained Masahiro Yamada (14): kbuild: remove headers_{install,check}_all kbuild: remove stale de

Re: [PATCH] kbuild: use more portable 'command -v' for cc-cross-prefix

On Mon, Jun 3, 2019 at 9:43 PM David Laight wrote: > > From: Masahiro Yamada > > Sent: 03 June 2019 12:45 > > On Mon, Jun 3, 2019 at 8:16 PM David Laight wrote: > > > > > > From: Masahiro Yamada > > > > Sent: 03 June 2019 11:49 > > > >

Re: [PATCH] kbuild: use more portable 'command -v' for cc-cross-prefix

On Mon, Jun 3, 2019 at 10:09 PM David Laight wrote: > > From: Masahiro Yamada > > Sent: 03 June 2019 12:38 > > Hi David, > > > > On Mon, Jun 3, 2019 at 8:14 PM David Laight wrote: > > > > > > From: Masahiro Yamada > > > > Sent: 03 Jun

Re: [PATCH] ARC: build: Try to guess CROSS_COMPILE with cc-cross-prefix

t/?id=fc2b47b55f17 > > > > A side note: even though "cc-cross-prefix" does its job it pollutes > > console with output of "which" for all the prefixes it didn't manage to find > > a matching cross-compiler for like that: > > | # ARCH=arc make de

Re: [PATCH] kbuild: use more portable 'command -v' for cc-cross-prefix

On Mon, Jun 3, 2019 at 8:16 PM David Laight wrote: > > From: Masahiro Yamada > > Sent: 03 June 2019 11:49 > > > > To print the pathname that will be used by shell in the current > > environment, 'command -v' is a standardized way. [1] > > > > &#

Re: [PATCH] kbuild: use more portable 'command -v' for cc-cross-prefix

Hi David, On Mon, Jun 3, 2019 at 8:14 PM David Laight wrote: > > From: Masahiro Yamada > > Sent: 03 June 2019 11:49 > > > > To print the pathname that will be used by shell in the current > > environment, 'command -v' is a standardized way. [1] > &g

[PATCH] kbuild: use more portable 'command -v' for cc-cross-prefix

rk the subshell, I added a shell special character '~'. We may be able to get rid of this workaround someday, but it is very far into the future. [1] http://pubs.opengroup.org/onlinepubs/9699919799/utilities/command.html Fixes: bd55f96fa9fc ("kbuild: refactor cc-cross-prefix implementation&

Re: [PATCH] ARC: build: Try to guess CROSS_COMPILE with cc-cross-prefix

Hi Alexey, On Mon, Jun 3, 2019 at 5:34 PM Alexey Brodkin wrote: > > Hi Masahiro-san, > > > -Original Message- > > From: Masahiro Yamada > > Sent: Monday, June 3, 2019 11:18 AM > > To: Alexey Brodkin > > Cc: arcml ; Linux Kernel Mailing List >

Re: [PATCH] ARC: build: Try to guess CROSS_COMPILE with cc-cross-prefix

Oh really? masahiro@pug:~$ which arc-linux-gcc /home/masahiro/tools/arc/bin/arc-linux-gcc masahiro@pug:~$ which dummy-linux-gcc masahiro@pug:~$ echo $? 1 When 'which' cannot find the given command, it does not print anything to stderr. Does it work differently on your machine?

[PATCH] clocksource: arc_timer: use BIT() instead of _BITUL()

This is in-kernel C code, so there is no reason to use _BITUL(). Replace it with equivalent BIT(). I added #include explicitly although it has been included by other headers eventually. Signed-off-by: Masahiro Yamada --- drivers/clocksource/arc_timer.c | 3 ++- 1 file changed, 2 insertions

Re: [PATCH] arc: remove redundant kernel-space generic-y

Hi Vineet, On Thu, Dec 20, 2018 at 4:52 AM Vineet Gupta wrote: > > On 12/19/18 7:16 AM, Masahiro Yamada wrote: > > Could you pick this up to your arc tree? > > Done, will push it in a day or so ! > > Thx, > -Vineet This was not sent in the previous MW. (Nor did I se

Re: [PATCH] arc: remove redundant kernel-space generic-y

Hi Vineet, On Mon, Dec 17, 2018 at 2:07 PM Vineet Gupta wrote: > > On 12/16/18 6:17 AM, Masahiro Yamada wrote: > > This commit removes redundant generic-y defines in > > arch/arc/include/asm/Kbuild. > > > > It is redundant to define generic-y when arch-specific imp

[PATCH] arc: remove redundant kernel-space generic-y

: Masahiro Yamada --- arch/arc/include/asm/Kbuild | 4 1 file changed, 4 deletions(-) diff --git a/arch/arc/include/asm/Kbuild b/arch/arc/include/asm/Kbuild index feed50c..caa2702 100644 --- a/arch/arc/include/asm/Kbuild +++ b/arch/arc/include/asm/Kbuild @@ -3,23 +3,19 @@ generic-y += bugs.h

[PATCH 3/3] arch: remove redundant generic-y defines

Now that Kbuild automatically creates asm-generic wrappers for missing mandatory headers, it is redundant to list the same headers in generic-y and mandatory-y. Suggested-by: Sam Ravnborg Signed-off-by: Masahiro Yamada --- arch/alpha/include/uapi/asm/Kbuild | 5 - arch/arc/include

Re: [PATCH v4 6/9] kbuild: consolidate Devicetree dtb build rules

be > dtc. > > This change enables support 'dtbs_install' on some arches which were > missing the target. > > Acked-by: Will Deacon > Acked-by: Paul Burton > Acked-by: Ley Foon Tan > Cc: Masahiro Yamada Please change this to Acked-by: Masahiro Yamada Thanks.

Re: [PATCH v3 6/9] kbuild: consolidate Devicetree dtb build rules

Hi Rob, 2018年10月1日(月) 22:26 Rob Herring : > > On Mon, Oct 1, 2018 at 12:49 AM Masahiro Yamada > wrote: > > > > Hi Rob, > > > > > > 2018年9月29日(土) 0:43 Rob Herring : > > > > > +# > > > --

  1   2   >