Re: WARNING in kvm_arch_vcpu_ioctl_run (2)

2017-12-03 Thread Dmitry Vyukov
On Mon, Nov 20, 2017 at 10:19 AM, Wanpeng Li wrote: > Hi Dmitry, > 2017-11-20 12:09 GMT+08:00 Wanpeng Li : >> 2017-11-18 4:22 GMT+08:00 syzbot >> : >>> Hello, >>> >>> syzbot has

Re: WARNING in kvm_arch_vcpu_ioctl_run (2)

2017-12-03 Thread Dmitry Vyukov
On Mon, Nov 20, 2017 at 10:19 AM, Wanpeng Li wrote: > Hi Dmitry, > 2017-11-20 12:09 GMT+08:00 Wanpeng Li : >> 2017-11-18 4:22 GMT+08:00 syzbot >> : >>> Hello, >>> >>> syzbot has tested the proposed patch but the reproducer still triggered >>> crash: >>> WARNING in kvm_arch_vcpu_ioctl_run >>> >>>

Re: [alsa-devel] [PATCH v4 03/15] soundwire: Add Master registration

2017-12-03 Thread Vinod Koul
On Fri, Dec 01, 2017 at 04:10:55PM -0600, Pierre-Louis Bossart wrote: > On 12/1/17 3:56 AM, Vinod Koul wrote: > >A Master registers with SoundWire bus and scans the firmware provided > > nitpick: is the 'register' correct? You create a bus instance for each > hardware master interface. Or is my

Re: [alsa-devel] [PATCH v4 03/15] soundwire: Add Master registration

2017-12-03 Thread Vinod Koul
On Fri, Dec 01, 2017 at 04:10:55PM -0600, Pierre-Louis Bossart wrote: > On 12/1/17 3:56 AM, Vinod Koul wrote: > >A Master registers with SoundWire bus and scans the firmware provided > > nitpick: is the 'register' correct? You create a bus instance for each > hardware master interface. Or is my

Re: [PATCH v6 0/2] memory: Introduce ti-emif-sram driver

2017-12-03 Thread Tony Lindgren
* santosh.shilim...@oracle.com [171203 04:03]: > > > On 11/30/17 2:56 PM, Dave Gerlach wrote: > > This is a resend of v5 of this series found here [1]. It introduces > > relocatable PM handlers for the emif that are copied to sram and > > run from there during low

Re: [PATCH v6 0/2] memory: Introduce ti-emif-sram driver

2017-12-03 Thread Tony Lindgren
* santosh.shilim...@oracle.com [171203 04:03]: > > > On 11/30/17 2:56 PM, Dave Gerlach wrote: > > This is a resend of v5 of this series found here [1]. It introduces > > relocatable PM handlers for the emif that are copied to sram and > > run from there during low power mode entry. > > > > The

Re: [PATCH v5 net-next,mips 6/7] netdev: octeon-ethernet: Add Cavium Octeon III support.

2017-12-03 Thread David Miller
From: David Daney Date: Fri, 1 Dec 2017 15:18:06 -0800 > +static char *mix_port; > +module_param(mix_port, charp, 0444); > +MODULE_PARM_DESC(mix_port, "Specifies which ports connect to MIX > interfaces."); > + > +static char *pki_port; > +module_param(pki_port, charp,

Re: [PATCH v5 net-next,mips 6/7] netdev: octeon-ethernet: Add Cavium Octeon III support.

2017-12-03 Thread David Miller
From: David Daney Date: Fri, 1 Dec 2017 15:18:06 -0800 > +static char *mix_port; > +module_param(mix_port, charp, 0444); > +MODULE_PARM_DESC(mix_port, "Specifies which ports connect to MIX > interfaces."); > + > +static char *pki_port; > +module_param(pki_port, charp, 0444); >

Re: [GIT PULL] revert ARM SCPI changes for v4.15-rc1

2017-12-03 Thread Heiner Kallweit
Am 03.12.2017 um 02:04 schrieb Olof Johansson: > On Fri, Dec 01, 2017 at 11:53:05AM -0800, Kevin Hilman wrote: >> Arnd, Olof, >> >> These ARM SCPI changes caused SCPI regressions resulting in CPUfreq >> failures on most Amlogic SoCs (found by kernelci.org.) >> >> Unfortunately, this was not caught

Re: [GIT PULL] revert ARM SCPI changes for v4.15-rc1

2017-12-03 Thread Heiner Kallweit
Am 03.12.2017 um 02:04 schrieb Olof Johansson: > On Fri, Dec 01, 2017 at 11:53:05AM -0800, Kevin Hilman wrote: >> Arnd, Olof, >> >> These ARM SCPI changes caused SCPI regressions resulting in CPUfreq >> failures on most Amlogic SoCs (found by kernelci.org.) >> >> Unfortunately, this was not caught

Linux 4.15-rc2

2017-12-03 Thread Linus Torvalds
It's Sunday, but a few hours earlier than usual, since I'm on the east coast, three hours ahead of my normal release schedule. It's a slightly bigger rc2 than I would have wished for, but this early in the release process I don't worry about it. The appended shortlog gives the details, it's fixes

Linux 4.15-rc2

2017-12-03 Thread Linus Torvalds
It's Sunday, but a few hours earlier than usual, since I'm on the east coast, three hours ahead of my normal release schedule. It's a slightly bigger rc2 than I would have wished for, but this early in the release process I don't worry about it. The appended shortlog gives the details, it's fixes

Re: [PATCH RFC] stat.2: Document that stat can fail with EINTR

2017-12-03 Thread walter harms
Am 03.12.2017 01:23, schrieb Keno Fischer: > Particularly on network file systems, a stat call may require > submitting a message over the network and waiting interruptably > for a reply. > > Signed-off-by: Keno Fischer > --- > > The catalyst for this patch was me

Re: [PATCH RFC] stat.2: Document that stat can fail with EINTR

2017-12-03 Thread walter harms
Am 03.12.2017 01:23, schrieb Keno Fischer: > Particularly on network file systems, a stat call may require > submitting a message over the network and waiting interruptably > for a reply. > > Signed-off-by: Keno Fischer > --- > > The catalyst for this patch was me experiencing EINTR errors

[PATCH] [FIX] staging: pi433: fix last commit indentation

2017-12-03 Thread Victor Carvalho
fix indentation in commit: - 928150e1dbb8638f309493b7dc72d56c20896704 staging: pi433: fix coding style in function rf69_set_rx_cfg() Signed-off-by: Victor Carvalho --- drivers/staging/pi433/pi433_if.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)

