Re: [PATCH 2/2] staging: iio: ad7606: move out of staging

On 12/11/16 14:32, Eva Rachel Retuya wrote: > On Sat, Nov 12, 2016 at 02:26:51PM +, Jonathan Cameron wrote: >> On 11/11/16 14:22, Lars-Peter Clausen wrote: >>> On 11/11/2016 07:34 AM, Eva Rachel Retuya wrote: Move the ad7606 driver from staging/iio/adc to iio/adc. Also, update the

Re: [PATCH] ARM64: dts: bcm2837-rpi-3-b: remove incorrect pwr LED

Hi Eric, [add Gerd to CC] > Eric Anholt hat am 11. November 2016 um 18:38 geschrieben: > > > From: Andrea Merello > > We are incorrectly defining the pwr LED, attaching it to a gpio line > that is wired to the Wi-Fi SDIO module (which fails due to

Re: [PATCH v3 28/28] staging: iio: tsl2583: move out of staging

On 10/11/16 09:26, Brian Masney wrote: > Move tsl2580, tsl2581, tsl2583 driver out of staging into mainline. > > Signed-off-by: Brian Masney The rest of the series (that I haven't commented on) looks good to me. I'll pick them up once those few minor changes are sorted.

[PATCH] iscsi: fix spelling mistakes in dev_warn messages

From: Colin Ian King Trivial fix to spelling mistake "suspeneded" to "suspended" in dev_warn messages Signed-off-by: Colin Ian King --- drivers/scsi/isci/remote_node_context.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

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

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

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

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

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/5]staging:speakup:synth.c Modified block text

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

dm-crypt accepts '+' in the key

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

[PATCH] regulator: stw481x-vmmc: fix ages old enable error

The regulator has never been properly enabled, it has been dormant all the time. It's strange that MMC was working at all, but it likely worked by the signals going through the levelshifter and reaching the card anyways. Cc: sta...@vger.kernel.org Fixes: 3615a34ea1a6 ("regulator: add STw481x VMMC

Re: [PATCH v3 16/28] staging: iio: tsl2583: updated code comment to match what the code does

On 10/11/16 09:25, Brian Masney wrote: > 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. > > Signed-off-by: Brian Masney

Re: [PATCH v3 03/28] staging: iio: tsl2583: check if chip is suspended in in_illuminance_calibrate_store

On Sat, Nov 12, 2016 at 04:27:32PM +, Jonathan Cameron wrote: > On 12/11/16 16:24, Jonathan Cameron wrote: > > On 10/11/16 09:25, Brian Masney wrote: > >> in_illuminance_calibrate_store() did not check to see if the chip is > >> suspended. This patch adds the proper check. The return value

Re: [PATCH v4 1/8] iio:core: add a callback to allow drivers to provide _available attributes

On 08/11/16 11:58, Peter Rosin wrote: > From: Jonathan Cameron > > A large number of attributes can only take a limited range of values. > Currently in IIO this is handled by directly registering additional > *_available attributes thus providing this information to userspace.

Re: [PATCH v4 2/8] iio: inkern: add helpers to query available values from channels

On 08/11/16 11:58, Peter Rosin wrote: > Specifically a helper for reading the available maximum raw value of a > channel and a helper for forwarding read_avail requests for raw values > from one iio driver to an iio channel that is consumed. > > These rather specific helpers are in turn built

[PATCH] drm/vc4: Add threading support

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

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

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

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

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: [PATCH v2 1/2] arm64: dts: Add level for cpu dt node for exynos7

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

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

> 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

kvm: use-after-free/GPF in kvm_irq_delivery_to_apic_fast

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).

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

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

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

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 ---

Re: [PATCH] x86/cpuid: Deal with broken firmware once more

On 11/10/2016 01:50 PM, Charles (Chas) Williams wrote: > > > On 11/09/2016 10:57 PM, M. Vefa Bicakci wrote: >> [0.002000] mvb: CPU: Physical Processor ID: 0 >> [0.002000] mvb: CPU: Processor Core ID: 0 >> [0.002000] mvb: identify_cpu:1112: c: 880013b0a040, >> c->logical_proc_id:

Re: [PATCH] x86/cpuid: Deal with broken firmware once more

On 11/10/2016 06:31 PM, Boris Ostrovsky wrote: > On 11/10/2016 10:05 AM, Charles (Chas) Williams wrote: >> >> >> On 11/10/2016 09:02 AM, Boris Ostrovsky wrote: >>> On 11/10/2016 06:13 AM, Thomas Gleixner wrote: On Thu, 10 Nov 2016, M. Vefa Bicakci wrote: > I have found that your

Re: [PATCH 1/2] staging: iio: ad7606: replace range/range_available with corresponding scale

