Re: [PATCH V11 10/10] arm/arm64: KVM: add guest SEA support

2017-02-28 Thread Baicar, Tyler
Hello James, On 2/24/2017 3:42 AM, James Morse wrote: On 21/02/17 21:22, Tyler Baicar wrote: Currently external aborts are unsupported by the guest abort handling. Add handling for SEAs so that the host kernel reports SEAs which occur in the guest kernel. diff --git a/arch/arm/include/asm/kvm_

[PATCH v2 0/3] perf annotate: Introduce the new source code view

2017-02-28 Thread Taeung Song
Hi, Currently perf-annotate have several problems. - Wrong line numbers on perf-annotate (both stdio and TUI) - Wrong sum of overhead(percent) matching source lines - It's so confusing that the output is mixed with both source code and assembly code. (related to depending on 'objdump -S

[PATCH v2 1/3] perf annotate: Get correct line numbers matched with addr

2017-02-28 Thread Taeung Song
Currently perf-annotate show wrong line numbers. For example, Actual source code is as below ... 21 }; 22 23 unsigned int limited_wgt; 24 25 unsigned int get_cond_maxprice(int wgt) 26 { ... However, the output of perf-annotate is as below. 4 Disassembly of section .text: 6

[PATCH v2 3/3] perf annotate: Support the new source code view for TUI

2017-02-28 Thread Taeung Song
Current source code view in TUI also has problems showing wrong line number and confusing mixed source code & dissambly view. So fix it with the new source code view. We can toggle the new source code view by a 's' key. Before: │11 void pack_knapsack(struct jewelry *jewelry) │12

[PATCH -next] watchdog/kempld_wdt: fix build for COMPILE_TEST

2017-02-28 Thread Randy Dunlap
(.text+0x19b1a4): undefined reference to `kempld_get_mutex' kempld_wdt.c:(.text+0x19b1bd): undefined reference to `kempld_release_mutex' Signed-off-by: Randy Dunlap Cc: Michael Brunner --- drivers/watchdog/Kconfig |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- linux-next-201702

Re: [PATCH v2 2/3] spi: allocate spi_board_info entries one by one

2017-02-28 Thread Mark Brown
On Tue, Feb 28, 2017 at 10:24:17AM -0800, Dmitry Torokhov wrote: > On Tue, Feb 28, 2017 at 09:16:50AM +, Mark Brown wrote: > > On Mon, Feb 27, 2017 at 08:18:56PM -0800, Dmitry Torokhov wrote: > > > Lists of spi_board_info entries registered with spi_register_board_info() > > > can be quite long

Re: [Outreachy kernel] Re: [PATCH 5/5] staging: gdm724x: Remove unnecessary else after return

2017-02-28 Thread Julia Lawall
On Tue, 28 Feb 2017, Joe Perches wrote: > On Wed, 2017-03-01 at 00:41 +0530, SIMRAN SINGHAL wrote: > > On Tue, Feb 28, 2017 at 1:49 AM, SIMRAN SINGHAL > > wrote: > > > On Tue, Feb 28, 2017 at 1:11 AM, Joe Perches wrote: > > > > On Mon, 2017-02-27 at 23:44 +0530, simran singhal wrote: > > > > >

[PATCH] staging/fbtft: Hush checkpatch.pl warning about unnecessary line continuations.

2017-02-28 Thread Alexander Kapshuk
Use a single fmt string with appropriate verbs as conversion specifiers, followed by the original string literals and the integer argument instead of using a backslash to escape a new line embedded inbetween quoted string literals passed as fmt arguments to dev_err() invoked in drivers/staging/fbtf

[PATCH v2 2/3] perf annotate: Introduce the new source code view

2017-02-28 Thread Taeung Song
The current source code view using 'objdump -S' has several limitations regarding line numbers of each soure code line and confusing mixed soure code & diassembly view. So not use the '-S' option any more and add the new reable source code view with correct line numbers and source code per each sy

Re: [PATCH 5/5] staging: gdm724x: Remove unnecessary else after return

2017-02-28 Thread SIMRAN SINGHAL
On Tue, Feb 28, 2017 at 1:49 AM, SIMRAN SINGHAL wrote: > On Tue, Feb 28, 2017 at 1:11 AM, Joe Perches wrote: >> On Mon, 2017-02-27 at 23:44 +0530, simran singhal wrote: >>> This patch fixes the checkpatch warning that else is not generally >>> useful after a break or return. >> >>> This was done

kvm: WARNING in nested_vmx_merge_msr_bitmap

2017-02-28 Thread Dmitry Vyukov
Hello, I've got the following WARNING while running syzkaller fuzzer: [ cut here ] WARNING: CPU: 0 PID: 7832 at arch/x86/kvm/vmx.c:9709 nested_vmx_merge_msr_bitmap arch/x86/kvm/vmx.c:9709 [inline] WARNING: CPU: 0 PID: 7832 at arch/x86/kvm/vmx.c:9709 nested_get_vmcs12_pages

Re: [PATCH v5 06/10] seccomp,landlock: Handle Landlock events per process hierarchy

2017-02-28 Thread Andy Lutomirski
On Tue, Feb 21, 2017 at 5:26 PM, Mickaël Salaün wrote: > The seccomp(2) syscall can be use to apply a Landlock rule to the > current process. As with a seccomp filter, the Landlock rule is enforced > for all its future children. An inherited rule tree can be updated > (append-only) by the owner of

Re: [Outreachy kernel] [PATCH 3/5] staging: rtl8712: Remove unnecessary else after return

2017-02-28 Thread SIMRAN SINGHAL
On Tue, Feb 28, 2017 at 2:49 AM, Julia Lawall wrote: > > > On Mon, 27 Feb 2017, simran singhal wrote: > >> This patch fixes the checkpatch warning that else is not generally >> useful after a break or return. >> >> This was done using Coccinelle: >> @@ >> expression e2; >> statement s1; >> @@ >> i

Re: [PATCH] jump_label: align jump_entry table to at least 4-bytes

2017-02-28 Thread David Daney
On 02/28/2017 11:34 AM, Jason Baron wrote: On 02/28/2017 02:22 PM, David Daney wrote: On 02/28/2017 11:05 AM, David Daney wrote: On 02/28/2017 10:39 AM, Jason Baron wrote: [...] I suspect that the alignment of the __jump_table section in the .ko files is not correct, and you are seeing so

Re: [PATCH v2 2/3] spi: allocate spi_board_info entries one by one

2017-02-28 Thread Geert Uytterhoeven
On Tue, Feb 28, 2017 at 7:54 PM, Mark Brown wrote: > On Tue, Feb 28, 2017 at 10:24:17AM -0800, Dmitry Torokhov wrote: >> On Tue, Feb 28, 2017 at 09:16:50AM +, Mark Brown wrote: >> > On Mon, Feb 27, 2017 at 08:18:56PM -0800, Dmitry Torokhov wrote: >> > > Lists of spi_board_info entries register

KASAN errors after 21c8e72037fb ("crypto: testmgr - use calculated count for number of test vectors")

2017-02-28 Thread Laura Abbott
Hi, While attempting to debug something else, I saw KASAN errors from the crypto test with ccm: [ 28.231615] == [ 28.232007] BUG: KASAN: global-out-of-bounds in __test_aead+0x9d9/0x2200 at addr 8212fca0 [ 28.232007] R

Re: [PATCH v4] reset: Add i.MX7 SRC reset driver

2017-02-28 Thread Andrey Smirnov
On Tue, Feb 28, 2017 at 6:38 AM, Philipp Zabel wrote: > On Tue, 2017-02-21 at 08:13 -0800, Andrey Smirnov wrote: >> Add reset controller driver exposing various reset faculties, >> implemented by System Reset Controller IP block. >> >> Cc: Lucas Stach >> Cc: Rob Herring >> Cc: Mark Rutland >> C

[PATCH -next] watchdog/wm831x: fix build when COMPILE_TEST is enabled

2017-02-28 Thread Randy Dunlap
watchdog/wm831x_wdt.ko] undefined! ERROR: "wm831x_set_bits" [drivers/watchdog/wm831x_wdt.ko] undefined! Signed-off-by: Randy Dunlap Cc: patc...@opensource.wolfsonmicro.com Cc: Mark Brown --- drivers/watchdog/Kconfig |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- linux-next-201

