Re: [PATCH v2 00/11] getting back -Wmaybe-uninitialized

On 11/11/16 19:49, Arnd Bergmann wrote: > On Friday, November 11, 2016 9:13:00 AM CET Linus Torvalds wrote: >> On Thu, Nov 10, 2016 at 8:44 AM, Arnd Bergmann wrote: >>> >>> Please merge these directly if you are happy with the result. >> >> I will take this. > > Thanks a lot! >

Re: Children Under Miserable Condition

Good Day, As an Aging widow with no child, suffering from long time incurable illness; presently admitted in a private hospital. I decided to give away my estate to the vulnerable through a sincere person. This is because at this point, life seems miserable for there is nobody to inherit my

Re: PM regression with LED changes in next-20161109

Hi, On 11-11-16 23:12, Pavel Machek wrote: Hi! Reason #1: Hmm. So userland can read the LED state, and it can get _some_ value back, but it can not know if it is current state or not. That is not correct, the current behavior for eading the brightness atrribute is to always return the

[PATCH 1/1 linux-next] ALSA: opl3: don't opencode IS_REACHABLE()

Signed-off-by: Fabian Frederick --- sound/drivers/opl3/opl3_lib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/drivers/opl3/opl3_lib.c b/sound/drivers/opl3/opl3_lib.c index 369cef2..cd9e9f3 100644 --- a/sound/drivers/opl3/opl3_lib.c +++

[PATCH 1/1 linux-next] ALSA: rawmidi: don't opencode IS_REACHABLE()

Signed-off-by: Fabian Frederick --- sound/core/rawmidi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/core/rawmidi.c b/sound/core/rawmidi.c index b450a27..2096bb0 100644 --- a/sound/core/rawmidi.c +++ b/sound/core/rawmidi.c @@ -1610,7 +1610,7 @@ static int

[PATCH 1/1 linux-next] ALSA: opl4: don't opencode IS_REACHABLE()

Signed-off-by: Fabian Frederick --- sound/drivers/opl4/opl4_lib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/drivers/opl4/opl4_lib.c b/sound/drivers/opl4/opl4_lib.c index 89c7aa0..240656e 100644 --- a/sound/drivers/opl4/opl4_lib.c +++

[PATCH 1/1 linux-next] ALSA: oss: don't opencode IS_REACHABLE()

Signed-off-by: Fabian Frederick --- sound/core/oss/pcm_oss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/core/oss/pcm_oss.c b/sound/core/oss/pcm_oss.c index ebc9fdf..698a014 100644 --- a/sound/core/oss/pcm_oss.c +++ b/sound/core/oss/pcm_oss.c @@ -2501,7 +2501,7 @@

[PATCH 1/1 linux-next] ALSA: pci: don't opencode IS_REACHABLE()

Signed-off-by: Fabian Frederick --- sound/pci/au88x0/au88x0_game.c | 2 +- sound/pci/azt3328.c | 2 +- sound/pci/cs4281.c | 2 +- sound/pci/cs46xx/cs46xx_lib.c| 2 +- sound/pci/emu10k1/emu10k1.c | 2 +- sound/pci/trident/trident_main.c | 2 +- 6 files

[RFC 1/1 linux-next] ALSA: pci: declare SUPPORT_JOYSTICK in gameport.h

Currently, all gameport callsites declare SUPPORT_JOYSTICK under opencoded IS_REACHABLE(). This patch moves definition in gameport.h and removes opencoding. Signed-off-by: Fabian Frederick --- This is untested include/linux/gameport.h| 3 ++- sound/pci/als4000.c | 4

[PATCH 1/1 linux-next] drm/nouveau: don't opencode IS_REACHABLE()

Signed-off-by: Fabian Frederick --- drivers/gpu/drm/nouveau/nouveau_hwmon.c | 6 +++--- drivers/gpu/drm/nouveau/nvkm/subdev/pci/agp.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_hwmon.c b/drivers/gpu/drm/nouveau/nouveau_hwmon.c

[PATCH 1/1 linux-next] usb: isp1301-omap: don't opencode IS_REACHABLE()

Signed-off-by: Fabian Frederick --- drivers/usb/phy/phy-isp1301-omap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy-isp1301-omap.c b/drivers/usb/phy/phy-isp1301-omap.c index 8d111ec..042c5a8 100644 --- a/drivers/usb/phy/phy-isp1301-omap.c +++

Re: [RFC 1/1 linux-next] ALSA: pci: declare SUPPORT_JOYSTICK in gameport.h