[PATCH] [FIX] staging: pi433: fix last commit indentation

2017-12-03 Thread Victor Carvalho
fix indentation in commit: - 928150e1dbb8638f309493b7dc72d56c20896704 staging: pi433: fix coding style in function rf69_set_rx_cfg() Signed-off-by: Victor Carvalho --- drivers/staging/pi433/pi433_if.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

Hello

2017-12-03 Thread MISS BORTE OGORTAI
-- GREETINGS, I AM BORTE ,I WAS DIAGNOSE OF OVARIAN CANCER,WHICH DOCTORS HAVE CONFIRMED THAT I HAVE ONLY FEW WEEKS TO LIVE, SO I HAVE DECIDED TO DONATE EVERYTHING I HAVE TO THE ORPHANAGE AND THE POOR WIDOWS THROUGH YOU .PLEASE KINDLY REPLY ME(missbortiogo...@gmail.com) AS SOON AS POSIBLE

Hello

2017-12-03 Thread MISS BORTE OGORTAI
-- GREETINGS, I AM BORTE ,I WAS DIAGNOSE OF OVARIAN CANCER,WHICH DOCTORS HAVE CONFIRMED THAT I HAVE ONLY FEW WEEKS TO LIVE, SO I HAVE DECIDED TO DONATE EVERYTHING I HAVE TO THE ORPHANAGE AND THE POOR WIDOWS THROUGH YOU .PLEASE KINDLY REPLY ME(missbortiogo...@gmail.com) AS SOON AS POSIBLE

What Has Kept You Waiting?

2017-12-03 Thread Mr Light Adam
Attention We are surprised as you have not responded to our previous notification concerning your $650,000, United Nations has decided to compensate you as part of the empowerment program to reinstate those who have lost money to scammers in the past. To commence facilitating your transfer,

What Has Kept You Waiting?

2017-12-03 Thread Mr Light Adam
Attention We are surprised as you have not responded to our previous notification concerning your $650,000, United Nations has decided to compensate you as part of the empowerment program to reinstate those who have lost money to scammers in the past. To commence facilitating your transfer,

[PATCH 2/6] staging: pi433: Capitalize constant definitions

2017-12-03 Thread Simon Sandström
Fixes checkpatch.pl warnings "Avoid CamelCase ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 32 - drivers/staging/pi433/rf69_registers.h | 44 +- 2 files changed, 38 insertions(+), 38

[PATCH 2/6] staging: pi433: Capitalize constant definitions

2017-12-03 Thread Simon Sandström
Fixes checkpatch.pl warnings "Avoid CamelCase ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 32 - drivers/staging/pi433/rf69_registers.h | 44 +- 2 files changed, 38 insertions(+), 38 deletions(-) diff

[PATCH 4/6] staging: pi433: Rename enum optionOnOff in rf69_enum.h

2017-12-03 Thread Simon Sandström
Renames the enum optionOnOff and its values optionOn, optionOff to enum option_on_off and OPTION_ON, OPTION_OFF. Fixes checkpatch.pl warnings: "Avoid CamelCase: , , ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 34 ++---

[PATCH 3/6] staging: pi433: Rename variable in struct pi433_rx_cfg

