Re: [PATCH V15 01/11] acpi: apei: read ack upon ghes record consumption

2017-04-19 Thread Baicar, Tyler
On 4/19/2017 12:31 PM, Borislav Petkov wrote: On Tue, Apr 18, 2017 at 05:05:13PM -0600, Tyler Baicar wrote: A RAS (Reliability, Availability, Serviceability) controller may be a separate processor running in parallel with OS execution, and may generate error records for consumption by the OS.

Re: [PATCH V15 01/11] acpi: apei: read ack upon ghes record consumption

2017-04-19 Thread Baicar, Tyler
On 4/19/2017 12:31 PM, Borislav Petkov wrote: On Tue, Apr 18, 2017 at 05:05:13PM -0600, Tyler Baicar wrote: A RAS (Reliability, Availability, Serviceability) controller may be a separate processor running in parallel with OS execution, and may generate error records for consumption by the OS.

[PATCH] ARM: dts: am335x-phycore-som: fix rv4162 compatible

2017-04-19 Thread Alexandre Belloni
The rv4162 compatbile string is missing the vendor part, add it. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/am335x-phycore-som.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/am335x-phycore-som.dtsi

[PATCH] ARM: dts: am335x-phycore-som: fix rv4162 compatible

2017-04-19 Thread Alexandre Belloni
The rv4162 compatbile string is missing the vendor part, add it. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/am335x-phycore-som.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/am335x-phycore-som.dtsi

Re: [PATCH] soc: bcm: brcmstb: Correctly match 7435 SoC

