Re: [PATCH v2] hexagon: switch to NO_BOOTMEM

2018-07-25 Thread Mike Rapoport
On Wed, Jul 25, 2018 at 08:44:57PM -0500, Richard Kuo wrote: > On Wed, Jul 25, 2018 at 08:29:54AM +0300, Mike Rapoport wrote: > > This patch adds registration of the system memory with memblock, eliminates > > bootmem initialization and converts early memory reservations from bootmem > > to

Re: [PATCH v2 3/4] dma: Add Actions Semi Owl family S900 DMA driver

2018-07-25 Thread Manivannan Sadhasivam
Hi Vinod, On Tue, Jul 24, 2018 at 06:39:43PM +0530, Vinod wrote: > somehow this got stuck so sending again... > > On 24-07-18, 18:16, Vinod wrote: > > On 23-07-18, 09:47, Manivannan Sadhasivam wrote: > > > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > >

Re: [PATCH 1/2] drivers: clk: st: warn on iomap failure

2018-07-25 Thread Nicholas Mc Guire
On Wed, Jul 25, 2018 at 01:41:38PM -0700, Stephen Boyd wrote: > Quoting Nicholas Mc Guire (2018-07-15 03:18:23) > > While the return value of clkgen_get_register_base() is being checked > > at the call site, there is no indication of failure cause thus making > > diagnosis of the issues hard. The

Re: [PATCH 10/14] arm64: dts: meson-axg: add linein codec

2018-07-25 Thread jbrunet
On Wed, 2018-07-25 at 21:18 +0200, Martin Blumenstingl wrote: > Hi Jerome, > > On Tue, Jul 24, 2018 at 3:09 PM Jerome Brunet wrote: > > > > Add the es7241 analog to digital converter which is fed by the > > lienin jack of the s400 > > > > Signed-off-by: Jerome Brunet > > --- > >

Re: [PATCH] fsi: master-ast-cf: Fix memory leak

2018-07-25 Thread Gustavo A. R. Silva
On 07/25/2018 06:45 PM, Benjamin Herrenschmidt wrote: > On Wed, 2018-07-25 at 08:38 -0500, Gustavo A. R. Silva wrote: >> In case memory resources for *fw* were allocated, release them >> before return. >> >> Addresses-Coverity-ID: 1472044 ("Resource leak") >> Fixes: 6a794a27daca ("fsi:

Re: [PATCH] reset: hisilicon: fix potential NULL pointer dereference

2018-07-25 Thread Gustavo A. R. Silva
Hi Stephen, On 07/25/2018 06:45 PM, Stephen Boyd wrote: > Quoting Gustavo A. R. Silva (2018-07-18 18:58:45) >> There is a potential execution path in which function >> platform_get_resource() returns NULL. If this happens, >> we will end up having a NULL pointer dereference. >> >> Fix this by

Re: [PATCH mmc-next 3/3] mmc: sdhci-of-dwcmshc: solve 128MB DMA boundary limitation

2018-07-25 Thread Jisheng Zhang
On Wed, 25 Jul 2018 17:47:49 +0800 Jisheng Zhang wrote: > When using DMA, if the DMA addr spans 128MB boundary, we have to split > the DMA transfer into two so that each one doesn't exceed the boundary. > > Signed-off-by: Jisheng Zhang > --- > drivers/mmc/host/sdhci-of-dwcmshc.c | 41

Re: [PATCH 3/6] irqchip: RISC-V Local Interrupt Controller Driver

2018-07-25 Thread Anup Patel
On Wed, Jul 25, 2018 at 4:54 PM, Christoph Hellwig wrote: > On Wed, Jul 25, 2018 at 12:18:39PM +0100, Marc Zyngier wrote: >> This feels odd. It means that you cannot have the following sequence: >> >> local_irq_disable(); >> enable_irq(x); // where x is owned by a remote hart >> >> as

Re: [PATCH] watchdog: sp805: Add clock-frequency property

2018-07-25 Thread Srinath Mannam
Hi Guenter, It was my mistake sorry for that. I thought to add as Rivewed-by your name.. Many changes are made based on your review comments and suggestions. So, Can I add your name in Reviewed list? I will modify and send next patchset. thank you. Regards, Srinath. On Thu, Jul 26, 2018 at

[PATCH v2] watchdog: sp805: Add clock-frequency property

2018-07-25 Thread Srinath Mannam
Use clock-frequency property given in _DSD object of ACPI device to calculate Watchdog rate as binding clock devices are not available as device tree. Note: There is no formal review process for _DSD properties Signed-off-by: Srinath Mannam Reviewed-by: Guenter Roeck Reviewed-by: Ray Jui

[PATCH] ASoC: AMD: Fix build warning

2018-07-25 Thread Akshu Agrawal
Fixes sound/soc/amd/acp-da7219-max98357a.c: In function 'cz_probe': sound/soc/amd/acp-da7219-max98357a.c:367:3: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized] dev_err(>dev, "Failed to register regulator: %d\n", ret); Signed-off-by: Akshu Agrawal ---

linux-next: manual merge of the block tree with the rdma tree

2018-07-25 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the block tree got a conflict in: drivers/nvme/target/rdma.c between commit: 23f96d1f15a7 ("nvmet-rdma: Simplify ib_post_(send|recv|srq_recv)() calls") 202093848cac ("nvmet-rdma: add an error flow for post_recv failures") from the rdma tree and

Re: [PATCH] watchdog: sp805: Add clock-frequency property