Hello, Thanks for explaining it. Now I understand better why read_raw is formatted in that manner. I have some questions in-line: On Fri, Nov 11, 2016 at 03:18:37PM +0100, Lars-Peter Clausen wrote: > On 11/11/2016 07:34 AM, Eva Rachel Retuya wrote: > > Eliminate the non-standard attribute

Re: [PATCH] iio: cros_ec_sensors_core: fix unsigned compared less than zero on status

On 10/11/16 03:16, Guenter Roeck wrote: > On Wed, Nov 9, 2016 at 3:12 PM, Colin King wrote: >> From: Colin Ian King >> >> status is a u8 hence the check if status is less than zero has no effect. >> Fix this by replacing status with int ret so

Re: [PATCH V6 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

On 11/12/2016 11:01 AM, agust...@codeaurora.org wrote: Hey Lorenzo, Hanjun, On 2016-11-11 08:33, Hanjun Guo wrote: Hi Lorenzo, On 11/11/2016 01:58 AM, Lorenzo Pieralisi wrote: On Thu, Nov 10, 2016 at 10:02:35AM -0500, agust...@codeaurora.org wrote: Hey Hanjun, On 2016-11-09 21:36, Hanjun

Re: [PATCH v3 03/28] staging: iio: tsl2583: check if chip is suspended in in_illuminance_calibrate_store

On 12/11/16 16:24, Jonathan Cameron wrote: > On 10/11/16 09:25, Brian Masney wrote: >> in_illuminance_calibrate_store() did not check to see if the chip is >> suspended. This patch adds the proper check. The return value from >> taos_als_calibrate() was also not checked in this function, so the >>

Re: [PATCH v3 04/28] staging: iio: tsl2583: remove unnecessary chip status check in taos_get_lux

On 10/11/16 09:25, Brian Masney wrote: > 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.

[PATCH] [STYLE 2/5]staging:speakup:speakup_soft.c Space preferred around

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

Re: [PATCH v2 2/6] mfd: stm32-adc: Add support for stm32 ADC

On 10/11/16 16:18, Fabrice Gasnier wrote: > Add core driver for STMicroelectronics STM32 ADC (Analog to Digital > Converter). STM32 ADC can be composed of up to 3 ADCs with shared > resources like clock prescaler, common interrupt line and analog > reference voltage. > This core driver basically

Re: [PATCH v2 3/6] iio: adc: Add support for STM32 ADC

On 10/11/16 16:18, Fabrice Gasnier wrote: > This patch adds support for STMicroelectronics STM32 MCU's analog to > digital converter. > > Signed-off-by: Fabrice Gasnier Nice and clean - few minor things inline. Jonathan > --- > drivers/iio/adc/Kconfig | 10 + >

Re: [PATCH v4 3/8] iio: mcp4531: provide range of available raw values

On 08/11/16 11:58, Peter Rosin wrote: > Example: > > $ cat '/sys/bus/iio/devices/iio:device0/out_resistance_raw_available' > [0 1 256] > > Meaning: min 0, step 1 and max 256. > > Signed-off-by: Peter Rosin Applied. > --- > .../testing/sysfs-bus-iio-potentiometer-mcp4531|

[PATCH] ps3_gelic: fix spelling mistake in debug message

From: Colin Ian King Trivial fix to spelling mistake "unmached" to "unmatched" in debug message. Signed-off-by: Colin Ian King --- drivers/net/ethernet/toshiba/ps3_gelic_wireless.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH v4 4/8] dt-bindings: add axentia to vendor-prefixes

On 08/11/16 11:58, Peter Rosin wrote: > Acked-by: Rob Herring > Signed-off-by: Peter Rosin Applied. > --- > Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git

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

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

[PATCH] drm/sun4i: constify component_ops structures

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: Source address fib invalidation on IPv6

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

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

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/

kvm: BUG in pte_list_remove

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

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

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

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

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:

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

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

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

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

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

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

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

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

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

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

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

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

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

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 > light sensor, battery voltage monitoring or any general purpose. > > Additionally it can measure the chip

