RE: [Regression 4.15-rc2] New messages `tpm tpm0: A TPM error (2314) occurred continue selftest`

2017-12-15 Thread Mario.Limonciello
> -Original Message- > From: Paul Menzel [mailto:pmen...@molgen.mpg.de] > Sent: Friday, December 15, 2017 5:54 AM > To: Limonciello, Mario ; Alexander Steffen > ; Jason Gunthorpe > Cc: linux-integr...@vger.kernel.org; linux-kernel@vger.kernel.org; Rafael J. > Wysocki ; Len Brown > Subject

Re: [PATCH v2 2/5] arm64: dts: rockchip: add extcon nodes and enable tcphy.

2017-12-15 Thread Heiko Stübner
Am Mittwoch, 13. Dezember 2017, 11:32:16 CET schrieb Enric Balletbo i Serra: > Enable tcphy and create the cros-ec's extcon node for the USB Type-C port. > > Signed-off-by: Enric Balletbo i Serra > Reviewed-by: Brian Norris Just for people reading along, with the extcon driver-patch applied, En

[PATCH] net: qcom/emac: Reduce timeout for mdio read/write

2017-12-15 Thread Hemanth Puranik
Currently mdio read/write takes around ~115us as the timeout between status check is set to 100us. By reducing the timeout to 1us mdio read/write takes ~15us to complete. This improves the link up event response. Signed-off-by: Hemanth Puranik --- drivers/net/ethernet/qualcomm/emac/emac-phy.c |

Re: [PATCH v2 3/3] backlight: tdo24m: add model and status sysfs entries

2017-12-15 Thread Daniel Thompson
On Fri, Oct 13, 2017 at 09:42:49PM +0200, Robert Jarzmik wrote: > Add entries to query the panel ASIC to acquire the model number and the > status field. This comes in handy to check that the SPI connection is > functional and that the panel's firmware is alive. > > Signed-off-by: Robert Jarzmik

Re: [PATCH] xen/balloon: Mark unallocated host memory as UNUSABLE

2017-12-15 Thread Boris Ostrovsky
>> + >> +hostmem_resource->start = max_addr; >> +hostmem_resource->end = entry->addr + entry->size; >> +for (; i < memmap.nr_entries; i++) { >> +entry = &xen_e820_table->entries[i]; >> +if (entry->type == E820_TYPE_RAM) > Shouldn't that be != ? No, the idea her

Re: [PATCH] PCI / PM: Force devices to D0 in pci_pm_thaw_noirq()

2017-12-15 Thread Rafael J. Wysocki
On Fri, Dec 15, 2017 at 3:07 AM, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > It is incorrect to call pci_restore_state() for devices in low-power > states (D1-D3), as that involves the restoration of MSI setup which > requires MMIO to be operational and that is only the case in D0. > >

Re: arm64: unhandled level 0 translation fault

2017-12-15 Thread Will Deacon
On Fri, Dec 15, 2017 at 02:30:00PM +0100, Geert Uytterhoeven wrote: > On Fri, Dec 15, 2017 at 12:23 PM, Dave Martin wrote: > > The two important differences here seem to be > > > > 1) Staging the state via current->thread.fpsimd_state instead of loading > > directly: > > > > - fpsimd_load_st

Re: Warning in i915/intel_audio.c

2017-12-15 Thread Mika Kahola
On Fri, 2017-12-15 at 19:20 +0530, Jaswinder Singh Rajput wrote: > On Fri, 2017-12-15 at 14:44 +0200, Mika Kahola wrote: > > > > Hi, > > > > This is a known issue. Could you try out this patch to see if that > > would fix this issue for you? > > > > https://patchwork.freedesktop.org/series/35389

Re: Linux 4.15.0-rc3 perf probe/uprobe issue with address randomization

2017-12-15 Thread Peter Zijlstra
On Fri, Dec 15, 2017 at 03:14:37PM +0100, Thomas-Mich Richter wrote: > During debugging of perf probe tool I discovered an issue with > uprobes and address randomization. > > To set a uprobe on a function named inet_pton in libc library, you > obtain the address of the symbol inet_pton using comma

Re: [PATCH v8] arm64: dts: meson-axg: switch uart_ao clock to CLK81

2017-12-15 Thread Neil Armstrong
On 15/12/2017 15:17, Yixun Lan wrote: > Switch the uart_ao pclk to CLK81 since the clock driver is ready. > > Signed-off-by: Yixun Lan > > --- > This is a fixup for the previous version > > Changes in v8 since [1] > - move clock DT info into soc.dtsi > > [1] > http://lists.infradead.org/piper

[PATCH v8] arm64: dts: meson-axg: switch uart_ao clock to CLK81

2017-12-15 Thread Yixun Lan
Switch the uart_ao pclk to CLK81 since the clock driver is ready. Signed-off-by: Yixun Lan --- This is a fixup for the previous version Changes in v8 since [1] - move clock DT info into soc.dtsi [1] http://lists.infradead.org/pipermail/linux-amlogic/2017-December/005741.html --- arch/arm64/b

Re: [PATCH 1/2] bdisp: Fix a possible sleep-in-atomic bug in bdisp_hw_reset

2017-12-15 Thread Hans Verkuil
Fabien or Benjamin, can you take a look at these two patches? I'm a bit hesitant applying this since e.g. this bdisp_hw_reset() function might wait for up to a second, which is a mite long for an interrupt :-) Regards, Hans On 12/12/17 14:47, Jia-Ju Bai wrote: > The driver may sleep un

[PATCH 1/6] nvmem: core: Deduplicate bus_find_device() by name matching

