[PATCH 2/2 V2] power:sysfs: Expose device wakeup_event_count.

2019-07-22 Thread Ravi Chandra Sadineni
Device level event_count can help user level daemon to track if a praticular device has seen an wake interrupt during a suspend resume cycle. Thus expose it via sysfs. Signed-off-by: Ravi Chandra Sadineni --- V2: Address comments from patchset 1. Documentation/ABI/testing/sysfs-devices-power |

Issue with sequence to switch to HS400

2019-07-22 Thread Alan Cooper
I'm having a problem with a new SD/MMC controller and PHY in our latest SoC's. The issue I'm seeing is that I can't switch into HS400 mode. This looks like something the driver is doing that doesn't meet the JEDEC spec. In the "HS400 timing mode selection" section of the JEDEC spec , in step 7 it

Re: [PATCH 3/3] MIPS: qi_lb60: Move MMC configuration to devicetree

2019-07-22 Thread Paul Burton
Hello, Paul Cercueil wrote: > Move the MMC configuration from the board C file to devicetree. > > The 'power' GPIO was removed and instead the vmmc regulator is used, > to follow the changes introduced in the jz4740-mmc driver. > > Signed-off-by: Paul Cercueil Applied to mips-next. Thanks,

[PATCH] drm/exynos: fix missing decrement of retry counter

2019-07-22 Thread Colin King
From: Colin Ian King Currently the retry counter is not being decremented, leading to a potential infinite spin if the scalar_reads don't change state. Addresses-Coverity: ("Infinite loop") Fixes: 280e54c9f614 ("drm/exynos: scaler: Reset hardware before starting the operation") Signed-off-by:

Re: [PATCH 2/2] MIPS: dts: ingenic: Add 'cpus' node

