Re: [RFC PATCH v2 1/2] ata: ahci: Support state with min power and Partial low power state

2018-07-13 Thread Hans de Goede
Hi, On 13-07-18 00:27, Srinivas Pandruvada wrote: Currently when min_power policy is selected, the partial low power state is not entered and link will try aggressively enter to only slumber state. Add a new policy which still enable DEVSLP but also try to enter partial low power state. For

Re: [RFC PATCH v2 1/2] ata: ahci: Support state with min power and Partial low power state

2018-07-13 Thread Hans de Goede
Hi, On 13-07-18 00:27, Srinivas Pandruvada wrote: Currently when min_power policy is selected, the partial low power state is not entered and link will try aggressively enter to only slumber state. Add a new policy which still enable DEVSLP but also try to enter partial low power state. For

Re: [PATCH v2] tools/memory-model: Add extra ordering for locks and remove it for ordinary release/acquire

2018-07-13 Thread Andrea Parri
On Thu, Jul 12, 2018 at 07:05:39PM -0700, Daniel Lustig wrote: > On 7/12/2018 11:10 AM, Linus Torvalds wrote: > > On Thu, Jul 12, 2018 at 11:05 AM Peter Zijlstra > > wrote: > >> > >> The locking pattern is fairly simple and shows where RCpc comes apart > >> from expectation real nice. > > > >

Re: [PATCH v2] tools/memory-model: Add extra ordering for locks and remove it for ordinary release/acquire

2018-07-13 Thread Andrea Parri
On Thu, Jul 12, 2018 at 07:05:39PM -0700, Daniel Lustig wrote: > On 7/12/2018 11:10 AM, Linus Torvalds wrote: > > On Thu, Jul 12, 2018 at 11:05 AM Peter Zijlstra > > wrote: > >> > >> The locking pattern is fairly simple and shows where RCpc comes apart > >> from expectation real nice. > > > >

[PATCH] tty: serial: jsm: remove redundant pointer ch

2018-07-13 Thread Colin King
From: Colin Ian King Pointer ch is being assigned but is never used hence it is redundant and can be removed. Cleans up clang warning: warning: variable 'ch' set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King --- drivers/tty/serial/jsm/jsm_tty.c | 3 +-- 1 file

[PATCH] tty: serial: jsm: remove redundant pointer ch

2018-07-13 Thread Colin King
From: Colin Ian King Pointer ch is being assigned but is never used hence it is redundant and can be removed. Cleans up clang warning: warning: variable 'ch' set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King --- drivers/tty/serial/jsm/jsm_tty.c | 3 +-- 1 file

[PATCH] thunderbolt: remove redundant variable 'approved'

2018-07-13 Thread Colin King
From: Colin Ian King Variable 'approved' is being assigned but is never used hence it is redundant and can be removed. Cleans up clang warning: warning: variable 'approved' set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King --- drivers/thunderbolt/icm.c | 2 -- 1 file

[PATCH] thunderbolt: remove redundant variable 'approved'

2018-07-13 Thread Colin King
From: Colin Ian King Variable 'approved' is being assigned but is never used hence it is redundant and can be removed. Cleans up clang warning: warning: variable 'approved' set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King --- drivers/thunderbolt/icm.c | 2 -- 1 file

Re: [PATCH v2 2/2] pinctrl: nuvoton: add NPCM7xx pinctrl and GPIO driver

2018-07-13 Thread Linus Walleij
On Thu, Jul 12, 2018 at 11:42 PM Tomer Maimon wrote: > Add Nuvoton BMC NPCM750/730/715/705 Pinmux and > GPIO controller driver. > > Signed-off-by: Tomer Maimon (...) > +++ b/drivers/pinctrl/nuvoton/pinctrl-npcm7xx.c > @@ -0,0 +1,2089 @@ > +// SPDX-License-Identifier: GPL-2.0 > +// Copyright

Re: [PATCH v2 2/2] pinctrl: nuvoton: add NPCM7xx pinctrl and GPIO driver

2018-07-13 Thread Linus Walleij
On Thu, Jul 12, 2018 at 11:42 PM Tomer Maimon wrote: > Add Nuvoton BMC NPCM750/730/715/705 Pinmux and > GPIO controller driver. > > Signed-off-by: Tomer Maimon (...) > +++ b/drivers/pinctrl/nuvoton/pinctrl-npcm7xx.c > @@ -0,0 +1,2089 @@ > +// SPDX-License-Identifier: GPL-2.0 > +// Copyright

RE: [PATCH v2 4/6] arm: dts: add support for Laird WB50N cpu module and DVK

2018-07-13 Thread Ben Whitten
> Subject: Re: [PATCH v2 4/6] arm: dts: add support for Laird > WB50N cpu module and DVK > > Hi, > > I've now applied the whole series after fixing two small > whitespace > issues. Thanks! > On 15/06/2018 14:40:53+0100, Ben Whitten wrote: > > +_clk { > > + atmel,clk-output-range = <0

RE: [PATCH v2 4/6] arm: dts: add support for Laird WB50N cpu module and DVK

2018-07-13 Thread Ben Whitten
> Subject: Re: [PATCH v2 4/6] arm: dts: add support for Laird > WB50N cpu module and DVK > > Hi, > > I've now applied the whole series after fixing two small > whitespace > issues. Thanks! > On 15/06/2018 14:40:53+0100, Ben Whitten wrote: > > +_clk { > > + atmel,clk-output-range = <0

Re: [PATCH] kconfig: remove EXPERT from CHECKPOINT_RESTORE

2018-07-13 Thread Pavel Emelyanov
On 07/12/2018 07:33 PM, Eric W. Biederman wrote: > > Adrian Reber writes: > >> The CHECKPOINT_RESTORE configuration option was introduced in 2012 and >> combined with EXPERT. CHECKPOINT_RESTORE is already enabled in many >> distribution kernels and also part of the defconfigs of various >>

