Re: [PATCH] mm: Use special value SHRINKER_REGISTERING instead list_empty() check

2018-08-04 Thread Kirill Tkhai
On 05.08.2018 03:03, Matthew Wilcox wrote: > On Sat, Aug 04, 2018 at 09:42:05PM +0300, Kirill Tkhai wrote: >> This is exactly the thing the patch makes. Instead of inserting a shrinker >> pointer >> to idr, it inserts a fake value SHRINKER_REGISTERING there. The patch makes >> impossible >> to

Re: [PATCH] mm: Use special value SHRINKER_REGISTERING instead list_empty() check

2018-08-04 Thread Kirill Tkhai
On 05.08.2018 03:03, Matthew Wilcox wrote: > On Sat, Aug 04, 2018 at 09:42:05PM +0300, Kirill Tkhai wrote: >> This is exactly the thing the patch makes. Instead of inserting a shrinker >> pointer >> to idr, it inserts a fake value SHRINKER_REGISTERING there. The patch makes >> impossible >> to

Re: [rt-patch 4/3] arm,KVM: Move phys_timer handling to hard irq context

2018-08-04 Thread Mike Galbraith
On Sat, 2018-08-04 at 14:25 +0200, Mike Galbraith wrote: > > Besides, there are more interesting fish in the arm64 sea than kvm. > > virgin 4.16.18-rt12-rt > > [ 537.236131] ITS queue timeout (65440 65504 4640) > [ 537.236150] ITS cmd its_build_inv_cmd failed FWIW, I had thought that

Re: [rt-patch 4/3] arm,KVM: Move phys_timer handling to hard irq context

2018-08-04 Thread Mike Galbraith
On Sat, 2018-08-04 at 14:25 +0200, Mike Galbraith wrote: > > Besides, there are more interesting fish in the arm64 sea than kvm. > > virgin 4.16.18-rt12-rt > > [ 537.236131] ITS queue timeout (65440 65504 4640) > [ 537.236150] ITS cmd its_build_inv_cmd failed FWIW, I had thought that

[PATCH v4 1/2] leds: core: Introduce LED pattern trigger

2018-08-04 Thread Baolin Wang
Some LED controllers have support for autonomously controlling brightness over time, according to some preprogrammed pattern or function. This patch adds pattern trigger that LED device can configure the pattern and trigger it. Signed-off-by: Raphael Teysseyre Signed-off-by: Baolin Wang ---

[PATCH v4 1/2] leds: core: Introduce LED pattern trigger

2018-08-04 Thread Baolin Wang
Some LED controllers have support for autonomously controlling brightness over time, according to some preprogrammed pattern or function. This patch adds pattern trigger that LED device can configure the pattern and trigger it. Signed-off-by: Raphael Teysseyre Signed-off-by: Baolin Wang ---

Re: [LKP] [mm, oom] c1e4c54f9c: BUG:KASAN:null-ptr-deref_in_d

2018-08-04 Thread 禹舟键
Hi Michal and David, AFAICS, the null pointer is the task_struct of p in the dump_header. Why don't we just add a if before printing the victim task's informaiton. diff --git a/mm/oom_kill.c b/mm/oom_kill.c --- a/mm/oom_kill.c +++ b/mm/oom_kill.c @@ -438,14 +438,6 @@ static void

Re: [LKP] [mm, oom] c1e4c54f9c: BUG:KASAN:null-ptr-deref_in_d

2018-08-04 Thread 禹舟键
Hi Michal and David, AFAICS, the null pointer is the task_struct of p in the dump_header. Why don't we just add a if before printing the victim task's informaiton. diff --git a/mm/oom_kill.c b/mm/oom_kill.c --- a/mm/oom_kill.c +++ b/mm/oom_kill.c @@ -438,14 +438,6 @@ static void

Re: [PATCH v8 2/6] arm64: dts: actions: Add pinctrl definition for S900 I2C controller

