[PATCH] nvme: Makefile: remove dead build rule

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 <vrothb...@suse.com> --- drivers/nvme/host/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/nvme/host/Ma

[PATCH] nvme: Makefile: remove dead build rule

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

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,

rcu: NO_HZ_FULL_SYSIDLE leftovers

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,

[PATCH] exofs: ORE: remove dead dependence

Remove last reference on PNFS_OBJLAYOUT. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- 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

[PATCH] exofs: ORE: remove dead dependence

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

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

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

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

[PATCH] tracing: fix DEBUG_PREEMPT typo

s/PREEMPT_DEBUG/DEBUG_PREEMPT/ Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- 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/i

[PATCH] tracing: fix DEBUG_PREEMPT typo

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

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

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

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

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

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. > > > >

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

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. > > > >

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

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,

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

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,

Re: mm: testcases for RODATA: renamed Kconfig option

newbie... > > Thanks. > Jinbum Park. > > > 2017. 2. 8. 오후 6:34에 "Valentin Rothberg" <valentinrothb...@gmail.com>님이 작성: > > Hi Jinbum, > > your commit bf9058f08772 ("mm: add arch-independent testcases for > RODATA") has shown up in today

Re: mm: testcases for RODATA: renamed Kconfig option

gt; 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 > another co

mm: testcases for RODATA: renamed Kconfig option

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

mm: testcases for RODATA: renamed Kconfig option

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

net: dsa: remaining references on NET_DSA_HWMON

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:

net: dsa: remaining references on NET_DSA_HWMON

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:

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

Hi Krzysztof, On 19/01/2017 10:20, Krzysztof Kozlowski wrote: > On Thu, Jan 19, 2017 at 11:18 AM, Krzysztof Kozlowski <k...@kernel.org> wrote: >> On Thu, Jan 19, 2017 at 11:11 AM, Valentin Rothberg >> <valentinrothb...@gmail.com> wrote: >>> Commit 22dfab102

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

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

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 <valetinrothb...@gmail.com> --- Detected by

[PATCH] ARM: Exynos4415: remove dead build rule

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

Don't sort the list of string-similar Kconfig symbols alphabetically to preserve the correct order of string similarity. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- scripts/checkkconfigsymbols.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

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

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.py b

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

Add the _APPARMOR substring to reference the intended Kconfig option. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- Detected by using ./scripts/checkkconfigsymbols.py (--diff on linux-next) security/apparmor/lsm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

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

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

The Kconfig option for OR12000 is OR1K_1200. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- 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..669887

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

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

s/UPROBE_EVENTS/UPROBE_EVENT/ Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- 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 b2e650

[PATCH] sparc64_defconfig: correctly set user probes

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

[PATCH] ARM: multi_v7_defconfig: set bcm47xx watchdog

Correct the bcm47xx watchdog option. The convention of bcm watchdogs is the _WDT suffix. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- Detected by running scripts/checkkconfigsymbols.py arch/arm/configs/multi_v7_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 de

[PATCH] ARM: multi_v7_defconfig: set bcm47xx watchdog

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

s/CONFIG_CONFIG_BCM47XX_NVRAM/CONFIG_BCM47XX_NVRAM/ Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- 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/c

[PATCH] ARM: multi_v7_defconfig: fix config typo

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

Hi Paul, On Thu, Nov 17, 2016 at 12:55 PM, Paul Bolle <pebo...@tiscali.nl> 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

Re: i2c: undefined option I2C_ALGO_BUSCLEAR

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

> 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 selects I2C_

Re: i2c: undefined option I2C_ALGO_BUSCLEAR

> 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 selects I2C_

i2c: undefined option I2C_ALGO_BUSCLEAR

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

i2c: undefined option I2C_ALGO_BUSCLEAR

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

[PATCH] ASoC: samsung: Makefile cleanup

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 <valentinrothb...@gmail.com

[PATCH] ASoC: samsung: Makefile cleanup

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

Support git's "^" syntax for diffing two commits, for instance via "--diff HEAD^^^..HEAD". Signed-off-by: Michael Ellermann <m...@ellerman.id.au> Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- scripts/checkkconfigsymbols.py | 2 +- 1 file cha

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

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/chec

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

On Wed, Oct 26, 2016 at 12:40 PM, Michael Ellerman <m...@ellerman.id.au> wrote: > Valentin Rothberg <valentinrothb...@gmail.com> writes: > >> On Wed, Oct 26, 2016 at 7:52 AM, Michael Ellerman <m...@ellerman.id.au> >> wrote: >>> Cyril Bur <cyril..

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

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