2018-07-25 Thread Guenter Roeck
On 07/25/2018 09:47 PM, Srinath Mannam wrote: Hi Guenter, It was my mistake sorry for that. I thought to add as Rivewed-by your name.. Many changes are made based on your review comments and suggestions. So, Can I add your name in Reviewed list? You did that already below, and I am ok with

RE: [PATCH v3 2/2] fpga: zynq-fpga: Add support for readback

2018-07-25 Thread Appana Durga Kedareswara Rao
Hi Alan, Thanks for the review... > > In Zynq Case it supports two types of the readback (Configuration registers, > Configuration data(fpga image)) which may not be the same case for other > vendors. > > Since I need to support both the use cases I have differentiated them using >

Re: [PATCH] watchdog: sp805: Add clock-frequency property

2018-07-25 Thread Srinath Mannam
Hi Guenter, Thank you.. I sent next version patch with modification. Regards, Srinath. On Thu, Jul 26, 2018 at 10:39 AM, Guenter Roeck wrote: > On 07/25/2018 09:47 PM, Srinath Mannam wrote: >> >> Hi Guenter, >> >> It was my mistake sorry for that. I thought to add as Rivewed-by your >> name..

[PATCH v2 2/3] selftest/ftrace: Move kprobe selftest function to separate compile unit

2018-07-25 Thread Masami Hiramatsu
From: Francis Deslauriers Move selftest function to its own compile unit so it can be compiled with the ftrace cflags (CC_FLAGS_FTRACE) allowing it to be probed during the ftrace startup tests. Signed-off-by: Francis Deslauriers Acked-by: Masami Hiramatsu --- kernel/trace/Makefile

BUG: soft lockup in snd_virmidi_output_trigger

2018-07-25 Thread Dae R. Jeong
Reporting the crash: BUG: soft lockup in snd_virmidi_output_trigger This crash has been found in v4.18-rc3 using RaceFuzzer (a modified version of Syzkaller), which we describe more at the end of this report. Note that this bug is previously reported by Syzkaller a few month ago.

[PATCH v2 1/3] tracing: kprobes: Prohibit probing on notrace function

2018-07-25 Thread Masami Hiramatsu
Prohibit kprobe-events probing on notrace function. Since probing on the notrace function can cause recursive event call. In most case those are just skipped, but in some case it falls into infinit recursive call. This protection can be disabled by the kconfig CONFIG_KPROBE_EVENTS_ON_NOTRACE=y,

Re: [PATCH 2/2] drivers: clk: st: address sparse warnings

2018-07-25 Thread Nicholas Mc Guire
On Wed, Jul 25, 2018 at 01:40:53PM -0700, Stephen Boyd wrote: > Quoting Nicholas Mc Guire (2018-07-15 03:18:24) > > Refactoring of code to make it more readable and at the same time make > > sparse happy again. > > > > Signed-off-by: Nicholas Mc Guire > > --- > > > > sparse complained about: >

[PATCH v2 3/3] selftests/ftrace: Fix kprobe string testcase to not probe notrace function

2018-07-25 Thread Masami Hiramatsu
Fix kprobe string argument testcase to not probe notrace function. Instead, it probes tracefs function which must be available with ftrace. Signed-off-by: Masami Hiramatsu --- .../ftrace/test.d/kprobe/kprobe_args_string.tc | 30 1 file changed, 12 insertions(+), 18

[PATCH v2 0/3] tracing: kprobes: Prohibit probing on notrace functions

2018-07-25 Thread Masami Hiramatsu
Hello, this is the 2nd version of the series to prohibit kprobe on notrace functions which Francis sent before. Here is v1 series: https://lkml.org/lkml/2018/7/12/678 In this version, I've add a kconfig option to remove notrace protection just for debugging, and also fix ftracetest testcase to

Re: [PATCH 0/3] PTI for x86-32 Fixes and Updates