2018-08-04 Thread Manivannan Sadhasivam
On Sat, Aug 04, 2018 at 11:04:40PM +0200, Wolfram Sang wrote: > On Wed, Aug 01, 2018 at 11:32:28PM +0530, Manivannan Sadhasivam wrote: > > Add pinctrl definition for Actions Semiconductor S900 I2C controller. > > Pinctrl definitions are only available for I2C0, I2C1, and I2C2. > > > >

Re: [PATCH v8 2/6] arm64: dts: actions: Add pinctrl definition for S900 I2C controller

2018-08-04 Thread Manivannan Sadhasivam
On Sat, Aug 04, 2018 at 11:04:40PM +0200, Wolfram Sang wrote: > On Wed, Aug 01, 2018 at 11:32:28PM +0530, Manivannan Sadhasivam wrote: > > Add pinctrl definition for Actions Semiconductor S900 I2C controller. > > Pinctrl definitions are only available for I2C0, I2C1, and I2C2. > > > >

Re: [PATCH v3 1/2] leds: core: Introduce LED pattern trigger

2018-08-04 Thread Baolin Wang
Hi Jacek, On 5 August 2018 at 05:41, Jacek Anaszewski wrote: > Hi Baolin, > > Thank you for the new version of the patch set. Thanks for your quick response. > > On 08/04/2018 06:51 PM, Baolin Wang wrote: >> Some LED controllers have support for autonomously controlling >> brightness over

Re: [PATCH v3 1/2] leds: core: Introduce LED pattern trigger

2018-08-04 Thread Baolin Wang
Hi Jacek, On 5 August 2018 at 05:41, Jacek Anaszewski wrote: > Hi Baolin, > > Thank you for the new version of the patch set. Thanks for your quick response. > > On 08/04/2018 06:51 PM, Baolin Wang wrote: >> Some LED controllers have support for autonomously controlling >> brightness over

Re: [PATCH] arm64: clean the additional checks before calling ghes_notify_sea()

2018-08-04 Thread gengdongjiu
2018-07-27 18:06 GMT+08:00 Will Deacon : > On Thu, Jul 26, 2018 at 05:01:47PM -0400, Dongjiu Geng wrote: >> In order to remove the additional check before calling the >> ghes_notify_sea(), make stub definition when !CONFIG_ACPI_APEI_SEA. >> >> Signed-off-by: Dongjiu Geng >> --- > > Acked-by: Will

Re: [PATCH] arm64: clean the additional checks before calling ghes_notify_sea()

2018-08-04 Thread gengdongjiu
2018-07-27 18:06 GMT+08:00 Will Deacon : > On Thu, Jul 26, 2018 at 05:01:47PM -0400, Dongjiu Geng wrote: >> In order to remove the additional check before calling the >> ghes_notify_sea(), make stub definition when !CONFIG_ACPI_APEI_SEA. >> >> Signed-off-by: Dongjiu Geng >> --- > > Acked-by: Will

[PATCH] PM / hibernate: Mark expected switch fall-through

2018-08-04 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 114713 ("Missing break in switch") Signed-off-by: Gustavo A. R. Silva --- kernel/power/hibernate.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH] PM / hibernate: Mark expected switch fall-through

2018-08-04 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 114713 ("Missing break in switch") Signed-off-by: Gustavo A. R. Silva --- kernel/power/hibernate.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH] power: supply: max77693_charger: fix unintentional fall-through

2018-08-04 Thread Gustavo A. R. Silva
Hi Sebastian, On 07/22/2018 05:17 PM, Sebastian Reichel wrote: > Hi, > > On Wed, Jul 18, 2018 at 08:14:50AM +0200, Krzysztof Kozlowski wrote: >> On 17 July 2018 at 23:47, Gustavo A. R. Silva wrote: >>> It seems that a *break* is missing in order to avoid a fall-through. >>> Otherwise, the

Re: [PATCH] power: supply: max77693_charger: fix unintentional fall-through

