Re: [PATCH 7/8] lib/lzo: separate lzo-rle from lzo

2018-12-01 Thread Herbert Xu
On Fri, Nov 30, 2018 at 02:26:30PM +, Dave Rodgman wrote: > To prevent any issues with persistent data, separate lzo-rle > from lzo so that it is treated as a separate algorithm, and > lzo is still available. > > Link: http://lkml.kernel.org/r/20181127161913.23863-8-dave.rodg...@arm.com >

[PATCH] cpu: Bool tests don't need comparisons

2018-12-01 Thread Wen Yang
This is the patch to the file cpu.c which fixes the following coccinelle warning: WARNING: Comparison to bool Signed-off-by: Wen Yang CC: Thomas Gleixner CC: Ingo Molnar CC: Konrad Rzeszutek Wilk CC: Josh Poimboeuf CC: "Peter Zijlstra (Intel)" CC: Peter Zijlstra CC: Mukesh Ojha CC:

Re: [PATCH 4.19 000/110] 4.19.6-stable review

2018-12-01 Thread Greg Kroah-Hartman
On Fri, Nov 30, 2018 at 02:29:45PM -0800, Guenter Roeck wrote: > On Thu, Nov 29, 2018 at 03:11:31PM +0100, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 4.19.6 release. > > There are 110 patches in this series, all will be posted as a response > > to this one.

[PATCH -next] coresight: stm: remove set but not used variable 'res_size'

2018-12-01 Thread YueHaibing
Fixes gcc '-Wunused-but-set-variable' warning: drivers/hwtracing/coresight/coresight-stm.c: In function 'stm_probe': drivers/hwtracing/coresight/coresight-stm.c:796:9: warning: variable 'res_size' set but not used [-Wunused-but-set-variable] It never used since introduction in commit

[PATCH] locktorture: Fix assignment of boolean variables

2018-12-01 Thread Wen Yang
Fix the following warnings reported by coccinelle: kernel/locking/locktorture.c:703:6-10: WARNING: Assignment of bool to 0/1 kernel/locking/locktorture.c:918:2-20: WARNING: Assignment of bool to 0/1 kernel/locking/locktorture.c:949:3-20: WARNING: Assignment of bool to 0/1

Re: [PATCH v2] signal: add procfd_signal() syscall

2018-12-01 Thread Arnd Bergmann
On Sat, Dec 1, 2018 at 12:54 AM Andy Lutomirski wrote: > On Fri, Nov 30, 2018 at 2:10 PM Arnd Bergmann wrote: > > On Fri, Nov 30, 2018 at 5:36 PM Andy Lutomirski wrote: > > > On Fri, Nov 30, 2018 at 3:41 AM Arnd Bergmann wrote: > > > > siginfo_t as it is now still has a number of other

[PATCH] sched/fair: Fix assignment of boolean variables

2018-12-01 Thread Wen Yang
Fix the following warnings reported by coccinelle: kernel//sched/fair.c:7958:3-12: WARNING: Assignment of bool to 0/1 This also makes the code more readable. Signed-off-by: Wen Yang CC: Ingo Molnar CC: Peter Zijlstra CC: linux-kernel@vger.kernel.org --- kernel/sched/fair.c | 2 +- 1 file

Re: XFS patches for stable

2018-12-01 Thread Amir Goldstein
> >> It's getting to the point that with the amount of known issues with XFS > >> on LTS kernels it makes sense to mark it as CONFIG_BROKEN. > > > >Really? Where are the bug reports? > > In 'git log'! You report these every time you fix something in upstream > xfs but don't backport it to stable

[PATCH] locktorture: style fix - spaces required around

2018-12-01 Thread Wen Yang
This patch fixes the following checkpatch.pl errors: ERROR: spaces required around that ':' (ctx:VxW) +torture_type, tag, cxt.debug_lock ? " [debug]": "", ^ Signed-off-by: Wen Yang CC: Davidlohr Bueso CC: "Paul E.

Re: [PATCH v2] signal: add procfd_signal() syscall

2018-12-01 Thread Christian Brauner
On December 1, 2018 9:51:18 PM GMT+13:00, Arnd Bergmann wrote: >On Sat, Dec 1, 2018 at 12:54 AM Andy Lutomirski >wrote: >> On Fri, Nov 30, 2018 at 2:10 PM Arnd Bergmann wrote: >> > On Fri, Nov 30, 2018 at 5:36 PM Andy Lutomirski >wrote: >> > > On Fri, Nov 30, 2018 at 3:41 AM Arnd Bergmann

[PATCH] sched/fair: Fix typos

2018-12-01 Thread Wen Yang
Fix a few typos detected by the checkpatch script: WARNING: 'intialized' may be misspelled - perhaps 'initialized'? WARNING: 'Substract' may be misspelled - perhaps 'Subtract'? Signed-off-by: Wen Yang CC: Ingo Molnar CC: Peter Zijlstra CC: linux-kernel@vger.kernel.org --- kernel/sched/fair.c

Re: [PATCH 1/4] selftests: timers: move PIE tests out of rtctest

2018-12-01 Thread Alexandre Belloni
Hello, On 29/11/2018 17:57:05-0200, Rafael David Tinoco wrote: > On 4/19/18 9:50 AM, Alexandre Belloni wrote: > > Since commit 6610e0893b8bc ("RTC: Rework RTC code to use timerqueue for > > events"), PIE are completely handled using hrtimers, without actually using > > any underlying hardware

[PATCH] i2c: mux: demux-pinctrl: handle failure case of devm_kstrdup()