On Wed, Oct 26, 2016 at 7:52 AM, Michael Ellerman <m...@ellerman.id.au> wrote: > Cyril Bur <cyril...@gmail.com> writes: > >> On Wed, 2016-10-05 at 07:57 +0200, Valentin Rothberg wrote: >>> s/ALIVEC/ALTIVEC/ >>> >> >> Oops, nice catch

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

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

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

drm/i915: undefined STOP_MACHINE

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

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

s/ALIVEC/ALTIVEC/ Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- 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 ---

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

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

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

samples: blackfin: dead config options

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

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

The Kconfig symbol for 32bit ARM is 'ARM', not 'ARM32'. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- 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

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

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 b63fd72295c6..608b1f608e13

Re: staging/greybus: undefined Kconfig symbols

On Wed, Sep 21, 2016 at 9:39 AM, Greg KH <gre...@linuxfoundation.org> 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 greyb

Re: staging/greybus: undefined Kconfig symbols

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. >> checkkconfigsymbols.py

staging/greybus: undefined Kconfig symbols

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:

staging/greybus: undefined Kconfig symbols

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:

drm/i915: undefined symbol I915_SW_FENCE_CHECK_DAG

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

drm/i915: undefined symbol I915_SW_FENCE_CHECK_DAG

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

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

Fix pylint and pep8 warnings to have a consistent syntax and style. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- scripts/checkkconfigsymbols.py | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/s

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

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

'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 <valentinrothb...@gmail.com> --- scripts/checkkconfigsymb

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

Python 2 is slowly dying, so port the script to Python 3. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- scripts/checkkconfigsymbols.py | 33 + 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/scripts/checkkconfigsymbol

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

'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 --- scripts/checkkconfigsymbols.py | 134

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

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

Replace the deprecated OptionParser with ArgumentParser, as recommended by pylint. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- scripts/checkkconfigsymbols.py | 142 - 1 file changed, 69 insertions(+), 73 deletions(-) diff

[PATCH v2 2/5] checkkconfigsymbols: use ArgumentParser

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

, 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 terminology

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

Splitting a command string could lead to unintended arguments. Use an argument list in the execute() function instead. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- scripts/checkkconfigsymbols.py | 27 --- 1 file changed, 16 insertions(

[PATCH v2 0/5] checkkconfigsymbols: several updates

, 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 terminology

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

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

On Sat, Aug 27, 2016 at 7:53 PM, Greg KH <gre...@linuxfoundation.org> 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

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

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

Make the term 'symbol' consitent in the entire scripts. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- scripts/checkkconfigsymbols.py | 134 - 1 file changed, 67 insertions(+), 67 deletions(-) diff --git a/scripts/checkkconfigsymb

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

Splitting a command string could lead to unintended arguments. Use an argument list in the execute() function instead. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- scripts/checkkconfigsymbols.py | 29 ++--- 1 file changed, 18 insertions(

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

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 b/scripts

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

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

Port the script to python3 and fix some pylint warnings. Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- scripts/checkkconfigsymbols.py | 280 +++-- 1 file changed, 156 insertions(+), 124 deletions(-) diff --git a/s

[PATCH 1/3] checkkconfigsymbols: port to python3

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

Use subprocess and set shell to False to avoid potential shell injections. Reported-by: Bernd Dietzel <tc...@t-online.de> Signed-off-by: Valentin Rothberg <valentinrothb...@gmail.com> --- Note that I don't see how it could be exploited currently. There is no user input used in

[PATCH] checkkconfigsymbols.py: avoid shell injection

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 preventive fix

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

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

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

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

option, --no-color, if the user wants to disable colour > output for whatever reason. > > Signed-off-by: Andrew Donnellan <andrew.donnel...@au1.ibm.com> Acked-by: Valentin Rothberg <valentinrothb...@gmail.com> > --- > > V1->V2: > - a

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

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

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 <valentinrothb...@gmail.com> --- I found the issue using scripts/checkkconfigsymbols.py drivers/iio/l

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

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

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.

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

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

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 <andrew.donnel...@au1.ibm.com>

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

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-by: Valentin Rothber

fou: dead build rule for fou6.o

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

fou: dead build rule for fou6.o

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

  1   2   3   4   5   6   >