2018-08-04 Thread Gustavo A. R. Silva
Hi Sebastian, On 07/22/2018 05:17 PM, Sebastian Reichel wrote: > Hi, > > On Wed, Jul 18, 2018 at 08:14:50AM +0200, Krzysztof Kozlowski wrote: >> On 17 July 2018 at 23:47, Gustavo A. R. Silva wrote: >>> It seems that a *break* is missing in order to avoid a fall-through. >>> Otherwise, the

Re: [PATCH v2 1/2] Support the nonstring variable attribute (gcc >= 8)

2018-08-04 Thread Joe Perches
On Wed, 2018-08-01 at 19:54 +0200, Miguel Ojeda wrote: > From the GCC manual: > > The nonstring variable attribute specifies that an object or member > declaration with type array of char or pointer to char is intended to > store character arrays that do not necessarily contain a terminating NUL

Re: [PATCH v2 1/2] Support the nonstring variable attribute (gcc >= 8)

2018-08-04 Thread Joe Perches
On Wed, 2018-08-01 at 19:54 +0200, Miguel Ojeda wrote: > From the GCC manual: > > The nonstring variable attribute specifies that an object or member > declaration with type array of char or pointer to char is intended to > store character arrays that do not necessarily contain a terminating NUL

Re: [PATCH 4/4] watchdog: hpwdt: Update version number.

2018-08-04 Thread Guenter Roeck
On 08/02/2018 02:15 PM, Jerry Hoemann wrote: Bump version number to reflect recent bug fixes. Signed-off-by: Jerry Hoemann As usual, my recommendation is to drop version numbers. The version number is reflected by the Linux kernel version and SHA. Yet, I understand that people feel quite

Re: [PATCH 4/4] watchdog: hpwdt: Update version number.

2018-08-04 Thread Guenter Roeck
On 08/02/2018 02:15 PM, Jerry Hoemann wrote: Bump version number to reflect recent bug fixes. Signed-off-by: Jerry Hoemann As usual, my recommendation is to drop version numbers. The version number is reflected by the Linux kernel version and SHA. Yet, I understand that people feel quite

Re: [PATCH 3/4] watchdog: hpwdt: Display module parameters.

2018-08-04 Thread Guenter Roeck
On 08/02/2018 02:15 PM, Jerry Hoemann wrote: Print module parameters when the driver is loaded. Signed-off-by: Jerry Hoemann --- drivers/watchdog/hpwdt.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/watchdog/hpwdt.c b/drivers/watchdog/hpwdt.c index

Re: [PATCH 3/4] watchdog: hpwdt: Display module parameters.

2018-08-04 Thread Guenter Roeck
On 08/02/2018 02:15 PM, Jerry Hoemann wrote: Print module parameters when the driver is loaded. Signed-off-by: Jerry Hoemann --- drivers/watchdog/hpwdt.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/watchdog/hpwdt.c b/drivers/watchdog/hpwdt.c index

Re: [PATCH 2/4] watchdog: hpwdt: Claim NMI from iLO

2018-08-04 Thread Guenter Roeck
On 08/02/2018 02:15 PM, Jerry Hoemann wrote: The hwpdt driver is overloaded for handling both the iLO watchdog and the explicit "Generate NMI to System" virutal button. Claim the iLO NMI virtual button even if we are not claiming the iLO watchdog pretimeout. Signed-off-by: Jerry Hoemann

Re: [PATCH 2/4] watchdog: hpwdt: Claim NMI from iLO

2018-08-04 Thread Guenter Roeck
On 08/02/2018 02:15 PM, Jerry Hoemann wrote: The hwpdt driver is overloaded for handling both the iLO watchdog and the explicit "Generate NMI to System" virutal button. Claim the iLO NMI virtual button even if we are not claiming the iLO watchdog pretimeout. Signed-off-by: Jerry Hoemann

Re: [PATCH 1/4] watchdog: hpwdt: Initialize pretimeout from module parameter.