2018-12-01 Thread Nicholas Mc Guire
devm_kstrdup() may return NULL if internal allocation failed. Thus using name, value is unsafe without being checked. As i2c_demux_pinctrl_probe() can return -ENOMEM in other cases a dev_err() message is included to make the failure location clear. Signed-off-by: Nicholas Mc Guire Fixes:

Re: [PATCH 22/27] locking/lockdep: Reuse list entries that are no longer in use

2018-12-01 Thread Peter Zijlstra
On Thu, Nov 29, 2018 at 08:48:50AM -0800, Bart Van Assche wrote: > On Thu, 2018-11-29 at 13:01 +0100, Peter Zijlstra wrote: > > On Thu, Nov 29, 2018 at 11:49:02AM +0100, Peter Zijlstra wrote: > > > On Wed, Nov 28, 2018 at 03:43:20PM -0800, Bart Van Assche wrote: > > > > /* > > > >

Re: [PATCH 04/24] dt-bindings: leds: Add function and color properties

2018-12-01 Thread Jacek Anaszewski
On 11/30/2018 11:19 PM, Rob Herring wrote: > On Fri, Nov 30, 2018 at 3:08 PM Pavel Machek wrote: >> >> Hi! >> Pavel gave following examples: eth0:green:link adsl0:green:link adsl0:red:error So we would have e.g.: associated-vl42-device = <>;

Re: [PATCH] mtd: remove DEBUGFS_RO_ATTR()

2018-12-01 Thread kbuild test robot
Hi Yangtao, Thank you for the patch! Yet something to improve: [auto build test ERROR on mtd/master] [also build test ERROR on v4.20-rc4 next-20181130] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[PATCH] s390: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-01 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/s390/block/dasd.c | 15 +-- drivers/s390/cio/qdio_debug.c | 16 ++-- 2 files changed, 3 insertions(+), 28 deletions(-) diff --git a/drivers/s390/block/dasd.c

[PATCH v11 0/2] cpufreq: qcom-hw: Add support for QCOM cpufreq HW

2018-12-01 Thread Taniya Das
[v11] * Updated the code logic as per Stephen. * Default boost enabled is removed. * Update the clock name to use "alternate" for GPLL0 source in code and Documentation binding. * Module description updated. * perf_base updated to perf_state_reg. [v10] * Update

Re: [PATCH] printk: don't unconditionally shortcut print_time()

2018-12-01 Thread Tetsuo Handa
On 2018/12/02 8:49, kbuild test robot wrote: >> kernel/printk/printk.c:2396:5: error: 'printk_time' undeclared (first use in >> this function) ^~~ Thanks. printk_time depends on CONFIG_PRINTK=y. Added a dummy definition. >From f903b9fa36159472a207f93e2405e45e3999f650 Mon Sep 17

Re: [PATCH v2 -next] platform/x86: intel-ips: fix 'passing zero to PTR_ERR()' warning

2018-12-01 Thread kbuild test robot
Hi YueHaibing, Thank you for the patch! Yet something to improve: [auto build test ERROR on next-20181126] url: https://github.com/0day-ci/linux/commits/YueHaibing/platform-x86-intel-ips-fix-passing-zero-to-PTR_ERR-warning/20181126-213205 config: x86_64-rhel-7.2-clear (attached as .config)

Re: [PATCH v2] mtd: remove DEBUGFS_RO_ATTR()

2018-12-01 Thread Boris Brezillon
On Sat, 1 Dec 2018 20:54:17 -0500 Yangtao Li wrote: > We already have the DEFINE_SHOW_ATTRIBUTE.There is no need to define > such a macro,so remove DEBUGFS_RO_ATTR.Also use DEFINE_SHOW_ATTRIBUTE > to simplify some code. > > Signed-off-by: Yangtao Li > --- > Changes in v2: > -Remove a missing

Re: BUG: corrupted list in freeary

2018-12-01 Thread Manfred Spraul
Hi Dmitry, On 11/30/18 6:58 PM, Dmitry Vyukov wrote: On Thu, Nov 29, 2018 at 9:13 AM, Manfred Spraul wrote: Hello together, On 11/27/18 4:52 PM, syzbot wrote: Hello, syzbot found the following crash on: HEAD commit:e195ca6cb6f2 Merge branch 'for-linus' of git://git.kernel... git tree:

Re: [PATCH v7 1/4] dt-bindings: pps: descriptor-based gpio, capture-clear addition

2018-12-01 Thread tom burkart
Quoting Rob Herring : On Wed, Nov 28, 2018 at 8:05 PM tom burkart wrote: Quoting Rob Herring : > On Mon, Nov 26, 2018 at 9:57 PM tom burkart wrote: >> >> Quoting Rob Herring : >> >> > On Sat, Nov 17, 2018 at 6:35 PM tom burkart wrote: >> >> >> >> Quoting Rob Herring : >> >> >> >> > On

Re: [PATCH v2] signal: add procfd_signal() syscall

2018-12-01 Thread Eric W. Biederman
Andy Lutomirski writes: >> On Dec 1, 2018, at 7:28 AM, Eric W. Biederman wrote: >> >> >> It just occurs to me that the simple way to implement >> procfd_sigqueueinfo info is like: >> >> int copy_siginfo_from_user_any(kernel_siginfo_t *info, siginfo_t *uinfo) >> { >> #ifdef CONFIG_COMPAT >>

Re: [PATCH v2] signal: add procfd_signal() syscall

2018-12-01 Thread Andy Lutomirski
On Sat, Dec 1, 2018 at 4:07 PM Eric W. Biederman wrote: > > Andy Lutomirski writes: > > >> On Dec 1, 2018, at 7:28 AM, Eric W. Biederman > >> wrote: > >> > >> > >> It just occurs to me that the simple way to implement > >> procfd_sigqueueinfo info is like: > >> > >> int

Re: [PATCH v2 5/5] x86/mm: Drop usage of __flush_tlb_all() in kernel_physical_mapping_init()

2018-12-01 Thread Sasha Levin
On Fri, Nov 30, 2018 at 04:35:32PM -0800, Dan Williams wrote: Commit f77084d96355 "x86/mm/pat: Disable preemption around __flush_tlb_all()" addressed a case where __flush_tlb_all() is called without preemption being disabled. It also left a warning to catch other cases where preemption is not

[PATCH v2] mtd: remove DEBUGFS_RO_ATTR()

2018-12-01 Thread Yangtao Li
We already have the DEFINE_SHOW_ATTRIBUTE.There is no need to define such a macro,so remove DEBUGFS_RO_ATTR.Also use DEFINE_SHOW_ATTRIBUTE to simplify some code. Signed-off-by: Yangtao Li --- Changes in v2: -Remove a missing DEBUGFS_RO_ATTR --- drivers/mtd/devices/docg3.c| 20

[PATCH] mfd: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-01 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/mfd/ab3100-core.c | 15 ++- drivers/mfd/ab3100-otp.c | 16 +++- drivers/mfd/tps65010.c| 14 ++ 3 files changed, 7 insertions(+), 38 deletions(-) diff --git

Re: [PATCH v10 1/2] dt-bindings: cpufreq: Introduce QCOM CPUFREQ Firmware bindings

2018-12-01 Thread Taniya Das
Hello Matthias, On 11/21/2018 11:32 PM, Matthias Kaehlcke wrote: On Wed, Nov 21, 2018 at 04:12:46PM +0530, Taniya Das wrote: Add QCOM cpufreq firmware device bindings for Qualcomm Technology Inc's SoCs. This is required for managing the cpu frequency transitions which are controlled by the

Re: [PATCH v10 1/2] dt-bindings: cpufreq: Introduce QCOM CPUFREQ Firmware bindings

2018-12-01 Thread Taniya Das
Hello Rob, On 11/27/2018 12:28 AM, Rob Herring wrote: On Wed, Nov 21, 2018 at 10:02:36AM -0800, Matthias Kaehlcke wrote: On Wed, Nov 21, 2018 at 04:12:46PM +0530, Taniya Das wrote: Add QCOM cpufreq firmware device bindings for Qualcomm Technology Inc's SoCs. This is required for managing the

Re: [PATCH v10 2/2] cpufreq: qcom-hw: Add support for QCOM cpufreq HW driver

2018-12-01 Thread Taniya Das
Hello Matthias, On 11/22/2018 12:11 AM, Matthias Kaehlcke wrote: Hi Taniya, thanks for respinning, a few nits inline. On Wed, Nov 21, 2018 at 04:12:47PM +0530, Taniya Das wrote: The CPUfreq HW present in some QCOM chipsets offloads the steps necessary for changing the frequency of CPUs. The

Re: [PATCH v10 2/2] cpufreq: qcom-hw: Add support for QCOM cpufreq HW driver

2018-12-01 Thread Taniya Das
Hello Stephen, Thanks for the patch, I have updated the latest series with the patch and few comments from Matthias. On 11/21/2018 11:53 PM, Stephen Boyd wrote: Quoting Taniya Das (2018-11-21 02:42:47) diff --git a/drivers/cpufreq/Makefile b/drivers/cpufreq/Makefile index d5ee456..789b2e0

Re: [PATCH] workqueue: remove some duplicated includes

2018-12-01 Thread Nikolay Borisov
On 1.12.18 г. 8:46 ч., Frank Lee wrote: > It seems that the trivial tree has not been maintained for some time. > Are there aother tree to choose? perhaps mmotm, maintained by Andrew (cc'ed) > > MBR, > Yangtao > On Sat, Dec 1, 2018 at 2:36 AM Tejun Heo wrote: >> >> On Mon, Nov 26, 2018 at

Re: [GIT pull] x86/pti: Cure the STIBP fallout

2018-12-01 Thread pr-tracker-bot
The pull request you sent on Sat, 1 Dec 2018 21:34:38 +0100 (CET): > git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86-pti-for-linus has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/4b78317679c4f3782a3cff0ddb269c1fcfde7621 Thank you! -- Deet-doot-dot, I

Re:

2018-12-01 Thread Ahmed Adama
-- I solicit your cooperation in a $7 Million transaction business that will benefit both of us and i need your trust reply for more details. Kind regards Ahmed.

[PATCH v2 2/9] arch/arm/mm/dma-mapping.c: Convert to use vm_insert_range

2018-12-01 Thread Souptick Joarder
Convert to use vm_insert_range() to map range of kernel memory to user vma. Signed-off-by: Souptick Joarder --- arch/arm/mm/dma-mapping.c | 21 +++-- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c index

[PATCH v2 3/9] drivers/firewire/core-iso.c: Convert to use vm_insert_range

2018-12-01 Thread Souptick Joarder
Convert to use vm_insert_range to map range of kernel memory to user vma. Signed-off-by: Souptick Joarder Reviewed-by: Matthew Wilcox --- drivers/firewire/core-iso.c | 15 ++- 1 file changed, 2 insertions(+), 13 deletions(-) diff --git a/drivers/firewire/core-iso.c

[PATCH v11 1/2] dt-bindings: cpufreq: Introduce QCOM CPUFREQ Firmware bindings

2018-12-01 Thread Taniya Das
Add QCOM cpufreq firmware device bindings for Qualcomm Technology Inc's SoCs. This is required for managing the cpu frequency transitions which are controlled by the hardware engine. Signed-off-by: Taniya Das --- .../bindings/cpufreq/cpufreq-qcom-hw.txt | 172 + 1

[PATCH v11 2/2] cpufreq: qcom-hw: Add support for QCOM cpufreq HW driver

2018-12-01 Thread Taniya Das
The CPUfreq HW present in some QCOM chipsets offloads the steps necessary for changing the frequency of CPUs. The driver implements the cpufreq driver interface for this hardware engine. Signed-off-by: Saravana Kannan Signed-off-by: Stephen Boyd Signed-off-by: Taniya Das ---

[PATCH v3] mtd: remove DEBUGFS_RO_ATTR()

2018-12-01 Thread Yangtao Li
We already have the DEFINE_SHOW_ATTRIBUTE.There is no need to define such a macro,so remove DEBUGFS_RO_ATTR.Also use DEFINE_SHOW_ATTRIBUTE to simplify some code. Signed-off-by: Yangtao Li --- changes in v3: -remove the blank line between the function definition and DEFINE_SHOW_ATTRIBUTE() ---

Re: [PATCH] mtd: remove DEBUGFS_RO_ATTR()

2018-12-01 Thread kbuild test robot
Hi Yangtao, Thank you for the patch! Yet something to improve: [auto build test ERROR on mtd/master] [also build test ERROR on v4.20-rc4 next-20181130] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH RFC 14/15] lib: replace **** with a hug

2018-12-01 Thread Sergey Senozhatsky
On (11/30/18 12:59), Jarkko Sakkinen wrote: > On Fri, Nov 30, 2018 at 02:41:11PM -0500, Steven Rostedt wrote: > > Since the code has been greatly modified since that comment was added, > > I would say the comment is simply out of date. > > > > Just nuke the comment, and that will be an accurate

Re: WARNING: CPU: 0 PID: 0 at drivers/irqchip/irq-gic-v3-its.c

2018-12-01 Thread Qian Cai
On 11/12/18 3:39 AM, Marc Zyngier wrote: > On Fri, 09 Nov 2018 18:41:03 +, > Qian Cai wrote: >> >> >> >>> On Nov 9, 2018, at 12:41 PM, Marc Zyngier wrote: >>> >>> On 09/11/18 17:28, Sudeep Holla wrote: On Fri, Nov 9, 2018 at 4:10 PM Marc Zyngier wrote: > [...] >

[GIT pull] x86/pti: Cure the STIBP fallout

2018-12-01 Thread Thomas Gleixner
Linus, please pull the latest x86-pti-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86-pti-for-linus The performance destruction department finally got it's act together and came up with a cure for the STIPB regression: - Provide a command line option

Re: [PATCH] locktorture: Fix assignment of boolean variables

2018-12-01 Thread Paul E. McKenney
On Sat, Dec 01, 2018 at 04:31:49PM +0800, Wen Yang wrote: > Fix the following warnings reported by coccinelle: > > kernel/locking/locktorture.c:703:6-10: WARNING: Assignment of bool to 0/1 > kernel/locking/locktorture.c:918:2-20: WARNING: Assignment of bool to 0/1 >

Re: [PATCH] locktorture: style fix - spaces required around

2018-12-01 Thread Paul E. McKenney
On Sat, Dec 01, 2018 at 04:40:39PM +0800, Wen Yang wrote: > This patch fixes the following checkpatch.pl errors: > > ERROR: spaces required around that ':' (ctx:VxW) > +torture_type, tag, cxt.debug_lock ? " [debug]": "", >

Re: [PATCH] sched/fair: Fix assignment of boolean variables

2018-12-01 Thread Joey Pabalinas
On Sat, Dec 01, 2018 at 05:09:36PM +0800, Wen Yang wrote: > Fix the following warnings reported by coccinelle: > kernel//sched/fair.c:7958:3-12: WARNING: Assignment of bool to 0/1 > > This also makes the code more readable. > > Signed-off-by: Wen Yang > CC: Ingo Molnar > CC: Peter Zijlstra >

Re: linux-next: manual merge of the selinux tree with the vfs tree

2018-12-01 Thread Ondrej Mosnacek
On Thu, Nov 29, 2018 at 11:07 AM Ondrej Mosnacek wrote: > On Wed, Nov 28, 2018 at 10:52 PM Paul Moore wrote: > > On Tue, Nov 27, 2018 at 6:50 AM Stephen Rothwell > > wrote: > > > Hi Ondrej, > > > > > > On Tue, 27 Nov 2018 09:53:32 +0100 Ondrej Mosnacek > > > wrote: > > > > > > > > Hm...

RE: Fwd: [Bug 201647] New: Intel Wireless card 3165 does not get detected but bluetooth works

2018-12-01 Thread Grumbach, Emmanuel
> > [+cc Emmanuel, LKML] > > On Fri, Nov 09, 2018 at 03:43:06PM -0600, Bjorn Helgaas wrote: > > -- Forwarded message - > > From: > > Date: Fri, Nov 9, 2018 at 4:10 AM > > Subject: [Bug 201647] New: Intel Wireless card 3165 does not get > > detected but bluetooth works > > > >

Re: [RFC PATCH 0/5] x86: dynamic indirect call promotion

2018-12-01 Thread Josh Poimboeuf
On Sat, Dec 01, 2018 at 06:52:45AM +, Nadav Amit wrote: > > On Nov 29, 2018, at 7:19 AM, Josh Poimboeuf wrote: > > > > On Wed, Nov 28, 2018 at 10:06:52PM -0800, Andy Lutomirski wrote: > >> On Wed, Nov 28, 2018 at 7:24 PM Andy Lutomirski > >> wrote: > >>> On Nov 28, 2018, at 6:06 PM, Nadav

Re: [PATCH] kbuild: fix UML build error with CONFIG_GCC_PLUGINS

2018-12-01 Thread Masahiro Yamada
On Thu, Nov 29, 2018 at 12:01 PM Masahiro Yamada wrote: > > UML fails to build with CONFIG_GCC_PLUGINS=y. > > $ make -s ARCH=um mrproper > $ make -s ARCH=um allmodconfig > $ make ARCH=um > UPD include/generated/uapi/linux/version.h > WRAP

Re: [PATCH v2 1/2] power: supply: add input voltage limit property.

2018-12-01 Thread Pavel Machek
Hi! > >> We have a problem with USBPD chargers which under certain conditions > >> can result in system overheating if the voltage provided by the USBPD > >> port is too high. While the preferred means to control this would be > >> through devicetree or ACPI settings, this is not always possible,

Re: [PATCH v3 3/7] staging:iio:ad2s90: Add max frequency check at probe

2018-12-01 Thread Jonathan Cameron
On Fri, 23 Nov 2018 22:23:08 -0200 Matheus Tavares wrote: > From: Alexandru Ardelean > > This patch adds a max frequency check at the beginning of ad2s90_probe > function so that when it is set to a value above 0.83Mhz, dev_err is > called with an appropriate message and -EINVAL is returned. >

Re: [PATCH v3 4/7] dt-bindings:iio:resolver: Add docs for ad2s90

2018-12-01 Thread Jonathan Cameron
On Mon, 26 Nov 2018 19:52:27 -0600 Rob Herring wrote: > On Fri, 23 Nov 2018 22:23:09 -0200, Matheus Tavares wrote: > > This patch adds the device tree binding documentation for the ad2s90 > > resolver-to-digital converter. > > > > Signed-off-by: Matheus Tavares > > --- > > Changes in v3: > >

[PATCH 00/13 v2] Regulator ena_gpiod fixups

2018-12-01 Thread Linus Walleij
Here is a second iteration of these fixups after thinking over Charles Keepax excellent comments on the first series of fixes. To make sure GPIO descriptors are never left dangling (as far as I can tell!) I use this stepwise approach: 1. Fix the regulator_register() in the core to guarantee

[PATCH] perf util config : Modify size factor of snprintf

2018-12-01 Thread Sihyeon Jang
According to definition of snprintf, it gets size factor including null('\0') byte. So '-1' is not neccessary. Also it will be helpful unfied style with other cases. (eg. builtin-script.c) Cc: Jiri Olsa Cc: Namhyung Kim Signed-off-by: Sihyeon Jang --- tools/perf/util/config.c | 4 ++-- 1

[PATCH] perf util config : Modify size factor of snprintf

2018-12-01 Thread Sihyeon Jang
According to definition of snprintf, it gets size factor including null('\0') byte. So '-1' is not neccessary. Also it will be helpful unfied style with other cases. (eg. builtin-script.c) Cc: Jiri Olsa Cc: Namhyung Kim Signed-off-by: Sihyeon Jang --- tools/perf/util/config.c | 4 ++-- 1

[PATCH] platform/x86: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-01 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/platform/x86/intel_pmc_core.c | 46 --- .../platform/x86/intel_telemetry_debugfs.c| 42 +++-- 2 files changed, 14 insertions(+), 74 deletions(-) diff --git

Re: [PATCH v2] signal: add procfd_signal() syscall

2018-12-01 Thread Andy Lutomirski
> On Dec 1, 2018, at 7:28 AM, Eric W. Biederman wrote: > > > It just occurs to me that the simple way to implement > procfd_sigqueueinfo info is like: > > int copy_siginfo_from_user_any(kernel_siginfo_t *info, siginfo_t *uinfo) > { > #ifdef CONFIG_COMPAT >if (in_compat_syscall) >

Re: [PATCH] iio: adc: Replace license text w/ SPDX identifier

2018-12-01 Thread Matheus Tavares Bernardino
On Wed, Nov 28, 2018 at 4:54 PM Scott Branden wrote: >> >> >> On 2018-11-28 9:53 a.m., Matheus Tavares wrote: >> >> From: Lucas Santos >> >> This patch removes all license boilerplate texts from the .c and .h >> > I know other people don't follow the patch guidelines, > > but more info on proper

Re: [PATCH 2/3] iio: chemical: add support for Sensirion SPS30 sensor

2018-12-01 Thread Jonathan Cameron
On Mon, 26 Nov 2018 21:48:07 +0100 Tomasz Duszynski wrote: > On Sun, Nov 25, 2018 at 04:14:34PM +0530, Himanshu Jha wrote: > > On Sat, Nov 24, 2018 at 11:14:14PM +0100, Tomasz Duszynski wrote: > > > Add support for Sensirion SPS30 particulate matter sensor. > > > > > > Signed-off-by: Tomasz

[PATCH] pcie: portdrv: Fix Unnecessary space before function pointer arguments

2018-12-01 Thread Benjamin Young
Made spacing more consistent in the code for function pointer declarations based on checkpatch.pl Signed-off-by: Benjamin Young --- drivers/pci/pcie/portdrv.h | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/pci/pcie/portdrv.h

Re: [PATCH] watchdog: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-01 Thread Guenter Roeck
On 12/1/18 2:14 AM, Yangtao Li wrote: Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li One patch per file, please. Guenter --- drivers/watchdog/bcm_kona_wdt.c | 16 +++- drivers/watchdog/ie6xx_wdt.c| 16 +++- 2 files changed,

Re: [PATCH] printk: Add caller information to printk() output.

2018-12-01 Thread Tetsuo Handa
On 2018/12/01 0:40, Petr Mladek wrote: >> Some examples for console output: >> >> [0.293000] [T1] smpboot: CPU0: Intel(R) Core(TM) i5-4440S CPU @ >> 2.80GHz (family: 0x6, model: 0x3c, stepping: 0x3) >> [0.299733] [T1] Performance Events: Haswell events, core PMU driver. >> [

Re: Regression: very quiet speakers on Thinkpad T570s

2018-12-01 Thread Takashi Iwai
On Fri, 30 Nov 2018 17:51:33 +0100, Jeremy Cline wrote: > > On 11/30/18 11:00 AM, Takashi Iwai wrote: > > On Fri, 30 Nov 2018 15:49:17 +0100, > > Jeremy Cline wrote: > >> > >> Hi, > >> > >> Some folks have reported on the Fedora bug tracker[0] that the laptop > >> speaker volume is very low on

Re: [PATCH v2] signal: add procfd_signal() syscall

2018-12-01 Thread Eric W. Biederman
Arnd Bergmann writes: > On Fri, Nov 30, 2018 at 7:56 AM Christian Brauner > wrote: >> On Thu, Nov 29, 2018 at 11:13:57PM -0600, Eric W. Biederman wrote: >> > Arnd Bergmann writes: >> > > On Thu, Nov 29, 2018 at 9:14 PM Andy Lutomirski >> > > wrote: >> > > >> > > It looks like we already

[PATCH 2/2] watchdog: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-01 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/watchdog/ie6xx_wdt.c | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/drivers/watchdog/ie6xx_wdt.c b/drivers/watchdog/ie6xx_wdt.c index 78c2541f5d52..8de9fb1ed371

[PATCH 0/2] watchdog: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-01 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Yangtao Li (2): watchdog: bcm281xx: convert to DEFINE_SHOW_ATTRIBUTE watchdog: convert to DEFINE_SHOW_ATTRIBUTE drivers/watchdog/bcm_kona_wdt.c | 16 +++- drivers/watchdog/ie6xx_wdt.c| 16 +++- 2 files

[PATCH 1/2] watchdog: bcm281xx: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-01 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/watchdog/bcm_kona_wdt.c | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/drivers/watchdog/bcm_kona_wdt.c b/drivers/watchdog/bcm_kona_wdt.c index

Re: [PATCH 1/3] modpost: refactor seen flag clearing in add_depends()

2018-12-01 Thread Masahiro Yamada
On Sat, Nov 24, 2018 at 5:08 PM Masahiro Yamada wrote: > > You do not need to iterate over all modules for resetting ->seen flag > because add_depends() is only interested in modules that export symbols > referenced from the given 'mod'. > > This also avoids shadowing the 'modules' parameter of

Re: [PATCH] watchdog: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-01 Thread Frank Lee
On Sat, Dec 1, 2018 at 10:25 PM Guenter Roeck wrote: > > On 12/1/18 2:14 AM, Yangtao Li wrote: > > Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. > > > > Signed-off-by: Yangtao Li > > One patch per file, please. > > Guenter Already split into small files. MBR, Yangtao

Re: [PATCH v2 1/2] kbuild: make 'archprepare' depend on 'scripts'

2018-12-01 Thread Masahiro Yamada
On Thu, Nov 29, 2018 at 12:57 PM Masahiro Yamada wrote: > > Before start descending, Kbuild needs to run 'prepare' and 'scripts', > which has been orthogonal to each other. > > Going forward, let's consider 'scripts' is a part of the preparation. > This will allow more cleanups. > > Move

Re: [PATCH v2 2/2] kbuild: descend into scripts/gcc-plugins/ via scripts/Makefile

2018-12-01 Thread Masahiro Yamada
On Thu, Nov 29, 2018 at 12:57 PM Masahiro Yamada wrote: > > Now that 'archprepare' depends on 'scripts', Kbuild can descend into > scripts/gcc-plugins in a more standard way. > > Signed-off-by: Masahiro Yamada > Reviewed-by: Kees Cook > --- Applied to linux-kbuild. > Changes in v2: > - It

Re: [PATCH] pcmcia: remove per-arch PCMCIA config entry

2018-12-01 Thread Masahiro Yamada
On Mon, Nov 26, 2018 at 6:24 PM Dominik Brodowski wrote: > > On Mon, Nov 26, 2018 at 05:15:41PM +0900, Masahiro Yamada wrote: > > Now that all architectures include drivers/pcmcia/Kconfig where > > the PCMCIA config is defined, the PCMCIA config entries in per-arch > > Kconfig files are

Re: siginfo pid not populated from ptrace?

2018-12-01 Thread Eric W. Biederman
Kees Cook writes: > On Tue, Nov 27, 2018 at 8:44 PM Eric W. Biederman > wrote: >> >> Kees Cook writes: >> >> > On Tue, Nov 27, 2018 at 4:38 PM, Kees Cook wrote: >> >> On Tue, Nov 27, 2018 at 3:21 PM, Tycho Andersen wrote: >> >>> On Mon, Nov 12, 2018 at 12:24:43PM -0700, Tycho Andersen

Re: [PATCH 2/2] iio: adc: ti_am335x_tscadc: Improve accuracy of measurement

2018-12-01 Thread Jonathan Cameron
On Wed, 28 Nov 2018 09:14:32 + Lee Jones wrote: > On Mon, 19 Nov 2018, Vignesh R wrote: > > > When performing single ended measurements with TSCADC, its recommended > > to set negative input (SEL_INM_SWC_3_0) of ADC step to ADC's VREFN in the > > corresponding STEP_CONFIGx register. > >

Re: [PATCH] staging: iio: ad7780: Add gain & filter gpio support

2018-12-01 Thread Jonathan Cameron
On Thu, 29 Nov 2018 12:19:08 + "Ardelean, Alexandru" wrote: > On Tue, 2018-11-27 at 06:11 -0500, Popa, Stefan Serban wrote: > > On Lu, 2018-11-26 at 17:24 -0200, Giuliano Belinassi wrote: > > Hi, please see bellow > > > > One note from me here. > > > > Hi, thank you for the review > > >

[PATCH] MAINTAINERS: Remove myself from the list

2018-12-01 Thread Carlo Caione
Kevin is basically doing all the work. Remove my name as co-maintainer. Signed-off-by: Carlo Caione --- MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 01645edc2388..63a2ad093524 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1297,7 +1297,6 @@ F:

Attention Please !!!

2018-12-01 Thread MR RECHARD THOMAS
-- My Dear Friend, I never want to bother you at all with my business proposal. However is sure i contact you as my foreign partner. I would like you to give me your full consideration to participate and share the amounts with me. You have to follow up so that you know what to do next. I need

[PATCH] mmc: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-01 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/mmc/core/mmc_test.c | 14 ++ drivers/mmc/host/atmel-mci.c | 31 +-- drivers/mmc/host/omap_hsmmc.c | 14 ++ drivers/mmc/host/s3cmci.c | 30

Re: [PATCH v2] signal: add procfd_signal() syscall

2018-12-01 Thread Eric W. Biederman
It just occurs to me that the simple way to implement procfd_sigqueueinfo info is like: int copy_siginfo_from_user_any(kernel_siginfo_t *info, siginfo_t *uinfo) { #ifdef CONFIG_COMPAT if (in_compat_syscall) return copy_siginfo_from_user32(info, uinfo); #endif

Re: [PATCH v3 2/7] staging:iio:ad2s90: Remove spi setup that should be done via dt

2018-12-01 Thread Jonathan Cameron
On Fri, 23 Nov 2018 22:23:07 -0200 Matheus Tavares wrote: > The ad2s90 driver currently sets some spi settings (max_speed_hz and > mode) at ad2s90_probe. Since the maximum frequency is a required element > in DT binding for spi slave devices and because the spi mode for the > device can be

Re: [PATCH v3 1/7] staging:iio:ad2s90: Add device tree support

2018-12-01 Thread Jonathan Cameron
On Fri, 23 Nov 2018 22:23:06 -0200 Matheus Tavares wrote: > This patch adds device tree support to ad2s90 with standard > device tree id table. > > Signed-off-by: Matheus Tavares Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks,

Re: [PATCH v3 6/7] staging:iio:ad2s90: Add comment to device state mutex

2018-12-01 Thread Jonathan Cameron
On Fri, 23 Nov 2018 22:23:11 -0200 Matheus Tavares wrote: > From: Victor Colombo > > Fix the checkpatch.pl issue: > "CHECK: struct mutex definition without comment". > > Signed-off-by: Victor Colombo > Signed-off-by: Matheus Tavares Applied, thanks Jonathan > --- > Changes in v3: > - none

Re: [PATCH v3 net] mv88e6060: disable hardware level MAC learning

2018-12-01 Thread Andrew Lunn
On Fri, Nov 30, 2018 at 09:58:36PM -0200, Anderson Luiz Alves wrote: > Disable hardware level MAC learning because it breaks station roaming. > When enabled it drops all frames that arrive from a MAC address > that is on a different port at learning table. > > Signed-off-by: Anderson Luiz Alves

Re: [PATCH v3 5/7] staging:iio:ad2s90: Replace license text w/ SPDX identifier

2018-12-01 Thread Jonathan Cameron
On Fri, 23 Nov 2018 22:23:10 -0200 Matheus Tavares wrote: > This patch removes the license boilerplate text at the top of ad2s90.c > and, instead, adds the SPDX GPL-2.0 license identifier, which solves the > checkpatch.pl warning: > "WARNING: Missing or malformed SPDX-License-Identifier tag in

Re: [PATCH v3 7/7] staging:iio:ad2s90: Move out of staging

2018-12-01 Thread Jonathan Cameron
On Mon, 26 Nov 2018 07:34:35 + "Ardelean, Alexandru" wrote: > On Fri, 2018-11-23 at 22:23 -0200, Matheus Tavares wrote: > > Move ad2s90 resolver driver out of staging to the main tree. > > > > Acked-by: Alexandru Ardelean Hi Alexandru, I took that to apply to the whole series, hope you

Re: [PATCH 1/3] iio: add IIO_MASSCONCENTRATION channel type

2018-12-01 Thread Jonathan Cameron
On Sun, 25 Nov 2018 16:44:23 +0100 Tomasz Duszynski wrote: > On Sun, Nov 25, 2018 at 06:14:44AM -0800, Matt Ranostay wrote: > > On Sun, Nov 25, 2018 at 6:03 AM Jonathan Cameron > > wrote: > > > > > > On Sun, 25 Nov 2018 05:51:32 -0800 > > > Matt Ranostay wrote: > > > > > > > On Sat, Nov

Re: ext4 file system corruption with v4.19.3 / v4.19.4

2018-12-01 Thread Huang Yan
On Tue, 27 Nov 2018 23:15:55 -0500, Theodore Y. Ts'o wrote: > I'm trying to figure out common factors from those people who are > reporting problems. Hello, I experienced the ext4-randomly-switching-to-read-only issue with Linux kernel 4.19.5 (from http://kernel.ubuntu.com/~kernel-ppa/mainline/ ;

[PATCH 13/13 v2] regulator: s2mps11: Hand over GPIO to regulator core

2018-12-01 Thread Linus Walleij
The GPIO descriptors used by the S2MPS11 driver are retrieved during probe() and it is really helpful to have those under devres management because of all the errorpaths in the intialization. Using the new dev_gpiod_unhinge() call we can remove the devres management of the descriptor right before

[PATCH 06/13 v2] regulator: max8973: Let core handle GPIO descriptor

2018-12-01 Thread Linus Walleij
Use the gpiod_get() rather than the devm_* version so that the regulator core can handle the lifecycle of these descriptors. Fixes: e7d2be696faa ("regulator: max8973: Pass descriptor instead of GPIO number") Signed-off-by: Linus Walleij --- ChangeLog v1->v2: - Drop the gpiod_put() on the

[PATCH 09/13 v2] gpio: Add devm_gpiod_unhinge()

2018-12-01 Thread Linus Walleij
This adds a function named devm_gpiod_unhinge() that removes the resource management from a GPIO descriptor. I am not sure if this is the best anglosaxon name for the function, no other managed resources have an equivalent currently, but I chose "unhinge" as the closest intuitive thing I could

[PATCH 02/13 v2] regulator: fixed: Let core handle GPIO descriptor

2018-12-01 Thread Linus Walleij
Use the gpiod_get() rather than the devm_* version so that the regulator core can handle the lifecycle of these descriptors. Fixes: efdfeb079cc3 ("regulator: fixed: Convert to use GPIO descriptor only") Signed-off-by: Linus Walleij --- ChangeLog v1->v2: - Drop the conditional gpiod_put() on the

[PATCH 03/13 v2] regulator: lm363x: Let core handle GPIO descriptor

2018-12-01 Thread Linus Walleij
Use the gpiod_get() rather than the devm_* version so that the regulator core can handle the lifecycle of these descriptors. Fixes: b2d751b7f69b ("regulator: lm363x: Pass descriptor instead of GPIO number") Signed-off-by: Linus Walleij --- ChangeLog v1->v2: - Drop the gpiod_put() on the

[PATCH 01/13 v2] regulator: core: Track dangling GPIO descriptors

2018-12-01 Thread Linus Walleij
If a GPIO descriptor is passed to the regulator_register() function inside the config->ena_gpiod callers must be sure that once they call this API the regulator core owns that descriptor and will make sure to issue gpiod_put() on it, no matter whether the call is successful or not. For device

[PATCH 04/13 v2] regulator: lp8788-ldo: Let core handle GPIO descriptor

2018-12-01 Thread Linus Walleij
Use the gpiod_get() rather than the devm_* version so that the regulator core can handle the lifecycle of these descriptors. Fixes: 2468f0d51548 ("regulator: lp8788-ldo: Pass descriptor instead of GPIO number") Signed-off-by: Linus Walleij --- ChangeLog v1->v2: - Drop the gpiod_put() on the

[PATCH 07/13 v2] gpio: Export gpiod_get_from_of_node()

2018-12-01 Thread Linus Walleij
This function already exist inside gpiolib, we were just reluctant to make it available to the kernel at large as the devm_* seemed to be enough for anyone. However we found out that regulators need to do their own lifecycle/refcounting on GPIO descriptors and explicitly call gpiod_put() when

[PATCH 10/13 v2] regulator: da9211: Hand over GPIO to regulator core

2018-12-01 Thread Linus Walleij
The GPIO descriptors used by the DA9211 driver are retrieved during probe() and it is really helpful to have those under devres management because of all the errorpaths in the intialization. Using the new dev_gpiod_unhinge() call we can remove the devres management of the descriptor right before

[PATCH 08/13 v2] regulator: max77686: Let core handle GPIO descriptor

2018-12-01 Thread Linus Walleij
Use the gpiod_get_from_of_node() rather than the devm_* version so that the regulator core can handle the lifecycle of these descriptors. Fix up the errorpath so that we free this descriptor if an error occurs in the callback. Rely on the regulator core to deal with it after this point: a

[PATCH 05/13 v2] regulator: max8952: Let core handle GPIO descriptor

2018-12-01 Thread Linus Walleij
Use the gpiod_get() rather than the devm_* version so that the regulator core can handle the lifecycle of these descriptors. Fixes: d7a261c2d1f2 ("regulator: max8952: Pass descriptor instead of GPIO number") Signed-off-by: Linus Walleij --- ChangeLog v1->v2: - Drop the gpiod_put() on the

  1   2   3   4   >