Spurious code in commit 1bf40ada6290 ("amd-xgbe: Add support for clause 37 auto-negotiation"

Hi, in commit 1bf40ada6290 ("amd-xgbe: Add support for clause 37 auto-negotiation"), we can find: diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-common.h b/drivers/net/ethernet/amd/xgbe/xgbe-common.h index 695e982..8bcf4ef 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe-common.h +++

Re: [PATCH] iio: imu: inv_mpu6050: inform the i2c mux core about how it is used

On 10/11/16 09:23, Peter Rosin wrote: > On 2016-09-03 18:30, Jonathan Cameron wrote: >> On 31/08/16 09:02, Peter Rosin wrote: >>> The i2c mux core can then take appropriate action depending on if it is >>> used for an actual i2c mux, or for an arbitrator or gate. In this case >>> it is used as a

Re: [PATCH v3 05/28] staging: iio: tsl2583: remove unnecessary chip status checks in suspend/resume

On 10/11/16 09:25, Brian Masney wrote: > 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

[PATCH] net: atheros: atl1c: use new api ethtool_{get|set}_link_ksettings

The ethtool api {get|set}_settings is deprecated. We move this driver to new api {get|set}_link_ksettings. Signed-off-by: Philippe Reynes --- drivers/net/ethernet/atheros/atl1c/atl1c_ethtool.c | 54 +++- 1 files changed, 30 insertions(+), 24 deletions(-)

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

On Fri, 11 Nov 2016, Florian Fainelli wrote: > Le 11/11/2016 à 17:49, Nicolas Pitre a écrit : > > On Fri, 11 Nov 2016, 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.

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

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 +++-

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

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

[PATCH] ppdev: check before attaching port

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

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

[PATCH 8/9] x86/efi: Retrieve and assign Apple device properties

From: Lukas Wunner Apple's EFI drivers supply device properties which are needed to support Macs optimally. They contain vital information which cannot be obtained any other way (e.g. Thunderbolt Device ROM). They're also used to convey the current device state so that OS

[PATCH 3/9] efi: Add support for seeding the RNG from a UEFI config table

From: Ard Biesheuvel Specify a Linux specific UEFI configuration table that carries some random bits, and use the contents during early boot to seed the kernel's random number generator. This allows much strong random numbers to be generated early on. The entropy is

[PATCH 7/9] efi: Allow bitness-agnostic protocol calls

From: Lukas Wunner We already have a macro to invoke boot services which on x86 adapts automatically to the bitness of the EFI firmware: efi_call_early(). The macro allows sharing of functions across arches and bitness variants as long as those functions only call boot

[GIT PULL 0/9] EFI changes for v4.10

Folks, please pull the following v4.10 material. There isn't a huge amount of stuff here. The biggest change is the EFI dev path parser code from Lukas to get thunderbolt working on his macbook. [ The thunderbolt patch has been ACK'd by Andreas and given the OK to take it through the EFI tree

kvm: WARNING in em_ret_far

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

Re: [PATCH 2/2] staging: iio: ad7606: move out of staging

On 11/11/16 14:22, Lars-Peter Clausen wrote: > On 11/11/2016 07:34 AM, Eva Rachel Retuya wrote: >> Move the ad7606 driver from staging/iio/adc to iio/adc. Also, update the >> corresponding Makefile and Kconfig associated with the change. > > This is obviously OK, but when you generate a patch

Re: [srcu] Can we suppress sparse warning?

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? Sorry, I wrote this code for only showing warning message. This

Re: [PATCH v3 03/28] staging: iio: tsl2583: check if chip is suspended in in_illuminance_calibrate_store

On 10/11/16 09:25, Brian Masney wrote: > in_illuminance_calibrate_store() did not check to see if the chip is > suspended. 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

[PATCH] scsi: megaraid_sas: add in missing white spaces in error messages text

From: Colin Ian King A couple of dev_printk messages spans two lines and the literal string is missing a white space between words. Add the white space. Signed-off-by: Colin Ian King --- drivers/scsi/megaraid/megaraid_sas_base.c | 6 ++

Re: [PATCH v3 21/28] staging: iio: tsl2583: move from a global to a per device lux table

On 10/11/16 09:25, Brian Masney wrote: > 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

Re: [PATCH v3 16/28] staging: iio: tsl2583: updated code comment to match what the code does

On Sat, Nov 12, 2016 at 04:36:37PM +, Jonathan Cameron wrote: > On 10/11/16 09:25, Brian Masney wrote: > > 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

[PATCH] soc/fsl: fix spelling mistakes in critical error messages

From: Colin Ian King Trivial fix to spelling mistake "uncommited" to "uncommitted" in critical error messages. Signed-off-by: Colin Ian King --- drivers/soc/fsl/qbman/bman.c | 2 +- drivers/soc/fsl/qbman/qman.c | 2 +- 2 files changed, 2

[PATCH] [STYLE 5/5]staging:speakup:speakup_soft.c Blank lines

Made suggested modifications from checkpatch in reference to: CHECK: Please don't use multiple blank lines CHECK: Blank lines aren't necessary after an open brace '{' Signed-off-by: Walt Feasel --- drivers/staging/speakup/speakup_soft.c | 2 -- 1 file changed, 2

[PATCH] amdkfd: fix spelling mistake in kfd_ioctl_dbg_unrgesiter

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

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

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

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

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

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

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()

Looks fine, Reviewed-by: Christoph Hellwig

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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 |

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

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(-)

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

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()

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

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

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

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

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

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

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.

  1   2   3   4   5   6   7   >