Re: [PATCH] kconfig: remove EXPERT from CHECKPOINT_RESTORE

2018-07-13 Thread Pavel Emelyanov
On 07/12/2018 07:33 PM, Eric W. Biederman wrote: > > Adrian Reber writes: > >> The CHECKPOINT_RESTORE configuration option was introduced in 2012 and >> combined with EXPERT. CHECKPOINT_RESTORE is already enabled in many >> distribution kernels and also part of the defconfigs of various >>

Re: [PATCH] staging: speakup: fix wraparound in uaccess length check

2018-07-13 Thread Greg Kroah-Hartman
On Thu, Jul 12, 2018 at 04:12:39PM -0700, Jann Horn wrote: > On Thu, Jul 12, 2018 at 3:47 PM Al Viro wrote: > > > > On Fri, Jul 13, 2018 at 12:29:36AM +0200, Jann Horn wrote: > > > From: Samuel Thibault > > > > > > From: Samuel Thibault > > > > > > If softsynthx_read() is called with `count <

Re: [PATCH] staging: speakup: fix wraparound in uaccess length check

2018-07-13 Thread Greg Kroah-Hartman
On Thu, Jul 12, 2018 at 04:12:39PM -0700, Jann Horn wrote: > On Thu, Jul 12, 2018 at 3:47 PM Al Viro wrote: > > > > On Fri, Jul 13, 2018 at 12:29:36AM +0200, Jann Horn wrote: > > > From: Samuel Thibault > > > > > > From: Samuel Thibault > > > > > > If softsynthx_read() is called with `count <

Re: [PATCH 1/2] dt-bindings: pfuze100: add optional pfuze-disable-sw binding

2018-07-13 Thread Marco Felsch
Hi Mark, On 18-07-12 16:31, Mark Brown wrote: > On Thu, Jul 12, 2018 at 01:02:39PM +0200, Marco Felsch wrote: > > > +Optional properties: > > +- pfuze-disable-sw: Disable all unused switch regulators to save power > > + consumption. Attention, some platforms are using the switch regulators > >

Re: [PATCH 1/2] dt-bindings: pfuze100: add optional pfuze-disable-sw binding

2018-07-13 Thread Marco Felsch
Hi Mark, On 18-07-12 16:31, Mark Brown wrote: > On Thu, Jul 12, 2018 at 01:02:39PM +0200, Marco Felsch wrote: > > > +Optional properties: > > +- pfuze-disable-sw: Disable all unused switch regulators to save power > > + consumption. Attention, some platforms are using the switch regulators > >

Re: [PATCH v3 3/3] clk: qcom: Add display clock controller driver for SDM845

2018-07-13 Thread spanda
On 2018-07-13 01:11, Stephen Boyd wrote: Quoting Taniya Das (2018-07-12 10:21:33) ++ Display driver team, On 7/9/2018 8:36 PM, Stephen Boyd wrote: > Quoting Taniya Das (2018-07-09 02:34:07) >> >> >> On 7/9/2018 1:07 PM, Stephen Boyd wrote: >>> Quoting Taniya Das (2018-07-09 00:07:21)

Re: [PATCH v3 3/3] clk: qcom: Add display clock controller driver for SDM845

2018-07-13 Thread spanda
On 2018-07-13 01:11, Stephen Boyd wrote: Quoting Taniya Das (2018-07-12 10:21:33) ++ Display driver team, On 7/9/2018 8:36 PM, Stephen Boyd wrote: > Quoting Taniya Das (2018-07-09 02:34:07) >> >> >> On 7/9/2018 1:07 PM, Stephen Boyd wrote: >>> Quoting Taniya Das (2018-07-09 00:07:21)

Re: [PATCH] kconfig: remove EXPERT from CHECKPOINT_RESTORE

2018-07-13 Thread Pavel Emelyanov
On 07/12/2018 04:07 PM, Adrian Reber wrote: > The CHECKPOINT_RESTORE configuration option was introduced in 2012 and > combined with EXPERT. CHECKPOINT_RESTORE is already enabled in many > distribution kernels and also part of the defconfigs of various > architectures. > > To make it easier for

Re: [PATCH] kconfig: remove EXPERT from CHECKPOINT_RESTORE

2018-07-13 Thread Pavel Emelyanov
On 07/12/2018 04:07 PM, Adrian Reber wrote: > The CHECKPOINT_RESTORE configuration option was introduced in 2012 and > combined with EXPERT. CHECKPOINT_RESTORE is already enabled in many > distribution kernels and also part of the defconfigs of various > architectures. > > To make it easier for

Re: [PATCH 1/3] tty/serial_core: add ISO7816 infrastructure

2018-07-13 Thread Greg KH
On Fri, Jul 13, 2018 at 09:56:48AM +0200, Ludovic Desroches wrote: > On Thu, Jul 12, 2018 at 05:02:29PM +0200, Greg KH wrote: > > On Wed, Jul 11, 2018 at 03:16:36PM +0200, Ludovic Desroches wrote: > > > From: Nicolas Ferre > > > > > > Add the ISO7816 ioctl and associated accessors and data

Re: [PATCH 1/3] tty/serial_core: add ISO7816 infrastructure

2018-07-13 Thread Greg KH
On Fri, Jul 13, 2018 at 09:56:48AM +0200, Ludovic Desroches wrote: > On Thu, Jul 12, 2018 at 05:02:29PM +0200, Greg KH wrote: > > On Wed, Jul 11, 2018 at 03:16:36PM +0200, Ludovic Desroches wrote: > > > From: Nicolas Ferre > > > > > > Add the ISO7816 ioctl and associated accessors and data

Re: [PATCH] pinctrl: mt7622: fix probe fail by misuse the selector

