[PATCH] nvme: Makefile: remove dead build rule

2017-06-28 Thread Valentin Rothberg
Remove dead build rule for drivers/nvme/host/scsi.c which has been removed by commit ("nvme: Remove SCSI translations"). Signed-off-by: Valentin Rothberg --- drivers/nvme/host/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/nvme/host/Makefile b/drivers/nvme/hos

rcu: NO_HZ_FULL_SYSIDLE leftovers

2017-05-16 Thread Valentin Rothberg
Hi Paul, I stumbled upon your commit 1cb830954ef9 ("rcu: Remove nohz_full full-system-idle state machine") as ./scripts/checkkconfigsymbols.py reported that there are some remaining references on the removed Kconfig symbol 'NO_HZ_FULL_SYSIDLE'. I couldn't find any commit removing them on LKML, and

[PATCH] exofs: ORE: remove dead dependence

2017-04-21 Thread Valentin Rothberg
Remove last reference on PNFS_OBJLAYOUT. Signed-off-by: Valentin Rothberg --- fs/exofs/Kconfig.ore | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/exofs/Kconfig.ore b/fs/exofs/Kconfig.ore index 2daf2329c28d..73cab8d92dfd 100644 --- a/fs/exofs/Kconfig.ore +++ b/fs/exofs

Re: [PATCH 3/4] nfs: remove the objlayout driver

2017-04-21 Thread Valentin Rothberg
Hi Christoph, I just came across this patch in linux-next commit 6d22323b2e9f. scripts/checkkconfigsymbols.py reports that there is a leftover reference on PNFS_OBJLAYOUT in fs/exofs/Kconfig.ore line 10 (depends on EXOFS_FS || PNFS_OBJLAYOUT). I assume that this reference can safely be removed now

[PATCH] tracing: fix DEBUG_PREEMPT typo

2017-04-13 Thread Valentin Rothberg
s/PREEMPT_DEBUG/DEBUG_PREEMPT/ Signed-off-by: Valentin Rothberg --- include/linux/ftrace.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h index c4552953a59a..e1bbfbb741a5 100644 --- a/include/linux/ftrace.h +++ b/include

Re: [for-next][PATCH 4/7] tracing: Rename trace_active to disable_stack_tracer and inline its modification

2017-04-13 Thread Valentin Rothberg
Hi Steven, I just found this patch in linux-next commit 8aaf1ee70e19 with scripts/checkkconfigsymbols.py complaining about an undefined CONFIG_PREEMPT_DEBUG (see below). I guess it's just a typo, since there is a symbol named DEBUG_PREEMPT. If you want to, I can send a trivial patch to rename bot

Re: [PATCH] Staging: comedi: drivers: comedi_test: Avoid multiple line dereference

2017-02-22 Thread Valentin Rothberg
On Feb 21 '17 09:22, Joe Perches wrote: > On Wed, 2017-02-22 at 00:31 +0800, Cheah Kok Cheong wrote: > > > Another option could be using the typedefs from include/linux/types.h, > > > e.g. ushort. However, this might require changing other declarations as > > > well to keep consistency. > > > > T

Re: [PATCH] Staging: comedi: drivers: comedi_test: Avoid multiple line dereference

2017-02-21 Thread Valentin Rothberg
On Feb 21 '17 10:12, Ian Abbott wrote: > On 21/02/2017 09:33, Cheah Kok Cheong wrote: > > On Mon, Feb 20, 2017 at 05:36:52PM +, Ian Abbott wrote: > > > On 20/02/17 16:02, Cheah Kok Cheong wrote: > > > > On Mon, Feb 20, 2017 at 10:03:39AM +, Ian Abbott wrote: > > > > > On 20/02/17 08:28, Che

Re: mm: testcases for RODATA: renamed Kconfig option

2017-02-08 Thread Valentin Rothberg
> Thanks. > Jinbum Park. > > > 2017. 2. 8. 오후 6:34에 "Valentin Rothberg" 님이 작성: > > Hi Jinbum, > > your commit bf9058f08772 ("mm: add arch-independent testcases for > RODATA") has shown up in today's linux next and ran into an issue with

mm: testcases for RODATA: renamed Kconfig option

2017-02-08 Thread Valentin Rothberg
Hi Jinbum, your commit bf9058f08772 ("mm: add arch-independent testcases for RODATA") has shown up in today's linux next and ran into an issue with another commit. The added testcases depend on DEBUG_RODATA_TEST, which further depends on DEBUG_RODATA. DEBUG_RODATA, however, has been renamed to S

net: dsa: remaining references on NET_DSA_HWMON

2017-01-23 Thread Valentin Rothberg
Hi Andrew, your commit cf1a56a4cf19 ("net: dsa: Remove hwmon support") has popped up in today's linux-next. The commit removes all references on NET_DSA_HWMON but the following: Documentation/networking/dsa/dsa.txt:501:These callbacks are only available if CONFIG_NET_DSA_HWMON is enabled: inclu

Re: [PATCH] ARM: Exynos4415: remove dead build rule

