[PATCH v7 13/34] i2c: tegra: Move out all device-tree parsing into tegra_i2c_parse_dt()

2020-09-08 Thread Dmitry Osipenko
Move out code related to device-tree parsing from the probe function into tegra_i2c_parse_dt() in order to make code more consistent. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-)

[PATCH v7 15/34] i2c: tegra: Reorder location of functions in the code

2020-09-08 Thread Dmitry Osipenko
Reorder location of functions in the code in order to have definition of functions closer to the place of the invocation. This change makes easier to navigate around the code and removes the need to have a prototype for tegra_i2c_init(). Reviewed-by: Michał Mirosław Signed-off-by: Dmitry

[PATCH v7 02/34] i2c: tegra: Add missing pm_runtime_put()

2020-09-08 Thread Dmitry Osipenko
The pm_runtime_get_sync() always bumps refcount regardless of whether it succeeds or fails. Hence driver is responsible for restoring of the RPM refcounting. This patch adds missing RPM puts which restore refcounting in a case of pm_runtime_get_sync() error. Suggested-by: Andy Shevchenko

[PATCH v7 27/34] i2c: tegra: Check errors for both positive and negative values

2020-09-08 Thread Dmitry Osipenko
The driver's code is inconsistent in regards to the error values checking. The correct way should be to check both positive and negative values. This patch cleans up the error-checks in the code. Note that the pm_runtime_get_sync() could return positive value on success, hence only relevant parts

[PATCH v7 24/34] i2c: tegra: Factor out packet header setup from tegra_i2c_xfer_msg()

2020-09-08 Thread Dmitry Osipenko
The code related to packet header setting up is a bit messy and makes tegra_i2c_xfer_msg() more difficult to read than it could be. Let's factor the packet header setup from tegra_i2c_xfer_msg() into separate function in order to make code easier to read and follow. Reviewed-by: Michał Mirosław

[PATCH v7 30/34] i2c: tegra: Clean up variable names

2020-09-08 Thread Dmitry Osipenko
Rename "ret" variables to "err" in order to make code a bit more expressive, emphasizing that the returned value is an error code. Same vice versa, where appropriate. Rename variable "reg" to "val" in order to better reflect the actual usage of the variable in the code and to make naming

[PATCH v7 14/34] i2c: tegra: Clean up probe function

2020-09-08 Thread Dmitry Osipenko
The driver's probe function code is a bit difficult to read. This patch reorders code of the probe function, forming groups of code that are easy to work with. The probe tear-down order now matches the driver-removal order. Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 100

[PATCH v7 29/34] i2c: tegra: Improve formatting of variables

2020-09-08 Thread Dmitry Osipenko
Reorder definition of variables in the code to have them sorted by length and grouped logically, also replace "unsigned long" with "u32". Do this in order to make code easier to read. Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 97 -- 1

[PATCH v7 22/34] i2c: tegra: Rename wait/poll functions

2020-09-08 Thread Dmitry Osipenko
Drop '_timeout' postfix from the wait/poll completion function names in order to make the names shorter, making code cleaner a tad. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 32 ++-- 1 file changed, 14

[PATCH v7 23/34] i2c: tegra: Factor out error recovery from tegra_i2c_xfer_msg()

2020-09-08 Thread Dmitry Osipenko
Factor out error recovery code from tegra_i2c_xfer_msg() in order to make this function easier to read and follow. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 46 ++ 1 file changed, 30 insertions(+), 16

[PATCH v7 26/34] i2c: tegra: Factor out hardware initialization into separate function

2020-09-08 Thread Dmitry Osipenko
Factor out hardware initialization into a separate function from the probe function. The only place where runtime PM needs to be resumed during probe is the place of hardware initialization, hence it makes sense to factor out it in order to have a bit cleaner error handling in tegra_i2c_probe().

[PATCH v7 28/34] i2c: tegra: Consolidate error handling in tegra_i2c_xfer_msg()

2020-09-08 Thread Dmitry Osipenko
Consolidate error handling in tegra_i2c_xfer_msg() into a common code path in order to make code cleaner. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git

[PATCH v7 25/34] i2c: tegra: Factor out register polling into separate function

2020-09-08 Thread Dmitry Osipenko
Factor out register polling into a separate function in order to remove boilerplate code and make code cleaner. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 57 +++--- 1 file changed, 25 insertions(+), 32

Re: [Intel-gfx] [PATCH 0/8] Convert the intel iommu driver to the dma-iommu api

2020-09-08 Thread Tom Murphy
On Tue, 8 Sep 2020 at 16:56, Tvrtko Ursulin wrote: > > > On 08/09/2020 16:44, Logan Gunthorpe wrote: > > On 2020-09-08 9:28 a.m., Tvrtko Ursulin wrote: > >>> > >>> diff --git a/drivers/gpu/drm/i915/i915_scatterlist.h > >>> b/drivers/gpu/drm/i915/i915 > >>> index b7b59328cb76..9367ac801f0c 100644

[PATCH v7 18/34] i2c: tegra: Remove likely/unlikely from the code

2020-09-08 Thread Dmitry Osipenko
The likely/unlikely annotations should be used only in a hot paths of performance-critical code. The I2C driver doesn't have such paths, and thus, there is no justification for usage of likely/unlikely annotations in the code. Hence remove them. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry

[PATCH v7 21/34] i2c: tegra: Don't fall back to PIO mode if DMA configuration fails

2020-09-08 Thread Dmitry Osipenko
The DMA code path has been tested well enough and the DMA configuration performed by tegra_i2c_config_fifo_trig() shouldn't ever fail in practice. Hence let's remove the obscure transfer-mode switching in order to have a cleaner and simpler code. Now I2C transfer will be failed if DMA

[PATCH v7 19/34] i2c: tegra: Remove redundant check in tegra_i2c_issue_bus_clear()

2020-09-08 Thread Dmitry Osipenko
The tegra_i2c_wait_for_config_load() checks for 'has_config_load_reg' by itself, hence there is no need to duplicate the check. Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git

[PATCH v7 16/34] i2c: tegra: Clean up variable types

2020-09-08 Thread Dmitry Osipenko
Don't use signed types for unsigned values and use consistent types for sibling variables. Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 38 +- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/drivers/i2c/busses/i2c-tegra.c

[PATCH v7 20/34] i2c: tegra: Remove "dma" variable from tegra_i2c_xfer_msg()

2020-09-08 Thread Dmitry Osipenko
The "dma" variable of tegra_i2c_xfer_msg() function doesn't bring much in regards to readability and generation of the code. Besides readability, it's also not very nice that the is_curr_dma_xfer is initialized in tegra_i2c_xfer_msg() and then could be overridden by tegra_i2c_config_fifo_trig().

RE: [GIT PULL] First batch of KVM changes for Linux 5.9

2020-09-08 Thread Christopherson, Sean J
Ingo Molnar wrote: > * Paolo Bonzini wrote: > > > Paolo Bonzini (11): > > Merge branch 'kvm-async-pf-int' into HEAD > > kvmtool broke in this merge window, hanging during bootup right after CPU > bringup: > > [1.289404] #63 > [0.012468] kvm-clock: cpu 63, msr 6ff69fc1,

[PATCH v7 09/34] i2c: tegra: Use reset_control_reset()

2020-09-08 Thread Dmitry Osipenko
Use a single reset_control_reset() instead of assert/deasset couple in order to make code cleaner a tad. Note that the reset_control_reset() uses 1 microsecond delay instead of 2 that was used previously, but this shouldn't matter because one microsecond is a default reset time for most of Tegra

[PATCH v7 00/34] Improvements for Tegra I2C driver

2020-09-08 Thread Dmitry Osipenko
Hello! This series performs refactoring of the Tegra I2C driver code and hardens the atomic-transfer mode. Changelog: v7: - Reworked the "Clean up probe function" patch by moving out all variable renamings into the "Clean up variable names" patch. This results in a nicer diff, which

[PATCH v7 17/34] i2c: tegra: Remove outdated barrier()

2020-09-08 Thread Dmitry Osipenko
The barrier() was intended to reduce possibility of racing with the interrupt handler, but driver's code evolved significantly and today's driver enables interrupt only when it waits for completion notification. Hence barrier() has no good use anymore, let's remove it. Reviewed-by: Michał

[PATCH v7 07/34] i2c: tegra: Runtime PM always available on Tegra

2020-09-08 Thread Dmitry Osipenko
The runtime PM is guaranteed to be always available on Tegra after commit 40b2bb1b132a ("ARM: tegra: enforce PM requirement"). Hence let's remove all the RPM-availability checking and handling from the code. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry Osipenko ---

[PATCH v7 08/34] i2c: tegra: Remove error message used for devm_request_irq() failure

2020-09-08 Thread Dmitry Osipenko
The error message prints number of vIRQ, which isn't a useful information. In practice devm_request_irq() never fails, hence let's remove the bogus message in order to make code cleaner. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 4 +--- 1

[PATCH v7 06/34] i2c: tegra: Remove i2c_dev.clk_divisor_non_hs_mode member

2020-09-08 Thread Dmitry Osipenko
The "non_hs_mode" divisor value is fixed, thus there is no need to have the variable i2c_dev.clk_divisor_non_hs_mode struct member. Let's remove it and move the mode selection into tegra_i2c_init() where it can be united with the timing selection. Reviewed-by: Michał Mirosław Signed-off-by:

[PATCH v7 11/34] i2c: tegra: Use platform_get_irq()

2020-09-08 Thread Dmitry Osipenko
Use common helper for retrieval of the interrupt number in order to make code cleaner. Note that platform_get_irq() prints error message by itself. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 9 +++-- 1 file changed, 3 insertions(+), 6

[PATCH v7 01/34] i2c: tegra: Make tegra_i2c_flush_fifos() usable in atomic transfer

2020-09-08 Thread Dmitry Osipenko
The tegra_i2c_flush_fifos() shouldn't sleep in atomic transfer and jiffies are not updating if interrupts are disabled. Let's switch to use iopoll API helpers for register-polling. The iopoll API provides helpers for both atomic and non-atomic cases. Note that this patch doesn't fix any known

[PATCH v7 10/34] i2c: tegra: Use devm_platform_get_and_ioremap_resource()

2020-09-08 Thread Dmitry Osipenko
Driver now uses devm_platform_get_and_ioremap_resource() which replaces the typical boilerplate code and makes code cleaner. Reviewed-by: Michał Mirosław Signed-off-by: Dmitry Osipenko --- drivers/i2c/busses/i2c-tegra.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [PATCH v2 2/2] venus: firmware: Set virtual address ranges