2017-12-15 Thread srinivas . kandagatla
From: Lukas Wunner No need to reinvent the wheel, we have bus_find_device_by_name(). Signed-off-by: Lukas Wunner Signed-off-by: Srinivas Kandagatla --- drivers/nvmem/core.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c in

[PATCH 2/6] nvmem: core: let stride and word_size default to 1

2017-12-15 Thread srinivas . kandagatla
From: Heiner Kallweit If the caller doesn't set stride and/or word_size in struct nvmem_config then nvmem_register accepts this but we may face strange effects later due to both values being 0. Therefore use 1 as default for both values. Signed-off-by: Heiner Kallweit Signed-off-by: Srinivas Ka

Linux 4.15.0-rc3 perf probe/uprobe issue with address randomization

2017-12-15 Thread Thomas-Mich Richter
During debugging of perf probe tool I discovered an issue with uprobes and address randomization. To set a uprobe on a function named inet_pton in libc library, you obtain the address of the symbol inet_pton using command nm and then use the following command to set the uprobe: # echo "p:probe_li

[PATCH 4/6] nvmem: uniphier: change access unit from 32bit to 8bit

2017-12-15 Thread srinivas . kandagatla
From: Kunihiko Hayashi The efuse on UniPhier allows 8bit access according to the specification. Since bit offset of nvmem is limited to 0-7, it is desiable to change access unit of nvmem to 8bit. Signed-off-by: Kunihiko Hayashi Signed-off-by: Srinivas Kandagatla --- drivers/nvmem/uniphier-efu

Re: [PATCH v2 net-next 1/3] net: dsa: mediatek: add VLAN support for MT7530

