[PATCH v4 03/11] kconfig: remove unneeded directory generation from local*config

dconfig from local*config"), no sub-directory is required. Signed-off-by: Masahiro Yamada --- Changes in v4: None scripts/kconfig/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile index f3c0e62..96a1c3d 100644 --- a/scripts/

[PATCH v4 02/11] kconfig: split out useful helpers in confdata.c

Split out helpers: is_present() - check if the given path exists is_dir() - check if the given path exists and it is a directory make_parent_dir() - create the parent directories of the given path These helpers will be reused in later commits. Signed-off-by: Masahiro Yamada --- Changes in

[PATCH v4 11/11] kbuild: remove auto.conf from prerequisite of phony targets

f . I moved tristate.conf here because it is only included from scripts/Makefile.modbuiltin. Signed-off-by: Masahiro Yamada --- Changes in v4: None Makefile | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/Makefile b/Makefile index 0ec1c27..ceb9c1e 1006

[PATCH v4 07/11] kbuild: use 'include' directive to load auto.conf from top Makefile

om a pristine source tree: Makefile:605: include/config/auto.conf: No such file or directory The previous commit made sure include/config/auto.conf exists after the 'make *config' stage. Now, we can use the 'include' directive without showing the warning. Signed-off-by:

[PATCH 1/2] clk: uniphier: add NAND 200MHz clock

this. Signed-off-by: Masahiro Yamada --- drivers/clk/uniphier/clk-uniphier-sys.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/drivers/clk/uniphier/clk-uniphier-sys.c b/drivers/clk/uniphier/clk-uniphier-sys.c index 4f5ff9f..a582446 100644 --- a

[PATCH 2/2] clk: uniphier: add more USB3 PHY clocks

Add USB3 PHY clocks where missing. Use fixed-factor clocks for those without gating. For clarification, prefix clock names with 'ss' or 'hs'. Signed-off-by: Masahiro Yamada --- drivers/clk/uniphier/clk-uniphier-sys.c | 26 +- 1 file changed,

[PATCH 1/2] ARM: uniphier: dts: add more clocks to Denali NAND controller node

Catch up with the new binding of the Denali IP where three clocks, "nand", "nand_x", "ecc" are required. For UniPhier SoCs, the "nand_x" and "ecc" are tied up because they are both 200MHz. Signed-off-by: Masahiro Yamada --- arch/arm/boot

[PATCH 2/2] arm64: uniphier: dts: add more clocks to Denali NAND controller node

Catch up with the new binding of the Denali IP where three clocks, "nand", "nand_x", "ecc" are required. For UniPhier SoCs, the "nand_x" and "ecc" are tied up because they are both 200MHz. Signed-off-by: Masahiro Yamada --- arch/arm64/boot/dt

Re: [GIT PULL 3/3] ARM: uniphier: UniPhier SoC updates for v4.19

Hi Olof, 2018-07-22 6:24 GMT+09:00 Olof Johansson : > On Thu, Jul 19, 2018 at 08:05:03AM +0900, Masahiro Yamada wrote: >> Hi Arnd, Olof, >> >> Please pull UniPhier SoC changes for the v4.19 MW. >> >> Thanks! >> >> >> The following changes si

Re: [GIT PULL 3/3] ARM: uniphier: UniPhier SoC updates for v4.19

Hi Olof, 2018-07-22 8:24 GMT+09:00 Masahiro Yamada : > Hi Olof, > > 2018-07-22 6:24 GMT+09:00 Olof Johansson : >> On Thu, Jul 19, 2018 at 08:05:03AM +0900, Masahiro Yamada wrote: >>> Hi Arnd, Olof, >>> >>> Please pull UniPhier SoC changes for the

Re: [PATCH 1/2] mmc: tmio: rename files and CONFIG options to have the same prefix