2019-07-22 Thread Paul Burton
Hi Paul, On Mon, Jul 22, 2019 at 01:55:48PM -0400, Paul Cercueil wrote: > Add 'cpus' node to the jz4740.dtsi, jz4770.dtsi, jz4780.dtsi files. What's the motivation for this? If it's to silence the "cacheinfo: Unable to detect cache hierarchy" messages, does commit b8bea8a5e5d9 ("mips: fix

Re: [PATCH] checkpatch: Added warnings in favor of strscpy().

2019-07-22 Thread Jonathan Corbet
On Mon, 22 Jul 2019 15:24:33 -0700 Joe Perches wrote: > > If the functions themselves are fully defined in the .h file, I'd just add > > the kerneldoc there as well. That's how it's usually done, and you want > > to keep the documentation and the prototypes together. > > In this case, it's a

Re: [PATCH] checkpatch: Added warnings in favor of strscpy().

2019-07-22 Thread Joe Perches
On Mon, 2019-07-22 at 15:57 -0600, Jonathan Corbet wrote: > On Mon, 22 Jul 2019 14:50:09 -0700 > Joe Perches wrote: > > > On Mon, 2019-07-22 at 23:01 +0200, Stephen Kitt wrote: > > > How about you submit your current patch set, and I follow up with the > > > above > > > adapted to stracpy? >

Re: [PATCH][next] clk: Si5341/Si5340: remove redundant assignment to n_den

2019-07-22 Thread Stephen Boyd
Quoting Christophe JAILLET (2019-07-22 15:00:24) > > I don't use it explicitly, but the suggestions I get include some git > history, so I guess that it is on by default. > > I was thinking at parsing files to see if MODULE_AUTHOR includes an email. > Ok. Feel free to write a patch. Just know

linux-next: build warning after merge of the char-misc.current tree

2019-07-22 Thread Stephen Rothwell
Hi all, After merging the char-misc.current tree, today's linux-next build (arm multi_v7_defconfig) produced this warning: In file included from drivers/base/firmware_loader/main.c:41: drivers/base/firmware_loader/firmware.h:145:12: warning: 'fw_map_paged_buf' defined but not used

Re: [PATCH 39/79] libperf: Add perf_evlist__add function

2019-07-22 Thread Jiri Olsa
On Mon, Jul 22, 2019 at 04:09:35PM -0300, Arnaldo Carvalho de Melo wrote: SNIP > > Right after applying this patch my alias for building perf crashes: > > I.e. I have: > > alias m='perf stat -e cycles:u,instructions:u make -k O=/tmp/build/perf -C > tools/perf install-bin' > > And I'm

Re: Linux 5.3-rc1

2019-07-22 Thread Guenter Roeck
On Sun, Jul 21, 2019 at 02:33:38PM -0700, Linus Torvalds wrote: [ ... ] > > Go test, > Things looked pretty good until a few days ago. Unfortunately, the last few days brought in a couple of issues. riscv:virt:defconfig:scsi[virtio] riscv:virt:defconfig:scsi[virtio-pci] Boot tests crash with

Re: Resend: [PATCH] ktest: Fix typos in sample.conf

2019-07-22 Thread Randy Dunlap
On 7/22/19 3:17 PM, Masanari Iida wrote: > This patch fixes some spelling typos in sample.conf. > > Signed-off-by: Masanari Iida Acked-by: Randy Dunlap Thanks. > --- > tools/testing/ktest/sample.conf | 20 ++-- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff

Resend: [PATCH] ktest: Fix typos in sample.conf

2019-07-22 Thread Masanari Iida
This patch fixes some spelling typos in sample.conf. Signed-off-by: Masanari Iida --- tools/testing/ktest/sample.conf | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/tools/testing/ktest/sample.conf b/tools/testing/ktest/sample.conf index

Re: [PATCH 38/79] libperf: Add perf_evlist__init function

2019-07-22 Thread Jiri Olsa
On Mon, Jul 22, 2019 at 04:39:25PM -0300, Arnaldo Carvalho de Melo wrote: SNIP > > I'm applying it to fix this issue and avoid a bisection break. I'm now > going to run 'perf test' after each cset too. And probably the next cset > has this issue as well, i.e. reordering of initialization in the

Re: [RFC 07/11] dt-bindings: media: meson-ao-cec: add SM1 compatible

2019-07-22 Thread Rob Herring
On Mon, 1 Jul 2019 12:47:01 +0200, Neil Armstrong wrote: > Add AO-CEC compatible string for the Amlogic SM1 SoC family, > a derivate of the G12A AO-CECB controller. > > Signed-off-by: Neil Armstrong > --- > Documentation/devicetree/bindings/media/meson-ao-cec.txt | 8 +--- > 1 file

Re: [RFC 05/11] dt-bindings: soc: amlogic: clk-measure: Add SM1 compatible

2019-07-22 Thread Rob Herring
On Mon, 1 Jul 2019 12:46:59 +0200, Neil Armstrong wrote: > Add the Amlogic SM1 Compatible for the clk-measurer IP. > > Signed-off-by: Neil Armstrong > --- > Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt | 1 + > 1 file changed, 1 insertion(+) > Reviewed-by: Rob Herring

Re: [PATCH v2 2/2] x86/purgatory: use CFLAGS_REMOVE rather than reset KBUILD_CFLAGS

2019-07-22 Thread Nick Desaulniers
On Mon, Jul 22, 2019 at 2:33 PM Nick Desaulniers wrote: > > KBUILD_CFLAGS is very carefully built up in the top level Makefile, > particularly when cross compiling or using different build tools. > Resetting KBUILD_CFLAGS via := assignment is an antipattern. > > The comment above the reset

Re: [PATCH v10 03/12] dt-binding: gce: add binding for gce client reg property

2019-07-22 Thread Rob Herring
On Mon, Jul 01, 2019 at 03:48:33PM +0800, Bibby Hsieh wrote: > cmdq driver provide a function that get the relationship > of sub system number from device node for client. > add specification for #subsys-cells, mediatek,gce-client-reg. > > Signed-off-by: Bibby Hsieh > --- >

Re: [PATCH] clk: imx: imx8mm: fix audio pll setting

2019-07-22 Thread Stephen Boyd
Quoting Peng Fan (2019-07-14 19:55:43) > From: Peng Fan > > The AUDIO PLL max support 650M, so the original clk settings violate > spec. This patch makes the output 786432000 -> 393216000, > and 722534400 -> 361267200 to aligned with NXP vendor kernel without any > impact on audio functionality

Re: [PATCH v1 1/2] mm/page_idle: Add support for per-pid page_idle using virtual indexing

2019-07-22 Thread Andrew Morton
On Mon, 22 Jul 2019 17:32:04 -0400 "Joel Fernandes (Google)" wrote: > The page_idle tracking feature currently requires looking up the pagemap > for a process followed by interacting with /sys/kernel/mm/page_idle. > This is quite cumbersome and can be error-prone too. If between > accessing the

Re: [PATCH v2 10/11] dt-bindings: mmc: arasan: Document 'xlnx,zynqmp-8.9a' controller

2019-07-22 Thread Rob Herring
On Mon, 1 Jul 2019 10:59:50 +0530, Manish Narani wrote: > Add documentation for 'xlnx,zynqmp-8.9a' SDHCI controller and optional > properties followed by example. > > Signed-off-by: Manish Narani > --- > .../devicetree/bindings/mmc/arasan,sdhci.txt | 19 > +++ > 1

Re: [PATCH] clk: renesas: cpg-mssr: Fix reset control race condition

2019-07-22 Thread Stephen Boyd
Quoting Geert Uytterhoeven (2019-07-11 06:03:59) > The module reset code in the Renesas CPG/MSSR driver uses > read-modify-write (RMW) operations to write to a Software Reset Register > (SRCRn), and simple writes to write to a Software Reset Clearing > Register (SRSTCLRn), as was mandated by the

Re: [PATCH v2 07/11] dt-bindings: mmc: arasan: Add optional properties for Arasan SDHCI

2019-07-22 Thread Rob Herring
On Mon, Jul 01, 2019 at 10:59:47AM +0530, Manish Narani wrote: > Add optional propeties for Arasan SDHCI which are used to set clk delays > for different speed modes in the controller. > > Signed-off-by: Manish Narani > --- > Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 15

Re: [PATCH][next] clk: Si5341/Si5340: remove redundant assignment to n_den

2019-07-22 Thread Christophe JAILLET
Le 22/07/2019 à 23:53, Stephen Boyd a écrit : Quoting Christophe JAILLET (2019-07-22 14:43:32) Le 22/07/2019 à 23:24, Stephen Boyd a écrit : Please Cc authors of drivers so they can ack/review. Adding Mike to take a look. Quoting Colin King (2019-07-01 09:50:20) From: Colin Ian King The

Re: [PATCH] phy: rockchip-dp: Avoid power leak by leaving the PHY power on

2019-07-22 Thread Doug Anderson
Kishon, On Mon, Jun 3, 2019 at 8:22 AM Doug Anderson wrote: > > Kishon, > > On Mon, Jun 3, 2019 at 4:22 AM Kishon Vijay Abraham I wrote: > > > > Hi, > > > > On 20/05/19 1:34 PM, Caesar Wang wrote: > > > Hi Doug, > > > > > > For now, nobody of rockchip is responsible for this driver. > > > Cc:

Re: [PATCH] checkpatch: Added warnings in favor of strscpy().

2019-07-22 Thread Jonathan Corbet
On Mon, 22 Jul 2019 14:50:09 -0700 Joe Perches wrote: > On Mon, 2019-07-22 at 23:01 +0200, Stephen Kitt wrote: > > How about you submit your current patch set, and I follow up with the above > > adapted to stracpy? > > OK, I will shortly after I figure out how to add kernel-doc > for

Re: [PATCH v2 05/11] dt-bindings: mmc: arasan: Update Documentation for the input clock

2019-07-22 Thread Rob Herring
On Mon, Jul 01, 2019 at 10:59:45AM +0530, Manish Narani wrote: > Add documentation for an optional input clock which is essentially used > in sampling the input data coming from the card. > > Signed-off-by: Manish Narani > --- > Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 14

Re: [PATCH v1 1/2] clk: tegra: divider: Fix missing check for enable-bit on rate's recalculation

2019-07-22 Thread Stephen Boyd
Quoting Dmitry Osipenko (2019-07-17 14:33:36) > 17.07.2019 23:08, Stephen Boyd пишет: > > Quoting Dmitry Osipenko (2019-07-15 10:35:26) > >> Unset "enable" bit means that divider is in bypass mode, hence it doesn't > >> have any effect in that case. > >> > >> Signed-off-by: Dmitry Osipenko > > >

Re: [PATCH v3] nvme-pci: Support shared tags across queues for Apple 2018 controllers

2019-07-22 Thread Benjamin Herrenschmidt
On Fri, 2019-07-19 at 23:51 +1000, Benjamin Herrenschmidt wrote: > On Fri, 2019-07-19 at 14:28 +0200, Christoph Hellwig wrote: > > Yikes, that things looks worse and worse. I think at this point > > we'll > > have to defer the support to 5.4 unfortunately as it is getting more > > and more

Re: [PATCH v2 01/11] dt-bindings: mmc: arasan: Update documentation for SD Card Clock

2019-07-22 Thread Rob Herring
On Mon, Jul 01, 2019 at 10:59:41AM +0530, Manish Narani wrote: > The clock handling is to be updated in the Arasan SDHCI. As the > 'devm_clk_register' is deprecated in the clock framework, this needs to > specify one more clock named 'clk_sdcard' to get the clock in the driver > via

Re: [PATCH] clk: sprd: Select REGMAP_MMIO to avoid compile errors

2019-07-22 Thread Stephen Boyd
Quoting Chunyan Zhang (2019-07-17 22:36:16) > From: Chunyan Zhang > > Make REGMAP_MMIO selected to avoid undefined reference to regmap symbols. > > Fixes: d41f59fd92f2 ("clk: sprd: Add common infrastructure") > Signed-off-by: Chunyan Zhang > --- Applied to clk-fixes

[PATCH V2 3/4] drivers: qcom: rpmh: switch over from spinlock irq variants

2019-07-22 Thread Lina Iyer
From: "Raju P.L.S.S.S.N" Switch over from using _irqsave/_irqrestore variants since we no longer race with a lock from the interrupt handler. While we are at it, rename the cache_lock to just lock to allow use of the lock to synchronize controller access. Signed-off-by: Raju P.L.S.S.S.N

[PATCH V2 1/4] drivers: qcom: rpmh-rsc: simplify TCS locking

2019-07-22 Thread Lina Iyer
From: "Raju P.L.S.S.S.N" The tcs->lock was introduced to serialize access with in TCS group. But, drv->lock is still needed to synchronize core aspects of the communication. This puts the drv->lock in the critical and high latency path of sending a request. drv->lock provides the all necessary

[PATCH V2 4/4] drivers: qcom: rpmh-rsc: remove redundant register access

2019-07-22 Thread Lina Iyer
Since drv->tcs_in_use is updated when the DRV_STATUS is updated, we could simply use the former to determine if the TCS is idle or not. Therefore, remove redundant TCS register read. Signed-off-by: Lina Iyer --- drivers/soc/qcom/rpmh-rsc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)