2020-09-08 Thread Stanimir Varbanov
On 9/8/20 5:39 PM, Bjorn Andersson wrote: > On Mon 17 Aug 03:27 CDT 2020, Stanimir Varbanov wrote: > >> In order to boot some of the new Venus firmware versions TZ call to set >> virtual address ranges is needed. Add virtual address ranges for CP and >> CP_NONPIX in resource structure and use

Re: [PATCH v2] dt-bindings: convert spmi.txt to spmi.yaml

2020-09-08 Thread Rob Herring
On Wed, 26 Aug 2020 06:36:49 +0200, Mauro Carvalho Chehab wrote: > Convert the SPMI bus documentation to JSON/yaml. > > Signed-off-by: Mauro Carvalho Chehab > --- > > v2: > - addressed issues pointed by Rob; > - made clear that group ID is a future extension, that it is not > currently

Mailing list about low levels of Linux on cellphones

2020-09-08 Thread Pavel Machek
Hi! It seems there is quite a lot of efforts porting kernel to various cellphones. Librem 5 and PinePhone have their own hardware, people around Maemo Leste work with Nokia N900 and Droid 4, there's group working with Sony cellphones, there are postmarketOS people and there are probably groups I

RE: [RFC PATCH] cma: make number of CMA areas dynamic, remove CONFIG_CMA_AREAS

2020-09-08 Thread Song Bao Hua (Barry Song)
> -Original Message- > From: Mike Kravetz [mailto:mike.krav...@oracle.com] > Sent: Wednesday, September 9, 2020 6:29 AM > To: Song Bao Hua (Barry Song) ; > linux...@kvack.org; linux-kernel@vger.kernel.org; > linux-arm-ker...@lists.infradead.org; linux-m...@vger.kernel.org > Cc: Roman

Re: [PATCH v8 6/7] iommu/uapi: Handle data and argsz filled by users

2020-09-08 Thread Alex Williamson
On Mon, 31 Aug 2020 11:24:59 -0700 Jacob Pan wrote: > IOMMU user APIs are responsible for processing user data. This patch > changes the interface such that user pointers can be passed into IOMMU > code directly. Separate kernel APIs without user pointers are introduced > for in-kernel users of

RE: [PATCH net v2] hv_netvsc: Fix hibernation for mlx5 VF driver