2018-08-04 Thread Guenter Roeck
On 08/02/2018 02:15 PM, Jerry Hoemann wrote: When the pretimeout is specified as a module parameter, the value should be reflected in hpwdt_dev.pretimeout. The default (on) case is correct. But, when disabling pretimeout, the value should be set to zero in hpwdt_dev. Signed-off-by: Jerry

Re: [PATCH 1/4] watchdog: hpwdt: Initialize pretimeout from module parameter.

2018-08-04 Thread Guenter Roeck
On 08/02/2018 02:15 PM, Jerry Hoemann wrote: When the pretimeout is specified as a module parameter, the value should be reflected in hpwdt_dev.pretimeout. The default (on) case is correct. But, when disabling pretimeout, the value should be set to zero in hpwdt_dev. Signed-off-by: Jerry

Re: [PATCH 00/46] Use dmaenginem_async_device_register to simplify code

2018-08-04 Thread Huang Shijie
On Fri, Aug 03, 2018 at 11:20:49AM +0300, Peter Ujfalusi wrote: > Hi, > > On 2018-08-03 10:19, Huang Shijie wrote: > > All the patches are using dmaenginem_async_device_register to simplify code > > except the last one: > > dmaengine: add COMPILE_TEST for the drivers > > > > I use the last one

Re: [PATCH 00/46] Use dmaenginem_async_device_register to simplify code

2018-08-04 Thread Huang Shijie
On Fri, Aug 03, 2018 at 11:20:49AM +0300, Peter Ujfalusi wrote: > Hi, > > On 2018-08-03 10:19, Huang Shijie wrote: > > All the patches are using dmaenginem_async_device_register to simplify code > > except the last one: > > dmaengine: add COMPILE_TEST for the drivers > > > > I use the last one

[PATCH] soc: mediatek: pwrap: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This issue was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/soc/mediatek/mtk-pmic-wrap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] soc: mediatek: pwrap: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This issue was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/soc/mediatek/mtk-pmic-wrap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] s390/qeth: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This issue was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/s390/net/qeth_l3_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] s390/qeth: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This issue was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/s390/net/qeth_l3_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 0/3] Kbuild: fix and clean-up arch/um/Makefile

2018-08-04 Thread Randy Dunlap
On 08/03/2018 09:47 PM, Masahiro Yamada wrote: > 1/3 fixes the build failure reported by Randy. > (I may have seen a similar report before, but I cannot recall it.) > > 2/3 and 3/3 clean-up the Makefile a bit more. > > I'd like to apply this series to kbuild tree. > Ack from UML folks are

Re: [PATCH 0/3] Kbuild: fix and clean-up arch/um/Makefile

2018-08-04 Thread Randy Dunlap
On 08/03/2018 09:47 PM, Masahiro Yamada wrote: > 1/3 fixes the build failure reported by Randy. > (I may have seen a similar report before, but I cannot recall it.) > > 2/3 and 3/3 clean-up the Makefile a bit more. > > I'd like to apply this series to kbuild tree. > Ack from UML folks are

Re: [RFC PATCH v1] memory: tegra: Block DMA for clients HW on a faulty memory access

2018-08-04 Thread Dmitry Osipenko
On Sunday, 5 August 2018 03:11:57 MSK Mikko Perttunen wrote: > One potential issue is with host1x clients where userspace processes can > submit jobs with invalid memory accesses (addresses not mapped to > IOMMU). If when such a failure happens, we disable the DMA for the whole > host1x client,

Re: [RFC PATCH v1] memory: tegra: Block DMA for clients HW on a faulty memory access

2018-08-04 Thread Dmitry Osipenko
On Sunday, 5 August 2018 03:11:57 MSK Mikko Perttunen wrote: > One potential issue is with host1x clients where userspace processes can > submit jobs with invalid memory accesses (addresses not mapped to > IOMMU). If when such a failure happens, we disable the DMA for the whole > host1x client,