2018-07-31 0:07 GMT+09:00 Ulf Hansson : > On 26 July 2018 at 05:28, Masahiro Yamada > wrote: >> As commit b6147490e6aa ("mmc: tmio: split core functionality, DMA and >> MFD glue") said, these MMC controllers use the IP from Panasonic. >> >> TMIO (Toshiba

Re: include architecture Kconfig files from top-level Kconfig v4

rch/arm/Kconfig > - various fixes from Randy > - various UML patches to be able to use the common Kconfig > -- > To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html Applied. Thanks! -- Best Regards Masahiro Yamada

Re: [PATCH 3/3] microblaze: add endianness options to LDFLAGS instead of LD

Hi Michal, Ping? 2018-07-03 10:22 GMT+09:00 Masahiro Yamada : > With the recent syntax extension, Kconfig is now able to evaluate the > compiler / toolchain capability. > > However, accumulating flags to 'LD' is not compatible with the way > it works; 'LD' mus

Re: [RFC][PATCH] kconfig: Add "m or y" and "y or m" answers for oldconfig

lar use case I have in mind is continuous integration > where you probably want to test build any new features. Currently > you would have to either blindly try both 'y' and 'm' answers > or parse the output to know which one will be accepted. > > Cc: Masahiro Yamada &g

Re: include architecture Kconfig files from top-level Kconfig v3

eviously never enabled because kernel/Kconfig.preempt was not included. Now, CONFIG_PREEMPT_COUNT can be enabled if it is select'ed by someone. I guess this change will be OK, but I'd like you and Randy to double-check it just in case. -- Best Regards Masahiro Yamada

[PATCH 2/2] kbuild: do not redirect the first prerequisite for filechk

. Each rule can explicitly adds it if necessary. Signed-off-by: Masahiro Yamada --- Makefile | 2 +- arch/s390/kernel/syscalls/Makefile | 6 +++--- arch/um/Makefile | 2 +- scripts/Kbuild.include | 2 +- scripts/Makefile.lib | 2 +-

[PATCH 1/2] ARM: at91: remove unused duplicated filechk_offsets

The filechk_offsets in arch/arm/mach-at91/Makefile is never used because it is always overridden by the equivalent one in scripts/Makefile.lib Signed-off-by: Masahiro Yamada --- I will queue this to my kbuild tree. arch/arm/mach-at91/Makefile | 25 - 1 file changed

[RESEND PATCH] drm/bridge/synopsys: remove commented-out flag in Makefile

Please do not comment out unneeded code. Remove. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/bridge/synopsys/Makefile | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/synopsys/Makefile b/drivers/gpu/drm/bridge/synopsys/Makefile index 5dad97d..3e1b1e3 100644

[PATCH] mmc: tmio: remove unneeded variable in tmio_mmc_start_command()

Pass TMIO_MASK_CMD to tmio_mmc_enable_mmc_irqs() directly, and remove the variable, irq_mask. Signed-off-by: Masahiro Yamada --- drivers/mmc/host/tmio_mmc_core.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/mmc/host/tmio_mmc_core.c b/drivers/mmc/host

Re: [PATCH v4 04/35] mtd: rawnand: denali: convert to nand_scan()

x_banks); > if (ret) > - goto free_buf; > + goto disable_irq; > > ret = mtd_device_register(mtd, NULL, 0); > if (ret) { > dev_err(denali->dev, "Failed to register MTD: %d\n", ret); > goto cleanup_nand; > } > + > return 0; > > cleanup_nand: > nand_cleanup(chip); > -free_buf: > - kfree(denali->buf); > disable_irq: > denali_disable_irq(denali); > > -- > 2.14.1 > > > __ > Linux MTD discussion mailing list > http://lists.infradead.org/mailman/listinfo/linux-mtd/ You need to remove kfree(denali->buf) from denali_remove(), right? void denali_remove(struct denali_nand_info *denali) { struct mtd_info *mtd = nand_to_mtd(&denali->nand); nand_release(mtd); kfree(denali->buf); < REMOVE !! denali_disable_irq(denali); } Otherwise, denali_remove() will free denali->buf twice because kfree(denali->buf) is called from denali_detach_chip(). -- Best Regards Masahiro Yamada

Re: [PATCH v4 04/35] mtd: rawnand: denali: convert to nand_scan()