2017-12-03 Thread Simon Sandström
Renames variable thresholdDecrement in struct pi433_rx_cfg to threshold_decrement to get rid of checkpatch.pl warning "Avoid CamelCase: ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 2 +- drivers/staging/pi433/pi433_if.h | 2 +- 2 files changed, 2

[PATCH 4/6] staging: pi433: Rename enum optionOnOff in rf69_enum.h

2017-12-03 Thread Simon Sandström
Renames the enum optionOnOff and its values optionOn, optionOff to enum option_on_off and OPTION_ON, OPTION_OFF. Fixes checkpatch.pl warnings: "Avoid CamelCase: , , ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 34 ++---

[PATCH 3/6] staging: pi433: Rename variable in struct pi433_rx_cfg

2017-12-03 Thread Simon Sandström
Renames variable thresholdDecrement in struct pi433_rx_cfg to threshold_decrement to get rid of checkpatch.pl warning "Avoid CamelCase: ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 2 +- drivers/staging/pi433/pi433_if.h | 2 +- 2 files changed, 2 insertions(+), 2

[PATCH 6/6] staging: pi433: Rename enum modShaping in rf69_enum.h

2017-12-03 Thread Simon Sandström
Renames enum modShaping and its values to get rid of checkpatch.pl warnings: "Avoid CamelCase: ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 2 +- drivers/staging/pi433/pi433_if.h | 2 +- drivers/staging/pi433/rf69.c | 21

[PATCH 6/6] staging: pi433: Rename enum modShaping in rf69_enum.h

2017-12-03 Thread Simon Sandström
Renames enum modShaping and its values to get rid of checkpatch.pl warnings: "Avoid CamelCase: ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 2 +- drivers/staging/pi433/pi433_if.h | 2 +- drivers/staging/pi433/rf69.c | 21 +++--

Re: Difficulties for compilation without extra optimisation

2017-12-03 Thread Trond Myklebust
On Sun, 2017-12-03 at 15:15 +0100, SF Markus Elfring wrote: > Hello, > > I came along some software modules where I suggested source code > adjustments. > > Example: > nfs/write: Use common error handling code in > nfs_lock_and_join_requests() > > https://lkml.org/lkml/2017/11/7/599 >

Re: Difficulties for compilation without extra optimisation

2017-12-03 Thread Trond Myklebust
On Sun, 2017-12-03 at 15:15 +0100, SF Markus Elfring wrote: > Hello, > > I came along some software modules where I suggested source code > adjustments. > > Example: > nfs/write: Use common error handling code in > nfs_lock_and_join_requests() > > https://lkml.org/lkml/2017/11/7/599 >

[PATCH 1/6] staging: pi433: Fix indentation in rf69_enum.h

2017-12-03 Thread Simon Sandström
Basically just 's//\t/', to fix checkpatch.pl warnings: "please, no spaces at the start of a line". Signed-off-by: Simon Sandström --- drivers/staging/pi433/rf69_enum.h | 207 +++--- 1 file changed, 103 insertions(+), 104 deletions(-) diff

[PATCH 5/6] staging: pi433: Rename enum dataMode in rf69_enum.h

2017-12-03 Thread Simon Sandström
Renames enum dataMode and its values packet, continuous, continuousNoSync to enum data_mode and PACKET, CONTINUOUS, CONTINUOUS_NO_SYNC. Fixes checkpatch.pl warnings: "Avoid CamelCase: , ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 2 +-

[PATCH 1/6] staging: pi433: Fix indentation in rf69_enum.h

2017-12-03 Thread Simon Sandström
Basically just 's//\t/', to fix checkpatch.pl warnings: "please, no spaces at the start of a line". Signed-off-by: Simon Sandström --- drivers/staging/pi433/rf69_enum.h | 207 +++--- 1 file changed, 103 insertions(+), 104 deletions(-) diff --git

[PATCH 5/6] staging: pi433: Rename enum dataMode in rf69_enum.h

2017-12-03 Thread Simon Sandström
Renames enum dataMode and its values packet, continuous, continuousNoSync to enum data_mode and PACKET, CONTINUOUS, CONTINUOUS_NO_SYNC. Fixes checkpatch.pl warnings: "Avoid CamelCase: , ". Signed-off-by: Simon Sandström --- drivers/staging/pi433/pi433_if.c | 2 +- drivers/staging/pi433/rf69.c

[PATCH 0/6] Fix indentation and CamelCase issues in staging/pi433

2017-12-03 Thread Simon Sandström
These patches fixes a bunch of code style issues in staging/pi433. The first patch fixes indentation in rf69_enum.h and the rest of the patches fixes CamelCase issues in all of staging/pi433. In total the patches get rids of around 140 warnings generated by checkpatch.pl. - Simon --- Simon

[PATCH 0/6] Fix indentation and CamelCase issues in staging/pi433

