Re: MIPS: BMIPS: Enable GZIP ramdisk and timed printks

2015-11-12 Thread Andreas Ziegler
Hi Florian, your patch "MIPS: BMIPS: Enable GZIP ramdisk and timed printks" showed up as commit fb9e5642aa9e in linux-next today (that is, next-20151112). I noticed it because we (a research group from Erlangen[0]) are running daily checks on linux-next. Your commit adds two entries to

Re: tpm, tpm_tis: fix tpm_tis ACPI detection issue with TPM 2.0

2015-10-20 Thread Andreas Ziegler
Hi Jarkko, your patch "tpm, tpm_tis: fix tpm_tis ACPI detection issue with TPM 2.0" showed up as commit 399235dc6e95 in linux-next today (that is, next-20151020). I noticed it because we (a research group from Erlangen[0]) are running daily checks on linux-next. Your commit creates the following

Re: spi/bcm63xx: fix standard accessors and compile guard

2015-10-13 Thread Andreas Ziegler
Hi Jonas, your patch "spi/bcm63xx: fix standard accessors and compile guard" showed up as commit 682b5280bf00 in linux-next today (that is, next-20151013). I noticed it because we (a research group from Erlangen[0]) are running daily checks on linux-next. Your commit fixes two #ifdef statements

Re: parisc: adjust L1_CACHE_BYTES to 128 bytes on PA8800 and PA8900 CPUs

2015-09-03 Thread Andreas Ziegler
Hi Helge, today's linux-next tree (next-20150903) contains commit 20f924902ff6 ("parisc: adjust L1_CACHE_BYTES to 128 bytes on PA8800 and PA8900 CPUs") which you authored. I noticed it because we[0] are running a daily analysis on all commits in linux-next as part of our research and our tools

Re: Weird Kconfig condition (IPV6 [=n] || IPV6 [=n]=n)

2015-09-14 Thread Andreas Ziegler
Hi, On 09/14/2015 10:41, Mason wrote: > Hello, > > I did make menuconfig in v4.2 > > In the help for CONFIG_CRYPTO_HMAC, I have > > │ Selected by: IP_SCTP [=n] && NET [=y] && INET [=y] && (IPV6 [=n] || > IPV6 [=n]=n) || INET_AH [=n] && NET [=y] && INET [=y] || \ │ > │

[PATCH] fsl/qe: Do not prefix header guard with CONFIG_

2016-06-08 Thread Andreas Ziegler
The CONFIG_ prefix should only be used for options which can be configured through Kconfig and not for guarding headers. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- include/soc/fsl/qe/qe_tdm.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/s

[PATCH] drivers/net/fsl_ucc: Do not prefix header guard with CONFIG_

2016-06-08 Thread Andreas Ziegler
The CONFIG_ prefix should only be used for options which can be configured through Kconfig and not for guarding headers. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- drivers/net/wan/fsl_ucc_hdlc.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drive

select on non-existing Kconfig option CRC32C

2016-06-15 Thread Andreas Ziegler
Hi Hendrik, your patch "s390/crc32-vx: add crypto API module for optimized CRC-32 algorithms" showed up in linux-next today (next-20160615) as commit 364148e0b195. The patch defines the Kconfig option CRYPTO_CRC32_S390 which 'select's CRC32C. However, this should probably have been

[PATCH] mm: Fix two typos in comments for to_vmem_altmap()

2016-01-25 Thread Andreas Ziegler
p. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- kernel/memremap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/memremap.c b/kernel/memremap.c index e517a16..2e84aa8 100644 --- a/kernel/memremap.c +++ b/kernel/memremap.c @@ -377,7 +377,7 @@ st

[PATCH] drm/i915: Remove select to deleted STOP_MACHINE from Kconfig

2016-01-25 Thread Andreas Ziegler
Commit 86fffe4a61dd ("kernel: remove stop_machine() Kconfig dependency") removed the option STOP_MACHINE from init/Kconfig, but missed a select to this option from the DRM_I915 option. Remove this now obsolete select statement. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de&

Select on now-gone Kconfig option STOP_MACHINE

2016-01-25 Thread Andreas Ziegler
Hi Chris, your commit 5bab6f60cb4d ("drm/i915: Serialise updates to GGTT with access through GGTT on Braswell") added a select to STOP_MACHINE to the DRM_I915 Kconfig option. In 4.5-rc1, your commit 86fffe4a61dd ("kernel: remove stop_machine() Kconfig dependency") then removed STOP_MACHINE

[PATCH v2] drm/i915: Remove select to deleted STOP_MACHINE from Kconfig

2016-01-25 Thread Andreas Ziegler
option STOP_MACHINE from init/Kconfig and ensured that stop_machine() universally works. Due to the order in which the patches were applied, removing the select from DRM_I915 got lost during merging. Remove the now obsolete select statement. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau

[PATCH] cgroup: Fix misspelling of CONFIG_SOCK_CGROUP_DATA in comments

2016-01-26 Thread Andreas Ziegler
. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- include/linux/cgroup.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/cgroup.h b/include/linux/cgroup.h index 2162dca..48ce12f 100644 --- a/include/linux/cgroup.h +++ b/include/linux/cgroup.h

Use of deleted Kconfig option B43_PCMCIA