2018-07-25 21:47 GMT+09:00 Miquel Raynal : > Hi Boris, > > Boris Brezillon wrote on Wed, 25 Jul 2018 > 11:51:16 +0200: > >> On Wed, 25 Jul 2018 18:42:44 +0900 >> Masahiro Yamada wrote: >> >> > >> > You need to remove kfree(denali->b

Re: [PATCH] scripts: Add Python 3 support to tracing/draw_functrace.py

gt; > import sys, re > @@ -123,7 +124,7 @@ def main(): > tree = tree.getParent(caller) > tree = tree.calls(callee, calltime) > > - print CallTree.ROOT > + print(CallTree.ROOT) > > if __name__ == "__main__": > main() > -- > 2.17.1 > -- Best Regards Masahiro Yamada

Re: [PATCH v4 00/11] kbuild/kconfig: do not update config during installation

2018-07-20 16:46 GMT+09:00 Masahiro Yamada : > > The main motivation of this patch series is to suppress the syncconfig > during running installation targets. > > V1 consisted of only two patches: > https://patchwork.kernel.org/patch/10468105/ > https://patchwork.kern

Re: [PATCH v3 05/12] kconfig: make syncconfig update .config regardless of sym_change_count

2018-07-09 20:39 GMT+09:00 Dirk Gouders : > Dirk Gouders writes: > >> Dirk Gouders writes: >> >>> Masahiro Yamada writes: >>> >>>> syncconfig updates the .config only when sym_change_count > 0, i.e. >>>> any change in config

[PATCH 6/6] kbuild: rename LDFLAGS to KBUILD_LDFLAGS

UILD_ prefixed variables for the internally appended options, so here is one more conversion to sanitize the naming convention. I did not touch Makefiles under tools/ since the tools build system is a different world. Signed-off-by: Masahiro Yamada --- Makefile | 6

[PATCH 4/6] kbuild: replace $(LDFLAGS) $(ldflags-y) with $(ld_flags)

$(LDFLAGS) $(ldflags-y) is equivalent to $(ld_flags). Signed-off-by: Masahiro Yamada --- scripts/Makefile.lib | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index 1bb594f..1915574 100644 --- a/scripts/Makefile.lib +++ b/scripts

[PATCH 0/6] kbuild: clean-up and rename LDFLAGS to KBUILD_LDFLAGS

This series renames LDFLAGS to KBUILD_LDFLAGS after some Makefile cleanups. Currently, the last patch does not apply to Linus' tree due to missing pre-requisite patches. I will resolve conflicts as needed. Masahiro Yamada (6): um: remove redundant 'export LDFLAGS' in arch/

[PATCH 2/6] ARM: vfp: clean-up arch/arm/vfp/Makefile

like this: core-$(CONFIG_VFP) += arch/arm/vfp/ ..., so $(CONFIG_VFP) is always evaluated to y in arch/arm/vfp/Makefile. Also, there is no point to use pseudo object, vfp.o, which never becomes a module. Add all objects directly to obj-y. Signed-off-by: Masahiro Yamada --- ar

[PATCH 3/6] kbuild: remove redundant LDFLAGS clearing in arch/*/Makefile

Since commit ce99d0bf312d ("kbuild: clear LDFLAGS in the top Makefile"), the top-level Makefile caters to this. Signed-off-by: Masahiro Yamada --- arch/arm/Makefile | 3 --- arch/openrisc/Makefile | 1 - arch/riscv/Makefile| 1 - 3 files changed, 5 deletions(-) diff --git

[PATCH 1/6] um: remove redundant 'export LDFLAGS' in arch/x86/Makefile.um

This is already exported by the top-level Makefile. Signed-off-by: Masahiro Yamada --- arch/x86/Makefile.um | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/x86/Makefile.um b/arch/x86/Makefile.um index 45af199..5296f8c 100644 --- a/arch/x86/Makefile.um +++ b/arch/x86/Makefile.um

[PATCH 5/6] kbuild: change ld_flags to contain LDFLAGS_$(@F)

Put $(LDFLAGS_$(@F)) into ld_flags so that $(LDFLAGS_pcap.o) and $(LDFLAGS_vde.o) in arch/um/drivers/Makefile are absorbed. Signed-off-by: Masahiro Yamada --- arch/um/drivers/Makefile | 4 ++-- scripts/Makefile.lib | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a

Re: [PATCH v3 00/12] kbuild/kconfig: do not update config during installation

2018-07-10 20:34 GMT+09:00 Dirk Gouders : > Masahiro Yamada writes: > >> The main motivation of this patch series is to suppress the syncconfig >> during running installation targets. >> >> V1 consisted of only two patches: >> https://patchwork.k

Re: [PATCHv2] treewide: Rename HOSTCFLAGS -> KBUILD_HOSTCFLAGS

_HOSTCFLAGS). To set flags that will take effect for all host programs created in that Makefile, use the variable HOST_EXTRACFLAGS. I will fold this. -- Best Regards Masahiro Yamada

[PATCH] kbuild: suppress warnings from 'getconf LFS_*'

upport for hostprogs") Reported-by: Chen Feng Cc: Uwe Kleine-König Signed-off-by: Masahiro Yamada --- Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 0f035ae..abc74f8 100644 --- a/Makefile +++ b/Makefile @@ -353,9 +353,9 @@ CONFIG

Re: [PATCH] kbuild: suppress warnings from 'getconf LFS_*'

2018-07-12 20:08 GMT+09:00 Uwe Kleine-König : > On Thu, Jul 12, 2018 at 07:38:36PM +0900, Masahiro Yamada wrote: >> Suppress warnings for systems that do not recognize LFS_*. >> >> getconf: no such configuration parameter `LFS_CFLAGS' >> getconf: no such conf

Re: [PATCH] scripts/tags.sh: add __ro_after_init

\ > + -I __initdata,__exitdata,__initconst,__ro_after_init\ > -I __initdata_memblock \ > -I __refdata,__attribute,__maybe_unused,__always_unused \ > -I __acquires,__releases,__deprecated \ > -- > 2.17.1 > -- Best Regards Masahiro Yamada

Re: [PATCH v3] tools/build/Build.include: fix # escaping in .cmd files for future Make

aracters: > > printf '\# cannot find fixdep (%s)\n' $(fixdep) > $(dot-target).cmd; \ > printf '\# using basic dep data\n\n' >> $(dot-target).cmd; \ > > This completes commit 9564a8cf (Kbuild: fix # escaping in .cmd files for > fut

Re: [PATCHv2 1/7] tools: build: Fixup host c flags

> JDIR = pmu-events/arch/$(SRCARCH) >> JSON = $(shell [ -d $(JDIR) ] &&\ >> -- >> 2.17.1 >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best Regards Masahiro Yamada

Re: [PATCHv2 2/7] tools: build: Use HOSTLDFLAGS with fixdep

TPUT)fixdep: $(OUTPUT)fixdep-in.o >> - $(QUIET_LINK)$(HOSTCC) $(LDFLAGS) -o $@ $< >> + $(QUIET_LINK)$(HOSTCC) $(HOSTLDFLAGS) -o $@ $< >> >> FORCE: >> >> -- >> 2.17.1 >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best Regards Masahiro Yamada

[GIT PULL] Kbuild fixes for v4.18-rc5

parameters Adam Borowski (1): scripts: teach extract-vmlinux about LZ4 and ZSTD Constantine Shulyupin (1): scripts/tags.sh: add __ro_after_init Laura Abbott (2): tools: build: Fixup host c flags tools: build: Use HOSTLDFLAGS with fixdep Masahiro

Re: [PATCH v3 05/12] kconfig: make syncconfig update .config regardless of sym_change_count

2018-07-12 20:32 GMT+09:00 Dirk Gouders : > Masahiro Yamada writes: > >> 2018-07-09 20:39 GMT+09:00 Dirk Gouders : >>> Dirk Gouders writes: >>> >>>> Dirk Gouders writes: >>>> >>>>> Masahiro Yamada writes: >>&g

Re: Mutli-directory module Makefiles

$(kbuild-file) > +-include $(kbuild-file) > > # If the save-* variables changed error out > ifeq ($(KBUILD_NOPEDANTIC),) > > Or we could create empty Makefiles in subdirectories... Is there a > better way of handling this? Which is preferred? -- Best Regards Masahiro Yamada

Re: [PATCH] kbuild: make Makefile|Kbuild in each directory optional

If the save-* variables changed error out > ifeq ($(KBUILD_NOPEDANTIC),) > -- > 2.15.1 > -- Best Regards Masahiro Yamada

[PATCH] gpio: zynq: remove unneeded (void *) casts in of_match_table

of_device_id::data is an opaque pointer. No explicit cast is needed. Signed-off-by: Masahiro Yamada --- drivers/gpio/gpio-zynq.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpio/gpio-zynq.c b/drivers/gpio/gpio-zynq.c index 6b4d10d..ed87c9a 100644 --- a

Re: [PATCH] kbuild: add ARM support for tar-pkg

016 +0100 arm: Use full path in KBUILD_IMAGE definition -- Best Regards Masahiro Yamada

Re: [PATCH] arm64: dts: uniphier: add input-delay properties to Cadence eMMC node

2017-03-31 13:20 GMT+09:00 Masahiro Yamada : > Since commit a04e2b383401 ("mmc: sdhci-cadence: Update PHY delay > configuration"), PHY parameters must be specified by DT. > > The hard-coded settings have been converted as follows: > - SDHCI_CDNS_PHY_DLY_SD_DEFAULT ->

Re: [PATCH] arm64: dts: uniphier: add cdns,phy-dll-delay-sdclk(-hsmmc) for eMMC

2017-04-04 11:04 GMT+09:00 Masahiro Yamada : > Adjust the PHY parameters for more stable access to the eMMC device. > Set the SDCLK output delay value to 21 (including HS200/400 modes). > > Signed-off-by: Masahiro Yamada > --- > > arch/arm64/boot/dts/socionext/uniphier-ld1

Re: [PATCH V5 6/7] ARM: uniphier: Use - instead of @ for DT OPP entries

des will never have a > "reg" property. > > Reported-by: Krzysztof Kozlowski > Reported-by: Masahiro Yamada > Suggested-by: Mark Rutland > Signed-off-by: Viresh Kumar > Acked-by: Masahiro Yamada > Acked-by: Rob Herring > --- Applied to linux-uniphier. Thanks! -- Best Regards Masahiro Yamada

[PATCH 2/2] arm64: dts: uniphier: fix simple-bus unit address format error

Compiling the UniPhier DT files with W=1, DTC warns like follows: Warning (simple_bus_reg): Node /soc/smpctrl@5980 simple-bus unit address format error, expected "59801000" Signed-off-by: Masahiro Yamada --- arch/arm64/boot/dts/socionext/uniphier-ld11.dtsi | 2 +- arch/arm6

[PATCH 1/2] ARM: dts: uniphier: fix simple-bus unit address format error

@1,1f0/uart@000b simple-bus unit address format error, expected "b" Warning (simple_bus_reg): Node /soc/smpctrl@5980 simple-bus unit address format error, expected "59801000" Signed-off-by: Masahiro Yamada --- arch/arm/boot/dts/uniphier-ld4.dtsi | 2

[PATCH 1/5] of: remove drivers/of/testcase-data from include search path for CPP

) introduced dynamic addition of testcase data, but it missed to delete this search path. Moreover, the directory drivers/of/testcase-data does not exist since commit 19fd74879a32 ("of/unittest: Rename selftest.c to unittest.c"). Signed-off-by: Masahiro Yamada --- scripts/Makefil

[PATCH 4/5] of: remove arch/$(SRCARCH)/boot/dts from include search path for CPP

path. Signed-off-by: Masahiro Yamada --- scripts/Makefile.lib | 1 - 1 file changed, 1 deletion(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index 011d464..9c8f38a 100644 --- a/scripts/Makefile.lib +++ b/scripts/Makefile.lib @@ -174,7 +174,6 @@ cpp_flags = -Wp,-MD,$(depfile

[PATCH 3/5] powerpc: dts: use #include "..." to include DT in the same directory

Most of DT files in PowerPC use #include "..." to make pre-processor include DT in the same directory, but we have 3 exceptional files that use #include <...> for that. They must be fixed to remove -I$(srctree)/arch/$(SRCARCH)/boot/dts path from dtc_cpp_flags. Signed-off-by:

[PATCH 5/5] of: add scripts/dtc/include-prefixes to search path for both CPP and DTC

hieve this, scripts/dtc/include-prefixes should be passed to both CPP and DTC. I refactored Makefile.lib a bit to not repeat the same path. Signed-off-by: Masahiro Yamada --- scripts/Makefile.lib | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/scripts/Makefile.lib b/scr

[PATCH 2/5] ARM: dts: use #include "..." to include DT in the same directory

Most of DT files in ARM use #include "..." to make pre-processor include DT in the same directory, but we have 3 exceptional files that use #include <...> for that. They must be fixed to remove -I$(srctree)/arch/$(SRCARCH)/boot/dts path from dtc_cpp_flags. Signed-off-by:

[PATCH 0/5] devicetree: cleanup and improve DT cross-arch reference

(needed for 4/5) 4/5: Remove arch/$(SRCARCH)/boot/dts include path 5/5: Make DT cross-arch reference available for /include/ directive Masahiro Yamada (5): of: remove drivers/of/testcase-data from include search path for CPP ARM: dts: use #include "..." to include DT in the same dire

Re: [PATCH 0/6] kconfig: Fix memory leaks during parsing

uments during parsing > Kconfig: Don't leak main menus during parsing > kconfig: Don't leak help strings during parsing > kconfig: Regenerate parser I applied 1-5. I left some comments on 5. -- Best Regards Masahiro Yamada

Re: [PATCH 5/6] kconfig: Don't leak help strings during parsing

u node with two help strings */ > + free(current_entry->help); > current_entry->help = $2; > }; I applied this, but maybe is it better to warning message? zconfprint("warning: ...") or zconf_error()? If you send v2, I will replace this. -- Best Regards Masahiro Yamada