Re: [PATCH v4 0/3] Add Broadcom SPU Crypto Driver

2017-02-28 Thread Florian Fainelli
On 02/13/2017 07:11 AM, Jon Mason wrote: > On Sat, Feb 11, 2017 at 5:54 AM, Herbert Xu > wrote: >> On Fri, Feb 03, 2017 at 12:55:31PM -0500, Rob Rice wrote: >>> Changes in v4: >>> - Added Rob Herring's Acked-by to patch 1/3 for bindings doc >>> - In response to Herbert's comment, in ahash_export(

Re: [PATCH] soc: bcm: brcmstb: Match additional compatible strings

2017-02-28 Thread Florian Fainelli
On 02/14/2017 04:41 PM, Florian Fainelli wrote: > From: Florian Fainelli > > Match all known sun-top-ctrl compatible strings from our MIPS chips > counterparts. This allows us to properly report the SoC information to > user-space through our SoC driver. > > Signed-off-by: Florian Fainelli App

Re: [PATCH RESEND] drm/via: use get_user_pages_unlocked()

2017-02-28 Thread Lorenzo Stoakes
On 28 February 2017 at 19:35, Al Viro wrote: > On Tue, Feb 28, 2017 at 10:01:10AM +0100, Daniel Vetter wrote: > >> > + ret = get_user_pages_unlocked((unsigned long)xfer->mem_addr, >> > + vsg->num_pages, vsg->pages, >> > + (vsg->direction == DMA_FROM_DEVICE) ?

Re: [PATCH] staging/fbtft: Hush checkpatch.pl warning about unnecessary line continuations.

2017-02-28 Thread Joe Perches
On Tue, 2017-02-28 at 21:52 +0200, Alexander Kapshuk wrote: > Use a single fmt string with appropriate verbs as conversion specifiers, > followed by the original string literals and the integer argument > instead of using a backslash to escape a new line embedded inbetween > quoted string literals

Re: [PATCH 1/2] ARM: dts: BCM5301X: Add NAND entries to bcm953012k

2017-02-28 Thread Florian Fainelli
On 02/08/2017 12:45 PM, Jon Mason wrote: > Add the NAND entry in the DTS > > Signed-off-by: Jon Mason Applied, thanks! -- Florian

Re: [LSF/MM TOPIC] cpuset vs mempolicy related issues

2017-02-28 Thread Balbir Singh
On 03/02/17 20:17, Vlastimil Babka wrote: > Hi, > > this mail tries to summarize the problems with current cpusets implementation > wrt memory restrictions, especially when used together with mempolicies. > The issues were initially discovered when working on the series fixing recent > premature

Re: [PATCH 3/3] gpu: drm: drivers: Convert printk(KERN_ to pr_

2017-02-28 Thread Patrik Jakobsson
On Tue, Feb 28, 2017 at 1:55 PM, Joe Perches wrote: > Use a more common logging style. > > Miscellanea: > > o Coalesce formats and realign arguments > o Neaten a few macros now using pr_ > > Signed-off-by: Joe Perches For the gma500 changes: Acked-by: Patrik Jakobsson > --- > drivers/gpu/drm

Re: [PATCH v3 04/25] clocksource: Add Owl timer

2017-02-28 Thread Thomas Gleixner
On Tue, 28 Feb 2017, Andreas Färber wrote: > This is a callback, which I thought is re-entrant. It's not reentrant at least not on the same CPU. On a SMP machine this function might be called concurrently on several cores (assumed that the whole thing is replicated across cores). > VAL changes wh

Re: Microphone gain on N900

2017-02-28 Thread Jarkko Nikula
On 02/27/17 21:53, Pali Rohár wrote: > On Sunday 26 February 2017 21:28:32 Pavel Machek wrote: >> Hi! >> >> v4.10 works quite nicely on N900, but I still have problems with >> audio. Even GSM calls would be usable, if I had reasonable volume on >> microphone and speaker... but I don't. >> >> Both s

[PATCH net-next v3 1/3] net: ethernet: bgmac: use #defines for MAX size

2017-02-28 Thread Jon Mason
The maximum frame size is really just the standard ethernet frame size and FCS. So use those existing defines to make the code a little more beautiful. Signed-off-by: Jon Mason --- drivers/net/ethernet/broadcom/bgmac.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/

Re: [PATCH 1/1] ARM: dts: NSP: Add crypto (SPU) to dtsi

2017-02-28 Thread Florian Fainelli
On 02/22/2017 01:22 PM, Steve Lin wrote: > Adds crypto hardware (SPU) to Northstar Plus device tree file. > > Signed-off-by: Steve Lin Applied, thanks! -- Florian

Re: [PATCH 1/2] ARM: dts: bcm5301x: Add TWD WD Support to DT

2017-02-28 Thread Rafał Miłecki
On 28 February 2017 at 21:31, Jon Mason wrote: > @@ -70,10 +70,19 @@ > clocks = <&periph_clk>; > }; > > - local-timer@20600 { > + timer@20600 { Is there a reason for this rename? I don't really have preference for any name, it's

Re: [PATCH 1/2] ARM: dts: bcm5301x: Add TWD WD Support to DT

2017-02-28 Thread Jon Mason
On Tue, Feb 28, 2017 at 3:43 PM, Rafał Miłecki wrote: > On 28 February 2017 at 21:31, Jon Mason wrote: >> @@ -70,10 +70,19 @@ >> clocks = <&periph_clk>; >> }; >> >> - local-timer@20600 { >> + timer@20600 { > > Is there a reason f

Re: [linux-sunxi] [PATCH 2/5] arm64: only select PINCTRL_SUNXI for Allwinner platforms

2017-02-28 Thread Andre Przywara
Hi, On 28/02/17 17:24, Icenowy Zheng wrote: > As the pinctrl driver selecting is refactored in Kconfig file of > pinctrl-sunxi, now we can select only PINCTRL_SUNXI for Allwinner > platform, and the default value of several pinctrl drivers useful on > ARM64 Allwinner SoCs will become Y. > > Drop

Re: [PATCH v2 2/3] spi: allocate spi_board_info entries one by one

2017-02-28 Thread Lars-Peter Clausen
On 02/28/2017 07:54 PM, Mark Brown wrote: > On Tue, Feb 28, 2017 at 10:24:17AM -0800, Dmitry Torokhov wrote: >> On Tue, Feb 28, 2017 at 09:16:50AM +, Mark Brown wrote: >>> On Mon, Feb 27, 2017 at 08:18:56PM -0800, Dmitry Torokhov wrote: Lists of spi_board_info entries registered with spi_r

[PATCH v2] sched/rt: Add comments describing the RT IPI pull method

2017-02-28 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" While looking into optimizations for the RT scheduler IPI logic, I realized that the comments are lacking to describe it efficiently. It deserves a lengthy description describing its design. Signed-off-by: Steven Rostedt (VMware) --- Changes since v1. Added

Re: [PATCH] arm: exynos increase CONFIG_CMA_SIZE_MBYTES to 96

2017-02-28 Thread Shuah Khan
On 02/28/2017 09:42 AM, Javier Martinez Canillas wrote: > Hello Shuah, > > The subject line isn't consistent with what's used for exynos_defconfig > changes, please use instead: > > ARM: exynos_defconfig: Increase CONFIG_CMA_SIZE_MBYTES to 96 > oops - I meant o make it exynos_defconfig > On 02

Re: [PATCH 1/4] MIPS: Handle non word sized instructions when examining frame

2017-02-28 Thread Maciej W. Rozycki
On Tue, 28 Feb 2017, Matt Redfearn wrote: > Since the instruction modifying the stack pointer is usually the first > in the function, that one is ususally handled correctly. But the s/ususally/usually/ > diff --git a/arch/mips/kernel/process.c b/arch/mips/kernel/process.c > index 803e255b6fc3..

Re: [PATCH] sched/rt: Add comments describing the RT IPI pull method

2017-02-28 Thread Steven Rostedt
On Mon, 20 Feb 2017 12:12:35 +0100 Peter Zijlstra wrote: Hi Peter, I'm finally getting around to updating this patch. > > +/* > > + * When a high priority task schedules out from a CPU and a lower priority > > + * task is scheduled in, a check is made to see if there's any RT tasks > > + * on

[PATCH 2/2] ARM: dts: bcm5301x: Add I2C support to the DT

2017-02-28 Thread Jon Mason
From: Jon Mason Add I2C support to the bcm5301x Device Tree. Since no driver changes are needed to enable this hardware, only the device tree changes are required to make this functional. Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm5301x.dtsi | 9 + 1 file changed, 9 insertions(

Re: [Lsf-pc] [LSF/MM TOPIC] do we really need PG_error at all?

2017-02-28 Thread NeilBrown
On Mon, Feb 27 2017, Jeff Layton wrote: > On Tue, 2017-02-28 at 10:32 +1100, NeilBrown wrote: >> On Mon, Feb 27 2017, Andreas Dilger wrote: >> >> > >> > My thought is that PG_error is definitely useful for applications to get >> > correct errors back when doing write()/sync_file_range() so that

Re: KASAN errors after 21c8e72037fb ("crypto: testmgr - use calculated count for number of test vectors")

2017-02-28 Thread Ard Biesheuvel
On 28 February 2017 at 20:17, Laura Abbott wrote: > Hi, > > While attempting to debug something else, I saw KASAN errors > from the crypto test with ccm: > > [ 28.231615] > == > [ 28.232007] BUG: KASAN: global-out-of-bounds in __

Re: kvm: WARNING in nested_vmx_merge_msr_bitmap

2017-02-28 Thread Jim Mattson
I believe this happens when the VMCS12 MSR bitmap address is valid, but no device is configured to respond to the bus request. I agree that the warning should be removed. However, in this case, the VMCS12 MSR bitmap should read as all 1s. The same is true everywhere that nested_get_page returns NUL

Re: [PATCH 1/2] drivers: pwm: pwm-atmel: switch to atomic PWM

2017-02-28 Thread Boris Brezillon
Hi Claudiu, On Tue, 28 Feb 2017 12:40:14 +0200 Claudiu Beznea wrote: > The currently Atmel PWM controllers supported by this driver > could change period and duty factor without channel disable > (sama5d3 supports this by using period and duty factor update > registers, sam9rl support this by wr

[PATCH 0/8] watchdog fixes for COMPILE_TEST change

2017-02-28 Thread Arnd Bergmann
Hi Guenter, I noticed your change to enable lots of watchdog drivers for compile testing in linux-next today. While this is generally a great idea, I'm not entirely sure why this appeared during the merge window. I case you plan to have this merged into v4.11, please make sure to get the fixups i

[PATCH 7/8] watchdog: bcm2835: add CONFIG_OF dependency

2017-02-28 Thread Arnd Bergmann
Without CONFIG_OF, the driver fails to link: drivers/watchdog/built-in.o: In function `bcm2835_power_off': bcm2835_wdt.c:(.text+0x1946): undefined reference to `of_find_device_by_node' This adds a new dependency, to allow the COMPILE_TEST check to succeed. Fixes: da2a68b3eb47 ("watchdog: Enable

[PATCH 3/8] watchdog: wm831x watchdog really needs mfd

2017-02-28 Thread Arnd Bergmann
The wm831x watchdog driver can now be built without the wm831x mfd driver, which results in a link error: (.text+0x1a95c): undefined reference to `wm831x_set_bits' (.text+0x1a95c): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `wm831x_set_bits' (.text+0x1a968): undefined

[PATCH 1/8] mfd: retu: add inline alternatives for CONFIG_RETU=n

2017-02-28 Thread Arnd Bergmann
The retu watchdog calls into the respective mfd driver, but fails to link if that is diabled: drivers/watchdog/built-in.o: In function `retu_wdt_set_timeout': ziirave_wdt.c:(.text+0x8c88): undefined reference to `retu_write' ziirave_wdt.c:(.text+0x8c88): relocation truncated to fit: R_AARCH64_CALL

RE: [Intel-wired-lan] [PATCH] e1000e: fix timing for 82579 Gigabit Ethernet controller

2017-02-28 Thread Keller, Jacob E
> -Original Message- > From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On > Behalf Of Neftin, Sasha > Sent: Monday, February 27, 2017 12:40 AM > To: Bernd Faust ; Kirsher, Jeffrey T > ; intel-wired-...@lists.osuosl.org; > net...@vger.kernel.org; linux-kernel@vger.ker

[PATCH 4/8] watchdog: geode: restore hard CS5535_MFGPT dependency

2017-02-28 Thread Arnd Bergmann
Wtihout CONFIG_CS5535_MFGPT, the driver does not link right: drivers/watchdog/built-in.o: In function `geodewdt_probe': geodewdt.c:(.init.text+0xca3): undefined reference to `cs5535_mfgpt_alloc_timer' geodewdt.c:(.init.text+0xcd4): undefined reference to `cs5535_mfgpt_write' geodewdt.c:(.init.text

[PATCH 6/8] watchdog: sp805: add back AMBA dependency

2017-02-28 Thread Arnd Bergmann
The driver fails to link if ARM_AMBA is disabled: drivers/watchdog/sp805_wdt.o: In function `sp805_wdt_driver_init': sp805_wdt.c:(.init.text+0x4): undefined reference to `amba_driver_register' It seems that the COMPILE_TEST was added in the wrong place, as there is no architecture dependency, but

[PATCH 8/8] watchdog: kempld: revert to full dependency

2017-02-28 Thread Arnd Bergmann
The kempld watchdog driver requires the respective MFD driver: drivers/watchdog/built-in.o: In function `kempld_wdt_probe': kempld_wdt.c:(.text+0x5c78): undefined reference to `kempld_get_mutex' kempld_wdt.c:(.text+0x5c84): undefined reference to `kempld_read8' kempld_wdt.c:(.text+0x5c8e): undefin

[PATCH 2/8] mfd: db8500-prcmu: fix stub helper interface

2017-02-28 Thread Arnd Bergmann
When the db8500 watchdog is enabled without the PRCMU, we get a lot of warnings about duplicate or missing helper functions: In file included from drivers/watchdog/ux500_wdt.c:21:0: include/linux/mfd/dbx500-prcmu.h:422:19: error: redefinition of 'prcmu_abb_read' static inline int prcmu_abb_read(u

[PATCH] remoteproc: qcom: fix QCOM_SMD dependencies

2017-02-28 Thread Arnd Bergmann
qcom_smd_register_edge() is provided by either QCOM_SMD or RPMSG_QCOM_SMD, and if both of them are disabled, it does nothing. The check for the PIL drivers however only checks for QCOM_SMD, so it breaks with QCOM_SMD=n && RPMSG_QCOM_SMD=m: drivers/remoteproc/built-in.o: In function `smd_subdev_re

[PATCH] tpm: select CONFIG_CRYPTO

2017-02-28 Thread Arnd Bergmann
We get a newly introduced harmless warning when CONFIG_CRYPTO is disabled: warning: (TCG_TPM && TRUSTED_KEYS && IMA) selects CRYPTO_HASH_INFO which has unmet direct dependencies (CRYPTO) This adds another select to avoid the warning, consistent with other users of the crypto code. Fixes: c1f92b

Re: [patch 0/3] KVM CPU frequency change hypercalls

2017-02-28 Thread Marcelo Tosatti
On Fri, Feb 24, 2017 at 04:34:52PM +0100, Paolo Bonzini wrote: > > > On 24/02/2017 14:04, Marcelo Tosatti wrote: > > Whats the current usecase, or forseeable future usecase, for > > save/restore > > across preemption again? (which would validate the broken by design > > claim). >

[PATCH 2/2] hwmon: Aspeed AST2400/AST2500 ADC

2017-02-28 Thread Rick Altherr
Aspeed AST2400/AST2500 BMC SoCs include a 16 channel, 10-bit ADC. This driver implements reading the ADC values, enabling channels via device tree, and optionally providing channel labels via device tree. Low and high threshold interrupts are supported by the hardware but not implemented. Signed-

[PATCH v8 10/11] uapi: export all headers under uapi directories

2017-02-28 Thread Nicolas Dichtel
Regularly, when a new header is created in include/uapi/, the developer forgets to add it in the corresponding Kbuild file. This error is usually detected after the release is out. In fact, all headers under uapi directories should be exported, thus it's useless to have an exhaustive list. After

Re: [PATCH 1/2] ARM: dts: bcm5301x: Add TWD WD Support to DT

2017-02-28 Thread Rafał Miłecki
On 28 February 2017 at 21:46, Jon Mason wrote: > On Tue, Feb 28, 2017 at 3:43 PM, Rafał Miłecki wrote: >> On 28 February 2017 at 21:31, Jon Mason wrote: >>> @@ -70,10 +70,19 @@ >>> clocks = <&periph_clk>; >>> }; >>> >>> - local-timer@20600 {

[PATCH v8 02/11] nios2: put setup.h in uapi

2017-02-28 Thread Nicolas Dichtel
This header file is exported, but from a userland pov, it's just a wrapper to asm-generic/setup.h. Signed-off-by: Nicolas Dichtel Reviewed-by: Tobias Klauser --- arch/nios2/include/uapi/asm/Kbuild | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/nios2/include/uapi/asm/Kbuild b/arch/nio

[PATCH v8 03/11] x86: stop exporting msr-index.h to userland

2017-02-28 Thread Nicolas Dichtel
Even if this file was not in an uapi directory, it was exported because it was listed in the Kbuild file. Fixes: b72e7464e4cf ("x86/uapi: Do not export as part of the user API headers") Suggested-by: Borislav Petkov CC: Ingo Molnar Signed-off-by: Nicolas Dichtel Acked-by: Ingo Molnar Acked-b

[PATCH v8 05/11] Makefile.headersinst: remove destination-y option

2017-02-28 Thread Nicolas Dichtel
This option was added in commit c7bb349e7c25 ("kbuild: introduce destination-y for exported headers") but never used in-tree. Signed-off-by: Nicolas Dichtel Acked-by: Paul Bolle --- Documentation/kbuild/makefiles.txt | 23 --- scripts/Makefile.headersinst | 2 +- 2 fi

[PATCH v8 00/11] uapi: export all headers under uapi directories

2017-02-28 Thread Nicolas Dichtel
Patches #1 and #2 are just cleanup: some exported headers were still under a non-uapi directory. Patch #3 is a fix to avoid exporting a file that was not under an uapi directory. After these three patches, all exported headers are under an uapi directory: path #4 stops searching files in non uapi

[PATCH] [media] tw5864: use dev_warn instead of WARN to shut up warning

2017-02-28 Thread Arnd Bergmann
tw5864_frameinterval_get() only initializes its output when it successfully identifies the video standard in tw5864_input. We get a warning here because gcc can't always track the state if initialized warnings across a WARN() macro, and thinks it might get used incorrectly in tw5864_s_parm: media/

Re: [PATCH] scsi: lpfc: use proper format string for dma_addr_t

2017-02-28 Thread James Smart
Arnd, Thank you. Looks good. -- james Signed-off-by: James Smart On 2/27/2017 12:37 PM, Arnd Bergmann wrote: dma_addr_t may be either u32 or u64, depending on the kernel configuration, and we get a warning for the 32-bit case: drivers/scsi/lpfc/lpfc_nvme.c: In function 'lpfc_nvme_ls_req':

[PATCH v8 07/11] btrfs_tree.h: fix include from userland

2017-02-28 Thread Nicolas Dichtel
This patch prepares the uapi export by fixing the following errors: .../linux/btrfs_tree.h:283:2: error: #error "UUID items require BTRFS_UUID_SIZE == 16!" #error "UUID items require BTRFS_UUID_SIZE == 16!" .../linux/btrfs_tree.h:390:12: error: ‘BTRFS_UUID_SIZE’ undeclared here (not in a funct

Re: nios2 crash/hang in mainline due to 'lib: update LZ4 compressor module'

2017-02-28 Thread Sandra Loosemore
On 02/28/2017 08:53 AM, Tobias Klauser wrote: (adding Sandra Loosemore to Cc due to possible relation to gcc/binutils for nios2) On 2017-02-26 at 22:03:38 +0100, Guenter Roeck wrote: Hi Sven, my qemu test for nios2 started failing with commit 4e1a33b105dd ("lib: update LZ4 compressor module")

[PATCH v8 11/11] uapi: export all arch specifics directories

2017-02-28 Thread Nicolas Dichtel
This patch removes the need of subdir-y. Now all files/directories under arch//include/uapi/ are exported. The only change for userland is the layout of the command 'make headers_install_all': directories asm- are replaced by arch-/. Those new directories contains all files/directories of the spec

[GIT PULL] nfsd changes for 4.11

2017-02-28 Thread J. Bruce Fields
Please pull nfsd changes for 4.11 from git://linux-nfs.org/~bfields/linux.git tags/nfsd-4.11 The nfsd update this round is mainly a lot of miscellaneous cleanups and bugfixes. A couple changes could theoretically break working setups on upgrade. I don't expect complaints in practice, but they

[PATCH v8 09/11] smc_diag.h: fix include from userland

2017-02-28 Thread Nicolas Dichtel
This patch prepares the uapi export by fixing the following error: .../linux/smc_diag.h:6:27: fatal error: rdma/ib_verbs.h: No such file or directory #include Signed-off-by: Nicolas Dichtel --- include/rdma/ib_verbs.h | 3 +-- include/uapi/linux/smc_diag.h | 2 +- include/uapi/

[PATCH v8 01/11] h8300: put bitsperlong.h in uapi

2017-02-28 Thread Nicolas Dichtel
This header file is exported, thus move it to uapi. Signed-off-by: Nicolas Dichtel --- arch/h8300/include/asm/bitsperlong.h | 14 -- arch/h8300/include/uapi/asm/bitsperlong.h | 14 ++ 2 files changed, 14 insertions(+), 14 deletions(-) delete mode 100644 arch/h8300/i

[PATCH v8 04/11] Makefile.headersinst: cleanup input files

2017-02-28 Thread Nicolas Dichtel
After the last four patches, all exported headers are under uapi/, thus input-files2 are not needed anymore. The side effect is that input-files1-name is exactly header-y. Note also that input-files3-name is genhdr-y. Signed-off-by: Nicolas Dichtel --- scripts/Makefile.headersinst | 34

[PATCH v8 08/11] cryptouser.h: fix include from userland

2017-02-28 Thread Nicolas Dichtel
This patch prepares the uapi export by fixing the following error: .../linux/cryptouser.h:58:16: error: ‘CRYPTO_MAX_ALG_NAME’ undeclared here (not in a function) char cru_name[CRYPTO_MAX_ALG_NAME]; Signed-off-by: Nicolas Dichtel --- include/linux/crypto.h | 2 +- include/uapi/linux/

Re: [PATCH] tpm: select CONFIG_CRYPTO

2017-02-28 Thread Peter Huewe
Am 28. Februar 2017 22:12:55 MEZ schrieb Arnd Bergmann : >We get a newly introduced harmless warning when CONFIG_CRYPTO is >disabled: > >warning: (TCG_TPM && TRUSTED_KEYS && IMA) selects CRYPTO_HASH_INFO >which has unmet direct dependencies (CRYPTO) > >This adds another select to avoid the warnin

[PATCH v8 06/11] uapi: includes linux/types.h before exporting files

2017-02-28 Thread Nicolas Dichtel
Some files will be exported after the next patch. 0-day tests report the following warning/error: ./usr/include/linux/bcache.h:8: include of is preferred over ./usr/include/linux/bcache.h:11: found __[us]{8,16,32,64} type without #include ./usr/include/linux/qrtr.h:8: found __[us]{8,16,32,64}

Re: [PATCH v2] staging: iio: accel: Move header file content to source file

2017-02-28 Thread SIMRAN SINGHAL
On Wed, Mar 1, 2017 at 1:04 AM, Lars-Peter Clausen wrote: > On 02/28/2017 08:32 PM, SIMRAN SINGHAL wrote: >> On Wed, Mar 1, 2017 at 12:53 AM, Jonathan Cameron wrote: >>> On 28/02/17 19:17, Jonathan Cameron wrote: On 28/02/17 18:51, simran singhal wrote: > The contents of the header file

Re: [PATCH] pci: dwc: Fix NULL pointer dereference due to missing assignment

2017-02-28 Thread Bjorn Helgaas
On Tue, Feb 28, 2017 at 01:38:18PM +0530, Vivek Gautam wrote: > Add missing assignment of dw_pci core structure for > designware-plat. > > Commit 8b130f815a6b ("PCI: dwc: Fix crashes seen due to missing > assignments") fixed all other platforms, but missed > designware-plat file. > > Fixes: 442ec

Re: sounds: deadlocked processed in snd_seq_pool_done

2017-02-28 Thread Takashi Iwai
On Tue, 28 Feb 2017 17:26:27 +0100, Dmitry Vyukov wrote: > > Hello, > > The following program creates deadlocked processed in snd_seq_pool_done: > > https://gist.githubusercontent.com/dvyukov/1c9b1e4641f46c16a42cdb4458bbe8fd/raw/a09912f3cff4d86f412cedf5f5b4a5c98a9f0040/gistfile1.txt > > # cat /

Re: [PATCH 0/8] watchdog fixes for COMPILE_TEST change

2017-02-28 Thread Guenter Roeck
On Tue, Feb 28, 2017 at 10:01:15PM +0100, Arnd Bergmann wrote: > Hi Guenter, > > I noticed your change to enable lots of watchdog drivers for compile > testing in linux-next today. While this is generally a great idea, > I'm not entirely sure why this appeared during the merge window. > Problem i

Re: [RFC PATCH v4] IV Generation algorithms for dm-crypt

2017-02-28 Thread Milan Broz
On 02/22/2017 07:12 AM, Binoy Jayan wrote: > > I was wondering if this is near to be ready for submission (apart from > the testmgr.c > changes) or I need to make some changes to make it similar to the IPSec > offload? I just tried this and except it registers the IV for every new device again,

[PATCH] net/mlx5e: add IPV6 dependency

2017-02-28 Thread Arnd Bergmann
The ethernet support now calls directly into the ipv6 core code, which fails if IPV6 is a loadable module but mlx5 is built-in: drivers/net/ethernet/mellanox/mlx5/core/en_tc.o: In function `mlx5e_create_encap_header_ipv6': en_tc.c:(.text.mlx5e_create_encap_header_ipv6+0x110): undefined reference

Re: [PATCH] clk: meson: fix SET_PARM macro

2017-02-28 Thread Stephen Boyd
On 02/28, Jerome Brunet wrote: > parameter val is not enclosed in parenthesis which is buggy when given an > expression instead of a simple value > > Signed-off-by: Jerome Brunet Fixes tag? Is there a place in the code that is using a complex expression for val right now? -- Qualcomm Innovatio

[PATCH] intel_idle: stop exposing platform acronyms in sysfs

2017-02-28 Thread Len Brown
From: Len Brown Cosmetic only -- no functional change in this patch. sysfs before: state4/desc:MWAIT 0x20 state4/name:C6-HSW sysfs after: state4/desc:MWAIT 0x20 state4/name:C6 We remove the platform acronyms from the end of the state name (-HSW in this case) for three reasonse. 1. more cons

RE: IB on s390 broken with commit 99db94940 "IB/core: Remove ib_device.dma_device"

2017-02-28 Thread Parav Pandit
Hi Bart, I am using Linux-block tree testing on x86_64. git://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git Commit ac1820fb286b552b6885d40ab34f1e59b815f1f1 introduced dma_ops related change that you made. With this change I am hitting below error in mlx5_ib driver. "DMAR: Allocat

Re: [PATCH] staging/fbtft: Hush checkpatch.pl warning about unnecessary line continuations.

2017-02-28 Thread Alexander Kapshuk
On Tue, Feb 28, 2017 at 10:34 PM, Joe Perches wrote: > On Tue, 2017-02-28 at 21:52 +0200, Alexander Kapshuk wrote: >> Use a single fmt string with appropriate verbs as conversion specifiers, >> followed by the original string literals and the integer argument >> instead of using a backslash to esc

Re: [PATCH] PCI: altera: Fix TLP_CFG_DW0 for TLP write

2017-02-28 Thread Bjorn Helgaas
On Tue, Feb 28, 2017 at 06:37:16PM +0800, Ley Foon Tan wrote: > Commit eb5767122feba1 used the TLP_FMTTYPE_CFGRD* for TLP write operation > and this cause writing to configuration space will fail. This patch > fix it by using correct FMTTYPE for write operation. > > Signed-off-by: Ley Foon Tan >

[PATCH v2] video: fbdev: imxfb: support AUS mode

2017-02-28 Thread Martin Kaiser
Some displays require setting AUS mode in the LDCD AUS Mode Control Register to work with the imxfb driver. Like the value of the Panel Configuration Register, the AUS Mode Control Register's value depends on the display mode. Allow setting this register from the device tree. Make the device tree

[PATCH 5/8] watchdog: menf21bmc: add I2C dependency

2017-02-28 Thread Arnd Bergmann
This driver fails to link when CONFIG_I2C is disabled or a loadable module while the watchdog is built-in: drivers/watchdog/built-in.o: In function `menf21bmc_wdt_shutdown': menf21bmc_wdt.c:(.text+0x9b44): undefined reference to `i2c_smbus_write_word_data' menf21bmc_wdt.c:(.text+0x9b44): relocati

Re: [PATCH 3/8] watchdog: wm831x watchdog really needs mfd

2017-02-28 Thread Guenter Roeck
On Tue, Feb 28, 2017 at 10:01:18PM +0100, Arnd Bergmann wrote: > The wm831x watchdog driver can now be built without the wm831x mfd > driver, which results in a link error: > > (.text+0x1a95c): undefined reference to `wm831x_set_bits' > (.text+0x1a95c): relocation truncated to fit: R_AARCH64_CALL2

Re: [PATCH 1/8] mfd: retu: add inline alternatives for CONFIG_RETU=n

2017-02-28 Thread Guenter Roeck
On Tue, Feb 28, 2017 at 10:01:16PM +0100, Arnd Bergmann wrote: > The retu watchdog calls into the respective mfd driver, but fails to > link if that is diabled: > > drivers/watchdog/built-in.o: In function `retu_wdt_set_timeout': > ziirave_wdt.c:(.text+0x8c88): undefined reference to `retu_write'

[PATCH 1/2] ARM: dts: bcm5301x: Add TWD WD Support to DT

2017-02-28 Thread Jon Mason
From: Jon Mason Add support for the ARM TWD Watchdog to the bcm5301x device tree. The ARM TWD timer allocated the register space for the WDT, so this patch necessitated shrinking that. Also, the GIC masks were added for these. Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm5301x.dtsi | 15

RE: [PATCH V2] PCI/ASPM: reconfigure ASPM following hotplug for POLICY_DEFAULT

2017-02-28 Thread Patel, Mayurkumar
> >On 2/28/2017 6:03 AM, Patel, Mayurkumar wrote: >>> - /* Save default state */ >>> - link->aspm_default = link->aspm_enabled; >> But, I am finding a problem with this change, if Policy is set to default, >> BIOS enables ASPM L1, but pcie_config_aspm_link() disables ASPM L1 >> due to link->asp

Re: [PATCH] clk: meson: fix SET_PARM macro

2017-02-28 Thread Jerome Brunet
On Tue, 2017-02-28 at 13:26 -0800, Stephen Boyd wrote: > On 02/28, Jerome Brunet wrote: > > parameter val is not enclosed in parenthesis which is buggy when > > given an > > expression instead of a simple value > > > > Signed-off-by: Jerome Brunet > > Fixes tag? Is there a place in the code that

Re: [RFC PATCH 2/2] mtd: devices: m25p80: Enable spi-nor bounce buffer support

2017-02-28 Thread Richard Weinberger
Vignesh, Am 27.02.2017 um 13:08 schrieb Vignesh R: > Many SPI controller drivers use DMA to read/write from m25p80 compatible > flashes. Therefore enable bounce buffers support provided by spi-nor > framework to take care of handling vmalloc'd buffers which may not be > DMA'able. > > Signed-off-b

Re: [PATCH] jump_label: align jump_entry table to at least 4-bytes

2017-02-28 Thread David Daney
On 02/28/2017 10:39 AM, Jason Baron wrote: On 02/28/2017 01:16 PM, David Daney wrote: On 02/28/2017 08:21 AM, Steven Rostedt wrote: On Tue, 28 Feb 2017 10:25:46 +0530 Sachin Sant wrote: File: ./net/ipv4/xfrm4_input.o [12] __jump_table PROGBITS 000639 18 1

Re: [PATCH 2/8] mfd: db8500-prcmu: fix stub helper interface

2017-02-28 Thread Guenter Roeck
On Tue, Feb 28, 2017 at 10:01:17PM +0100, Arnd Bergmann wrote: > When the db8500 watchdog is enabled without the PRCMU, we get a lot of > warnings about duplicate or missing helper functions: > > In file included from drivers/watchdog/ux500_wdt.c:21:0: > include/linux/mfd/dbx500-prcmu.h:422:19: er

Re: [Outreachy kernel] [PATCH 4/5] staging: sm750fb: Remove unnecessary else after return

2017-02-28 Thread SIMRAN SINGHAL
On Tue, Feb 28, 2017 at 2:45 AM, Julia Lawall wrote: > > > On Mon, 27 Feb 2017, simran singhal wrote: > >> This patch fixes the checkpatch warning that else is not generally >> useful after a break or return. >> >> This was done using Coccinelle: >> @@ >> expression e2; >> statement s1; >> @@ >> i

Re: [PATCH 4/8] watchdog: geode: restore hard CS5535_MFGPT dependency

2017-02-28 Thread Guenter Roeck
On Tue, Feb 28, 2017 at 10:01:19PM +0100, Arnd Bergmann wrote: > Wtihout CONFIG_CS5535_MFGPT, the driver does not link right: > > drivers/watchdog/built-in.o: In function `geodewdt_probe': > geodewdt.c:(.init.text+0xca3): undefined reference to > `cs5535_mfgpt_alloc_timer' > geodewdt.c:(.init.tex

Re: [PATCH 6/8] watchdog: sp805: add back AMBA dependency

2017-02-28 Thread Guenter Roeck
On Tue, Feb 28, 2017 at 10:01:21PM +0100, Arnd Bergmann wrote: > The driver fails to link if ARM_AMBA is disabled: > > drivers/watchdog/sp805_wdt.o: In function `sp805_wdt_driver_init': > sp805_wdt.c:(.init.text+0x4): undefined reference to `amba_driver_register' > > It seems that the COMPILE_TES

Re: [PATCH 5/8] watchdog: menf21bmc: add I2C dependency

2017-02-28 Thread Guenter Roeck
On Tue, Feb 28, 2017 at 10:01:20PM +0100, Arnd Bergmann wrote: > This driver fails to link when CONFIG_I2C is disabled or a loadable module > while > the watchdog is built-in: > > drivers/watchdog/built-in.o: In function `menf21bmc_wdt_shutdown': > menf21bmc_wdt.c:(.text+0x9b44): undefined refere

Re: [PATCH 3/8] watchdog: wm831x watchdog really needs mfd

2017-02-28 Thread Arnd Bergmann
On Tue, Feb 28, 2017 at 10:36 PM, Guenter Roeck wrote: >> config WM831X_WATCHDOG >> tristate "WM831x watchdog" >> - depends on MFD_WM831X || COMPILE_TEST >> + depends on MFD_WM831X > > I prefer the fix proposed by Randy: > > - depends on MFD_WM831X || COMPILE_TEST > + d

Re: [PATCH 11/13] fs: fix unsigned enum warning with gcc-4.2

2017-02-28 Thread Brendan Gregg
On Tue, Jan 3, 2017 at 2:47 PM, Brendan Gregg wrote: > > On Fri, Dec 16, 2016 at 2:56 AM, Arnd Bergmann wrote: > > > > With arm-linux-gcc-4.2, almost every file we build in the kernel ends > > up with this warning: > > > > include/linux/fs.h:2648: warning: comparison of unsigned expression < 0 is

<    1   2   3   4   5   6   7   8   9   >