2016-01-26 Thread Andreas Ziegler
Hi Rafał, your commit 399500da18f7 ("ssb: pick PCMCIA host code support from b43 driver") removed the Kconfig option B43_PCMCIA from Kconfig, but left one reference to it in b43/main.c, inside the b43_print_driverinfo() function. Can this #ifdef be removed, should it be replaced with an #ifdef

Re: [PATCH] cgroup: Fix misspelling of CONFIG_SOCK_CGROUP_DATA in comments

2016-01-26 Thread Andreas Ziegler
On 01/26/2016 16:08, Rosen, Rami wrote: > Hi, > > --- a/include/linux/cgroup.h > +++ b/include/linux/cgroup.h > @@ -604,11 +604,11 @@ static inline struct cgroup *sock_cgroup_ptr(struct > sock_cgroup_data *skcd) > #endif > } > > In this occasion, seems that maybe something else is also

'select' on deleted KEYS_DEBUG_PROC_KEYS option

2016-01-26 Thread Andreas Ziegler
Hi Dmitry, your commit f4dc37785e9b ("integrity: define '.evm' as a builtin 'trusted' keyring") introduces the INTEGRITY_TRUSTED_KEYRING Kconfig option, which has a 'select' to KEYS_DEBUG_PROC_KEYS. This option, however, has been removed over a year ago by commit dabd39cc2fb1 ("KEYS: Make

[PATCH] security: Remove select to deleted KEYS_DEBUG_PROC_KEYS option

2016-01-26 Thread Andreas Ziegler
f CONFIG_KEYS=y"). This patch removes the inoperative select. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- security/integrity/Kconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/security/integrity/Kconfig b/security/integrity/Kconfig index 21d7568..7543398 100644 --- a/

[PATCH] ARC: Don't source drivers/pci/pcie/Kconfig ourselves

2016-03-15 Thread Andreas Ziegler
lue used outside its choice group drivers/pci/pcie/Kconfig:74:warning: choice value used outside its choice group This change updates the Kconfig file for ARC, dropping the now unnecessary 'source' statement, which makes the warning disappear. Signed-off-by: Andreas Ziegler <andreas.zieg..

[PATCH] PCI: cleanup pci/pcie/Kconfig

2016-03-15 Thread Andreas Ziegler
This change cleans up style issues in drivers/pci/pcie/Kconfig, in particular all indentation is now done using tabs, not spaces, and the definition of PCIEASPM_DEBUG is now separated from the definition of PCIEASPM with a newline. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau

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

2016-03-31 Thread Andreas Ziegler
Fix a typo in the help message for the -d parameter by removing one 'm'. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- scripts/checkkconfigsymbols.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts/checkkconfigsymb

Left-over select to PUBLIC_KEY_ALGO_RSA

2016-03-04 Thread Andreas Ziegler
Hi David, your patch "akcipher: Move the RSA DER encoding check to the crypto layer" showed up in linux-next today as commit d43de6c780a8 (that is, next-20160304). It removes the CONFIG_PUBLIC_KEY_ALGO_RSA option, but does leave one reference to it in place inside security/integrity/Kconfig, in

[PATCH] security: integrity: Remove select to deleted option PUBLIC_KEY_ALGO_RSA

2016-03-04 Thread Andreas Ziegler
Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- security/integrity/Kconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/security/integrity/Kconfig b/security/integrity/Kconfig index 979be65..da95658 100644 --- a/security/integrity/Kconfig +++ b/security/integrity/Kconfig @@ -35,7

[PATCH] printk: Remove unnecessary #ifdef CONFIG_PRINTK

2016-08-04 Thread Andreas Ziegler
the newly introduced #ifdef is unnecessary. Let's remove it to avoid confusion. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- kernel/printk/printk.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index a5ef95c..a37fc8c 100644 ---

Re: [PATCHv2 0/7] eBPF JIT for PPC64

2016-06-30 Thread Andreas Ziegler
Hi Naveen, this patchset makes a change to arch/powerpc/net/Makefile in order to only compile the previously existing bpf_jit_comp.c if !CONFIG_PPC64, and use bpf_jit_comp64.c if CONFIG_PPC64 is enabled. Inside arch/powerpc/net/bpf_jit_comp.c, however, there is still an #ifdef CONFIG_PPC64 block

Re: [PATCH v3] module: extend 'rodata=off' boot cmdline parameter to module mappings

2016-11-28 Thread Andreas Ziegler
Hi Akashi, your patch "module: extend 'rodata=off' boot cmdline parameter to module mappings" showed up in linux-next today, and I noticed a small error in it. The first modified #ifdef is fine, the second one, however, has a spelling mistake in it: the CONFIG_ variable should be

Re: [PATCH v3] cpufreq: brcmstb-cpufreq: CPUfreq driver for older Broadcom STB SoCs

2016-11-28 Thread Andreas Ziegler
Forgot to CC linux-kernel and linux-pm in my first message. On 11/28/2016 10:28, Andreas Ziegler wrote: > Hi Markus, > > your patch "cpufreq: brcmstb-cpufreq: CPUfreq driver for older Broadcom STB > SoCs" showed up in linux-next today, and I noticed a small error in it. &

Re: [PATCH] regmap: make LZO cache optional

2017-06-08 Thread Andreas Ziegler
Hi Jonas, I noticed your patch 'regmap: make LZO cache optional' as it recently showed up in linux-next. In your patch, you modify drivers/base/regmap/regcache.c by adding an #if IS_ENABLED() statement. However, this statement contains a spelling error, as it references REGCHACHE_COMPRESSED