[PATCH] ASoC: wm8804: Use common error handling code in wm8804_probe()

2017-11-18 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 19 Nov 2017 07:55:49 +0100 * Add a jump target so that a specific error message is stored only once at the end of this function implementation. * Replace two calls of the function "dev_err" by goto statements. This issue was

[PATCH] ASoC: wm8804: Use common error handling code in wm8804_probe()

2017-11-18 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 19 Nov 2017 07:55:49 +0100 * Add a jump target so that a specific error message is stored only once at the end of this function implementation. * Replace two calls of the function "dev_err" by goto statements. This issue was detected by using the Coccinelle

Re: [PATCH] ALSA: via82xx: Use common error handling code in snd_via82xx_create()

2017-11-18 Thread SF Markus Elfring
>> @@ -1153,6 +1150,14 @@ static int snd_via82xx_create(struct snd_card *card, >> >> *r_via = chip; >> return 0; >> + >> +disable_device: >> +pci_disable_device(pci); >> +goto exit; >> +free_chip: >> +snd_via82xx_free(chip); >> +exit: >> +return err; > > Doubly goto

Re: [PATCH] ALSA: via82xx: Use common error handling code in snd_via82xx_create()

2017-11-18 Thread SF Markus Elfring
>> @@ -1153,6 +1150,14 @@ static int snd_via82xx_create(struct snd_card *card, >> >> *r_via = chip; >> return 0; >> + >> +disable_device: >> +pci_disable_device(pci); >> +goto exit; >> +free_chip: >> +snd_via82xx_free(chip); >> +exit: >> +return err; > > Doubly goto

Re: [PATCH V2 2/2] mfd: cros ec: spi: Simplify delay handling between SPI messages

2017-11-18 Thread Benson Leung
Hi Jon, On Tue, Nov 14, 2017 at 02:43:28PM +, Jon Hunter wrote: > The EC SPI driver prevents SPI transfers being to rapidly by keeping > track of the time the last transfer was issued via the > 'last_transfer_ns' variable. Previously, if the 'last_transfer_ns' > variable was zero, this

Re: [PATCH V2 2/2] mfd: cros ec: spi: Simplify delay handling between SPI messages

2017-11-18 Thread Benson Leung
Hi Jon, On Tue, Nov 14, 2017 at 02:43:28PM +, Jon Hunter wrote: > The EC SPI driver prevents SPI transfers being to rapidly by keeping > track of the time the last transfer was issued via the > 'last_transfer_ns' variable. Previously, if the 'last_transfer_ns' > variable was zero, this

Re: [PATCH V2 1/2] mfd: cros ec: spi: Don't send first message too soon

2017-11-18 Thread Benson Leung
Hi Jon, On Tue, Nov 14, 2017 at 02:43:27PM +, Jon Hunter wrote: > On the Tegra124 Nyan-Big chromebook the very first SPI message sent to > the EC is failing. > > The Tegra SPI driver configures the SPI chip-selects to be active-high > by default (and always has for many years). The EC SPI

Re: [PATCH V2 1/2] mfd: cros ec: spi: Don't send first message too soon

2017-11-18 Thread Benson Leung
Hi Jon, On Tue, Nov 14, 2017 at 02:43:27PM +, Jon Hunter wrote: > On the Tegra124 Nyan-Big chromebook the very first SPI message sent to > the EC is failing. > > The Tegra SPI driver configures the SPI chip-selects to be active-high > by default (and always has for many years). The EC SPI

[PATCH 1/2] MAINTAINERS: regulator: Add Documentation/power/regulator/

2017-11-18 Thread Jonathan Neuschäfer
Signed-off-by: Jonathan Neuschäfer --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2811a211632c..a644d41e088c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -14447,6 +14447,7 @@ W: http://www.slimlogic.co.uk/?p=48 T:

[PATCH 2/2] regulator: Update code examples in documentation

2017-11-18 Thread Jonathan Neuschäfer
This involves using the REGULATOR_SUPPLY initializer macro and reindenting some of the code. Signed-off-by: Jonathan Neuschäfer --- Documentation/power/regulator/machine.txt | 36 ++- 1 file changed, 16 insertions(+), 20 deletions(-) diff

[PATCH 1/2] MAINTAINERS: regulator: Add Documentation/power/regulator/

2017-11-18 Thread Jonathan Neuschäfer
Signed-off-by: Jonathan Neuschäfer --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2811a211632c..a644d41e088c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -14447,6 +14447,7 @@ W: http://www.slimlogic.co.uk/?p=48 T: git

[PATCH 2/2] regulator: Update code examples in documentation

2017-11-18 Thread Jonathan Neuschäfer
This involves using the REGULATOR_SUPPLY initializer macro and reindenting some of the code. Signed-off-by: Jonathan Neuschäfer --- Documentation/power/regulator/machine.txt | 36 ++- 1 file changed, 16 insertions(+), 20 deletions(-) diff --git

[PATCH 3/6 v3] mmc: sdhci-acpi: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : return -EINVAL instead of host->irq.

[PATCH 2/6 v3] mmc: s3cmci: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'.

[PATCH 6/6 v3] mmc: sunxi-mmc: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : return -EINVAL instead of host->irq.

[PATCH 3/6 v3] mmc: sdhci-acpi: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : return -EINVAL instead of host->irq. drivers/mmc/host/sdhci-acpi.c | 4

[PATCH 2/6 v3] mmc: s3cmci: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes

[PATCH 6/6 v3] mmc: sunxi-mmc: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : return -EINVAL instead of host->irq. drivers/mmc/host/sunxi-mmc.c | 5

[PATCH 1/6 v3] mmc: meson-gx-mmc: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'.

[PATCH 4/6 v3] mmc: sdhci-spear: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : return -EINVAL instead of host->irq.

[PATCH 1/6 v3] mmc: meson-gx-mmc: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes

[PATCH 4/6 v3] mmc: sdhci-spear: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : return -EINVAL instead of host->irq. drivers/mmc/host/sdhci-spear.c | 4

[PATCH 2/2 v3] misc: atmel-ssc: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'.

[PATCH 1/2 v3] misc: aspeed-lpc-snoop: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'.

[PATCH 2/2 v3] misc: atmel-ssc: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ 0 is not valid. changes

[PATCH 1/2 v3] misc: aspeed-lpc-snoop: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ 0 is not valid. changes

[PATCH 3/3 v3] mfd: sun4i-gpadc: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. If IRQ0 is not valid. changes in v3 : return -EINVAL insted of irq.

[PATCH 1/3 v3] mfd: ipaq-micro: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'.

[PATCH 1/3 v3] mfd: ipaq-micro: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. If IRQ0 is not valid.

[PATCH 3/3 v3] mfd: sun4i-gpadc: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. If IRQ0 is not valid. changes in v3 : return -EINVAL insted of irq. drivers/mfd/sun4i-gpadc.c | 7 +--

Re: [PATCH v2] serial: 8250_pci: Add Amazon PCI serial device ID

2017-11-18 Thread Matt Wilson
On Mon, Nov 13, 2017 at 11:31:31AM -0800, Matt Wilson wrote: > From: Matt Wilson > > This device will be used in future Amazon EC2 instances as the primary > serial port (i.e., data sent to this port will be available via the > GetConsoleOuput [1] EC2 API). Ping? --msw > [1]

Re: [PATCH v2] serial: 8250_pci: Add Amazon PCI serial device ID

2017-11-18 Thread Matt Wilson
On Mon, Nov 13, 2017 at 11:31:31AM -0800, Matt Wilson wrote: > From: Matt Wilson > > This device will be used in future Amazon EC2 instances as the primary > serial port (i.e., data sent to this port will be available via the > GetConsoleOuput [1] EC2 API). Ping? --msw > [1] >

Re: [PATCH v2] iio: mma8452: replace license description with SPDX specifier

2017-11-18 Thread harinath Nampally
> This replaces the custom license information text with the appropriate > SPDX identifier. While the information here stays the same, it is easier > to read. > Signed-off-by: Martin Kepplinger > Acked-by: Peter Meerwald-Stadler Acked-by: Harinath Nampally

Re: [PATCH v2] iio: mma8452: replace license description with SPDX specifier

2017-11-18 Thread harinath Nampally
> This replaces the custom license information text with the appropriate > SPDX identifier. While the information here stays the same, it is easier > to read. > Signed-off-by: Martin Kepplinger > Acked-by: Peter Meerwald-Stadler Acked-by: Harinath Nampally On Sat, Nov 18, 2017 at 11:29 AM,

Re: [RFC v2] prctl: prctl(PR_SET_IDLE, PR_IDLE_MODE_KILLME), for stateless idle loops

2017-11-18 Thread Matthew Wilcox
On Fri, Nov 17, 2017 at 08:45:03PM -0800, Shawn Landden wrote: > On Fri, Nov 3, 2017 at 2:09 AM, Michal Hocko wrote: > > On Thu 02-11-17 23:35:44, Shawn Landden wrote: > > > 16 bytes per process is kinda spendy, but I want to keep > > > lru behavior, which mem_score_adj does

Re: [RFC v2] prctl: prctl(PR_SET_IDLE, PR_IDLE_MODE_KILLME), for stateless idle loops

2017-11-18 Thread Matthew Wilcox
On Fri, Nov 17, 2017 at 08:45:03PM -0800, Shawn Landden wrote: > On Fri, Nov 3, 2017 at 2:09 AM, Michal Hocko wrote: > > On Thu 02-11-17 23:35:44, Shawn Landden wrote: > > > 16 bytes per process is kinda spendy, but I want to keep > > > lru behavior, which mem_score_adj does not allow. When a

[PATCH 5/5 v3] ASoC: intel: mfld: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : Return EIO insted of ctx->irq_num.

[PATCH 5/5 v3] ASoC: intel: mfld: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : Return EIO insted of ctx->irq_num. sound/soc/intel/boards/mfld_machine.c

[PATCH 3/5 v4] ASoC: nuc900: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : irq was unsigned. so using signed variable

[PATCH 3/5 v4] ASoC: nuc900: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : irq was unsigned. so using signed variable ret. changes in v3 :

[PATCH 2/5 v4] ASoC: mt8173: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : irq was unsigned. so changed it to signed.

[PATCH 4/5 v3] ASoC: intel: sst: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : Return EIO insted of ctx->irq_num.

[PATCH 2/5 v4] ASoC: mt8173: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : irq was unsigned. so changed it to signed. changes in v3 :

[PATCH 4/5 v3] ASoC: intel: sst: Handle return value of platform_get_irq

2017-11-18 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- changes in v2 : Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid. changes in v3 : Return EIO insted of ctx->irq_num. sound/soc/intel/atom/sst/sst_acpi.c |

[PATCH 1/5 v4] ASoC: ep93xx-ac97: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : irq was unsigned. so changed it to signed.

[PATCH 1/5 v4] ASoC: ep93xx-ac97: Fix platform_get_irq's error checking

2017-11-18 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2 : irq was unsigned. so changed it to signed. changes in v3 :

Re: [22/26] MIPS: generic: Introduce generic DT-based board support

2017-11-18 Thread Guenter Roeck
On Fri, Aug 26, 2016 at 04:37:21PM +0100, Paul Burton wrote: > Introduce a "generic" platform, which aims to be board-agnostic by > making use of device trees passed by the boot protocol defined in the > MIPS UHI (Universal Hosting Interface) specification. Provision is made > for supporting

Re: [22/26] MIPS: generic: Introduce generic DT-based board support

2017-11-18 Thread Guenter Roeck
On Fri, Aug 26, 2016 at 04:37:21PM +0100, Paul Burton wrote: > Introduce a "generic" platform, which aims to be board-agnostic by > making use of device trees passed by the boot protocol defined in the > MIPS UHI (Universal Hosting Interface) specification. Provision is made > for supporting

Re: [PATCH v2 net-next] net/tcp: trace all TCP/IP state transition with tcp_set_state tracepoint

2017-11-18 Thread David Miller
From: Yafang Shao Date: Sat, 18 Nov 2017 15:32:36 + > The TCP/IP transition from TCP_LISTEN to TCP_SYN_RECV and some other > transitions are not traced with tcp_set_state tracepoint. > > In order to trace the whole tcp lifespans, two helpers are introduced, > void

Re: [PATCH v2 net-next] net/tcp: trace all TCP/IP state transition with tcp_set_state tracepoint

2017-11-18 Thread David Miller
From: Yafang Shao Date: Sat, 18 Nov 2017 15:32:36 + > The TCP/IP transition from TCP_LISTEN to TCP_SYN_RECV and some other > transitions are not traced with tcp_set_state tracepoint. > > In order to trace the whole tcp lifespans, two helpers are introduced, > void __tcp_set_state(struct

Re: [PATCH] usbnet: ipheth: fix potential null pointer dereference in ipheth_carrier_set

2017-11-18 Thread David Miller
From: "Gustavo A. R. Silva" Date: Fri, 17 Nov 2017 14:02:09 -0600 > _dev_ is being dereferenced before it is null checked, hence there > is a potential null pointer dereference. > > Fix this by moving the pointer dereference after _dev_ has been null > checked. > >

Re: [PATCH] usbnet: ipheth: fix potential null pointer dereference in ipheth_carrier_set

2017-11-18 Thread David Miller
From: "Gustavo A. R. Silva" Date: Fri, 17 Nov 2017 14:02:09 -0600 > _dev_ is being dereferenced before it is null checked, hence there > is a potential null pointer dereference. > > Fix this by moving the pointer dereference after _dev_ has been null > checked. > > Addresses-Coverity-ID:

[PATCH] perf probe: Add a tip about define a tracepoint event

2017-11-18 Thread minchae
perf probe command defines dynamic tracepoint events, by symbol and registers without debuginfo, or by C expressions with debuginfo. --- tools/perf/Documentation/tips.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/Documentation/tips.txt

[PATCH] perf probe: Add a tip about define a tracepoint event

2017-11-18 Thread minchae
perf probe command defines dynamic tracepoint events, by symbol and registers without debuginfo, or by C expressions with debuginfo. --- tools/perf/Documentation/tips.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/Documentation/tips.txt

Re: [PATCH] RCU: Remove have_rcu_nocb_mask from tree_plugin.h

2017-11-18 Thread Paul E. McKenney
On Fri, Nov 17, 2017 at 09:40:15PM +0600, Rakib Mullick wrote: > Currently have_rcu_nocb_mask is used to avoid double allocation of > rcu_nocb_mask during boot up. Due to different representation of > cpumask_var_t on different kernel config CPUMASK=y(or n) it was okay. > But now we have a helper

Re: [PATCH] RCU: Remove have_rcu_nocb_mask from tree_plugin.h

2017-11-18 Thread Paul E. McKenney
On Fri, Nov 17, 2017 at 09:40:15PM +0600, Rakib Mullick wrote: > Currently have_rcu_nocb_mask is used to avoid double allocation of > rcu_nocb_mask during boot up. Due to different representation of > cpumask_var_t on different kernel config CPUMASK=y(or n) it was okay. > But now we have a helper

Re: [PATCH RFC 00/25] Replacing net_mutex with rw_semaphore

2017-11-18 Thread Eric W. Biederman
Kirill Tkhai writes: > Hi, > > this is continuation of discussion from here: > > https://lkml.org/lkml/2017/11/14/298 > > The plan has changed a little bit, so I'd be happy to hear > people's comments, before I dived into all 400+ pernet subsys > and devices. > > The patch

Re: [PATCH RFC 00/25] Replacing net_mutex with rw_semaphore

2017-11-18 Thread Eric W. Biederman
Kirill Tkhai writes: > Hi, > > this is continuation of discussion from here: > > https://lkml.org/lkml/2017/11/14/298 > > The plan has changed a little bit, so I'd be happy to hear > people's comments, before I dived into all 400+ pernet subsys > and devices. > > The patch set adds pernet sys

Re: [PATCH] HID: elecom: fix the descriptor of the EX-G trackball

2017-11-18 Thread Tomasz Kramkowski
On Sun, Nov 19, 2017 at 12:31:45AM +, Diego Elio Pettenò wrote: > Please do not drop the explicit documentation of the diff. Looking at what > a driver does in three years is not going to be obvious, whether you know > HID or not. I'm sure there are situations where this is true but I've just

Re: [PATCH] HID: elecom: fix the descriptor of the EX-G trackball

2017-11-18 Thread Tomasz Kramkowski
On Sun, Nov 19, 2017 at 12:31:45AM +, Diego Elio Pettenò wrote: > Please do not drop the explicit documentation of the diff. Looking at what > a driver does in three years is not going to be obvious, whether you know > HID or not. I'm sure there are situations where this is true but I've just

[PATCH] iio: cros_ec: Remove unused variables

2017-11-18 Thread Paolo Cretaro
Fix gcc warnings about variable 'ec_device' being set but not used in these files: common/cros_ec_sensors/cros_ec_sensors.c:194:25 light/cros_ec_light_prox.c:184:25 Signed-off-by: Paolo Cretaro --- drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c | 2 --

[PATCH] iio: cros_ec: Remove unused variables

2017-11-18 Thread Paolo Cretaro
Fix gcc warnings about variable 'ec_device' being set but not used in these files: common/cros_ec_sensors/cros_ec_sensors.c:194:25 light/cros_ec_light_prox.c:184:25 Signed-off-by: Paolo Cretaro --- drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c | 2 --

[PATCH] perf trace: Add tips about perf trace

2017-11-18 Thread Sangwon Hong
Add two tips that summary perf trace and perf trace record to know Intuitively. Cc: Jiri Olsa Cc: Namhyung Kim Cc: Taeung Song Signed-off-by: Sangwon Hong --- tools/perf/Documentation/tips.txt | 2 ++ 1 file changed,

[PATCH] perf trace: Add tips about perf trace

2017-11-18 Thread Sangwon Hong
Add two tips that summary perf trace and perf trace record to know Intuitively. Cc: Jiri Olsa Cc: Namhyung Kim Cc: Taeung Song Signed-off-by: Sangwon Hong --- tools/perf/Documentation/tips.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/perf/Documentation/tips.txt

Re: [PATCH] HID: elecom: fix the descriptor of the EX-G trackball

2017-11-18 Thread Tomasz Kramkowski
On Sat, Nov 18, 2017 at 10:27:26PM +, Tomasz Kramkowski wrote: > I was going to do this just now actually but then I noticed that someone > had beat me to the punch with the EX-G (I was already surprised when I > found someone had patched the HUGE and DEFT). Actually, I'll put my money where

Re: [PATCH] HID: elecom: fix the descriptor of the EX-G trackball

2017-11-18 Thread Tomasz Kramkowski
On Sat, Nov 18, 2017 at 10:27:26PM +, Tomasz Kramkowski wrote: > I was going to do this just now actually but then I noticed that someone > had beat me to the punch with the EX-G (I was already surprised when I > found someone had patched the HUGE and DEFT). Actually, I'll put my money where

Re: [PATCH v2 06/15] ima: add parser of digest lists metadata

2017-11-18 Thread Mimi Zohar
Hi Serge, On Fri, 2017-11-17 at 22:20 -0600, Serge E. Hallyn wrote: > On Tue, Nov 07, 2017 at 11:37:01AM +0100, Roberto Sassu wrote: > > from a predefined position (/etc/ima/digest_lists/metadata), when rootfs > > becomes available. Digest lists must be loaded before IMA appraisal is in > >

Re: [PATCH v2 06/15] ima: add parser of digest lists metadata

2017-11-18 Thread Mimi Zohar
Hi Serge, On Fri, 2017-11-17 at 22:20 -0600, Serge E. Hallyn wrote: > On Tue, Nov 07, 2017 at 11:37:01AM +0100, Roberto Sassu wrote: > > from a predefined position (/etc/ima/digest_lists/metadata), when rootfs > > becomes available. Digest lists must be loaded before IMA appraisal is in > >

Re: [alsa-devel] [PATCH] ASoC: fsl_asrc: Fix typo in a field define

2017-11-18 Thread Fabio Estevam
On Thu, Nov 16, 2017 at 5:55 PM, Nicolin Chen wrote: > ASRFSTi_IAEi has an 11-bit offset as its _SHIFT macro defines. > > So this patch just fixes that. > > Reported-by: Laurent Charpentier > Signed-off-by: Nicolin Chen

Re: [alsa-devel] [PATCH] ASoC: fsl_asrc: Fix typo in a field define

2017-11-18 Thread Fabio Estevam
On Thu, Nov 16, 2017 at 5:55 PM, Nicolin Chen wrote: > ASRFSTi_IAEi has an 11-bit offset as its _SHIFT macro defines. > > So this patch just fixes that. > > Reported-by: Laurent Charpentier > Signed-off-by: Nicolin Chen Reviewed-by: Fabio Estevam

Re: [PATCH] HID: elecom: fix the descriptor of the EX-G trackball

2017-11-18 Thread Tomasz Kramkowski
Since the device only has 6 physical buttons, I don't think unmasking all 8 bits in the HID report descriptor is appropriate. If you do that then userspace sees a mouse with 8 buttons and you never know when software might actually behave differently depending on that number. (I can imagine fancy

Re: [PATCH] HID: elecom: fix the descriptor of the EX-G trackball

2017-11-18 Thread Tomasz Kramkowski
Since the device only has 6 physical buttons, I don't think unmasking all 8 bits in the HID report descriptor is appropriate. If you do that then userspace sees a mouse with 8 buttons and you never know when software might actually behave differently depending on that number. (I can imagine fancy

Re: seccomp() SECCOMP_RET_KILL_PROCESS text for man page

2017-11-18 Thread Michael Kerrisk (man-pages)
Hi Kees! On 18 November 2017 at 21:52, Kees Cook wrote: > On Sat, Nov 18, 2017 at 12:04 PM, Michael Kerrisk (man-pages) > wrote: >> Hi Kees, >> >> I came up with the following text (patch below) to describe the >> SECCOMP_RET_KILL_PROCESS action

Re: seccomp() SECCOMP_RET_KILL_PROCESS text for man page

2017-11-18 Thread Michael Kerrisk (man-pages)
Hi Kees! On 18 November 2017 at 21:52, Kees Cook wrote: > On Sat, Nov 18, 2017 at 12:04 PM, Michael Kerrisk (man-pages) > wrote: >> Hi Kees, >> >> I came up with the following text (patch below) to describe the >> SECCOMP_RET_KILL_PROCESS action that you added in 4.14. Does it >> look okay? >>

transaction charged twice.

2017-11-18 Thread Ahalan Kamal Arnab
Hello, Two days ago i bought from your website at value 215.60 dedic. I used my credit card to pay for 215.60 for those service. Yesterday morning when I checked my account, I had the transaction charged twice. I have attached my bank statement that shows the double charge. Plase let

transaction charged twice.

2017-11-18 Thread Ahalan Kamal Arnab
Hello, Two days ago i bought from your website at value 215.60 dedic. I used my credit card to pay for 215.60 for those service. Yesterday morning when I checked my account, I had the transaction charged twice. I have attached my bank statement that shows the double charge. Plase let

Re: [PATCH] drivers/char/random.c: remove unused dont_count_entropy

2017-11-18 Thread Rasmus Villemoes
On 28 October 2017 at 00:30, Rasmus Villemoes wrote: > Ever since "random: kill dead extract_state struct" [1], the > dont_count_entropy member of struct timer_rand_state has been > effectively unused. Since it hasn't found a new use in 12 years, it's > probably safe to

Re: [PATCH] drivers/char/random.c: remove unused dont_count_entropy

2017-11-18 Thread Rasmus Villemoes
On 28 October 2017 at 00:30, Rasmus Villemoes wrote: > Ever since "random: kill dead extract_state struct" [1], the > dont_count_entropy member of struct timer_rand_state has been > effectively unused. Since it hasn't found a new use in 12 years, it's > probably safe to finally kill it. Ping.

Re: [PATCH] dt-bindings: trivial-devices: Remove fsl,mc13892

2017-11-18 Thread Fabio Estevam
On Sat, Nov 18, 2017 at 12:22 AM, Jonathan Neuschäfer wrote: > This device's bindings are not trivial: Additional properties are > documented in in Documentation/devicetree/bindings/mfd/mc13xxx.txt. > > Signed-off-by: Jonathan Neuschäfer

Re: [PATCH] dt-bindings: trivial-devices: Remove fsl,mc13892

2017-11-18 Thread Fabio Estevam
On Sat, Nov 18, 2017 at 12:22 AM, Jonathan Neuschäfer wrote: > This device's bindings are not trivial: Additional properties are > documented in in Documentation/devicetree/bindings/mfd/mc13xxx.txt. > > Signed-off-by: Jonathan Neuschäfer Reviewed-by: Fabio Estevam Thanks

Re: [git pull] vfs.git get_user_pages_fast() conversion

2017-11-18 Thread Dan Williams
On Fri, Nov 17, 2017 at 1:32 PM, Al Viro wrote: > On Fri, Nov 17, 2017 at 12:50:47PM -0800, Linus Torvalds wrote: > >> Not because the conversion was wrong, but because the original code is >> so broken. >> >> In particular, that "1" that is unchanged in the arguments is

Re: [git pull] vfs.git get_user_pages_fast() conversion

2017-11-18 Thread Dan Williams
On Fri, Nov 17, 2017 at 1:32 PM, Al Viro wrote: > On Fri, Nov 17, 2017 at 12:50:47PM -0800, Linus Torvalds wrote: > >> Not because the conversion was wrong, but because the original code is >> so broken. >> >> In particular, that "1" that is unchanged in the arguments is correct >> in the

transaction charged twice.

2017-11-18 Thread Davi |||
Hello, Two days ago i bought from your website at value 215.60 dedic. I used my credit card to pay for 215.60 for those service. Yesterday morning when I checked my account, I had the transaction charged twice. I have attached my bank statement that shows the double charge. Plase let me

transaction charged twice.

2017-11-18 Thread Davi |||
Hello, Two days ago i bought from your website at value 215.60 dedic. I used my credit card to pay for 215.60 for those service. Yesterday morning when I checked my account, I had the transaction charged twice. I have attached my bank statement that shows the double charge. Plase let me

Re: [git pull] vfs.git get_user_pages_fast() conversion

2017-11-18 Thread Al Viro
On Fri, Nov 17, 2017 at 09:32:15PM +, Al Viro wrote: > And for get_user_pages() itself it's even more ridiculous - vmalist (the last > argument) is non-NULL in only one caller. Which uses it only to check if all > of the VMAs happen to be hugetlb ones, apparently. > > FWIW, I wanted to trim

Re: [git pull] vfs.git get_user_pages_fast() conversion

2017-11-18 Thread Al Viro
On Fri, Nov 17, 2017 at 09:32:15PM +, Al Viro wrote: > And for get_user_pages() itself it's even more ridiculous - vmalist (the last > argument) is non-NULL in only one caller. Which uses it only to check if all > of the VMAs happen to be hugetlb ones, apparently. > > FWIW, I wanted to trim

Re: [PATCH 00/10] x86: Add support for running as secondary Jailhouse guest

2017-11-18 Thread H. Peter Anvin
On 11/15/17 23:26, Jan Kiszka wrote: > This series paves the way to run Linux in so-called non-root cells > (guest partitions) of the Jailhouse hypervisor. > > Jailhouse [1] was started 4 years ago as an open-source (GPL) leight- > weight hypervisor that statically partitions SMP systems. It's

Re: [PATCH 00/10] x86: Add support for running as secondary Jailhouse guest

2017-11-18 Thread H. Peter Anvin
On 11/15/17 23:26, Jan Kiszka wrote: > This series paves the way to run Linux in so-called non-root cells > (guest partitions) of the Jailhouse hypervisor. > > Jailhouse [1] was started 4 years ago as an open-source (GPL) leight- > weight hypervisor that statically partitions SMP systems. It's

[PATCH] ALSA: cs5530: Use common error handling code in snd_cs5530_probe()

2017-11-18 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 18 Nov 2017 22:10:19 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH] ALSA: cs5530: Use common error handling code in snd_cs5530_probe()

2017-11-18 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 18 Nov 2017 22:10:19 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- sound/pci/cs5530.c | 17

Re: [RFC PATCH v3 for 4.15 08/24] Provide cpu_opv system call

2017-11-18 Thread Andy Lutomirski
On Fri, Nov 17, 2017 at 12:07 PM, Thomas Gleixner wrote: > On Fri, 17 Nov 2017, Andi Kleen wrote: >> > The most straight forward is to have a mechanism which forces everything >> > into the slow path in case of debugging, lack of progress, etc. The slow >> >> That's the abort

Re: [RFC PATCH v3 for 4.15 08/24] Provide cpu_opv system call

2017-11-18 Thread Andy Lutomirski
On Fri, Nov 17, 2017 at 12:07 PM, Thomas Gleixner wrote: > On Fri, 17 Nov 2017, Andi Kleen wrote: >> > The most straight forward is to have a mechanism which forces everything >> > into the slow path in case of debugging, lack of progress, etc. The slow >> >> That's the abort address, right? > >

Re: seccomp() SECCOMP_RET_KILL_PROCESS text for man page

2017-11-18 Thread Kees Cook
On Sat, Nov 18, 2017 at 12:04 PM, Michael Kerrisk (man-pages) wrote: > Hi Kees, > > I came up with the following text (patch below) to describe the > SECCOMP_RET_KILL_PROCESS action that you added in 4.14. Does it > look okay? > >SECCOMP_RET_KILL_PROCESS (since

Re: seccomp() SECCOMP_RET_KILL_PROCESS text for man page

2017-11-18 Thread Kees Cook
On Sat, Nov 18, 2017 at 12:04 PM, Michael Kerrisk (man-pages) wrote: > Hi Kees, > > I came up with the following text (patch below) to describe the > SECCOMP_RET_KILL_PROCESS action that you added in 4.14. Does it > look okay? > >SECCOMP_RET_KILL_PROCESS (since Linux 4.14) >

Re: [patch V2 02/11] LICENSES: Add the GPL 2.0 license

2017-11-18 Thread Charlemagne Lasse
2017-11-18 20:14 GMT+01:00 Linus Torvalds : > You may be confusing things because of a newer version. > > I refuse to change the original copyright wording due to idiotic > internal FSF politics that tried to change history. But you are accepting commit messages

Re: [patch V2 02/11] LICENSES: Add the GPL 2.0 license

2017-11-18 Thread Charlemagne Lasse
2017-11-18 20:14 GMT+01:00 Linus Torvalds : > You may be confusing things because of a newer version. > > I refuse to change the original copyright wording due to idiotic > internal FSF politics that tried to change history. But you are accepting commit messages which are factually wrong? I am

[PATCH 2/2] ALSA: vx222: Use common error handling code in two functions

2017-11-18 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 18 Nov 2017 21:23:27 +0100 Add jump targets so that a bit of exception handling can be better reused at the end of these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH 2/2] ALSA: vx222: Use common error handling code in two functions

2017-11-18 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 18 Nov 2017 21:23:27 +0100 Add jump targets so that a bit of exception handling can be better reused at the end of these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- sound/pci/vx222/vx222.c | 44

[PATCH 1/2] ALSA: vx222: Adjust ten function calls together with a variable assignment

2017-11-18 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 18 Nov 2017 21:10:37 +0100 The script "checkpatch.pl" pointed information out like the following. ERROR: do not use assignment in if condition Thus fix the affected source code places. Signed-off-by: Markus Elfring

[PATCH 1/2] ALSA: vx222: Adjust ten function calls together with a variable assignment

2017-11-18 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 18 Nov 2017 21:10:37 +0100 The script "checkpatch.pl" pointed information out like the following. ERROR: do not use assignment in if condition Thus fix the affected source code places. Signed-off-by: Markus Elfring --- sound/pci/vx222/vx222.c | 19

  1   2   3   4   5   6   >