2017-12-03 Thread Simon Sandström
These patches fixes a bunch of code style issues in staging/pi433. The first patch fixes indentation in rf69_enum.h and the rest of the patches fixes CamelCase issues in all of staging/pi433. In total the patches get rids of around 140 warnings generated by checkpatch.pl. - Simon --- Simon

Re: [PATCH v3] doc: add maintainer book

2017-12-03 Thread Jonathan Corbet
On Thu, 30 Nov 2017 12:55:07 +1100 "Tobin C. Harding" wrote: > Create a new kernel documentation book 'how to be a maintainer' > (suggested by Jonathan Corbet). Add chapters on 'configuring git' and > 'creating a pull request'. So I've finally gotten around to giving this a look.

Re: [PATCH v3] doc: add maintainer book

2017-12-03 Thread Jonathan Corbet
On Thu, 30 Nov 2017 12:55:07 +1100 "Tobin C. Harding" wrote: > Create a new kernel documentation book 'how to be a maintainer' > (suggested by Jonathan Corbet). Add chapters on 'configuring git' and > 'creating a pull request'. So I've finally gotten around to giving this a look. It seems like

[PULL REQUEST] i2c for 4.15

2017-12-03 Thread Wolfram Sang
Linus, here are two bugfixes for I2C, fixing a memleak in the core and irq allocation for i801. Also, three bugfixes for the at24 eeprom driver which Bartosz collected while taking over maintainership for this driver. Good start, Bartosz! Please pull. Thanks, Wolfram The following changes

[PULL REQUEST] i2c for 4.15

2017-12-03 Thread Wolfram Sang
Linus, here are two bugfixes for I2C, fixing a memleak in the core and irq allocation for i801. Also, three bugfixes for the at24 eeprom driver which Bartosz collected while taking over maintainership for this driver. Good start, Bartosz! Please pull. Thanks, Wolfram The following changes

Re: [PATCH net] rxrpc: Use correct netns source in rxrpc_release_sock()

2017-12-03 Thread David Miller
From: David Howells Date: Fri, 01 Dec 2017 11:09:53 + > In rxrpc_release_sock() there may be no rx->local value to access, so we > can't unconditionally follow it to the rxrpc network namespace information > to poke the connection reapers. > > Instead, use the socket's

Re: [PATCH net] rxrpc: Fix the MAINTAINERS record

2017-12-03 Thread David Miller
From: David Howells Date: Fri, 01 Dec 2017 11:10:37 + > Fix the MAINTAINERS record so that it's more obvious who the maintainer for > AF_RXRPC is. > > Reported-by: Joe Perches > Reported-by: David Miller > Signed-off-by: David

Re: [PATCH net] rxrpc: Use correct netns source in rxrpc_release_sock()

2017-12-03 Thread David Miller
From: David Howells Date: Fri, 01 Dec 2017 11:09:53 + > In rxrpc_release_sock() there may be no rx->local value to access, so we > can't unconditionally follow it to the rxrpc network namespace information > to poke the connection reapers. > > Instead, use the socket's namespace pointer to

Re: [PATCH net] rxrpc: Fix the MAINTAINERS record

2017-12-03 Thread David Miller
From: David Howells Date: Fri, 01 Dec 2017 11:10:37 + > Fix the MAINTAINERS record so that it's more obvious who the maintainer for > AF_RXRPC is. > > Reported-by: Joe Perches > Reported-by: David Miller > Signed-off-by: David Howells Applied.

Re: [PATCH] liquidio: fix incorrect indentation of assignment statement

2017-12-03 Thread David Miller
From: Colin King Date: Fri, 1 Dec 2017 12:38:11 + > From: Colin Ian King > > Remove one extraneous level of indentation on assignment statement. > > Signed-off-by: Colin Ian King Applied.

Re: [PATCH] liquidio: fix incorrect indentation of assignment statement

2017-12-03 Thread David Miller
From: Colin King Date: Fri, 1 Dec 2017 12:38:11 + > From: Colin Ian King > > Remove one extraneous level of indentation on assignment statement. > > Signed-off-by: Colin Ian King Applied.

Re: BUG: workqueue lockup (2)

2017-12-03 Thread Thomas Gleixner
On Sun, 3 Dec 2017, Dmitry Vyukov wrote: > On Sun, Dec 3, 2017 at 3:31 PM, syzbot > > wrote: > > Hello, > > > > syzkaller hit the following crash on > > 2db767d9889cef087149a5eaa35c1497671fa40f > >

Re: BUG: workqueue lockup (2)

2017-12-03 Thread Thomas Gleixner
On Sun, 3 Dec 2017, Dmitry Vyukov wrote: > On Sun, Dec 3, 2017 at 3:31 PM, syzbot > > wrote: > > Hello, > > > > syzkaller hit the following crash on > > 2db767d9889cef087149a5eaa35c1497671fa40f > > git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/master > > compiler: gcc (GCC)