2017-12-15 Thread kbuild test robot
Hi Sean, I love your patch! Yet something to improve: [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/sean-wang-mediatek-com/add-VLAN-support-to-DSA-MT7530/20171215-214450 config: i386-randconfig-x019-201750 (attached as .config) compiler: gcc-7

[PATCH 03/17] mm: don't export __add_pages

2017-12-15 Thread Christoph Hellwig
This function isn't used by any modules, and is only to be called from core MM code. This includes the calls for the add_pages wrapper that might be inlined. Signed-off-by: Christoph Hellwig --- mm/memory_hotplug.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/memory_hotplug.c b/mm/memo

[PATCH 01/17] memremap: provide stubs for vmem_altmap_offset and vmem_altmap_free

2017-12-15 Thread Christoph Hellwig
Currently all calls to those functions are eliminated by the compiler when CONFIG_ZONE_DEVICE is not set, but this soon won't be the case. Signed-off-by: Christoph Hellwig --- include/linux/memremap.h | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/include

[PATCH 04/17] mm: pass the vmem_altmap to arch_add_memory and __add_pages

2017-12-15 Thread Christoph Hellwig
We can just pass this on instead of having to do a radix tree lookup without proper locking 2 levels into the callchain. Signed-off-by: Christoph Hellwig --- arch/ia64/mm/init.c| 5 +++-- arch/powerpc/mm/mem.c | 5 +++-- arch/s390/mm/init.c| 5 +++-- arch/sh/m

[PATCH 06/17] mm: pass the vmem_altmap to arch_remove_memory and __remove_pages

2017-12-15 Thread Christoph Hellwig
We can just pass this on instead of having to do a radix tree lookup without proper locking 2 levels into the callchain. Signed-off-by: Christoph Hellwig wip --- arch/ia64/mm/init.c| 4 ++-- arch/powerpc/mm/mem.c | 6 ++ arch/s390/mm/init.c| 2 +- arch/sh/mm/i

[PATCH 09/17] mm: split altmap memory map allocation from normal case

2017-12-15 Thread Christoph Hellwig
No functional changes, just untangling the call chain. Signed-off-by: Christoph Hellwig Reviewed-by: Logan Gunthorpe --- arch/powerpc/mm/init_64.c | 5 - arch/x86/mm/init_64.c | 5 - include/linux/mm.h| 9 ++--- mm/sparse-vmemmap.c | 15 +++ 4 files

[PATCH 11/17] mm: move get_dev_pagemap out of line

2017-12-15 Thread Christoph Hellwig
This is a pretty big function, which should be out of line in general, and a no-op stub if CONFIG_ZONE_DEVICЕ is not set. Signed-off-by: Christoph Hellwig Reviewed-by: Logan Gunthorpe --- include/linux/memremap.h | 39 --- kernel/memremap.c| 36 ++

[PATCH 16/17] memremap: change devm_memremap_pages interface to use struct dev_pagemap

2017-12-15 Thread Christoph Hellwig
From: Logan Gunthorpe This new interface is similar to how struct device (and many others) work. The caller initializes a 'struct dev_pagemap' as required and calls 'devm_memremap_pages'. This allows the pagemap structure to be embedded in another structure and thus container_of can be used. In t

[PATCH 14/17] memremap: simplify duplicate region handling in devm_memremap_pages

2017-12-15 Thread Christoph Hellwig
__radix_tree_insert already checks for duplicates and returns -EEXIST in that case, so remove the duplicate (and racy) duplicates check. Signed-off-by: Christoph Hellwig Reviewed-by: Logan Gunthorpe --- kernel/memremap.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/kernel/mem

[PATCH 17/17] memremap: merge find_dev_pagemap into get_dev_pagemap

2017-12-15 Thread Christoph Hellwig
There is only one caller of the trivial function find_dev_pagemap left, so just merge it into the caller. Signed-off-by: Christoph Hellwig --- kernel/memremap.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/kernel/memremap.c b/kernel/memremap.c index fd0e7c44e6bd.

[PATCH 15/17] memremap: drop private struct page_map

2017-12-15 Thread Christoph Hellwig
From: Logan Gunthorpe 'struct page_map' is a private structure of 'struct dev_pagemap' but the latter replicates all the same fields as the former so there isn't much value in it. Thus drop it in favour of a completely public struct. This is a clean up in preperation for a more generally useful

[PATCH 05/17] mm: pass the vmem_altmap to vmemmap_populate

2017-12-15 Thread Christoph Hellwig
We can just pass this on instead of having to do a radix tree lookup without proper locking a few levels into the callchain. Signed-off-by: Christoph Hellwig --- arch/arm64/mm/mmu.c| 6 -- arch/ia64/mm/discontig.c | 3 ++- arch/powerpc/mm/init_64.c | 7 ++- arch

[PATCH 13/17] memremap: remove to_vmem_altmap

2017-12-15 Thread Christoph Hellwig
All callers are gone now. Signed-off-by: Christoph Hellwig --- include/linux/memremap.h | 9 - kernel/memremap.c| 26 -- 2 files changed, 35 deletions(-) diff --git a/include/linux/memremap.h b/include/linux/memremap.h index 26e8aaba27d5..3fddcfe57bb0 10

[PATCH 10/17] mm: merge vmem_altmap_alloc into altmap_alloc_block_buf

2017-12-15 Thread Christoph Hellwig
There is no clear separation between the two, so merge them. Signed-off-by: Christoph Hellwig Reviewed-by: Logan Gunthorpe --- mm/sparse-vmemmap.c | 45 - 1 file changed, 16 insertions(+), 29 deletions(-) diff --git a/mm/sparse-vmemmap.c b/mm/sparse-

[PATCH 12/17] mm: optimize dev_pagemap reference counting around get_dev_pagemap

2017-12-15 Thread Christoph Hellwig
Change the calling convention so that get_dev_pagemap always consumes the previous reference instead of doing this using an explicit earlier call to put_dev_pagemap in the callers. The callers will still need to put the final reference after finishing the loop over the pages. Signed-off-by: Chris

[PATCH 02/17] mm: don't export arch_add_memory

2017-12-15 Thread Christoph Hellwig
Only x86_64 and sh export this symbol, and it is not used by any modular code. Signed-off-by: Christoph Hellwig --- arch/sh/mm/init.c | 1 - arch/x86/mm/init_64.c | 1 - 2 files changed, 2 deletions(-) diff --git a/arch/sh/mm/init.c b/arch/sh/mm/init.c index bf726af5f1a5..afc54d593a26 10064

[PATCH 07/17] mm: pass the vmem_altmap to vmemmap_free

2017-12-15 Thread Christoph Hellwig
We can just pass this on instead of having to do a radix tree lookup without proper locking a few levels into the callchain. Signed-off-by: Christoph Hellwig --- arch/arm64/mm/mmu.c| 3 +- arch/ia64/mm/discontig.c | 3 +- arch/powerpc/mm/init_64.c | 5 ++-- arch/s390/mm

[PATCH 08/17] mm: pass the vmem_altmap to memmap_init_zone

2017-12-15 Thread Christoph Hellwig
Pass the vmem_altmap two levels down instead of needing a lookup. Signed-off-by: Christoph Hellwig --- arch/ia64/mm/init.c| 9 + include/linux/memory_hotplug.h | 2 +- include/linux/mm.h | 4 ++-- kernel/memremap.c | 2 +- mm/hmm.c

revamp vmem_altmap / dev_pagemap handling V2

2017-12-15 Thread Christoph Hellwig
Hi all, this series started with two patches from Logan that now are in the middle of the series to kill the memremap-internal pgmap structure and to redo the dev_memreamp_pages interface to be better suitable for future PCI P2P uses. I reviewed them and noticed that there isn't really any good r

[PATCH 5/6] nvmem: rockchip-efuse: parse 'rockchip,efuse-size'

2017-12-15 Thread srinivas . kandagatla
From: Finley Xiao The eFuse size is defined in property before, but the length of registers is not equal to the size on some platforms, so we add a new property to redefine it. Signed-off-by: Finley Xiao Acked-by: Rob Herring Signed-off-by: Heiko Stuebner Signed-off-by: Srinivas Kandagatla

[PATCH 6/6] nvmem: rockchip-efuse: add support for rk3328-efuse

2017-12-15 Thread srinivas . kandagatla
From: Finley Xiao This adds the necessary data for handling eFuse on the rk3328. Signed-off-by: Finley Xiao Acked-by: Rob Herring Signed-off-by: Heiko Stuebner Signed-off-by: Srinivas Kandagatla --- .../devicetree/bindings/nvmem/rockchip-efuse.txt | 1 + drivers/nvmem/rockchip-efuse.c

[PATCH 3/6] nvmem: core: switch to device_property_present for reading property "read-only"

2017-12-15 Thread srinivas . kandagatla
From: Heiner Kallweit Switch to more generic device_property_present to consider also non-DT properties. Signed-off-by: Heiner Kallweit Signed-off-by: Srinivas Kandagatla --- drivers/nvmem/core.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/nvmem/core.c b/

[PATCH 0/6] nvmem: patches for v4.16 (Set 1)

2017-12-15 Thread srinivas . kandagatla
From: Srinivas Kandagatla Hi Greg, Here are some nvmem enhancements plus some new SOCs support in rockchip. Can you queue them up for 4.16. Wish you a Merry Christmas and Happy New Year 2018!! Thanks, Srini Finley Xiao (2): nvmem: rockchip-efuse: parse 'rockchip,efuse-size' nvmem: rockchi

Re: [PATCH v2 2/4] sched/fair: add util_est on top of PELT

2017-12-15 Thread Peter Zijlstra
On Fri, Dec 15, 2017 at 02:02:18PM +, Patrick Bellasi wrote: > On 13-Dec 17:05, Peter Zijlstra wrote: > > On Tue, Dec 05, 2017 at 05:10:16PM +, Patrick Bellasi wrote: > > > + if (cfs_rq->nr_running > 0) { > > > + util_est = cfs_rq->util_est_runnable; > > > + util_est -= tas

[PATCH v2] Adjustments: lock/unlock task in context_switch

2017-12-15 Thread rodrigosiqueira
Function prepare_lock_switch have an unused parameter, and also the function name was not descriptive. To improve the readability and remove the extra parameter, the following changes were made: * Moved prepare_lock_switch from kernel/sched/sched.h to kernel/sched/core.c, renamed it to acquire_l

Re: [PATCH v2 01/17] mm/gup: Fixup p*_access_permitted()

2017-12-15 Thread Peter Zijlstra
On Thu, Dec 14, 2017 at 03:37:30PM +0100, Peter Zijlstra wrote: > Kirill did point out that my patch(es) break FOLL_DUMP in that it would > now exclude pkey protected pages from core-dumps. > > My counter argument is that it will now properly exclude !_PAGE_USER > pages. > > If we change p??_acc

Re: Regression with a0747a859ef6 ("bdi: add error handle for bdi_debug_register")

2017-12-15 Thread weiping zhang
2017-12-15 19:10 GMT+08:00 Bruno Wolff III : > On Fri, Dec 15, 2017 at 10:04:32 +0800, > weiping zhang wrote: >> >> I just want to know WARN_ON WHAT in device_add_disk, >> if bdi_register_owner return error code, it may fail at any step of >> following: > > > Was that output in the original boot

Re: [PATCH v2 2/4] sched/fair: add util_est on top of PELT

2017-12-15 Thread Patrick Bellasi
On 13-Dec 17:05, Peter Zijlstra wrote: > On Tue, Dec 05, 2017 at 05:10:16PM +, Patrick Bellasi wrote: > > + if (cfs_rq->nr_running > 0) { > > + util_est = cfs_rq->util_est_runnable; > > + util_est -= task_util_est(p); > > + if (util_est < 0) > > +

[PATCH v2] gpio: fix "gpio-line-names" property retrieval

2017-12-15 Thread Christophe Leroy
Following commit 9427ecbed46cc ("gpio: Rework of_gpiochip_set_names() to use device property accessors"), "gpio-line-names" DT property is not retrieved anymore when chip->parent is not set by the driver. This is due to OF based property reads having been replaced by device based property reads. T

Re: [BUG]kernel softlockup due to sidtab_search_context run for long time because of too many sidtab context node

2017-12-15 Thread Stephen Smalley
On Fri, 2017-12-15 at 03:09 +, yangjihong wrote: > On 12/15/2017 10:31 PM, yangjihong wrote: > > On 12/14/2017 12:42 PM, Casey Schaufler wrote: > > > On 12/14/2017 9:15 AM, Stephen Smalley wrote: > > > > On Thu, 2017-12-14 at 09:00 -0800, Casey Schaufler wrote: > > > > > On 12/14/2017 8:42 AM,

[PATCH v2] gpio: fix "gpio-line-names" property retrieval

2017-12-15 Thread chleroy
Following commit 9427ecbed46cc ("gpio: Rework of_gpiochip_set_names() to use device property accessors"), "gpio-line-names" DT property is not retrieved anymore when chip->parent is not set by the driver. This is due to OF based property reads having been replaced by device based property reads. T

Re: Warning in i915/intel_audio.c

2017-12-15 Thread Jaswinder Singh Rajput
On Fri, 2017-12-15 at 14:44 +0200, Mika Kahola wrote: > Hi, > > This is a known issue. Could you try out this patch to see if that > would fix this issue for you? > > https://patchwork.freedesktop.org/series/35389/ > Yes, Thanks. It works :-) When this patch will merge with linus kernel. Than

Re: [PATCH 2/6] blk-mq: replace timeout synchronization with a RCU and generation based scheme

2017-12-15 Thread t...@kernel.org
Hello, Peter. On Thu, Dec 14, 2017 at 09:20:42PM +0100, Peter Zijlstra wrote: > But now that I look at this again, TJ, why can't the below happen? > > write_seqlock_begin(); > blk_mq_rq_update_state(rq, IN_FLIGHT); > blk_add_timer(rq); > > read_seqcount_begi

Re: [PATCHv4 09/12] x86/mm: Provide pmdp_establish() helper

2017-12-15 Thread Kirill A. Shutemov
On Wed, Dec 13, 2017 at 04:36:39PM -0800, Andrew Morton wrote: > On Thu, 14 Dec 2017 03:33:18 +0300 "Kirill A. Shutemov" > wrote: > > > On Wed, Dec 13, 2017 at 04:09:51PM -0800, Andrew Morton wrote: > > > > @@ -181,6 +182,40 @@ static inline pmd_t > > > > native_pmdp_get_and_clear(pmd_t *pmdp)

Re: [PATCH] clk: fix spin_lock/unlock imbalance on bad clk_enable() reentrancy

2017-12-15 Thread Jerome Brunet
On Tue, 2017-12-12 at 22:14 -0600, David Lechner wrote: > On 12/12/2017 05:43 PM, David Lechner wrote: > > If clk_enable() is called in reentrant way and spin_trylock_irqsave() is > > not working as expected, it is possible to get a negative enable_refcnt > > which results in a missed call to spin_

Re: [PATCH 0/2] Use SPDX-License-Identifier for rockchip devicetree files

2017-12-15 Thread Philippe Ombredanne
Klaus, On Fri, Dec 15, 2017 at 12:44 PM, Klaus Goger wrote: > This patch series replaces all the license text in rockchip devicetree > files text with a proper SPDX-License-Identifier. > It follows the guidelines submitted[1] by Thomas Gleixner that are not > yet merged. > > These series also fix

Re: [PATCH v10 3/4] [media] platform: Add Synopsys DesignWare HDMI RX PHY e405 Driver

2017-12-15 Thread Hans Verkuil
On 11/12/17 18:41, Jose Abreu wrote: > This adds support for the Synopsys DesignWare HDMI RX PHY e405. This > phy receives and decodes HDMI video that is delivered to a controller. > > Main features included in this driver are: > - Equalizer algorithm that chooses the phy best settings >

[PATCH] nvmem: meson-mx-efuse: fix reading from an offset other than 0

2017-12-15 Thread srinivas . kandagatla
From: Martin Blumenstingl meson_mx_efuse_read calculates the address internal to the eFuse based on the offset and the word size. This works fine with any given offset. However, the offset is also included when writing to the output buffer. This means that reading 4 bytes at offset 500 tries to w

Re: [PATCH 14/25] arm: stm32: dts: Remove leading 0x and 0s from bindings notation

2017-12-15 Thread Alexandre Torgue
Hi Mathieu On 12/15/2017 01:46 PM, Mathieu Malaterre wrote: Improve the DTS files by removing all the leading "0x" and zeros to fix the following dtc warnings: Warning (unit_address_format): Node /XXX unit name should not have leading "0x" and Warning (unit_address_format): Node /XXX unit nam

[PATCH v3] firmware: qcom: scm: Fix incorrect of_node_put calls in scm_init

2017-12-15 Thread Loys Ollivier
When using other platform architectures, in the init of the qcom_scm driver, of_node_put is called on /firmware if no qcom dt is found. This results in a kernel error: Bad of_node_put() on /firmware. These calls to of_node_put from the qcom_scm init are unnecessary as of_find_matching_node and of_

Re: [PATCH 2/8] ARM: dts: keystone*: Use a single soc0 instance

2017-12-15 Thread Nishanth Menon
Crap.. couple of typos crept in. Apologies - Santosh, if you dont want to manualy change, I can rebase and repost if you like to any branch of your choice. On 12/15/2017 07:20 AM, Nishanth Menon wrote: Provide and soc0 node and reference the same to simplify dts. This also ^^ - sho

Re: [patch v2 1/2] mm, mmu_notifier: annotate mmu notifiers with blockable invalidate callbacks

2017-12-15 Thread Dimitri Sivanich
On Thu, Dec 14, 2017 at 01:30:56PM -0800, David Rientjes wrote: > Commit 4d4bbd8526a8 ("mm, oom_reaper: skip mm structs with mmu notifiers") > prevented the oom reaper from unmapping private anonymous memory with the > oom reaper when the oom victim mm had mmu notifiers registered. > > The rationa

Re: [PATCH] drm/tegra: mark t186 display hub PM functions __maybe_unused

2017-12-15 Thread Arnd Bergmann
On Fri, Dec 15, 2017 at 2:33 PM, Thierry Reding wrote: > On Fri, Dec 15, 2017 at 01:51:52PM +0100, Arnd Bergmann wrote: >> The newly introduced driver has optional suspend/resume functions, >> causing a warning when CONFIG_PM is disabled: >> >> drivers/gpu/drm/tegra/hub.c:749:12: error: 'tegra_dis

Re: [PATCH v10 2/4] MAINTAINERS: Add entry for Synopsys DesignWare HDMI drivers

2017-12-15 Thread Hans Verkuil
On 15/12/17 14:35, Hans Verkuil wrote: > On 11/12/17 18:41, Jose Abreu wrote: >> Add an entry for Synopsys DesignWare HDMI Receivers drivers >> and phys. >> >> Signed-off-by: Jose Abreu >> Cc: Joao Pinto > > Reviewed-by: Hans Verkuil Oops, I was a bit too quick here. You're missing this file:

Re: [PATCH v10 1/4] dt-bindings: media: Document Synopsys DesignWare HDMI RX

2017-12-15 Thread Hans Verkuil
On 11/12/17 18:41, Jose Abreu wrote: > Document the bindings for the Synopsys DesignWare HDMI RX. > > Signed-off-by: Jose Abreu > Acked-by: Rob Herring (v8) > Cc: Joao Pinto > Cc: Rob Herring > Cc: Mark Rutland > Cc: Mauro Carvalho Chehab > Cc: Hans Verkuil > Cc: Sylwester Nawrocki > Cc: d

Re: [PATCH v10 2/4] MAINTAINERS: Add entry for Synopsys DesignWare HDMI drivers

2017-12-15 Thread Hans Verkuil
On 11/12/17 18:41, Jose Abreu wrote: > Add an entry for Synopsys DesignWare HDMI Receivers drivers > and phys. > > Signed-off-by: Jose Abreu > Cc: Joao Pinto Reviewed-by: Hans Verkuil Regards, Hans > --- > MAINTAINERS | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git

Re: [PATCH v2 00/22] mmc: tmio: various fixes and cleanups

2017-12-15 Thread Ulf Hansson
On 15 December 2017 at 12:12, Wolfram Sang wrote: > >> After 2, COMPILE_TEST will work correctly. >> >> Then, Wolfram mentioned we would need to include from tmio_mmc.h >> >> https://patchwork.kernel.org/patch/10074333/ >> >> >> I was waiting for a patch from him. > > Yes, I am sorry. I am curren

Re: [PATCH 07/25] arm: keystone: dts: Remove leading 0x and 0s from bindings notation

2017-12-15 Thread Nishanth Menon
On 12/15/2017 06:46 AM, Mathieu Malaterre wrote: Improve the DTS files by removing all the leading "0x" and zeros to fix the following dtc warnings: Warning (unit_address_format): Node /XXX unit name should not have leading "0x" and Warning (unit_address_format): Node /XXX unit name should not

Re: [PATCH] cpupower: Remove FSF address

2017-12-15 Thread Dominik Brodowski
On Thu, Dec 14, 2017 at 05:16:29PM -0800, Laura Abbott wrote: > > Checkpatch in the kernel now complains about having the FSF address > in comments. Other tools such as rpmlint are now starting to do the > same thing. Remove the FSF address to reduce warnings on multiple tools. > > Signed-off-by:

Re: [PATCH] drm/tegra: mark t186 display hub PM functions __maybe_unused

2017-12-15 Thread Thierry Reding
On Fri, Dec 15, 2017 at 01:51:52PM +0100, Arnd Bergmann wrote: > The newly introduced driver has optional suspend/resume functions, > causing a warning when CONFIG_PM is disabled: > > drivers/gpu/drm/tegra/hub.c:749:12: error: 'tegra_display_hub_resume' defined > but not used [-Werror=unused-func

Re: [PATCH] rtc: ds1307: call the platform's logic for handle IRQs.

2017-12-15 Thread Enric Balletbo Serra
Hi Alexandre, 2017-12-15 14:04 GMT+01:00 Alexandre Belloni : > Hi, > > On 15/12/2017 at 12:44:49 +0100, Enric Balletbo Serra wrote: >> Hi, >> >> 2017-08-24 12:30 GMT+02:00 Enric Balletbo i Serra >> : >> > On some systems the nIRQ pin is often connect to a GPIO, then, if a given >> > interrupt line

Re: arm64: unhandled level 0 translation fault

2017-12-15 Thread Geert Uytterhoeven
Hi Dave, On Fri, Dec 15, 2017 at 12:23 PM, Dave Martin wrote: > On Thu, Dec 14, 2017 at 07:08:27PM +0100, Geert Uytterhoeven wrote: >> On Thu, Dec 14, 2017 at 4:24 PM, Dave P Martin wrote: >> > On Thu, Dec 14, 2017 at 02:34:50PM +, Geert Uytterhoeven wrote: >> >> On Tue, Dec 12, 2017 at 11:2

Re: [PATCH 2/6] blk-mq: replace timeout synchronization with a RCU and generation based scheme

2017-12-15 Thread t...@kernel.org
Hello, Bart. On Thu, Dec 14, 2017 at 09:13:32PM +, Bart Van Assche wrote: ... > however is called before a every use of a request. Sorry but I'm not > enthusiast about the code in blk_rq_init() that reinitializes state > information that should survive request reuse. If it wasn't clear, me ne

[PATCH net-next] net: dsa: lan9303: lan9303_csr_reg_wait cleanups

2017-12-15 Thread Egil Hjelmeland
Non-functional cleanups in lan9303_csr_reg_wait(): - Change type of param 'mask' from int to u32. - Remove param 'value' (will probably never be used) - Reduced retries from 1000 to 25, consistent with lan9303_read_wait. - Corrected comments Signed-off-by: Egil Hjelmeland --- drivers/net/dsa

Re: [PATCH] percpu: percpu_counter_initialized can be boolean

2017-12-15 Thread Tejun Heo
On Fri, Dec 15, 2017 at 02:48:32PM +0800, guoyayun wrote: > This patch makes percpu_counter_initialized return bool due to this > particular function only using either one or zero as its return > value. > > No functional change. > > Signed-off-by: guoyayun Applied to percpu/for-4.16. Thanks.

Re: [PATCH v4 2/2] misc: xlnx_vcu: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-15 Thread Arnd Bergmann
In Fri, Dec 15, 2017 at 8:24 AM, Dhaval Shah wrote: > Xilinx ZYNQMP logicoreIP Init driver is based on the new > LogiCoreIP design created. This driver provides the processing system > and programmable logic isolation. Set the frequency based on the clock > information get from the logicoreIP regi

RE: [PATCH v9 0/2] media: ov7740: Add a V4L2 sensor-level driver

2017-12-15 Thread Wenyou.Yang
Hi Sakari, Do you have some comments on this version? Best Regards, Wenyou Yang > -Original Message- > From: linux-media-ow...@vger.kernel.org [mailto:linux-media- > ow...@vger.kernel.org] On Behalf Of Wenyou Yang > Sent: 2017年12月11日 9:32 > To: Mauro Carvalho Chehab ; Rob Herring > ; Ma

[PATCH 3/8] ARM: dts: keystone: Get rid of usage of skeleton.dtsi

2017-12-15 Thread Nishanth Menon
skeleton.dtsi does'nt offer us any real benefits with most of the parameters being overriden. So, just drop the same entirely and introduce appropriate changes for chosen node and memory back to our top level definition. This also squashes the following warnings with W=1: arch/arm/boot/dts/keysto

Re: [PATCH v5 0/3] Fix find_first_zero_bit() usage

2017-12-15 Thread Lorenzo Pieralisi
On Thu, Dec 14, 2017 at 02:01:43PM +0100, Niklas Cassel wrote: > find_first_zero_bit()'s parameter 'size' is defined in bits, > not in bytes. > > Calling find_first_zero_bit() with the wrong size unit > will lead to insidious bugs. > > Fix all uses of find_first_zero_bit() called with > sizeof()

[PATCH 5/8] ARM: dts: keystone-clocks: Add missing unit name to clock nodes that have regs

2017-12-15 Thread Nishanth Menon
Add the control register as the base for the clock nodes which are missing them. This squashes some 78 warnings of the effect when built with W=1. Reported-by: Rob Herring Signed-off-by: Nishanth Menon --- arch/arm/boot/dts/keystone-clocks.dtsi | 52 +- 1 file ch

Re: rsi_91x: Low bandwidth: sends ~100 Kbits/sec, receives ~1.5 Mbits/sec

2017-12-15 Thread Alexey Brodkin
Hi Prameela, On Fri, 2017-12-15 at 18:14 +0530, Prameela Rani Garnepudi wrote: > Hi Alexey, > > > On Friday 15 December 2017 04:50 PM, Alexey Brodkin wrote: > > Hi Prameela, > > > > On Fri, 2017-12-15 at 11:13 +0530, Prameela Rani Garnepudi wrote: > >> Hi Alexey, > >> > >> Please use the

Re: [PATCH v19 1/7] xbitmap: Introduce xbitmap

2017-12-15 Thread Matthew Wilcox
On Fri, Dec 15, 2017 at 07:05:07PM +0800, kbuild test robot wrote: > 21struct radix_tree_node *node; > 22void **slot; ^^^ missing __rcu annotation here. Wei, could you fold that change into your next round? Thanks!

[PATCH 2/8] ARM: dts: keystone*: Use a single soc0 instance

2017-12-15 Thread Nishanth Menon
Provide and soc0 node and reference the same to simplify dts. This also resolves the following warnings when built with W=1: arch/arm/boot/dts/keystone-k2hk-evm.dtb: Warning (unit_address_vs_reg): Node /soc has a reg or ranges property, but no unit name arch/arm/boot/dts/keystone-k2l-evm.dtb: Warn

[PATCH 1/8] ARM: dts: keystone*: Standardize license with SPDX tag

2017-12-15 Thread Nishanth Menon
Standardize the licenses with SPDX license tag and standardize TI's copyright statement to be consistent. Signed-off-by: Nishanth Menon --- arch/arm/boot/dts/keystone-clocks.dtsi | 7 ++- arch/arm/boot/dts/keystone-k2e-clocks.dtsi | 7 ++- arch/arm/boot/dts/keystone-k2e-evm.dts

[PATCH 7/8] ARM: dts: keystone-k2e-clocks: Add missing unit name to clock nodes that have regs

2017-12-15 Thread Nishanth Menon
Add the control register as the base for the clock nodes which are missing them. This squashes the following warnings of the effect when built with W=1: arch/arm/boot/dts/keystone-k2e-evm.dtb: Warning (unit_address_vs_reg): Node /soc@0/clocks/clkusb1 has a reg or ranges property, but no unit name

[PATCH 8/8] ARM: dts: keystone-k2l-clocks: Add missing unit name to clock nodes that have regs

2017-12-15 Thread Nishanth Menon
Add the control register as the base for the clock nodes which are missing them. This squashes some 22 warnings of the effect when built with W=1. Reported-by: Rob Herring Signed-off-by: Nishanth Menon --- arch/arm/boot/dts/keystone-k2l-clocks.dtsi | 44 +++--- 1 file c

[PATCH 6/8] ARM: dts: keystone-hk-clocks: Add missing unit name to clock nodes that have regs

2017-12-15 Thread Nishanth Menon
Add the control register as the base for the clock nodes which are missing them. This squashes some 36 warnings of the effect when built with W=1. Reported-by: Rob Herring Signed-off-by: Nishanth Menon --- arch/arm/boot/dts/keystone-k2hk-clocks.dtsi | 74 ++--- 1 file ch

Re: [PATCH v2] PCI: keystone: fix interrupt-controller-node lookup

2017-12-15 Thread Lorenzo Pieralisi
On Fri, Nov 17, 2017 at 02:38:31PM +0100, Johan Hovold wrote: > Fix child-node lookup during initialisation which was using the wrong > OF-helper and ended up searching the whole device tree depth-first > starting at the parent rather than just matching on its children. > > To make things worse, t

Re: [PATCH] cgroup/cpuset: fix circular locking dependency

2017-12-15 Thread Tejun Heo
Hello, Prateek. On Fri, Dec 15, 2017 at 02:24:55PM +0530, Prateek Sood wrote: > Following are two ways to improve cgroup_transfer_tasks(). In > both cases task in PF_EXITING state would be left in source > cgroup. It would be removed from cgroup_exit() in exit path. > > diff --git a/kernel/cgroup

[PATCH 4/8] ARM: dts: keystone: Add missing unit name to interrupt controller

2017-12-15 Thread Nishanth Menon
Add base address for GIC as unit address. This also squashes the following warnings when built with W=1: arch/arm/boot/dts/keystone-k2hk-evm.dtb: Warning (unit_address_vs_reg): Node /interrupt-controller has a reg or ranges property, but no unit name arch/arm/boot/dts/keystone-k2l-evm.dtb: Warning

[PATCH 0/8] ARM: dts: keystone*: Stage 1 cleanup for W=1

2017-12-15 Thread Nishanth Menon
Hi, Series to squash some of the build warnings with dts. At least try and get through 2/3rd of the warnings low hanging ones with this series. Warning status: As of master: http://pastebin.ubuntu.com/26188581/ After this series: http://pastebin.ubuntu.com/26188578/ Diff: 243 warnings squashed.

[GIT PULL] tee dynamic shm for v4.16

2017-12-15 Thread Jens Wiklander
Hello arm-soc maintainers, Please pull these tee driver changes. This implements support for dynamic shared memory support in OP-TEE. More specifically is enables mapping of user space memory in secure world to be used as shared memory. This has been reviewed and refined by the OP-TEE community a

[PATCH] cgroup, docs: document the root cgroup behavior of cpu and io controllers

2017-12-15 Thread Maciej S. Szmigiero
Currently, cgroups v2 documentation contains only a generic remark that "How resource consumption in the root cgroup is governed is up to each controller", which isn't really telling users much, who need to dig in the code and / or commit messages to learn the exact behavior. In cgroups v1 at leas

Re: sysfs_kf_seq_show() spends 87% of time in memset()

2017-12-15 Thread Tejun Heo
Hello, On Fri, Dec 15, 2017 at 01:59:55PM +0100, Christophe LEROY wrote: > Doing a 'perf record' on an application using GPIOs a lot, I > discovered that most of the time spent in the read() system call of > the 'value' sysfs file of that GPIO (which returns "0\n" or "1\n") > is indeed spent in me

Re: [PATCH 03/25] arm: bcm: dts: Remove leading 0x and 0s from bindings notation

2017-12-15 Thread Stefan Wahren
[Add Eric] Am 15.12.2017 um 13:46 schrieb Mathieu Malaterre: Improve the DTS files by removing all the leading "0x" and zeros to fix the following dtc warnings: Warning (unit_address_format): Node /XXX unit name should not have leading "0x" and Warning (unit_address_format): Node /XXX unit na

Re: [PATCH] drm/tegra: mark t186 display hub PM functions __maybe_unused

2017-12-15 Thread Arnd Bergmann
On Fri, Dec 15, 2017 at 2:10 PM, Dmitry Osipenko wrote: > On 15.12.2017 15:51, Arnd Bergmann wrote: >> --- a/drivers/gpu/drm/tegra/hub.c >> +++ b/drivers/gpu/drm/tegra/hub.c >> @@ -730,7 +730,7 @@ static int tegra_display_hub_remove(struct >> platform_device *pdev) >> return err; >> } >> >

[PATCH] x86: Remove unused parameter of prepare_switch_to

2017-12-15 Thread rodrigosiqueira
Commit e37e43a497d5 ("x86/mm/64: Enable vmapped stacks (CONFIG_HAVE_ARCH_VMAP_STACK=y)") added prepare_switch_to with one extra parameter which is not used by the function, remove it. Signed-off-by: Rodrigo Siqueira --- arch/x86/include/asm/switch_to.h | 5 ++--- 1 file changed, 2 insertions(+),

Re: [PATCH v6] mfd: syscon: Add hardware spinlock support

2017-12-15 Thread Arnd Bergmann
On Fri, Dec 15, 2017 at 11:42 AM, Lee Jones wrote: >> @@ -87,6 +88,30 @@ static struct syscon *of_syscon_register(struct >> device_node *np) >> if (ret) >> reg_io_width = 4; >> >> + ret = of_hwspin_lock_get_id(np, 0); >> + if (ret > 0) { >> + syscon_config

Re: [PATCH] drm/tegra: mark t186 display hub PM functions __maybe_unused

2017-12-15 Thread Dmitry Osipenko
On 15.12.2017 15:51, Arnd Bergmann wrote: > The newly introduced driver has optional suspend/resume functions, > causing a warning when CONFIG_PM is disabled: > > drivers/gpu/drm/tegra/hub.c:749:12: error: 'tegra_display_hub_resume' defined > but not used [-Werror=unused-function] > drivers/gpu/d

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

2017-12-15 Thread Tejun Heo
On Fri, Dec 15, 2017 at 10:58:04AM +0100, Arnd Bergmann wrote: > As long as cft->name is guaranteed to be NUL-terminated, using strlcpy() would > work just as well and avoid that warning, so the change below could be folded > into that commit. > > Signed-off-by: Arnd Bergmann Applied to cgroup/f

Re: [PATCH 4.14 00/52] 4.14.7-stable review

2017-12-15 Thread Greg Kroah-Hartman
On Fri, Dec 15, 2017 at 11:09:00AM +0100, Nikola Ciprich wrote: > Hi Greg, > > On Fri, Dec 15, 2017 at 10:51:37AM +0100, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 4.14.7 release. > > There are 52 patches in this series, all will be posted as a response > >

Re: [PATCH 4.4 000/105] 4.4.106-stable review

2017-12-15 Thread Greg Kroah-Hartman
On Fri, Dec 15, 2017 at 03:01:10AM -0700, Nathan Chancellor wrote: > > Merged, compiled, and flashed onto my Pixel 2 XL and OnePlus 5. > > No initial issues noticed in general usage or dmesg. Wonderful, thanks for testing and letting me know. greg k-h

Re: [PATCH linux ipmi for BMC] ipmi: add an Aspeed KCS IPMI BMC driver

2017-12-15 Thread Corey Minyard
On 12/14/2017 09:07 PM, Haiyue Wang wrote: Thanks Corey, yes, I ran through checkpatch. Fix most of warnings and errors. I thought 2 and 3 would be acceptable, will fix it in new patch,and together with fix for comments from all of you. And how to fix item 1 ? 1. WARNING: added, moved or delet

Re: [PATCH] rtc: ds1307: call the platform's logic for handle IRQs.

2017-12-15 Thread Alexandre Belloni
Hi, On 15/12/2017 at 12:44:49 +0100, Enric Balletbo Serra wrote: > Hi, > > 2017-08-24 12:30 GMT+02:00 Enric Balletbo i Serra > : > > On some systems the nIRQ pin is often connect to a GPIO, then, if a given > > interrupt line is supposed to wake up the system, the corresponding input > > of that

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