2018-07-13 Thread Linus Walleij
On Thu, Jul 12, 2018 at 7:50 AM wrote: > From: Sean Wang > > After the commit acf137951367 ("pinctrl: core: Return selector to the > pinctrl driver") and the commit 47f1242d19c3 ("pinctrl: pinmux: Return > selector to the pinctrl driver"), it's necessary to add the fixes > needed for the pin

Re: [PATCH] pinctrl: mt7622: fix probe fail by misuse the selector

2018-07-13 Thread Linus Walleij
On Thu, Jul 12, 2018 at 7:50 AM wrote: > From: Sean Wang > > After the commit acf137951367 ("pinctrl: core: Return selector to the > pinctrl driver") and the commit 47f1242d19c3 ("pinctrl: pinmux: Return > selector to the pinctrl driver"), it's necessary to add the fixes > needed for the pin

Re: [PATCH v2 3/6] ARM: trusted_foundations: do not use naked function

2018-07-13 Thread Stefan Agner
On 13.07.2018 01:01, Russell King - ARM Linux wrote: > On Thu, Jul 12, 2018 at 03:43:10PM -0700, Kees Cook wrote: >> On Tue, Apr 17, 2018 at 1:11 AM, Thierry Reding wrote: >> > On Mon, Apr 16, 2018 at 08:21:09PM +0200, Stefan Agner wrote: >> >> On 16.04.2018 18:08, Stephen Warren wrote: >> >> >

Re: [PATCH v2 3/6] ARM: trusted_foundations: do not use naked function

2018-07-13 Thread Stefan Agner
On 13.07.2018 01:01, Russell King - ARM Linux wrote: > On Thu, Jul 12, 2018 at 03:43:10PM -0700, Kees Cook wrote: >> On Tue, Apr 17, 2018 at 1:11 AM, Thierry Reding wrote: >> > On Mon, Apr 16, 2018 at 08:21:09PM +0200, Stefan Agner wrote: >> >> On 16.04.2018 18:08, Stephen Warren wrote: >> >> >

[PATCH 2/3] ARM: dts: imx6sll-evk: correct lcd regulator GPIO pin

2018-07-13 Thread Anson Huang
On i.MX6SLL EVK board, lcd regulator is controlled by GPIO4 IO03 using MX6SLL_PAD_KEY_ROW5__GPIO4_IO03 pin, NOT MX6SLL_PAD_ECSPI1_SCLK__GPIO4_IO08, correct it. Signed-off-by: Anson Huang --- arch/arm/boot/dts/imx6sll-evk.dts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 1/3] ARM: dts: imx6sll-evk: enable PWM1 for backlight driver

2018-07-13 Thread Anson Huang
Enable pwm1 module on i.MX6SLL EVK board to make backlight driver really work with LCD panel connected. Signed-off-by: Anson Huang --- arch/arm/boot/dts/imx6sll-evk.dts | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/boot/dts/imx6sll-evk.dts

[PATCH 3/3] ARM: dts: imx6sll-evk: enable SEIKO 43WVF1G lcdif panel

2018-07-13 Thread Anson Huang
Enable SEIKO 43WVF1G lcdif panel for DRM driver, add necessary properties according to SEIKO 43WVF1G driver's requirement, such as "dvdd-supply", "avdd-supply" and "backlight" etc.. Signed-off-by: Anson Huang --- arch/arm/boot/dts/imx6sll-evk.dts | 76 --- 1

[PATCH 2/3] ARM: dts: imx6sll-evk: correct lcd regulator GPIO pin

2018-07-13 Thread Anson Huang
On i.MX6SLL EVK board, lcd regulator is controlled by GPIO4 IO03 using MX6SLL_PAD_KEY_ROW5__GPIO4_IO03 pin, NOT MX6SLL_PAD_ECSPI1_SCLK__GPIO4_IO08, correct it. Signed-off-by: Anson Huang --- arch/arm/boot/dts/imx6sll-evk.dts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 1/3] ARM: dts: imx6sll-evk: enable PWM1 for backlight driver

2018-07-13 Thread Anson Huang
Enable pwm1 module on i.MX6SLL EVK board to make backlight driver really work with LCD panel connected. Signed-off-by: Anson Huang --- arch/arm/boot/dts/imx6sll-evk.dts | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/boot/dts/imx6sll-evk.dts

[PATCH 3/3] ARM: dts: imx6sll-evk: enable SEIKO 43WVF1G lcdif panel

2018-07-13 Thread Anson Huang
Enable SEIKO 43WVF1G lcdif panel for DRM driver, add necessary properties according to SEIKO 43WVF1G driver's requirement, such as "dvdd-supply", "avdd-supply" and "backlight" etc.. Signed-off-by: Anson Huang --- arch/arm/boot/dts/imx6sll-evk.dts | 76 --- 1

Re: [PATCH 1/3] tty/serial_core: add ISO7816 infrastructure

2018-07-13 Thread Ludovic Desroches
On Thu, Jul 12, 2018 at 04:58:08PM +0200, Greg KH wrote: > On Wed, Jul 11, 2018 at 03:16:36PM +0200, Ludovic Desroches wrote: > > From: Nicolas Ferre > > > > Add the ISO7816 ioctl and associated accessors and data structure. > > Drivers can then use this common implementation to handle ISO7816.

Re: [PATCH 1/3] tty/serial_core: add ISO7816 infrastructure

2018-07-13 Thread Ludovic Desroches
On Thu, Jul 12, 2018 at 04:58:08PM +0200, Greg KH wrote: > On Wed, Jul 11, 2018 at 03:16:36PM +0200, Ludovic Desroches wrote: > > From: Nicolas Ferre > > > > Add the ISO7816 ioctl and associated accessors and data structure. > > Drivers can then use this common implementation to handle ISO7816.

[PATCH] staging/rtl8192u: hide unused procfs helpers