Re: [PATCH] kbuild: move cc-option and cc-disable-warning after incl. arch Makefile

2018-01-11 1:21 GMT+09:00 Geert Uytterhoeven : > Hi Yamada-san, > > On Sun, Dec 10, 2017 at 6:28 AM, Masahiro Yamada > wrote: >> 2017-11-27 21:25 GMT+09:00 Geert Uytterhoeven : >>> On Mon, Nov 27, 2017 at 1:15 PM, Masahiro Yamada >>> wrote: >>>>

[PATCH] genksyms: drop *.hash.c from .gitignore

This is a left-over of commit bb3290d91695 ("Remove gperf usage from toolchain"). We do not generate a hash function any more. Signed-off-by: Masahiro Yamada --- scripts/genksyms/.gitignore | 1 - 1 file changed, 1 deletion(-) diff --git a/scripts/genksyms/.gitignore b/script

[PATCH v2 5/7] fixdep: move global variables to local variables of main()

. Signed-off-by: Masahiro Yamada --- Changes in v2: - Squash print_cmdline() into main() scripts/basic/fixdep.c | 42 -- 1 file changed, 16 insertions(+), 26 deletions(-) diff --git a/scripts/basic/fixdep.c b/scripts/basic/fixdep.c index dfba77b..d33b5a4