On Sat, 12 Nov 2016 09:44:05 +0100, Fabian Frederick wrote: > > Currently, all gameport callsites declare SUPPORT_JOYSTICK > under opencoded IS_REACHABLE(). > This patch moves definition in gameport.h and removes opencoding. Not all callers. azt3328 defines SUPPORT_GAMEPORT, for example.

Re: [PATCH 1/1 linux-next] ALSA: opl3: don't opencode IS_REACHABLE()

On Sat, 12 Nov 2016 09:28:33 +0100, Fabian Frederick wrote: > > Signed-off-by: Fabian Frederick Applied, thanks. Takashi > --- > sound/drivers/opl3/opl3_lib.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sound/drivers/opl3/opl3_lib.c

Re: [PATCH 1/1 linux-next] ALSA: rawmidi: don't opencode IS_REACHABLE()

On Sat, 12 Nov 2016 09:34:49 +0100, Fabian Frederick wrote: > > Signed-off-by: Fabian Frederick Applied, thanks. Takashi > --- > sound/core/rawmidi.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/sound/core/rawmidi.c b/sound/core/rawmidi.c > index

Re: [PATCH 1/1 linux-next] ALSA: oss: don't opencode IS_REACHABLE()

On Sat, 12 Nov 2016 09:36:01 +0100, Fabian Frederick wrote: > > Signed-off-by: Fabian Frederick Applied, thanks. Takashi > --- > sound/core/oss/pcm_oss.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sound/core/oss/pcm_oss.c b/sound/core/oss/pcm_oss.c > index

Re: [PATCH 1/1 linux-next] ALSA: pci: don't opencode IS_REACHABLE()

