Re: [PATCH v2] HID: udraw: Add support for the uDraw tablet for PS3

2016-11-12 Thread Bastien Nocera
On Tue, 2016-11-08 at 14:30 +0100, Bastien Nocera wrote: > On Tue, 2016-11-08 at 14:23 +0100, Jiri Kosina wrote: > > > > Hi Bastien, > > > > overall the driver looks ok to me, thanks. Probably the only > > concern > > I'd  > > have -- traditionally, we've always been naming drivers by the > >

[PATCH] [media] zoran: fix spelling mistake in dprintk message

2016-11-12 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake "unnsupported" to "unsupported" in debug message. Signed-off-by: Colin Ian King --- drivers/media/pci/zoran/zoran_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/zoran/zoran_driver.c

[PATCH] [STYLE 2/2]staging:speakup:speakup_spkout.c Spaces preferred around

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to: CHECK: spaces preferred around that '|' Signed-off-by: Walt Feasel --- drivers/staging/speakup/speakup_spkout.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git

Re: [PATCH v4 8/8] iio: envelope-detector: ADC driver based on a DAC and a comparator

2016-11-12 Thread Jonathan Cameron
On 08/11/16 17:03, Peter Rosin wrote: > On 2016-11-08 16:59, Thomas Gleixner wrote: >> On Tue, 8 Nov 2016, Peter Rosin wrote: >>> +/* >>> + * The envelope_detector_comp_latch function works together with the >>> compare >>> + * interrupt service routine below (envelope_detector_comp_isr) as a

Re: [PATCH v4 8/8] iio: envelope-detector: ADC driver based on a DAC and a comparator

2016-11-12 Thread Jonathan Cameron
On 11/11/16 11:37, Peter Rosin wrote: > On 2016-11-09 16:06, Thomas Gleixner wrote: >> On Wed, 9 Nov 2016, Peter Rosin wrote: >>> On 2016-11-08 22:47, Thomas Gleixner wrote: I don't think you need extra race handling with that, but I might be wrong as usual. >>> >>> There's obviously no

[PATCH] [STYLE 1/2]staging:speakup:speakup_txprt.c Block comment align

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to: WARNING: Block comments should align the * on each line Modified multiline comment to single Removed blank line at end Signed-off-by: Walt Feasel --- drivers/staging/speakup/speakup_txprt.c | 8 +++- 1 file changed, 3

[PATCH] amdkfd: fix spelling mistake in kfd_ioctl_dbg_unrgesiter

2016-11-12 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake, rename kfd_ioctl_dbg_unrgesiter to kfd_ioctl_dbg_unregister Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdkfd/kfd_chardev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH] [STYLE 2/2]staging:speakup:speakup_txprt.c Spaces preferred around

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to: CHECK: spaces preferred around that '|' Signed-off-by: Walt Feasel --- drivers/staging/speakup/speakup_txprt.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git

[PATCH] drm/vc4: Add threading support

2016-11-12 Thread Eric Anholt
From: Jonas Pfeil Add support for threaded fragment shaders. The validation code checks for thread switch signals and ensures that the registers of the other thread are not touched, and that our clamps are not live across thread switches. It also checks that the threading and branching

[PATCH] drm/sun4i: constify component_ops structures

2016-11-12 Thread Julia Lawall
These component_ops structures are only used as the second argument to component_add and component_del, which are declared as const, so the structures can be declared as const as well. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @r disable

Re: [PATCH 2/6] pinctrl: rockchip: add support for rk1108

2016-11-12 Thread 陈豪
2016-11-03 20:34 GMT+08:00 Andy Yan : > Add basic support for rk1108 soc > > Signed-off-by: Andy Yan > --- > > drivers/pinctrl/pinctrl-rockchip.c | 27 ++- > 1 file changed, 26 insertions(+), 1 deletion(-) > > diff --git a/drivers/pinctrl/pinctrl-rockchip.c >

[PATCH] [STYLE]staging:speakup:spk_priv_keyinfo.h Spaces preferred around

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to CHECK: spaces preferred around that '+' Signed-off-by: Walt Feasel --- drivers/staging/speakup/spk_priv_keyinfo.h | 44 +++--- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git

[PATCH] net: ethernet: ixp4xx_eth: fix spelling mistake in debug message

2016-11-12 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake "successed" to "succeeded" in debug message. Also unwrap multi-line literal string. Signed-off-by: Colin Ian King --- drivers/net/ethernet/xscale/ixp4xx_eth.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH] [STYLE]staging:speakup:spk_types.h Align match parenthesis

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to CHECK: Alignment should match open parenthesis Signed-off-by: Walt Feasel --- drivers/staging/speakup/spk_types.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/spk_types.h

[PATCH] [STYLE 1/5]staging:speakup:synth.c Modified block text

2016-11-12 Thread Walt Feasel
Modified block comment for alignment consistency Signed-off-by: Walt Feasel --- drivers/staging/speakup/synth.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/synth.c b/drivers/staging/speakup/synth.c index 54b2f39..29efdb1 100644 ---

[PATCH v2] [STYLE 1/5]staging:speakup:synth.c Modified block text

2016-11-12 Thread Walt Feasel
Modified block comment for alignment consistency Signed-off-by: Walt Feasel --- drivers/staging/speakup/synth.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/synth.c b/drivers/staging/speakup/synth.c index 54b2f39..800fbbf 100644 ---

[PATCH] misc: ibmasm: fix typo in error message

2016-11-12 Thread Colin King
From: Colin Ian King Trivial fix to typo "repsonse" to "response" in error message. Signed-off-by: Colin Ian King --- drivers/misc/ibmasm/module.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/misc/ibmasm/module.c b/drivers/misc/ibmasm/module.c index

Re: [PATCH] Fix: disable sys_membarrier when nohz_full is enabled

2016-11-12 Thread Paul E. McKenney
On Tue, Nov 08, 2016 at 11:15:51AM +, Mathieu Desnoyers wrote: > - On Nov 7, 2016, at 3:03 PM, Paul E. McKenney paul...@linux.vnet.ibm.com > wrote: > > > On Mon, Nov 07, 2016 at 06:10:14PM +, Mathieu Desnoyers wrote: > >> > >> - On Nov 7, 2016, at 1:06 PM, Paul E. McKenney > >>

Re: [PATCH 03/12] block: floppy: use bio_add_page()

2016-11-12 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig

Re: [PATCH 01/12] block: bio: pass bvec table to bio_init()

2016-11-12 Thread Christoph Hellwig
On Fri, Nov 11, 2016 at 08:05:29PM +0800, Ming Lei wrote: > Some drivers often use external bvec table, so introduce > this helper for this case. It is always safe to access the > bio->bi_io_vec in this way for this case. > > After converting to this usage, it will becomes a bit easier > to

Re: [PATCH v3 3/3] Coccinelle: misc: Add support for devm variant in all modes

2016-11-12 Thread Julia Lawall
On Mon, 24 Oct 2016, Vaishali Thakkar wrote: > Add missing support for the devm_request_threaded_irq in > the rules of context, report and org modes. > > Misc: > > To be consistent with other scripts, change confidence level > of the script to 'Moderate'. > > Signed-off-by: Vaishali Thakkar

Re: [PATCH v3 2/3] Coccinelle: misc: Improve the result given by context mode

2016-11-12 Thread Julia Lawall
On Mon, 24 Oct 2016, Vaishali Thakkar wrote: > To eliminate false positives given by the context mode, add > necessary arguments for the function request_threaded_irq. > > Signed-off-by: Vaishali Thakkar Acked-by: Julia Lawall > --- > Changes since v2: > - Add missing declaration of

[PATCH] [STYLE 2/5]staging:speakup:synth.c Logical continuation previous

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to CHECK: Logical continuations should be on the previous line Signed-off-by: Walt Feasel --- drivers/staging/speakup/synth.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/speakup/synth.c

Re: [PATCH] locktorture: Fix potential memory leak with rw lock test

2016-11-12 Thread Paul E. McKenney
On Thu, Nov 10, 2016 at 01:06:39PM -0800, Yang Shi wrote: > When running locktorture module with the below commands with kmemleak enabled: > > $ modprobe locktorture torture_type=rw_lock_irq > $ rmmod locktorture > > The below kmemleak got caught: > > root@10:~# echo scan >

[PATCH] [STYLE 3/5]staging:speakup:synth.c Space preferred around

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to CHECK: spaces preferred around that '/,+,-' Signed-off-by: Walt Feasel --- drivers/staging/speakup/synth.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/speakup/synth.c

Re: [PATCH v3 1/3] Coccinelle: misc: Improve the matching of rules

2016-11-12 Thread Julia Lawall
On Mon, 24 Oct 2016, Vaishali Thakkar wrote: > Currently because of the left associativity of the operators, pattern > IRQF_ONESHOT | flags does not match with the pattern when we have more > than one flag after the disjunction. This eventually results in giving > false positives by the script.

[PATCH] [STYLE 4/5]staging:speakup:synth.c Align match parenthesis

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to CHECK: Alignment should match open parenthesis Signed-off-by: Walt Feasel --- drivers/staging/speakup/synth.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/speakup/synth.c

[PATCH] [STYLE 5/5]staging:speakup:synth.c Blankline before close }

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to CHECK: Blank lines aren't necessary before a close brace '}' Signed-off-by: Walt Feasel --- drivers/staging/speakup/synth.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/speakup/synth.c

[PATCH] [STYLE 1/2]staging:speakup:thread.c Modified block comment

2016-11-12 Thread Walt Feasel
Modified block comment to multiline style Signed-off-by: Walt Feasel --- drivers/staging/speakup/thread.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/speakup/thread.c b/drivers/staging/speakup/thread.c index 90c383e..e4de7dc 100644 ---

Re: Source address fib invalidation on IPv6

2016-11-12 Thread David Ahern
On 11/12/16 8:40 AM, Jason A. Donenfeld wrote: > Hi again, > > I've done some pretty in depth debugging now to determine exactly what > the behavior of ipv6_stub->ipv6_dst_lookup is. First I'll start with > ip_route_output_flow, which I believe to be well behaved, and then > I'll show

[PATCH] [STYLE 2/2]staging:speakup:thread.c Align match parenthesis

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to: CHECK: Alignment should match open parenthesis Signed-off-by: Walt Feasel --- drivers/staging/speakup/thread.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/thread.c

[PATCH v4 23/26] staging: iio: tsl2583: remove unnecessary memset call

2016-11-12 Thread Brian Masney
The entries in the lux table (als_device_lux) can be updated via sysfs through the function in_illuminance_lux_table_store(). The last row in the table must be terminated with values that are zero. The sysfs code already ensures that the last row is all zeros. The call to memset to clear out the

[PATCH v4 26/26] staging: iio: tsl2583: move out of staging

2016-11-12 Thread Brian Masney
Move tsl2580, tsl2581, tsl2583 driver out of staging into mainline. Signed-off-by: Brian Masney --- .../ABI/testing/sysfs-bus-iio-light-tsl2583| 20 + drivers/iio/light/Kconfig | 7 + drivers/iio/light/Makefile | 1 +

[PATCH v4 24/26] staging: iio: tsl2583: remove unnecessary variable initialization

2016-11-12 Thread Brian Masney
The ret variable in tsl2583_suspend() and tsl2583_resume() was initialized to 0. This is not necessary so this patch removes the initialization. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v4 25/26] staging: iio: tsl2583: add copyright and MODULE_AUTHOR

2016-11-12 Thread Brian Masney
Add Brian Masney's copyright to the header and to the MODULE_AUTHOR for all of the staging cleanups that has been done to this driver. The original MODULE_AUTHOR() did not have a space between his name and email address. This patch also adds the missing space. Signed-off-by: Brian Masney ---

[PATCH v4 22/26] staging: iio: tsl2583: remove comment for tsl2583_probe()

2016-11-12 Thread Brian Masney
The comment for tsl2583_probe() does not provide any useful value. This patch removes the comment. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/iio/light/tsl2583.c

[PATCH v4 21/26] staging: iio: tsl2583: clarified comment about clearing interrupts

2016-11-12 Thread Brian Masney
The comment that describes the code that clears the interrupt bit was vague and didn't provide much value. This patch adds more detail about why that bit needs to be cleared. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 5 +++-- 1 file changed, 3 insertions(+), 2

[PATCH v4 20/26] staging: iio: tsl2583: add tsl2583 to list of supported devices in the header

2016-11-12 Thread Brian Masney
The header only listed the tsl2580 and tsl2581 devices as supported by this driver. This patch adds the tsl2583 since it is also supported by this driver. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v4 16/26] staging: iio: tsl2583: change tsl2583_als_calibrate() to return 0 on success

2016-11-12 Thread Brian Masney
tsl2583_als_calibrate() returns the newly computed gain_trim if the calibration was successful. This function is only called by in_illuminance_calibrate_store() and the return value inside that sysfs attribute is only checked to see if an error was returned. This patch changes

[PATCH v4 19/26] staging: iio: tsl2583: move from a global to a per device lux table

2016-11-12 Thread Brian Masney
The driver contains a global lux table that can be updated via sysfs. Change this to a per device lux table so that multiple devices can be hooked up to the same system with different lux tables. There are 10 entries, plus 1 for the termination segment, set aside for the entries in the lux table.

[PATCH v4 15/26] staging: iio: tsl2583: moved code block inside else statement

2016-11-12 Thread Brian Masney
The check for ch1lux > ch0lux inside tsl2583_get_lux is only valid if the ratio is not equal to zero. Move the code block inside the else statement. This does away with the need to initialize the variables to zero. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 23

[PATCH v4 13/26] staging: iio: tsl2583: fix multiline comment syntax

2016-11-12 Thread Brian Masney
The definition of the tsl2583_device_lux struct has a series of single line comments. There are two other cases where the multiline comments did not have an initial blank line. Change these comments to use the proper multiline syntax. Signed-off-by: Brian Masney ---

[PATCH] [STYLE 2/2]staging:speakup:varhandlers.c Align match parenthesis

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to CHECK: Alignment should match open parenthesis Signed-off-by: Walt Feasel --- drivers/staging/speakup/varhandlers.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/speakup/varhandlers.c

[PATCH v4 02/26] staging: iio: tsl2583: remove unnecessary chip status check in taos_get_lux

2016-11-12 Thread Brian Masney
taos_get_lux checks to see if the chip is in a working state. This check is not necessary since it is only called from tsl2583_read_raw and in_illuminance_calibrate_store (via taos_als_calibrate). The chip state is already checked by these functions. Signed-off-by: Brian Masney ---

[PATCH v4 18/26] staging: iio: tsl2583: don't assume an unsigned int is 32 bits

2016-11-12 Thread Brian Masney
in_illuminance_lux_table_store assumes that an unsigned int is 32 bits. Replace this with sizeof(value[1]). Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/light/tsl2583.c

[PATCH v4 01/26] staging: iio: tsl2583: check if chip is in a working state in in_illuminance_calibrate_store

2016-11-12 Thread Brian Masney
in_illuminance_calibrate_store() did not check to see if the chip is in a working state. This patch adds the proper check. The return value from taos_als_calibrate() was also not checked in this function, so the proper check was also added while changes are being made here. Signed-off-by: Brian

[PATCH v4 08/26] staging: iio: tsl2583: unify function and variable prefix to tsl2583_

2016-11-12 Thread Brian Masney
Some functions and variables were prefixed with either taos, tsl258x, taos2583, or tsl2583. Change everything to use the tsl2583 prefix since that is the name of the .c file. The taos_settings member inside the taos_settings struct was renamed to als_settings. Signed-off-by: Brian Masney ---

[PATCH v4 14/26] staging: iio: tsl2583: updated code comment to match what the code does

2016-11-12 Thread Brian Masney
If channel 0 does not have any data, then the code sets the lux to zero. The corresponding comment says that the last value is returned. This updates the comment to correctly reflect what the code does. It also clarifies the comment about why 0 is returned. Signed-off-by: Brian Masney ---

[PATCH v4 17/26] staging: iio: tsl2583: remove unnecessary parentheses

2016-11-12 Thread Brian Masney
in_illuminance_lux_table_store() contains some unnecessary parentheses. This patch removes them since they provide no value. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v4 10/26] staging: iio: tsl2583: fix comparison between signed and unsigned integers

2016-11-12 Thread Brian Masney
Fixed warning found by make W=2: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

[PATCH v4 11/26] staging: iio: tsl2583: change newlines to improve readability

2016-11-12 Thread Brian Masney
Add and remove newlines to improve code readability in preparation for moving the driver out of staging. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/light/tsl2583.c

[PATCH v4 12/26] staging: iio: tsl2583: combine sysfs documentation

2016-11-12 Thread Brian Masney
There are two separate files describing the tsl2583 sysfs attributes. Combine the two files into one. Updated the name of the sysfs attributes to match the current ABI. Signed-off-by: Brian Masney Suggested-by: Peter Meerwald-Stadler --- .../Documentation/light/sysfs-bus-iio-light-tsl2583 |

[PATCH v4 04/26] staging: iio: tsl2583: change current chip state from a tristate to a bool

2016-11-12 Thread Brian Masney
The current chip state is represented as a tristate (working, suspended, and unknown). The unknown state was not used. This patch changes the chip state so that it is now represented as a single boolean value (suspended). Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 20

[PATCH v4 09/26] staging: iio: tsl2583: fix alignment of #define values

2016-11-12 Thread Brian Masney
Most of the values in the #defines have their values aligned on a single column, but some do not. This changes the remaining defines to use consistent alignment with the majority to improve code readability. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 10 +- 1

[PATCH v4 03/26] staging: iio: tsl2583: remove unnecessary chip status checks in suspend/resume

2016-11-12 Thread Brian Masney
The device probing and the suspend/resume code checks a flag internal to the driver that determines whether or not the chip is in a working state. These checks are not needed. This patch removes the unnecessary checks. It will do no harm to the hardware if the chip is reinitialized if it is

[PATCH v4 05/26] staging: iio: tsl2583: remove redundant write to the control register in taos_probe()

2016-11-12 Thread Brian Masney
taos_probe() calls i2c_smbus_write_byte() to select the control register, however there are no subsequent calls to i2c_smbus_read_byte(). The write call is unnecessary and is removed by this patch. Verified that the driver still functions correctly using a TSL2581 hooked up to a Raspberry Pi 2.

[PATCH v4 06/26] staging: iio: tsl2583: remove the FSF's mailing address

2016-11-12 Thread Brian Masney
Address warning from checkpatch: CHECK: Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. Linux already includes a copy of the GPL. Signed-off-by: Brian Masney

[PATCH v4 07/26] staging: iio: tsl2583: cleaned up logging

2016-11-12 Thread Brian Masney
There are several places in the code where the function name is hardcoded in the log message. Use the __func__ constant string to build the log message. This also clarifies some of the error messages to match the code and ensures that the correct priority is used since the message is already being

[PATCH v4 00/26] staging: iio: tsl2583: move out of staging

2016-11-12 Thread Brian Masney
This patch set continues my work to clean up the tsl2583 driver to move it out of staging. Some highlights include: - Eliminated some unnecessary i2c calls to the sensor. - Fixed issue with changes to calibscale and int_time not being set on the chip. - Moved from a global lux table to a per

[PATCH] [STYLE 1/2]staging:speakup:varhandlers.c Moddified block comment

2016-11-12 Thread Walt Feasel
Modified block comment to multiline style Signed-off-by: Walt Feasel --- drivers/staging/speakup/varhandlers.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/speakup/varhandlers.c b/drivers/staging/speakup/varhandlers.c index 21186e3..5ab7245 100644 ---

[PATCH] isci: fix typo in deg_dbg message

2016-11-12 Thread Colin King
From: Colin Ian King Trivial fix to typo "repsonse" to "response" in dev_dbg message. Signed-off-by: Colin Ian King --- drivers/scsi/isci/request.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/isci/request.c b/drivers/scsi/isci/request.c index

[PATCH] [STYLE]staging:MAINTAINERS email revision speakup

2016-11-12 Thread Walt Feasel
Modified email address per the TODO file in speakup's email listing, also verified email address from speakup's website Signed-off-by: Walt Feasel --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index ec1ee3e..40d272c 100644 ---

Re: Source address fib invalidation on IPv6

2016-11-12 Thread Jason A. Donenfeld
Hi David, On Sat, Nov 12, 2016 at 7:14 PM, David Ahern wrote: > I believe that is coming from __ip_route_output_key_hash(), line 2232 with > __ip_dev_find not finding a device with that address. It's possible we simply are looking at different source trees, but I have the -EINVAL return in 4.8

Re: [PATCH v2 1/2] arm64: dts: Add level for cpu dt node for exynos7

2016-11-12 Thread Krzysztof Kozlowski
On Sat, Nov 12, 2016 at 6:00 PM, Alim Akhtar wrote: > Hi Javier, > > On Sat, Nov 12, 2016 at 7:54 PM, Javier Martinez Canillas > wrote: >> Hello Alim, >> >> On 11/12/2016 07:17 AM, Alim Akhtar wrote: >>> This patch adds level for cpu dt node, so that these levels can be used >> >> Do you mean

Re: [PATCH] [STYLE]staging:MAINTAINERS email revision speakup

2016-11-12 Thread Chris Brannon
Walt Feasel writes: > Modified email address per the TODO file in > speakup's email listing, also verified email > address from speakup's website NAK. The website needs updating, because it has Kirk's old addresses. k...@reisers.ca is correct.

[PATCH] Input: psmouse - disable automatic probing of BYD touchpads

2016-11-12 Thread Dmitry Torokhov
BYD automatic protocol detection is extremely unreliable and is often triggers false positives on regular mice, Sentelic touchpads, and other devices. BYD has several documents that have recommended detection sequence, but they conflict with each other and, as far as I can see, still would not

Re: PM regression with LED changes in next-20161109

2016-11-12 Thread Jacek Anaszewski
Hi, On 11/12/2016 11:33 AM, Hans de Goede wrote: Hi, On 12-11-16 11:24, Jacek Anaszewski wrote: Hi, On 11/11/2016 08:28 PM, Hans de Goede wrote: Hi, On 11-11-16 18:03, Jacek Anaszewski wrote: On 11/11/2016 01:01 PM, Pavel Machek wrote: On Thu 2016-11-10 22:34:07, Jacek Anaszewski wrote:

Re: [srcu] Can we suppress sparse warning?

2016-11-12 Thread Paul E. McKenney
On Sat, Nov 12, 2016 at 11:26:15PM +0900, Tetsuo Handa wrote: > Paul E. McKenney wrote: > > Notwithstanding my confusion about what your self-referential > > srcu_dereference() is intended to achieve, what happens if you change the > > "void *ptr = srcu_dereference(ptr, );" to add __rcu? > >

Re: [PATCH v7 4/9] drivers:input:tsc2007: add iio interface to read external ADC input and temperature

2016-11-12 Thread H. Nikolaus Schaller
> Am 12.11.2016 um 15:19 schrieb Jonathan Cameron : > > On 12/11/16 14:04, Jonathan Cameron wrote: >> On 11/11/16 19:02, H. Nikolaus Schaller wrote: >>> The tsc2007 chip not only has a resistive touch screen controller but >>> also an external AUX adc imput which can be used for an ambient >>>

[PATCH] [STYLE]staging:speakup:TODO Modify current email

2016-11-12 Thread Walt Feasel
Modified email address to reflect current address Signed-off-by: Walt Feasel --- drivers/staging/speakup/TODO | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/TODO b/drivers/staging/speakup/TODO index 3094799..993410c 100644 ---

Re: [PATCH RFC] mm: Add debug_virt_to_phys()

2016-11-12 Thread Florian Fainelli
Le 11/11/2016 à 21:43, Will Deacon a écrit : > On Fri, Nov 11, 2016 at 04:44:43PM -0800, Florian Fainelli wrote: >> When CONFIG_DEBUG_VM is turned on, virt_to_phys() maps to >> debug_virt_to_phys() which helps catch vmalloc space addresses being >> passed. This is helpful in debugging bogus

[PATCH] Revert "[STYLE]staging:MAINTAINERS email revision speakup"

2016-11-12 Thread Walt Feasel
This reverts commit 0970517be9d3d3e3a2fd5aa5a9938318b375f2d0. --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 40d272c..ec1ee3e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11540,7 +11540,7 @@ F: drivers/staging/slicoss/

Re: [PATCH] Input: psmouse - disable automatic probing of BYD touchpads

2016-11-12 Thread Pali Rohár
On Saturday 12 November 2016 20:15:45 Dmitry Torokhov wrote: > BYD automatic protocol detection is extremely unreliable and is often > triggers false positives on regular mice, Sentelic touchpads, and > other devices. BYD has several documents that have recommended > detection sequence, but they

[PATCH] [STYLE]staging:skein:threefish_block.c remove blank lines

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to CHECK: Please don't use multiple blank lines Signed-off-by: Walt Feasel --- drivers/staging/skein/threefish_block.c | 16 1 file changed, 16 deletions(-) diff --git a/drivers/staging/skein/threefish_block.c

kvm: use-after-free/GPF in kvm_irq_delivery_to_apic_fast

2016-11-12 Thread Dmitry Vyukov
Hello, The following program triggers use-after-free in kvm_irq_delivery_to_apic_fast: https://gist.githubusercontent.com/dvyukov/68a25fb4f8f48807fb7cdf3ebbb84e58/raw/b7b85810a1070c93387ece6d2388da8dbe937452/gistfile1.txt On commit 015ed9433be2b476ec7e2e6a9a411a56e3b5b035 (Nov 11).

dm-crypt accepts '+' in the key

2016-11-12 Thread Mikulas Patocka
Hi dm-crypt uses the function kstrtou8 to decode the encryption key. kstrtou8 calls kstrtoull and kstrtoull skips the first character if it is '+'. Consequently, it is possible to load keys with '+' in it. For example, this is possible: dmsetup create cr --table "0 131072 crypt

kvm: out-of-bounds write in __rtc_irq_eoi_tracking_restore_one

2016-11-12 Thread Dmitry Vyukov
Hello, The following program triggers slab-ouf-of-bound write: https://gist.githubusercontent.com/dvyukov/c4941c67e2eb5be314b902b17dc089df/raw/4f1844d19f6308135ca14c7f28e0898da1b363de/gistfile1.txt On commit 015ed9433be2b476ec7e2e6a9a411a56e3b5b035 (Nov 11). BUG: KASAN: slab-out-of-bounds in

kvm: BUG in pte_list_remove

2016-11-12 Thread Dmitry Vyukov
Hello, The following program triggers BUG in pte_list_remove: https://gist.githubusercontent.com/dvyukov/c763720296f6f4e51934c58ade173a71/raw/2f78e3e634dcd8c3d39d2fbbdcd2afb02bc736f5/gistfile1.txt On commit 015ed9433be2b476ec7e2e6a9a411a56e3b5b035 (nov 11) kernel BUG at arch/x86/kvm/mmu.c:1016!

kvm: recursive lock in kvm_clear_async_pf_completion_queue

2016-11-12 Thread Dmitry Vyukov
Hello, I've got the following report while running syzkaller fuzzer. On commit 015ed9433be2b476ec7e2e6a9a411a56e3b5b035 (Nov 11). [ INFO: possible recursive locking detected ] 4.9.0-rc4+ #49 Not tainted - kworker/2:1/5658 is trying to acquire lock:

[PATCH 1/2] x86/efi: Fix EFI memmap pointer size warning

2016-11-12 Thread Matt Fleming
From: Borislav Petkov Fix this when building on 32-bit: arch/x86/platform/efi/efi.c: In function ‘__efi_enter_virtual_mode’: arch/x86/platform/efi/efi.c:911:5: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] (efi_memory_desc_t *)pa); ^

[PATCH 2/2] x86/efi: Prevent mixed mode boot corruption with CONFIG_VMAP_STACK

2016-11-12 Thread Matt Fleming
Booting an EFI mixed mode kernel has been crashing since commit: e37e43a497d5 ("x86/mm/64: Enable vmapped stacks (CONFIG_HAVE_ARCH_VMAP_STACK=y)") The user-visible effect in my test setup was the kernel being unable to find the root file system ramdisk. This was likely caused by silent memory

[GIT PULL 0/2] EFI urgent fixes

2016-11-12 Thread Matt Fleming
Folks, please pull the following two EFI patches. The first fixes a build warning for PAE that Boris hit. The second makes mixed-mode EFI boot again after the vmap'd stack changes introduced during the merge window. The following changes since commit bc33b0ca11e3df46a4fa7639ba488c9d4911:

[PATCH] [STYLE]staging:dgnc:dgnc_cls.h Spelling correction

2016-11-12 Thread Walt Feasel
Corrected a spelling mistake for control Signed-off-by: Walt Feasel --- drivers/staging/dgnc/dgnc_cls.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_cls.h b/drivers/staging/dgnc/dgnc_cls.h index 2597e36..463ad30 100644 ---

[PATCH v2] cpufreq: conservative: Decrease frequency faster when the update deferred

2016-11-12 Thread Stratos Karafotis
Conservative governor changes the CPU frequency in steps. That means that if a CPU runs at max frequency, it will need several sampling periods to return to min frequency when the workload is finished. If the update function that calculates the load and target frequency is deferred, the governor

Re: PM regression with LED changes in next-20161109

2016-11-12 Thread Hans de Goede
Hi, On 12-11-16 20:14, Jacek Anaszewski wrote: Why a dedicated file? Are we going to mirror brightness here wrt r/w (show/store) behavior ? If not userspace now needs 2 open fds which is not really nice. If we are and we are not going to use poll for something else on brightness itself then

[PATCH] [STYLE]staging:dgnc:dgnc_driver.c Blankline before }

2016-11-12 Thread Walt Feasel
Made suggested modifications from checkpatch in reference to CHECK: Blank lines aren't necessary before a close brace '}' Signed-off-by: Walt Feasel --- drivers/staging/dgnc/dgnc_driver.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c

[tip:perf/urgent] perf hists: Fix column length on --hierarchy

2016-11-12 Thread tip-bot for Namhyung Kim
Commit-ID: c72ab446cac1d6c9551fd26c4cfef1b2fc5041fd Gitweb: http://git.kernel.org/tip/c72ab446cac1d6c9551fd26c4cfef1b2fc5041fd Author: Namhyung Kim AuthorDate: Tue, 8 Nov 2016 22:08:33 +0900 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 9 Nov 2016 11:55:29 -0300 perf hists: Fix

[tip:perf/urgent] perf/x86/intel/uncore: Add more Intel uncore IMC PCI IDs for SkyLake

2016-11-12 Thread tip-bot for Kan Liang
Commit-ID: d786810b2f896854506e7b698a137f074942e410 Gitweb: http://git.kernel.org/tip/d786810b2f896854506e7b698a137f074942e410 Author: Kan Liang AuthorDate: Tue, 8 Nov 2016 13:54:41 -0500 Committer: Ingo Molnar CommitDate: Fri, 11 Nov 2016 08:30:22 +0100 perf/x86/intel/uncore: Add

[tip:sched/core] sched/deadline: Fix typo in a comment

2016-11-12 Thread tip-bot for Daniel Bristot de Oliveira
Commit-ID: 9846d50df3def5bff9d8a408a958722e79bcaa10 Gitweb: http://git.kernel.org/tip/9846d50df3def5bff9d8a408a958722e79bcaa10 Author: Daniel Bristot de Oliveira AuthorDate: Tue, 8 Nov 2016 11:15:23 +0100 Committer: Ingo Molnar CommitDate: Fri, 11 Nov 2016 08:28:52 +0100

[tip:perf/urgent] perf hists browser: Fix indentation of folded sign on --hierarchy

2016-11-12 Thread tip-bot for Namhyung Kim
Commit-ID: 3d9f4683929a968dc9b9493f4e608b109ad292a2 Gitweb: http://git.kernel.org/tip/3d9f4683929a968dc9b9493f4e608b109ad292a2 Author: Namhyung Kim AuthorDate: Tue, 8 Nov 2016 22:08:30 +0900 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 9 Nov 2016 11:20:56 -0300 perf hists

[tip:locking/core] locking/lockdep: Remove unused parameter from the add_lock_to_list() function

2016-11-12 Thread tip-bot for Tahsin Erdogan
Commit-ID: 83f06168ef15da5dc735c7ea14fae67609ed9538 Gitweb: http://git.kernel.org/tip/83f06168ef15da5dc735c7ea14fae67609ed9538 Author: Tahsin Erdogan AuthorDate: Tue, 8 Nov 2016 00:02:07 -0800 Committer: Ingo Molnar CommitDate: Fri, 11 Nov 2016 08:25:20 +0100 locking/lockdep: Remove

[tip:ras/core] x86/MCE: Correct TSC timestamping of error records

2016-11-12 Thread tip-bot for Borislav Petkov
Commit-ID: 54467353a96577f840cd2348981417c559b21b4b Gitweb: http://git.kernel.org/tip/54467353a96577f840cd2348981417c559b21b4b Author: Borislav Petkov AuthorDate: Thu, 10 Nov 2016 14:10:53 +0100 Committer: Ingo Molnar CommitDate: Fri, 11 Nov 2016 08:08:24 +0100 x86/MCE: Correct TSC

[tip:x86/cpu] x86/cpu/AMD: Clean up cpu_llc_id assignment per topology feature

2016-11-12 Thread tip-bot for Yazen Ghannam
Commit-ID: b6a50cddbcbda7105355898ead18f1a647c22520 Gitweb: http://git.kernel.org/tip/b6a50cddbcbda7105355898ead18f1a647c22520 Author: Yazen Ghannam AuthorDate: Tue, 8 Nov 2016 16:30:54 +0100 Committer: Ingo Molnar CommitDate: Wed, 9 Nov 2016 17:07:43 +0100 x86/cpu/AMD: Clean up

[tip:perf/urgent] perf hists browser: Show folded sign properly on --hierarchy

2016-11-12 Thread tip-bot for Namhyung Kim
Commit-ID: 131d51eb1d17aac3a604cf929fd99ff4dd34f495 Gitweb: http://git.kernel.org/tip/131d51eb1d17aac3a604cf929fd99ff4dd34f495 Author: Namhyung Kim AuthorDate: Tue, 8 Nov 2016 22:08:31 +0900 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 9 Nov 2016 11:30:47 -0300 perf hists

[tip:perf/urgent] perf hist browser: Fix hierarchy column counts

2016-11-12 Thread tip-bot for Namhyung Kim
Commit-ID: 9cba9844547731d2f14d79485c43192ffaa37b76 Gitweb: http://git.kernel.org/tip/9cba9844547731d2f14d79485c43192ffaa37b76 Author: Namhyung Kim AuthorDate: Tue, 25 Oct 2016 01:21:10 +0900 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 9 Nov 2016 11:19:28 -0300 perf hist

[tip:perf/urgent] perf hists browser: Fix column indentation on --hierarchy

2016-11-12 Thread tip-bot for Namhyung Kim
Commit-ID: b9bf911e990a189f89147ee6b0a153ed0125 Gitweb: http://git.kernel.org/tip/b9bf911e990a189f89147ee6b0a153ed0125 Author: Namhyung Kim AuthorDate: Tue, 8 Nov 2016 22:08:32 +0900 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 9 Nov 2016 11:45:58 -0300 perf hists

[PATCH] ppdev: check before attaching port

2016-11-12 Thread Sudip Mukherjee
After parport starts using the device model, all pardevice drivers should decide in their match_port callback function if they want to attach with that particulatr port. ppdev has been converted to use the new parport device-model code but pp_attach() tried to attach with all the ports. Create a

[PATCH] [STYLE]staging:dgnc:dgnc_driver.h Spelling correction

2016-11-12 Thread Walt Feasel
Corrected spelling mistake for statements Signed-off-by: Walt Feasel --- drivers/staging/dgnc/dgnc_driver.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_driver.h b/drivers/staging/dgnc/dgnc_driver.h index 8792026..5770a85 100644 ---

kvm: GPF in irqfd_shutdown/eventfd_ctx_remove_wait_queue

2016-11-12 Thread Dmitry Vyukov
Hello, I've got the following crash while running syzkaller fuzzer. On commit 015ed9433be2b476ec7e2e6a9a411a56e3b5b035 (Nov 11). Unfortunately it is not reproducible. May be related to: https://groups.google.com/forum/#!topic/syzkaller/NKlClJzOOww

kvm: WARNING in em_ret_far

2016-11-12 Thread Dmitry Vyukov
Hello, The following program triggers WARNING in em_ret_far: https://gist.githubusercontent.com/dvyukov/12bc06a5f638d91953f94bcbe49bcc89/raw/6d863470808a695c085620664b7134864b2eb2a9/gistfile1.txt On commit 015ed9433be2b476ec7e2e6a9a411a56e3b5b035 (Nov 11). WARNING: CPU: 2 PID: 3668 at

<    1   2   3   4   5   6   7   >