[PATCH V2 2/4] drivers: qcom: rpmh-rsc: avoid locking in the interrupt handler

2019-07-22 Thread Lina Iyer
Avoid locking in the interrupt context to improve latency. Since we don't lock in the interrupt context, it is possible that we now could race with the DRV_CONTROL register that writes the enable register and cleared by the interrupt handler. For fire-n-forget requests, the interrupt may be raised

Re: [PATCH][next] clk: Si5341/Si5340: remove redundant assignment to n_den

2019-07-22 Thread Stephen Boyd
Quoting Christophe JAILLET (2019-07-22 14:43:32) > Le 22/07/2019 à 23:24, Stephen Boyd a écrit : > > Please Cc authors of drivers so they can ack/review. > > > > Adding Mike to take a look. > > > > Quoting Colin King (2019-07-01 09:50:20) > >> From: Colin Ian King > >> > >> The variable n_den is

Re: [GIT PULL] Wimplicit-fallthrough patches for 5.3-rc2

2019-07-22 Thread Stephen Rothwell
Hi Gustavo, On Mon, 22 Jul 2019 14:58:13 -0500 "Gustavo A. R. Silva" wrote: > > The following changes since commit 5f9e832c137075045d15cd6899ab0505cfb2ca4b: > > Linus 5.3-rc1 (2019-07-21 14:05:38 -0700) > > are available in the Git repository at: > >

Re: [PATCH 3/5] dt: bindings: add mt7621-pll dt binding documentation

2019-07-22 Thread Stephen Boyd
Quoting Chuanhong Guo (2019-07-09 11:20:16) > This commit adds device tree binding documentation for MT7621 > PLL controller. > > Signed-off-by: Chuanhong Guo > --- > .../bindings/clock/mediatek,mt7621-pll.txt| 19 +++ > 1 file changed, 19 insertions(+) > create mode 100644