Re: BUG: workqueue lockup (2)

2017-12-03 Thread Dmitry Vyukov
On Sun, Dec 3, 2017 at 3:31 PM, syzbot wrote: > Hello, > > syzkaller hit the following crash on > 2db767d9889cef087149a5eaa35c1497671fa40f > git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/master > compiler:

Re: BUG: workqueue lockup (2)

2017-12-03 Thread Dmitry Vyukov
On Sun, Dec 3, 2017 at 3:31 PM, syzbot wrote: > Hello, > > syzkaller hit the following crash on > 2db767d9889cef087149a5eaa35c1497671fa40f > git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/master > compiler: gcc (GCC) 7.1.1 20170620 > .config is attached > Raw console output is

Re: [PATCH v6 4/6] dt: bindings: lp8860: Update the bindings to the standard

2017-12-03 Thread Jacek Anaszewski
On 12/03/2017 02:49 PM, Jacek Anaszewski wrote: > Hi Dan, > > Thanks for the patch. > > On 12/01/2017 05:56 PM, Dan Murphy wrote: >> Update the lp8860 dt binding to the LED standard where >> the LED should have a child node and also adding a >> LED trigger entry. >> >> Signed-off-by: Dan Murphy

Re: [PATCH v6 4/6] dt: bindings: lp8860: Update the bindings to the standard

2017-12-03 Thread Jacek Anaszewski
On 12/03/2017 02:49 PM, Jacek Anaszewski wrote: > Hi Dan, > > Thanks for the patch. > > On 12/01/2017 05:56 PM, Dan Murphy wrote: >> Update the lp8860 dt binding to the LED standard where >> the LED should have a child node and also adding a >> LED trigger entry. >> >> Signed-off-by: Dan Murphy

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