2018-07-13 Thread YueHaibing
When CONFIG_PROC_FS isn't set, gcc warning this: drivers/staging/rtl8192u/r8192U_core.c:508:12: warning: ‘proc_get_stats_ap’ defined but not used [-Wunused-function] static int proc_get_stats_ap(struct seq_file *m, void *v) ^ drivers/staging/rtl8192u/r8192U_core.c:527:12: warning:

[PATCH] staging/rtl8192u: hide unused procfs helpers

2018-07-13 Thread YueHaibing
When CONFIG_PROC_FS isn't set, gcc warning this: drivers/staging/rtl8192u/r8192U_core.c:508:12: warning: ‘proc_get_stats_ap’ defined but not used [-Wunused-function] static int proc_get_stats_ap(struct seq_file *m, void *v) ^ drivers/staging/rtl8192u/r8192U_core.c:527:12: warning:

[PATCH] ARM: dts: imx6sl-evk: add missing GPIO iomux setting

2018-07-13 Thread Anson Huang
On i.MX6SL EVK board, the MX6SL_PAD_KEY_ROW5 pin is used as lcd 3v3 regulator control pin, need to make sure MX6SL_PAD_KEY_ROW5 is muxed as GPIO function for controlling lcd 3v3 regulator. Signed-off-by: Anson Huang --- arch/arm/boot/dts/imx6sl-evk.dts | 1 + 1 file changed, 1 insertion(+)

Re: [Cocci] [PATCH 01/18] coccinelle: change strncpy+truncation to strlcpy

2018-07-13 Thread Dominique Martinet
Himanshu Jha wrote on Fri, Jul 13, 2018: > > I expect each maintainer will pick their share of the patchs if they > > agree with it and the rest will just be dropped? > > Masahiro Yamada takes coccinelle patches, > so please cc him or your patch would be lost. Thanks, will do. > > +virtual

[PATCH] ARM: dts: imx6sl-evk: add missing GPIO iomux setting

2018-07-13 Thread Anson Huang
On i.MX6SL EVK board, the MX6SL_PAD_KEY_ROW5 pin is used as lcd 3v3 regulator control pin, need to make sure MX6SL_PAD_KEY_ROW5 is muxed as GPIO function for controlling lcd 3v3 regulator. Signed-off-by: Anson Huang --- arch/arm/boot/dts/imx6sl-evk.dts | 1 + 1 file changed, 1 insertion(+)

Re: [Cocci] [PATCH 01/18] coccinelle: change strncpy+truncation to strlcpy

2018-07-13 Thread Dominique Martinet
Himanshu Jha wrote on Fri, Jul 13, 2018: > > I expect each maintainer will pick their share of the patchs if they > > agree with it and the rest will just be dropped? > > Masahiro Yamada takes coccinelle patches, > so please cc him or your patch would be lost. Thanks, will do. > > +virtual

Re: [PATCH 1/2] dt-bindings: pfuze100: add optional pfuze-disable-sw binding

2018-07-13 Thread Marco Felsch
Hi Fabio, On 18-07-12 12:19, Fabio Estevam wrote: > On Thu, Jul 12, 2018 at 8:02 AM, Marco Felsch wrote: > > > +Optional properties: > > +- pfuze-disable-sw: Disable all unused switch regulators to save power > > + consumption. Attention, some platforms are using the switch regulators > > as

Re: [PATCH 1/2] dt-bindings: pfuze100: add optional pfuze-disable-sw binding

2018-07-13 Thread Marco Felsch
Hi Fabio, On 18-07-12 12:19, Fabio Estevam wrote: > On Thu, Jul 12, 2018 at 8:02 AM, Marco Felsch wrote: > > > +Optional properties: > > +- pfuze-disable-sw: Disable all unused switch regulators to save power > > + consumption. Attention, some platforms are using the switch regulators > > as

Re: [PATCH 1/3] tty/serial_core: add ISO7816 infrastructure

2018-07-13 Thread Ludovic Desroches
On Thu, Jul 12, 2018 at 05:02:29PM +0200, Greg KH wrote: > On Wed, Jul 11, 2018 at 03:16:36PM +0200, Ludovic Desroches wrote: > > From: Nicolas Ferre > > > > Add the ISO7816 ioctl and associated accessors and data structure. > > Drivers can then use this common implementation to handle ISO7816.

Re: [PATCH 1/3] tty/serial_core: add ISO7816 infrastructure

2018-07-13 Thread Ludovic Desroches
On Thu, Jul 12, 2018 at 05:02:29PM +0200, Greg KH wrote: > On Wed, Jul 11, 2018 at 03:16:36PM +0200, Ludovic Desroches wrote: > > From: Nicolas Ferre > > > > Add the ISO7816 ioctl and associated accessors and data structure. > > Drivers can then use this common implementation to handle ISO7816.

Re: [PATCH v3 00/12] kbuild/kconfig: do not update config during installation

2018-07-13 Thread Dirk Gouders
Masahiro Yamada writes: > 2018-07-10 20:34 GMT+09:00 Dirk Gouders : >> Masahiro Yamada writes: >> >>> The main motivation of this patch series is to suppress the syncconfig >>> during running installation targets. >>> >>> V1 consisted of only two patches: >>>

Re: [PATCH v3 00/12] kbuild/kconfig: do not update config during installation

2018-07-13 Thread Dirk Gouders
Masahiro Yamada writes: > 2018-07-10 20:34 GMT+09:00 Dirk Gouders : >> Masahiro Yamada writes: >> >>> The main motivation of this patch series is to suppress the syncconfig >>> during running installation targets. >>> >>> V1 consisted of only two patches: >>>

Re: [PATCH] pinctrl: ocelot: fix gpio4 twi function