Re: [PATCH 2/5] MIPS: ralink: fix cpu clock of mt7621 and add dt clk devices

2019-07-22 Thread Stephen Boyd
Quoting Chuanhong Guo (2019-07-09 11:20:15) > For a long time the mt7621 uses a fixed cpu clock which causes a problem > if the cpu frequency is not 880MHz. > > This patch fixes the cpu clock calculation and adds the cpu/bus clkdev > which will be used in dts. > > Ported from OpenWrt: >

Re: [PATCH] checkpatch: Added warnings in favor of strscpy().

2019-07-22 Thread Joe Perches
On Mon, 2019-07-22 at 23:01 +0200, Stephen Kitt wrote: > How about you submit your current patch set, and I follow up with the above > adapted to stracpy? OK, I will shortly after I figure out how to add kernel-doc for stracpy/stracpy_pad to lib/string.c. It doesn't seem appropriate to add the

[PATCH v3 1/3] HID: core: reformat and reduce hid_printk macros

2019-07-22 Thread stillcompiling
From: Joshua Clayton Reformat hid_printk macros to use standard __VA_ARGS__ syntax Remove hid_printk(), hid_emerg(), hid_crit(), and hid_alert(). Per Joe Perches these unused and likely never to be used. Signed-off-by: Joshua Clayton --- include/linux/hid.h | 32

[PATCH v3 3/3] HID: core: only warn once of oversize hid report

2019-07-22 Thread stillcompiling
From: Joshua Clayton On HP spectre x360 convertible the message: hid-sensor-hub 001F:8087:0AC2.0002: hid_field_extract() called with n (192) > 32! (kworker/1:2) is continually printed many times per second, crowding out all else Protect dmesg by printing the warning only one time.

[PATCH v3 2/3] HID: core: Add printk_once variants to hid_warn() etc

2019-07-22 Thread stillcompiling
From: Joshua Clayton hid_warn_once() is needed, add the others as part of the block Signed-off-by: Joshua Clayton --- include/linux/hid.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/include/linux/hid.h b/include/linux/hid.h index e6c7efdb0458..cd41f209043f 100644 ---

Re: [PATCH 3/3] riscv: dts: Add DT node for SiFive FU540 Ethernet controller driver

2019-07-22 Thread Paul Walmsley
On Fri, 19 Jul 2019, Yash Shah wrote: > DT node for SiFive FU540-C000 GEMGXL Ethernet controller driver added > > Signed-off-by: Yash Shah Thanks, queuing this one for v5.3-rc with Andrew's suggested change to change phy1 to phy0. Am assuming patches 1 and 2 will go in via -net. - Paul

Re: [PATCH v9 04/21] mips: mm: Add p?d_leaf() definitions

2019-07-22 Thread Paul Burton
Hi Steven, On Mon, Jul 22, 2019 at 04:41:53PM +0100, Steven Price wrote: > walk_page_range() is going to be allowed to walk page tables other than > those of user space. For this it needs to know when it has reached a > 'leaf' entry in the page tables. This information is provided by the >

Re: [PATCH 2/3] macb: Update compatibility string for SiFive FU540-C000

2019-07-22 Thread Paul Walmsley
On Fri, 19 Jul 2019, Yash Shah wrote: > Update the compatibility string for SiFive FU540-C000 as per the new > string updated in the binding doc. > Reference: https://lkml.org/lkml/2019/7/17/200 > > Signed-off-by: Yash Shah Tested-by: Paul Walmsley - Paul

Re: [PATCH] raid1: use an int as the return value of raise_barrier()

2019-07-22 Thread Song Liu
On Tue, Jul 2, 2019 at 7:30 AM Hou Tao wrote: > > Using a sector_t as the return value is misleading, because > raise_barrier() only return 0 or -EINTR. > > Also add comments for the return values of raise_barrier(). > > Signed-off-by: Hou Tao Applied. Thanks for the patch. Song

Re: [PATCH] clk: rockchip: Fix -Wunused-const-variable

2019-07-22 Thread Heiko Stuebner
Hi Stephen, Am Montag, 22. Juli 2019, 23:35:18 CEST schrieb Stephen Boyd: > Quoting Nathan Huckleberry (2019-06-27 15:22:20) > > Clang produces the following warning > > > > drivers/clk/rockchip/clk-rv1108.c:125:7: warning: unused variable > > 'mux_pll_src_3plls_p' [-Wunused-const-variable] > >

[PATCH v2] tracing: kmem: convert call_site addresses to user friendly symbols

2019-07-22 Thread George G. Davis
While attempting to debug slub freelist pointer corruption bugs caused by a module, I discovered that the kmem call_site addresses are not at all user friendly for modules unless you manage to save a copy of kallsyms for the running kernel beforehand. So convert kmem call_site addresses to user

Re: [PATCH][next] clk: Si5341/Si5340: remove redundant assignment to n_den

2019-07-22 Thread Christophe JAILLET
Le 22/07/2019 à 23:24, Stephen Boyd a écrit : Please Cc authors of drivers so they can ack/review. Adding Mike to take a look. Quoting Colin King (2019-07-01 09:50:20) From: Colin Ian King The variable n_den is initialized however that value is never read as n_den is re-assigned a little

Re: [PATCH v2 3/3] HID: core: only warn once of oversize hid report

2019-07-22 Thread Joshua Clayton
On Mon, Jul 22, 2019 at 3:30 PM Joe Perches wrote: > > On Mon, 2019-07-22 at 15:26 -0600, stillcompil...@gmail.com wrote: > > From: Joshua Clayton > > Thanks Joshua > > > On HP spectre x360 convertible the message: > > hid-sensor-hub 001F:8087:0AC2.0002: hid_field_extract() called with n (192)