2018-07-25 Thread David H. Gutteridge
On Mon, 2018-07-23 at 09:29 +0200, Joerg Roedel wrote: > Hey David, > > On Sun, Jul 22, 2018 at 11:49:00PM -0400, David H. Gutteridge wrote: > > Unfortunately, I can trigger a bug in KVM+QEMU with the Bochs VGA > > driver. (This is the same VM definition I shared with you in a PM > > back on Feb.

[PATCH v5] ARM: mvebu: use dt_fixup to provide fallback for enable-method

2018-07-25 Thread Chris Packham
We need to maintain backwards compatibility with device trees that don't define an enable method. At the same time we want the device tree to be able to specify an enable-method and have it stick. Previously by having smp assigned in the DT_MACHINE definition this would be picked up by

[PATCH 2/4] ARM: dts: mvebu: db-dxbc2: use new style nand binding

2018-07-25 Thread Chris Packham
Update the nand flash binding to the new style. Signed-off-by: Chris Packham --- arch/arm/boot/dts/armada-xp-db-dxbc2.dts | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/arch/arm/boot/dts/armada-xp-db-dxbc2.dts b/arch/arm/boot/dts/armada-xp-db-dxbc2.dts

[PATCH 3/4] ARM: dts: mvebu: db-xc3-24g4: use new style nand binding

2018-07-25 Thread Chris Packham
Update the nand flash binding to the new style. Signed-off-by: Chris Packham --- arch/arm/boot/dts/armada-xp-db-xc3-24g4xg.dts | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/arch/arm/boot/dts/armada-xp-db-xc3-24g4xg.dts

[PATCH 4/4] ARM: dts: mvebu: Add device tree for db-88f6820-amc board

2018-07-25 Thread Chris Packham
This board is a plugin card for some of Marvell's switch development kits. It's similar to the non-amc board except that it has no SATA support. Signed-off-by: Chris Packham --- arch/arm/boot/dts/Makefile| 1 + .../boot/dts/armada-385-db-88f6820-amc.dts| 147

Re: [PATCH] watchdog: sp805: Add clock-frequency property

2018-07-25 Thread Guenter Roeck
On 07/23/2018 01:37 AM, Srinath Mannam wrote: Use clock-frequency property given in _DSD object of ACPI device to calculate Watchdog rate as binding clock devices are not available as device tree. Note: There is no formal review process for _DSD properties Signed-off-by: Srinath Mannam

linux-next: build failure after merge of the block tree

2018-07-25 Thread Stephen Rothwell
Hi all, After merging the block tree, today's linux-next build (x86_64 allmodconfig) failed like this: drivers/nvme/target/rdma.c: In function 'nvmet_rdma_find_get_device': drivers/nvme/target/rdma.c:894:26: error: 'struct ib_device_attr' has no member named 'max_sge'; did you mean 'max_cqe'?

Re: [PATCH] net/rds/Kconfig: RDS should depend on IPV6

2018-07-25 Thread Ka-Cheong Poon
On 07/26/2018 06:36 AM, Santosh Shilimkar wrote: On 7/25/2018 3:20 PM, Anders Roxell wrote: Build error, implicit declaration of function __inet6_ehashfn shows up When RDS is enabled but not IPV6. net/rds/connection.c: In function ‘rds_conn_bucket’: net/rds/connection.c:67:9: error: implicit

Re: linux-next: build warning after merge of the rdma tree

2018-07-25 Thread Bart Van Assche
On Wed, 2018-07-25 at 21:05 -0600, Jason Gunthorpe wrote: > On Thu, Jul 26, 2018 at 10:55:53AM +1000, Stephen Rothwell wrote: > > Hi all, > > > > After merging the rdma tree, today's linux-next build (powerpc > > ppc64_defconfig) produced this warning: > > > > net/sunrpc/xprtrdma/svc_rdma_rw.c:

Re: [PATCH] ACPI / LPSS: Avoid PM quirks on suspend and resume from S3

2018-07-25 Thread Kai-Heng Feng
Hi Rafael, > On 2018Jul24, at 18:36, Rafael J. Wysocki wrote: > > On Tuesday, July 24, 2018 11:13:42 AM CEST Rafael J. Wysocki wrote: >> On Tuesday, July 24, 2018 10:46:09 AM CEST Kai Heng Feng wrote: >>> Hi Rafael, >>> On Jun 13, 2018, at 7:17 PM, Rafael J. Wysocki wrote:

[PATCH 1/4] ARM: dts: mvebu: 98dx3236: Rename nand controller node

2018-07-25 Thread Chris Packham
Update the 98dx3236 SoC and dependent boards to use "nand-controller" instead of "nand". Signed-off-by: Chris Packham --- arch/arm/boot/dts/armada-xp-98dx3236.dtsi | 2 +- arch/arm/boot/dts/armada-xp-db-dxbc2.dts | 2 +- arch/arm/boot/dts/armada-xp-db-xc3-24g4xg.dts | 2 +- 3 files

[PATCH 0/4] ARM: dts: mvebu: updates and new board

2018-07-25 Thread Chris Packham
This series updates the armada-xp-98dx3236 SoC and related boards to use the new style dts bindings for nand. I've also added a new db-88f6820-amc board which is an Armada-385 based reference board from Marvell's switch team. It's a plugin card for either the db-dxbc2 or db-xc3-24g4 which can be

[PATCH v3 1/4] dt-bindings: dmaengine: Add binding for Actions Semi Owl SoCs

2018-07-25 Thread Manivannan Sadhasivam
Add devicetree binding for Actions Semi Owl SoCs DMA controller. Signed-off-by: Manivannan Sadhasivam --- .../devicetree/bindings/dma/owl-dma.txt | 47 +++ 1 file changed, 47 insertions(+) create mode 100644 Documentation/devicetree/bindings/dma/owl-dma.txt diff --git

[PATCH v3 2/4] arm64: dts: actions: Add Actions Semi S900 DMA Controller

2018-07-25 Thread Manivannan Sadhasivam
Add DMA controller node for Actions Semi S900 SoC. Signed-off-by: Manivannan Sadhasivam --- arch/arm64/boot/dts/actions/s900.dtsi | 13 + 1 file changed, 13 insertions(+) diff --git a/arch/arm64/boot/dts/actions/s900.dtsi b/arch/arm64/boot/dts/actions/s900.dtsi index

[PATCH v3 0/4] Add Actions Semi Owl family S900 DMA Controller support

2018-07-25 Thread Manivannan Sadhasivam
This patchset adds DMA controller support for Actions Semi Owl family S900 SoC. This driver has been structured in a way such that there will be only one controller driver for the whole Owl family series (S500, S700 and S900 SoCs). There are 12 physical channels and 46 logical channels supported

[PATCH v3 3/4] dmaengine: Add Actions Semi Owl family S900 DMA driver

2018-07-25 Thread Manivannan Sadhasivam
Add Actions Semi Owl family S900 DMA driver. Signed-off-by: Manivannan Sadhasivam --- drivers/dma/Kconfig | 8 + drivers/dma/Makefile | 1 + drivers/dma/owl-dma.c | 971 ++ 3 files changed, 980 insertions(+) create mode 100644

Re: [PATCH] mmc: host: iproc: Add ACPI support to IPROC SDHCI

2018-07-25 Thread Srinath Mannam
Hi Ulf Hansson, Kindly review this patch. Thank you, Regards, Srinath. On Tue, Jul 17, 2018 at 10:15 PM, Srinath Mannam wrote: > Add ACPI support to all IPROC SDHCI varients > > Signed-off-by: Srinath Mannam > Reviewed-by: Ray Jui > Reviewed-by: Scott Branden > Reviewed-by: Vladimir

[PATCH v3 4/4] MAINTAINERS: Add entry for Actions Semi Owl SoCs DMA driver

2018-07-25 Thread Manivannan Sadhasivam
Add entry for Actions Semi Owl SoCs DMA driver under ARM/ACTIONS. Signed-off-by: Manivannan Sadhasivam --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 09b54e9ebc6f..56d9c7715c2a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1145,12 +1145,14

Re: linux-next: build warning after merge of the sound-asoc tree

2018-07-25 Thread Agrawal, Akshu
On 7/26/2018 7:49 AM, Stephen Rothwell wrote: > Hi all, > > After merging the sound-asoc tree, today's linux-next build (x86_64 > allmodconfig) produced this warning: > > sound/soc/amd/acp-da7219-max98357a.c: In function 'cz_probe': > sound/soc/amd/acp-da7219-max98357a.c:367:3: warning: 'ret'

Re: [PATCH 2/3] mm: calculate deferred pages after skipping mirrored memory

2018-07-25 Thread Andrew Morton
On Tue, 24 Jul 2018 21:46:25 -0400 Pavel Tatashin wrote: > > > +static inline bool defer_init(int nid, unsigned long pfn, unsigned long > > > end_pfn) > > > { > > > + static unsigned long prev_end_pfn, nr_initialised; > > > > So answer me quick, what happens with a static variable in an

Re: [PATCH -next] fs/epoll: simply CONFIG_NET_RX_BUSY_POLL ifdefery

2018-07-25 Thread Andrew Morton
On Wed, 25 Jul 2018 13:06:33 -0700 Davidlohr Bueso wrote: > >> +#else > >> +#define ep_busy_loop(ep, nonblock)do { } while (0) > >> +#define ep_reset_busy_poll_napi_id(ep)do { } while (0) > >> +#define ep_set_busy_poll_napi_id(epi) do { } while (0) > >> +

Re: [PATCH v2 0/6] kaddr and pfn can be NULL to ->direct_access()

2018-07-25 Thread Dave Jiang
Pretty straight forward series. Huaisheng, I can apply the whole series to libnvdimm if we can get ack's from maintainer of dcssblk and dm-writecache for the respective bits. On 07/25/2018 09:28 AM, Huaisheng Ye wrote: > From: Huaisheng Ye > > Changes since v1 [1]: > * Involve the previous

Re: [PATCH 1/6] RISC-V: simplify software interrupt / IPI code

2018-07-25 Thread Palmer Dabbelt
On Wed, 25 Jul 2018 02:36:44 PDT (-0700), Christoph Hellwig wrote: Rename handle_ipi to riscv_software_interrupt, drop the unused return value and provide a stub for the !SMP build. This allows simplifying the upcoming interrupt controller driver by not providing a wrapper for it.

Re: [PATCH 2/6] RISC-V: remove INTERRUPT_CAUSE_* defines from asm/irq.h

2018-07-25 Thread Palmer Dabbelt
On Wed, 25 Jul 2018 02:36:45 PDT (-0700), Christoph Hellwig wrote: These are only of use to the local irq controller driver, so add them in that driver implementation instead, which will be submitted soon. Signed-off-by: Christoph Hellwig --- arch/riscv/include/asm/irq.h | 4 1 file

Re: [PATCH v1 1/4] igb: Remove unnecessary include of

2018-07-25 Thread Jeff Kirsher
On Wed, 2018-07-25 at 14:52 -0500, Bjorn Helgaas wrote: > From: Bjorn Helgaas > > The igb driver doesn't need anything provided by pci-aspm.h, so > remove > the unnecessary include of it. > > Signed-off-by: Bjorn Helgaas Acked-by: Jeff Kirsher I am fine with you picking up this change. >

OOM in secondary cgroup leading to networking lose

2018-07-25 Thread Àbéjídé Àyodélé
Hi friends, We observed a sequence of events starting from an OOM in a secondary cgroup which ends up in the bond interface being down for a period of up to 12 seconds. Below is some piece of dmesg about when the bond interface went down: [Wed Jul 25 19:20:45 2018] Call Trace: [Wed Jul 25

Re: [PATCH v1 2/3] thermal: tsens: switch from of_iomap() to devm_ioremap_resource()

2018-07-25 Thread Matthias Kaehlcke
On Wed, Jul 18, 2018 at 12:55:02PM +0530, Amit Kucheria wrote: > devm_ioremap_resources() automatically requests resources (so that the I/O > region shows up in /proc/iomem) and devm_ wrappers do better error handling > and unmapping of the I/O region when needed. > > Signed-off-by: Amit Kucheria

[PATCH v1] checkpatch: check for #if 0/#if 1

2018-07-25 Thread Prakruthi Deepak Heragu
The #if 0 or #if 1 is used to toggle features. Warn if #if 0 or #if 1 is present and suggest that they can be removed. Signed-off-by: Abhijeet Dharmapurikar Signed-off-by: Prakruthi Deepak Heragu --- Changes in v1: - Rephrase the warning message to fit in a single line without 80 column

Re: [PATCH] dt-bindings: thermal: qcom-spmi-temp-alarm: Improve thermal zone in example

2018-07-25 Thread Doug Anderson
Hi, On Tue, Jul 24, 2018 at 4:50 PM, Matthias Kaehlcke wrote: > The current example for a thermal zone isn't very useful as reference > since it would result in a hardware shutdown at 145°C, instead of > allowing the system to try to shutdown gracefully. Without an ADC > channel a maximum of two

Re: [PATCH] clk: tegra: probe deferral error reporting

2018-07-25 Thread Stephen Boyd
Quoting Marcel Ziswiler (2018-07-20 00:54:22) > From: Marcel Ziswiler > > Actually report the error code from devm_regulator_get() which may as > well just be a probe deferral. > > Signed-off-by: Marcel Ziswiler > > --- > > drivers/clk/tegra/clk-dfll.c | 5 +++-- > 1 file changed, 3

Re: [PATCH] clk: uniphier: add clock frequency support for SPI

2018-07-25 Thread Stephen Boyd
Quoting Keiji Hayashibara (2018-07-18 22:23:48) > From: Kunihiko Hayashi > > Add clock control for SPI controller on UniPhier SoCs. > > Signed-off-by: Kunihiko Hayashi > Signed-off-by: Masahiro Yamada > --- Signed-off-by chain is a little weird, but I'll go with it. Applied to clk-next

Re: [PATCH] thermal: qcom-spmi: Delete unused field 'prev_stage'

2018-07-25 Thread Doug Anderson
Hi, On Tue, Jul 24, 2018 at 4:56 PM, Matthias Kaehlcke wrote: > The field 'prev_stage' in struct qpnp_tm_chip is not used, remove it. > > Signed-off-by: Matthias Kaehlcke > --- > drivers/thermal/qcom-spmi-temp-alarm.c | 1 - > 1 file changed, 1 deletion(-) Reviewed-by: Douglas Anderson

Re: linux-next: Signed-off-by missing for commit in the xarray tree

2018-07-25 Thread Matthew Wilcox
On Thu, Jul 26, 2018 at 07:28:14AM +1000, Stephen Rothwell wrote: > Hi Matthew, > > Commits > > 890e537e2b42 ("filesystem-dax: Introduce dax_lock_mapping_entry()") > aaf149902c79 ("filesystem-dax: Set page->index") > > are missing a Signed-off-by from their committers. Oh, hah. I assume

Re: [PATCH] input: pxrc - do not store USB device in private struct

2018-07-25 Thread Dmitry Torokhov
On Tue, Jul 24, 2018 at 08:09:59PM +0200, Marcus Folkesson wrote: > Hello Dmitry, > > On Tue, Jul 24, 2018 at 02:38:04AM +, Dmitry Torokhov wrote: > > Hi Marcus, > > > > On Mon, Jul 16, 2018 at 04:40:14PM +0200, Marcus Folkesson wrote: > > > The USB device is only needed during setup, so put

Re: [PATCH v1 3/3] thermal: tsens: Fix negative temperature reporting

2018-07-25 Thread Matthias Kaehlcke
On Wed, Jul 18, 2018 at 12:55:03PM +0530, Amit Kucheria wrote: > The current code will always return 0x in case of negative > temperatures due to a bug in how the binary sign extension is being done. > > Use sign_extend32() instead. > > Signed-off-by: Amit Kucheria > --- >

Re: [PATCH v2] EDAC, sb_edac: Add support for systems with segmented PCI buses

2018-07-25 Thread Masayoshi Mizuma
Hi Boris, On 07/25/2018 05:22 AM, Borislav Petkov wrote: > On Tue, Jul 24, 2018 at 03:02:13PM -0400, Masayoshi Mizuma wrote: >> [*] KASAN report is as follows. > > That KASAN report is an arbitrary side-effect from the missing segmented > support so I ripped it out from the commit message and

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-25 Thread Singh, Balbir
On 7/25/18 1:15 AM, Johannes Weiner wrote: > Hi Balbir, > > On Tue, Jul 24, 2018 at 07:14:02AM +1000, Balbir Singh wrote: >> Does the mechanism scale? I am a little concerned about how frequently >> this infrastructure is monitored/read/acted upon. > > I expect most users to poll in the

Re: [PATCH] tracing: do not leak kernel addresses

2018-07-25 Thread Steven Rostedt
On Wed, 25 Jul 2018 13:22:36 -0700 Mark Salyzyn wrote: > From: Nick Desaulniers > > Switch from 0x%lx to 0x%pK to print the kernel addresses. > > Fixes: CVE-2017-0630 Wait This breaks perf and trace-cmd! They require this to be able to print various strings in trace events. This file is

[PATCH 09/10] staging:rtl8192u: Remove typedef of struct cmpk_intr_sta_t - Style

2018-07-25 Thread John Whitmore
Remove the typedef of struct cmpk_intr_sta_t. This clears the checkpatch issue with creating new types in code. The change is purely a coding style change and should have no impact on runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.c | 6 +++---

[PATCH 04/10] staging:rtl8192u: Removed unused CMPK_RX_DBG_MSG_SIZE - Style

2018-07-25 Thread John Whitmore
Remove the unused macro definition CMPK_RX_DBG_MSG_SIZE. This is a coding style change which should have no impact on runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.h | 1 - 1 file changed, 1 deletion(-) diff --git

[PATCH 00/10] staging:rtl8192u: Coding style - r819xU_cmdpkt.h

2018-07-25 Thread John Whitmore
This series of patches contain purely simple coding style changes. I considered renaming the struct cmpk_set_cfg_t as it is never actually used. Following its definition however it is used in a #define: #define cmpk_query_cfg_tcmpk_set_cfg_t So whilst the struct cmpk_set_cfg_t

[PATCH 07/10] staging:rtl8192u: Remove multiple blank lines - Style

2018-07-25 Thread John Whitmore
Remove multiple blank lines, raise a checkpatch issue. This is purely a coding style change which should not impact runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.h | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH 08/10] staging:rtl8192u: Correct spelling in comments - style

2018-07-25 Thread John Whitmore
Correct the spelling of a number of comments, which caused a checkpatch issue. This is purely a coding style change which should have no impact on runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.h | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 06/10] staging:rtl8192u: Rename ISR_BcnTimerIntr - Style

2018-07-25 Thread John Whitmore
Rename the bit definition ISR_BcnTimerIntr to ISR_BCN_TIMER_INTR. This change clears the checkpatch issue with CamelCase naming. The change is purely a style change and should not impact runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.c | 2 +-

[PATCH 10/10] staging:rtl8192u: Remove typedef from struct cmpk_set_cfg_t - Style

2018-07-25 Thread John Whitmore
Remove the typedef from struct cmpk_set_cfg_t. This change clears the checkpatch issue with defining new types. The change is purely a coding style change which should have no impact on runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.c | 2 +-

[PATCH 02/10] staging:rtl8192u: Remove unused CMPK_TX_SET_CONFIG_SIZE - Style

2018-07-25 Thread John Whitmore
Remove the unused macro CMPK_TX_SET_CONFIG_SIZE. This is a coding style change which should not impact runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r819xU_cmdpkt.h

[PATCH 01/10] staging:rtl8192u: Remove typedef of struct cmpk_txfb_t - Style

2018-07-25 Thread John Whitmore
Remove the typedef of the structure cmpk_txfb_t. This clears the checkpatch issue with defining new types. This change is purely a coding style change which should not impact runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.c | 8

[PATCH 03/10] staging:rtl8192u: Rename bit definition ISR_TxBcnOk - Style

2018-07-25 Thread John Whitmore
Rename the bit definition ISR_TxBcnOk to ISR_TX_BCN_OK. This change clears the checkpatch issue with CamelCase naming. The change is a coding style change which should not impact runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.c | 2 +-

[PATCH 05/10] staging:rtl8192u: Rename ISR_TxBcnErr bit definition - Style

2018-07-25 Thread John Whitmore
Rename the bit definition ISR_TxBcnErr to ISR_TX_BCN_ERR. This change clears the checkpatch issue with CamelCase naming. The change is purely a coding style change which should not impact runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_cmdpkt.c | 2 +-

Re: [PATCH 1/1] rpmsg: virtio_rpmsg_bus: Fix rpmsg_remove() release order

2018-07-25 Thread Suman Anna
Hi Loic, On 07/25/2018 04:07 PM, Loic Pallardy wrote: > In rpmsg_remove() procedure, children are removed then Name > Service announcement endpoint is destroyed and device released. > > Issue with current implementation is if a NS announcement > creation request is ongoing (received before

Re: [PATCH] tracing: Fix double free of event_trigger_data

2018-07-25 Thread Masami Hiramatsu
On Wed, 25 Jul 2018 15:53:21 -0400 Steven Rostedt wrote: > On Wed, 25 Jul 2018 15:29:04 -0400 > Steven Rostedt wrote: > > > Nope, this doesn't work. It's a little more complex than the other one. > > I'll just leave it, and fix the reg code for 4.19. > > I take this back. Looks like it just

Re: [PATCH] x86/bugs: protect against userspace-userspace spectreRSB

2018-07-25 Thread Jiri Kosina
On Wed, 25 Jul 2018, Linus Torvalds wrote: > > Mitigate userspace-userspace attacks by always unconditionally filling RSB > > on > > context switch when generic spectrev2 mitigation has been enabled. > > Shouldn't this also do something like > > x86_spec_ctrl_base |= x86_spec_ctrl_mask &

Re: [PATCH v5 1/3] thermal: qcom-spmi: Use PMIC thermal stage 2 for critical trip points

2018-07-25 Thread Doug Anderson
Hi, On Tue, Jul 24, 2018 at 4:46 PM, Matthias Kaehlcke wrote: > +static int qpnp_tm_update_critical_trip_temp(struct qpnp_tm_chip *chip, > +int temp) > +{ > + u8 reg; > + bool disable_s2_shutdown = false; > + int ret; > + > +

Re: [PATCH 1/3] clk: s2mps11: Fix matching when built as module and DT node contains compatible

2018-07-25 Thread Stephen Boyd
Quoting Krzysztof Kozlowski (2018-07-25 08:55:15) > When driver is built as module and DT node contains clocks compatible > (e.g. "samsung,s2mps11-clk"), the module will not be autoloaded because > module aliases won't match. > > The modalias from uevent: of:NclocksTCsamsung,s2mps11-clk > The

[PATCH v4 1/9] proc/kcore: don't grab lock for kclist_add()

2018-07-25 Thread Omar Sandoval
From: Omar Sandoval kclist_add() is only called at init time, so there's no point in grabbing any locks. We're also going to replace the rwlock with a rwsem, which we don't want to try grabbing during early boot. While we're here, mark kclist_add() with __init so that we'll get a warning if

Re: Reminder to review a few patches sent two weeks ago

2018-07-25 Thread pheragu
On 2018-07-24 17:33, Joe Perches wrote: On Tue, 2018-07-24 at 14:56 -0700, pher...@codeaurora.org wrote: A reminder to review a few patches I had sent last week. Below are the links for the patches. https://lkml.org/lkml/2018/7/5/798 I have no fundamental object to this one, but the 80

Re: [PATCH 1/2] tracing: kprobes: Prohibit probing on notrace functions

2018-07-25 Thread Masami Hiramatsu
On Fri, 13 Jul 2018 08:18:03 -0400 Steven Rostedt wrote: > On Fri, 13 Jul 2018 11:53:01 +0900 > Masami Hiramatsu wrote: > > > On Thu, 12 Jul 2018 13:54:12 -0400 > > Francis Deslauriers wrote: > > > > > From: Masami Hiramatsu > > > > > > Prohibit kprobe-events probing on notrace function. >

linux-next: build warning after merge of the rdma tree

2018-07-25 Thread Stephen Rothwell
Hi all, After merging the rdma tree, today's linux-next build (powerpc ppc64_defconfig) produced this warning: net/sunrpc/xprtrdma/svc_rdma_rw.c: In function 'svc_rdma_post_chunk_ctxt': net/sunrpc/xprtrdma/svc_rdma_rw.c:350:5: warning: 'bad_wr' may be used uninitialized in this function

Re: Showing /sys/fs/cgroup/memory/memory.stat very slow on some machines

2018-07-25 Thread Singh, Balbir
On 7/19/18 3:40 AM, Bruce Merry wrote: > On 18 July 2018 at 17:49, Shakeel Butt wrote: >> On Wed, Jul 18, 2018 at 8:37 AM Bruce Merry wrote: >>> That sounds promising. Is there any way to tell how many zombies there >>> are, and is there any way to deliberately create zombies? If I can >>>

Re: [PATCH 1/2] tracing: kprobes: Prohibit probing on notrace functions

2018-07-25 Thread Steven Rostedt
On Thu, 26 Jul 2018 09:41:06 +0900 Masami Hiramatsu wrote: > On Fri, 13 Jul 2018 08:18:03 -0400 > Steven Rostedt wrote: > > > On Fri, 13 Jul 2018 11:53:01 +0900 > > Masami Hiramatsu wrote: > > > > > On Thu, 12 Jul 2018 13:54:12 -0400 > > > Francis Deslauriers wrote: > > > > > > > From:

Re: [PATCH v5 1/3] thermal: qcom-spmi: Use PMIC thermal stage 2 for critical trip points

2018-07-25 Thread Matthias Kaehlcke
Hi Doug, On Wed, Jul 25, 2018 at 04:19:56PM -0700, Doug Anderson wrote: > On Tue, Jul 24, 2018 at 4:46 PM, Matthias Kaehlcke wrote: > > +static int qpnp_tm_update_critical_trip_temp(struct qpnp_tm_chip *chip, > > +int temp) > > +{ > > + u8 reg;

Re: [PATCH v1] checkpatch: check for #if 0/#if 1

2018-07-25 Thread Joe Perches
On Wed, 2018-07-25 at 16:14 -0700, Prakruthi Deepak Heragu wrote: > The #if 0 or #if 1 is used to toggle features. Warn if #if 0 or #if 1 > is present and suggest that they can be removed. > > Signed-off-by: Abhijeet Dharmapurikar > Signed-off-by: Prakruthi Deepak Heragu > --- > Changes in v1:

Re: [PATCHv3 0/2] Stackleak for arm64

2018-07-25 Thread Laura Abbott
On 07/25/2018 04:49 AM, Will Deacon wrote: Hi Laura, On Tue, Jul 24, 2018 at 05:38:07PM +0100, Will Deacon wrote: On Fri, Jul 20, 2018 at 02:41:52PM -0700, Laura Abbott wrote: This is the version of stackleak for arm64, hopefully ready for queueing Thanks. I'll push these into linux-next

Re: [PATCH v2 1/1] remoteproc: correct rproc_free_vring() to avoid invalid kernel paging

2018-07-25 Thread Suman Anna
Hi Loic, On 07/06/2018 02:46 AM, Loic Pallardy wrote: > If rproc_start() failed, rproc_resource_cleanup() is called to clean > debugfs entries, then associated iommu mappings, carveouts and vdev. > Issue occurs when rproc_free_vring() is trying to reset vring resource > table entry. > At this

Re: [PATCH 5/7] x86/vdso: Add vDSO functions for direct store instructions

2018-07-25 Thread Fenghua Yu
On Mon, Jul 23, 2018 at 10:27:34PM -0700, Andy Lutomirski wrote: > On Mon, Jul 23, 2018 at 8:42 PM, Fenghua Yu wrote: > > On Mon, Jul 23, 2018 at 06:48:00PM -0700, Andy Lutomirski wrote: > >> On 07/23/2018 05:55 AM, Fenghua Yu wrote: > >> >The instructions can be implemented in intrinsic

[PATCH v2 2/3] pinctrl: msm: Mux out gpio function with gpio_request()

2018-07-25 Thread Stephen Boyd
We rely on devices to use pinmuxing configurations in DT to select the GPIO function (function 0) if they're going to use the gpio in GPIO mode. Let's simplify things for driver authors by implementing gpio_request_enable() for this pinctrl driver to mux out the GPIO function when the gpio is use

[PATCH v2 1/3] pinctrl: msm: Really mask level interrupts to prevent latching

2018-07-25 Thread Stephen Boyd
The interrupt controller hardware in this pin controller has two status enable bits. The first "normal" status enable bit enables or disables the summary interrupt line being raised when a gpio interrupt triggers and the "raw" status enable bit allows or prevents the hardware from latching an

RE: [PATCH] clk: Add functions to get optional clocks

2018-07-25 Thread Stephen Boyd
Quoting Phil Edworthy (2018-07-18 06:56:26) > Hi Russell, > > On 18 July 2018 14:19, Geert Uytterhoeven wrote: > > On Wed, Jul 18, 2018 at 3:02 PM Russell King - ARM Linux wrote: > > > On Wed, Jul 18, 2018 at 01:57:38PM +0100, Phil Edworthy wrote: > > > > Behaves the same as (devm_)clk_get

Re: [PATCH 1/1] rpmsg: virtio_rpmsg_bus: Fix rpmsg_create_channel memory leak

2018-07-25 Thread Suman Anna
Hi Loic, On 07/25/2018 04:08 PM, Loic Pallardy wrote: > In case of rpmsg_register_device() failure, vch previously > allocated must be free. > Isn't this already handled through the virtio_rpmsg_release_device() callback as part of the put_device() in rpmsg_register_device() failure. regards

Re: [PATCH v1 1/3] thermal: tsens: Rename variable

2018-07-25 Thread Matthias Kaehlcke
On Wed, Jul 18, 2018 at 12:55:01PM +0530, Amit Kucheria wrote: > We're actually reading the temperature from the status register. Fix the > variable name to reflect that. > > Signed-off-by: Amit Kucheria > --- > drivers/thermal/qcom/tsens-common.c | 6 +++--- > drivers/thermal/qcom/tsens-v2.c

[PATCH] PCI: Equalize hotplug memory for non/occupied slots

2018-07-25 Thread Jon Derrick
Currently, a hotplug bridge will be given hpmemsize additional memory if available, in order to satisfy any future hotplug allocation requirements. These calculations don't consider the current memory size of the hotplug bridge/slot, so hotplug bridges/slots which have downstream devices will get

Re: [PATCHv3 2/2] mtd: m25p80: restore the status of SPI flash when exiting

2018-07-25 Thread Brian Norris
Hi, On Wed, Jul 25, 2018 at 07:48:21AM +1000, NeilBrown wrote: > On Tue, Jul 24 2018, Brian Norris wrote: > > On Tue, Jul 24, 2018 at 11:51:49AM +1000, NeilBrown wrote: > >> On Tue, Jul 24 2018, Boris Brezillon wrote: > >> > On Tue, 24 Jul 2018 08:46:33 +1000 > >> > NeilBrown wrote: > >> >> One

Re: [PATCH 1/2] clk: uniphier: add NAND 200MHz clock

2018-07-25 Thread Stephen Boyd
Quoting Masahiro Yamada (2018-07-20 01:37:35) > The Denali NAND controller IP needs three clocks: > > - clk: controller core clock > > - clk_x: bus interface clock > > - ecc_clk: clock at which ECC circuitry is run > > Currently, only the first one (50MHz) is provided. The rest of the >

Re: [PATCH 2/2] clk: uniphier: add more USB3 PHY clocks

2018-07-25 Thread Stephen Boyd
Quoting Masahiro Yamada (2018-07-20 01:37:36) > Add USB3 PHY clocks where missing. Use fixed-factor clocks for those > without gating. > > For clarification, prefix clock names with 'ss' or 'hs'. > > Signed-off-by: Masahiro Yamada > --- Applied to clk-next

Re: [PATCH v7 1/5] clk: actions: Add missing REGMAP_MMIO dependency

2018-07-25 Thread Stephen Boyd
Quoting Saravanan Sekar (2018-07-19 02:06:45) > Add REGMAP_MMIO as dependency to avoid undefined > reference to regmap symbols. > > Fixes: d85d20053e19 ("clk: actions: Add S900 SoC clock support") > Signed-off-by: Saravanan Sekar > Reviewed-by: Andreas Färber > Reviewed-by: Manivannan

Re: [PATCH v7 2/5] dt-bindings: clock: Add S700 support for Actions Semi Soc's

2018-07-25 Thread Stephen Boyd
Quoting Saravanan Sekar (2018-07-19 02:06:46) > Add clock bindings constants for action S700 > Maintain common clock dt-bindings for Actions Semi SoC's > S700 and S900. > > Signed-off-by: Parthiban Nallathambi > Signed-off-by: Saravanan Sekar > Reviewed-by: Rob Herring > --- Applied to

Re: [PATCH v7 3/5] clk: actions: Add S700 SoC clock support

2018-07-25 Thread Stephen Boyd
Quoting Saravanan Sekar (2018-07-19 02:06:47) > Add Actions Semi S700 SoC clock support > > Signed-off-by: Parthiban Nallathambi > Signed-off-by: Saravanan Sekar > Reviewed-by: Manivannan Sadhasivam > --- Applied to clk-next

Re: [PATCH] clk: tegra: probe deferral error reporting

2018-07-25 Thread Peter Geis
On 7/25/2018 7:24 PM, Stephen Boyd wrote: Quoting Marcel Ziswiler (2018-07-20 00:54:22) From: Marcel Ziswiler Actually report the error code from devm_regulator_get() which may as well just be a probe deferral. Signed-off-by: Marcel Ziswiler --- drivers/clk/tegra/clk-dfll.c | 5 +++--

Re: linux-next: Signed-off-by missing for commit in the xarray tree

2018-07-25 Thread Dave Jiang
On 07/25/2018 05:03 PM, Stephen Rothwell wrote: > Hi Matthew, > > On Wed, 25 Jul 2018 16:36:21 -0700 Matthew Wilcox wrote: >> >> On Thu, Jul 26, 2018 at 07:28:14AM +1000, Stephen Rothwell wrote: >>> >>> Commits >>> >>> 890e537e2b42 ("filesystem-dax: Introduce dax_lock_mapping_entry()") >>>

  1   2   3   4   5   6   7   8   9   10   >