Re: [PATCH] Makefile: enable dochelp run from main make level

2017-10-17 Thread Masahiro Yamada
2017-10-11 10:06 GMT+09:00 Shuah Khan : > On 10/10/2017 06:58 PM, Randy Dunlap wrote: >> On 10/10/17 17:54, Masahiro Yamada wrote: >>> 2017-10-11 5:09 GMT+09:00 Shuah Khan : >>>> On 10/07/2017 05:24 AM, Masahiro Yamada wrote: >>>>> 2017-10-03 8:44 G

Re: [GIT PULL (RESEND)] ARM: uniphier: UniPhier fixes for v4.12

2017-05-25 Thread Masahiro Yamada
Hi Olof, 2017-05-26 8:59 GMT+09:00 Olof Johansson : > On Mon, May 22, 2017 at 11:03:49AM +0900, Masahiro Yamada wrote: >> Hi Arnd, Olof, >> >> Last week, I thought I found a problem in my pull request, >> but it turned out my misunderstanding. >> >> So, m

[PATCH] arm64: dts: uniphier: reserve more memory for LD11/LD20

2017-05-26 Thread Masahiro Yamada
Reserve enough space below the kernel base. The assumed address map is: 8000 - 80ff : for IPP 8100 - 81ff : for ARM secure 8200 - : for Linux Signed-off-by: Masahiro Yamada --- arch/arm64/boot/dts/socionext/uniphier-ld11.dtsi | 2 +- arch/arm64/boot/dts

[PATCH] arm64: dts: uniphier: reserve more memory for LD11/LD20

2017-05-26 Thread Masahiro Yamada
Reserve enough space below the kernel base. The assumed address map is: 8000 - 80ff : for IPP 8100 - 81ff : for ARM secure 8200 - : for Linux Signed-off-by: Masahiro Yamada --- arch/arm64/boot/dts/socionext/uniphier-ld11.dtsi | 2 +- arch/arm64/boot/dts

Re: [PATCH] x86, build: Improve the isolinux searching of isoimage generation

2017-10-31 Thread Masahiro Yamada
ot/bzImage arch/x86/boot/isoimage/linux > > Could we make it something more streamlined and similar to the rest of the > build > as well, like: > > GEN arch/x86/boot/bzImage > GEN arch/x86/boot/isoimage > GEN arch/x86/boot/isoimage/linux > > I.e. only mention the new files built, with an appropriate prefix. > > I've Cc:-ed the kbuild maintainers, maybe they have a better suggestion > instead of > the 'GEN' abbreviation? > Generally, the abbreviation is the tool that has processed the target, but if you do not find an appropriate one, 'GEN' is fine. -- Best Regards Masahiro Yamada

Re: [PATCH 1/2] usb: ehci-platform: use reset array API

2017-11-01 Thread Masahiro Yamada
Hi Alan, 2017-11-02 0:32 GMT+09:00 Alan Stern : > On Mon, 30 Oct 2017, Masahiro Yamada wrote: > >> Generic drivers like this need to control arbitrary numbers of reset >> lines. Instead of hard-coding the maximum number of resets, use the >> reset array API. It can

[PATCH v2 2/2] usb: ohci-platform: use reset array API

2017-11-01 Thread Masahiro Yamada
Generic drivers like this need to control arbitrary number of reset lines. Instead of hard-coding the maximum number of resets, use the reset array API. It can manage a bunch of resets behind the scene. Signed-off-by: Masahiro Yamada --- Changes in v2: - Goto err_put_clks when it fails to

[PATCH v2 1/2] usb: ehci-platform: use reset array API

2017-11-01 Thread Masahiro Yamada
Generic drivers like this need to control arbitrary number of reset lines. Instead of hard-coding the maximum number of resets, use the reset array API. It can manage a bunch of resets behind the scene. Signed-off-by: Masahiro Yamada --- Changes in v2: - Goto err_put_clks when it fails to

Re: [PATCH] sh: remove pointless select of KBUILD_DEFCONFIG

2017-11-01 Thread Masahiro Yamada
SH developers, No comment? Looks like the SH maintainers are no more picking up any patches. If nobody takes care of this patch, I will apply it to Kbuild tree. 2017-10-04 18:33 GMT+09:00 Masahiro Yamada : > KBUILD_DEFCONFIG := shx3_defconfig > > is never used in a sen

Re: linux-next: build failure after merge of the gpio tree