2017-12-03 Thread kbuild test robot
drivers/input/touchscreen/cyttsp5.c:1098:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Fixes: 68d0bd1e4815 ("Input: Add driver for Cypress Generation 5

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

2017-12-03 Thread kbuild test robot
drivers/input/touchscreen/cyttsp5.c:1098:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Fixes: 68d0bd1e4815 ("Input: Add driver for Cypress Generation 5

Re: [PATCH v4 1/2] Input: Add driver for Cypress Generation 5 touchscreen

2017-12-03 Thread kbuild test robot
/linux/commits/Myl-ne-Josserand/Input-Add-driver-for-Cypress-Generation-5-touchscreen/20171203-203557 base: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next coccinelle warnings: (new ones prefixed by >>) >> drivers/input/touchscreen/cyttsp5.c:1098:3-8: No need t

Re: [PATCH v4 1/2] Input: Add driver for Cypress Generation 5 touchscreen

2017-12-03 Thread kbuild test robot
/linux/commits/Myl-ne-Josserand/Input-Add-driver-for-Cypress-Generation-5-touchscreen/20171203-203557 base: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next coccinelle warnings: (new ones prefixed by >>) >> drivers/input/touchscreen/cyttsp5.c:1098:3-8: No need t

Difficulties for compilation without extra optimisation

2017-12-03 Thread SF Markus Elfring
Hello, I came along some software modules where I suggested source code adjustments. Example: nfs/write: Use common error handling code in nfs_lock_and_join_requests() https://lkml.org/lkml/2017/11/7/599 https://patchwork.kernel.org/patch/10047013/

Difficulties for compilation without extra optimisation

2017-12-03 Thread SF Markus Elfring
Hello, I came along some software modules where I suggested source code adjustments. Example: nfs/write: Use common error handling code in nfs_lock_and_join_requests() https://lkml.org/lkml/2017/11/7/599 https://patchwork.kernel.org/patch/10047013/

Re: [PATCH v5.1 1/2] ARM64: dts: meson-gx: use stable UART bindings with correct gate clock

2017-12-03 Thread Andreas Färber
Hi, Am 21.06.2017 um 16:42 schrieb Neil Armstrong: > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi > b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi > index 17d3efd..ea53cc2 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi >

Re: [PATCH v5.1 1/2] ARM64: dts: meson-gx: use stable UART bindings with correct gate clock

2017-12-03 Thread Andreas Färber
Hi, Am 21.06.2017 um 16:42 schrieb Neil Armstrong: > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi > b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi > index 17d3efd..ea53cc2 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi >

[PATCH 03/10] staging: ccree: remove more unnecessary parentheses

2017-12-03 Thread Gilad Ben-Yossef
The removal of likely/unlikely unearthed some more unnecessary parentheses. Remove them for better readability. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_buffer_mgr.c | 4 ++-- drivers/staging/ccree/ssi_driver.c | 6 +++---

[PATCH 03/10] staging: ccree: remove more unnecessary parentheses

2017-12-03 Thread Gilad Ben-Yossef
The removal of likely/unlikely unearthed some more unnecessary parentheses. Remove them for better readability. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_buffer_mgr.c | 4 ++-- drivers/staging/ccree/ssi_driver.c | 6 +++--- drivers/staging/ccree/ssi_request_mgr.c | 2

Re: [PATCH v6 5/6] leds: lp8860: Update the LED label generation

2017-12-03 Thread Jacek Anaszewski
Dan, On 12/01/2017 05:59 PM, Dan Murphy wrote: > On 12/01/2017 10:56 AM, Dan Murphy wrote: >> Fix the LED label generation for the LP8860 to >> conform with the >> >> Documentation/devicetree/bindings/leds/common.txt >> >> document indicating the LED label should be part of a >> child node to the

Re: [PATCH v6 5/6] leds: lp8860: Update the LED label generation

2017-12-03 Thread Jacek Anaszewski
Dan, On 12/01/2017 05:59 PM, Dan Murphy wrote: > On 12/01/2017 10:56 AM, Dan Murphy wrote: >> Fix the LED label generation for the LP8860 to >> conform with the >> >> Documentation/devicetree/bindings/leds/common.txt >> >> document indicating the LED label should be part of a >> child node to the

[PATCH 06/10] staging: ccree: remove interim DT docs

2017-12-03 Thread Gilad Ben-Yossef
As proper DT bindings doc where submitted and ACKed, remove the interim one. Signed-off-by: Gilad Ben-Yossef --- .../devicetree/bindings/crypto/arm-cryptocell.txt | 27 -- 1 file changed, 27 deletions(-) delete mode 100644

[PATCH 06/10] staging: ccree: remove interim DT docs

2017-12-03 Thread Gilad Ben-Yossef
As proper DT bindings doc where submitted and ACKed, remove the interim one. Signed-off-by: Gilad Ben-Yossef --- .../devicetree/bindings/crypto/arm-cryptocell.txt | 27 -- 1 file changed, 27 deletions(-) delete mode 100644

[PATCH 09/10] staging: ccree: fix AEAD func naming convention

2017-12-03 Thread Gilad Ben-Yossef
The aead files was using a func naming convention which was inconsistent (ssi vs. cc), included a useless prefix (ssi_aead) and often used too long function names producing monster func names such as ssi_aead_gcm_setup_ghash_desc() that made the call site code hard to read. Make the code more

[PATCH 10/10] staging: ccree: amend aead func def for readability

2017-12-03 Thread Gilad Ben-Yossef
Func definitions in the aead implementation were did not adhere to coding style. Fix them for better readability. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c | 141 ++- 1 file changed, 51 insertions(+), 90

[PATCH 10/10] staging: ccree: amend aead func def for readability

2017-12-03 Thread Gilad Ben-Yossef
Func definitions in the aead implementation were did not adhere to coding style. Fix them for better readability. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c | 141 ++- 1 file changed, 51 insertions(+), 90 deletions(-) diff --git

[PATCH 09/10] staging: ccree: fix AEAD func naming convention

2017-12-03 Thread Gilad Ben-Yossef
The aead files was using a func naming convention which was inconsistent (ssi vs. cc), included a useless prefix (ssi_aead) and often used too long function names producing monster func names such as ssi_aead_gcm_setup_ghash_desc() that made the call site code hard to read. Make the code more

[PATCH 08/10] staging: ccree: NULLify backup_info when unused

2017-12-03 Thread Gilad Ben-Yossef
backup_info field is only allocated for decrypt code path. The field was not nullified when not used causing a kfree in an error handling path to attempt to free random addresses as uncovered in stress testing. Fixes: 737aed947f9b ("staging: ccree: save ciphertext for CTS IV") Cc:

[PATCH 08/10] staging: ccree: NULLify backup_info when unused

2017-12-03 Thread Gilad Ben-Yossef
backup_info field is only allocated for decrypt code path. The field was not nullified when not used causing a kfree in an error handling path to attempt to free random addresses as uncovered in stress testing. Fixes: 737aed947f9b ("staging: ccree: save ciphertext for CTS IV") Cc:

[PATCH 07/10] staging: ccree: update TODO list

2017-12-03 Thread Gilad Ben-Yossef
Update the ccree staging TODO list in light of recent work. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/TODO | 23 ++- 1 file changed, 2 insertions(+), 21 deletions(-) diff --git a/drivers/staging/ccree/TODO b/drivers/staging/ccree/TODO

[PATCH 04/10] staging: ccree: fix indentation of var assignment

2017-12-03 Thread Gilad Ben-Yossef
Fix indentation of var assignment split across lines for better readability. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ccree/ssi_aead.c

[PATCH 01/10] staging: ccree: remove inline qualifiers

2017-12-03 Thread Gilad Ben-Yossef
The ccree drivers was marking a lot of big functions in C file as static inline for no good reason. Remove the inline qualifier from any but the few truly single line functions. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c| 40

[PATCH 02/10] staging: ccree: remove unproven likely/unlikely

2017-12-03 Thread Gilad Ben-Yossef
The ccree code made a lot of use of likely/unlikely qualifiers without proven measurements showing any benefits. Remove them all until we see what is justified and what is not. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c| 57 ++--

[PATCH 07/10] staging: ccree: update TODO list

2017-12-03 Thread Gilad Ben-Yossef
Update the ccree staging TODO list in light of recent work. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/TODO | 23 ++- 1 file changed, 2 insertions(+), 21 deletions(-) diff --git a/drivers/staging/ccree/TODO b/drivers/staging/ccree/TODO index c9f5754..f44edcd

[PATCH 04/10] staging: ccree: fix indentation of var assignment

2017-12-03 Thread Gilad Ben-Yossef
Fix indentation of var assignment split across lines for better readability. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ccree/ssi_aead.c b/drivers/staging/ccree/ssi_aead.c index

[PATCH 01/10] staging: ccree: remove inline qualifiers

2017-12-03 Thread Gilad Ben-Yossef
The ccree drivers was marking a lot of big functions in C file as static inline for no good reason. Remove the inline qualifier from any but the few truly single line functions. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c| 40 -

[PATCH 02/10] staging: ccree: remove unproven likely/unlikely

2017-12-03 Thread Gilad Ben-Yossef
The ccree code made a lot of use of likely/unlikely qualifiers without proven measurements showing any benefits. Remove them all until we see what is justified and what is not. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c| 57 ++--

[PATCH 00/10] staging: ccree: cleanups & fixes

2017-12-03 Thread Gilad Ben-Yossef
Another batch of ccree work. Includes the usual mix of mostly cleanups and one fix. Also includes an updated TODO to reflect recent progress and removal of interim documentation of device tree strings as the official string and documentation thereof got blessed into the crypto tree. Gilad

[PATCH 05/10] staging: ccree: remove braces for single statement blocks

2017-12-03 Thread Gilad Ben-Yossef
Remove braces from single statement if clause. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_request_mgr.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/ccree/ssi_request_mgr.c

[PATCH 00/10] staging: ccree: cleanups & fixes

2017-12-03 Thread Gilad Ben-Yossef
Another batch of ccree work. Includes the usual mix of mostly cleanups and one fix. Also includes an updated TODO to reflect recent progress and removal of interim documentation of device tree strings as the official string and documentation thereof got blessed into the crypto tree. Gilad

[PATCH 05/10] staging: ccree: remove braces for single statement blocks

2017-12-03 Thread Gilad Ben-Yossef
Remove braces from single statement if clause. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_request_mgr.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/ccree/ssi_request_mgr.c b/drivers/staging/ccree/ssi_request_mgr.c index

Re: [PATCH v6 5/6] leds: lp8860: Update the LED label generation

2017-12-03 Thread Jacek Anaszewski
Dan, On 12/01/2017 05:56 PM, Dan Murphy wrote: > Fix the LED label generation for the LP8860 to > conform with the > > Documentation/devicetree/bindings/leds/common.txt > > document indicating the LED label should be part of a > child node to the device parent. If no label is > in the child

Re: [PATCH v6 5/6] leds: lp8860: Update the LED label generation

2017-12-03 Thread Jacek Anaszewski
Dan, On 12/01/2017 05:56 PM, Dan Murphy wrote: > Fix the LED label generation for the LP8860 to > conform with the > > Documentation/devicetree/bindings/leds/common.txt > > document indicating the LED label should be part of a > child node to the device parent. If no label is > in the child

Re: [PATCH v6 4/6] dt: bindings: lp8860: Update the bindings to the standard

2017-12-03 Thread Jacek Anaszewski
Hi Dan, Thanks for the patch. On 12/01/2017 05:56 PM, Dan Murphy wrote: > Update the lp8860 dt binding to the LED standard where > the LED should have a child node and also adding a > LED trigger entry. > > Signed-off-by: Dan Murphy > --- > > v6 - New patch to fix binding

[PATCH] perf tools: set GUI mode after returning from perf_session__new()

2017-12-03 Thread Song Seok Ho
If perf_session__new() returns NULL with use_browser set to 2 via --gtk option previously, perf dies quietly without printing any errors. The reason behind this is that GTK is not yet initialized when the caller inside perf_session__new() is trying to print error message to the screen. Reorder

Re: [PATCH v6 4/6] dt: bindings: lp8860: Update the bindings to the standard

2017-12-03 Thread Jacek Anaszewski
Hi Dan, Thanks for the patch. On 12/01/2017 05:56 PM, Dan Murphy wrote: > Update the lp8860 dt binding to the LED standard where > the LED should have a child node and also adding a > LED trigger entry. > > Signed-off-by: Dan Murphy > --- > > v6 - New patch to fix binding documentation > >

[PATCH] perf tools: set GUI mode after returning from perf_session__new()

2017-12-03 Thread Song Seok Ho
If perf_session__new() returns NULL with use_browser set to 2 via --gtk option previously, perf dies quietly without printing any errors. The reason behind this is that GTK is not yet initialized when the caller inside perf_session__new() is trying to print error message to the screen. Reorder

Re: [PATCH v4 2/2] arm64: Add software workaround for Falkor erratum 1041

2017-12-03 Thread Shanker Donthineni
Hi Will, thanks for your review comments. On 12/01/2017 05:24 AM, Will Deacon wrote: > On Mon, Nov 27, 2017 at 05:18:00PM -0600, Shanker Donthineni wrote: >> The ARM architecture defines the memory locations that are permitted >> to be accessed as the result of a speculative instruction fetch

Re: [PATCH v4 2/2] arm64: Add software workaround for Falkor erratum 1041

2017-12-03 Thread Shanker Donthineni
Hi Will, thanks for your review comments. On 12/01/2017 05:24 AM, Will Deacon wrote: > On Mon, Nov 27, 2017 at 05:18:00PM -0600, Shanker Donthineni wrote: >> The ARM architecture defines the memory locations that are permitted >> to be accessed as the result of a speculative instruction fetch

Re: [PATCH v6 1/6] leds: Add new API to derive a LED name

2017-12-03 Thread Jacek Anaszewski
Dan, On 12/01/2017 05:56 PM, Dan Murphy wrote: > Create an API that is called to derive the > LED name from either the DT label in the child > node or if that does not exist from the parent > node name and an alternate label that is passed in. > > Signed-off-by: Dan Murphy > ---

Re: [PATCH v6 1/6] leds: Add new API to derive a LED name

2017-12-03 Thread Jacek Anaszewski
Dan, On 12/01/2017 05:56 PM, Dan Murphy wrote: > Create an API that is called to derive the > LED name from either the DT label in the child > node or if that does not exist from the parent > node name and an alternate label that is passed in. > > Signed-off-by: Dan Murphy > --- > > v6 - New

Re: KASAN: slab-out-of-bounds Read in strcmp

2017-12-03 Thread Tetsuo Handa
Tetsuo Handa wrote: > which will allow strcmp() to trigger out of bound read when "size" is > larger than strlen(initial_sid_to_string[i]). Oops. "smaller" than. > > Thus, I guess the simplest fix is to use strncmp() instead of strcmp(). Can somebody test below patch? (My CentOS 7 environment

Re: KASAN: slab-out-of-bounds Read in strcmp

2017-12-03 Thread Tetsuo Handa
Tetsuo Handa wrote: > which will allow strcmp() to trigger out of bound read when "size" is > larger than strlen(initial_sid_to_string[i]). Oops. "smaller" than. > > Thus, I guess the simplest fix is to use strncmp() instead of strcmp(). Can somebody test below patch? (My CentOS 7 environment

Re: [PATCH v6 4/6] dt: bindings: lp8860: Update the bindings to the standard

2017-12-03 Thread Jacek Anaszewski
Dan, On 12/01/2017 05:56 PM, Dan Murphy wrote: > Update the lp8860 dt binding to the LED standard where > the LED should have a child node and also adding a > LED trigger entry. > > Signed-off-by: Dan Murphy > --- > > v6 - New patch to fix binding documentation > >

Re: [PATCH v6 4/6] dt: bindings: lp8860: Update the bindings to the standard

2017-12-03 Thread Jacek Anaszewski
Dan, On 12/01/2017 05:56 PM, Dan Murphy wrote: > Update the lp8860 dt binding to the LED standard where > the LED should have a child node and also adding a > LED trigger entry. > > Signed-off-by: Dan Murphy > --- > > v6 - New patch to fix binding documentation > >

Re: [PATCH] cpu/hotplug: merge cpuhp_bp_states_ap_states as cpuhp_hp_states

2017-12-03 Thread Lai Jiangshan
On Sun, Dec 3, 2017 at 2:33 PM, Paul E. McKenney wrote: > On Fri, Dec 01, 2017 at 09:50:05PM +0800, Lai Jiangshan wrote: >> cpuhp_bp_states_ap_states have diffent set of steps >> without any conflicting configed steps, so that they can >> be merged. >> >> The original

Re: [PATCH] cpu/hotplug: merge cpuhp_bp_states_ap_states as cpuhp_hp_states

2017-12-03 Thread Lai Jiangshan
On Sun, Dec 3, 2017 at 2:33 PM, Paul E. McKenney wrote: > On Fri, Dec 01, 2017 at 09:50:05PM +0800, Lai Jiangshan wrote: >> cpuhp_bp_states_ap_states have diffent set of steps >> without any conflicting configed steps, so that they can >> be merged. >> >> The original `[CPUHP_BRINGUP_CPU] = { },`

<    1   2   3   4   5   6   7   >