2018-07-13 Thread Linus Walleij
On Wed, Jul 11, 2018 at 3:01 PM Alexandre Belloni wrote: > the TWI function on GPIO4 is actually a multiplexed SCL, not an original > TWI SDA or SCL. Fix it. > > Signed-off-by: Alexandre Belloni Patch applied. Yours, Linus Walleij

Re: [PATCH] pinctrl: ocelot: fix gpio4 twi function

2018-07-13 Thread Linus Walleij
On Wed, Jul 11, 2018 at 3:01 PM Alexandre Belloni wrote: > the TWI function on GPIO4 is actually a multiplexed SCL, not an original > TWI SDA or SCL. Fix it. > > Signed-off-by: Alexandre Belloni Patch applied. Yours, Linus Walleij

Re: [LINUX PATCH v11 2/3] memory: pl353: Add driver for arm pl353 static memory controller

2018-07-13 Thread Linus Walleij
On Wed, Jul 11, 2018 at 9:37 AM Naga Sureshkumar Relli wrote: > Add driver for arm pl353 static memory controller. This controller is used in > Xilinx Zynq SoC for interfacing the NAND and NOR/SRAM memory devices. > > Signed-off-by: Naga Sureshkumar Relli > --- > Changes in v11: > - Added amba

Re: [LINUX PATCH v11 2/3] memory: pl353: Add driver for arm pl353 static memory controller

2018-07-13 Thread Linus Walleij
On Wed, Jul 11, 2018 at 9:37 AM Naga Sureshkumar Relli wrote: > Add driver for arm pl353 static memory controller. This controller is used in > Xilinx Zynq SoC for interfacing the NAND and NOR/SRAM memory devices. > > Signed-off-by: Naga Sureshkumar Relli > --- > Changes in v11: > - Added amba

Re: [LINUX PATCH v11 1/3] dt-bindings: memory: Add pl353 smc controller devicetree binding information

2018-07-13 Thread Linus Walleij
On Wed, Jul 11, 2018 at 9:37 AM Naga Sureshkumar Relli wrote: > Add pl353 static memory controller devicetree binding information. > > Signed-off-by: Naga Sureshkumar Relli > --- > Changes in v11: Reviewed-by: Linus Walleij Thanks! Yours, Linus Walleij

Re: [LINUX PATCH v11 1/3] dt-bindings: memory: Add pl353 smc controller devicetree binding information

2018-07-13 Thread Linus Walleij
On Wed, Jul 11, 2018 at 9:37 AM Naga Sureshkumar Relli wrote: > Add pl353 static memory controller devicetree binding information. > > Signed-off-by: Naga Sureshkumar Relli > --- > Changes in v11: Reviewed-by: Linus Walleij Thanks! Yours, Linus Walleij

Re: [PATCH v10 0/6] Driver for at91 usart in spi mode

2018-07-13 Thread Lee Jones
On Wed, 11 Jul 2018, Nicolas Ferre wrote: > On 25/06/2018 at 19:22, Radu Pirea wrote: > > Hello, > > > > This is the second version of driver. I added a mfd driver which by > > default probes atmel_serial driver and if in dt is specified to probe > > the spi driver, then the spi-at91-usart

Re: [PATCH v10 0/6] Driver for at91 usart in spi mode

2018-07-13 Thread Lee Jones
On Wed, 11 Jul 2018, Nicolas Ferre wrote: > On 25/06/2018 at 19:22, Radu Pirea wrote: > > Hello, > > > > This is the second version of driver. I added a mfd driver which by > > default probes atmel_serial driver and if in dt is specified to probe > > the spi driver, then the spi-at91-usart

Re: [PATCH v4] regulator: fixed: Convert to use GPIO descriptor only

2018-07-13 Thread Linus Walleij
On Tue, Jul 10, 2018 at 7:56 PM Janusz Krzysztofik wrote: > > - .gpio = AMS_DELTA_GPIO_PIN_MODEM_NRESET, > > This is OK but not enough for clean build of board-ams-delta.c when merged > into current linux-next as one more struct fixed_voltage_config introduced > there >

Re: [PATCH v4] regulator: fixed: Convert to use GPIO descriptor only

2018-07-13 Thread Linus Walleij
On Tue, Jul 10, 2018 at 7:56 PM Janusz Krzysztofik wrote: > > - .gpio = AMS_DELTA_GPIO_PIN_MODEM_NRESET, > > This is OK but not enough for clean build of board-ams-delta.c when merged > into current linux-next as one more struct fixed_voltage_config introduced > there >

Re: [PATCH 6/6] mfd: rave-sp: Emulate CMD_GET_STATUS on device that don't support it

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > CMD_GET_STATUS is not supported by some devices implementing > RDU2-compatible ICD as well as "legacy" devices. To account for that > fact, add code that obtains the same information (app/bootloader FW > version) using several different commands. > >

Re: [PATCH 5/6] mfd: rave-sp: Add legacy watchdog ping command translation

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > This is needed to make rave-sp-wdt driver to properly ping the > watchdog on "legacy" firmware. > > Cc: linux-kernel@vger.kernel.org > Cc: cphe...@gmail.com > Cc: Lucas Stach > Cc: Nikita Yushchenko > Cc: Lee Jones > Signed-off-by: Andrey Smirnov

Re: [PATCH 6/6] mfd: rave-sp: Emulate CMD_GET_STATUS on device that don't support it

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > CMD_GET_STATUS is not supported by some devices implementing > RDU2-compatible ICD as well as "legacy" devices. To account for that > fact, add code that obtains the same information (app/bootloader FW > version) using several different commands. > >

Re: [PATCH 5/6] mfd: rave-sp: Add legacy watchdog ping command translation

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > This is needed to make rave-sp-wdt driver to properly ping the > watchdog on "legacy" firmware. > > Cc: linux-kernel@vger.kernel.org > Cc: cphe...@gmail.com > Cc: Lucas Stach > Cc: Nikita Yushchenko > Cc: Lee Jones > Signed-off-by: Andrey Smirnov