[PATCH v2 1/7] fixdep: remove unnecessary inclusion

was included for ntohl(), but it was removed by commit dee81e988674 ("fixdep: faster CONFIG_ search"). Signed-off-by: Masahiro Yamada --- Changes in v2: - newly added scripts/basic/fixdep.c | 1 - 1 file changed, 1 deletion(-) diff --git a/scripts/basic/fixdep.c b/scripts/basi

[PATCH v2 4/7] fixdep: remove unneeded memcpy() in parse_dep_file()

needed for PATH_MAX). Signed-off-by: Masahiro Yamada --- Changes in v2: None scripts/basic/fixdep.c | 22 +- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git a/scripts/basic/fixdep.c b/scripts/basic/fixdep.c index 9f9238e..dfba77b 100644 --- a/scripts/basic/fixdep.c

[PATCH v2 2/7] fixdep: use malloc() and read() to load dep_file to buffer

content into a malloc'ed buffer, we can insert '\0' after each token, and save memcpy(). (I will do this in the commit after next.) Signed-off-by: Masahiro Yamada --- Changes in v2: - Remove #include scripts/basic/fixdep.c | 56

[PATCH v2 7/7] fixdep: use existing helper to check modular CONFIG options

str_ends_with() tests if the given token ends with a particular string. Currently, it is used to check file paths without $(srctree). Actually, we have one more place where this helper is useful. Use it to check if CONFIG option ends with _MODULE. Signed-off-by: Masahiro Yamada --- Changes in