[PATCH] genwqe: card_base: Use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/misc/genwqe/card_base.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] genwqe: card_base: Use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/misc/genwqe/card_base.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] Input: raydium_i2c_ts - use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/input/touchscreen/raydium_i2c_ts.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH] Input: raydium_i2c_ts - use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/input/touchscreen/raydium_i2c_ts.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH] clk: qcom: clk-branch: Use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/clk/qcom/clk-branch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [RFC 0/4] Virtio uses DMA API for all devices

2018-08-04 Thread Michael S. Tsirkin
On Fri, Aug 03, 2018 at 08:16:21PM -0500, Benjamin Herrenschmidt wrote: > On Fri, 2018-08-03 at 22:07 +0300, Michael S. Tsirkin wrote: > > On Fri, Aug 03, 2018 at 10:58:36AM -0500, Benjamin Herrenschmidt wrote: > > > On Fri, 2018-08-03 at 00:05 -0700, Christoph Hellwig wrote: > > > > > 2- Make

[PATCH] clk: qcom: clk-branch: Use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/clk/qcom/clk-branch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [RFC 0/4] Virtio uses DMA API for all devices

2018-08-04 Thread Michael S. Tsirkin
On Fri, Aug 03, 2018 at 08:16:21PM -0500, Benjamin Herrenschmidt wrote: > On Fri, 2018-08-03 at 22:07 +0300, Michael S. Tsirkin wrote: > > On Fri, Aug 03, 2018 at 10:58:36AM -0500, Benjamin Herrenschmidt wrote: > > > On Fri, 2018-08-03 at 00:05 -0700, Christoph Hellwig wrote: > > > > > 2- Make

[PATCH] platform/x86: acer-wmi: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/platform/x86/acer-wmi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] platform/x86: acer-wmi: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- drivers/platform/x86/acer-wmi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [RFC PATCH v1] memory: tegra: Block DMA for clients HW on a faulty memory access

2018-08-04 Thread Mikko Perttunen
One potential issue is with host1x clients where userspace processes can submit jobs with invalid memory accesses (addresses not mapped to IOMMU). If when such a failure happens, we disable the DMA for the whole host1x client, unrelated userspace processes may see failures even though there is

Re: [RFC PATCH v1] memory: tegra: Block DMA for clients HW on a faulty memory access

2018-08-04 Thread Mikko Perttunen
One potential issue is with host1x clients where userspace processes can submit jobs with invalid memory accesses (addresses not mapped to IOMMU). If when such a failure happens, we disable the DMA for the whole host1x client, unrelated userspace processes may see failures even though there is

Re: [PATCH] mm: Use special value SHRINKER_REGISTERING instead list_empty() check

2018-08-04 Thread Matthew Wilcox
On Sat, Aug 04, 2018 at 09:42:05PM +0300, Kirill Tkhai wrote: > This is exactly the thing the patch makes. Instead of inserting a shrinker > pointer > to idr, it inserts a fake value SHRINKER_REGISTERING there. The patch makes > impossible > to dereference a shrinker unless it's completely

Re: [PATCH] mm: Use special value SHRINKER_REGISTERING instead list_empty() check

2018-08-04 Thread Matthew Wilcox
On Sat, Aug 04, 2018 at 09:42:05PM +0300, Kirill Tkhai wrote: > This is exactly the thing the patch makes. Instead of inserting a shrinker > pointer > to idr, it inserts a fake value SHRINKER_REGISTERING there. The patch makes > impossible > to dereference a shrinker unless it's completely

Re: Re: [PATCH] [PATCH] mm: disable preemption before swapcache_free

2018-08-04 Thread Hugh Dickins
On Fri, 27 Jul 2018, zhaowu...@wingtech.com wrote: > >On Thu, 26 Jul 2018 15:03:23 +0800 "zhaowu...@wingtech.com" > > wrote: > > > >> >On Thu 26-07-18 10:21:40, zhaowu...@wingtech.com wrote: > >> >[...] > >> >> Our project really needs a fix to this issue > >> > > >> >Could you be more specific

