Re: [PATCH v2] PCI: imx: Add imx6sx suspend/resume support

2018-11-22 Thread Lorenzo Pieralisi
On Wed, Nov 07, 2018 at 01:57:03PM +, Leonard Crestez wrote: > Enable PCI suspend/resume support on imx6sx socs. This is similar to > imx7d with a few differences: > > * The PM_Turn_Off bit is exposed through an IOMUX GPR, like all other > pcie control bits on 6sx. > * The pcie_inbound_axi clk

[PATCH 00/17] Add support for TI PRU ICSS

2018-11-22 Thread Roger Quadros
Hi, The Programmable Real-Time Unit and Industrial Communication Subsystem (PRU-ICSS) is present on various TI SoCs such as AM335x, AM437x, AM57x, Keystone 66AK2G, etc. A PRUSS consists of dual 32-bit RISC cores (Programmable Real-Time Units, or PRUs), data and instruction RAMs, shard RAM, and int

[PATCH 01/17] dt-bindings: remoteproc: Add TI PRUSS bindings

2018-11-22 Thread Roger Quadros
From: Suman Anna This patch adds the bindings for the Programmable Real-Time Unit and Industrial Communication Subsystem (PRU-ICSS) present on various TI SoCs. The IP is present on multiple TI SoC architecture families including the OMAP architecture SoCs such as AM33xx, AM437x and AM57xx; and on

[PATCH 02/17] soc: ti: pruss: Define platform data for PRUSS bus driver

2018-11-22 Thread Roger Quadros
From: Suman Anna The PRUSS can have a PRCM reset line associated with the IP on some OMAP architecture based SoCs. The reset needs to be programmed properly before accessing any of the internal registers in the PRUSS. This functionality is achieved through the omap_device layer, which is not expo

[PATCH 04/17] soc: ti: pruss: Fix system suspend/MStandby config issues

2018-11-22 Thread Roger Quadros
From: Suman Anna The PRU-ICSS subsystem has a separate PRUSS_CFG module that contains various configuration registers. This includes a control bit STANDBY_INIT in PRUSS_CFG register to initiate a Standby sequence (when set) and trigger a MStandby request to the SoC's PRCM module. This same bit is

[PATCH 03/17] soc: ti: pruss: Add pruss_soc_bus platform driver

2018-11-22 Thread Roger Quadros
From: Suman Anna The Programmable Real-Time Unit - Industrial Communication Subsystem (PRU-ICSS) is present of various TI SoCs such as AM335x or AM437x or the Keystone 66AK2G. Each SoC can have one or more PRUSS instances that may or may not be identical. For example, AM335x SoCs have a single PR

[PATCH 06/17] soc: ti: pruss: Add a platform driver for PRUSS in TI SoCs