[PATCH v2 6/7] fixdep: refactor parse_dep_file()

ction argument. With this, we can swap the order of strrcmp() and "*p = \0;" Also, strrcmp() is an ambiguous function name. Flip the logic and rename it to str_ends_with(). I added a new helper is_ignored_file() - this returns 1 if the token represents a file that should be ignored.

[PATCH v2 3/7] fixdep: factor out common code for reading files

fer for an empty file. strstr() will immediately return NULL, and this is what we expect. On the other hand, an empty dep_file should be treated as an error. In this case, parse_dep_file() will error out with "no targets found" and it is a correct error message. Signed-off-by: Masahiro

[PATCH 3/3] kconfig: make conf_unsaved a local variable of conf_read()

conf_unsaved is initialized by conf_read_simple(), but it is possible to move it to conf_read() so that it can be a local variable. Signed-off-by: Masahiro Yamada --- scripts/kconfig/confdata.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/kconfig/confdata.c b

[PATCH 1/3] kconfig: make input_mode static

Sparse reports: warning: symbol 'input_mode' was not declared. Should it be static? Signed-off-by: Masahiro Yamada --- scripts/kconfig/conf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c index 8b9cdf4..31c0

[PATCH 2/3] kconfig: make xfgets() really static

Sparse reports: warning: symbol 'xfgets' was not declared. Should it be static? It is declared as static, but it is missing in the definition part. Move the definition up and remove the forward declaration. Signed-off-by: Masahiro Yamada --- scripts/kconfig/conf.c | 17 +++---

Re: [PATCH] kconfig: Document automatic submenu creation code

> scripts/kconfig/menu.c | 38 ++ > 1 file changed, 38 insertions(+) Applied to linux-kbuild/kconfig. Thanks! -- Best Regards Masahiro Yamada

Re: [PATCH 0/3] kconfig: Fix expression memory leaks

menu creation mem leak > kconfig: Fix expr_free() E_NOT leak > kconfig: Fix choice symbol expression leak > Applied to linux-kbuild/kconfig. Thanks! -- Best Regards Masahiro Yamada

Re: [PATCH] kconfig: Document important expression functions

is prevents memory errors. > > No functional changes. Only comments added. > > Signed-off-by: Ulf Magnusson Applied to linux-kbuild/kconfig. Thanks! -- Best Regards Masahiro Yamada

Re: [PATCH v2] kconfig: Remove menu_end_entry()