Re: Re: [PATCH] [PATCH] mm: disable preemption before swapcache_free

2018-08-04 Thread Hugh Dickins
On Fri, 27 Jul 2018, zhaowu...@wingtech.com wrote: > >On Thu, 26 Jul 2018 15:03:23 +0800 "zhaowu...@wingtech.com" > > wrote: > > > >> >On Thu 26-07-18 10:21:40, zhaowu...@wingtech.com wrote: > >> >[...] > >> >> Our project really needs a fix to this issue > >> > > >> >Could you be more specific

Re: [PATCH 1/7] mm: allocate mm_cpumask dynamically based on nr_cpu_ids

2018-08-04 Thread Guenter Roeck
On Mon, Jul 16, 2018 at 03:03:31PM -0400, Rik van Riel wrote: > The mm_struct always contains a cpumask bitmap, regardless of > CONFIG_CPUMASK_OFFSTACK. That means the first step can be to > simplify things, and simply have one bitmask at the end of the > mm_struct for the mm_cpumask. > > This

Re: [PATCH 1/7] mm: allocate mm_cpumask dynamically based on nr_cpu_ids

2018-08-04 Thread Guenter Roeck
On Mon, Jul 16, 2018 at 03:03:31PM -0400, Rik van Riel wrote: > The mm_struct always contains a cpumask bitmap, regardless of > CONFIG_CPUMASK_OFFSTACK. That means the first step can be to > simplify things, and simply have one bitmask at the end of the > mm_struct for the mm_cpumask. > > This

[PATCH] mm/kasan/kasan_init: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- mm/kasan/kasan_init.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH] mm/kasan/kasan_init: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- mm/kasan/kasan_init.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH 12/12] ASoC: wm8903: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/wm8903.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 12/12] ASoC: wm8903: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/wm8903.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 10/12] ASoC: wm5100-tables: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/wm5100-tables.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-)

[PATCH 10/12] ASoC: wm5100-tables: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/wm5100-tables.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-)

[PATCH 11/12] ASoC: da9055: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/da9055.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 11/12] ASoC: da9055: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/da9055.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 09/12] ASoC: da7213: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/da7213.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 09/12] ASoC: da7213: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/da7213.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 08/12] ASoC: twl6040: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/twl6040.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 08/12] ASoC: twl6040: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/twl6040.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 07/12] ASoC: da7219: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/da7219.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 07/12] ASoC: da7219: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/da7219.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 06/12] ASoC: wm8996: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/wm8996.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 06/12] ASoC: wm8996: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/wm8996.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 05/12] ASoC: cs4270: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/cs4270.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 05/12] ASoC: cs4270: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/cs4270.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 04/12] ASoC: wm8990: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/wm8990.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 04/12] ASoC: wm8990: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/wm8990.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 03/12] ASoC: tda7419: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/tda7419.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 03/12] ASoC: tda7419: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/tda7419.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 02/12] ASoC: rt5631: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/rt5631.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 02/12] ASoC: rt5631: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/rt5631.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 01/12] ASoC: max9850: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/max9850.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 01/12] ASoC: max9850: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Return statements in functions returning bool should use true or false instead of an integer value. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva --- sound/soc/codecs/max9850.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 00/12] ASoC: codecs: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Hi all, This patchset aims to replace the use of integer values with a proper true or false in return statements for functions returning bool. Thanks Gustavo A. R. Silva (12): ASoC: max9850: use true and false for boolean values ASoC: rt5631: use true and false for boolean values ASoC:

[PATCH 00/12] ASoC: codecs: use true and false for boolean values

2018-08-04 Thread Gustavo A. R. Silva
Hi all, This patchset aims to replace the use of integer values with a proper true or false in return statements for functions returning bool. Thanks Gustavo A. R. Silva (12): ASoC: max9850: use true and false for boolean values ASoC: rt5631: use true and false for boolean values ASoC:

Re: [PATCH v3 1/2] leds: core: Introduce LED pattern trigger

2018-08-04 Thread Jacek Anaszewski
Hi Baolin, Thank you for the new version of the patch set. On 08/04/2018 06:51 PM, Baolin Wang wrote: > Some LED controllers have support for autonomously controlling > brightness over time, according to some preprogrammed pattern or > function. > > This patch adds pattern trigger that LED

Re: [PATCH v3 1/2] leds: core: Introduce LED pattern trigger

2018-08-04 Thread Jacek Anaszewski
Hi Baolin, Thank you for the new version of the patch set. On 08/04/2018 06:51 PM, Baolin Wang wrote: > Some LED controllers have support for autonomously controlling > brightness over time, according to some preprogrammed pattern or > function. > > This patch adds pattern trigger that LED

Re: [PATCH 5/7] x86/mm/init: remove freed kernel image areas from alias mapping

2018-08-04 Thread Andy Lutomirski
On Thu, Aug 2, 2018 at 3:58 PM, Dave Hansen wrote: > > From: Dave Hansen > > The kernel image is mapped into two places in the virtual address > space (addresses without KASLR, of course): > > 1. The kernel direct map (0x8800) > 2. The "high kernel map"

Re: [PATCH 5/7] x86/mm/init: remove freed kernel image areas from alias mapping

2018-08-04 Thread Andy Lutomirski
On Thu, Aug 2, 2018 at 3:58 PM, Dave Hansen wrote: > > From: Dave Hansen > > The kernel image is mapped into two places in the virtual address > space (addresses without KASLR, of course): > > 1. The kernel direct map (0x8800) > 2. The "high kernel map"

Re: [PATCH v2, simpler] ia64: fix ptrace(PTRACE_GETREGS) (unbreaks strace, gdb)

2018-08-04 Thread Sergei Trofimovich
On Fri, 9 Mar 2018 23:15:55 + Sergei Trofimovich wrote: I tried to explain in more detail breakage mechanics of unwinder and gcc code generation quirks at: https://trofi.github.io/posts/210-ptrace-and-accidental-boot-fix-on-ia64.html Hopefully it gives better intuition of code change

Re: [PATCH v2, simpler] ia64: fix ptrace(PTRACE_GETREGS) (unbreaks strace, gdb)

2018-08-04 Thread Sergei Trofimovich
On Fri, 9 Mar 2018 23:15:55 + Sergei Trofimovich wrote: I tried to explain in more detail breakage mechanics of unwinder and gcc code generation quirks at: https://trofi.github.io/posts/210-ptrace-and-accidental-boot-fix-on-ia64.html Hopefully it gives better intuition of code change

Re: [PATCH] ext4/mballoc: Remove unneeded variable "err"

2018-08-04 Thread Theodore Y. Ts'o
On Sat, Aug 04, 2018 at 07:04:56PM +0800, zhong jiang wrote: > The err is not used after initalization. So just remove the variable. > > Signed-off-by: zhong jiang I'll apply this patch, but how did you generate the diff? The function name here is all wrong: > diff --git a/fs/ext4/mballoc.c

Re: [PATCH] ext4/mballoc: Remove unneeded variable "err"

2018-08-04 Thread Theodore Y. Ts'o
On Sat, Aug 04, 2018 at 07:04:56PM +0800, zhong jiang wrote: > The err is not used after initalization. So just remove the variable. > > Signed-off-by: zhong jiang I'll apply this patch, but how did you generate the diff? The function name here is all wrong: > diff --git a/fs/ext4/mballoc.c

Re: [PATCH i2c-next v2] i2c: aspeed: Add an explicit type casting for *get_clk_reg_val