On Sat, 12 Nov 2016 09:40:53 +0100, Fabian Frederick wrote: > > Signed-off-by: Fabian Frederick This one is postponed. Please resubmit a patch covering all in sound/pci/*, as I mentioned in the reply to your RFC patch. thanks, Takashi > --- > sound/pci/au88x0/au88x0_game.c | 2 +- >

Re: [PATCH 0/6 v7] sched: reflect sched_entity move into task_group's load

On 10 November 2016 at 18:04, Dietmar Eggemann wrote: > On 08/11/16 09:53, Vincent Guittot wrote: >> Ensure that the move of a sched_entity will be reflected in load and >> utilization of the task_group hierarchy. >> >> When a sched_entity moves between groups or CPUs, load and utilization >> of

I Hope You Get My Message This Time

-- This is the second time i am sending you this mail. I, Friedrich Mayrhofer Donate $ 1,000,000.00 to You, Email Me personally for more details. Regards. Friedrich Mayrhofer

[PATCH v2 2/2] arm64: dts: Add ARM PMU node for exynos7

This patch adds ARM Performance Monitor Unit dt node for exynos7. PMU provides various statistics on the operation of the CPU and memory system at runtime, which are very useful when debugging or profiling code. This enables the same. Signed-off-by: Alim Akhtar ---

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

This patch adds level for cpu dt node, so that these levels can be used as a phandle whenever required. For example, adding a "interrupt-affinity" for arm pmu node. Signed-off-by: Alim Akhtar --- arch/arm64/boot/dts/exynos/exynos7.dtsi |8 1 file changed, 4 insertions(+), 4

Re: PM regression with LED changes in next-20161109

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: Hi, On 11/10/2016 09:29 PM, Pavel Machek wrote: On Thu 2016-11-10 10:55:37, Tony Lindgren wrote:

Re: kvm: GPF in __get_kvmclock_ns

This one is already known - see "[PATCH] KVM: x86: do not go through vcpu in __get_kvmclock_ns" which I posted yesterday. Paolo - Original Message - > From: "Dmitry Vyukov" > To: "Paolo Bonzini" , rkrc...@redhat.com, "Thomas > Gleixner" , "Ingo Molnar" > , "H. Peter Anvin" ,

[GIT PULL] KVM updates for 4.9-rc5

Linus, The following changes since commit bc33b0ca11e3df46a4fa7639ba488c9d4911: Linux 4.9-rc4 (2016-11-05 16:23:36 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/virt/kvm/kvm.git tags/for-linus for you to fetch changes up to

[PATCH] phy: twl4030-usb: make driver DT only

All users are DT-only and it makes no sense to keep unused code Signed-off-by: Nicolae Rosia --- drivers/phy/Kconfig | 1 + drivers/phy/phy-twl4030-usb.c | 22 +++--- 2 files changed, 8 insertions(+), 15 deletions(-) diff --git a/drivers/phy/Kconfig

Re: PM regression with LED changes in next-20161109

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: Hi, On 11/10/2016 09:29 PM, Pavel Machek wrote:

Re: [GIT PULL 0/5] perf/urgent hists hierarchy fixes

* Arnaldo Carvalho de Melo wrote: > Hi Ingo, > > Please consider pulling these 'perf report/top --hierarchy' fixes into > perf/urgent, they don't fix segfaults, just some oddities in scrolling, etc. > > If you think it is too late in the 4.9-rc window, then please pull it >

Re: [PATCH -v4 1/8] locking/drm: Kill mutex trickery

* Peter Zijlstra wrote: > On Fri, Nov 11, 2016 at 12:22:02PM +0100, Daniel Vetter wrote: > > > Once all your locking rework is assembled it might be good to have a > > topic branch I could pull in. Both for testing and to handle conflicts > > before it goes boom in the merge window ;-) Not

Re: [RFC PATCH] x86/debug: Dump more detailed segfault info

* Borislav Petkov wrote: > From: Borislav Petkov > > I found out recently that this is very helpful when trying to look at > opcodes around the rIP when the segfault happens, and also poke at > architectural registers. When enabled, it looks something like this: > > strsep[3702]: segfault

Re: [PATCH] timers: Fix timer inaccuracy

On Fri, 11 Nov 2016, Saravana Kannan wrote: > On 11/10/2016 02:07 AM, Thomas Gleixner wrote: > > Deferrable timers shouldn't have been invented in the first place and yes, > > they are not going to happen on hrtimers, quite the contrary, I'm working > > on eliminating them completely. > > If you

Re: [PATCH v4 1/4] usb: dbc: early driver for xhci debug capability

On Sat, 12 Nov 2016, Lu Baolu wrote: > On 11/11/2016 08:28 PM, Peter Zijlstra wrote: > > Again, a UART rules. Make a virtual UART in hardware, that'd be totally > > awesome. This thing, I'm not convinced its worth having. > > This is the initial work. It helps at least in cases where people need

Re: [PATCH 1/3] clk: Hi6220: enable stub clock driver for ARCH_HISI

Hi Stephen, On Wed, Aug 31, 2016 at 03:40:49PM -0700, Stephen Boyd wrote: > On 08/31, Leo Yan wrote: > > In current kernel config 'CONFIG_STUB_CLK_HI6220' is disabled by > > default, as result stub clock driver has not been registered and > > CPUFreq driver cannot work. > > > > This patch is to

[PATCH] perf tools: Remove jvmti/Makefile file

On Fri, Nov 11, 2016 at 03:18:03PM -0800, Stephane Eranian wrote: > Jiri, > > On Wed, Nov 2, 2016 at 6:35 AM, Jiri Olsa wrote: > > hi, > > adding jvmti compilation under perf build, so it's easy > > to put it under rpm. > > > > I plan on updating fedora/rhel perf rpm, to have the jvmti > >

Re: [PATCH 2/3] thermal: hisilicon: fix for dependency

Hi Rui, Eduardo, On Wed, Aug 31, 2016 at 04:50:16PM +0800, Leo Yan wrote: > The thermal driver is standalone driver which is used to enable > thermal sensors, so it can be used with any cooling device and > should not bind with CPU cooling device driver. > > This original patch is suggested by

Re: sched/autogroup: race if !sysctl_sched_autogroup_enabled ?

On Wed, 2016-11-09 at 18:50 +0100, Peter Zijlstra wrote: > On Wed, Nov 09, 2016 at 05:59:33PM +0100, Oleg Nesterov wrote: > > > We need to ensure that autogroup/tg returned by autogroup_task_group() > > can't go away if we race with autogroup_move_group(), and unless the > > caller holds

Re: [RFC PATCH] x86/debug: Dump more detailed segfault info

On Sat, Nov 12, 2016 at 12:06:49PM +0100, Ingo Molnar wrote: > Note that on recent kernels, with printk log timestamping enabled, this looks > like: > > [ 206.721243] CR2: CR3: 00042ab75000 CR4: > 001406e0 > [ 206.729217] Code: > [ 206.731271] 55 > [

[PATCH 2/4] regulator: twl: kill unused functions

This code was used by OMAP platform based boards which are now DT only. Proper support for SMPS is missing in this driver. Signed-off-by: Nicolae Rosia --- drivers/regulator/twl-regulator.c | 29 ++--- 1 file changed, 2 insertions(+), 27 deletions(-) diff --git

[PATCH 1/4] regulator: twl: make driver DT only

All users are DT only, remove unused code. Signed-off-by: Nicolae Rosia --- drivers/regulator/twl-regulator.c | 37 ++--- 1 file changed, 6 insertions(+), 31 deletions(-) diff --git a/drivers/regulator/twl-regulator.c b/drivers/regulator/twl-regulator.c index

[PATCH 0/4] regulator: twl: preparation for TWL6032 support

Hello, I'm planning on adding TWL6032 support but working with the existing twl regulator driver is hard because of a lot of legacy code. This series removes unused code and splits up the TWL6030 logic from TWL4030 so it is easier to build on. I'm keeping the same name for TWL4030

[PATCH 4/4] regulator: twl: split twl6030 logic into its own file

In order to not break existing users, we keep using the same CONFIG symbol. This makes it easier to add support for TWL6032 and refactor mfd/twl-core. Checkpatch warnings are inherited from twl-regulator.c and will be addressed in a subsequent patch. Signed-off-by: Nicolae Rosia --- MAINTAINERS

[PATCH 3/4] regulator: twl-regulator: rework fixed regulator definition

TWL603X and TWL4030 are different and have different code logic. Rework the regulator definition method so we can split the file easily in twl4030 and twl6030. Signed-off-by: Nicolae Rosia --- drivers/regulator/twl-regulator.c | 38 -- 1 file changed, 24

Re: Children Under Miserable Condition

Good Day, As an Aging widow with no child, suffering from long time incurable illness; presently admitted in a private hospital. I decided to give away my estate to the vulnerable through a sincere person. This is because at this point, life seems miserable for there is nobody to inherit my

Re: [PATCH v6 3/3] x86/kvm: Add AVX512_4VNNIW and AVX512_4FMAPS support

On Fri, Nov 11, 2016 at 05:25:36PM +0800, He Chen wrote: > Add two new AVX512 subfeatures support for KVM guest. > > AVX512_4VNNIW: > Vector instructions for deep learning enhanced word variable precision. > > AVX512_4FMAPS: > Vector instructions for deep learning floating-point single

Re: [PATCH net 2/2] r8152: rx descriptor check

On 16-11-11 07:13 AM, Francois Romieu wrote: > Hayes Wang : >> For some platforms, the data in memory is not the same with the one >> from the device. That is, the data of memory is unbelievable. The >> check is used to find out this situation. > > Invalid packet size corrupted receive

Re: [PATCH v2 00/11] getting back -Wmaybe-uninitialized

On 11/11/16 19:49, Arnd Bergmann wrote: > On Friday, November 11, 2016 9:13:00 AM CET Linus Torvalds wrote: >> On Thu, Nov 10, 2016 at 8:44 AM, Arnd Bergmann wrote: >>> >>> Please merge these directly if you are happy with the result. >> >> I will take this. > > Thanks a lot! > >> I do see two

Re: Children Under Miserable Condition

Good Day, As an Aging widow with no child, suffering from long time incurable illness; presently admitted in a private hospital. I decided to give away my estate to the vulnerable through a sincere person. This is because at this point, life seems miserable for there is nobody to inherit my

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

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

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 >> light sensor, battery voltage monitoring or any general

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

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 s/level/label here? I'm asking because you are using level consistently in the subject line and commit message but I'm not sure what it means in this

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

On 11/11/16 14:18, Lars-Peter Clausen wrote: > On 11/11/2016 07:34 AM, Eva Rachel Retuya wrote: >> Eliminate the non-standard attribute in_voltage_range and move its >> functionality under the scale attribute. read_raw() has been taken care >> of previously so only write_raw() is handled here. >>

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 2/2] staging: iio: ad7606: move out of staging

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 > >> corresponding Makefile and Kconfig associated

Re: [PATCH v2 2/2] arm64: dts: Add ARM PMU node for exynos7

Hello Alim, On 11/12/2016 07:17 AM, Alim Akhtar wrote: > This patch adds ARM Performance Monitor Unit dt node for exynos7. > PMU provides various statistics on the operation of the CPU and > memory system at runtime, which are very useful when debugging or > profiling code. This enables the same.

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 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 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 the less than zero compare >> will correctly detect

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 this). i agree with the intention of this

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

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 ipv6_stub->ipv6_dst_lookup, which seems ill-behaved: Userspace:

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

On 11/12/2016 8:34 AM, Marion & Christophe JAILLET wrote: > 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

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

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 s/level/label here? I'm asking because you are using level >

Re: [PATCH 0/6] Add basic support for support for Rockchip RK1108 SOC

patches 1-5 on rk1108-cvr-v10 board. Tested-by: Jacob Chen 2016-11-03 20:26 GMT+08:00 Andy Yan : > > RK1108 is embedded with an ARM Cortex-A7 single core and a DSP core. > It is designed for varies application scenario such as car DVR, sports > DV, secure camera and UAV camera. > This patch

Re: [PATCH v3 01/28] staging: iio: tsl2583: split out functionality of taos_chip_on()

On 10/11/16 09:25, Brian Masney wrote: > taos_chip_on() reads an eight member array called taos_config > that contains the desired state of the chip's registers. Only four > of the registers actually need to be written to. The four that do > not need to be written to are for the {low,high} byte of

Re: [PATCH v3 02/28] staging: iio: tsl2583: fix issue with changes to calibscale and int_time not being set on the chip

On 10/11/16 09:25, Brian Masney wrote: > When updating the in_illuminance_calibscale and > in_illuminance_integration_time sysfs attributes, these values were not > actually written to the chip. The chip would continue to use the old > parameters. Extracted out tsl2583_set_als_gain() and >

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 ++ 1 file changed, 2 insertions(+), 4 deletions(-)

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] ASoC: mioa701_wm9713: add missing white space in dev_err message

From: Colin Ian King There is a missing whitespace in the dev_err message between "will" and "lead". Add the whitespace. Signed-off-by: Colin Ian King --- sound/soc/pxa/mioa701_wm9713.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/pxa/mioa701_wm9713.c

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

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] 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(-) diff --git

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

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 v3 20/28] staging: iio: tsl2583: don't assume an unsigned int is 32 bits

On 10/11/16 09:25, Brian Masney wrote: > in_illuminance_lux_table_store assumes that an unsigned int is 32 bits. > Replace this with sizeof(unsigned int). > > Signed-off-by: Brian Masney Preferred to use sizeof(value[0]) in case it changes type sometime in the future? > --- >

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 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. Thanks, Jonathan > ---

[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 --git a/drivers/scsi/isci/remote_node_context.c

[PATCH] [STYLE 1/5]staging:speakup:speakup_soft.c Modify block text

Modified multiline comment to single Signed-off-by: Walt Feasel --- drivers/staging/speakup/speakup_soft.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/speakup/speakup_soft.c b/drivers/staging/speakup/speakup_soft.c index 6b1d0f5..c14a136 100644 ---

[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

[PATCH] [STYLE 3/5]staging:speakup:speakup_soft.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/speakup_soft.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/speakup_soft.c

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 insertions(+), 2 deletions(-) diff --git

[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 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 + > drivers/iio/adc/Makefile

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

On 12/11/16 16:59, Brian Masney wrote: > 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

Re: [PATCH v4 0/8] IIO wrapper drivers, dpot-dac and envelope-detector

On 08/11/16 11:58, Peter Rosin wrote: > Hi! > > This is a respin with a handful of nitpicks fixed from v3. No major > changes. And a couple of acks was added too, thanks! I also added > Thomas Gleixner as Cc, since Jonathan was hoping for some comments > on the somewhat odd interrupt handling in

[PATCH] [STYLE 4/5]staging:speakup:speakup_soft.c Logical continuations

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/speakup_soft.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

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. > > It is

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

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

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 a/Documentation/devicetree/bindings/vendor-prefixes.txt >

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

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

Re: [PATCH v4 6/8] iio: dpot-dac: DAC driver based on a digital potentiometer

On 08/11/16 11:58, Peter Rosin wrote: > It is assumed that the dpot is used as a voltage divider between the > current dpot wiper setting and the maximum resistance of the dpot. The > divided voltage is provided by a vref regulator. > > .--. >.---. | | >

Re: [PATCH v4 7/8] dt-bindings: iio: document envelope-detector bindings

On 08/11/16 11:58, Peter Rosin wrote: > Acked-by: Rob Herring > Signed-off-by: Peter Rosin Applied. > --- > .../bindings/iio/adc/envelope-detector.txt | 54 > ++ > MAINTAINERS| 6 +++ > 2 files changed, 60 insertions(+) >

<    1   2   3   4   5   6   7   >