cause > unnecessary merge conflicts. Regenerate it manually instead: > > $ make REGENERATE_PARSERS=1 conf > Applied to linux-kbuild/kconfig. Thanks! With this, I think I picked up all the patches you posted in the last October. (let me know if I am missing something.) Thanks a lot for all of your contribution! (and also Kconfiglib, I used in U-Boot) -- Best Regards Masahiro Yamada

Re: [PATCH 1/2] kconfig: use bool instead of boolean for type definition attributes, again

2017-12-16 0:38 GMT+09:00 Masahiro Yamada : > Commit 6341e62b212a ("kconfig: use bool instead of boolean for type > definition attributes") did treewide replacement of 'boolean', and > also mentioned the keyword 'boolean' would be dropped later on. >

Re: [PATCH 2/2] kconfig: drop 'boolean' keyword

2017-12-16 0:38 GMT+09:00 Masahiro Yamada : > No more users of this keyword. Drop it according to the notice by > commit 6341e62b212a ("kconfig: use bool instead of boolean for type > definition attributes"). > > Signed-off-by: Masahiro Yamada > --- > Applied to l

[PATCH 2/2] kbuild: remove unnecessary LEX_PREFIX and YACC_PREFIX

Kconfig was the only user of these. With Kconfig converted to use the default 'yy' prefix, we do not need them any more. Signed-off-by: Masahiro Yamada --- scripts/Makefile.lib | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/scripts/Makefile.lib

[PATCH 1/2] kconfig: use default 'yy' prefix for lexer and parser

default 'yy' prefix. Signed-off-by: Masahiro Yamada --- scripts/kconfig/Makefile | 3 --- scripts/kconfig/zconf.l | 16 scripts/kconfig/zconf.y | 22 +++--- 3 files changed, 19 insertions(+), 22 deletions(-) diff --git a/scripts/kconfig/Makefile

Re: [PATCH v2 09/22] mmc: tmio: use mmc_can_gpio_cd() instead of checking TMIO_MMC_USE_GPIO_CD

Hi Ulf, 2018-01-02 21:56 GMT+09:00 Wolfram Sang : > On Sat, Nov 25, 2017 at 01:24:44AM +0900, Masahiro Yamada wrote: >> To use a GPIO line for card detection, TMIO_MMC_USE_GPIO_CD is set >> by a legacy board (arch/sh/boards/mach-ecovec24). >> >> For DT platforms, t

[GIT PULL] Kbuild fixes for v4.15-rc8

architectures that setup CROSS_COMPILE in their arch Makefile - fix Kconfig rational operators for bool / tristate - drop a gperf-generated file from .gitignore Masahiro Yamada (2): kbuild: move cc-option and cc-disable-warning

Re: [PATCH] genksyms: drop *.hash.c from .gitignore

2018-01-11 18:28 GMT+09:00 Masahiro Yamada : > This is a left-over of commit bb3290d91695 ("Remove gperf usage from > toolchain"). > > We do not generate a hash function any more. > > Signed-off-by: Masahiro Yamada > --- > Applied to linux-kbuild/fixes. -- Best Regards Masahiro Yamada

Re: [PATCH] kconfig: Warn if there is more than one help text

lf Magnusson > --- Applied to linux-kbuild/kconfig replacing "kconfig: Don't leak help strings during parsing" Thanks! -- Best Regards Masahiro Yamada

Re: [PATCH 00/11] Remove blank help texts from Kconfig files and add warning

; the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best Regards Masahiro Yamada

Re: [PATCH 01/10] net/sched: kconfig: Remove empty help texts

gnusson > > Applied. > -- > To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html I had applied the whole series into linux-kbui

Re: [PATCH] Coccinelle: coccicheck: fix typo

2018-02-01 18:20 GMT+09:00 Julia Lawall : > Correct spelling of "coccinelle". > > Signed-off-by: Julia Lawall > > --- Applied to linux-kbuild/misc. Thanks! -- Best Regards Masahiro Yamada

[GIT PULL 1/3] Kbuild updates for v4.16

() Masahiro Yamada (8): fixdep: remove unnecessary inclusion fixdep: use malloc() and read() to load dep_file to buffer fixdep: factor out common code for reading files fixdep: remove unneeded memcpy() in parse_dep_file() fixdep: move global variables to local variables

[GIT PULL 2/3] Kbuild misc updates for v4.16

: coccicheck: fix typo Jérémy Lefaure (1): Coccinelle: array_size: report even if include is missing Masahiro Yamada (1): kbuild: pkg: make out-of-tree rpm/deb-pkg build immediately fail Paolo Pisati (1): scripts/package: snap-pkg target .gitignore