Re: [PATCH v2 0/2] Support kexec/kdump for clang built kernel

2019-07-22 Thread Nick Desaulniers
Joe, Is it possible to have scripts/get_maintainer.pl always cc linux-kernel@vger.kernel.org? I just sent out a series, and it seems the cover letter didn't get sent to LKML. I usually use this shell function to send patches: ``` function kpatch () { patch=$1 shift git send-email \

Re: [PATCH 1/5] MIPS: ralink: add dt binding header for mt7621-pll

2019-07-22 Thread Stephen Boyd
The subject of this patch is confusing. Not sure what it has to do with "MIPS:" so maybe remove that and prefix it "dt-bindings: clock:" instead. Quoting Chuanhong Guo (2019-07-09 11:20:14) > This patch adds dt binding header for mediatek,mt7621-pll > > Signed-off-by: Weijie Gao >

[PATCH] ARM: davinci: dm646x: Fix a typo in the comment

2019-07-22 Thread Christophe JAILLET
The driver is dedicated to DM646x. So update the description in the top most comment accordingly. It must have been derived from dm644x.c, but looks DM646 speecific now. Signed-off-by: Christophe JAILLET --- arch/arm/mach-davinci/dm646x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

linux-next: Fixes tag needs some work in the net-next tree

2019-07-22 Thread Stephen Rothwell
Hi all, In commit 66b5f1c43984 ("net-ipv6-ndisc: add support for RFC7710 RA Captive Portal Identifier") Fixes tag Fixes: e35f30c131a56 has these problem(s): - missing subject Did you mean Fixes: e35f30c131a5 ("Treat ND option 31 as userland (DNSSL support)") -- Cheers, Stephen

Re: [PATCH] clk: rockchip: Fix -Wunused-const-variable

2019-07-22 Thread Stephen Boyd
Quoting Nathan Huckleberry (2019-06-27 15:22:20) > Clang produces the following warning > > drivers/clk/rockchip/clk-rv1108.c:125:7: warning: unused variable > 'mux_pll_src_3plls_p' [-Wunused-const-variable] > PNAME(mux_pll_src_3plls_p) = { "apll", "gpll", "dpll" }; > > Looks like this

[PATCH v2 2/2] x86/purgatory: use CFLAGS_REMOVE rather than reset KBUILD_CFLAGS

2019-07-22 Thread Nick Desaulniers
KBUILD_CFLAGS is very carefully built up in the top level Makefile, particularly when cross compiling or using different build tools. Resetting KBUILD_CFLAGS via := assignment is an antipattern. The comment above the reset mentions that -pg is problematic. Other Makefiles like

[PATCH v2 1/2] x86/purgatory: do not use __builtin_memcpy and __builtin_memset

2019-07-22 Thread Nick Desaulniers
Implementing memcpy and memset in terms of __builtin_memcpy and __builtin_memset is problematic. GCC at -O2 will replace calls to the builtins with calls to memcpy and memset (but will generate an inline implementation at -Os). Clang will replace the builtins with these calls regardless of

Re: [PATCH v3] clk: mediatek: mt8183: Register 13MHz clock earlier for clocksource

2019-07-22 Thread Stephen Boyd
Quoting Weiyi Lu (2019-06-28 00:22:34) > The 13MHz clock should be registered before clocksource driver is > initialized. Use CLK_OF_DECLARE_DRIVER() to guarantee. > > Fixes: acddfc2c261b ("clk: mediatek: Add MT8183 clock support") > Cc: > Signed-off-by: Weiyi Lu > --- Applied to clk-fixes

Re: [RESEND][PATCH] clk: at91: generated: Truncate divisor to GENERATED_MAX_DIV + 1

2019-07-22 Thread Stephen Boyd
Quoting nicolas.fe...@microchip.com (2019-07-03 08:05:24) > On 25/06/2019 at 11:10, Codrin Ciubotariu wrote: > > In clk_generated_determine_rate(), if the divisor is greater than > > GENERATED_MAX_DIV + 1, then the wrong best_rate will be returned. > > If clk_generated_set_rate() will be called

Re: [RESEND][PATCH] clk: at91: generated: Truncate divisor to GENERATED_MAX_DIV + 1

2019-07-22 Thread Stephen Boyd
Quoting Codrin Ciubotariu (2019-06-25 02:10:02) > In clk_generated_determine_rate(), if the divisor is greater than > GENERATED_MAX_DIV + 1, then the wrong best_rate will be returned. > If clk_generated_set_rate() will be called later with this wrong > rate, it will return -EINVAL, so the

[PATCH v1 1/2] mm/page_idle: Add support for per-pid page_idle using virtual indexing

2019-07-22 Thread Joel Fernandes (Google)
The page_idle tracking feature currently requires looking up the pagemap for a process followed by interacting with /sys/kernel/mm/page_idle. This is quite cumbersome and can be error-prone too. If between accessing the per-PID pagemap and the global page_idle bitmap, if something changes with the

Re: [PATCH] clk: Add missing documentation of devm_clk_bulk_get_optional() argument

2019-07-22 Thread Stephen Boyd
Quoting Sylwester Nawrocki (2019-07-01 04:46:51) > Fix an incomplete devm_clk_bulk_get_optional() function documentation > by adding description of the num_clks argument as in other *clk_bulk* > functions. > > Fixes: 9bd5ef0bd874 ("clk: Add devm_clk_bulk_get_optional() function") > Reported-by:

Re: [RFC PATCH 0/3] md: export internal stats through debugfs

2019-07-22 Thread Song Liu
On Tue, Jul 2, 2019 at 6:25 AM Hou Tao wrote: > > Hi, > > There are so many io counters, stats and flags in md, so I think > export these info to userspace will be helpful for online-debugging, > especially when the vmlinux file and the crash utility are not > available. And these info can also

Re: [PATCH 5/6] clk: imx8mq: Remove CLK_IS_CRITICAL flag for IMX8MQ_CLK_TMU_ROOT

2019-07-22 Thread Stephen Boyd
Quoting anson.hu...@nxp.com (2019-07-04 21:56:11) > From: Anson Huang > > IMX8MQ_CLK_TMU_ROOT is ONLY used for thermal module, the driver > should manage this clock, so no need to have CLK_IS_CRITICAL flag > set. > > Signed-off-by: Anson Huang > --- Acked-by: Stephen Boyd

Re: [PATCH v2 3/3] HID: core: only warn once of oversize hid report

2019-07-22 Thread Joe Perches
On Mon, 2019-07-22 at 15:26 -0600, stillcompil...@gmail.com wrote: > From: Joshua Clayton Thanks Joshua > On HP spectre x360 convertible the message: > hid-sensor-hub 001F:8087:0AC2.0002: hid_field_extract() called with n (192) > > 32! (kworker/1:2) > is continually printed many times per

Re: [PATCH 10/11] firmware: arm_scmi: Drop config flag in clk_ops->rate_set

2019-07-22 Thread Stephen Boyd
Quoting Sudeep Holla (2019-07-08 08:47:29) > CLOCK_PROTOCOL_ATTRIBUTES provides attributes to indicate the maximum > number of pending asynchronous clock rate changes supported by the > platform. If it's non-zero, then we should be able to use asynchronous > clock rate set for any clocks until the

Re: [PATCH 11/11] firmware: arm_scmi: Use asynchronous CLOCK_RATE_SET when possible

2019-07-22 Thread Stephen Boyd
Quoting Sudeep Holla (2019-07-08 08:47:30) > CLOCK_PROTOCOL_ATTRIBUTES provides attributes to indicate the maximum > number of pending asynchronous clock rate changes supported by the > platform. If it's non-zero, then we should be able to use asynchronous > clock rate set for any clocks until the

Re: [PATCH 2/5] x86_64, -march=native: POPCNT support

2019-07-22 Thread Alexey Dobriyan
On Tue, Jul 23, 2019 at 12:15:39AM +0300, Alexey Dobriyan wrote: > On Mon, Jul 22, 2019 at 11:12:10PM +0200, Peter Zijlstra wrote: > > On Mon, Jul 22, 2019 at 11:27:20PM +0300, Alexey Dobriyan wrote: > > > Detect POPCNT instruction support and inline hweigth*() functions > > > if it is supported

[PATCH] ASoC: rt5665: Fix a typo in the name of a function

2019-07-22 Thread Christophe JAILLET
All function names start with rt5665_, except 'rt5655_set_verf()'. It is likely a typo. Fix it to be consistent. Signed-off-by: Christophe JAILLET --- sound/soc/codecs/rt5665.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sound/soc/codecs/rt5665.c

[PATCH v2 2/3] HID: core: Add printk_once variants to hid_warn() etc

2019-07-22 Thread stillcompiling
From: Joshua Clayton hid_warn_once() is needed, add the others as part of the block Signed-off-by: Joshua Clayton --- include/linux/hid.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/include/linux/hid.h b/include/linux/hid.h index b5e73331100e..306dde3760a4 100644 ---

[PATCH v2 1/3] HID: core: reformat and reduce hid_printk macros

2019-07-22 Thread stillcompiling
From: Joshua Clayton Reformat hid_printk macros to use standard __VA_ARGS__ syntax Remove hid_printk(), hid_emerg(), hid_crit(), and hid_alert(). Per Joe Perches these unused and likely never to be used. Signed-off-by: Joshua Clayton --- include/linux/hid.h | 32

[PATCH v2 3/3] HID: core: only warn once of oversize hid report

2019-07-22 Thread stillcompiling
From: Joshua Clayton On HP spectre x360 convertible the message: hid-sensor-hub 001F:8087:0AC2.0002: hid_field_extract() called with n (192) > 32! (kworker/1:2) is continually printed many times per second, crowding out all else Protect dmesg by printing the warning only one time.

Re: [PATCH] clk: imx8mq: Mark AHB clock as critical

2019-07-22 Thread Stephen Boyd
Quoting Abel Vesa (2019-07-05 01:52:19) > On 19-06-25 15:32:22, Stephen Boyd wrote: > > Quoting Abel Vesa (2019-06-25 02:01:56) > > > Keep the AHB clock always on since there is no driver to control it and > > > all the other clocks that use it as parent rely on it being always > > > enabled. > >

Re: [PATCH][next] clk: Si5341/Si5340: remove redundant assignment to n_den

2019-07-22 Thread Stephen Boyd
Please Cc authors of drivers so they can ack/review. Adding Mike to take a look. Quoting Colin King (2019-07-01 09:50:20) > From: Colin Ian King > > The variable n_den is initialized however that value is never read > as n_den is re-assigned a little later in the two paths of a > following

Re: [PATCH] tracing: kmem: convert call_site addresses to user friendly symbols

2019-07-22 Thread George G. Davis
Hello Steve, On Mon, Jul 22, 2019 at 09:43:43AM -0400, Steven Rostedt wrote: > > Looking at the kbuild report... > > On Fri, 19 Jul 2019 22:22:40 -0400 > "George G. Davis" wrote: > > > diff --git a/include/trace/events/kmem.h b/include/trace/events/kmem.h > > index eb57e3037deb..ae18e61fa1c0