2020-09-08 Thread Dexuan Cui
> From: Michael Kelley > Sent: Tuesday, September 8, 2020 1:49 PM > > @@ -2635,6 +2632,15 @@ static int netvsc_resume(struct hv_device *dev) > > netvsc_devinfo_put(device_info); > > net_device_ctx->saved_netvsc_dev_info = NULL; > > > > + /* A NIC driver (e.g. mlx5) may keep the VF

Re: [PATCH 1/3] clk: samsung: Add clk ID definitions for the CPU parent clocks

2020-09-08 Thread Rob Herring
On Wed, 26 Aug 2020 19:15:27 +0200, Sylwester Nawrocki wrote: > Add clock ID definitions for the CPU parent clocks for SoCs > which don't have such definitions yet. This will allow us to > reference the parent clocks directly by cached struct clk_hw > pointers in the clock provider, rather than

Re: [PATCH v2] fs: Handle I_DONTCACHE in iput_final() instead of generic_drop_inode()

2020-09-08 Thread Ira Weiny
On Fri, Sep 04, 2020 at 03:59:39PM +0800, Hao Li wrote: > If generic_drop_inode() returns true, it means iput_final() can evict > this inode regardless of whether it is dirty or not. If we check > I_DONTCACHE in generic_drop_inode(), any inode with this bit set will be > evicted unconditionally.

Re: [PATCH 1/2] dt-bindings: usb: dwc3-xilinx: Add documentation for Versal DWC3 Controller

2020-09-08 Thread Rob Herring
On Thu, Aug 27, 2020 at 12:14:00AM +0530, Manish Narani wrote: > Add documentation for Versal DWC3 controller. Add required property > 'reg' for the same. Also add optional properties for snps,dwc3. > > Signed-off-by: Manish Narani > --- > .../devicetree/bindings/usb/dwc3-xilinx.txt |

[PATCH v6 0/2] Add Intel LGM soc DMA support

2020-09-08 Thread Amireddy Mallikarjuna reddy
Add DMA controller driver for Lightning Mountain(LGM) family of SoCs. The main function of the DMA controller is the transfer of data from/to any DPlus compliant peripheral to/from the memory. A memory to memory copy capability can also be configured. This ldma driver is used for configure the

Re: Mailing list about low levels of Linux on cellphones

2020-09-08 Thread Bhaskar Chowdhury
On 00:56 Wed 09 Sep 2020, Pavel Machek wrote: Hi! It seems there is quite a lot of efforts porting kernel to various cellphones. Librem 5 and PinePhone have their own hardware, people around Maemo Leste work with Nokia N900 and Droid 4, there's group working with Sony cellphones, there are

[PATCH v6 3/3] arm64: Add IMA kexec buffer to DTB

2020-09-08 Thread Lakshmi Ramasubramanian
Any existing FDT_PROP_IMA_KEXEC_BUFFER property in the device tree needs to be removed and its corresponding memory reservation in the currently running kernel needs to be freed. The address and size of the current kernel's IMA measurement log need to be added to the device tree's IMA kexec

[PATCH v6 2/3] arm64: Store IMA log information in kimage used for kexec

2020-09-08 Thread Lakshmi Ramasubramanian
Address and size of the buffer containing the IMA measurement log need to be passed from the current kernel to the next kernel on kexec. Add address and size fields to "struct kimage_arch" for ARM64 platform to hold the address and size of the IMA measurement log buffer. Define an architecture

[PATCH v6 1/3] powerpc: Refactor kexec functions to move arch independent code to IMA

2020-09-08 Thread Lakshmi Ramasubramanian
The functions ima_get_kexec_buffer(), ima_free_kexec_buffer(), remove_ima_buffer(), and delete_fdt_mem_rsv() that handle carrying forward the IMA measurement logs on kexec for powerpc do not have architecture specific code, but they are currently defined for powerpc only. Move these functions to

[PATCH v6 0/3] Carry forward IMA measurement log on kexec on ARM64

2020-09-08 Thread Lakshmi Ramasubramanian
On kexec file load Integrity Measurement Architecture(IMA) subsystem may verify the IMA signature of the kernel and initramfs, and measure it. The command line parameters passed to the kernel in the kexec call may also be measured by IMA. A remote attestation service can verify the measurement

Re: [PATCH] fs: Eliminate a local variable to make the code more clear

2020-09-08 Thread Hao Lee
On Tue, Sep 08, 2020 at 07:48:57PM +0100, Al Viro wrote: > On Tue, Sep 08, 2020 at 01:06:56PM +, Hao Lee wrote: > > ping > > > > On Wed, Jul 29, 2020 at 03:21:28PM +, Hao Lee wrote: > > > The dentry local variable is introduced in 'commit 84d17192d2afd ("get > > > rid of full-hash scan on

[PATCH v2] PCI: hv: Fix hibernation in case interrupts are not re-created

2020-09-08 Thread Dexuan Cui
Hyper-V doesn't trap and emulate the accesses to the MSI/MSI-X registers, and we must use hv_compose_msi_msg() to ask Hyper-V to create the IOMMU Interrupt Remapping Table Entries. This is not an issue for a lot of PCI device drivers (e.g. NVMe driver, Mellanox NIC drivers), which destroy and

Re: [PATCH -next] selftests/seccomp: Use bitwise instead of arithmetic operator for flags

2020-09-08 Thread Kees Cook
On Wed, 15 Apr 2020 12:15:01 +0800, Zou Wei wrote: > This silences the following coccinelle warning: > > "WARNING: sum of probable bitmasks, consider |" > > tools/testing/selftests/seccomp/seccomp_bpf.c:3131:17-18: WARNING: sum of > probable bitmasks, consider | >

Re: [PATCH v2 27/28] x86, relocs: Ignore L4_PAGE_OFFSET relocations

2020-09-08 Thread Sami Tolvanen
On Thu, Sep 03, 2020 at 03:47:32PM -0700, Kees Cook wrote: > On Thu, Sep 03, 2020 at 01:30:52PM -0700, Sami Tolvanen wrote: > > L4_PAGE_OFFSET is a constant value, so don't warn about absolute > > relocations. > > > > Signed-off-by: Sami Tolvanen > > Any other details on this? I assume this is

Re: [ptrace] 201766a20e: kernel_selftests.seccomp.make_fail

2020-09-08 Thread Kees Cook
On Mon, Aug 05, 2019 at 12:47:19PM +0300, Dmitry V. Levin wrote: > On Mon, Jul 29, 2019 at 05:35:30PM +0800, kernel test robot wrote: > > FYI, we noticed the following commit (built with gcc-7): > > > > commit: 201766a20e30f982ccfe36bebfad9602c3ff574a ("ptrace: add > > PTRACE_GET_SYSCALL_INFO

Re: [PATCH] mm/mmu_notifier.c: micro-optimization substitute kzalloc with kmalloc

2020-09-08 Thread Jason Gunthorpe
On Tue, Sep 08, 2020 at 09:42:45AM +0300, Mike Rapoport wrote: > On Sun, Sep 06, 2020 at 06:06:39PM +0200, Mateusz Nosek wrote: > > Hi, > > > > I performed simple benchmarks using custom kernel module with the code > > fragment in question 'copy-pasted' in there in both versions. In case of 1k, >

[PATCH] net: qrtr: Reintroduce ARCH_QCOM as a dependency for QRTR

2020-09-08 Thread Anant Thazhemadam
Removing ARCH_QCOM, as a dependency for QRTR begins to give rise to issues with respect to maintaining reference count integrity and suspicious rcu usage. The bugs resolved by making QRTR dependent on ARCH_QCOM include: * WARNING: refcount bug in qrtr_node_lookup Reported-by:

Re: [PATCH next] io_uring: fix task hung in io_uring_setup

2020-09-08 Thread Jens Axboe
On 9/7/20 6:03 PM, Hillf Danton wrote: > > On Mon, 7 Sep 2020 06:55:04 Jens Axboe wrote: >> On 9/7/20 2:50 AM, Pavel Begunkov wrote: >>> >>> BTW, I don't see the patch itself, and it's neither in io_uring, block >>> nor fs mailing lists. Hillf, could you please CC proper lists next time? > >

Re: [PATCH] net: qrtr: Reintroduce ARCH_QCOM as a dependency for QRTR

2020-09-08 Thread Anant Thazhemadam
On 09/09/20 5:03 am, Anant Thazhemadam wrote: > Removing ARCH_QCOM, as a dependency for QRTR begins to give rise to > issues with respect to maintaining reference count integrity and > suspicious rcu usage. > > The bugs resolved by making QRTR dependent on ARCH_QCOM include: > > * WARNING:

Re: [PATCH v18 00/32] per memcg lru_lock: reviews

2020-09-08 Thread Hugh Dickins
Miscellaneous Acks and NAKs and other comments on the beginning and the end of the series, but not much yet on the all-important middle. I'm hoping to be spared sending ~20 email replies to ~20 patches. [PATCH v18 01/32] mm/memcg: warning on !memcg after readahead page charged Acked-by: Hugh

Re: [PATCH v2 1/1] samples/seccomp: eliminate two compile warnings in user-trap.c

2020-09-08 Thread Kees Cook
On Wed, Sep 02, 2020 at 09:33:06AM +0800, Leizhen (ThunderTown) wrote: > On 2020/9/1 16:39, Zhen Lei wrote: > > samples/seccomp/user-trap.c is compiled with $(userccflags), and the > > latter does not contain -fno-strict-aliasing, so the warnings reported as > > below. Due to add "userccflags +=

Re: [PATCH v2 00/28] Add support for Clang LTO

2020-09-08 Thread Sami Tolvanen
On Sun, Sep 06, 2020 at 09:24:38AM +0900, Masahiro Yamada wrote: > On Fri, Sep 4, 2020 at 5:30 AM Sami Tolvanen wrote: > > > > This patch series adds support for building x86_64 and arm64 kernels > > with Clang's Link Time Optimization (LTO). > > > > In addition to performance, the primary

[PATCH] drm: amd/display: fix spelling of "function"

2020-09-08 Thread Randy Dunlap
deletions(-) --- linux-next-20200908.orig/drivers/gpu/drm/amd/display/dc/dml/dcn20/display_rq_dlg_calc_20.h +++ linux-next-20200908/drivers/gpu/drm/amd/display/dc/dml/dcn20/display_rq_dlg_calc_20.h @@ -33,7 +33,7 @@ struct display_mode_lib; // Function: dml_rq_dlg_get_rq_reg // Main entry poin

Re: [PATCH v4 1/2] pinctrl: single: parse #pinctrl-cells = 2

2020-09-08 Thread Trent Piepho
On Tuesday, June 30, 2020 6:33:19 PM PDT Drew Fustini wrote: > If "pinctrl-single,pins" has 3 arguments (offset, conf, mux), then > pcs_parse_one_pinctrl_entry() does an OR operation on conf and mux to > get the value to store in the register. > - vals[found].val =

Re: linux-next: build warnings in Linus' tree

2020-09-08 Thread Stephen Rothwell
Hi Josh, On Tue, 8 Sep 2020 08:14:39 -0500 Josh Poimboeuf wrote: > > On Tue, Sep 08, 2020 at 09:11:02AM +1000, Stephen Rothwell wrote: > > > > Building Linus' tree, today's linux-next build (x86_64 allmodconfig) > > produced these warnings: > > > > arch/x86/kernel/cpu/mce/core.o: warning:

general protection fault in unlink_file_vma

2020-09-08 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:59126901 Merge tag 'perf-tools-fixes-for-v5.9-2020-09-03' .. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=1166cb5d90 kernel config: https://syzkaller.appspot.com/x/.config?x=3c5f6ce8d5b68299

[PATCH net-next 2/5] net: ipa: manage endpoints separate from clock

2020-09-08 Thread Alex Elder
Currently, when (before) the last IPA clock reference is dropped, all endpoints are suspended. And whenever the first IPA clock reference is taken, all endpoints are resumed (or started). In most cases there's no need to start endpoints when the clock starts. So move the calls to

[PATCH net-next 0/5] net: ipa: wake up system on RX available

2020-09-08 Thread Alex Elder
This series arranges for the IPA driver to wake up a suspended system if the IPA hardware has a packet to deliver to the AP. Currently, the GSI interrupt is set up to be a waking interrupt. But the GSI interrupt won't actually fire for a stopped channel (or a channel that underlies a suspended

[PATCH net-next 1/5] net: ipa: use atomic exchange for suspend reference

2020-09-08 Thread Alex Elder
We take a single IPA clock reference to keep the clock running until we get a system suspend operation. When a system suspend request arrives, we drop that reference, and if that's the last reference (likely) we'll proceed with suspending endpoints and disabling the IPA core clock and

[PATCH net-next 5/5] net: ipa: do not enable GSI interrupt for wakeup

2020-09-08 Thread Alex Elder
We now trigger a system resume when we receive an IPA SUSPEND interrupt. We should *not* wake up on GSI interrupts. Signed-off-by: Alex Elder --- drivers/net/ipa/gsi.c | 17 - drivers/net/ipa/gsi.h | 1 - 2 files changed, 4 insertions(+), 14 deletions(-) diff --git

[PATCH net-next 4/5] net: ipa: enable wakeup on IPA interrupt

2020-09-08 Thread Alex Elder
Now that we handle wakeup interrupts properly, arrange for the IPA interrupt to be treated as a wakeup interrupt. Signed-off-by: Alex Elder --- drivers/net/ipa/ipa_interrupt.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/net/ipa/ipa_interrupt.c

[PATCH net-next 3/5] net: ipa: use device_init_wakeup()

2020-09-08 Thread Alex Elder
The call to wakeup_source_register() in ipa_probe() does not do what it was intended to do. Call device_init_wakeup() in ipa_setup() instead, to set the IPA device as wakeup-capable and to initially enable wakeup capability. When we receive a SUSPEND interrupt, call pm_wakeup_dev_event() with a

Re: [PATCH net-next] rcu: prevent RCU_LOCKDEP_WARN() from swallowing the condition

2020-09-08 Thread Jakub Kicinski
On Tue, 08 Sep 2020 21:15:56 +0300 niko...@cumulusnetworks.com wrote: > Ah, you want to solve it for all. :) > Looks and sounds good to me, > Reviewed-by: Nikolay Aleksandrov Actually, I give up, lockdep_is_held() is not defined without CONFIG_LOCKDEP, let's just go with your patch..

Re: [PATCH v4 2/2] ARM: dts: am33xx-l4: change #pinctrl-cells from 1 to 2

2020-09-08 Thread Trent Piepho
On Tuesday, June 30, 2020 6:33:20 PM PDT Drew Fustini wrote: > Increase #pinctrl-cells to 2 so that mux and conf be kept separate. This > requires the AM33XX_PADCONF macro in omap.h to also be modified to keep pin > conf and pin mux values separate. > --- a/arch/arm/boot/dts/am33xx-l4.dtsi > +++

RE: [PATCH v7 2/3] dt-bindings: phy: intel: Add Keem Bay eMMC PHY bindings

2020-09-08 Thread Wan Mohamad, Wan Ahmad Zainie
Resend the reply. > -Original Message- > From: Vinod Koul > Sent: Tuesday, September 1, 2020 1:52 PM > To: Wan Mohamad, Wan Ahmad Zainie > > Cc: kis...@ti.com; robh...@kernel.org; Shevchenko, Andriy > ; eswara.k...@linux.intel.com; > vadivel.muruganx.ramuthe...@linux.intel.com; Raja

Re: [PATCH v2] drivers: input: Use single i2c_transfer transaction when using RM_CMD_BANK_SWITCH

2020-09-08 Thread Dmitry Torokhov
Hi Furquan, On Thu, Aug 20, 2020 at 07:40:06PM -0700, Furquan Shaikh wrote: > On an AMD chromebook, where the same I2C bus is shared by both Raydium > touchscreen and a trackpad device, it is observed that interleaving of > I2C messages when `raydium_i2c_read_message()` is called leads to the >

Re: linux-next: Tree for Sep 2 (lib/ubsan.c)

2020-09-08 Thread Stephen Rothwell
Hi Randy, On Tue, 8 Sep 2020 07:38:31 -0700 Randy Dunlap wrote: > > On 9/4/20 12:59 AM, Brendan Higgins wrote: > > On Thu, Sep 3, 2020 at 11:12 PM Randy Dunlap wrote: > > > >> > >> On 9/2/20 8:44 AM, Randy Dunlap wrote: > >>> On 9/2/20 1:09 AM, Stephen Rothwell wrote: > Hi all, >

Re: linux-next: Tree for Sep 2 (lib/ubsan.c)

2020-09-08 Thread Randy Dunlap
On 9/8/20 5:46 PM, Stephen Rothwell wrote: > Hi Randy, > > On Tue, 8 Sep 2020 07:38:31 -0700 Randy Dunlap wrote: >> >> On 9/4/20 12:59 AM, Brendan Higgins wrote: >>> On Thu, Sep 3, 2020 at 11:12 PM Randy Dunlap wrote: >>> On 9/2/20 8:44 AM, Randy Dunlap wrote: > On 9/2/20

Re: [PATCH 15/19] md: use bdev_check_media_change

2020-09-08 Thread Song Liu
On Tue, Sep 8, 2020 at 7:55 AM Christoph Hellwig wrote: > > The md driver does not have a ->revalidate_disk method, so it can just > use bdev_check_media_change without any additional changes. > > Signed-off-by: Christoph Hellwig > Reviewed-by: Johannes Thumshirn Acked-by: Song Liu > --- >

Re: [PATCH V2 2/5] iommu: Add iommu_dma_free_cpu_cached_iovas function

2020-09-08 Thread Lu Baolu
On 9/4/20 4:18 AM, Tom Murphy wrote: to dma-iommu ops Add a iommu_dma_free_cpu_cached_iovas function to allow drivers which use the dma-iommu ops to free cached cpu iovas. Signed-off-by: Tom Murphy --- drivers/iommu/dma-iommu.c | 9 + include/linux/dma-iommu.h | 3 +++ 2 files

Re: [PATCH 3/9] md: compare bd_disk instead of bd_contains

2020-09-08 Thread Song Liu
On Wed, Sep 2, 2020 at 10:43 PM Christoph Hellwig wrote: > > To check for partitions of the same disk bd_contains works as well, but > bd_disk is way more obvious. > > Signed-off-by: Christoph Hellwig Acked-by: Song Liu > --- > drivers/md/md.c | 7 +++ > 1 file changed, 3 insertions(+),

Re: [PATCH 4/9] md: don't detour through bd_contains for the gendisk

2020-09-08 Thread Song Liu
On Wed, Sep 2, 2020 at 10:43 PM Christoph Hellwig wrote: > > bd_disk is set on all block devices, including those for partitions. > > Signed-off-by: Christoph Hellwig Acked-by: Song Liu > --- > drivers/md/md.c | 2 +- > drivers/md/md.h | 2 +- > 2 files changed, 2 insertions(+), 2

Re: linux-next: Tree for Sep 2 (lib/ubsan.c)

2020-09-08 Thread Shuah Khan
On 9/8/20 6:49 PM, Randy Dunlap wrote: On 9/8/20 5:46 PM, Stephen Rothwell wrote: Hi Randy, On Tue, 8 Sep 2020 07:38:31 -0700 Randy Dunlap wrote: On 9/4/20 12:59 AM, Brendan Higgins wrote: On Thu, Sep 3, 2020 at 11:12 PM Randy Dunlap wrote: On 9/2/20 8:44 AM, Randy Dunlap wrote: On

[PATCH trivial] usb: phy: phy-ab8500-usb: fix spello of "function"

2020-09-08 Thread Randy Dunlap
From: Randy Dunlap Fix typo/spello of "function". Signed-off-by: Randy Dunlap Cc: Felipe Balbi Cc: linux-...@vger.kernel.org Cc: Jiri Kosina --- drivers/usb/phy/phy-ab8500-usb.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- linux-next-20200908.orig/drivers/usb/phy/

Re: [PATCH v18 31/32] mm: Add explicit page decrement in exception path for isolate_lru_pages

2020-09-08 Thread Matthew Wilcox
On Mon, Aug 24, 2020 at 08:55:04PM +0800, Alex Shi wrote: > +++ b/mm/vmscan.c > @@ -1688,10 +1688,13 @@ static unsigned long isolate_lru_pages(unsigned long > nr_to_scan, > > if (!TestClearPageLRU(page)) { > /* > -

Re: [PATCH] x86/msr: do not warn on writes to OC_MAILBOX

2020-09-08 Thread Srinivas Pandruvada
On Tue, 2020-09-08 at 21:30 +0200, Borislav Petkov wrote: > On Tue, Sep 08, 2020 at 12:18:38PM -0700, Sultan Alsawaf wrote: > > I'd like to point out that on Intel's recent 14nm parts, > > undervolting > > is not so much for squeezing every last drop of performance out of > > the > > SoC as it is

[PATCH] lib/mpi: fix spello of "functions"

2020-09-08 Thread Randy Dunlap
From: Randy Dunlap Fix typo/spello of "functions". Signed-off-by: Randy Dunlap Cc: Andrew Morton --- lib/mpi/mpi-bit.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- linux-next-20200908.orig/lib/mpi/mpi-bit.c +++ linux-next-20200908/lib/mpi/mpi-bit.c @@ -1,4 +1,4

Re: [PATCH v6 00/20] gpio: cdev: add uAPI v2

2020-09-08 Thread Kent Gibson
On Tue, Sep 08, 2020 at 10:04:05AM -0600, Shuah Khan wrote: > On 9/8/20 9:54 AM, Bartosz Golaszewski wrote: > > On Tue, Sep 8, 2020 at 5:24 PM Shuah Khan wrote: > > > > > > On 9/4/20 7:02 AM, Bartosz Golaszewski wrote: > > > > On Fri, Sep 4, 2020 at 2:52 PM Kent Gibson wrote: > > > > > > > > >

arch/powerpc/platforms/44x/ppc476.c:242:34: sparse: sparse: incorrect type in argument 1 (different address spaces)

2020-09-08 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 34d4ddd359dbcdf6c5fb3f85a179243d7a1cb7f8 commit: 8f28ca6bd8211214faf717677bbffe375c2a6072 iomap: constify ioreadX() iomem argument (as in generic implementation) date: 4 weeks ago config:

[PATCH] soundwire: Add generic bandwidth allocation algorithm

2020-09-08 Thread Bard Liao
This algorithm computes bus parameters like clock frequency, frame shape and port transport parameters based on active stream(s) running on the bus. Developers can also implement their own .compute_params() callback for specific resource management algorithm, and set if before calling

Re: [PATCH v6 00/20] gpio: cdev: add uAPI v2

2020-09-08 Thread Shuah Khan
On 9/8/20 7:09 PM, Kent Gibson wrote: On Tue, Sep 08, 2020 at 10:04:05AM -0600, Shuah Khan wrote: On 9/8/20 9:54 AM, Bartosz Golaszewski wrote: On Tue, Sep 8, 2020 at 5:24 PM Shuah Khan wrote: On 9/4/20 7:02 AM, Bartosz Golaszewski wrote: On Fri, Sep 4, 2020 at 2:52 PM Kent Gibson wrote:

Re: [PATCH v7 00/20] gpio: cdev: add uAPI v2

2020-09-08 Thread Kent Gibson
On Tue, Sep 08, 2020 at 06:07:49PM +0200, Bartosz Golaszewski wrote: > On Sat, Sep 5, 2020 at 3:36 PM Kent Gibson wrote: > > [ snip] > > > > drivers/gpio/Kconfig| 29 +- > > drivers/gpio/Makefile |2 +- > > drivers/gpio/gpiolib-cdev.c | 1277

Re: [PATCH v7 0/9] Add UEFI support for RISC-V

2020-09-08 Thread Palmer Dabbelt
On Fri, 28 Aug 2020 10:20:27 PDT (-0700), Atish Patra wrote: This series adds UEFI support for RISC-V. Linux kernel: v5.9-rc2 U-Boot: v2020.07 OpenSBI: master Patch 1-3 are generic riscv feature addition required for UEFI support. Patch 4-7 adds the efi stub support for RISC-V which was

RE: [PATCH] blkcg: add plugging support for punt bio

2020-09-08 Thread Tianxianting
Thanks TJ and your previous guide to me. I will summarize and resubmit the patch. -Original Message- From: Tejun Heo [mailto:hte...@gmail.com] On Behalf Of Tejun Heo Sent: Wednesday, September 09, 2020 2:03 AM To: tianxianting (RD) Cc: ax...@kernel.dk; cgro...@vger.kernel.org;

[PATCH v2 2/7] soundwire: bus: filter-out unwanted interrupt reports

2020-09-08 Thread Bard Liao
From: Pierre-Louis Bossart Unlike the traditional usage, in the SoundWire specification the interrupt masks only gate the propagation of an interrupt condition to the PING frame status. They do not gate the changes of the INT_STAT registers, which will happen regardless of the mask settings. See

[PATCH v2 0/7] ASoC/soundwire: filter out invalid PARITY errors

2020-09-08 Thread Bard Liao
Some codecs may report fake PARITY errors in the initial state. This series will filter them out. Pierre-Louis Bossart (7): ASoC/soundwire: bus: use property to set interrupt masks soundwire: bus: filter-out unwanted interrupt reports soundwire: slave: add first_interrupt_done status

Re: [PATCH 5.8 000/186] 5.8.8-rc1 review

2020-09-08 Thread Shuah Khan
On 9/8/20 9:22 AM, Greg Kroah-Hartman wrote: This is the start of the stable review cycle for the 5.8.8 release. There are 186 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made

[PATCH v2 4/7] soundwire: bus: use quirk to filter out invalid parity errors

2020-09-08 Thread Bard Liao
From: Pierre-Louis Bossart If a Slave device reports with a quirk that its initial parity check may be incorrect, filter it but keep the parity checks active in steady state. Signed-off-by: Pierre-Louis Bossart Reviewed-by: Kai Vehmanen Reviewed-by: Guennadi Liakhovetski Signed-off-by: Bard

Re: [PATCH 5.4 000/129] 5.4.64-rc1 review

2020-09-08 Thread Shuah Khan
On 9/8/20 9:24 AM, Greg Kroah-Hartman wrote: This is the start of the stable review cycle for the 5.4.64 release. There are 129 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made

[PATCH v2 7/7] soundwire: cadence: add parity error injection through debugfs

2020-09-08 Thread Bard Liao
From: Pierre-Louis Bossart The Cadence IP can inject errors, let's make use of this capability to test Slave parity error checks. See e.g. example log where both the master and slave detect the parity error injected on a dummy read command. cd /sys/kernel/debug/soundwire/master-1/intel-sdw/

Re: [PATCH 4.19 00/88] 4.19.144-rc1 review

2020-09-08 Thread Shuah Khan
On 9/8/20 9:25 AM, Greg Kroah-Hartman wrote: This is the start of the stable review cycle for the 4.19.144 release. There are 88 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made

Re: [PATCH 4.14 00/65] 4.14.197-rc1 review

2020-09-08 Thread Shuah Khan
On 9/8/20 9:25 AM, Greg Kroah-Hartman wrote: This is the start of the stable review cycle for the 4.14.197 release. There are 65 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made

Re: [PATCH V2 5/5] DO NOT MERGE: iommu: disable list appending in dma-iommu

2020-09-08 Thread Lu Baolu
Hi Christoph, On 9/8/20 2:23 PM, Christoph Hellwig wrote: On Tue, Sep 08, 2020 at 02:04:53PM +0800, Lu Baolu wrote: Do you mind telling where can I find Marek's series? [PATCH v10 00/30] DRM: fix struct sg_table nents vs. orig_nents misuse on various lists including the iommu one. It

Re: [IB/srpt] c804af2c1d: last_state.test.blktests.exit_code.143

2020-09-08 Thread Bart Van Assche
On 2020-09-08 11:22, Jason Gunthorpe wrote: > It is reasonable to consider the cq_pool as a built-in client, so I > would suggest moving it to right around the time the dynamic clients > are handled. Something like this: > > diff --git a/drivers/infiniband/core/device.c >

Re: [PATCH v1 1/1] scsi: ufshcd: Allow zero value setting to Auto-Hibernate Timer

2020-09-08 Thread Martin K. Petersen
Bao, > The zero value Auto-Hibernate Timer is a valid setting, and it > indicates the Auto-Hibernate feature being disabled. Correctly support > this setting. In addition, when this value is queried from sysfs, read > from the host controller's register and return that value instead of > using

Re: [PATCH v3 1/2] scsi: ibmvfc: use compiler attribute defines instead of __attribute__()

2020-09-08 Thread Martin K. Petersen
Tyrel, > Update ibmvfc.h structs to use the preferred __packed and __aligned() > attribute macros defined in include/linux/compiler_attributes.h in place > of __attribute__(). Applied 1+2 to my 5.10 staging tree. Thanks! -- Martin K. Petersen Oracle Linux Engineering

<    4   5   6   7   8   9   10   11   12   13   >