[GIT PULL 3/3] Kconfig updates for v4.16

ig: Remove silentoldconfig from help and docs; fix kconfig/conf's help Masahiro Yamada (12): kconfig: display recursive dependency resolution hint just once kbuild: add LEX and YACC variables kbuild: prepare to remove C files pre-generated by flex and bison kconfig: generat

Re: [PATCH 00/11] Remove blank help texts from Kconfig files and add warning

2018-02-01 5:46 GMT+09:00 Randy Dunlap : > On 01/31/2018 07:27 AM, Masahiro Yamada wrote: >> 2018-01-31 18:34 GMT+09:00 Ulf Magnusson : >>> This patchset first removes all blank help texts from Kconfig files for all >>> arches, and then makes Kconfig print a w

Re: [PATCH 02/11] video: fbdev: kconfig: Remove blank help text

d because I need this to suppress warning messages introduced by 11/11. I am planning to send a PR for this series. -- Best Regards Masahiro Yamada

Re: [PATCH 03/11] mmc: kconfig: Remove blank help text

2018-01-31 18:34 GMT+09:00 Ulf Magnusson : > Blank help texts are probably either a typo, a Kconfig misunderstanding, > or some kind of half-committing to adding a help text (in which case a > TODO comment would be clearer, if the help text really can't be added > right away). > > Best to remove th

Re: [PATCH 04/11] Staging: rtl8192u: kconfig: Remove blank help text

2018-01-31 18:34 GMT+09:00 Ulf Magnusson : > Blank help texts are probably either a typo, a Kconfig misunderstanding, > or some kind of half-committing to adding a help text (in which case a > TODO comment would be clearer, if the help text really can't be added > right away). > > Best to remove th

Re: [PATCH 05/11] Staging: rtl8192e: kconfig: Remove blank help text

gt; > Best to remove them, IMO. > > Signed-off-by: Ulf Magnusson > --- FYI. I picked up this patch to kbuild because I need this to suppress warning messages introduced by 11/11. I am planning to send a PR for this series next week. -- Best Regards Masahiro Yamada

Re: [PATCH 06/11] lib/Kconfig.debug: Remove blank help text

2018-01-31 18:34 GMT+09:00 Ulf Magnusson : > Blank help texts are probably either a typo, a Kconfig misunderstanding, > or some kind of half-committing to adding a help text (in which case a > TODO comment would be clearer, if the help text really can't be added > right away). > > Best to remove th

Re: [PATCH 08/11] MIPS: kconfig: Remove blank help text

gt; > Best to remove them, IMO. > > Signed-off-by: Ulf Magnusson > --- FYI. I picked up this patch to kbuild because I need this to suppress warning messages introduced by 11/11. I am planning to send a PR for this series next week. -- Best Regards Masahiro Yamada

Re: [PATCH 10/11] nios2: kconfig: Remove blank help text

2018-01-31 18:34 GMT+09:00 Ulf Magnusson : > Blank help texts are probably either a typo, a Kconfig misunderstanding, > or some kind of half-committing to adding a help text (in which case a > TODO comment would be clearer, if the help text really can't be added > right away). > > Best to remove th

Re: [PATCH 09/11] arm: vt8500: kconfig: Remove blank help text

2018-01-31 18:34 GMT+09:00 Ulf Magnusson : > Blank help texts are probably either a typo, a Kconfig misunderstanding, > or some kind of half-committing to adding a help text (in which case a > TODO comment would be clearer, if the help text really can't be added > right away). > > Best to remove th

Re: [PATCH] kconfig: Clarify choice dependency propagation

symbols" mean "choice values"? The "choice" itself is a symbol with NULL name, so I'd like to clarify it. -- Best Regards Masahiro Yamada

Re: [PATCH] kconfig: Clarify choice dependency propagation

Hi Ulf, 2018-01-19 1:58 GMT+09:00 Ulf Magnusson : > On Thu, Jan 18, 2018 at 5:47 PM, Masahiro Yamada > wrote: >> 2018-01-14 23:12 GMT+09:00 Ulf Magnusson : >>> It's easy to miss that choices are special-cased to pass on their mode >>> as the parent depend

Re: [PATCH] mmc: tmio: hide unused tmio_mmc_clk_disable/tmio_mmc_clk_enable functions

allers. > > Fixes: b21fc294387e ("mmc: tmio: move clk_enable/disable out of > tmio_mmc_host_probe()") > Signed-off-by: Arnd Bergmann > --- Good catch, thanks! Reviewed-by: Masahiro Yamada -- Best Regards Masahiro Yamada

<    1   2   3   4   5   6   7   8   9   10   >