2017-11-01 Thread Masahiro Yamada
struct irq_data *data, bool early) > struct uniphier_gpio_priv *priv = domain->host_data; > struct gpio_chip *chip = &priv->chip; > > gpiochip_lock_as_irq(chip, data->hwirq + UNIPHIER_GPIO_IRQ_OFFSET); > + return 0; > } > > static void uniphier_gpio_irq_domain_deactivate(struct irq_domain *domain, > -- > 2.14.1 > > -- > Cheers, > Stephen Rothwell -- Best Regards Masahiro Yamada

Re: [PATCH] sh: remove pointless select of KBUILD_DEFCONFIG

2017-11-02 Thread Masahiro Yamada
Hi Geert, 2017-11-02 18:18 GMT+09:00 Geert Uytterhoeven : > Hi Yamada-san, > > On Wed, Oct 4, 2017 at 11:33 AM, Masahiro Yamada > wrote: >> KBUILD_DEFCONFIG := shx3_defconfig >> >> is never used in a sensible way. KBUILD_DEFCONFIG specifies the >> defconfi

Re: [PATCH] sh: remove pointless select of KBUILD_DEFCONFIG

2017-11-02 Thread Masahiro Yamada
2017-11-02 18:26 GMT+09:00 Masahiro Yamada : > Hi Geert, > > 2017-11-02 18:18 GMT+09:00 Geert Uytterhoeven : >> Hi Yamada-san, >> >> On Wed, Oct 4, 2017 at 11:33 AM, Masahiro Yamada >> wrote: >>> KBUILD_DEFCONFIG := shx3_defconfig >>> >

Re: [GIT PULL] License cleanup: add SPDX license identifiers to some kernel files

2017-11-02 Thread Masahiro Yamada
g at the top line is preferred, should existing files be fixed? Some projects already adopted SPDX, and the tag in the copyright block looks nice (at least to me)... https://github.com/u-boot/u-boot/blob/master/common/board_f.c https://github.com/ARM-software/arm-trusted-firmware/blob/master/bl31/bl31_main.c -- Best Regards Masahiro Yamada

[PATCH] sh: select KBUILD_DEFCONFIG depending on ARCH

2017-11-02 Thread Masahiro Yamada
RCH, like arch/sparc/Makefile. Signed-off-by: Masahiro Yamada --- arch/sh/Makefile | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/arch/sh/Makefile b/arch/sh/Makefile index 280bbff..65300193 100644 --- a/arch/sh/Makefile +++ b/arch/sh/Makefile @@ -15,6 +15,12 @@ ifne

Re: [PATCH net-next v3 2/2] net: ethernet: socionext: add AVE ethernet driver

2017-10-24 Thread Masahiro Yamada
atic dma_addr_t ave_dma_map(struct net_device *ndev, struct ave_desc *desc, > + void *ptr, size_t len, > + enum dma_data_direction dir) > +{ > + dma_addr_t paddr; > + > + paddr = dma_map_single(ndev->dev.parent, ptr, len, dir); > + if (unlikely(dma_mapping_error(ndev->dev.parent, paddr))) { > + paddr = (dma_addr_t)-ENOMEM; Yuk! Re-write the code. > + } else { > + desc->skbs_dma = paddr; > + desc->skbs_dmalen = len; > + } > + > + return paddr; > +} -- Best Regards Masahiro Yamada

Re: [PATCH] kbuild: fix dependency of dtbs targets

2017-10-24 Thread Masahiro Yamada
Hi. 2017-10-10 0:05 GMT+09:00 Russell King - ARM Linux : > On Wed, Oct 04, 2017 at 01:27:20PM +0900, Masahiro Yamada wrote: >> The target "dtbs" should depend on "scripts" because it needs to >> build dtc. The "prepare" target is unneeded here. >

Re: [PATCH v2 3/5] kbuild: Use relative path for $(objtree)

2017-10-25 Thread Masahiro Yamada
> make -C /scratch/geert/build/linux-uml-amd64 \ > KBUILD_SRC=/scratch/geert/linux/linux \ > KBUILD_EXTMOD="" -f /scratch/geert/linux/linux/Makefile \ > archheaders > ... > > Unfortunately reverting that commit doesn't help, as that causes the > failure that > commit was supposed to fix: > > /scratch/geert/linux/linux/arch/x86/um/user-offsets.c:19:29: fatal > error: asm/syscalls_64.h: No such file or directory > #include > ^ > compilation terminated. > /scratch/geert/linux/linux/scripts/Makefile.build:154: recipe for > target 'arch/x86/um/user-offsets.s' failed > > Do you have any clue? Thanks! > > Gr{oetje,eeting}s, -- Best Regards Masahiro Yamada

Re: [PATCH v2 0/2] mtd: nand: wait for tWHR, and fix the setup_data_interface of Denali

2017-10-25 Thread Masahiro Yamada
Hi Marc, 2017-10-19 23:58 GMT+09:00 Marc Gonzalez : > On 13/10/2017 10:34, Masahiro Yamada wrote: > >> 2017-10-04 20:05, Marc Gonzalez wrote: >> >>> On 29/09/2017 16:33, Masahiro Yamada wrote: >>> >>>> tango_nand.c is the only driver that sets NAND_

Re: [PATCH] Coccinelle: make DEBUG_FILE option more useful

2017-10-25 Thread Masahiro Yamada
DEBUG_FILE="/dev/null" > +fi > + > if [ "$COCCI" = "" ] ; then > for f in `find $srctree/scripts/coccinelle/ -name '*.cocci' -type f | > sort`; do > coccinelle $f > -- Best Regards Masahiro Yamada

[PATCH] coccinelle: grep Options and Requires fields more precisely

2017-10-25 Thread Masahiro Yamada
pectation is fragile. Fix the .cocci file. I removed "-rc20" because ld-version.sh cannot handle it. Make the coccicheck script to see exact patterns for "Options:" and "Requires:" in order to avoid accidental matching to what just happens to appear in comment line

[PATCH] coccinelle: fix verbose message about .cocci file being run

2017-10-25 Thread Masahiro Yamada
ly without sed. Signed-off-by: Masahiro Yamada --- scripts/coccicheck | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/coccicheck b/scripts/coccicheck index 1bfa2d2..9d18662 100755 --- a/scripts/coccicheck +++ b/scripts/coccicheck @@ -186,7 +186,7 @@ coccinelle () {

[RFC PATCH] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-10-25 Thread Masahiro Yamada
ned-off-by: Masahiro Yamada --- scripts/coccicheck | 3 +++ 1 file changed, 3 insertions(+) diff --git a/scripts/coccicheck b/scripts/coccicheck index 9d18662..b74c789 100755 --- a/scripts/coccicheck +++ b/scripts/coccicheck @@ -69,6 +69,9 @@ if [ "$C" = "1" -o "$C"

[PATCH] x86/build: beautify build log of syscall headers

2017-10-26 Thread Masahiro Yamada
/x86/include/generated/uapi/asm/unistd_64.h SYSHDR arch/x86/include/generated/uapi/asm/unistd_x32.h Signed-off-by: Masahiro Yamada --- arch/x86/entry/syscalls/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/entry/syscalls/Makefile b/arch/x86/entry

Re: [PATCH RFC] kbuild: fixes in Makefile.lib

2017-11-12 Thread Masahiro Yamada
sing ')'. Stop. Please test your patch before submitting. Assuming the build error will be fixed, this change is correct. Also, modname-multi should be fixed. modname-multi = $(sort $(foreach m,$(multi-used),\ $(if $(filter $(subst $(obj)/,,$*.o), $($(m:.o=-objs)) $($(m:.o=-y)) $($(m:.o=-m))),$(m:.o= Thanks. -- Best Regards Masahiro Yamada

Re: [PATCH] coccinelle: orplus: reodganize to improve performance

2017-11-12 Thread Masahiro Yamada
; -| > -*c1@i1 +@p c > -) > +* c1 +@p c2 > > -@script:python depends on org@ > -p << s.p; > +@script:python depends on s && org@ > +p << r.p; > @@ > > cocci.print_main("sum of probable bitmasks, consider |",p) > > -@script:python depends on report@ > -p << s.p; > +@script:python depends on s && report@ > +p << r.p; > @@ > > msg = "WARNING: sum of probable bitmasks, consider |" -- Best Regards Masahiro Yamada

Re: [PATCH] radix-tree: replace with

2017-11-12 Thread Masahiro Yamada
Hi Andrew, 2017-10-13 22:03 GMT+09:00 Masahiro Yamada : > This header contains references to spinlock_t and lockdep_is_held(), > both of which are defined in > > Signed-off-by: Masahiro Yamada I have not got any comment on this patch. Could you take a look a

Re: [PATCH 4/4] kbuild: optimize object directory creation for incremental build

2017-11-12 Thread Masahiro Yamada
Hi Cao, 2017-11-10 19:58 GMT+09:00 Cao jin : > Masahiro-san > > On 11/09/2017 11:41 PM, Masahiro Yamada wrote: >> The previous commit largely optimized the object directory creation. >> We can optimize it more for incremental build. >> >> There are already *.

Re: [PATCH 3/4] kbuild: create object directories simpler and faster

2017-11-13 Thread Masahiro Yamada
2017-11-10 18:21 GMT+09:00 Masahiro Yamada : > Hi Cao, > > > 2017-11-10 17:45 GMT+09:00 Cao jin : > >>> +ifneq ($(KBUILD_SRC),) >>> +# Create directories for object files if directory does not exist >>> +obj-dirs := $(sort $(obj) $(patsubst %/,%, $(dir

[PATCH v2 6/6] kbuild: remove redundant mkdir from ./Kbuild

2017-11-13 Thread Masahiro Yamada
These two targets are added to "targets". Their directories are automatically created. Signed-off-by: Masahiro Yamada --- Changes in v2: None Kbuild | 2 -- 1 file changed, 2 deletions(-) diff --git a/Kbuild b/Kbuild index 94c7527..f1997d8 100644 --- a/Kbuild +++ b/Kbuild @@ -

[PATCH v2 1/6] kbuild: create directory for make cache only when necessary

2017-11-13 Thread Masahiro Yamada
invoked by the first call of __run-and-store Signed-off-by: Masahiro Yamada Reviewed-by: Douglas Anderson --- Changes in v2: - Use boolean flag "create-cache-dir" for clarification - Fix a typo in git-log: Obiously -> Obviously scripts/Kbuild.include | 13 + 1

[PATCH v2 3/6] kbuild: filter-out PHONY targets from "targets"

2017-11-13 Thread Masahiro Yamada
veral places. PHONY targets never create .*.cmd files. Signed-off-by: Masahiro Yamada --- Changes in v2: - Newly added scripts/Makefile.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.build index 62d5314..6f60377

[PATCH v2 2/6] kbuild: remove redundant $(wildcard ...) for cmd_files calculation

2017-11-13 Thread Masahiro Yamada
I do not see any reason why $(wildcard ...) needs to be called twice for computing cmd_files. Remove the first one. Signed-off-by: Masahiro Yamada --- Changes in v2: - Fix "$$" in scripts/Makefile.headersinst per Douglas Makefile | 3 +-- scripts/Make

[PATCH v2 0/6] kbuild: optimize output directory creation

2017-11-13 Thread Masahiro Yamada
lt. Before: real0m8.322s user0m4.324s sys 0m1.220s After: real0m6.989s user0m4.168s sys 0m1.080s 4/6 gave the biggest impact. Masahiro Yamada (6): kbuild: create directory for make cache only when necessary kbuild: remove redundant $(wildcard ...) for cmd_files ca

[PATCH v2 4/6] kbuild: create object directories simpler and faster

2017-11-13 Thread Masahiro Yamada
e and more comprehensive. I also removed ugly code in arch/x86/entry/vdso/Makefile. This is now really unnecessary. Signed-off-by: Masahiro Yamada Acked-by: Ingo Molnar Tested-by: Douglas Anderson --- Changes in v2: - Complete "host-objdirs" removal - Remove (obj) from mkdir argumen

[PATCH v2 5/6] kbuild: optimize object directory creation for incremental build

2017-11-13 Thread Masahiro Yamada
skip "mkdir -p". With this, Kbuild runs almost zero "mkdir -p" in incremental building. Signed-off-by: Masahiro Yamada --- Changes in v2: - Remove "." from the filter-out list scripts/Makefile.build | 5 + 1 file changed, 5 insertions(+) diff --git a/sc

[PATCH] kbuild: specify FORCE as .PHONY target in Makefile.headersinst

2017-11-13 Thread Masahiro Yamada
Swap the order of ".PHONY: $(PHONY)" and "PHONY += FORCE" so that FORCE is correctly specified as a .PHONY target. Use a preferred way for specifying $(subdirs) as .PHONY targets. Signed-off-by: Masahiro Yamada --- scripts/Makefile.headersinst | 5 +++-- 1 file changed

[PATCH] selinux: remove unnecessary assignment to subdir-

2017-11-13 Thread Masahiro Yamada
Makefile.clean descends into $(subdir-y). Dummy assignment to subdir- is meaningless. Signed-off-by: Masahiro Yamada --- I will pick this up for kbuild tree since this is very trivial from the build system point of view. scripts/selinux/Makefile | 1 - 1 file changed, 1 deletion(-) diff

[PATCH] frv: .gitignore: ignore vmlinux.lds

2017-11-13 Thread Masahiro Yamada
Signed-off-by: Masahiro Yamada --- arch/frv/kernel/.gitignore | 1 + 1 file changed, 1 insertion(+) create mode 100644 arch/frv/kernel/.gitignore diff --git a/arch/frv/kernel/.gitignore b/arch/frv/kernel/.gitignore new file mode 100644 index 000..c5f676c --- /dev/null +++ b/arch/frv

[PATCH] sh: decompressor: ignore shipped files

2017-11-13 Thread Masahiro Yamada
These files are copied from arch/sh/lib, so should be ignored by git. Signed-off-by: Masahiro Yamada --- arch/sh/boot/compressed/.gitignore | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/sh/boot/compressed/.gitignore b/arch/sh/boot/compressed/.gitignore index 2374a83..edff113

Re: linux-next: build failure after merge of the gpio tree

2017-11-13 Thread Masahiro Yamada
atic void uniphier_gpio_irq_domain_deactivate(struct irq_domain *domain, > > Just a reminder that this conflict still exists. > Thanks for the reminder. Linus, Is it possible to forward this one? http://patchwork.ozlabs.org/patch/837435/ I'd like to eliminate coding style warning where possible. Just a slight change of coding style. I kept Stephen's authorship and Signed-off-by as-is. -- Best Regards Masahiro Yamada

Re: powerpc-linux-gnu-ld: warning: orphan section `.data.rel.ro' from `arch/powerpc/kernel/head_44x.o' being placed in section `.data.rel.ro'.

2017-11-13 Thread Masahiro Yamada
ns explicitly > > Match powerpc/64 and include .data.rel* input sections in the .data output > section explicitly. > > This should solve the warning: > > powerpc-linux-gnu-ld: warning: orphan section `.data.rel.ro' from > `arch/powerpc/kernel/head_44x.o' being pl

Re: [PATCH] builddeb: Pass the kernel:debarch substvar to dpkg-genchanges

2017-11-13 Thread Masahiro Yamada
variable to dpkg-genchanges. > > Signed-off-by: Sven Joachim > --- Applied to linux-kbuild/misc. Thanks! -- Best Regards Masahiro Yamada

Re: [PATCH] sh: select KBUILD_DEFCONFIG depending on ARCH

2017-11-13 Thread Masahiro Yamada
2017-11-03 2:40 GMT+09:00 Masahiro Yamada : > You can not select KBUILD_DEFCONFIG depending on any CONFIG option > because include/config/auto.conf is not included when building config > targets. So, CONFIG_SUPERH32 is never set during the configuration, > and cayman_defconfig is chos

Re: [Cocci] [PATCH v2] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-11-13 Thread Masahiro Yamada
Hi Julia, 2017-11-11 16:30 GMT+09:00 Julia Lawall : > > > On Fri, 10 Nov 2017, Julia Lawall wrote: > >> >> >> On Thu, 9 Nov 2017, Masahiro Yamada wrote: >> >> > The command "make -j8 C=1 CHECK=scripts/coccicheck" produces lots of >>

Re: [PATCH v2] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-11-13 Thread Masahiro Yamada
Hi Julia, 2017-11-14 0:30 GMT+09:00 Julia Lawall : > > > On Thu, 9 Nov 2017, Masahiro Yamada wrote: > >> The command "make -j8 C=1 CHECK=scripts/coccicheck" produces lots of >> "coccicheck failed" error messages. >> >> I do not know the c

Re: [PATCH] docs: dev-tools: coccinelle: delete out of date wiki reference

2017-11-13 Thread Masahiro Yamada
the latest version released from the Coccinelle homepage at > http://coccinelle.lip6.fr/ > > -Information and tips about Coccinelle are also provided on the wiki > -pages at http://cocci.ekstranet.diku.dk/wiki/doku.php > - > Once you have it, run the following command:: > > ./configure -- Best Regards Masahiro Yamada

Re: [PATCH v2] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-11-13 Thread Masahiro Yamada
Hi Julia, 2017-11-14 1:45 GMT+09:00 Julia Lawall : > > > On Tue, 14 Nov 2017, Masahiro Yamada wrote: > >> Hi Julia, >> >> >> 2017-11-14 0:30 GMT+09:00 Julia Lawall : >> > >> > >> > On Thu, 9 Nov 2017, Masahiro Yamada wrote: >>

Coccinelle: badzero.cocci failure

2017-11-14 Thread Masahiro Yamada
exception Yes_prepare_ocamlcocci.LinkFailure("/tmp/ocaml_cocci_18c9f9.cmxs") -- Best Regards Masahiro Yamada

[PATCH v3] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-11-14 Thread Masahiro Yamada
is a sensible solution; spatch does not create the subdirectory. Besides, ONLINE=1 mode takes a single file input for each spatch invocation, so there is no reason to parallelize it in the first place. Signed-off-by: Masahiro Yamada --- Changes in v3: - Set NPROC=1 because this is a more sens

Re: [PATCH v2] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-11-14 Thread Masahiro Yamada
Hi Julia, 2017-11-14 15:44 GMT+09:00 Julia Lawall : > > > On Tue, 14 Nov 2017, Masahiro Yamada wrote: > >> Hi Julia, >> >> 2017-11-14 1:45 GMT+09:00 Julia Lawall : >> > >> > >> > On Tue, 14 Nov 2017, Masahiro Yamada wrote: >> >

[PATCH] kbuild: move coccicheck help in scripts/Makefile.help to top Makefile

2017-11-14 Thread Masahiro Yamada
I do not think it is helpful to have a separate file just for the coccicheck help message. Merge scripts/Makefile.help into the top-level Makefile. Signed-off-by: Masahiro Yamada --- Makefile | 2 +- scripts/Makefile.help | 3 --- 2 files changed, 1 insertion(+), 4 deletions

Re: [PATCH v3] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-11-14 Thread Masahiro Yamada
planation sentence (If coccinelle also runs > in parallel,...). > > julia OK. Which lines are unneeded? Is it OK to remove all the comments, then just add "NPROC=1"? -- Best Regards Masahiro Yamada

Re: Coccinelle: badzero.cocci failure

2017-11-14 Thread Masahiro Yamada
1.0.6-00345-g2ca0bef compiled with OCaml version 4.02.3 Flags passed to the configure script: --prefix=/home/masahiro Python scripting support: yes Syntax of regular expresssions: PCRE > I'm not sure why it doesn't give feedback on whether OCaml scripting is > supported. I will check on this. > Thanks! -- Best Regards Masahiro Yamada

Re: [PATCH v2] kbuild: comments correction & update on Makefile.host

2017-11-14 Thread Masahiro Yamada
t-cobjs and host-cxxobjs > notation > host-objdirs := $(dir $(__hostprogs) $(host-cobjs) $(host-cxxobjs)) > > host-objdirs := $(strip $(sort $(filter-out ./,$(host-objdirs > -- I am removing all host-objdirs stuff. I do not need this hunk. -- Best Regards Masahiro Yamada

Re: [PATCH v3] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-11-14 Thread Masahiro Yamada
2017-11-14 20:09 GMT+09:00 Julia Lawall : > > > On Tue, 14 Nov 2017, Masahiro Yamada wrote: > >> Hi Julia, >> >> >> 2017-11-14 18:49 GMT+09:00 Julia Lawall : >> >> + >> >> +# If -j option is given to Make, scripts/coccicheck runs i

[PATCH v4] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-11-14 Thread Masahiro Yamada
s a reasonable solution; spatch does not create the subdirectory. Besides, ONLINE=1 mode takes a single file input for each spatch invocation, so there is no reason to parallelize it in the first place. Signed-off-by: Masahiro Yamada --- Changes in v4: - Remove unnecessary comments. One lin

[URGENT PATCH] arm64: dts: uniphier: route on-board device IRQ to GPIO controller for PXs3

2017-11-14 Thread Masahiro Yamada
Commit 429f203eb712 ("arm64: dts: uniphier: route on-board device IRQ to GPIO controller") missed to update this DTS. It becames a real problem when arm and arm64 trees are merged together. Signed-off-by: Masahiro Yamada --- Arnd, Olof, I think you are sending pull-requests for

Re: [PATCH] kbuild: Set KBUILD_CFLAGS before incl. arch Makefile

2017-11-04 Thread Masahiro Yamada
-# These warnings generated too much noise in a regular build. > -# Use make W=1 to enable them (see scripts/Makefile.extrawarn) > -KBUILD_CFLAGS += $(call cc-disable-warning, unused-but-set-variable) > -KBUILD_CFLAGS += $(call cc-disable-warning, unused-const-variable) > -endif > - > ifdef CONFIG_FRAME_POINTER > KBUILD_CFLAGS += -fno-omit-frame-pointer -fno-optimize-sibling-calls > else > -- > 2.15.0.403.gc27cc4dac6-goog > > -- > 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

[PATCH 0/2] kbuild: move dtb-y and CONFIG_OF_ALL_DTBS to Kbuild core to fix some issues

2017-11-04 Thread Masahiro Yamada
This series applies on top of my previous cleanup patch: https://patchwork.kernel.org/patch/10038167/ This series must go to the same branch, so I am sending this to DT ML. Masahiro Yamada (2): MIPS: dts: remove bogus bcm96358nb4ser.dtb from dtb-y entry kbuild: handle dtb-y and

[PATCH 2/2] kbuild: handle dtb-y and CONFIG_OF_ALL_DTBS natively in Makefile.lib

2017-11-04 Thread Masahiro Yamada
ely, so it should not hurt to do so. Add $(dtb-y) to extra-y, and $(dtb-) as well if CONFIG_OF_ALL_DTBS is enabled. All clutter things in Makefiles go away. As a bonus clean-up, I also removed dts-dirs. Just use subdir-y directly to traverse sub-directories. Signed-off-by: Masahiro Yamada --- arc

[PATCH 1/2] MIPS: dts: remove bogus bcm96358nb4ser.dtb from dtb-y entry

2017-11-04 Thread Masahiro Yamada
arch/mips/boot/dts/brcm/bcm96358nb4ser.dts does not exist, so we cannot build bcm96358nb4ser.dtb . Signed-off-by: Masahiro Yamada --- arch/mips/boot/dts/brcm/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/mips/boot/dts/brcm/Makefile b/arch/mips/boot/dts/brcm/Makefile index

Re: [PATCH 1/2] MIPS: dts: remove bogus bcm96358nb4ser.dtb from dtb-y entry

2017-11-05 Thread Masahiro Yamada
+CC Ralf Baechle +CC linux-m...@linux-mips.org +CC Kevin Cernekee +CC Florian Fainelli I missed to CC MIPS maintainers. 2017-11-05 14:30 GMT+09:00 Masahiro Yamada : > arch/mips/boot/dts/brcm/bcm96358nb4ser.dts does not exist, so > we cannot build bcm96358nb4ser.dtb . > > S

Re: [PATCH 1/2] MIPS: dts: remove bogus bcm96358nb4ser.dtb from dtb-y entry

2017-11-06 Thread Masahiro Yamada
2017-11-06 19:41 GMT+09:00 James Hogan : > Hi, > > On Sun, Nov 05, 2017 at 11:11:38PM +0900, Masahiro Yamada wrote: >> +CC Ralf Baechle >> +CC linux-m...@linux-mips.org >> +CC Kevin Cernekee >> +CC Florian Fainelli >> >> >> I missed to CC MIPS

[PATCH 3/6] ARM: dts: uniphier: route on-board device IRQ to GPIO controller

2017-10-17 Thread Masahiro Yamada
Interrupt lines from on-board devices are connected to the GPIO controller. Handle this correctly. Signed-off-by: Masahiro Yamada --- arch/arm/boot/dts/uniphier-ld4-ref.dts | 2 +- arch/arm/boot/dts/uniphier-ld6b-ref.dts | 2 +- arch/arm/boot/dts/uniphier-pro4-ref.dts | 2

[PATCH 4/6] arm64: dts: uniphier: route on-board device IRQ to GPIO controller

2017-10-17 Thread Masahiro Yamada
Interrupt lines from on-board devices are connected to the GPIO controller. Handle this correctly. Signed-off-by: Masahiro Yamada --- arch/arm64/boot/dts/socionext/uniphier-ld11-ref.dts | 3 ++- arch/arm64/boot/dts/socionext/uniphier-ld20-ref.dts | 3 ++- 2 files changed, 4 insertions(+), 2

[PATCH 2/6] arm64: dts: uniphier: add GPIO controller nodes

2017-10-17 Thread Masahiro Yamada
The GPIO controller also acts as an interrupt controller and the interrupt lines are connected to the AIDET block. Signed-off-by: Masahiro Yamada --- arch/arm64/boot/dts/socionext/uniphier-ld11.dtsi | 25 arch/arm64/boot/dts/socionext/uniphier-ld20.dtsi | 19

[PATCH 1/6] ARM: dts: uniphier: add GPIO controller nodes

2017-10-17 Thread Masahiro Yamada
The GPIO controller also acts as an interrupt controller and the interrupt lines are connected to the AIDET block. Signed-off-by: Masahiro Yamada --- arch/arm/boot/dts/uniphier-ld4.dtsi | 14 ++ arch/arm/boot/dts/uniphier-pro4.dtsi | 14 ++ arch/arm/boot/dts/uniphier

[PATCH 0/6] ARM: dts: uniphier: add GPIO nodes and related properties

2017-10-17 Thread Masahiro Yamada
Masahiro Yamada (6): ARM: dts: uniphier: add GPIO controller nodes arm64: dts: uniphier: add GPIO controller nodes ARM: dts: uniphier: route on-board device IRQ to GPIO controller arm64: dts: uniphier: route on-board device IRQ to GPIO controller ARM: dts: uniphier: add GPIO hog

[PATCH 5/6] ARM: dts: uniphier: add GPIO hog definition

2017-10-17 Thread Masahiro Yamada
Interrupt lines from on-board devices are connected to the GPIO controller. Add GPIO hogging so that the corresponding GPIO line is automatically requested. Signed-off-by: Masahiro Yamada --- arch/arm/boot/dts/uniphier-ld4-ref.dts | 8 arch/arm/boot/dts/uniphier-ld6b-ref.dts | 8

[PATCH 6/6] arm64: dts: uniphier: add GPIO hog definition

2017-10-17 Thread Masahiro Yamada
Interrupt lines from on-board devices are connected to the GPIO controller. Add GPIO hogging so that the corresponding GPIO line is automatically requested. Signed-off-by: Masahiro Yamada --- arch/arm64/boot/dts/socionext/uniphier-ld11-ref.dts | 8 arch/arm64/boot/dts/socionext

Re: [PATCH] Makefile: add targets for config-help and pkg-help

2017-10-18 Thread Masahiro Yamada
lp: > + @$(MAKE) $(build)=$(package-dir) help > > help-board-dirs := $(addprefix help-,$(board-dirs)) > > -- What happened to "doc-help" ? (I want to see consistent hyphenation) Please follow Randy's suggestion. Also you need to add %-help pattern to no-dot-config-targets. -- Best Regards Masahiro Yamada

Re: [RFC PATCH] kbuild: Allow specifying some base host CFLAGS

2017-10-18 Thread Masahiro Yamada
nel's normal flags. > > One last note: I chose to add new variables rather than just having > the build system try to pass HOSTCFLAGS in somehow (either through the > environment or the command line) to avoid weird interactions with > recursive invocations of make. > > Signed-off-by: Douglas Anderson > --- I'd like to know for-instance cases where this is useful. -- Best Regards Masahiro Yamada

Re: [PATCH v4 0/2] kbuild: Cache exploratory calls to the compiler

2017-10-18 Thread Masahiro Yamada
n of try-run (oops) > - Add Tested-by for Ingo and Guenter since v2 and v3 are very similar > > Changes in v2: > - Abstract at a different level (like shell-cached) per Masahiro Yamada > - Include ld-version, which I missed the first time > > Douglas Anderson (2): > kbuil

Re: [PATCH] kbuild: do not call cc-option before KBUILD_CFLAGS initialization

2017-10-18 Thread Masahiro Yamada
2017-10-12 18:22 GMT+09:00 Masahiro Yamada : > Some $(call cc-option,...) are invoked very early, even before > KBUILD_CFLAGS, etc. are initialized. > > The returned string from $(call cc-option,...) depends on > KBUILD_CPPFLAGS, KBUILD_CFLAGS, and GCC_PLUGINS_CFLAGS. > > Si

Re: [PATCH v2] kbuild: shrink .cache.mk when it exceeds 1000 lines

2017-10-18 Thread Masahiro Yamada
2017-10-13 20:25 GMT+09:00 Masahiro Yamada : > The cache files are only cleaned away by "make clean". If you continue > incremental builds, the cache files will grow up little by little. > It is not a big deal in general use cases because compiler flags do not > change qu

Re: [PATCH net-next v2 2/2] net: ethernet: socionext: add AVE ethernet driver

2017-10-18 Thread Masahiro Yamada
l in the driver. > Referring to your suggested method, I'll fix the part of clk and reset. > Why is clk optional? -- Best Regards Masahiro Yamada

Re: more build problems with "Makefile: move stackprotector availability out of Kconfig"

2017-10-18 Thread Masahiro Yamada
nfig_cc_stackprotector_auto-fix-3.patch > > Yes, these should get dropped for the moment, thanks. Arnd and I have > been trying to get to the bottom of it. > > -Kees I see this series is repeating apply/drop, so I am not tracking down the latest status. When you have a chance for re-spin, please replace KBUILD_CFLAGS += -DCONFIG_CC_STACKPROTECTOR KBUILD_AFLAGS += -DCONFIG_CC_STACKPROTECTOR with KBUILD_CPPFLAGS += -DCONFIG_CC_STACKPROTECTOR -- Best Regards Masahiro Yamada

[PATCH] clk: uniphier: fix parent of miodmac clock data

2017-10-18 Thread Masahiro Yamada
The "miodmac" is not a child of "stdmac". They are independent from each other. Fix it. Signed-off-by: Masahiro Yamada --- drivers/clk/uniphier/clk-uniphier-mio.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/clk/uniphier/clk-uniphier-

Re: [PATCH] printk: simplify no_printk()

2017-10-19 Thread Masahiro Yamada
Hi Petr, 2017-10-02 23:56 GMT+09:00 Petr Mladek : > On Mon 2017-09-18 00:01:44, Masahiro Yamada wrote: >> Commit 069f0cd00df0 ("printk: Make the printk*once() variants return >> a value") surrounded the macro implementation with ({ ... }). >> >> Now, the inn

Re: [GIT PULL] irqchip/irqdomain updates for 4.15

2017-10-19 Thread Masahiro Yamada
Hi Marc, 2017-10-19 20:15 GMT+09:00 Marc Zyngier : > Masahiro Yamada (2): > irqdomain: Move revmap_trees_mutex to struct irq_domain > irqdomain: Add __rcu annotations to radix tree slot > Thomas had already pick this up. It is in Linus' t

Re: [PATCH] kbuild doc: a bundle of fixes on makefiles.txt

2017-10-19 Thread Masahiro Yamada
igned-off-by: Cao jin > --- Applied to linux-kbuild/fixes. Thanks! -- Best Regards Masahiro Yamada

Re: [PATCH] kbuild: fix linker feature test macros when cross compiling with Clang

2017-10-27 Thread Masahiro Yamada
do not like to add $(CLANG_TARGET) to a place for common helpers. Instead of $(CLANG_TARGET), please add $(KBUILD_CPPFLAGS) $(CC_OPTION_CFLAGS) to cc-ldoption and ld-option. I have two requests next time: - please include linux-kbu...@vger.kernel.org in your To list - please base your patch on linux-kbuild/kbuild branch The URL of the tree is described in MAINTAINERS. Thanks. -- Best Regards Masahiro Yamada

Re: [PATCH] coccinelle: fix verbose message about .cocci file being run

2017-10-28 Thread Masahiro Yamada
Hi Jim 2017-10-28 8:56 GMT+09:00 Jim Davis : > On Wed, Oct 25, 2017 at 9:55 PM, Masahiro Yamada > wrote: >> If you run coccicheck with V=1 and COCCI=, you will see a strange >> path to the semantic patch file. For example, run the following: >> >> $ make V=1

Re: [PATCH] kbuild: fix linker feature test macros when cross compiling with Clang

2017-10-28 Thread Masahiro Yamada
2017-10-28 3:28 GMT+09:00 Nick Desaulniers : > + linux-kbu...@vger.kernel.org > > On Fri, Oct 27, 2017 at 4:20 AM, Masahiro Yamada > wrote: >> I do not like to add $(CLANG_TARGET) to a place for common helpers. >> Instead of $(CLANG_TARGET), please add >> $(KBUIL

Re: [PATCH v2] kbuild: fix linker feature test macros when cross compiling with Clang

2017-10-28 Thread Masahiro Yamada
g: cannot find entry symbol _start; defaulting to 004002e4 > $ echo $? > 0 > > This causes conditional checks that invoke $(CC) without the target > triple, then $(LD) on the result, to always fail. > > Suggested-by: Masahiro Yamada > Signed-off-by: Nick Desaulniers

[GIT PULL] Kbuild fixes for v4.14-rc7

2017-10-28 Thread Masahiro Yamada
in Makefile.modpost kbuild doc: a bundle of fixes on makefiles.txt David Lin (1): kbuild: clang: fix build failures with sparse check Masahiro Yamada (1): kbuild: revert $(realpath ...) to $(shell cd ... && /bin/pwd) Randy Dunlap (1): Makefile: kselftest: fix gram

[PATCH 3/4] reset: minimize the number of headers included from

2017-10-28 Thread Masahiro Yamada
alse. Signed-off-by: Masahiro Yamada --- include/linux/reset.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/include/linux/reset.h b/include/linux/reset.h index 9329664..53e5e78 100644 --- a/include/linux/reset.h +++ b/include/linux/reset.h @@ -1,8 +1,10 @@ #ifndef _L

[PATCH 2/4] reset: remove remaining WARN_ON() in

2017-10-28 Thread Masahiro Yamada
check, but of_reset_control_get_exclusive() does not. Signed-off-by: Masahiro Yamada --- include/linux/reset.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/include/linux/reset.h b/include/linux/reset.h index c2fe7b6..9329664 100644 --- a/include/linux/reset.h +++ b/include/linux/reset.h @@ -12

[PATCH 1/4] reset: make device_reset_optional() really optional

2017-10-28 Thread Masahiro Yamada
Commit bb475230b8e5 ("reset: make optional functions really optional") converted *_get_optional* functions, but device_reset_optional() was left behind. Convert it in the same way. Signed-off-by: Masahiro Yamada --- drivers/reset/core.c | 9 + include/linux/re

[PATCH 4/4] reset: remove reset_control_get(_optional)

2017-10-28 Thread Masahiro Yamada
No more users of these two. Signed-off-by: Masahiro Yamada --- include/linux/reset.h | 12 1 file changed, 12 deletions(-) diff --git a/include/linux/reset.h b/include/linux/reset.h index 53e5e78..0aa3d0d 100644 --- a/include/linux/reset.h +++ b/include/linux/reset.h @@ -343,18

Re: [RFC PATCH] clk: move of_clk_get_parent_count() declaration to

2017-10-28 Thread Masahiro Yamada
Hi Stephen, 2017-10-13 8:17 GMT+09:00 Stephen Boyd : > On 10/05, Masahiro Yamada wrote: >> The clock consumer, drivers/video/fbdev/simplefb.c, includes >> just for calling of_clk_get_parent_count(). >> This is ugly. >> >> Looking at simplefb_clocks_get()

Re: [PATCH] Check all .c files for bad kernel-doc comments

2017-10-29 Thread Masahiro Yamada
$show_not_found == 1)) { > print STDERR "Was looking for '$_'.\n" for keys > %function_table; > } > -- > 2.11.0.301.g722e3be85.dirty > > -- > 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: [PATCH] Coccinelle: use false positive annotation

2017-10-29 Thread Masahiro Yamada
nit. The last /// should be turned into // if you want to avoid two blank lines in a series after the semantic patch information. (I can fix it locally if you agree with it and I am supposed to pick up this.) > // Confidence: Low > // Copyright: (C) 2010 Nicolas Palix, DIKU. GPLv2. > -- Best Regards Masahiro Yamada

Re: [PATCH v2] kbuild: fix linker feature test macros when cross compiling with Clang

2017-10-29 Thread Masahiro Yamada
2017-10-29 0:00 GMT+09:00 Masahiro Yamada : > 2017-10-28 5:13 GMT+09:00 Nick Desaulniers : >> I was not seeing my linker flags getting added when using ld-option when >> cross compiling with Clang. Upon investigation, this seems to be due to >> a difference in how GCC

[PATCH 2/2] usb: ohci-platform: use reset array API

2017-10-30 Thread Masahiro Yamada
Generic drivers like this need to control arbitrary numbers of reset lines. Instead of hard-coding the maximum number of resets, use the reset array API. It can manage a bunch of resets behind the scene. Signed-off-by: Masahiro Yamada --- drivers/usb/host/ohci-platform.c | 37

[PATCH 1/2] usb: ehci-platform: use reset array API

2017-10-30 Thread Masahiro Yamada
Generic drivers like this need to control arbitrary numbers of reset lines. Instead of hard-coding the maximum number of resets, use the reset array API. It can manage a bunch of resets behind the scene. Signed-off-by: Masahiro Yamada --- drivers/usb/host/ehci-platform.c | 33

[PATCH] arm64: defconfig: enable CONFIG_GPIO_UNIPHIER

2017-10-30 Thread Masahiro Yamada
Enable the GPIO controller driver used for UniPhier SoC family. Signed-off-by: Masahiro Yamada --- arch/arm64/configs/defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index ceb9ca2..6b4df78 100644 --- a/arch/arm64/configs

[PATCH] ARM: multi_v7_defconfig: enable CONFIG_GPIO_UNIPHIER

2017-10-30 Thread Masahiro Yamada
Enable the GPIO controller driver used for UniPhier SoC family. Signed-off-by: Masahiro Yamada --- arch/arm/configs/multi_v7_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig index f857d3f..28668e4 100644

Re: [GIT PULL] arm64: dts: uniphier: UniPhier DT updates (64bit) for v4.15

2017-10-30 Thread Masahiro Yamada
Hi Arnd, 2017-10-30 22:24 GMT+09:00 Arnd Bergmann : > On Mon, Oct 23, 2017 at 7:40 PM, Masahiro Yamada > wrote: > >> UniPhier ARM64 SoC DT updates for v4.15 >> >> - add thermal monitor and thermal zone nodes >> - add efuse nodes >> - fix W=2 warnings >

Re: [PATCH] Check all .c files for bad kernel-doc comments

2017-10-30 Thread Masahiro Yamada
Hi Matthew, 2017-10-30 22:40 GMT+09:00 Matthew Wilcox : > On Mon, Oct 30, 2017 at 12:40:20PM +0900, Masahiro Yamada wrote: >> 2017-10-28 4:41 GMT+09:00 Matthew Wilcox : >> > Implement a '-none' output mode for kernel-doc which will only output >> > warning

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