2017-01-19 Thread Valentin Rothberg
Hi Krzysztof, On 19/01/2017 10:20, Krzysztof Kozlowski wrote: > On Thu, Jan 19, 2017 at 11:18 AM, Krzysztof Kozlowski wrote: >> On Thu, Jan 19, 2017 at 11:11 AM, Valentin Rothberg >> wrote: >>> Commit 22dfab102cfa ("ARM: EXYNOS: Remove Exynos4415 arch code (

[PATCH] ARM: Exynos4415: remove dead build rule

2017-01-19 Thread Valentin Rothberg
Commit 22dfab102cfa ("ARM: EXYNOS: Remove Exynos4415 arch code (SoC not supported anymore)") removed the Kconfig option SOC_EXYNOS4415. Thus, we are safe to remove the last reference in the Makefile. Signed-off-by: Valentin Rothberg --- Detected by using ./scripts/checkkconfig

[PATCH] checkkconfigsymbols.py: don't sort similar symbols

2017-01-18 Thread Valentin Rothberg
Don't sort the list of string-similar Kconfig symbols alphabetically to preserve the correct order of string similarity. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/checkkconfigsymbols

[PATCH] security/apparmor/lsm.c: set debug messages

2017-01-17 Thread Valentin Rothberg
Add the _APPARMOR substring to reference the intended Kconfig option. Signed-off-by: Valentin Rothberg --- Detected by using ./scripts/checkkconfigsymbols.py (--diff on linux-next) security/apparmor/lsm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/security/apparmor

[PATCH] arch/openrisc/lib/memcpy.c: use correct OR1200 option

2017-01-17 Thread Valentin Rothberg
The Kconfig option for OR12000 is OR1K_1200. Signed-off-by: Valentin Rothberg --- arch/openrisc/lib/memcpy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/openrisc/lib/memcpy.c b/arch/openrisc/lib/memcpy.c index 4706f01a199a..669887a60e27 100644 --- a/arch/openrisc

[PATCH] sparc64_defconfig: correctly set user probes

2016-12-13 Thread Valentin Rothberg
s/UPROBE_EVENTS/UPROBE_EVENT/ Signed-off-by: Valentin Rothberg --- arch/sparc/configs/sparc64_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/sparc/configs/sparc64_defconfig b/arch/sparc/configs/sparc64_defconfig index b2e650d1764f..f912fd1f7e32 100644 --- a

[PATCH] ARM: multi_v7_defconfig: set bcm47xx watchdog

2016-12-02 Thread Valentin Rothberg
Correct the bcm47xx watchdog option. The convention of bcm watchdogs is the _WDT suffix. Signed-off-by: Valentin Rothberg --- Detected by running scripts/checkkconfigsymbols.py arch/arm/configs/multi_v7_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm

[PATCH] ARM: multi_v7_defconfig: fix config typo

2016-12-02 Thread Valentin Rothberg
s/CONFIG_CONFIG_BCM47XX_NVRAM/CONFIG_BCM47XX_NVRAM/ Signed-off-by: Valentin Rothberg --- Detected by running scripts/checkkconfigsymbols.py arch/arm/configs/multi_v7_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm

Re: i2c: undefined option I2C_ALGO_BUSCLEAR

2016-11-17 Thread Valentin Rothberg
Hi Paul, On Thu, Nov 17, 2016 at 12:55 PM, Paul Bolle wrote: > Hi Valentin, > > On Thu, 2016-11-17 at 12:33 +0100, Valentin Rothberg wrote: >> I tested your patch and it works properly for me. It even still applies >> on linux-next. > > Cool. > > (We should p

Re: i2c: undefined option I2C_ALGO_BUSCLEAR

2016-11-17 Thread Valentin Rothberg
> > Hi Valentin, > > On Wed, 2016-11-16 at 08:31 +0100, Valentin Rothberg wrote: > > your commit c3ca951fe41a ("i2c: Add Tegra BPMP I2C proxy driver") > > popped up in today's linux-next tree, adding Kconfig option > > I2C_TEGRA_BPMP, which further selec

i2c: undefined option I2C_ALGO_BUSCLEAR

2016-11-15 Thread Valentin Rothberg
Hi Shardar, your commit c3ca951fe41a ("i2c: Add Tegra BPMP I2C proxy driver") popped up in today's linux-next tree, adding Kconfig option I2C_TEGRA_BPMP, which further selects I2C_ALGO_BUSCLEAR, which is nowhere defined in Kconfig. Is there a patch queued somewhere to add I2C_ALGO_BUSCLEAR to Kco

[PATCH] ASoC: samsung: Makefile cleanup

2016-11-09 Thread Valentin Rothberg
Commit a076d418235f ("ASoC: samsung: Drop AC97 drivers") removed some unused code and the associated Kconfig options, but left those options referenced in the Makefile. Remove the leftover references in the Makefile. Signed-off-by: Valentin Rothberg --- I found the iss

[PATCH] checkkconfigsymbols.py: support git's "^" syntax

2016-10-27 Thread Valentin Rothberg
Support git's "^" syntax for diffing two commits, for instance via "--diff HEAD^^^..HEAD". Signed-off-by: Michael Ellermann Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts

Re: [PATCH] powerpc: process.c: fix Kconfig typo

2016-10-26 Thread Valentin Rothberg
On Wed, Oct 26, 2016 at 12:40 PM, Michael Ellerman wrote: > Valentin Rothberg writes: > >> On Wed, Oct 26, 2016 at 7:52 AM, Michael Ellerman >> wrote: >>> Cyril Bur writes: >>> >>>> On Wed, 2016-10-05 at 07:57 +0200, Valentin Rothberg wrote: &g

Re: [PATCH] powerpc: process.c: fix Kconfig typo

2016-10-25 Thread Valentin Rothberg
On Wed, Oct 26, 2016 at 7:52 AM, Michael Ellerman wrote: > Cyril Bur writes: > >> On Wed, 2016-10-05 at 07:57 +0200, Valentin Rothberg wrote: >>> s/ALIVEC/ALTIVEC/ >>> >> >> Oops, nice catch >> >>> Signed-off-by: Valentin Rothberg >&g

drm/i915: undefined STOP_MACHINE

2016-10-18 Thread Valentin Rothberg
Dear Chris, your commit 9f267eb8d2ea ("drm/i915: Stop the machine whilst capturing the GPU crash dump") has popped up in today's linux-next and adds a select from DRM_I915 on STOP_MACHINE, which is not defined anywhere in Kconfig. STOP_MACHINE has been removed a while ago (see https://lkml.org/lk

[PATCH] powerpc: process.c: fix Kconfig typo

2016-10-04 Thread Valentin Rothberg
s/ALIVEC/ALTIVEC/ Signed-off-by: Valentin Rothberg --- arch/powerpc/kernel/process.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c index 9e7c10fe205f..ce6dc61b15b2 100644 --- a/arch/powerpc/kernel/process.c

samples: blackfin: dead config options

2016-10-03 Thread Valentin Rothberg
Hi Shua, your commit c6d2995da540 ("samples: move blackfin gptimers-example from Documentation") has popped up in today's linux-next. scripts/checkkconfigsymbols.py complained about two issues. The refactoring from build rules to Kconfig preserved the 'CONFIG_' prefix (i.e., depends on CONFIG_BLA

[PATCH] soc/fsl: dpaa_sys.h: fix ARM32 typo

2016-09-27 Thread Valentin Rothberg
The Kconfig symbol for 32bit ARM is 'ARM', not 'ARM32'. Signed-off-by: Valentin Rothberg --- drivers/soc/fsl/qbman/dpaa_sys.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/fsl/qbman/dpaa_sys.h b/drivers/soc/fsl/qbman/dpaa_sys.h index b63fd7

Re: staging/greybus: undefined Kconfig symbols

2016-09-21 Thread Valentin Rothberg
On Wed, Sep 21, 2016 at 9:39 AM, Greg KH wrote: > On Wed, Sep 21, 2016 at 08:30:01AM +0200, Valentin Rothberg wrote: >> Hi Greg, > > I was a little surprised, too, but that's the one in the signed-off field in the greybus commits, so I used it. >> checkkconfigsymbo

staging/greybus: undefined Kconfig symbols

2016-09-20 Thread Valentin Rothberg
Hi Greg, checkkconfigsymbols.py found some undefined Kconfig symbols (see below) in the Makefile and Kconfig file. I could not find any patch adding those symbols. Are there patches queued somewhere else? Best regards, Valentin GREYBUS_AUDIO_MSM8994 Referencing files: drivers/staging/greybus/

drm/i915: undefined symbol I915_SW_FENCE_CHECK_DAG

2016-09-11 Thread Valentin Rothberg
Hi Chris, your commit e68a139f6bf3 ("drm/i915: Add a sw fence for collecting up dma fences") has shown up in today's linux-next (i.e., 20160912) adding the following the lines (184++): + if (!IS_ENABLED(CONFIG_I915_SW_FENCE_CHECK_DAG)) + return false; The Kconfig symbol isn't

[PATCH v2 3/5] checkkconfigsymbols.py: fix pylint and pep8 warnings

2016-08-27 Thread Valentin Rothberg
Fix pylint and pep8 warnings to have a consistent syntax and style. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts/checkkconfigsymbols.py index

[PATCH v2 4/5] checkkconfigsymblos: consistent symbol terminology

2016-08-27 Thread Valentin Rothberg
'symbol' and 'feature' are used synonymously to refer to Kconfig symbols (configs, menus, etc.). Use the term 'symbol' to have a consistent terminology and to make the code more comprehensible. Signed-off-by: Valentin Rothberg --- script

[PATCH v2 1/5] checkkconfigsymbols.py: port to Python 3

2016-08-27 Thread Valentin Rothberg
Python 2 is slowly dying, so port the script to Python 3. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 33 + 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts/checkkconfigsymbols.py

[PATCH v2 2/5] checkkconfigsymbols: use ArgumentParser

2016-08-27 Thread Valentin Rothberg
Replace the deprecated OptionParser with ArgumentParser, as recommended by pylint. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 142 - 1 file changed, 69 insertions(+), 73 deletions(-) diff --git a/scripts/checkkconfigsymbols.py

[PATCH v2 0/5] checkkconfigsymbols: several updates

2016-08-27 Thread Valentin Rothberg
port, ArgumentParser, pylint/pep8 warnings) - More descriptive commit messages Valentin Rothberg (5): checkkconfigsymbols.py: port to Python 3 checkkconfigsymbols: use ArgumentParser checkkconfigsymbols.py: fix pylint and pep8 warnings checkkconfigsymblos: consistent symbol termin

[PATCH v2 5/5] checkkconfigsymbols: use arglist instead of cmd string

2016-08-27 Thread Valentin Rothberg
Splitting a command string could lead to unintended arguments. Use an argument list in the execute() function instead. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 27 --- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git a/scripts

Re: [PATCH 1/3] checkkconfigsymbols: port to python3

2016-08-27 Thread Valentin Rothberg
On Sat, Aug 27, 2016 at 7:53 PM, Greg KH wrote: > On Sat, Aug 27, 2016 at 05:25:29PM +0200, Valentin Rothberg wrote: >> Port the script to python3 and fix some pylint warnings. > > "and" in a changelog description usually means this should be 2 > different patches. >

[PATCH 2/3] checkkconfigsymbols.py: consitent symbol terminology

2016-08-27 Thread Valentin Rothberg
Make the term 'symbol' consitent in the entire scripts. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 134 - 1 file changed, 67 insertions(+), 67 deletions(-) diff --git a/scripts/checkkconfigsymbols.py

[PATCH 3/3] checkkconfigsymbols: use arglist instead of cmd string

2016-08-27 Thread Valentin Rothberg
Splitting a command string could lead to unintended arguments. Use an argument list in the execute() function instead. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 29 ++--- 1 file changed, 18 insertions(+), 11 deletions(-) diff --git a/scripts

[PATCH 1/3] checkkconfigsymbols: port to python3

2016-08-27 Thread Valentin Rothberg
Port the script to python3 and fix some pylint warnings. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 280 +++-- 1 file changed, 156 insertions(+), 124 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts

[PATCH] checkkconfigsymbols.py: avoid shell injection

2016-08-27 Thread Valentin Rothberg
Use subprocess and set shell to False to avoid potential shell injections. Reported-by: Bernd Dietzel Signed-off-by: Valentin Rothberg --- Note that I don't see how it could be exploited currently. There is no user input used in the execute() function. I see the patch more as a preventiv

[PATCH] lib/Kconfig.debug: fix typo in select statement

2016-08-20 Thread Valentin Rothberg
Commit 484f29c7430b3 ("bug: Provide toggle for BUG on data corruption") added a Kconfig select statement on CONFIG_DEBUG_LIST, but the CONFIG_ prefix is only used in Make and C(PP) syntax. Remove the CONFIG_ prefix to correctly select the Kconfig option DEBUG_LIST. Signed-off-by

Re: [PATCH v2] checkkconfigsymbols.py: add --no-color option, don't print color to non-TTY

2016-08-16 Thread Valentin Rothberg
s to disable colour > output for whatever reason. > > Signed-off-by: Andrew Donnellan Acked-by: Valentin Rothberg > --- > > V1->V2: > - automatically detect non-TTYs and disable colour output. > Suggested by Josh Triplett > --- > scripts/check

[PATCH] drivers/iio/light/Kconfig: SENSORS_BH1780 cleanup

2016-08-15 Thread Valentin Rothberg
Commit 7ef9153d9af5 ("misc: delete bh1780 driver") has removed the Kconfig option SENSORS_BH1780. Remove the last reference on this option. Signed-off-by: Valentin Rothberg --- I found the issue using scripts/checkkconfigsymbols.py drivers/iio/light/Kconfig | 1 - 1 file changed,

Re: [PATCH v2] checkkconfigsymbols.py: add --no-color option, don't print color to non-TTY

2016-07-05 Thread Valentin Rothberg
Hi Andrew, I am fine with this version of the patch, thanks. Usually patches to this script go through Greg's misc tree. Best regards, Valentin On Tue, Jul 5, 2016 at 9:47 AM, Andrew Donnellan wrote: > Only print the ANSI colour escape codes if stdout is a TTY. Useful if > redirecting output

Re: [PATCH] checkkconfigsymbols.py: add --no-color option

2016-07-04 Thread Valentin Rothberg
e > output. Useful if redirecting output to a file or piping to another script. > > (It should really be --no-colour, but I'll accept US spelling in the name > of consistency with existing usage in the code comments...) > > Signed-off-by: Andrew Donnellan Acked-

fou: dead build rule for fou6.o

2016-05-30 Thread Valentin Rothberg
Hi Arnd, your commit fabb13db448e ("fou: add Kconfig options for IPv6 support") has shown up in today's linux-next. ./scripts/checkkconfigsymbols.py warned because of the following statement in net/ipv6/Makefile: +obj-$(CONFIG_NET_FOU_IPV6_TUNNELS) += fou6.o The referenced Kconfig option does no

[PATCH] powerpc/mm/radix: fix STD_MMU_64 typo

2016-05-03 Thread Valentin Rothberg
s/MMU_STD_64/STD_MMU_64/ Fixes: 11ffc1cfa4c2 ("powerpc/mm/radix: Use STD_MMU_64 to properly isolate hash related code") Signed-off-by: Valentin Rothberg --- I only compile tested this change. The issue has been detected with scripts/checkkconfigsymbols.py ar

[PATCH] arch:blackfin:adis162** cleanup

2016-04-06 Thread Valentin Rothberg
Remove dead #if blocks referencing previously removed Kconfig options. Signed-off-by: Valentin Rothberg --- arch/blackfin/mach-bf537/boards/stamp.c | 22 -- 1 file changed, 22 deletions(-) diff --git a/arch/blackfin/mach-bf537/boards/stamp.c b/arch/blackfin/mach-bf537

staging:iio:adis162** cleanup

2016-04-04 Thread Valentin Rothberg
Hi Lars-Peter, both of your commits 88ae3aedb8ec ("staging:iio:adis16204: Remove adis16204 driver") and 2bcdb3f2e05f ("staging:iio:adis16220: Remove adis16220 driver") have shown up in today's linux-next. The corresponding Kconfig options are still referenced in the __initdata struct in arch/blac

Re: [PATCH] checkkconfigsymbols.py: Fix typo in help message

2016-04-01 Thread Valentin Rothberg
On Mar 31 '16 09:24, Andreas Ziegler wrote: > Fix a typo in the help message for the -d parameter by removing one 'm'. > > Signed-off-by: Andreas Ziegler Acked-by: Valentin Rothberg > --- > scripts/checkkconfigsymbols.py | 2 +- > 1 file changed, 1 insertion(

[PATCH] drivers:block: cpqarray clean up

2016-03-15 Thread Valentin Rothberg
Commit d436641439e0 ("cpqarray: remove it from the kernel") removes the Kconfig option BLK_CPQ_DA and cpqarray. Remove the dead build rule in the Makefile. Signed-off-by: Valentin Rothberg --- Detected by ./scripts/checkkconfigsymbols.py (checkkconfigsymbols.py --diff next-2016

[PATCH] arc: SMP: CONFIG_ARC_IPI_DBG cleanup

2016-02-24 Thread Valentin Rothberg
Commit 0443ad7fb2eb ("ARC: SMP: No need for CONFIG_ARC_IPI_DBG") removed the Kconfig option ARC_IPI_DBG. Remove the last reference on this option. Signed-off-by: Valentin Rothberg --- arch/arc/kernel/mcip.c | 5 - 1 file changed, 5 deletions(-) diff --git a/arch/arc/kernel/mci

ARC: leftover reference on ARC_IPI_DBG

2016-02-23 Thread Valentin Rothberg
Hi Vineet, your commit 0443ad7fb2eb ("ARC: SMP: No need for CONFIG_ARC_IPI_DBG") has shown up in todays linux-next tree (i.e., next-20160224) and removes the Kconfig option ARC_IPI_DBG and all referencing #ifdefs but one: arch/arc/kernel/mcip.c:58:#ifdef CONFIG_ARC_IPI_DBG arch/arc/kernel/mcip.c-

Re: [PATCH] rtc: max77686: fix irqf_oneshot.cocci warnings

2016-02-18 Thread Valentin Rothberg
Hi Alexandre, On 2/18/16 9:51 AM, Alexandre Belloni wrote: > On 18/02/2016 at 17:13:18 +0900, Krzysztof Kozlowski wrote : >> On 18.02.2016 17:06, Valentin Rothberg wrote: >>> From: kbuild test robot >>> >>> Since commit 1c6c69525b40 ("genirq: Reject bo

Re: [PATCH] rtc: max77686: fix irqf_oneshot.cocci warnings

2016-02-18 Thread Valentin Rothberg
Hi Krzysztof, On 2/18/16 9:50 AM, Krzysztof Kozlowski wrote: > On 18.02.2016 17:46, Valentin Rothberg wrote: >> >> >> Hi Krzysztof, >> >> On 2/18/16 9:13 AM, Krzysztof Kozlowski wrote: >>> On 18.02.2016 17:06, Valentin Rothberg wrote: >>>

Re: [PATCH] rtc: max77686: fix irqf_oneshot.cocci warnings

2016-02-18 Thread Valentin Rothberg
Hi Krzysztof, On 2/18/16 9:13 AM, Krzysztof Kozlowski wrote: > On 18.02.2016 17:06, Valentin Rothberg wrote: >> From: kbuild test robot >> >> Since commit 1c6c69525b40 ("genirq: Reject bogus threaded irq requests") >> threaded IRQs without a prim

[PATCH] rtc: max77686: fix irqf_oneshot.cocci warnings

2016-02-18 Thread Valentin Rothberg
coccinelle/misc/irqf_oneshot.cocci CC: Laxman Dewangan Signed-off-by: Fengguang Wu Signed-off-by: Valentin Rothberg --- drivers/rtc/rtc-max77686.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/rtc/rtc-max77686.c b/drivers/rtc/rtc-max77686.c index 5e924f3cde90..02

[PATCH] firmware: qemu_fw_cfg.c: fix typo FW_CFG_DATA_OFF

2016-02-11 Thread Valentin Rothberg
s/FW_CTRL_DATA_OFF/FW_CFG_DATA_OFF/ Signed-off-by: Valentin Rothberg Signed-off-by: Anreads Ziegler --- drivers/firmware/qemu_fw_cfg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/firmware/qemu_fw_cfg.c b/drivers/firmware/qemu_fw_cfg.c index 19f6851be87f

[PATCH v2] firmware: qemu_fw_cfg.c: fix typo FW_CFG_DATA_OFF

2016-02-11 Thread Valentin Rothberg
s/FW_CTRL_DATA_OFF/FW_CFG_DATA_OFF/ Signed-off-by: Valentin Rothberg Signed-off-by: Andreas Ziegler --- v2: corrected typo in signed-off-by drivers/firmware/qemu_fw_cfg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/firmware/qemu_fw_cfg.c b/drivers/firmware

Re: [PATCH] arch/powerpc/Kconfig: fix typo in select statement

2015-12-11 Thread Valentin Rothberg
Hi Daniel, thank you for the links and the explanation. On Dec 11 '15 19:13, Daniel Axtens wrote: > Thanks Valentin. > > Ironically I first noticed the spelling mistake in the original UBSAN > patchset[1], but wanted it to work on powerpc straight away so I went > with the mis-spelled version. I

[PATCH] arch/powerpc/Kconfig: fix typo in select statement

2015-12-10 Thread Valentin Rothberg
The 'I' in sanitize was missing: s/ARCH_HAS_UBSAN_SANTIZE_ALL/ARCH_HAS_UBSAN_SANITIZE_ALL/ Fixes: 257e4ee82dbd ("powerpc: enable UBSAN support") Signed-off-by: Valentin Rothberg --- Detected with ./scripts/checkkconfigsymbols.py arch/powerpc/Kconfig | 2 +- 1 file change

[PATCH] iio: exynos-adc: fix irqf_oneshot.cocci warnings

2015-12-02 Thread Valentin Rothberg
ci Signed-off-by: Fengguang Wu Signed-off-by: Valentin Rothberg --- The patch looks fine to me: bad things can happen when interrupts are acked before the handler ran. exynos_adc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/iio/adc/exynos_adc.c +++ b/drive

kernel: stop_machine: report (un)dead code (and feedback request)

2015-11-26 Thread Valentin Rothberg
Hi Chris, your commit 4c477de14237 ("kernel: remove stop_machine() Kconfig dependency") has shown up in today's linux-next tree (20151126). The commit changes the #ifdef condition in kernel/stop_machine.c from/to: -#if defined(CONFIG_STOP_MACHINE) && defined(CONFIG_SMP) +#if defined(CONFI

[PATCH] checkkconfigsymbols.py: find similar symbols

2015-11-26 Thread Valentin Rothberg
OMMIT_1_MESSAGE") - $COMMIT_2_HASH ("$COMMIT_2_MESSAGE") or - no commit found Signed-off-by: Valentin Rothberg --- This patch bases on a previously sent patch [1] that will soon make it to linux-next but currently resides in Greg's patch queue. If you want to test this

Re: [PATCH] checkkconfigsymbols.py: multiprocessing of files

2015-11-20 Thread Valentin Rothberg
Hi Greg, On Nov 01 '15 19:13, Greg KH wrote: [...] > I'll queue this up after 4.4-rc1 is out, sorry for the delay, not your > fault. Can you queue the patch for checkkconfigsymbols.py? I would like to send another patch adding support to find similar symbols, and apply it ontop of the multi-proc

clk: tango4: undefined CONFIG_ARCH_TANGOX

2015-11-20 Thread Valentin Rothberg
Hi Marc, your commit ed12dfc92f01 ("clk: tango4: clkgen driver for Tango4 platforms") has shown up in today's linux-next tree (i.e., next-20151120) adding the following build condition to the tango4 clk driver: drivers/clk/Makefile:45:obj-$(CONFIG_ARCH_TANGOX) += clk-tango4.o However, ARCH_TANGO

[PATCH] HAVE_DMA_ATTRS: remove leftovers in Kconfig

2015-11-18 Thread Valentin Rothberg
The Kconfig option HAVE_DMA_ATTRS has been removed by commit f6f306eeb7ea ("dma-mapping: always provide the dma_map_ops based implementation"). Remove leftover references on this option. Signed-off-by: Valentin Rothberg --- arch/s390/Kconfig | 1 - drivers/gpu/drm/vc4/K

dma-mappig: HAVE_DMA_ATTR leftovers

2015-11-18 Thread Valentin Rothberg
Hi Christoph, your commit f6f306eeb7ea ("dma-mapping: always provide the dma_map_ops based implementation") has shown up in today's linux-next tree (i.e., next-20151119). The commit removes the Kconfig option HAVE_DMA_ATTR and does some related clean-ups, but it leaves two references in Kconfig u

[PATCH v2] wireless:b43: remove last reference on B43_PCMCIA

2015-11-02 Thread Valentin Rothberg
Remove last reference on Kconfig option B43_PCMCIA, which has been removed by commit 399500da18f7 ("ssb: pick PCMCIA host code support from b43 driver"). Signed-off-by: Valentin Rothberg --- drivers/net/wireless/b43/main.c | 13 - 1 file changed, 4 insertions(+), 9

Re: [PATCH] wireless:b43: remove last reference on B43_PCMCIA

2015-11-02 Thread Valentin Rothberg
On Nov 02 '15 10:31, Rafał Miłecki wrote: > On 2 November 2015 at 10:24, Valentin Rothberg > wrote: > > Remove last reference on Kconfig option B43_PCMCIA, which has been > > removed by commit 399500da18f7 ("ssb: pick PCMCIA host code support from > > b43 driver&

[PATCH] wireless:b43: remove last reference on B43_PCMCIA

2015-11-02 Thread Valentin Rothberg
Remove last reference on Kconfig option B43_PCMCIA, which has been removed by commit 399500da18f7 ("ssb: pick PCMCIA host code support from b43 driver"). Signed-off-by: Valentin Rothberg --- drivers/net/wireless/b43/main.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/d

Documentation: update of USB_CHIPIDEA_DEBUG

2015-11-02 Thread Valentin Rothberg
Hi Peter, your commit 2fb2884ed856 ("usb: chipidea: delete static debug support") has shown up in today's Linux next tree (i.e., next-20151102) and removes the Kconfig option USB_CHIPIDEA_DEBUG in favor of dynamic debug support. However, there are some documentary references on this option still

b43: removal of B43_PCMCIA

2015-11-02 Thread Valentin Rothberg
Hi Rafal, your commit 399500da18f7 ("ssb: pick PCMCIA host code support from b43 driver") has shown up in today's Linux next tree (i.e., next-20151102) and removes the Kconfig option B43_PCMCIA and references on it in the code. However the commit misses to remove the following reference: drivers

Re: [PATCH] checkkconfigsymbols.py: multiprocessing of files

2015-11-01 Thread Valentin Rothberg
On Nov 01 '15 19:13, Greg KH wrote: > On Sun, Nov 01, 2015 at 09:56:18PM +0100, Valentin Rothberg wrote: > > On Nov 01 '15 11:45, Greg KH wrote: > > > On Sun, Nov 01, 2015 at 07:30:14PM +0100, Valentin Rothberg wrote: > > > > On Nov 01 '15 10:13, Greg K

Re: [PATCH] checkkconfigsymbols.py: multiprocessing of files

2015-11-01 Thread Valentin Rothberg
On Nov 01 '15 11:45, Greg KH wrote: > On Sun, Nov 01, 2015 at 07:30:14PM +0100, Valentin Rothberg wrote: > > On Nov 01 '15 10:13, Greg KH wrote: > > > On Sun, Nov 01, 2015 at 01:53:40PM +0100, Valentin Rothberg wrote: > > > > Hi Greg, > > > >

Re: [PATCH] checkkconfigsymbols.py: multiprocessing of files

2015-11-01 Thread Valentin Rothberg
On Nov 01 '15 10:13, Greg KH wrote: > On Sun, Nov 01, 2015 at 01:53:40PM +0100, Valentin Rothberg wrote: > > Hi Greg, > > > > On Oct 15 '15 10:37, Valentin Rothberg wrote: > > > > Can you take the patch below? I use the patched version of the script &

Re: [PATCH] checkkconfigsymbols.py: multiprocessing of files

2015-11-01 Thread Valentin Rothberg
Hi Greg, On Oct 15 '15 10:37, Valentin Rothberg wrote: Can you take the patch below? I use the patched version of the script since sending it and it has proved to be much faster and hence more comfortable to use since then. Kind regards, Valentin > Distribute the parsing of so

Re: [tpmdd-devel] tpm, tpm_tis: fix tpm_tis ACPI detection issue with TPM 2.0

2015-10-23 Thread Valentin Rothberg
On Oct 21 '15 18:58, Jarkko Sakkinen wrote: > On Tue, Oct 20, 2015 at 05:58:35PM +0300, Jarkko Sakkinen wrote: > > On Tue, Oct 20, 2015 at 01:49:02PM +0200, Andreas Ziegler wrote: > > > Hi Jarkko, > > > > > > your patch "tpm, tpm_tis: fix tpm_tis ACPI detection issue with TPM 2.0" > > > showed up

[PATCH] checkkconfigsymbols.py: multiprocessing of files

2015-10-15 Thread Valentin Rothberg
Distribute the parsing of source and Kconfig files on all available cores to speed up processing. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 128 - 1 file changed, 102 insertions(+), 26 deletions(-) diff --git a/scripts

[PATCH] Documentation: locktorture.txt: corret MUTEXES typo

2015-10-07 Thread Valentin Rothberg
s/CONFIG_RT_MUTEX/CONFIG_RT_MUTEXES/ Signed-off-by: Valentin Rothberg --- I detected the typo with scripts/checkkconfigsymbols.py Documentation/locking/locktorture.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/locking/locktorture.txt b/Documentation

Re: [PATCH v3] kbuild: document recursive dependency limitation / resolution

2015-10-06 Thread Valentin Rothberg
Hi Paul, On Oct 06 '15 10:32, Paul Bolle wrote: > Hi Valentin, > > On di, 2015-10-06 at 10:19 +0200, Valentin Rothberg wrote: > > I think that a general remark that using selects should be discouraged > > as, besides causing the recursive issue, selects can also break

Re: [PATCH v3] kbuild: document recursive dependency limitation / resolution

2015-10-06 Thread Valentin Rothberg
On Oct 06 '15 01:03, Luis R. Rodriguez wrote: > On Sun, Oct 04, 2015 at 03:42:47PM +0200, Valentin Rothberg wrote: > > Hi Luis, > > > > I finally found some time to read your patch. Thanks for doing this > > work in such great detail. > > > > What I

Re: [PATCH v3] kbuild: document recursive dependency limitation / resolution

2015-10-04 Thread Valentin Rothberg
Hi Luis, I finally found some time to read your patch. Thanks for doing this work in such great detail. What I miss in the text is a general discussion of the widespread use of selects. In my opinion, selects should be (like gotos) considered harmful: First, selects ignore the user selection a

CRIS v32: leftover references on ETRAX_VIRTUAL_GPIO

2015-10-01 Thread Valentin Rothberg
Hi Rabin, your commit ("CRIS v32: remove old GPIO and LEDs code") is in today's linux-next tree (i.e., 20151002). Among other Kconfig options, the commit removes ETRAX_VIRTUAL_GPIO but leaves the following references in the code: 0 arch/cris/arch-v32/kernel/setup.c 132 #ifdef CONFIG_ETRAX_

Re: compaction.h: undefined CONFIG_ZONE_HIGHMEM_

2015-09-23 Thread Valentin Rothberg
On Sep 23 '15 11:34, Vlastimil Babka wrote: > On 09/23/2015 10:43 AM, Valentin Rothberg wrote: > > Hi Vlastimil, > > > > your commit 1434c81a47e3 ("mm, compaction: export tracepoints zone names > > to userspace") has shown up in todays linux-next tree

compaction.h: undefined CONFIG_ZONE_HIGHMEM_

2015-09-23 Thread Valentin Rothberg
Hi Vlastimil, your commit 1434c81a47e3 ("mm, compaction: export tracepoints zone names to userspace") has shown up in todays linux-next tree (i.e., 20150923) adding the following lines of code: --- a/include/trace/events/compaction.h +++ b/include/trace/events/compaction.h @@ -18,6 +18,31 @@ [...

Re: drivers/power/wm831x_power.c:569:7-27: ERROR: Threaded IRQ with no primary handler requested without IRQF_ONESHOT

2015-09-22 Thread Valentin Rothberg
and I couldn't figure out a relation between you and the driver, but somehow I trusted the report - a machine must know better :) Sorry for making the noise. I CCed Anton, who signed with Mark the responsible commit. Best regards, Valentin > Thanks, > Sasha > > On 09/22/2015 01:

Re: drivers/power/wm831x_power.c:569:7-27: ERROR: Threaded IRQ with no primary handler requested without IRQF_ONESHOT

2015-09-22 Thread Valentin Rothberg
n Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation >From bceb7cace76aba9212db56be939bfecaacb75bc2 Mon Sep 17 00:00:00 2001 From: Valentin Rothberg Date: Tue, 22 Sep 2015 19:00:40 +0200 Subject: [PATCH] wm831x_power: Use IRQF_ONESHOT to re

spi/bcm63xx: unknown CONFIG_BIG_ENDIAN

2015-09-14 Thread Valentin Rothberg
Hi Jonas, your commit 158fcc4e050a ("spi/bcm63xx: replace custom io accessors with standard ones") has shown up in today's Linux next tree (i.e., next-20150915). This commit adds four #ifdef blocks depending on CONFIG_BIG_ENDIAN, which is not defined in Kconfig breaking the #ifdef #else conditi

[PATCH] openswitch: fix typo CONFIG_NF_CONNTRACK_LABEL

2015-08-28 Thread Valentin Rothberg
Fix typo in conntrack.c s/CONFIG_NF_CONNTRACK_LABEL/CONFIG_NF_CONNTRACK_LABELS/ Signed-off-by: Valentin Rothberg --- The typo was added by commmit c2ac66735870 ("openvswitch: Allow matching on conntrack label"). I detected the issue scripts/checkkconfigsymbols.py net/openvswitch/c

[PATCH] lib/Makefile: remove CONFIG_AVERAGE build rule

2015-08-26 Thread Valentin Rothberg
The Kconfig option AVERAGE and its implementation has been removed by commit f4e774f55fe0 ("average: remove out-of-line implementation"). Remove the dead build rule in lib/Makefile. Signed-off-by: Valentin Rothberg Reviewed-by: Johannes Berg --- I detected the issue wi

[PATCH] drivers/nfc/s3fwrn5/Makefile: remove superfluous cflags

2015-08-25 Thread Valentin Rothberg
NFC_DEBUG is not defined in Kconfig and since DEBUG is not used anywhere in this directory, we can safely remove this line. Signed-off-by: Valentin Rothberg --- I detected the issue with scripts/checkkconfigsymbols.py drivers/nfc/s3fwrn5/Makefile | 2 -- 1 file changed, 2 deletions(-) diff

Re: [PATCH v2] scripts/checkkconfigsymbols.py: support default statements

2015-08-24 Thread Valentin Rothberg
Hi Michal, On Mon, Aug 24, 2015 at 4:49 PM, Michal Marek wrote: > On 2015-07-27 12:33, Valentin Rothberg wrote: >> Until now, checkkonfigsymbols.py did not check default statements for >> references on missing Kconfig symbols (i.e., undefined Kconfig options). >> Hence, add

Re: soc/tegra: fuse: undefined option ARCH_TEGRA_210_SOC

2015-08-21 Thread Valentin Rothberg
Hi Thierry, this is just a kind reminder that the issue (see below) is still present. On Fri, May 15, 2015 at 10:57 AM, Thierry Reding wrote: > On Thu, May 14, 2015 at 01:50:08PM +0200, Valentin Rothberg wrote: >> Hi Thierry, >> >> your commit e3cf5bf687df ("soc

Re: mips: MT: ptrace.c: undefined MIPS_MT_SMTC

2015-08-21 Thread Valentin Rothberg
On Fri, Jul 31, 2015 at 8:42 PM, Valentin Rothberg wrote: > Hi Ralf, > > your commit 23c59ee7e541 ("MIPS: Add uprobes support.") is in today's > linux-next tree (i.e., next-20150731) and it adds the following lines > of code to arch/mips/kernel/ptrace.c: &

[PATCH] lib/Makefile: remove CONFIG_AVERAGE build rule

2015-08-21 Thread Valentin Rothberg
The Kconfig option AVERAGE and its implementation has been removed by commit f4e774f55fe0 ("average: remove out-of-line implementation"). Remove the dead build rule in lib/Makefile. Signed-off-by: Valentin Rothberg --- I detected the issue with scripts/checkkconfigsymbols.py lib/Ma

  1   2   3   >