2018-08-04 Thread Wolfram Sang
On Tue, Jul 24, 2018 at 01:36:15PM -0700, Jae Hyun Yoo wrote: > This commit fixes this sparse warning: > drivers/i2c/busses/i2c-aspeed.c:875:38: warning: incorrect type in assignment > (different modifiers) > drivers/i2c/busses/i2c-aspeed.c:875:38:expected unsigned int ( > *get_clk_reg_val

Re: [PATCH i2c-next v2] i2c: aspeed: Add an explicit type casting for *get_clk_reg_val

2018-08-04 Thread Wolfram Sang
On Tue, Jul 24, 2018 at 01:36:15PM -0700, Jae Hyun Yoo wrote: > This commit fixes this sparse warning: > drivers/i2c/busses/i2c-aspeed.c:875:38: warning: incorrect type in assignment > (different modifiers) > drivers/i2c/busses/i2c-aspeed.c:875:38:expected unsigned int ( > *get_clk_reg_val

Re: [PATCH v8 6/6] MAINTAINERS: Add entry for Actions Semiconductor Owl I2C driver

2018-08-04 Thread Wolfram Sang
On Wed, Aug 01, 2018 at 11:32:32PM +0530, Manivannan Sadhasivam wrote: > Add entry for Actions Semiconductor Owl I2C driver under ARM/ACTIONS So, it goes upstream via arm-soc? > > Signed-off-by: Manivannan Sadhasivam Acked-by: Wolfram Sang signature.asc Description: PGP signature

Re: [PATCH v8 2/6] arm64: dts: actions: Add pinctrl definition for S900 I2C controller

2018-08-04 Thread Wolfram Sang
On Wed, Aug 01, 2018 at 11:32:28PM +0530, Manivannan Sadhasivam wrote: > Add pinctrl definition for Actions Semiconductor S900 I2C controller. > Pinctrl definitions are only available for I2C0, I2C1, and I2C2. > > Signed-off-by: Manivannan Sadhasivam Note that I don't take DTS patches, so

Re: [PATCH v8 2/6] arm64: dts: actions: Add pinctrl definition for S900 I2C controller

2018-08-04 Thread Wolfram Sang
On Wed, Aug 01, 2018 at 11:32:28PM +0530, Manivannan Sadhasivam wrote: > Add pinctrl definition for Actions Semiconductor S900 I2C controller. > Pinctrl definitions are only available for I2C0, I2C1, and I2C2. > > Signed-off-by: Manivannan Sadhasivam Note that I don't take DTS patches, so

Re: [PATCH v8 6/6] MAINTAINERS: Add entry for Actions Semiconductor Owl I2C driver

2018-08-04 Thread Wolfram Sang
On Wed, Aug 01, 2018 at 11:32:32PM +0530, Manivannan Sadhasivam wrote: > Add entry for Actions Semiconductor Owl I2C driver under ARM/ACTIONS So, it goes upstream via arm-soc? > > Signed-off-by: Manivannan Sadhasivam Acked-by: Wolfram Sang signature.asc Description: PGP signature

Re: [PATCH v2 14/18] serial: intel: Add CCF support

2018-08-04 Thread Arnd Bergmann
On Sat, Aug 4, 2018 at 2:43 PM, Greg Kroah-Hartman wrote: > On Sat, Aug 04, 2018 at 12:54:22PM +0200, Hauke Mehrtens wrote: >> On 08/03/2018 12:30 PM, Greg Kroah-Hartman wrote: >> > On Fri, Aug 03, 2018 at 03:33:38PM +0800, Wu, Songjun wrote: >> This patch makes it possible to use it with the

Re: [PATCH v2 14/18] serial: intel: Add CCF support

2018-08-04 Thread Arnd Bergmann
On Sat, Aug 4, 2018 at 2:43 PM, Greg Kroah-Hartman wrote: > On Sat, Aug 04, 2018 at 12:54:22PM +0200, Hauke Mehrtens wrote: >> On 08/03/2018 12:30 PM, Greg Kroah-Hartman wrote: >> > On Fri, Aug 03, 2018 at 03:33:38PM +0800, Wu, Songjun wrote: >> This patch makes it possible to use it with the

  1   2   3   4   5   6   7   8   >