Re: [PATCH 2/3] MIPS: DTS: jz4740: Add node for the MMC driver

2019-07-22 Thread Paul Burton
Hello, Paul Cercueil wrote: > Add a devicetree node for the jz4740-mmc driver. > > Signed-off-by: Paul Cercueil Applied to mips-next. Thanks, Paul [ This message was auto-generated; if you believe anything is incorrect then please email paul.bur...@mips.com to report it. ]

Re: [PATCH 2/2] MIPS: Decode config3 register on Ingenic SoCs

2019-07-22 Thread Paul Burton
Hello, Paul Cercueil wrote: > XBurst misses a config2 register, so config3 decode was skipped in > decode_configs(). > > Signed-off-by: Paul Cercueil Applied to mips-next. Thanks, Paul [ This message was auto-generated; if you believe anything is incorrect then please email

Re: [PATCH 5/5] MIPS: Undefine PMD_ORDER for 32-bit builds

2019-07-22 Thread Paul Burton
Hello, Paul Cercueil wrote: > From: Daniel Silsby > > During an update long ago to conform to 4-level page code, PMD_ORDER was > changed from 0 to 1, despite the fact that a PMD table is not used at > all in a 32-bit MIPS build. PMD_ORDER does not seem to be used in these > builds. Now, it

Re: [PATCH 3/5] MIPS: Decouple CPU_SUPPORTS_HUGEPAGES from 64BIT

2019-07-22 Thread Paul Burton
Hello, Paul Cercueil wrote: > From: Daniel Silsby > > We now have partial 32-bit MIPS huge page support, so there's no need > to restrict these config options only to 64-bit systems. > > Signed-off-by: Daniel Silsby > Signed-off-by: Paul Cercueil Applied to mips-next. Thanks, Paul [

Re: [PATCH 4/5] MIPS: ingenic: Add support for huge pages

2019-07-22 Thread Paul Burton
Hello, Paul Cercueil wrote: > From: Daniel Silsby > > The Ingenic jz47xx SoC series of 32-bit MIPS CPUs support huge pages. > > Signed-off-by: Daniel Silsby > Signed-off-by: Paul Cercueil Applied to mips-next. Thanks, Paul [ This message was auto-generated; if you believe anything is

Re: [RFC 7/7] x86/current: Aggressive caching of current

2019-07-22 Thread Nadav Amit
> On Jul 22, 2019, at 2:07 PM, Peter Zijlstra wrote: > > On Thu, Jul 18, 2019 at 10:41:10AM -0700, Nadav Amit wrote: >> The current_task is supposed to be constant in each thread and therefore >> does not need to be reread. There is already an attempt to cache it >> using inline assembly, using

Re: [PATCH 2/5] MIPS: Add partial 32-bit huge page support

2019-07-22 Thread Paul Burton
Hello, Paul Cercueil wrote: > From: Daniel Silsby > > This adds initial support for huge pages to 32-bit MIPS systems. > Systems with extended addressing enabled (EVA,XPA,Alchemy/Netlogic) > are not yet supported. > With huge pages enabled, this implementation will increase page table > memory

Re: [PATCH v2] arm64: dts: fsl: pico-pi: Add a device tree for the PICO-PI-IMX8M

2019-07-22 Thread Daniel Baluta
On Mon, Jul 22, 2019 at 9:30 PM Vaittinen, Matti wrote: > > Sorry for top posting. I'm replying using mobile phone and outlook web app... > > gpio_intr is not needed. Irq must be given using the standard irq property. > gpio_intr has been used in an old draft driver - I assume the dts originates

Re: [PATCH] checkpatch: Added warnings in favor of strscpy().

2019-07-22 Thread Stephen Kitt
On Mon, 22 Jul 2019 10:59:00 -0700, Joe Perches wrote: > On Mon, 2019-07-22 at 10:50 -0700, Kees Cook wrote: > > On Sat, Jul 06, 2019 at 02:42:04PM +0200, Stephen Kitt wrote: > > > On Tue, 2 Jul 2019 10:25:04 -0700, Kees Cook > > > wrote: > > > > On Sat, Jun 29, 2019 at 06:15:37PM +0200,

[PATCH] ASoC: pcm3168a: Fix a typo in the name of a constant

2019-07-22 Thread Christophe JAILLET
There is a typo in PCM1368A_MAX_SYSCLK, it should be PCM3168A_MAX_SYSCLK (1 and 3 switched in 3168) Signed-off-by: Christophe JAILLET --- sound/soc/codecs/pcm3168a.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/codecs/pcm3168a.c b/sound/soc/codecs/pcm3168a.c

Re: [PATCH v2 6/8] PCI: al: Add support for DW based driver type

2019-07-22 Thread Bjorn Helgaas
On Sun, Jul 21, 2019 at 03:08:18PM +, Chocron, Jonathan wrote: > On Fri, 2019-07-19 at 08:55 +, Gustavo Pimentel wrote: > > On Thu, Jul 18, 2019 at 10:47:16, Jonathan Chocron > > wrote: > > > +static int al_pcie_probe(struct platform_device *pdev) > > > +{ > > > + struct device *dev =

Re: [PATCH 2/5] x86_64, -march=native: POPCNT support

2019-07-22 Thread Alexey Dobriyan
On Mon, Jul 22, 2019 at 11:12:10PM +0200, Peter Zijlstra wrote: > On Mon, Jul 22, 2019 at 11:27:20PM +0300, Alexey Dobriyan wrote: > > Detect POPCNT instruction support and inline hweigth*() functions > > if it is supported by CPU. > > > > Detect POPCNT at boot time and conditionally refuse to

Re: [PATCH 1/5] MIPS: Disallow CPU_SUPPORTS_HUGEPAGES for XPA,EVA

2019-07-22 Thread Paul Burton
Hello, Paul Cercueil wrote: > From: Daniel Silsby > > In preparation for 32-bit MIPS huge page support. > > EVA,XPA are extended-addressing modes for 32-bit MIPS systems. Because > huge pages aren't currently supported in 32-bit MIPS, this doesn't take > any features away from EVA,XPA-enabled

Re: [PATCH 1/2] MIPS: Rename JZRISC to XBURST

2019-07-22 Thread Paul Burton
Hello, Paul Cercueil wrote: > The real name of the CPU present in the JZ line of SoCs from Ingenic is > XBurst, not JZRISC. > > Signed-off-by: Paul Cercueil Series applied to mips-next. Thanks, Paul [ This message was auto-generated; if you believe anything is incorrect then please

Re: [PATCH] be2net: fix adapter->big_page_size miscaculation

2019-07-22 Thread Qian Cai
On Fri, 2019-07-19 at 17:47 -0400, Qian Cai wrote: > On Thu, 2019-07-18 at 16:29 -0700, David Miller wrote: > > From: Qian Cai > > Date: Thu, 18 Jul 2019 19:26:47 -0400 > > > > >   > > >   > > > > On Jul 18, 2019, at 5:21 PM, Bill Wendling wrote: > > > >   > > > > [My previous response was

Re: [PATCH 2/5] x86_64, -march=native: POPCNT support

2019-07-22 Thread Peter Zijlstra
On Mon, Jul 22, 2019 at 11:27:20PM +0300, Alexey Dobriyan wrote: > Detect POPCNT instruction support and inline hweigth*() functions > if it is supported by CPU. > > Detect POPCNT at boot time and conditionally refuse to boot. > > Signed-off-by: Alexey Dobriyan > --- >

Re: [RFC 3/7] x86/percpu: Use C for percpu accesses when possible

2019-07-22 Thread Nadav Amit
> On Jul 22, 2019, at 1:52 PM, Peter Zijlstra wrote: > > On Thu, Jul 18, 2019 at 10:41:06AM -0700, Nadav Amit wrote: > >> diff --git a/arch/x86/include/asm/preempt.h b/arch/x86/include/asm/preempt.h >> index 99a7fa9ab0a3..60f97b288004 100644 >> --- a/arch/x86/include/asm/preempt.h >> +++

Re: [PATCH 1/2] x86/purgatory: add -mno-sse, -mno-mmx, -mno-sse2 to Makefile

2019-07-22 Thread Nick Desaulniers
On Fri, Jul 19, 2019 at 1:17 AM Peter Zijlstra wrote: > > On Thu, Jul 18, 2019 at 02:34:44PM -0700, Nick Desaulniers wrote: > > On Wed, Jul 17, 2019 at 5:02 PM Vaibhav Rustagi > > wrote: > > > > > > Compiling the purgatory code with clang results in using of mmx > > > registers. > > > > > > $

Re: [PATCH] linux/dim: fix -Wunused-const-variable warnings

2019-07-22 Thread Saeed Mahameed
On Mon, 2019-07-22 at 09:08 -0400, Qian Cai wrote: > There are a lot of compilation warnings due to tx_profile[] and > rx_profile[] are only used in lib/dim/net_dim.c but > include/linux/dim.h > is included elsewhere. > > In file included from ./include/rdma/ib_verbs.h:64, > from

Re: [RFC 0/7] x86/percpu: Use segment qualifiers

2019-07-22 Thread Peter Zijlstra
On Thu, Jul 18, 2019 at 10:41:03AM -0700, Nadav Amit wrote: > GCC 6+ supports segment qualifiers. Using them allows to implement > several optimizations: Overall I like this. Nice!

Re: [RFC 7/7] x86/current: Aggressive caching of current

2019-07-22 Thread Peter Zijlstra
On Thu, Jul 18, 2019 at 10:41:10AM -0700, Nadav Amit wrote: > The current_task is supposed to be constant in each thread and therefore > does not need to be reread. There is already an attempt to cache it > using inline assembly, using this_cpu_read_stable(), which hides the > dependency on the

Backlight in motorola Droid 4

2019-07-22 Thread Pavel Machek
Hi! So now the backlight LED can be controlled. Good. (And thanks!) But I seem to remember that backlight had range from "is it really on?" to "very bright"; now it seems to have range from "bright" to "very bright". Any ideas what goes on there? Thanks,

Re: [RFC 3/7] x86/percpu: Use C for percpu accesses when possible

2019-07-22 Thread Peter Zijlstra
On Thu, Jul 18, 2019 at 10:41:06AM -0700, Nadav Amit wrote: > diff --git a/arch/x86/include/asm/preempt.h b/arch/x86/include/asm/preempt.h > index 99a7fa9ab0a3..60f97b288004 100644 > --- a/arch/x86/include/asm/preempt.h > +++ b/arch/x86/include/asm/preempt.h > @@ -91,7 +91,8 @@ static

Re: [PATCH 0/3] arm64: Allow early timestamping of kernel log

2019-07-22 Thread Pavel Tatashin
On Mon, Jul 22, 2019 at 3:33 AM Marc Zyngier wrote: > > So far, we've let the arm64 kernel start its meaningful time stamping > of the kernel log pretty late, which is caused by sched_clock() being > initialised rather late compared to other architectures. > > Pavel Tatashin proposed[1] to move

<    1   2   3   4   5   6   7   8   9   10   >