Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

On Fri, Sep 23, 2016 at 08:42:51AM -0400, Mike Snitzer wrote: > On Fri, Sep 23 2016 at 8:26am -0400, > Peter Zijlstra wrote: > > > On Fri, Sep 23, 2016 at 02:17:10PM +0200, Mike Galbraith wrote: > > > On Fri, 2016-09-23 at 10:00 +0200, Thomas Gleixner wrote: > > > > On Fri, 23 Sep 2016, Peter

Re: taint/module: Clean up global and module taint flags handling

On Fri, 23 Sep 2016, Jessica Yu wrote: > Hm, quick question, which tree would this patch go to? Though the > cleanup is for modules, there is an indirect cross-tree dependency > (taint_flag.module needs to be true for TAINT_LIVEPATCH for Josh's patch > to still work as intended). The least

[PATCH v3 4/8] drivers:input:tsc2007: check for presence and power down tsc2007 during probe

1. check if chip is really present and don't succeed if it isn't. 2. if it succeeds, power down the chip until accessed Signed-off-by: H. Nikolaus Schaller --- drivers/input/touchscreen/tsc2007.c | 9 + 1 file changed, 9 insertions(+) diff --git

[PATCH v3 4/8] drivers:input:tsc2007: check for presence and power down tsc2007 during probe

1. check if chip is really present and don't succeed if it isn't. 2. if it succeeds, power down the chip until accessed Signed-off-by: H. Nikolaus Schaller --- drivers/input/touchscreen/tsc2007.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/input/touchscreen/tsc2007.c

Re: [PATCH v2 07/14] of/platform: Probe "isa" busses by default

On Mon, Sep 19, 2016 at 4:21 PM, Paul Burton wrote: > Since commit 44a7185c2ae6 ("of/platform: Add common method to populate > default bus") platforms calling of_platform_bus_probe from an initcall > is either a rather unsafe race with of_platform_default_populate_init or

Re: [PATCH v2 07/14] of/platform: Probe "isa" busses by default

On Mon, Sep 19, 2016 at 4:21 PM, Paul Burton wrote: > Since commit 44a7185c2ae6 ("of/platform: Add common method to populate > default bus") platforms calling of_platform_bus_probe from an initcall > is either a rather unsafe race with of_platform_default_populate_init or > a no-op. The MIPS

[PATCH v3 2/8] drivers:input:tsc2007: send pendown and penup only once like ads7846(+tsc2046) driver does

this should reduce unnecessary input events. Signed-off-by: H. Nikolaus Schaller --- drivers/input/touchscreen/tsc2007.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/input/touchscreen/tsc2007.c b/drivers/input/touchscreen/tsc2007.c

[PATCH v3 7/8] drivers:input:ads7846(+tsc2046): fix spi module table

Fix module table so that the driver is loaded if compiled as module and requested by DT. Signed-off-by: H. Nikolaus Schaller --- drivers/input/touchscreen/ads7846.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/input/touchscreen/ads7846.c

[PATCH v3 0/8] drivers: touchscreen: tsc2007 and ads7846/tsc2046 improvements (use common touchscreen bindings, pre-calibration, spi fix and provide iio raw values)

It was quite a while ago that this patch set was in review, but here is a new version for consideration. Changes V3: * fix an issue with swapping * remove hard clipping to min/max rectangle - some systems expect to handle negative coordinates * make use of commit ed7c9870c9bc ("Input:

[PATCH v3 2/8] drivers:input:tsc2007: send pendown and penup only once like ads7846(+tsc2046) driver does

this should reduce unnecessary input events. Signed-off-by: H. Nikolaus Schaller --- drivers/input/touchscreen/tsc2007.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/input/touchscreen/tsc2007.c b/drivers/input/touchscreen/tsc2007.c index

[PATCH v3 7/8] drivers:input:ads7846(+tsc2046): fix spi module table

Fix module table so that the driver is loaded if compiled as module and requested by DT. Signed-off-by: H. Nikolaus Schaller --- drivers/input/touchscreen/ads7846.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/input/touchscreen/ads7846.c

[PATCH v3 0/8] drivers: touchscreen: tsc2007 and ads7846/tsc2046 improvements (use common touchscreen bindings, pre-calibration, spi fix and provide iio raw values)

It was quite a while ago that this patch set was in review, but here is a new version for consideration. Changes V3: * fix an issue with swapping * remove hard clipping to min/max rectangle - some systems expect to handle negative coordinates * make use of commit ed7c9870c9bc ("Input:

Re: [PATCH 1/1] lib/ioremap.c: avoid endless loop under ioremapping page unaligned ranges

On Fri 23-09-16 20:29:20, zijun_hu wrote: > On 2016/9/23 16:45, Michal Hocko wrote: > > On Thu 22-09-16 23:13:17, zijun_hu wrote: > >> On 2016/9/22 20:47, Michal Hocko wrote: > >>> On Wed 21-09-16 12:19:53, zijun_hu wrote: > From: zijun_hu > > endless loop maybe

[PATCH v3 6/8] drivers:input:ads7846(+tsc2046): add new common binding names, pre-calibration and flipping

commit b98abe52fa8e ("Input: add common DT binding for touchscreens") introduced common DT bindings for touchscreens [1] and a helper function to parse the DT. commit ed7c9870c9bc ("Input: of_touchscreen - add support for inverted / swapped axes") added another helper for parsing axis inversion

Re: [PATCH v3 0/2] Exynos IOMMU: proper runtime PM support (use device dependencies)

On Tuesday, September 20, 2016 10:51:13 AM Marek Szyprowski wrote: > Hi All, > > On 2016-09-19 23:45, Tobias Jakobi wrote: > > I did some tests with the new version today. Sadly the reboot/shutdown > > issues are still present. > > Thanks for the report. I've managed to reproduce this issue and

Re: [PATCH 1/1] lib/ioremap.c: avoid endless loop under ioremapping page unaligned ranges

On Fri 23-09-16 20:29:20, zijun_hu wrote: > On 2016/9/23 16:45, Michal Hocko wrote: > > On Thu 22-09-16 23:13:17, zijun_hu wrote: > >> On 2016/9/22 20:47, Michal Hocko wrote: > >>> On Wed 21-09-16 12:19:53, zijun_hu wrote: > From: zijun_hu > > endless loop maybe happen if either of

[PATCH v3 6/8] drivers:input:ads7846(+tsc2046): add new common binding names, pre-calibration and flipping

commit b98abe52fa8e ("Input: add common DT binding for touchscreens") introduced common DT bindings for touchscreens [1] and a helper function to parse the DT. commit ed7c9870c9bc ("Input: of_touchscreen - add support for inverted / swapped axes") added another helper for parsing axis inversion

Re: [PATCH v3 0/2] Exynos IOMMU: proper runtime PM support (use device dependencies)

On Tuesday, September 20, 2016 10:51:13 AM Marek Szyprowski wrote: > Hi All, > > On 2016-09-19 23:45, Tobias Jakobi wrote: > > I did some tests with the new version today. Sadly the reboot/shutdown > > issues are still present. > > Thanks for the report. I've managed to reproduce this issue and

Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

On Fri, Sep 23 2016 at 8:26am -0400, Peter Zijlstra wrote: > On Fri, Sep 23, 2016 at 02:17:10PM +0200, Mike Galbraith wrote: > > On Fri, 2016-09-23 at 10:00 +0200, Thomas Gleixner wrote: > > > On Fri, 23 Sep 2016, Peter Zijlstra wrote: > > > > > > Is anybody still using

[PATCH v3 5/8] DT:omap3+tsc2007: use new common touchscreen bindings

While we fix the GTA04 we add proper pinmux for the penirq gpio. Tested on: GTA04A4 and Pyra-Handheld Signed-off-by: H. Nikolaus Schaller --- arch/arm/boot/dts/omap3-gta04.dtsi | 25 +++-- 1 file changed, 23 insertions(+), 2 deletions(-) diff --git

[PATCH v3 1/8] drivers:input:tsc2007: add new common binding names, pre-calibration, flipping and rotation

commit b98abe52fa8e ("Input: add common DT binding for touchscreens") introduced common DT bindings for touchscreens [1] and a helper function to parse the DT. commit ed7c9870c9bc ("Input: of_touchscreen - add support for inverted / swapped axes") added another helper for parsing axis inversion

[PATCH v3 8/8] DT:omap3+ads7846: use new common touchscreen bindings

The standard touch screen bindings [1] replace the private ti,swap-xy with touchscreen-swaped-x-y. And for the Openpandora we use touchscreen-size etc. to match the LCD screen size. [1]: Documentation/devicetree/bindings/input/touchscreen/touchscreen.txt Tested with OpenPandora. Signed-off-by:

Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

On Fri, Sep 23 2016 at 8:26am -0400, Peter Zijlstra wrote: > On Fri, Sep 23, 2016 at 02:17:10PM +0200, Mike Galbraith wrote: > > On Fri, 2016-09-23 at 10:00 +0200, Thomas Gleixner wrote: > > > On Fri, 23 Sep 2016, Peter Zijlstra wrote: > > > > > > Is anybody still using PREEMPT_NONE? Most

[PATCH v3 5/8] DT:omap3+tsc2007: use new common touchscreen bindings

While we fix the GTA04 we add proper pinmux for the penirq gpio. Tested on: GTA04A4 and Pyra-Handheld Signed-off-by: H. Nikolaus Schaller --- arch/arm/boot/dts/omap3-gta04.dtsi | 25 +++-- 1 file changed, 23 insertions(+), 2 deletions(-) diff --git

[PATCH v3 1/8] drivers:input:tsc2007: add new common binding names, pre-calibration, flipping and rotation

commit b98abe52fa8e ("Input: add common DT binding for touchscreens") introduced common DT bindings for touchscreens [1] and a helper function to parse the DT. commit ed7c9870c9bc ("Input: of_touchscreen - add support for inverted / swapped axes") added another helper for parsing axis inversion

[PATCH v3 8/8] DT:omap3+ads7846: use new common touchscreen bindings

The standard touch screen bindings [1] replace the private ti,swap-xy with touchscreen-swaped-x-y. And for the Openpandora we use touchscreen-size etc. to match the LCD screen size. [1]: Documentation/devicetree/bindings/input/touchscreen/touchscreen.txt Tested with OpenPandora. Signed-off-by:

[PATCH v3 3/8] drivers:input:tsc2007: add iio interface to read external ADC input, temperature and raw conversion values

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 temperature. This extension provides an iio interface for

[PATCH v3 3/8] drivers:input:tsc2007: add iio interface to read external ADC input, temperature and raw conversion values

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 temperature. This extension provides an iio interface for

Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

On Fri, 2016-09-23 at 14:26 +0200, Peter Zijlstra wrote: > On Fri, Sep 23, 2016 at 02:17:10PM +0200, Mike Galbraith wrote: > > On Fri, 2016-09-23 at 10:00 +0200, Thomas Gleixner wrote: > > > On Fri, 23 Sep 2016, Peter Zijlstra wrote: > > > > > > Is anybody still using PREEMPT_NONE? Most workloads

Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

On Fri, 2016-09-23 at 14:26 +0200, Peter Zijlstra wrote: > On Fri, Sep 23, 2016 at 02:17:10PM +0200, Mike Galbraith wrote: > > On Fri, 2016-09-23 at 10:00 +0200, Thomas Gleixner wrote: > > > On Fri, 23 Sep 2016, Peter Zijlstra wrote: > > > > > > Is anybody still using PREEMPT_NONE? Most workloads

[PATCH V2]usb: dwc2: Clear GUSBCFG.UsbTrdTim before setting

The USBTRDTIM field needs to be cleared before setting a new value. Otherwise it will result in an incorrect value if phyif == GUSBCFG_PHYIF8. Change-Id: Ib3e33cf4fd15ada41dc070ff7b93858daafbd10f Signed-off-by: Pengcheng Li Acked-by: John Youn ---

[PATCH V2]usb: dwc2: Clear GUSBCFG.UsbTrdTim before setting

The USBTRDTIM field needs to be cleared before setting a new value. Otherwise it will result in an incorrect value if phyif == GUSBCFG_PHYIF8. Change-Id: Ib3e33cf4fd15ada41dc070ff7b93858daafbd10f Signed-off-by: Pengcheng Li Acked-by: John Youn --- drivers/usb/dwc2/gadget.c | 4 ++-- 1 file

Re: [PATCH V11 0/8] Introduce ACPI world to ITS irqchip

Hi, On 12/09/2016 20:32, Tomasz Nowicki wrote: > The series builds the PCI/MSI domain stack based on initial IORT driver > which is added in first place. As a reference please see IORT spec: > http://infocenter.arm.com/help/topic/com.arm.doc.den0049b/DEN0049B_IO_Remapping_Table.pdf > > Tested on

Re: [PATCH V11 0/8] Introduce ACPI world to ITS irqchip

Hi, On 12/09/2016 20:32, Tomasz Nowicki wrote: > The series builds the PCI/MSI domain stack based on initial IORT driver > which is added in first place. As a reference please see IORT spec: > http://infocenter.arm.com/help/topic/com.arm.doc.den0049b/DEN0049B_IO_Remapping_Table.pdf > > Tested on

Re: [PATCH 2/2] armv8: aarch32: Add SMP support for 32-bit Linux kernel

On Fri, Sep 23, 2016 at 10:19:06AM +0800, Alison Wang wrote: > The patch adds SMP support for running 32-bit Linux kernel for > Layerscape platforms. Spin-table method is used for SMP support. So far, spin-table has not been defined for 32-bit, and the code below, while mostly a copy of the

Re: [PATCH 2/2] armv8: aarch32: Add SMP support for 32-bit Linux kernel

On Fri, Sep 23, 2016 at 10:19:06AM +0800, Alison Wang wrote: > The patch adds SMP support for running 32-bit Linux kernel for > Layerscape platforms. Spin-table method is used for SMP support. So far, spin-table has not been defined for 32-bit, and the code below, while mostly a copy of the

Re: [PATCH 1/1] lib/ioremap.c: avoid endless loop under ioremapping page unaligned ranges

On 2016/9/23 16:45, Michal Hocko wrote: > On Thu 22-09-16 23:13:17, zijun_hu wrote: >> On 2016/9/22 20:47, Michal Hocko wrote: >>> On Wed 21-09-16 12:19:53, zijun_hu wrote: From: zijun_hu endless loop maybe happen if either of parameter addr and end is not

Re: [PATCH 1/1] lib/ioremap.c: avoid endless loop under ioremapping page unaligned ranges

On 2016/9/23 16:45, Michal Hocko wrote: > On Thu 22-09-16 23:13:17, zijun_hu wrote: >> On 2016/9/22 20:47, Michal Hocko wrote: >>> On Wed 21-09-16 12:19:53, zijun_hu wrote: From: zijun_hu endless loop maybe happen if either of parameter addr and end is not page aligned for

[PATCH] clk: change the type of clk_hw_onecell_data.num to unsigned int

The "num" is the number of clk_hw entries in the structure, so "unsigned int" would be a better fit. (size_t looks like data size we count by byte.) Besides, struct clk_onecell_data already uses unsigned int for "clk_num". Signed-off-by: Masahiro Yamada ---

[PATCH] clk: change the type of clk_hw_onecell_data.num to unsigned int

The "num" is the number of clk_hw entries in the structure, so "unsigned int" would be a better fit. (size_t looks like data size we count by byte.) Besides, struct clk_onecell_data already uses unsigned int for "clk_num". Signed-off-by: Masahiro Yamada --- include/linux/clk-provider.h | 2

Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

On Fri, Sep 23, 2016 at 02:17:10PM +0200, Mike Galbraith wrote: > On Fri, 2016-09-23 at 10:00 +0200, Thomas Gleixner wrote: > > On Fri, 23 Sep 2016, Peter Zijlstra wrote: > > > > Is anybody still using PREEMPT_NONE? Most workloads also care about > > > latency to some extend. Lots of code has

Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

On Fri, Sep 23, 2016 at 02:17:10PM +0200, Mike Galbraith wrote: > On Fri, 2016-09-23 at 10:00 +0200, Thomas Gleixner wrote: > > On Fri, 23 Sep 2016, Peter Zijlstra wrote: > > > > Is anybody still using PREEMPT_NONE? Most workloads also care about > > > latency to some extend. Lots of code has

Re: [PATCH] drm/i2c: tda998x: don't register the connector

Am Freitag, den 23.09.2016, 12:58 +0200 schrieb Daniel Vetter: > On Fri, Sep 23, 2016 at 11:34 AM, Liviu Dudau > wrote: > > > > rmmod-ing the hdlcd module generates a WARN() splat as the vsync is > > still > > enabled, but we never got the call to turn off the CRTC. Brian is

Re: [PATCH] drm/i2c: tda998x: don't register the connector

Am Freitag, den 23.09.2016, 12:58 +0200 schrieb Daniel Vetter: > On Fri, Sep 23, 2016 at 11:34 AM, Liviu Dudau > wrote: > > > > rmmod-ing the hdlcd module generates a WARN() splat as the vsync is > > still > > enabled, but we never got the call to turn off the CRTC. Brian is > > still > >

Re: [PATCH] cfq: fix starvation of asynchronous writes

On Fri, Sep 23, 2016 at 7:28 AM, Glauber Costa wrote: > On Sep 23, 2016 6:22 AM, "Paolo Valente" wrote: >> >> >> > Il giorno 23 set 2016, alle ore 02:59, Glauber Costa >> > ha scritto: >> > >> > While debugging timeouts

Re: [PATCH] cfq: fix starvation of asynchronous writes

On Fri, Sep 23, 2016 at 7:28 AM, Glauber Costa wrote: > On Sep 23, 2016 6:22 AM, "Paolo Valente" wrote: >> >> >> > Il giorno 23 set 2016, alle ore 02:59, Glauber Costa >> > ha scritto: >> > >> > While debugging timeouts happening in my application workload >> > (ScyllaDB), I have >> > observed

Re: [PATCHv3 5/5] staging:android:ion: Remove empty release function

On Fri, Sep 23, 2016 at 12:50:15PM +0300, Yannis Damigos wrote: > This patch removes the empty function rmem_ion_device_release > > Signed-off-by: Yannis Damigos > --- > drivers/staging/android/ion/ion_of.c | 7 --- > 1 file changed, 7 deletions(-) Please just

Re: [PATCHv3 5/5] staging:android:ion: Remove empty release function

On Fri, Sep 23, 2016 at 12:50:15PM +0300, Yannis Damigos wrote: > This patch removes the empty function rmem_ion_device_release > > Signed-off-by: Yannis Damigos > --- > drivers/staging/android/ion/ion_of.c | 7 --- > 1 file changed, 7 deletions(-) Please just resend the whole series for

Re: [PATCH 1/2] armv8: aarch32: Execute 32-bit Linux for LayerScape platforms

Hi Alison, On 23/09/16 03:19, Alison Wang wrote: > The ARMv8 architecture supports: > 1. 64-bit execution state, AArch64. > 2. 32-bit execution state, AArch32, that is compatible with previous > versions of the ARM architecture. > > LayerScape platforms are compliant with ARMv8 architecture.

Re: GPU-DRM-TILCDC: Less function calls in tilcdc_convert_slave_node() after error detection

>> I see a need to improve not only correctness there but also a bit of >> software efficiency. > > If you can measure any performance difference and present some results > (esp. considering that this is something that just happens when the > driver is loaded), then we'll talk. Are you really

Re: [PATCHv2 3/5] staging:android:ion: Fix blank line issues

On Fri, Sep 23, 2016 at 11:05:15AM +0300, giannis.dami...@gmail.com wrote: > From: Yannis Damigos > > This patch fixes the following blank line issues: > > CHECK: Please don't use multiple blank lines > + > + What does that mean with the + + lines? > > CHECK:

Re: [PATCH 1/2] armv8: aarch32: Execute 32-bit Linux for LayerScape platforms

Hi Alison, On 23/09/16 03:19, Alison Wang wrote: > The ARMv8 architecture supports: > 1. 64-bit execution state, AArch64. > 2. 32-bit execution state, AArch32, that is compatible with previous > versions of the ARM architecture. > > LayerScape platforms are compliant with ARMv8 architecture.

Re: GPU-DRM-TILCDC: Less function calls in tilcdc_convert_slave_node() after error detection

>> I see a need to improve not only correctness there but also a bit of >> software efficiency. > > If you can measure any performance difference and present some results > (esp. considering that this is something that just happens when the > driver is loaded), then we'll talk. Are you really

Re: [PATCHv2 3/5] staging:android:ion: Fix blank line issues

On Fri, Sep 23, 2016 at 11:05:15AM +0300, giannis.dami...@gmail.com wrote: > From: Yannis Damigos > > This patch fixes the following blank line issues: > > CHECK: Please don't use multiple blank lines > + > + What does that mean with the + + lines? > > CHECK: Please use a blank line after

Re: [PATCHv2 2/5] staging:android:ion: Fix whitespace issues

On Fri, Sep 23, 2016 at 11:05:14AM +0300, giannis.dami...@gmail.com wrote: > From: Yannis Damigos > > This patch fixes the following whitespace issues: > > CHECK: Alignment should match open parenthesis > +int ion_parse_dt_heap_common(struct device_node *heap_node, >

Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

On Fri, 2016-09-23 at 10:00 +0200, Thomas Gleixner wrote: > On Fri, 23 Sep 2016, Peter Zijlstra wrote: > > Is anybody still using PREEMPT_NONE? Most workloads also care about > > latency to some extend. Lots of code has explicit cond_resched() and > > doesn't worry. > > Dunno. But I bet there

Re: [PATCHv2 2/5] staging:android:ion: Fix whitespace issues

On Fri, Sep 23, 2016 at 11:05:14AM +0300, giannis.dami...@gmail.com wrote: > From: Yannis Damigos > > This patch fixes the following whitespace issues: > > CHECK: Alignment should match open parenthesis > +int ion_parse_dt_heap_common(struct device_node *heap_node, > +

Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

On Fri, 2016-09-23 at 10:00 +0200, Thomas Gleixner wrote: > On Fri, 23 Sep 2016, Peter Zijlstra wrote: > > Is anybody still using PREEMPT_NONE? Most workloads also care about > > latency to some extend. Lots of code has explicit cond_resched() and > > doesn't worry. > > Dunno. But I bet there

Re: [PATCH 01/11 V2] staging: dgnc: remove redundant initialization for channel array

On Fri, Sep 23, 2016 at 10:25:02AM +0900, Daeseok Youn wrote: > The channel array in board_t was initialized in dgnc_found_board() > with NULL. But the channel is going to initialize in dgnc_tty_init() > again. So the channel array doesn't need to set NULL > for initailization in

Re: [PATCH 01/11 V2] staging: dgnc: remove redundant initialization for channel array

On Fri, Sep 23, 2016 at 10:25:02AM +0900, Daeseok Youn wrote: > The channel array in board_t was initialized in dgnc_found_board() > with NULL. But the channel is going to initialize in dgnc_tty_init() > again. So the channel array doesn't need to set NULL > for initailization in

Re: [PATCH 0/8] staging: rts5208: Fix several style warnings

On Thu, Sep 22, 2016 at 08:18:53PM +0200, Sergio Paracuellos wrote: > This patchset fix several style warnings reported by checkpatch.pl script. > > Sergio Paracuellos (8): > staging: rts5208: fix style warnings in rtsx.h > staging: rts5208: fix style warnings in rtsx.c > staging: rts5208:

Re: [PATCH] mlxsw: spectrum: remove redundant check if err is zero

Fri, Sep 23, 2016 at 01:02:45PM CEST, colin.k...@canonical.com wrote: >From: Colin Ian King > >There is an earlier check and return if err is non-zero, so >the check to see if it is zero is redundant in every iteration >of the loop and hence the check can be removed. >

Re: [PATCH 0/8] staging: rts5208: Fix several style warnings

On Thu, Sep 22, 2016 at 08:18:53PM +0200, Sergio Paracuellos wrote: > This patchset fix several style warnings reported by checkpatch.pl script. > > Sergio Paracuellos (8): > staging: rts5208: fix style warnings in rtsx.h > staging: rts5208: fix style warnings in rtsx.c > staging: rts5208:

Re: [PATCH] mlxsw: spectrum: remove redundant check if err is zero

Fri, Sep 23, 2016 at 01:02:45PM CEST, colin.k...@canonical.com wrote: >From: Colin Ian King > >There is an earlier check and return if err is non-zero, so >the check to see if it is zero is redundant in every iteration >of the loop and hence the check can be removed. > >Signed-off-by: Colin Ian

Re: [RFC PATCH 1/6] perf: Move mlock accounting to ring buffer allocation

On Fri, Sep 23, 2016 at 02:27:21PM +0300, Alexander Shishkin wrote: > In order to be able to allocate perf ring buffers in non-mmap path, we > need to make sure we can still account the memory to the user and that > they don't exceed their mlock limit. > > This patch moves ring buffer memory

Re: [RFC PATCH 1/6] perf: Move mlock accounting to ring buffer allocation

On Fri, Sep 23, 2016 at 02:27:21PM +0300, Alexander Shishkin wrote: > In order to be able to allocate perf ring buffers in non-mmap path, we > need to make sure we can still account the memory to the user and that > they don't exceed their mlock limit. > > This patch moves ring buffer memory

Re: [PATCH] staging: sm750fb: rename getChipType to get_chip_type

On Thu, Sep 22, 2016 at 09:15:45PM +0300, Moshe Green wrote: > Rename CamelCased function getChipType to get_chip_type. > This issue was found by checkpatch.pl As this is a global function, can you rename it to something like "sm750_get_chip_type()"? Having a driver-specific function called

Re: [PATCH] staging: sm750fb: rename getChipType to get_chip_type

On Thu, Sep 22, 2016 at 09:15:45PM +0300, Moshe Green wrote: > Rename CamelCased function getChipType to get_chip_type. > This issue was found by checkpatch.pl As this is a global function, can you rename it to something like "sm750_get_chip_type()"? Having a driver-specific function called

Re: [PATCH] i2c: uniphier-f: fix misdetection of incomplete STOP condition

On Fri, Sep 23, 2016 at 09:04:01PM +0900, Masahiro Yamada wrote: > Currently, the status register FI2C_SR is checked immediately after > a STOP condition is issued in case of the deferred STOP condition. > It takes typically 5-10 usec until the corresponding bits in the > register are set, so the

Re: [PATCH] i2c: uniphier-f: fix misdetection of incomplete STOP condition

On Fri, Sep 23, 2016 at 09:04:01PM +0900, Masahiro Yamada wrote: > Currently, the status register FI2C_SR is checked immediately after > a STOP condition is issued in case of the deferred STOP condition. > It takes typically 5-10 usec until the corresponding bits in the > register are set, so the

Re: [PATCH 1/2] armv8: aarch32: Execute 32-bit Linux for LayerScape platforms

On Friday, September 23, 2016 10:19:05 AM CEST Alison Wang wrote: > The ARMv8 architecture supports: > 1. 64-bit execution state, AArch64. > 2. 32-bit execution state, AArch32, that is compatible with previous > versions of the ARM architecture. > > LayerScape platforms are compliant with ARMv8

Re: [PATCH 1/2] armv8: aarch32: Execute 32-bit Linux for LayerScape platforms

On Friday, September 23, 2016 10:19:05 AM CEST Alison Wang wrote: > The ARMv8 architecture supports: > 1. 64-bit execution state, AArch64. > 2. 32-bit execution state, AArch32, that is compatible with previous > versions of the ARM architecture. > > LayerScape platforms are compliant with ARMv8

Re: [PATCH 0/4] reintroduce compaction feedback for OOM decisions

On Fri 23-09-16 12:55:23, Vlastimil Babka wrote: > On 09/23/2016 10:26 AM, Michal Hocko wrote: > >> include/linux/compaction.h | 5 +++-- > >> mm/compaction.c| 44 > >> +++- > >> mm/internal.h | 1 + > >> mm/vmscan.c

Re: [PATCH 0/4] reintroduce compaction feedback for OOM decisions

On Fri 23-09-16 12:55:23, Vlastimil Babka wrote: > On 09/23/2016 10:26 AM, Michal Hocko wrote: > >> include/linux/compaction.h | 5 +++-- > >> mm/compaction.c| 44 > >> +++- > >> mm/internal.h | 1 + > >> mm/vmscan.c

Re: [PATCHv3 5/5] staging:android:ion: Remove empty release function

On Fri, Sep 23, 2016 at 12:50:15PM +0300, Yannis Damigos wrote: > This patch removes the empty function rmem_ion_device_release > > Signed-off-by: Yannis Damigos > --- > drivers/staging/android/ion/ion_of.c | 7 --- > 1 file changed, 7 deletions(-) > > diff --git

Re: [PATCHv3 5/5] staging:android:ion: Remove empty release function

On Fri, Sep 23, 2016 at 12:50:15PM +0300, Yannis Damigos wrote: > This patch removes the empty function rmem_ion_device_release > > Signed-off-by: Yannis Damigos > --- > drivers/staging/android/ion/ion_of.c | 7 --- > 1 file changed, 7 deletions(-) > > diff --git

Re: [PATCH 2/4] mm, compaction: more reliably increase direct compaction priority

On Fri 23-09-16 12:47:23, Vlastimil Babka wrote: > On 09/23/2016 10:23 AM, Michal Hocko wrote: > > On Fri 23-09-16 08:55:33, Vlastimil Babka wrote: > > [...] > >> >From 1623d5bd441160569ffad3808aeeec852048e558 Mon Sep 17 00:00:00 2001 > >> From: Vlastimil Babka > >> Date: Thu, 22

Re: [PATCH 2/4] mm, compaction: more reliably increase direct compaction priority

On Fri 23-09-16 12:47:23, Vlastimil Babka wrote: > On 09/23/2016 10:23 AM, Michal Hocko wrote: > > On Fri 23-09-16 08:55:33, Vlastimil Babka wrote: > > [...] > >> >From 1623d5bd441160569ffad3808aeeec852048e558 Mon Sep 17 00:00:00 2001 > >> From: Vlastimil Babka > >> Date: Thu, 22 Sep 2016

[GIT PULL] MTD NAND fixes for v4.8-rc8

Linus, This pull request contains fixes for bugs which got introduced in -rc1. You may wonder why I'm sending this pull request. Usually Brian takes NAND patches from Boris, but since Brian is very busy these days with other stuff and Boris is not yet member of the kernel.org web of trust I

[GIT PULL] MTD NAND fixes for v4.8-rc8

Linus, This pull request contains fixes for bugs which got introduced in -rc1. You may wonder why I'm sending this pull request. Usually Brian takes NAND patches from Boris, but since Brian is very busy these days with other stuff and Boris is not yet member of the kernel.org web of trust I

Re: [PATCH net-next 0/4] net: dsa: add port fast ageing

From: Vivien Didelot Date: Thu, 22 Sep 2016 16:49:20 -0400 > Today the DSA drivers are in charge of flushing the MAC addresses > associated to a port when its STP state changes from Learning or > Forwarding, to Disabled or Blocking or Listening. > > This

Re: [PATCH net-next 0/4] net: dsa: add port fast ageing

From: Vivien Didelot Date: Thu, 22 Sep 2016 16:49:20 -0400 > Today the DSA drivers are in charge of flushing the MAC addresses > associated to a port when its STP state changes from Learning or > Forwarding, to Disabled or Blocking or Listening. > > This makes the drivers more complex and hides

Re: [PATCH] drm/tilcdc: fix wrong error handling

On Fri, Sep 23, 2016 at 5:01 AM, Jyri Sarha wrote: > On 09/23/16 14:47, Sean Paul wrote: >> On Fri, Sep 23, 2016 at 3:52 AM, Daniel Schultz wrote: >>> When 'component_bind_all' fails it should not try to unbind components >>> in the error handling. This will

Re: [PATCH] drm/tilcdc: fix wrong error handling

On Fri, Sep 23, 2016 at 5:01 AM, Jyri Sarha wrote: > On 09/23/16 14:47, Sean Paul wrote: >> On Fri, Sep 23, 2016 at 3:52 AM, Daniel Schultz wrote: >>> When 'component_bind_all' fails it should not try to unbind components >>> in the error handling. This will produce a null pointer kernel panic

[PATCH] i2c: uniphier-f: fix misdetection of incomplete STOP condition

Currently, the status register FI2C_SR is checked immediately after a STOP condition is issued in case of the deferred STOP condition. It takes typically 5-10 usec until the corresponding bits in the register are set, so the error check for "stop condition was not completed" is very likely to be

[PATCH] i2c: uniphier-f: fix misdetection of incomplete STOP condition

Currently, the status register FI2C_SR is checked immediately after a STOP condition is issued in case of the deferred STOP condition. It takes typically 5-10 usec until the corresponding bits in the register are set, so the error check for "stop condition was not completed" is very likely to be

Re: cifs mount regression in 4.8 and 4.4 stable

On Thu, Sep 22, 2016 at 10:27:56AM -0500, Seth Forshee wrote: > On Thu, Sep 22, 2016 at 04:17:09PM +0100, Sachin Prabhu wrote: > > On Thu, 2016-09-22 at 10:09 -0500, Seth Forshee wrote: > > > We've received reports from users of a cifs mount regression in our > > > 4.4-based kernel, e.g. [1]. It

Re: cifs mount regression in 4.8 and 4.4 stable

On Thu, Sep 22, 2016 at 10:27:56AM -0500, Seth Forshee wrote: > On Thu, Sep 22, 2016 at 04:17:09PM +0100, Sachin Prabhu wrote: > > On Thu, 2016-09-22 at 10:09 -0500, Seth Forshee wrote: > > > We've received reports from users of a cifs mount regression in our > > > 4.4-based kernel, e.g. [1]. It

Re: [PATCH] drm/tilcdc: fix wrong error handling

On 09/23/16 14:47, Sean Paul wrote: > On Fri, Sep 23, 2016 at 3:52 AM, Daniel Schultz wrote: >> When 'component_bind_all' fails it should not try to unbind components >> in the error handling. This will produce a null pointer kernel panic when >> no component exist. >> >>

Re: [PATCH] drm/tilcdc: fix wrong error handling

On 09/23/16 14:47, Sean Paul wrote: > On Fri, Sep 23, 2016 at 3:52 AM, Daniel Schultz wrote: >> When 'component_bind_all' fails it should not try to unbind components >> in the error handling. This will produce a null pointer kernel panic when >> no component exist. >> >> This patch changes the

Re: [PATCH 3/4] autofs - make mountpoint checks namespace aware

On Fri, 2016-09-23 at 12:26 +0800, Ian Kent wrote: > On Thu, 2016-09-22 at 20:37 -0500, Eric W. Biederman wrote: > > Ian Kent writes: > > > > > On Thu, 2016-09-22 at 10:43 -0500, Eric W. Biederman wrote: > > > > Ian Kent writes: > > > > > > > > > Eric,

Re: [PATCH 3/4] autofs - make mountpoint checks namespace aware

On Fri, 2016-09-23 at 12:26 +0800, Ian Kent wrote: > On Thu, 2016-09-22 at 20:37 -0500, Eric W. Biederman wrote: > > Ian Kent writes: > > > > > On Thu, 2016-09-22 at 10:43 -0500, Eric W. Biederman wrote: > > > > Ian Kent writes: > > > > > > > > > Eric, Mateusz, I appreciate your spending time

Re: [PATCH 2/2] armv8: aarch32: Add SMP support for 32-bit Linux kernel

On Friday, September 23, 2016 10:19:06 AM CEST Alison Wang wrote: > The patch adds SMP support for running 32-bit Linux kernel for > Layerscape platforms. Spin-table method is used for SMP support. > > Signed-off-by: Alison Wang > Signed-off-by: Chenhui Zhao

Re: [PATCH 2/2] armv8: aarch32: Add SMP support for 32-bit Linux kernel

On Friday, September 23, 2016 10:19:06 AM CEST Alison Wang wrote: > The patch adds SMP support for running 32-bit Linux kernel for > Layerscape platforms. Spin-table method is used for SMP support. > > Signed-off-by: Alison Wang > Signed-off-by: Chenhui Zhao > --- > arch/arm/mach-imx/common.h

[PATCH] sched/fair: Do not decay new task load on first enqueue

Since commit 7dc603c9028e ("sched/fair: Fix PELT integrity for new tasks") ::last_update_time will be set to a non-zero value in post_init_entity_util_avg(), which leads to p->se.avg.load_avg being decayed on enqueue before the task has even had a chance to run. For a NICE_0 task the sequence of

[PATCH] sched/fair: Do not decay new task load on first enqueue

Since commit 7dc603c9028e ("sched/fair: Fix PELT integrity for new tasks") ::last_update_time will be set to a non-zero value in post_init_entity_util_avg(), which leads to p->se.avg.load_avg being decayed on enqueue before the task has even had a chance to run. For a NICE_0 task the sequence of

[PATCH V3] clocksource/timer-imx-gpt: Preventing resource leakage in error case.

-Free previously allocated memory. -Unmap I/O memory from kernel address space. Signed-off-by: Arvind Yadav --- drivers/clocksource/timer-imx-gpt.c | 23 +-- 1 file changed, 17 insertions(+), 6 deletions(-) diff --git

[PATCH V3] clocksource/timer-imx-gpt: Preventing resource leakage in error case.

-Free previously allocated memory. -Unmap I/O memory from kernel address space. Signed-off-by: Arvind Yadav --- drivers/clocksource/timer-imx-gpt.c | 23 +-- 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/drivers/clocksource/timer-imx-gpt.c

Re: [PATCH] softirq: let ksoftirqd do its job

On Fri, Sep 23, 2016 at 01:35:59PM +0200, Daniel Borkmann wrote: > On 09/02/2016 08:39 AM, David Miller wrote: > > > >I'm just kind of assuming this won't go through my tree, but I can take > >it if that's what everyone agrees to. > > Was this actually picked up somewhere in the mean time? I can

Re: [PATCH] softirq: let ksoftirqd do its job

On Fri, Sep 23, 2016 at 01:35:59PM +0200, Daniel Borkmann wrote: > On 09/02/2016 08:39 AM, David Miller wrote: > > > >I'm just kind of assuming this won't go through my tree, but I can take > >it if that's what everyone agrees to. > > Was this actually picked up somewhere in the mean time? I can

<    7   8   9   10   11   12   13   14   15   16   >