2017-04-19 Thread Gregory Fong
On Wed, Apr 19, 2017 at 10:01 AM, Florian Fainelli wrote: > Remove the duplicate brcm,bcm7425-sun-top-ctrl compatible string and > replace it with brcm,bcm7435-sun-top-ctrl which was intentend. s/intentend/intended/ > > Fixes: bd0faf08dc7f ("soc: bcm: brcmstb: Match

Re: [PATCH] soc: bcm: brcmstb: Correctly match 7435 SoC

2017-04-19 Thread Gregory Fong
On Wed, Apr 19, 2017 at 10:01 AM, Florian Fainelli wrote: > Remove the duplicate brcm,bcm7425-sun-top-ctrl compatible string and > replace it with brcm,bcm7435-sun-top-ctrl which was intentend. s/intentend/intended/ > > Fixes: bd0faf08dc7f ("soc: bcm: brcmstb: Match additional compatible

Re: [PATCH 2/2] pid_ns: Introduce ioctl to set vector of ns_last_pid's on ns hierarhy

2017-04-19 Thread Serge E. Hallyn
Quoting Kirill Tkhai (ktk...@virtuozzo.com): > On implementing of nested pid namespaces support in CRIU > (checkpoint-restore in userspace tool) we run into > the situation, that it's impossible to create a task with > specific NSpid effectively. After commit 49f4d8b93ccf > "pidns: Capture the

Re: [PATCH 2/2] pid_ns: Introduce ioctl to set vector of ns_last_pid's on ns hierarhy

2017-04-19 Thread Serge E. Hallyn
Quoting Kirill Tkhai (ktk...@virtuozzo.com): > On implementing of nested pid namespaces support in CRIU > (checkpoint-restore in userspace tool) we run into > the situation, that it's impossible to create a task with > specific NSpid effectively. After commit 49f4d8b93ccf > "pidns: Capture the

Re: [PATCH v2 1/2] hp-wmi: Do not shadow error values

2017-04-19 Thread Carlo Caione
On Wed, Apr 19, 2017 at 10:11 PM, Darren Hart wrote: > On Wed, Apr 19, 2017 at 07:41:53PM +0200, Carlo Caione wrote: >> From: Carlo Caione >> >> All the helper functions (i.e. hp_wmi_dock_state, hp_wmi_tablet_state, >> ...) using hp_wmi_perform_query to

Re: [PATCH v2 1/2] hp-wmi: Do not shadow error values

2017-04-19 Thread Carlo Caione
On Wed, Apr 19, 2017 at 10:11 PM, Darren Hart wrote: > On Wed, Apr 19, 2017 at 07:41:53PM +0200, Carlo Caione wrote: >> From: Carlo Caione >> >> All the helper functions (i.e. hp_wmi_dock_state, hp_wmi_tablet_state, >> ...) using hp_wmi_perform_query to perform an HP WMI query shadow the >>

Re: [PATCH v4 1/1] pwm: pca9685: fix gpio-only operation.

2017-04-19 Thread Mika Westerberg
On Tue, Apr 18, 2017 at 11:52:49AM -0400, Sven Van Asbroeck wrote: > > __maybe_unused and remove ugly #ifdef:ery. > > If this works on non- CONFIG_PM systems, I'm all for it ! > Grepping the drivers/ directory, I see that some drivers use > #ifdef CONFIG_PM, some use __maybe_unused for

Re: [PATCH v4 1/1] pwm: pca9685: fix gpio-only operation.

2017-04-19 Thread Mika Westerberg
On Tue, Apr 18, 2017 at 11:52:49AM -0400, Sven Van Asbroeck wrote: > > __maybe_unused and remove ugly #ifdef:ery. > > If this works on non- CONFIG_PM systems, I'm all for it ! > Grepping the drivers/ directory, I see that some drivers use > #ifdef CONFIG_PM, some use __maybe_unused for

[PATCH 3/3] ARM: dts: imx7d-nitrogen7: fix rv4162 compatible

2017-04-19 Thread Alexandre Belloni
The rv4162 compatbile string is missing the vendor part, add it. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/imx7d-nitrogen7.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx7d-nitrogen7.dts

[PATCH 3/3] ARM: dts: imx7d-nitrogen7: fix rv4162 compatible

2017-04-19 Thread Alexandre Belloni
The rv4162 compatbile string is missing the vendor part, add it. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/imx7d-nitrogen7.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx7d-nitrogen7.dts b/arch/arm/boot/dts/imx7d-nitrogen7.dts index

Re: [PATCH 1/1] selftests: breakpoints: allow to cross-compile for aarch64/arm64

2017-04-19 Thread Shuah Khan
On 04/19/2017 11:31 AM, Fathi Boudra wrote: > gentle ping > > On 22 March 2017 at 20:04, Fathi Boudra wrote: >> To build breakpoint_test_arm64, ARCH value is only tested for "aarch64". >> It covers only the native build because it's computed from uname -m output. >> For

[PATCH 2/3] ARM: dts: imx6qdl-nitrogen6_som2: fix rv4162 compatible

2017-04-19 Thread Alexandre Belloni
The rv4162 vendor is microcrystal, not ST. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi

Re: [PATCH 1/1] selftests: breakpoints: allow to cross-compile for aarch64/arm64

2017-04-19 Thread Shuah Khan
On 04/19/2017 11:31 AM, Fathi Boudra wrote: > gentle ping > > On 22 March 2017 at 20:04, Fathi Boudra wrote: >> To build breakpoint_test_arm64, ARCH value is only tested for "aarch64". >> It covers only the native build because it's computed from uname -m output. >> For cross-compilation, ARCH

[PATCH 2/3] ARM: dts: imx6qdl-nitrogen6_som2: fix rv4162 compatible

2017-04-19 Thread Alexandre Belloni
The rv4162 vendor is microcrystal, not ST. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi index

[PATCH 1/3] ARM: dts: imx6qdl-nitrogen6_max: fix rv4162 compatible

2017-04-19 Thread Alexandre Belloni
The rv4162 vendor is microcrystal, not ST. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi

[PATCH 1/3] ARM: dts: imx6qdl-nitrogen6_max: fix rv4162 compatible

2017-04-19 Thread Alexandre Belloni
The rv4162 vendor is microcrystal, not ST. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi index

Re: [PATCH 1/3] drm/omap: displays: panel-dpi: add backlight dependency

2017-04-19 Thread Laurent Pinchart
Hi Arnd, Thank you for the patch. On Wednesday 19 Apr 2017 19:59:17 Arnd Bergmann wrote: > The panel driver gained support for backlight but fails to link now > when that is disabled: > > drivers/gpu/drm/omapdrm/displays/panel-dpi.o: In function > `panel_dpi_probe_of':

Re: [PATCH 1/3] drm/omap: displays: panel-dpi: add backlight dependency

2017-04-19 Thread Laurent Pinchart
Hi Arnd, Thank you for the patch. On Wednesday 19 Apr 2017 19:59:17 Arnd Bergmann wrote: > The panel driver gained support for backlight but fails to link now > when that is disabled: > > drivers/gpu/drm/omapdrm/displays/panel-dpi.o: In function > `panel_dpi_probe_of':

[PATCH] soc: brcmstb: enable drivers for ARM64 also

2017-04-19 Thread Markus Mayer
From: Markus Mayer We enable the BRCMSTB SoC drivers not only for ARM, but also ARM64. Signed-off-by: Markus Mayer --- drivers/soc/bcm/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/bcm/Kconfig

[PATCH] soc: brcmstb: enable drivers for ARM64 also

2017-04-19 Thread Markus Mayer
From: Markus Mayer We enable the BRCMSTB SoC drivers not only for ARM, but also ARM64. Signed-off-by: Markus Mayer --- drivers/soc/bcm/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/bcm/Kconfig b/drivers/soc/bcm/Kconfig index a39b0d5..b3834b9 100644

Re: [PATCH v1] platform/x86: hp-wireless: reuse module_acpi_driver

2017-04-19 Thread Darren Hart
On Wed, Apr 19, 2017 at 10:27:31PM +0300, Andy Shevchenko wrote: > There is a macro to register and unregister modules in simple cases, > Let's use it and clean up the driver. > > Cc: Alex Hung > Signed-off-by: Andy Shevchenko Thanks

Re: [PATCH v1] platform/x86: hp-wireless: reuse module_acpi_driver

2017-04-19 Thread Darren Hart
On Wed, Apr 19, 2017 at 10:27:31PM +0300, Andy Shevchenko wrote: > There is a macro to register and unregister modules in simple cases, > Let's use it and clean up the driver. > > Cc: Alex Hung > Signed-off-by: Andy Shevchenko Thanks Andy, Reviewed-by: Darren Hart (VMware) -- Darren Hart

Re: [PATCH] mtd: mtdswap: use MTDSWAP_ECNT_MIN/MAX

2017-04-19 Thread Brian Norris
On Tue, Dec 20, 2016 at 09:54:33PM +0800, Geliang Tang wrote: > Since macros MTDSWAP_ECNT_MIN() and MTDSWAP_ECNT_MAX() have been > defined in mtdswap.c, use them instead of open-coding. > > Signed-off-by: Geliang Tang Applied to l2-mtd.git

Re: [PATCH] mtd: mtdswap: use MTDSWAP_ECNT_MIN/MAX

2017-04-19 Thread Brian Norris
On Tue, Dec 20, 2016 at 09:54:33PM +0800, Geliang Tang wrote: > Since macros MTDSWAP_ECNT_MIN() and MTDSWAP_ECNT_MAX() have been > defined in mtdswap.c, use them instead of open-coding. > > Signed-off-by: Geliang Tang Applied to l2-mtd.git

[v4.9-rt PATCH v2] ARM: mm: remove tasklist locking from update_sections_early()

2017-04-19 Thread Grygorii Strashko
The below backtrace can be observed on -rt kernel with CONFIG_DEBUG_RODATA option enabled: BUG: sleeping function called from invalid context at kernel/locking/rtmutex.c:993 in_atomic(): 1, irqs_disabled(): 128, pid: 14, name: migration/0 1 lock held by migration/0/14: #0:

[v4.9-rt PATCH v2] ARM: mm: remove tasklist locking from update_sections_early()

2017-04-19 Thread Grygorii Strashko
The below backtrace can be observed on -rt kernel with CONFIG_DEBUG_RODATA option enabled: BUG: sleeping function called from invalid context at kernel/locking/rtmutex.c:993 in_atomic(): 1, irqs_disabled(): 128, pid: 14, name: migration/0 1 lock held by migration/0/14: #0:

Re: [PATCH 2/2] hp-wmi: Fix detection for dock and tablet mode

2017-04-19 Thread Darren Hart
On Wed, Apr 19, 2017 at 07:23:39PM +0300, Andy Shevchenko wrote: > On Fri, Apr 14, 2017 at 2:07 AM, Darren Hart wrote: > > On Thu, Apr 13, 2017 at 10:09:43PM +0200, Carlo Caione wrote: > >> On Thu, Apr 13, 2017 at 8:21 PM, Darren Hart wrote: > >> > On

Re: [PATCH 2/2] hp-wmi: Fix detection for dock and tablet mode

2017-04-19 Thread Darren Hart
On Wed, Apr 19, 2017 at 07:23:39PM +0300, Andy Shevchenko wrote: > On Fri, Apr 14, 2017 at 2:07 AM, Darren Hart wrote: > > On Thu, Apr 13, 2017 at 10:09:43PM +0200, Carlo Caione wrote: > >> On Thu, Apr 13, 2017 at 8:21 PM, Darren Hart wrote: > >> > On Sun, Apr 09, 2017 at 03:56:08PM +0200, Carlo

Re: [PATCH] ieee802154: don't select COMMON_CLK

2017-04-19 Thread Marcel Holtmann
Hi Arnd, > A device driver must not select the COMMON_CLK subsystem, as that conflicts > with platforms that provide a legacy implementation of the clk API: > > drivers/clk/clk.o: In function `clk_enable': > clk.c:(.text.clk_enable+0x0): multiple definition of `clk_enable' >

Re: [PATCH] ieee802154: don't select COMMON_CLK

2017-04-19 Thread Marcel Holtmann
Hi Arnd, > A device driver must not select the COMMON_CLK subsystem, as that conflicts > with platforms that provide a legacy implementation of the clk API: > > drivers/clk/clk.o: In function `clk_enable': > clk.c:(.text.clk_enable+0x0): multiple definition of `clk_enable' >

Re: [PATCH 24/35] drivers/mtd: Convert remaining uses of pr_warning to pr_warn

2017-04-19 Thread Brian Norris
On Thu, Feb 16, 2017 at 11:11:37PM -0800, Joe Perches wrote: > To enable eventual removal of pr_warning > > This makes pr_warn use consistent for drivers/mtd > > Prior to this patch, there were 7 uses of pr_warning and > 31 uses of pr_warn in drivers/mtd > > Signed-off-by: Joe Perches

Re: [PATCH 24/35] drivers/mtd: Convert remaining uses of pr_warning to pr_warn

2017-04-19 Thread Brian Norris
On Thu, Feb 16, 2017 at 11:11:37PM -0800, Joe Perches wrote: > To enable eventual removal of pr_warning > > This makes pr_warn use consistent for drivers/mtd > > Prior to this patch, there were 7 uses of pr_warning and > 31 uses of pr_warn in drivers/mtd > > Signed-off-by: Joe Perches Applied

Re: [PATCH v2 1/2] hp-wmi: Do not shadow error values

2017-04-19 Thread Darren Hart
On Wed, Apr 19, 2017 at 07:41:53PM +0200, Carlo Caione wrote: > From: Carlo Caione > > All the helper functions (i.e. hp_wmi_dock_state, hp_wmi_tablet_state, > ...) using hp_wmi_perform_query to perform an HP WMI query shadow the > returned value in case of error. > > We

Re: [PATCH v2 1/2] hp-wmi: Do not shadow error values

2017-04-19 Thread Darren Hart
On Wed, Apr 19, 2017 at 07:41:53PM +0200, Carlo Caione wrote: > From: Carlo Caione > > All the helper functions (i.e. hp_wmi_dock_state, hp_wmi_tablet_state, > ...) using hp_wmi_perform_query to perform an HP WMI query shadow the > returned value in case of error. > > We return -EINVAL only

Re: [PATCH] blutooth: try to improve CONFIG_SERIAL_DEV_BUS dependency

2017-04-19 Thread Marcel Holtmann
Hi Arnd, > With CONFIG_SERIAL_DEV_BUS=m, the hci_serdev.o file does not actually > get built into hci_uart.o as the Makefile doesn't pick it up, leading > to a link error with anything referring to it: > > ERROR: "hci_uart_register_device" [drivers/bluetooth/hci_nokia.ko] undefined! >

Re: [PATCH] blutooth: try to improve CONFIG_SERIAL_DEV_BUS dependency

2017-04-19 Thread Marcel Holtmann
Hi Arnd, > With CONFIG_SERIAL_DEV_BUS=m, the hci_serdev.o file does not actually > get built into hci_uart.o as the Makefile doesn't pick it up, leading > to a link error with anything referring to it: > > ERROR: "hci_uart_register_device" [drivers/bluetooth/hci_nokia.ko] undefined! >

[PATCH] rtc: m41t80: Add proper compatible for rv4162

2017-04-19 Thread Alexandre Belloni
The correct compatible for the rv4162 (microcrystal,rv4162) was not used upstream and so was not added by eb235c561d04e. Signed-off-by: Alexandre Belloni --- drivers/rtc/rtc-m41t80.c | 5 + 1 file changed, 5 insertions(+) diff --git

[PATCH] rtc: m41t80: Add proper compatible for rv4162

2017-04-19 Thread Alexandre Belloni
The correct compatible for the rv4162 (microcrystal,rv4162) was not used upstream and so was not added by eb235c561d04e. Signed-off-by: Alexandre Belloni --- drivers/rtc/rtc-m41t80.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c

Re: fs/notify/inotify: slab-out-of-bounds write in strcpy

2017-04-19 Thread Dmitry Vyukov
On Wed, Apr 19, 2017 at 9:31 PM, Dmitry Vyukov wrote: > On Wed, Apr 12, 2017 at 5:58 AM, Andrew Zhu Aday > wrote: >> Hi all, >> >> Running syzkaller we've found a "slab-out-of-bounds write in strcpy" error. >> >> Using kernel 4.10-rc7 from

Re: fs/notify/inotify: slab-out-of-bounds write in strcpy

2017-04-19 Thread Dmitry Vyukov
On Wed, Apr 19, 2017 at 9:31 PM, Dmitry Vyukov wrote: > On Wed, Apr 12, 2017 at 5:58 AM, Andrew Zhu Aday > wrote: >> Hi all, >> >> Running syzkaller we've found a "slab-out-of-bounds write in strcpy" error. >> >> Using kernel 4.10-rc7 from www.kernel.org/pub/linux/kernel/v4.x/testing/ >> >>

Re: [PATCH 1/2] clk: ti: divider: try to fix ti_clk_register_divider

2017-04-19 Thread Tony Lindgren
* Arnd Bergmann [170419 10:48]: > The newly introduced function is entirely bogus as I found when looking > at this warning: > > drivers/clk/ti/divider.c: In function 'ti_clk_register_divider': > drivers/clk/ti/divider.c:460:8: error: 'reg' may be used uninitialized in > this

Re: [PATCH 1/2] clk: ti: divider: try to fix ti_clk_register_divider

2017-04-19 Thread Tony Lindgren
* Arnd Bergmann [170419 10:48]: > The newly introduced function is entirely bogus as I found when looking > at this warning: > > drivers/clk/ti/divider.c: In function 'ti_clk_register_divider': > drivers/clk/ti/divider.c:460:8: error: 'reg' may be used uninitialized in > this function

[PATCH] pinctrl: make artpec6 explicitly non-modular

2017-04-19 Thread Paul Gortmaker
The Kconfig currently controlling compilation of this code is: drivers/pinctrl/Kconfig:config PINCTRL_ARTPEC6 drivers/pinctrl/Kconfig:bool "Axis ARTPEC-6 pin controller driver" ...meaning that it currently is not being built as a module by anyone. Lets remove the modular code that is

[PATCH] pinctrl: make artpec6 explicitly non-modular

2017-04-19 Thread Paul Gortmaker
The Kconfig currently controlling compilation of this code is: drivers/pinctrl/Kconfig:config PINCTRL_ARTPEC6 drivers/pinctrl/Kconfig:bool "Axis ARTPEC-6 pin controller driver" ...meaning that it currently is not being built as a module by anyone. Lets remove the modular code that is

Re: [v4.9-rt PATCH] ARM: mm: remove tasklist locking from update_sections_early()

2017-04-19 Thread Kees Cook
On Wed, Apr 19, 2017 at 12:58 PM, Grygorii Strashko wrote: > > > On 04/18/2017 07:15 PM, Kees Cook wrote: >> On Tue, Apr 18, 2017 at 1:48 PM, Grygorii Strashko >> wrote: >>> The below backtrace can be observed on -rt kernel with

Re: [v4.9-rt PATCH] ARM: mm: remove tasklist locking from update_sections_early()

2017-04-19 Thread Kees Cook
On Wed, Apr 19, 2017 at 12:58 PM, Grygorii Strashko wrote: > > > On 04/18/2017 07:15 PM, Kees Cook wrote: >> On Tue, Apr 18, 2017 at 1:48 PM, Grygorii Strashko >> wrote: >>> The below backtrace can be observed on -rt kernel with CONFIG_DEBUG_RODATA >>> option enabled: >>> >>> BUG: sleeping

Re: [v4.9-rt PATCH] ARM: mm: remove tasklist locking from update_sections_early()

2017-04-19 Thread Grygorii Strashko
On 04/18/2017 07:15 PM, Kees Cook wrote: > On Tue, Apr 18, 2017 at 1:48 PM, Grygorii Strashko > wrote: >> The below backtrace can be observed on -rt kernel with CONFIG_DEBUG_RODATA >> option enabled: >> >> BUG: sleeping function called from invalid context at >>

Re: [v4.9-rt PATCH] ARM: mm: remove tasklist locking from update_sections_early()

2017-04-19 Thread Grygorii Strashko
On 04/18/2017 07:15 PM, Kees Cook wrote: > On Tue, Apr 18, 2017 at 1:48 PM, Grygorii Strashko > wrote: >> The below backtrace can be observed on -rt kernel with CONFIG_DEBUG_RODATA >> option enabled: >> >> BUG: sleeping function called from invalid context at >> kernel/locking/rtmutex.c:993

Re: [PATCH] netfilter: ctnetlink: Make some parameters integer to avoid enum mismatch

2017-04-19 Thread Matthias Kaehlcke
El Wed, Apr 19, 2017 at 12:41:10PM -0700 Joe Perches ha dit: > On Wed, 2017-04-19 at 11:39 -0700, Matthias Kaehlcke wrote: > > Not all parameters passed to ctnetlink_parse_tuple() and > > ctnetlink_exp_dump_tuple() match the enum type in the signatures of these > > functions. > > Maybe that

Re: [PATCH] netfilter: ctnetlink: Make some parameters integer to avoid enum mismatch

2017-04-19 Thread Matthias Kaehlcke
El Wed, Apr 19, 2017 at 12:41:10PM -0700 Joe Perches ha dit: > On Wed, 2017-04-19 at 11:39 -0700, Matthias Kaehlcke wrote: > > Not all parameters passed to ctnetlink_parse_tuple() and > > ctnetlink_exp_dump_tuple() match the enum type in the signatures of these > > functions. > > Maybe that

[PATCH 5/5] clk: mvebu: Delete an unnecessary variable initialisation in kirkwood_fix_sscg_deviation()

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 21:31:43 +0200 A pointer is immediately assigned to the local variable "sscg_np". Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring ---

[PATCH 5/5] clk: mvebu: Delete an unnecessary variable initialisation in kirkwood_fix_sscg_deviation()

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 21:31:43 +0200 A pointer is immediately assigned to the local variable "sscg_np". Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/clk/mvebu/common.c | 2 +- 1 file changed, 1 insertion(+), 1

[GIT PULL] Backlight for v4.11

2017-04-19 Thread Daniel Thompson
Hi Linus Normally pull requests for backlight come from Lee Jones (and will continue to do so) but the bug fixed here is annoying for few people so I'm providing a little holiday cover. The following changes since commit 4f7d029b9bf009fbee76bb10c0c4351a1870d2f3: Linux 4.11-rc7 (2017-04-16

[GIT PULL] Backlight for v4.11

2017-04-19 Thread Daniel Thompson
Hi Linus Normally pull requests for backlight come from Lee Jones (and will continue to do so) but the bug fixed here is annoying for few people so I'm providing a little holiday cover. The following changes since commit 4f7d029b9bf009fbee76bb10c0c4351a1870d2f3: Linux 4.11-rc7 (2017-04-16

[PATCH 4/5] clk: mvebu: Adjust two checks for null pointers in kirkwood_fix_sscg_deviation()

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 21:17:19 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus

[PATCH 4/5] clk: mvebu: Adjust two checks for null pointers in kirkwood_fix_sscg_deviation()

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 21:17:19 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus fix the affected source code

[PATCH 3/5] clk: mvebu: Use kcalloc() in two functions

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 21:08:54 +0200 * Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus use the corresponding function "kcalloc". This issue was detected by

[PATCH 3/5] clk: mvebu: Use kcalloc() in two functions

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 21:08:54 +0200 * Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus use the corresponding function "kcalloc". This issue was detected by using the Coccinelle software.

[PATCH 2/5] clk: mvebu: Adjust two checks for null pointers in of_cpu_clk_setup()

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 20:25:59 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus

[PATCH 2/5] clk: mvebu: Adjust two checks for null pointers in of_cpu_clk_setup()

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 20:25:59 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus fix the affected source code

Re: irq/affinity: Fix extra vecs calculation

2017-04-19 Thread Andrei Vagin
On Wed, Apr 19, 2017 at 01:03:59PM -0400, Keith Busch wrote: > On Wed, Apr 19, 2017 at 09:20:27AM -0700, Andrei Vagin wrote: > > Hi, > > > > Something is wrong with this patch. We run CRIU tests for upstream kernels. > > And we found that a kernel with this patch can't be booted. > > > >

Re: irq/affinity: Fix extra vecs calculation

2017-04-19 Thread Andrei Vagin
On Wed, Apr 19, 2017 at 01:03:59PM -0400, Keith Busch wrote: > On Wed, Apr 19, 2017 at 09:20:27AM -0700, Andrei Vagin wrote: > > Hi, > > > > Something is wrong with this patch. We run CRIU tests for upstream kernels. > > And we found that a kernel with this patch can't be booted. > > > >

[PATCH] perf, tools: Fix display of data source snoop indication

2017-04-19 Thread Andi Kleen
From: Andi Kleen perf mem report doesn't display the data source snoop indication correctly. In the kernel API the definition is but the table used by the perf tools exchanged Hit and Miss "None", "Miss", "Hit", Fix the table in perf. Cc:

[PATCH] perf, tools: Fix display of data source snoop indication

2017-04-19 Thread Andi Kleen
From: Andi Kleen perf mem report doesn't display the data source snoop indication correctly. In the kernel API the definition is but the table used by the perf tools exchanged Hit and Miss "None", "Miss", "Hit", Fix the table in perf. Cc: eran...@google.com

[PATCH 1/5] clk: mvebu: Use kcalloc() in of_cpu_clk_setup()

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 20:15:21 +0200 Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus use the corresponding function "kcalloc". This issue was detected by using

[PATCH 1/5] clk: mvebu: Use kcalloc() in of_cpu_clk_setup()

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 20:15:21 +0200 Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus use the corresponding function "kcalloc". This issue was detected by using the Coccinelle software.

[PATCH 0/5] clk-mvebu: Fine-tuning for four function implementations

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 21:45:43 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (5): Use kcalloc() in of_cpu_clk_setup() Adjust two checks for null pointers in of_cpu_clk_setup()

[PATCH 0/5] clk-mvebu: Fine-tuning for four function implementations

2017-04-19 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 19 Apr 2017 21:45:43 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (5): Use kcalloc() in of_cpu_clk_setup() Adjust two checks for null pointers in of_cpu_clk_setup() Use kcalloc() in two functions

[PATCH net v3] bridge: ebtables: fix reception of frames DNAT-ed to bridge device/port

2017-04-19 Thread Linus Lüssing
When trying to redirect bridged frames to the bridge device itself or a bridge port (brouting) via the dnat target then this currently fails: The ethernet destination of the frame is dnat'ed to the MAC address of the bridge device or port just fine. However, the IP code drops it in the beginning

[PATCH net v3] bridge: ebtables: fix reception of frames DNAT-ed to bridge device/port

2017-04-19 Thread Linus Lüssing
When trying to redirect bridged frames to the bridge device itself or a bridge port (brouting) via the dnat target then this currently fails: The ethernet destination of the frame is dnat'ed to the MAC address of the bridge device or port just fine. However, the IP code drops it in the beginning

Re: [PATCH V2] clk: hi6220: Add the hi655x's pmic clock

2017-04-19 Thread Daniel Lezcano
On Wed, Apr 19, 2017 at 09:00:05AM -0700, Stephen Boyd wrote: > On 04/16, Daniel Lezcano wrote: > > On Wed, Apr 12, 2017 at 08:02:45AM -0700, Stephen Boyd wrote: > > > On 04/08, Daniel Lezcano wrote: [ ... ] > > > > + ret = clk_hw_register_clkdev(_clk->clk_hw, clk_name, > > > > NULL); > >

Re: [PATCH V2] clk: hi6220: Add the hi655x's pmic clock

2017-04-19 Thread Daniel Lezcano
On Wed, Apr 19, 2017 at 09:00:05AM -0700, Stephen Boyd wrote: > On 04/16, Daniel Lezcano wrote: > > On Wed, Apr 12, 2017 at 08:02:45AM -0700, Stephen Boyd wrote: > > > On 04/08, Daniel Lezcano wrote: [ ... ] > > > > + ret = clk_hw_register_clkdev(_clk->clk_hw, clk_name, > > > > NULL); > >

Re: [PATCH v3] usb: dwc3: add disable u2mac linestate check quirk

2017-04-19 Thread Guenter Roeck
On Wed, Apr 19, 2017 at 5:11 AM, William Wu wrote: > This patch adds a quirk to disable USB 2.0 MAC linestate check > during HS transmit. Refer the dwc3 databook, we can use it for > some special platforms if the linestate not reflect the expected > line state(J) during

Re: [PATCH v3] usb: dwc3: add disable u2mac linestate check quirk

2017-04-19 Thread Guenter Roeck
On Wed, Apr 19, 2017 at 5:11 AM, William Wu wrote: > This patch adds a quirk to disable USB 2.0 MAC linestate check > during HS transmit. Refer the dwc3 databook, we can use it for > some special platforms if the linestate not reflect the expected > line state(J) during transmission. > > When use

Re: [PATCH] arm64: xen: Implement EFI reset_system callback

2017-04-19 Thread Daniel Kiper
On Wed, Apr 19, 2017 at 08:37:38PM +0100, Matt Fleming wrote: > On Wed, 19 Apr, at 09:29:06PM, Daniel Kiper wrote: > > On Tue, Apr 18, 2017 at 02:46:50PM +0100, Matt Fleming wrote: > > > On Thu, 06 Apr, at 04:55:11PM, Mark Rutland wrote: > > > > > > > > Please, let's keep the Xen knowledge

Re: [PATCH] arm64: xen: Implement EFI reset_system callback

2017-04-19 Thread Daniel Kiper
On Wed, Apr 19, 2017 at 08:37:38PM +0100, Matt Fleming wrote: > On Wed, 19 Apr, at 09:29:06PM, Daniel Kiper wrote: > > On Tue, Apr 18, 2017 at 02:46:50PM +0100, Matt Fleming wrote: > > > On Thu, 06 Apr, at 04:55:11PM, Mark Rutland wrote: > > > > > > > > Please, let's keep the Xen knowledge

Re: [RFC 0/8] Copy Offload with Peer-to-Peer PCI Memory

2017-04-19 Thread Logan Gunthorpe
On 19/04/17 01:31 PM, Jason Gunthorpe wrote: > Try it with VT-D turned on. It shouldn't work or there is a notable > security hole in your platform.. Ah, ok. >>> const struct dma_map_ops *comp_ops = get_dma_ops(completer); >>> const struct dma_map_ops *init_ops =

Re: [RFC 0/8] Copy Offload with Peer-to-Peer PCI Memory

2017-04-19 Thread Logan Gunthorpe
On 19/04/17 01:31 PM, Jason Gunthorpe wrote: > Try it with VT-D turned on. It shouldn't work or there is a notable > security hole in your platform.. Ah, ok. >>> const struct dma_map_ops *comp_ops = get_dma_ops(completer); >>> const struct dma_map_ops *init_ops =

Re: [PATCH v5 5/8] i2c: i2c-cbus-gpio: Add vendor prefix to retu node in example

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:56PM -0300, Javier Martinez Canillas wrote: > The example contains a device node for a retu device, but > its compatible string doesn't have a vendor prefix. > > While being there, drop the -mfd suffix since isn't correct. > > Signed-off-by: Javier Martinez Canillas

Re: [PATCH v5 5/8] i2c: i2c-cbus-gpio: Add vendor prefix to retu node in example

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:56PM -0300, Javier Martinez Canillas wrote: > The example contains a device node for a retu device, but > its compatible string doesn't have a vendor prefix. > > While being there, drop the -mfd suffix since isn't correct. > > Signed-off-by: Javier Martinez Canillas

Re: [PATCH] netfilter: ctnetlink: Make some parameters integer to avoid enum mismatch

2017-04-19 Thread Joe Perches
On Wed, 2017-04-19 at 11:39 -0700, Matthias Kaehlcke wrote: > Not all parameters passed to ctnetlink_parse_tuple() and > ctnetlink_exp_dump_tuple() match the enum type in the signatures of these > functions. Maybe that should be changed/fixed. > Since this is intended change the argument type of

Re: [PATCH] netfilter: ctnetlink: Make some parameters integer to avoid enum mismatch

2017-04-19 Thread Joe Perches
On Wed, 2017-04-19 at 11:39 -0700, Matthias Kaehlcke wrote: > Not all parameters passed to ctnetlink_parse_tuple() and > ctnetlink_exp_dump_tuple() match the enum type in the signatures of these > functions. Maybe that should be changed/fixed. > Since this is intended change the argument type of

Re: [v5,4/8] ARM: dts: n8x0: Add vendor prefix to retu node

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:55PM -0300, Javier Martinez Canillas wrote: > The retu device node doesn't have a vendor prefix > in its compatible string, fix it by adding one. > > Signed-off-by: Javier Martinez Canillas > Acked-by: Rob Herring >

Re: [v5,4/8] ARM: dts: n8x0: Add vendor prefix to retu node

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:55PM -0300, Javier Martinez Canillas wrote: > The retu device node doesn't have a vendor prefix > in its compatible string, fix it by adding one. > > Signed-off-by: Javier Martinez Canillas > Acked-by: Rob Herring > Acked-by: Aaro Koskinen > Acked-by: Tony

Re: [v5,3/8] mfd: retu: Add OF device ID table

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:54PM -0300, Javier Martinez Canillas wrote: > The driver doesn't have a struct of_device_id table but supported devices > are registered via Device Trees. This is working on the assumption that a > I2C device registered via OF will always match a legacy I2C device ID

Re: [v5,3/8] mfd: retu: Add OF device ID table

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:54PM -0300, Javier Martinez Canillas wrote: > The driver doesn't have a struct of_device_id table but supported devices > are registered via Device Trees. This is working on the assumption that a > I2C device registered via OF will always match a legacy I2C device ID

Re: [v5,2/8] mfd: retu: Drop -mfd suffix from I2C device ID name

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:53PM -0300, Javier Martinez Canillas wrote: > It's not correct to encode the subsystem in the I2C device name, so > drop the -mfd suffix. To maintain bisect-ability, change driver and > platform code / DTS users in the same patch. > > Suggested-by: Lee Jones

Re: [v5,2/8] mfd: retu: Drop -mfd suffix from I2C device ID name

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:53PM -0300, Javier Martinez Canillas wrote: > It's not correct to encode the subsystem in the I2C device name, so > drop the -mfd suffix. To maintain bisect-ability, change driver and > platform code / DTS users in the same patch. > > Suggested-by: Lee Jones >

Re: [v5,1/8] dt-bindings: mfd: Add retu/tahvo ASIC chips bindings

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:52PM -0300, Javier Martinez Canillas wrote: > There are Device Tree source files defining a device node for the > retu/tahvo I2C chip, but there isn't a DT binding document for it. > > Signed-off-by: Javier Martinez Canillas > Acked-by: Rob

Re: [v5,1/8] dt-bindings: mfd: Add retu/tahvo ASIC chips bindings

2017-04-19 Thread Wolfram Sang
On Wed, Apr 12, 2017 at 02:27:52PM -0300, Javier Martinez Canillas wrote: > There are Device Tree source files defining a device node for the > retu/tahvo I2C chip, but there isn't a DT binding document for it. > > Signed-off-by: Javier Martinez Canillas > Acked-by: Rob Herring > Acked-by: Aaro

Re: [PATCH] arm64: xen: Implement EFI reset_system callback

2017-04-19 Thread Matt Fleming
On Wed, 19 Apr, at 09:29:06PM, Daniel Kiper wrote: > On Tue, Apr 18, 2017 at 02:46:50PM +0100, Matt Fleming wrote: > > On Thu, 06 Apr, at 04:55:11PM, Mark Rutland wrote: > > > > > > Please, let's keep the Xen knowledge constrained to the Xen EFI wrapper, > > > rather than spreading it further. > >

Re: [PATCH] arm64: xen: Implement EFI reset_system callback

2017-04-19 Thread Matt Fleming
On Wed, 19 Apr, at 09:29:06PM, Daniel Kiper wrote: > On Tue, Apr 18, 2017 at 02:46:50PM +0100, Matt Fleming wrote: > > On Thu, 06 Apr, at 04:55:11PM, Mark Rutland wrote: > > > > > > Please, let's keep the Xen knowledge constrained to the Xen EFI wrapper, > > > rather than spreading it further. > >

Re: [PATCH 2/3] drm/vc4: Don't try to initialize FBDEV if we're only bound to V3D.

2017-04-19 Thread Daniel Vetter
On Wed, Apr 19, 2017 at 7:55 PM, Eric Anholt wrote: > Daniel Vetter writes: >> On Tue, Apr 18, 2017 at 9:11 PM, Eric Anholt wrote: >>> The FBDEV initialization would throw an error in dmesg, when we just >>> want to silently not initialize

Re: [PATCH 2/3] drm/vc4: Don't try to initialize FBDEV if we're only bound to V3D.

2017-04-19 Thread Daniel Vetter
On Wed, Apr 19, 2017 at 7:55 PM, Eric Anholt wrote: > Daniel Vetter writes: >> On Tue, Apr 18, 2017 at 9:11 PM, Eric Anholt wrote: >>> The FBDEV initialization would throw an error in dmesg, when we just >>> want to silently not initialize fbdev on a V3D-only VC4 instance. >>> >>>

[PATCH v2] dma-buf: Rename dma-ops to prevent conflict with kunmap_atomic macro

2017-04-19 Thread Logan Gunthorpe
Seeing the kunmap_atomic dma_buf_ops share the same name with a macro in highmem.h, the former can be aliased if any dma-buf user includes that header. I'm personally trying to include highmem.h inside scatterlist.h and this breaks the dma-buf code proper. Christoph Hellwig suggested [1]

[PATCH v2] dma-buf: Rename dma-ops to prevent conflict with kunmap_atomic macro

2017-04-19 Thread Logan Gunthorpe
Seeing the kunmap_atomic dma_buf_ops share the same name with a macro in highmem.h, the former can be aliased if any dma-buf user includes that header. I'm personally trying to include highmem.h inside scatterlist.h and this breaks the dma-buf code proper. Christoph Hellwig suggested [1]

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