2018-11-22 Thread Roger Quadros
From: Suman Anna The PRUSS platform driver deals with the overall PRUSS and is used for managing the subsystem level resources like various memories. It is responsible for the creation and deletion of the platform devices for the child PRU devices and other child devices (Interrupt Controller or

[PATCH 07/17] soc: ti: pruss: enable OCP master ports in SYSCFG always

2018-11-22 Thread Roger Quadros
From: Suman Anna The PRUSS CFG module's SYSCFG register is used for managing the PRCM clock management settings at the PRU-ICSS subsystem level, and is being programmed for the PRCM Idle/Standby protocol properly during probe and remove. The register is also programmed to enable the OCP master po

[PATCH 09/17] soc: ti: pruss: add pruss_{request,release}_mem_region() API

2018-11-22 Thread Roger Quadros
From: "Andrew F. Davis" Add two new API - pruss_request_mem_region() & pruss_release_mem_region(), to the PRUSS platform driver to allow client drivers to acquire and release the common memory resources present within a PRU-ICSS subsystem. This allows the client drivers to directly manipulate the

[PATCH 08/17] soc: ti: pruss: Add a PRUSS irqchip driver for PRUSS interrupts

2018-11-22 Thread Roger Quadros
From: Suman Anna The Programmable Real-Time Unit Subsystem (PRUSS) contains an interrupt controller (INTC) that can handle various system input events and post interrupts back to the device-level initiators. The INTC can support upto 64 input events with individual control configuration and hardw

[PATCH 10/17] soc: ti: pruss_intc: Add API to trigger a PRU sysevent

2018-11-22 Thread Roger Quadros
From: Suman Anna The PRUSS INTC can generate an interrupt to various processor subsystems on the SoC through a set of 64 possible PRU system events. These system events can be used by PRU client drivers or applications for event notifications/signalling between PRUs and MPU or other processors. A

[PATCH 14/17] ARM: OMAP2+: use pdata quirks for PRUSS reset lines on AM335x

2018-11-22 Thread Roger Quadros
From: Suman Anna The omap_device API is needed to perform the reset management for any IP instances with PRCM RSTCTRL registers (hard reset lines). This API is limited to the mach-omap2 layer, and cannot be exposed to drivers layer directly. So use platform data ops and pdata quirks for the PRUSS

[PATCH 05/17] soc: ti: pruss: Configure SYSCFG properly during probe/remove

2018-11-22 Thread Roger Quadros
From: Suman Anna The PRUSS CFG module's SYSCFG register is used for managing the PRCM clock management settings at the PRU-ICSS subsystem level. Add two helper functions pruss_{enable/disable}_module() that programs this SYSCFG register during probe and remove. The register is currently programm

[PATCH 12/17] soc: ti: pruss: add pruss_cfg_read()/update() API

2018-11-22 Thread Roger Quadros
Add two new APIs pruss_cfg_read() and pruss_cfg_update() to the PRUSS platform driver to allow other drivers to read and update the registers in the CFG submodule within the PRUSS. This interface provides a simple way for client drivers without having them to include and parse these syscon nodes w

[PATCH 11/17] soc: ti: pruss: add pruss_get()/put() API

2018-11-22 Thread Roger Quadros
From: Tero Kristo Add two new get and put API, pruss_get() and pruss_put(), to the PRUSS platform driver to allow client drivers to request a handle to a PRUSS device. This handle will be used by client drivers to request various operations of the PRUSS platform driver through additional API that

[PATCH 13/17] soc: ti: pruss: export pruss_intc_configure/unconfigure APIs

2018-11-22 Thread Roger Quadros
The PRU remoteproc driver will need to configure the Interrupt controller based on the application specific interrupt map. Export pruss_intc_configure() and pruss_intc_unconfigure() to linux/pruss.h Signed-off-by: Roger Quadros --- drivers/soc/ti/pruss.h | 21 --- include/linux/

[PATCH 15/17] ARM: dts: AM33xx: Add the PRU-ICSS DT nodes

2018-11-22 Thread Roger Quadros
From: Suman Anna Add the DT nodes for the PRU-ICSS on AM33xx family of SoCs. The AM33xx SoCs contain a single PRU-ICSS instance and is represented by the pruss-soc-bus node and a child PRUSS node. PRU-ICSS is not supported on AM3352 SoC though in the AM33xx family, so the nodes are added in disab

[PATCH 16/17] ARM: dts: AM33xx: Add PRU system events for virtio

2018-11-22 Thread Roger Quadros
From: Suman Anna Two PRU system events "vring" and "kick" have been added to each of the PRU nodes in the PRU-ICSS remote processor subsystem to enable the virtio/rpmsg communication between MPU and that PRU core. The additions are done in the base am33xx.dtsi file, and so are inherited by all th

[PATCH 17/17] ARM: dts: am335x-*: Enable PRU-ICSS nodes

2018-11-22 Thread Roger Quadros
From: Suman Anna The PRU-ICSS processor bus node and subsystem nodes were left in disabled state in the base am33xx.dtsi file. PRU-ICSS is supported on these AM335x boards so enable the PRUSS node. Signed-off-by: Suman Anna Signed-off-by: Roger Quadros --- arch/arm/boot/dts/am335x-bone-common

Re: [PATCH] procfs: fix the output format in /proc/PID/wchan

2018-11-22 Thread Alexey Dobriyan
On Wed, Nov 21, 2018 at 07:28:44PM -0800, Andrew Morton wrote: > On Mon, 19 Nov 2018 19:17:52 +0800 Yafang Shao wrote: > > > Just add the missing newline. > > > > ... > > > > --- a/fs/proc/base.c > > +++ b/fs/proc/base.c > > @@ -370,11 +370,12 @@ static int proc_pid_wchan(struct seq_file *m, struc

Re: [PATCH] modpost: skip ELF local symbols during section mismatch check

2018-11-22 Thread Masahiro Yamada
On Thu, Nov 22, 2018 at 4:11 PM Paul Walmsley wrote: > > > During development of a serial console driver with a gcc 8.2.0 > toolchain for RISC-V, the following modpost warning appeared: > > > WARNING: vmlinux.o(.data+0x19b10): Section mismatch in reference from the > variable .LANCHOR1 to th

Re: [PATCH] ptrace: take into account saved_sigmask in PTRACE_{GET,SET}SIGMASK

2018-11-22 Thread Oleg Nesterov
On 11/19, Andrei Vagin wrote: > > case PTRACE_SETSIGMASK: { > sigset_t new_set; > @@ -962,6 +971,8 @@ int ptrace_request(struct task_struct *child, long > request, > child->blocked = new_set; > spin_unlock_irq(&child->sighand->siglock); > > +

Re: [PATCH v2] PCI: imx: Add imx6sx suspend/resume support

2018-11-22 Thread Lucas Stach
Am Donnerstag, den 22.11.2018, 11:38 + schrieb Lorenzo Pieralisi: > On Wed, Nov 07, 2018 at 01:57:03PM +, Leonard Crestez wrote: > > Enable PCI suspend/resume support on imx6sx socs. This is similar to > > imx7d with a few differences: > > > > * The PM_Turn_Off bit is exposed through an IO

Re: [PATCH 0/7] ACPI HMAT memory sysfs representation

2018-11-22 Thread Anshuman Khandual
On 11/19/2018 11:07 PM, Dave Hansen wrote: > On 11/18/18 9:44 PM, Anshuman Khandual wrote: >> IIUC NUMA re-work in principle involves these functional changes >> >> 1. Enumerating compute and memory nodes in heterogeneous environment >> (short/medium term) > > This patch set _does_ that, thoug

Re: [PATCH 1/2] module: Overwrite st_size instead of st_info

2018-11-22 Thread Dave Martin
On Mon, Nov 19, 2018 at 05:25:12PM +0100, Vincent Whitchurch wrote: > st_info is currently overwritten after relocation and used to store the > elf_type(). However, we're going to need it fix kallsyms on ARM's > Thumb-2 kernels, so preserve st_info and overwrite the st_size field > instead. st_si

[PATCH 2/2] scripts/decode_stacktrace: Only strip base path when a prefix of the path

2018-11-22 Thread Marc Zyngier
Running something like: decodecode vmlinux . leads to interested results where not only the leading "." gets stripped from the displayed paths, but also anywhere in the string, displaying something like: kvm_vcpu_check_block (arch/arm64/kvm/virt/kvm/kvm_mainc:2141) which doesn't

[PATCH 0/2] scripts/decode{code,_stacktrace} fixes

2018-11-22 Thread Marc Zyngier
Here's a couple of fixes that have been languishing in my tree for a few months. The first one is for an issue that can only be observed using decodecode natively on arm/arm64m and resulting in a pretty useless disassembly. The second one fixes the path trimming in decode_stacktrace, and probably

[PATCH 1/2] scripts/decodecode: Set ARCH when running natively on arm/arm64

2018-11-22 Thread Marc Zyngier
When running decodecode natively on arm64, ARCH is likely not to be set, and we end-up with .4byte instead of .inst when generating the disassembly. Similar effects would occur if running natively on a 32bit ARM platform, although that's even less popular. A simple workaround is to populate ARCH

Re: [PATCH 2/2] exec: increase BINPRM_BUF_SIZE to 256

2018-11-22 Thread Oleg Nesterov
On 11/16, Alan Cox wrote: > > On Mon, 12 Nov 2018 17:09:56 +0100 > Oleg Nesterov wrote: > > > Large enterprise clients often times run applications out of networked > > file systems where the IT mandated layout of project volumes can end up > > leading to paths that are longer than 128 characters.

[PATCH v2 0/3] Bug fixes for Qualcomm BT chip wcn3990

2018-11-22 Thread Balakrishna Godavarthi
The below issues are found in the recent testing. 1. Observed device is not going into off state or not responding. As wcn3990 require a power pulses to turn on the irrespctive of igniting regulators, it was observed that power on or power off pulses are not in sync with respective to

[PATCH v2 3/3] Bluetooth: hci_qca: Fix frame reassembly errors for wcn3990

2018-11-22 Thread Balakrishna Godavarthi
During initalization of wcn3990, we observed UART is reading some stray bytes on the Rx line. This is logging Frame reassembly errors on the serial console. This could be because of tristate of Tx line of wcn3990 during boot up. [ 176.929612] Bluetooth: hci_qca.c:qca_recv() hci0: Frame reassembly

[PATCH v2 1/3] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-11-22 Thread Balakrishna Godavarthi
wcn3990 requires a power pulse to turn ON/OFF along with regulators. Sometimes we are observing the power pulses are sent out with some time delay, due to queuing these commands. This is causing synchronization issues with chip, which intern delay the chip setup or may end up with communication iss

[PATCH v2 2/3] Bluetooth: hci_qca: Deassert RTS while baudrate change command

2018-11-22 Thread Balakrishna Godavarthi
This patch will help to stop frame reassembly errors while changing the baudrate. This is because host send a change baudrate request command to the chip with 115200 bps, Whereas chip will change their UART clocks to the enable for new baudrate and sends the response for the change request command

Re: [PATCH v2] UAPI: move RENAME_* definitions to separated file

2018-11-22 Thread Yury Norov
Ping? On Wed, Nov 14, 2018 at 07:26:07AM +, Yury Norov wrote: > Hi all, > > It seems that I forgot to rename guards in v2. This is the correct > version. My apologize for noise. > > Yury > > >From 61215b1cb559e69fb57f2a82864e72e3a1ef4b12 Mon Sep 17 00:00:00 2001 > From: Yury Norov > Date:

Re: [PATCH 1/2] module: Overwrite st_size instead of st_info

2018-11-22 Thread Vincent Whitchurch
On Thu, Nov 22, 2018 at 12:01:54PM +, Dave Martin wrote: > On Mon, Nov 19, 2018 at 05:25:12PM +0100, Vincent Whitchurch wrote: > > st_info is currently overwritten after relocation and used to store the > > elf_type(). However, we're going to need it fix kallsyms on ARM's > > Thumb-2 kernels,

Re: [patch 22/24] x86/speculation: Create PRCTL interface to restrict indirect branch speculation

2018-11-22 Thread Borislav Petkov
On Wed, Nov 21, 2018 at 09:14:52PM +0100, Thomas Gleixner wrote: > From: Tim Chen > > Add the PR_SPEC_INDIR_BRANCH option for the PR_GET_SPECULATION_CTRL and > PR_SET_SPECULATION_CTRL prctls to allow fine grained per task control of > indirect branch speculation via STIBP. > > Invocations: > Ch

Re: [patch 22/24] x86/speculation: Create PRCTL interface to restrict indirect branch speculation

2018-11-22 Thread Peter Zijlstra
On Thu, Nov 22, 2018 at 01:26:38PM +0100, Borislav Petkov wrote: > Perhaps merge the two DISABLE branches to make it obvious what the > difference between them is: > > diff --git a/arch/x86/kernel/cpu/bugs.c b/arch/x86/kernel/cpu/bugs.c > index 6eac074e3935..28cece3a067b 100644 > --- a/arch/x86/ke

[tip:x86/cleanups 1/2] arch/arm/include/asm/efi.h:70:20: error: static declaration of 'efifb_setup_from_dmi' follows non-static declaration

2018-11-22 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/cleanups head: c382986fe178d02abdfcd6f4e7b9783c2922872c commit: d37904c5b14317a2c76efec6b9e4dbcaa17380e5 [1/2] x86/headers: Fix -Wmissing-prototypes warning config: arm-multi_v4t_defconfig (attached as .config) compiler: arm

Re: [Question] smp_wmb() in prepare_uprobe()

2018-11-22 Thread Oleg Nesterov
Hi, On 11/21, Andrea Parri wrote: > > The comment for the smp_wmb() in prepare_uprobe() says: > > "pairs with rmb() in find_active_uprobe()" it seems that this comment was wrong from the very beginning, > but I see no (smp_)rmb() in find_active_uprobe(); I see the smp_rmb() in > handle_swbp()

[PATCH V2 1/2] base/drivers/arch_topology: Replace mutex with READ_ONCE / WRITE_ONCE

2018-11-22 Thread Daniel Lezcano
The mutex protects a per_cpu variable access. The potential race can happen only when the cpufreq governor module is loaded and at the same time the cpu capacity is changed in the sysfs. There is no real interest of using a mutex to protect a variable assignation when there is no situation where a

[PATCH] mtd: spi-nor: fix erase_type array to indicate current map conf

2018-11-22 Thread Tudor.Ambarus
From: Tudor Ambarus Bug reported for the out-of-tree S25FS128S flash memory. BFPT table advertises all the erase types supported by all the possible map configurations. Update the erase_type array to indicate which erase types are applicable to the current map configuration. Backward compatibil

[PATCH V2 2/2] base/drivers/arch_topology: Default dmips-mhz if they are not set in DT

2018-11-22 Thread Daniel Lezcano
In the case of asymmetric SoC with the same micro-architecture, we have a group of CPUs with smaller OPPs than the other group. One example is the 96boards dragonboard 820c. There is no dmips/MHz difference between both groups, so no need to specify the values in the DT. Unfortunately, without thes

Re: [PATCH] mtd: spi-nor: fix erase_type array to indicate current map conf

2018-11-22 Thread Tudor.Ambarus
Hi, Alexander, Can you please test this patch to see if it fixes your problem? Thanks, ta On 11/22/2018 02:36 PM, Tudor Ambarus - M18064 wrote: > From: Tudor Ambarus > > Bug reported for the out-of-tree S25FS128S flash memory. > > BFPT table advertises all the erase types supported by all the

[PATCH v2] ata: pata_macio: add of_node_put()

2018-11-22 Thread Yangtao Li
of_find_node_by_path() acquires a reference to the node returned by it and that reference needs to be dropped by its caller. This place doesn't do that, so fix it. Signed-off-by: Yangtao Li --- Changes in v2: -update changelog --- drivers/ata/pata_macio.c | 2 ++ 1 file changed, 2 insertions(+)

Re: [RFC][PATCH 00/14] function_graph: Rewrite to allow multiple users

2018-11-22 Thread Steven Rostedt
On Thu, 22 Nov 2018 11:08:12 +0100 Peter Zijlstra wrote: > On Wed, Nov 21, 2018 at 08:27:08PM -0500, Steven Rostedt wrote: > > Well the fuction graph tracer is arguably the strongest of the tracers. > > It shows both the entrance and exit of a function, can give the timings > > of a function, and

[PATCH] soc: ti: QMSS: Change to use DEFINE_SHOW_ATTRIBUTE macro

2018-11-22 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/soc/ti/knav_qmss_queue.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/drivers/soc/ti/knav_qmss_queue.c b/drivers/soc/ti/knav_qmss_queue.c index 8b418379272d..67f7c00

[PATCH v1] Bluetooth: hci_serdev: Remove setting of HCI_QUIRK_RESET_ON_CLOSE.

2018-11-22 Thread Balakrishna Godavarthi
HCI_QUIRK_RESET_ON_CLOSE quirk is required for BT v1.0 based devices, to send a reset command to the chip during hci device close. Serdev architecture is used for the latest BT chips, which doesn't require to send the reset command during close. If still chips required reset command during close, i

[PATCH] staging: iio: ad5933: replaced kfifo by triggered_buffer

2018-11-22 Thread Marcelo Schmitt
Previously, there was an implicit creation of a kfifo which was replaced by a call to triggered_buffer_setup, which is already implemented in iio infrastructure. Signed-off-by: Marcelo Schmitt --- .../staging/iio/impedance-analyzer/Kconfig| 2 +- .../staging/iio/impedance-analyzer/ad5933.c

[PATCH] soc: sunxi: Change to use DEFINE_SHOW_ATTRIBUTE macro

2018-11-22 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/soc/sunxi/sunxi_sram.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/soc/sunxi/sunxi_sram.c b/drivers/soc/sunxi/sunxi_sram.c index b4b0f3480bd3..71e3ee4a3f19 100

Re: [PATCH] module: make it clearer when we're handling kallsyms symbols vs exported symbols

2018-11-22 Thread Jessica Yu
+++ Miroslav Benes [22/11/18 11:19 +0100]: On Wed, 21 Nov 2018, Jessica Yu wrote: The module loader internally works with both exported symbols represented as struct kernel_symbol, as well as Elf symbols from a module's symbol table. It's hard to distinguish sometimes which type of symbol we're

Re: [PATCH v2 1/3] thermal: tegra: continue if sensor register fails

2018-11-22 Thread Daniel Lezcano
On 22/11/2018 08:10, Wei Ni wrote: > > > On 21/11/2018 8:51 PM, Daniel Lezcano wrote: >> On 21/11/2018 11:23, Wei Ni wrote: >>> >>> >>> On 21/11/2018 4:55 PM, Daniel Lezcano wrote: On 13/11/2018 11:06, Wei Ni wrote: > Don't bail when a sensor fails to register with the > thermal zone

[PATCH] soc: ti: Change to use DEFINE_SHOW_ATTRIBUTE macro

2018-11-22 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/soc/ti/knav_dma.c | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/drivers/soc/ti/knav_dma.c b/drivers/soc/ti/knav_dma.c index e05ab16d9a9e..2a66f36b9fd6 100644 --- a/

Re: [PATCH 2/2] scripts/decode_stacktrace: Only strip base path when a prefix of the path

2018-11-22 Thread Mark Rutland
On Thu, Nov 22, 2018 at 12:14:40PM +, Marc Zyngier wrote: > Running something like: > > decodecode vmlinux . > > leads to interested results where not only the leading "." gets stripped > from the displayed paths, but also anywhere in the string, displaying > something like: > >

[PATCH] soc/tegra: pmc: Change to use DEFINE_SHOW_ATTRIBUTE macro

2018-11-22 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/soc/tegra/pmc.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c index 1fa840e3d930..d42a1f5ced82 100644 --- a/drivers/soc

Re: [PATCHv3] panic: avoid deadlocks in re-entrant console drivers

2018-11-22 Thread Petr Mladek
On Thu 2018-11-01 09:08:08, Petr Mladek wrote: > On Thu 2018-11-01 10:48:21, Sergey Senozhatsky wrote: > > On (10/31/18 13:27), Petr Mladek wrote: > > > > > > > > Signed-off-by: Sergey Senozhatsky > > > > > > The patch makes sense to me. The locks should stay busted also for > > > console_flush_

Re: [PATCH v3 3/3] ALSA: hda: add support for Huawei WMI micmute LED

2018-11-22 Thread Pavel Machek
On Tue 2018-11-20 13:19:26, Takashi Iwai wrote: > On Tue, 20 Nov 2018 12:51:59 +0100, > Pavel Machek wrote: > > > > Hi! > > > > > > You have general-purpose LED, yet you are treating it as "something > > > > special". That means ugly code (quoted above) and lack of flexibility. > > > > > > > > F

Re: [PATCH v3 3/3] ALSA: hda: add support for Huawei WMI micmute LED

2018-11-22 Thread Takashi Iwai
On Thu, 22 Nov 2018 14:12:16 +0100, Pavel Machek wrote: > > On Tue 2018-11-20 13:19:26, Takashi Iwai wrote: > > On Tue, 20 Nov 2018 12:51:59 +0100, > > Pavel Machek wrote: > > > > > > Hi! > > > > > > > > You have general-purpose LED, yet you are treating it as "something > > > > > special". That

Re: [PATCH v3 3/3] ALSA: hda: add support for Huawei WMI micmute LED

2018-11-22 Thread Pavel Machek
On Thu 2018-11-22 13:36:43, Andy Shevchenko wrote: > On Tue, Nov 20, 2018 at 2:19 PM Takashi Iwai wrote: > > > > On Tue, 20 Nov 2018 12:51:59 +0100, > > Pavel Machek wrote: > > > > > > Hi! > > > > > > > > You have general-purpose LED, yet you are treating it as "something > > > > > special". That

Re: [PATCH] clocksource/drivers/timer-imx-tpm: convert the driver to timer-of

2018-11-22 Thread Daniel Lezcano
Hi Anson, On 06/11/2018 06:15, Anson Huang wrote: > Convert the driver to use the timer_of helpers. > This allows to handle timer base, clock and irq using > common timer_of driver and it simplifies the code. Can you do an extra move by passing the timer_of structure around for: - tpm_timer_d

Re: [PATCH 2/7] node: Add heterogenous memory performance

2018-11-22 Thread Anshuman Khandual
On 11/19/2018 09:16 PM, Keith Busch wrote: > On Mon, Nov 19, 2018 at 09:05:07AM +0530, Anshuman Khandual wrote: >> On 11/15/2018 04:19 AM, Keith Busch wrote: >>> Heterogeneous memory systems provide memory nodes with latency >>> and bandwidth performance attributes that are different from other

Re: [PATCH 4/7] node: Add memory caching attributes

2018-11-22 Thread Anshuman Khandual
On 11/20/2018 04:36 AM, Keith Busch wrote: > On Mon, Nov 19, 2018 at 09:44:00AM +0530, Anshuman Khandual wrote: >> On 11/15/2018 04:19 AM, Keith Busch wrote: >>> System memory may have side caches to help improve access speed. While >>> the system provided cache is transparent to the software ac

Re: [PATCH] procfs: fix the output format in /proc/PID/wchan

2018-11-22 Thread Yafang Shao
On Thu, Nov 22, 2018 at 7:40 PM Alexey Dobriyan wrote: > > On Wed, Nov 21, 2018 at 07:28:44PM -0800, Andrew Morton wrote: > > On Mon, 19 Nov 2018 19:17:52 +0800 Yafang Shao wrote: > > > > > Just add the missing newline. > > > > > > ... > > > > > > --- a/fs/proc/base.c > > > +++ b/fs/proc/base.c >

Re: [PATCH] procfs: fix the output format in /proc/PID/wchan

2018-11-22 Thread Alexey Dobriyan
On Thu, Nov 22, 2018 at 09:29:52PM +0800, Yafang Shao wrote: > On Thu, Nov 22, 2018 at 7:40 PM Alexey Dobriyan wrote: > > > > On Wed, Nov 21, 2018 at 07:28:44PM -0800, Andrew Morton wrote: > > > On Mon, 19 Nov 2018 19:17:52 +0800 Yafang Shao > > > wrote: > > > > > > > Just add the missing newlin

[PATCH] fs/xfs: fix f_ffree value for statfs when project quota is set

2018-11-22 Thread 尹烨
When project is set, we should use inode limit minus the used count Signed-off-by: Ye Yin --- fs/xfs/xfs_qm_bhv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/xfs/xfs_qm_bhv.c b/fs/xfs/xfs_qm_bhv.c index 73a1d77ec187..3091e4bc04ef 100644 --- a/fs/xfs/xfs_qm_bhv.c +++ b/

Re: [PATCH v15 1/2] Reorganize the oom report in dump_header

2018-11-22 Thread Michal Hocko
On Wed 21-11-18 19:29:58, ufo19890...@gmail.com wrote: > From: yuzhoujian > > OOM report contains several sections. The first one is the allocation > context that has triggered the OOM. Then we have cpuset context > followed by the stack trace of the OOM path. The tird one is the OOM > memory inf

Re: [PATCH v15 2/2] Add oom victim's memcg to the oom context information

2018-11-22 Thread Michal Hocko
On Wed 21-11-18 19:29:59, ufo19890...@gmail.com wrote: > From: yuzhoujian > > The current oom report doesn't display victim's memcg context during the > global OOM situation. While this information is not strictly needed, it > can be really helpful for containerized environments to locate which >

Re: [RFC][PATCH 00/14] function_graph: Rewrite to allow multiple users

2018-11-22 Thread Peter Zijlstra
On Thu, Nov 22, 2018 at 07:46:05AM -0500, Steven Rostedt wrote: > On Thu, 22 Nov 2018 11:08:12 +0100 > Peter Zijlstra wrote: > > > On Wed, Nov 21, 2018 at 08:27:08PM -0500, Steven Rostedt wrote: > > > Well the fuction graph tracer is arguably the strongest of the tracers. > > > It shows both the

Re: [PATCH v3 3/3] ALSA: hda: add support for Huawei WMI micmute LED

2018-11-22 Thread Takashi Iwai
On Thu, 22 Nov 2018 14:18:02 +0100, Pavel Machek wrote: > > On Thu 2018-11-22 13:36:43, Andy Shevchenko wrote: > > On Tue, Nov 20, 2018 at 2:19 PM Takashi Iwai wrote: > > > > > > On Tue, 20 Nov 2018 12:51:59 +0100, > > > Pavel Machek wrote: > > > > > > > > Hi! > > > > > > > > > > You have general

Re: [Question] smp_wmb() in prepare_uprobe()

2018-11-22 Thread Andrea Parri
On Thu, Nov 22, 2018 at 01:36:56PM +0100, Oleg Nesterov wrote: > Hi, > > On 11/21, Andrea Parri wrote: > > > > The comment for the smp_wmb() in prepare_uprobe() says: > > > > "pairs with rmb() in find_active_uprobe()" > > it seems that this comment was wrong from the very beginning, > > > > b

[PATCH 7/8] staging: rtl8188eu: cleanup declarations in usb_ops_linux.c

2018-11-22 Thread Michael Straube
Replace tabs with spaces and/or remove spaces in declarations to cleanup whitespace. Signed-off-by: Michael Straube --- .../staging/rtl8188eu/os_dep/usb_ops_linux.c | 40 +-- 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/drivers/staging/rtl8188eu/os_dep/usb_ops

[PATCH 4/8] staging: rtl8188eu: correct spelling mistake in a comment

2018-11-22 Thread Michael Straube
Correct spelling mistake in a comment reported by checkpatch. checksumed -> checksummed Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c b/driv

[PATCH 5/8] staging: rtl8188eu: remove unnecessary parentheses

2018-11-22 Thread Michael Straube
Remove unnecessary parentheses in usb_ops_linux.c. Reported by checkpatch. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c b/drivers/staging

[PATCH 1/8] staging: rtl8188eu: use __func__ in usb_ops_linux.c

2018-11-22 Thread Michael Straube
Use __func__ instead of hardcoded function names. Reported by checkpatch. Signed-off-by: Michael Straube --- .../staging/rtl8188eu/os_dep/usb_ops_linux.c | 84 --- 1 file changed, 53 insertions(+), 31 deletions(-) diff --git a/drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c b

linux-kernel@vger.kernel.org

2018-11-22 Thread Michael Straube
Add spaces around '>>' and '&' to follow kernel coding style. Reported by checkpatch. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8188eu/os_dep/usb_ops_

[PATCH 8/8] staging: rtl8188eu: remove variable from rtl8188eu_xmit_tasklet()

2018-11-22 Thread Michael Straube
The local variable ret is only used to test the return value of the call to rtl8188eu_xmitframe_complete(). Use the function directly in the if test and remove the variable ret. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c | 5 + 1 file changed, 1 inser

[PATCH 2/8] staging: rtl8188eu: cleanup line ending with a '('

2018-11-22 Thread Michael Straube
Cleanup line ending with a '(' to follow kernel coding style. Reported by checkpatch. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/staging/rtl8188eu/os_dep/usb_ops_linu

[PATCH 3/8] staging: rtl8188eu: remove braces from single if else statement

2018-11-22 Thread Michael Straube
Remove braces from single line if else statement. Reported by checkpatch. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c b/drivers/stagi

Re: [PATCH 3/4] mm: Reclaim small amounts of memory when an external fragmentation event occurs

2018-11-22 Thread Vlastimil Babka
On 11/21/18 11:14 AM, Mel Gorman wrote: > An external fragmentation event was previously described as > > When the page allocator fragments memory, it records the event using > the mm_page_alloc_extfrag event. If the fallback_order is smaller > than a pageblock order (order-9 on 64-bit

Re: [PATCH v9 13/15] sched/fair: Introduce an energy estimation helper function

2018-11-22 Thread Peter Zijlstra
On Wed, Nov 21, 2018 at 04:05:27PM +, Quentin Perret wrote: > On Wednesday 21 Nov 2018 at 15:28:03 (+0100), Peter Zijlstra wrote: > > On Mon, Nov 19, 2018 at 02:18:55PM +, Quentin Perret wrote: > > > +static long > > > +compute_energy(struct task_struct *p, int dst_cpu, struct perf_domain

[PATCH] perf tests record: Allow for 'sleep' being 'coreutils'

2018-11-22 Thread Adrian Hunter
If the 'sleep' command is provided by coreutils, then the "PERF_RECORD_* events & perf_sample fields" test will fail because the MMAP name is 'coreutils' not 'sleep', and there is an extra COMM event. Fix the test to detect that case. Signed-off-by: Adrian Hunter --- tools/perf/tests/perf-record

[PATCH] gpio: restore original GPLv2+ license of gpiolib-of.c sources

2018-11-22 Thread Vladimir Zapolskiy
It's easy to verify that the change of drivers/gpio/gpiolib-of.c license header to SPDX standard changes the license from GPLv2+ to GPLv2, and this change corrects it. Fixes: dae5f0afcfc3 ("gpio: Use SPDX header for core library") Signed-off-by: Vladimir Zapolskiy --- drivers/gpio/gpiolib-of.c |

Re: [PATCH v9 08/15] sched/topology: Make Energy Aware Scheduling depend on schedutil

2018-11-22 Thread Peter Zijlstra
On Mon, Nov 19, 2018 at 02:18:50PM +, Quentin Perret wrote: > + if (rd->pd) > + pr_warn("rd %*pbl: Disabling EAS, schedutil is > mandatory\n", > + cpumask_pr_args(cpu_map)); > + goto

[PATCH 21/24] iommu/vt-d: remove the mapping_error dma_map_ops method

2018-11-22 Thread Christoph Hellwig
Return DMA_MAPPING_ERROR instead of 0 on a dma mapping failure and let the core dma-mapping code handle the rest. Signed-off-by: Christoph Hellwig --- drivers/iommu/intel-iommu.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/drivers/iommu/intel-iommu.c b/driver

[PATCH] firmware: ti_sci: Change to use DEFINE_SHOW_ATTRIBUTE macro

2018-11-22 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/firmware/ti_sci.c | 21 ++--- 1 file changed, 2 insertions(+), 19 deletions(-) diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c index 69ed1464175c..3fbbb61012c4 100644 -

[PATCH 13/24] alpha: remove the mapping_error dma_map_ops method

2018-11-22 Thread Christoph Hellwig
Return DMA_MAPPING_ERROR instead of 0 on a dma mapping failure and let the core dma-mapping code handle the rest. Signed-off-by: Christoph Hellwig --- arch/alpha/kernel/pci_iommu.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/arch/alpha/kernel/pci_iommu.c b

[PATCH 23/24] xen-swiotlb: remove the mapping_error dma_map_ops method

2018-11-22 Thread Christoph Hellwig
Return DMA_MAPPING_ERROR instead of 0 on a dma mapping failure and let the core dma-mapping code handle the rest. Signed-off-by: Christoph Hellwig --- drivers/xen/swiotlb-xen.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/xen/swiotlb-xen.c b/drivers/x

[PATCH] perf test: Fix perf_event_attr test failure

2018-11-22 Thread Adrian Hunter
Fix inconsistent use of tabs and spaces error: # perf test 16 -v 16: Setup struct perf_event_attr : --- start --- test child forked, pid 20224 File "/usr/libexec/perf-core/tests/attr.py", line 119 log.warning("expected %s=%s, got %s" % (t, self[t], other[

Re: [PATCH v2 1/1] arm64: Early boot time stamps

2018-11-22 Thread Marc Zyngier
On Wed, 21 Nov 2018 17:58:41 +, Pavel Tatashin wrote: > > On 18-11-21 17:47:07, Will Deacon wrote: > > > + /* > > > + * The arm64 boot protocol mandates that CNTFRQ_EL0 reflects > > > + * the timer frequency. To avoid breakage on misconfigured > > > + * systems, do not register the early s

Re: [PATCH] clocksource/drivers/integrator-ap: add of_node_put()

2018-11-22 Thread Daniel Lezcano
Hi Yangtao, On 20/11/2018 14:18, Yangtao Li wrote: > use of_node_put() to release the refcount. In future please give a better description. No need to resend, I'll do the change. Linus? I'm about to take this patch. Is it fine for you? > Signed-off-by: Yangtao Li > --- > drivers/clocksourc

[PATCH] s390: Remove obsolete bust_spinlock() implementation

2018-11-22 Thread Petr Mladek
The commit cefc8be82403cf ("Consolidate bust_spinlocks()") kept the s390-specific implementation because of the absence of CONFIG_VT. In fact, the only difference was calling console_unblank() instead of unblank_screen(). The common implementation in lib/bust_spinlocks.c started to call unblank_sc

Re: [PATCH v2] regulator: wm8994: Don't use devres for enable GPIOs

2018-11-22 Thread Linus Walleij
On Wed, Nov 21, 2018 at 11:13 AM Charles Keepax wrote: > The regulator core takes over managing the lifetime of the enable GPIO > once the regulator is registered. As such we shouldn't register the > enable GPIO using devm, or it will be freed twice. > > Reported-by: Marek Szyprowski > Signed-of

Re: [PATCH v2] regulator: wm8994: Don't use devres for enable GPIOs

2018-11-22 Thread Linus Walleij
On Wed, Nov 21, 2018 at 11:42 AM Marek Szyprowski wrote: > Linus, Mark: Similar issue is probably in the other regulator drivers, > which use enable GPIO allocated by devm_gpio_get*(). This driver is > simply the first one, which we observed it. It would be great if one > would take a look into r

Re: [PATCH v5 04/15] sched/core: uclamp: add CPU's clamp groups refcounting

2018-11-22 Thread Patrick Bellasi
On 13-Nov 07:11, Patrick Bellasi wrote: > On 11-Nov 17:47, Peter Zijlstra wrote: > > On Mon, Oct 29, 2018 at 06:32:59PM +, Patrick Bellasi wrote: [...] > > > + /* Both min and max clamps are MAX aggregated */ > > > + if (max_value < rq->uclamp.group[clamp_id][group_id].value)

Re: [PATCH] clocksource/drivers/integrator-ap: add of_node_put()

2018-11-22 Thread Linus Walleij
On Tue, Nov 20, 2018 at 2:19 PM Yangtao Li wrote: > use of_node_put() to release the refcount. > > Signed-off-by: Yangtao Li Acked-by: Linus Walleij Yours, Linus Walleij

Re: [RCF PATCH,v2,2/2] pwm: imx: Configure output to GPIO in disabled state

2018-11-22 Thread Vokáč Michal
Ahoj Uwe, On 20.11.2018 17:54, Uwe Kleine-König wrote: > On Tue, Nov 20, 2018 at 01:14:33PM +, Vokáč Michal wrote: >> On 16.11.2018 10:51, Thierry Reding wrote: >>> On Thu, Nov 15, 2018 at 09:37:33PM +0100, Uwe Kleine-König wrote: On Thu, Nov 15, 2018 at 04:25:45PM +0100, Thierry Reding w

Re: [PATCH] clocksource/drivers/integrator-ap: add of_node_put()

2018-11-22 Thread Daniel Lezcano
On 20/11/2018 14:18, Yangtao Li wrote: > use of_node_put() to release the refcount. > > Signed-off-by: Yangtao Li Applied, thanks! -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook |

[PATCH] power: supply: axp288_fuel_gauge: Change to use DEFINE_SHOW_ATTRIBUTE macro

2018-11-22 Thread Yangtao Li
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Yangtao Li --- drivers/power/supply/axp288_fuel_gauge.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/drivers/power/supply/axp288_fuel_gauge.c b/drivers/power/supply/axp288_fuel_gauge.c in

Re: [PATCH] clocksource/drivers/integrator-ap: add of_node_put()

2018-11-22 Thread Daniel Lezcano
On 22/11/2018 15:22, Linus Walleij wrote: > Acked-by: Linus Walleij > Thanks! -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter |

Re: [tip:x86/mm] x86/fault: Clean up the page fault oops decoder a bit

2018-11-22 Thread Andy Lutomirski
On Thu, Nov 22, 2018 at 2:13 AM tip-bot for Ingo Molnar wrote: > > Commit-ID: a2aa52ab16efbee40ad118ebac4a5e438f5b43ee > Gitweb: > https://git.kernel.org/tip/a2aa52ab16efbee40ad118ebac4a5e438f5b43ee > Author: Ingo Molnar > AuthorDate: Thu, 22 Nov 2018 09:34:03 +0100 > Committer: Ingo M

Re: [PATCH v4 1/3] kernel.h: disable type-checks in container_of() for Sparse

2018-11-22 Thread Luc Van Oostenryck
On Thu, Nov 22, 2018 at 12:14:20PM +0900, Masahiro Yamada wrote: > When I tried to enable BUILD_BUG_ON for Sparse, the kbuild test robot > reported lots of "unknown expression" warnings from container_of(), > which seemed false positive. > > I addressed this in [1], but fixing Sparse is the right

<    1   2   3   4   5   6   >