Re: [PATCH 4/6] mfd: rave-sp: Add legacy EEPROM access command translation

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > This is needed to make rave-sp-eeprom driver work on "legacy" > firmware. > > Cc: linux-kernel@vger.kernel.org > Cc: cphe...@gmail.com > Cc: Lucas Stach > Cc: Nikita Yushchenko > Cc: Lee Jones > Signed-off-by: Andrey Smirnov > --- >

Re: [PATCH 4/6] mfd: rave-sp: Add legacy EEPROM access command translation

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > This is needed to make rave-sp-eeprom driver work on "legacy" > firmware. > > Cc: linux-kernel@vger.kernel.org > Cc: cphe...@gmail.com > Cc: Lucas Stach > Cc: Nikita Yushchenko > Cc: Lee Jones > Signed-off-by: Andrey Smirnov > --- >

Re: [PATCH 2/6] mfd: rave-sp: Fix incorrectly specified checksum type

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > RAVE SP firmware covered by "legacy" variant uses 16-bit CCITT > checksum algorithm. Change the code to correctly reflect that. > > Cc: linux-kernel@vger.kernel.org > Cc: cphe...@gmail.com > Cc: Lucas Stach > Cc: Nikita Yushchenko > Cc: Lee Jones >

Re: [PATCH 3/6] mfd: rave-sp: Initialize flow control and parity of the port

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > Relying on serial port defaults for flow control and parity can result > in complete breakdown of communication with RAVE SP on some platforms > where defaults are not what we need them to be. One such case is > VF610-base ZII SPU3 board (not supported

Re: [PATCH 2/6] mfd: rave-sp: Fix incorrectly specified checksum type

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > RAVE SP firmware covered by "legacy" variant uses 16-bit CCITT > checksum algorithm. Change the code to correctly reflect that. > > Cc: linux-kernel@vger.kernel.org > Cc: cphe...@gmail.com > Cc: Lucas Stach > Cc: Nikita Yushchenko > Cc: Lee Jones >

Re: [PATCH 3/6] mfd: rave-sp: Initialize flow control and parity of the port

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > Relying on serial port defaults for flow control and parity can result > in complete breakdown of communication with RAVE SP on some platforms > where defaults are not what we need them to be. One such case is > VF610-base ZII SPU3 board (not supported

Re: [PATCH 1/6] mfd: rave-sp: Remove unused defines

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > Remove unusded defines that are a leftover from earlier iterations of > the driver. > > Cc: linux-kernel@vger.kernel.org > Cc: cphe...@gmail.com > Cc: Lucas Stach > Cc: Nikita Yushchenko > Cc: Lee Jones > Signed-off-by: Andrey Smirnov > --- >

Re: [PATCH 1/6] mfd: rave-sp: Remove unused defines

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Andrey Smirnov wrote: > Remove unusded defines that are a leftover from earlier iterations of > the driver. > > Cc: linux-kernel@vger.kernel.org > Cc: cphe...@gmail.com > Cc: Lucas Stach > Cc: Nikita Yushchenko > Cc: Lee Jones > Signed-off-by: Andrey Smirnov > --- >

Re: [PATCH 24/32] vfs: syscall: Add fsopen() to prepare for superblock creation [ver #9]

2018-07-13 Thread David Howells
Andy Lutomirski wrote: > > Also you can't currently directly create a bind mount from userspace as you > > can only bind from another path point - which you may not be able to access > > (either by permission failure or because it's not in your mount namespace). > > > > Are you trying to

Re: [PATCH 24/32] vfs: syscall: Add fsopen() to prepare for superblock creation [ver #9]

2018-07-13 Thread David Howells
Andy Lutomirski wrote: > > Also you can't currently directly create a bind mount from userspace as you > > can only bind from another path point - which you may not be able to access > > (either by permission failure or because it's not in your mount namespace). > > > > Are you trying to

Re: [PATCH 2/3][RFC] PM / Hibernate: Encrypt the snapshot pages before submitted to the block device

2018-07-13 Thread Yu Chen
Hi, On Thu, Jul 12, 2018 at 06:10:37PM +0800, joeyli wrote: > Hi Yu Chen, > > Sorry for my delay... > > On Fri, Jul 06, 2018 at 11:28:56PM +0800, Yu Chen wrote: > > Hi Joey Lee, > > On Fri, Jun 29, 2018 at 08:59:43PM +0800, joeyli wrote: > > > On Thu, Jun 28, 2018 at 10:52:07PM +0800, Yu Chen

Re: [PATCH 2/3][RFC] PM / Hibernate: Encrypt the snapshot pages before submitted to the block device

2018-07-13 Thread Yu Chen
Hi, On Thu, Jul 12, 2018 at 06:10:37PM +0800, joeyli wrote: > Hi Yu Chen, > > Sorry for my delay... > > On Fri, Jul 06, 2018 at 11:28:56PM +0800, Yu Chen wrote: > > Hi Joey Lee, > > On Fri, Jun 29, 2018 at 08:59:43PM +0800, joeyli wrote: > > > On Thu, Jun 28, 2018 at 10:52:07PM +0800, Yu Chen

Re: [PATCH] mfd: hi655x: Fix regmap area declared size for hi655x

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Rafael David Tinoco wrote: > Fixes: https://bugs.linaro.org/show_bug.cgi?id=3903 > > LTP Functional tests have caused a bad paging request when triggering > the regmap_read_debugfs() logic of the device PMIC Hi6553 (reading > regmap/f800.pmic/registers file during

Re: [PATCH] hostap: hide unused procfs helpers

2018-07-13 Thread Arend van Spriel
+ Randy On 7/13/2018 9:03 AM, YueHaibing wrote: When CONFIG_PROC_FS isn't set, gcc warning this: drivers/net/wireless/intersil/hostap/hostap_hw.c:2901:12: warning: ‘prism2_registers_proc_show’ defined but not used [-Wunused-function] static int prism2_registers_proc_show(struct seq_file *m,

Re: [PATCH] mfd: hi655x: Fix regmap area declared size for hi655x

2018-07-13 Thread Lee Jones
On Fri, 06 Jul 2018, Rafael David Tinoco wrote: > Fixes: https://bugs.linaro.org/show_bug.cgi?id=3903 > > LTP Functional tests have caused a bad paging request when triggering > the regmap_read_debugfs() logic of the device PMIC Hi6553 (reading > regmap/f800.pmic/registers file during

Re: [PATCH] hostap: hide unused procfs helpers

2018-07-13 Thread Arend van Spriel
+ Randy On 7/13/2018 9:03 AM, YueHaibing wrote: When CONFIG_PROC_FS isn't set, gcc warning this: drivers/net/wireless/intersil/hostap/hostap_hw.c:2901:12: warning: ‘prism2_registers_proc_show’ defined but not used [-Wunused-function] static int prism2_registers_proc_show(struct seq_file *m,

Re: [PATCH 1/2] gpio: mt7621: add OF_GPIO dependency

2018-07-13 Thread Linus Walleij
On Tue, Jul 10, 2018 at 5:19 PM Arnd Bergmann wrote: > Compile-testing the driver fails unless OF_GPIO is enabled: > > drivers/gpio/gpio-mt7621.c: In function 'mediatek_gpio_bank_probe': > drivers/gpio/gpio-mt7621.c:228:10: error: 'struct gpio_chip' has no member > named 'of_node' > > Fixes:

Re: [PATCH 1/2] gpio: mt7621: add OF_GPIO dependency

2018-07-13 Thread Linus Walleij
On Tue, Jul 10, 2018 at 5:19 PM Arnd Bergmann wrote: > Compile-testing the driver fails unless OF_GPIO is enabled: > > drivers/gpio/gpio-mt7621.c: In function 'mediatek_gpio_bank_probe': > drivers/gpio/gpio-mt7621.c:228:10: error: 'struct gpio_chip' has no member > named 'of_node' > > Fixes:

Re: [PATCH v13 13/18] x86/tsc: calibrate tsc only once

2018-07-13 Thread Dou Liyang
At 07/12/2018 08:04 AM, Pavel Tatashin wrote: During boot tsc is calibrated twice: once in tsc_early_delay_calibrate(), and the second time in tsc_init(). Rename tsc_early_delay_calibrate() to tsc_early_init(), and rework it so the calibration is done only early, and make tsc_init() to use

Re: [PATCH v13 13/18] x86/tsc: calibrate tsc only once

2018-07-13 Thread Dou Liyang
At 07/12/2018 08:04 AM, Pavel Tatashin wrote: During boot tsc is calibrated twice: once in tsc_early_delay_calibrate(), and the second time in tsc_init(). Rename tsc_early_delay_calibrate() to tsc_early_init(), and rework it so the calibration is done only early, and make tsc_init() to use

Re: [PATCH 08/32] genirq: Synchronize only with single thread on free_irq()

2018-07-13 Thread Lukas Wunner
On Thu, Jul 12, 2018 at 05:21:09PM -0500, Bjorn Helgaas wrote: > On Sat, Jun 16, 2018 at 09:25:00PM +0200, Lukas Wunner wrote: > > When pciehp is converted to threaded IRQ handling, removal of unplugged > > devices below a PCIe hotplug port happens synchronously in the IRQ > > thread. Removal of

Re: [PATCH 08/32] genirq: Synchronize only with single thread on free_irq()

2018-07-13 Thread Lukas Wunner
On Thu, Jul 12, 2018 at 05:21:09PM -0500, Bjorn Helgaas wrote: > On Sat, Jun 16, 2018 at 09:25:00PM +0200, Lukas Wunner wrote: > > When pciehp is converted to threaded IRQ handling, removal of unplugged > > devices below a PCIe hotplug port happens synchronously in the IRQ > > thread. Removal of

[PATCH] ACPI: button: hide unused procfs helpers

2018-07-13 Thread YueHaibing
When CONFIG_PROC_FS isn't set, gcc warning this: drivers/acpi/button.c:255:12: warning: ‘acpi_button_state_seq_show’ defined but not used [-Wunused-function] static int acpi_button_state_seq_show(struct seq_file *seq, void *offset) ^ fix this by adding #ifdef around it.

[PATCH] ACPI: button: hide unused procfs helpers

2018-07-13 Thread YueHaibing
When CONFIG_PROC_FS isn't set, gcc warning this: drivers/acpi/button.c:255:12: warning: ‘acpi_button_state_seq_show’ defined but not used [-Wunused-function] static int acpi_button_state_seq_show(struct seq_file *seq, void *offset) ^ fix this by adding #ifdef around it.

Re: [PATCH 10/18] brcmsmac: change strncpy+truncation to strlcpy

2018-07-13 Thread Arend van Spriel
On 7/13/2018 3:25 AM, Dominique Martinet wrote: Generated by scripts/coccinelle/misc/strncpy_truncation.cocci Acked-by: Arend van Spriel Signed-off-by: Dominique Martinet --- Please see https://marc.info/?l=linux-kernel=153144450722324=2 (the first patch of the serie) for the motivation

Re: [PATCH 10/18] brcmsmac: change strncpy+truncation to strlcpy

2018-07-13 Thread Arend van Spriel
On 7/13/2018 3:25 AM, Dominique Martinet wrote: Generated by scripts/coccinelle/misc/strncpy_truncation.cocci Acked-by: Arend van Spriel Signed-off-by: Dominique Martinet --- Please see https://marc.info/?l=linux-kernel=153144450722324=2 (the first patch of the serie) for the motivation

Re: [PATCH] gpio: mxc: add power management support

2018-07-13 Thread Bartosz Golaszewski
2018-07-13 9:11 GMT+02:00 Linus Walleij : > On Tue, Jul 10, 2018 at 9:53 AM Anson Huang wrote: > >> GPIO registers could lose context on some i.MX SoCs, >> like on i.MX7D, when enter LPSR mode, the whole SoC >> will be powered off except LPSR domain, GPIO banks >> will lose context in this case,

Re: [PATCH] gpio: mxc: add power management support

2018-07-13 Thread Bartosz Golaszewski
2018-07-13 9:11 GMT+02:00 Linus Walleij : > On Tue, Jul 10, 2018 at 9:53 AM Anson Huang wrote: > >> GPIO registers could lose context on some i.MX SoCs, >> like on i.MX7D, when enter LPSR mode, the whole SoC >> will be powered off except LPSR domain, GPIO banks >> will lose context in this case,

Re: [PATCH 2/4] phy: socionext: add USB3 PHY driver for UniPhier SoC

2018-07-13 Thread Kishon Vijay Abraham I
Hi, On Wednesday 11 July 2018 05:35 PM, Kunihiko Hayashi wrote: > On Mon, 9 Jul 2018 20:23:19 +0900 wrote: > >> Hi Kishon, >> Thank you for your comments. >> >> On Mon, 9 Jul 2018 10:49:50 +0530 wrote: >> >>> Hi, >>> >>> On Friday 29 June 2018 02:08 PM, Kunihiko Hayashi wrote: Add a

Re: [PATCH 2/4] phy: socionext: add USB3 PHY driver for UniPhier SoC

2018-07-13 Thread Kishon Vijay Abraham I
Hi, On Wednesday 11 July 2018 05:35 PM, Kunihiko Hayashi wrote: > On Mon, 9 Jul 2018 20:23:19 +0900 wrote: > >> Hi Kishon, >> Thank you for your comments. >> >> On Mon, 9 Jul 2018 10:49:50 +0530 wrote: >> >>> Hi, >>> >>> On Friday 29 June 2018 02:08 PM, Kunihiko Hayashi wrote: Add a

Re: [PATCH v3] time: Fix incorrect sleeptime injection when suspend fails

2018-07-13 Thread Mukesh Ojha
Hi John, Thanks for your response Please find my comments inline. On 7/11/2018 1:43 AM, John Stultz wrote: On Fri, Jul 6, 2018 at 6:17 AM, Mukesh Ojha wrote: Currently, there exists a corner case assuming when there is only one clocksource e.g RTC, and system failed to go to suspend mode.

Re: [PATCH v3] time: Fix incorrect sleeptime injection when suspend fails

2018-07-13 Thread Mukesh Ojha
Hi John, Thanks for your response Please find my comments inline. On 7/11/2018 1:43 AM, John Stultz wrote: On Fri, Jul 6, 2018 at 6:17 AM, Mukesh Ojha wrote: Currently, there exists a corner case assuming when there is only one clocksource e.g RTC, and system failed to go to suspend mode.

Re: [PATCH] gpio: mxc: add power management support

2018-07-13 Thread Linus Walleij
On Tue, Jul 10, 2018 at 9:53 AM Anson Huang wrote: > GPIO registers could lose context on some i.MX SoCs, > like on i.MX7D, when enter LPSR mode, the whole SoC > will be powered off except LPSR domain, GPIO banks > will lose context in this case, need to restore > the context after resume from

Re: [PATCH] gpio: mxc: add power management support

2018-07-13 Thread Linus Walleij
On Tue, Jul 10, 2018 at 9:53 AM Anson Huang wrote: > GPIO registers could lose context on some i.MX SoCs, > like on i.MX7D, when enter LPSR mode, the whole SoC > will be powered off except LPSR domain, GPIO banks > will lose context in this case, need to restore > the context after resume from

Re: [PATCH] gpio: aspeed: fix compile testing warning

2018-07-13 Thread Linus Walleij
On Tue, Jul 10, 2018 at 1:53 AM Benjamin Herrenschmidt wrote: > On Mon, 2018-07-09 at 16:56 +0200, Arnd Bergmann wrote: > > Gcc cannot always see that BUG_ON(1) is guaranteed to not > > return, so we get a warning message in some configurations: > > > > drivers/gpio/gpio-aspeed.c: In function

Re: [PATCH] gpio: aspeed: fix compile testing warning

2018-07-13 Thread Linus Walleij
On Tue, Jul 10, 2018 at 1:53 AM Benjamin Herrenschmidt wrote: > On Mon, 2018-07-09 at 16:56 +0200, Arnd Bergmann wrote: > > Gcc cannot always see that BUG_ON(1) is guaranteed to not > > return, so we get a warning message in some configurations: > > > > drivers/gpio/gpio-aspeed.c: In function

Re: [PATCH] gpio: aspeed: fix compile testing warning

2018-07-13 Thread Linus Walleij
On Mon, Jul 9, 2018 at 4:56 PM Arnd Bergmann wrote: > Gcc cannot always see that BUG_ON(1) is guaranteed to not > return, so we get a warning message in some configurations: > > drivers/gpio/gpio-aspeed.c: In function 'bank_reg': > drivers/gpio/gpio-aspeed.c:244:1: error: control reaches end of

Re: [PATCH] gpio: aspeed: fix compile testing warning

2018-07-13 Thread Linus Walleij
On Mon, Jul 9, 2018 at 4:56 PM Arnd Bergmann wrote: > Gcc cannot always see that BUG_ON(1) is guaranteed to not > return, so we get a warning message in some configurations: > > drivers/gpio/gpio-aspeed.c: In function 'bank_reg': > drivers/gpio/gpio-aspeed.c:244:1: error: control reaches end of

<    4   5   6   7   8   9   10   >