Re: [PATCH v9 4/7] clk: qcom: Add A53 PLL support

2017-10-25 Thread Bjorn Andersson
On Thu 21 Sep 09:49 PDT 2017, Georgi Djakov wrote: > diff --git a/drivers/clk/qcom/a53-pll.c b/drivers/clk/qcom/a53-pll.c [..] > + > +static const struct of_device_id qcom_a53pll_match_table[] = { > + { .compatible = "qcom,msm8916-a53pll" }, > + { } > +}; Move the match table below the pro

Re: [PATCH v9 3/7] mailbox: qcom: Move the apcs struct into a separate header

2017-10-25 Thread Bjorn Andersson
On Thu 21 Sep 09:49 PDT 2017, Georgi Djakov wrote: > Move the structure shared by the APCS IPC device and its subdevices > into a separate header file. > As you're creating the apcs regmap with devm_regmap_init_mmio() you can just call dev_get_regmap(dev->parent) in your child to get the handle.

Re: sched/fair: Hard lockup from idle_balance()/task_numa_migrate() race

2017-10-25 Thread Ed Swierk
Ping? On Mon, Oct 16, 2017 at 4:11 PM, Ed Swierk wrote: > > Ping for Peter, Ingo and other sched maintainers: > > I'd appreciate any feedback on this hard lockup issue, which occurred > on a system running kernel 4.4.52-grsec. > > To recap: a dual-socket Xeon (E5 v4) server system had been run

Re: [PATCH v9 7/7] clk: qcom: Add APCS clock controller support

2017-10-25 Thread Bjorn Andersson
On Thu 21 Sep 09:49 PDT 2017, Georgi Djakov wrote: > Add a driver for the APCS clock controller. It is part of the APCS > hardware block, which among other things implements also a combined > mux and half integer divider functionality. It can choose between a > fixed-rate clock or the dedicated AP

Re: [PATCH v9 1/7] mailbox: qcom: Convert APCS IPC driver to use regmap

2017-10-25 Thread Bjorn Andersson
On Thu 21 Sep 09:49 PDT 2017, Georgi Djakov wrote: > This hardware block provides more functionalities that just IPC. Convert > it to regmap to allow other child platform devices to use the same regmap. > Acked-by: Bjorn Andersson Regards, Bjorn

Re: n900 in next-20170901

2017-10-25 Thread Joonsoo Kim
On Wed, Oct 25, 2017 at 10:31:38AM -0700, Tony Lindgren wrote: > * Joonsoo Kim [171022 21:51]: > > On Fri, Oct 20, 2017 at 10:31:47AM -0700, Tony Lindgren wrote: > > > * Joonsoo Kim [171019 18:53]: > > > > Oops... I made a mistak. Could you test with reverting commit > > > > c977ee2803787363187d6

Re: [PATCH v3] rpmsg: Allow RPMSG_VIRTIO to be enabled via menuconfig or defconfig

2017-10-25 Thread Bjorn Andersson
On Thu 05 Oct 09:43 PDT 2017, Anup Patel wrote: > Currently, RPMSG_VIRTIO can only be enabled if some other kconfig > option selects it. This does not allow it to be enabled for > virtualized systems where Virtio RPMSG is available over Virtio > MMIO or PCI transport. > > This patch updates RPMSG

Re: [PATCH v3 5/6] dt-bindings: add the rockchip, dual-channel for dw-mipi-dsi

2017-10-25 Thread Archit Taneja
On 10/25/2017 09:21 AM, Nickey Yang wrote: Configure dsi slave channel when driving a panel which needs 2 DSI links. Signed-off-by: Nickey Yang --- .../devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/d

Re: [PATCH v2] sparc: Provide cmpxchg64 for 32-bit builds

2017-10-25 Thread Joe Perches
On Wed, 2017-10-25 at 19:51 -0700, Guenter Roeck wrote: > Fix the following build error, seen when building sparc32:allmodconfig. > > drivers/net/ethernet/intel/i40e/i40e_ethtool.c: > In function 'i40e_set_priv_flags': > drivers/net/ethernet/intel/i40e/i40e_ethtool.c:4150:2: error: > i

[PATCH] coccinelle: grep Options and Requires fields more precisely

2017-10-25 Thread Masahiro Yamada
Currently, the required version for badzero.cocci is picked up from its "Comments:" line since it contains the word "Requires". Surprisingly, ld-version.sh can extract the version number from the string "Requires Coccinelle version 1.0.0-rc20 or later", but this expectation is fragile. Fix the .c

[PATCH] coccinelle: fix verbose message about .cocci file being run

2017-10-25 Thread Masahiro Yamada
If you run coccicheck with V=1 and COCCI=, you will see a strange path to the semantic patch file. For example, run the following: $ make V=1 COCCI=scripts/coccinelle/free/kfree.cocci coccicheck [ snip ] The semantic patch that makes this report is available in scriptcoccinelle/free/kfree.coc

Re: [PATCH V8 1/2] printk: remove tabular output for NULL pointer

2017-10-25 Thread Joe Perches
On Thu, 2017-10-26 at 13:53 +1100, Tobin C. Harding wrote: > Currently pointer() checks for a NULL pointer argument and then if so > attempts to print "(null)" with _some_ standard width. This width cannot > correctly be ascertained here because many of the printk specifiers > print pointers of var

[RFC PATCH] coccinelle: fix parallel build with CHECK=scripts/coccicheck

2017-10-25 Thread Masahiro Yamada
If I run the following: $ make -j8 C=1 CHECK=scripts/coccicheck lots of "coccicheck failed" messages are displayed. I do not know the coccinell internals, but I guess --jobs does not work well if spatch is invoked from Make running in parallel. Disable --jobs in this case. Signed-off-by: Masa

[PATCH] [irq] Fix boot failure when irqaffinity is passed.

2017-10-25 Thread Rakib Mullick
When irqaffinity kernel param is passed in a CPUMASK_OFFSTACK=y build kernel, it fails to boot. zalloc_cpumask_var() cannot be used before initializing mm stuff (slab allocator) to allocate cpumask. So, use alloc_bootmem_cpumask_var(). Also in init_irq_default_affinity() removes unneeded ifdef, the

Re: [PATCH] coccinelle: grep Options and Requires fields more precisely

2017-10-25 Thread Julia Lawall
On Thu, 26 Oct 2017, Masahiro Yamada wrote: > Currently, the required version for badzero.cocci is picked up from > its "Comments:" line since it contains the word "Requires". > > Surprisingly, ld-version.sh can extract the version number from the > string "Requires Coccinelle version 1.0.0-rc20

Re: [fstests PATCH v3] generic: add test for DAX MAP_SYNC support

2017-10-25 Thread Amir Goldstein
On Wed, Oct 25, 2017 at 11:47 PM, Ross Zwisler wrote: > Add a test that exercises DAX's new MAP_SYNC flag. > > This test creates a file and writes to it via an mmap(), but never syncs > via fsync/msync. This process is tracked via dm-log-writes, then replayed. > > If MAP_SYNC is working the dm-lo

Re: I2C-Octeon: Use common error handling code in octeon_i2c_probe()

2017-10-25 Thread SF Markus Elfring
> NAK.  We don't need any more Markus Elfring spam. Why do you not like another update suggestion for this use case? >> Add a jump target so that a specific error message is stored only once >> at the end of this function implementation. > > Please read C standard about duplicate literal string

Re: [PATCH 1/3] usb: host: remove ehci-msm.c

2017-10-25 Thread Manu Gautam
Hi, On 10/26/2017 3:31 AM, Alex Elder wrote: > No Qualcomm SoC requires the "ehci-msm.c" code any more. So remove it. > > Suggested-by: Stephen Boyd > Signed-off-by: Alex Elder > Acked-by: Bjorn Andersson > Acked-by: Andy Gross > --- > drivers/usb/host/Makefile | 1 - > drivers/usb/host

Re: [PATCH v3 4/6] drm/rockchip/dsi: add dual mipi channel support

2017-10-25 Thread Archit Taneja
On 10/25/2017 01:34 PM, Sean Paul wrote: On Wed, Oct 25, 2017 at 11:51:01AM +0800, Nickey Yang wrote: This patch add dual mipi channel support: 1.add definition of dsi1 register and grf operation. 2.dsi0 and dsi1 will work in master and slave mode when driving dual mipi panel. Signed-off-by:

Re: [PATCH 2/3] usb: phy: remove phy-msm-usb.c

2017-10-25 Thread Manu Gautam
Hi, On 10/26/2017 3:31 AM, Alex Elder wrote: > No Qualcomm SoC requires the "phy-msm-usb.c" USB phy driver support > any more, so remove the code. > > Suggested-by: Stephen Boyd > Signed-off-by: Alex Elder > Acked-by: Bjorn Andersson > Acked-by: Andy Gross > --- > drivers/usb/phy/Makefile

[PATCH v2] Bluetooth: hci_ldisc: Allow sleeping while proto locks are held.

2017-10-25 Thread =?UTF-8?q?Ronald=20Tschal=C3=A4r?=
Commit dec2c92880cc5435381d50e3045ef018a762a917 ("Bluetooth: hci_ldisc: Use rwlocking to avoid closing proto races") introduced locks in hci_ldisc that are held while calling the proto functions. These locks are rwlock's, and hence do not allow sleeping while they are held. However, the proto funct

[PATCH v4] Add /proc/PID/smaps support for DAX

2017-10-25 Thread Fan Du
Memory behind device DAX is not attached into normal memory management system, when user mmap /dev/dax, smaps part is currently missing, so no idea for user to check how much device DAX memory are actually used in practice. Whether vma is backed up by normal page,huge page, or both at the same tim

[PATCH] Bluetooth: hci_ldisc: Fix another race when closing the tty.

2017-10-25 Thread =?UTF-8?q?Ronald=20Tschal=C3=A4r?=
The following race condition still existed: P1P2 cancel_work_sync() hci_uart_tx_wakeup() hci_uart_write_work() hci_uart_dequeue() clear_bit(HC

[PATCH] power: supply: 88pm860x_battery array_soc first number is in mV

2017-10-25 Thread winton.liu
Fix wrong comments of array_soc description. First number is mV not mAh. Signed-off-by: winton.liu <18502523...@163.com> --- drivers/power/supply/88pm860x_battery.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/power/supply/88pm860x_battery.c b/drivers/power/supply/

Re: v4.14-rc3/arm64 DABT exception in atomic_inc() / __skb_clone()

2017-10-25 Thread Jason Wang
On 2017年10月26日 03:01, Eric Dumazet wrote: On Wed, Oct 25, 2017 at 11:49 AM, Willem de Bruijn wrote: From skb->dev and netdev_priv, the tun device has flags 0x1002 == IFF_TAP | IFF_NO_PI. This kernel precedes the recent support for IFF_NAPI and IFF_NAPI_FRAGS. The allocation most likely happ

Re: [PATCH] power: supply: 88pm860x_battery array_soc first number is in mV

2017-10-25 Thread Joe Perches
On Thu, 2017-10-26 at 13:21 +0800, winton.liu wrote: > Fix wrong comments of array_soc description. > First number is mV not mAh. [] > diff --git a/drivers/power/supply/88pm860x_battery.c > b/drivers/power/supply/88pm860x_battery.c [] > @@ -123,7 +123,7 @@ struct ccnt { > > /* > * State of Ch

[PATCH RESEND] f2fs: support soft block reservation

2017-10-25 Thread Chao Yu
From: Yunlong Song It supports to extend reserved_blocks sysfs interface to be soft threshold, which allows user configure it exceeding current available user space. Signed-off-by: Yunlong Song Signed-off-by: Chao Yu --- Documentation/ABI/testing/sysfs-fs-f2fs | 3 ++- fs/f2fs/f2fs.h

Re: [PATCH v5 9/9] block: Assign a lock_class per gendisk used for wait_for_completion()

2017-10-25 Thread Ingo Molnar
* Jens Axboe wrote: > On 10/25/2017 03:13 AM, Ingo Molnar wrote: > > > > * Byungchul Park wrote: > > > >> Darrick posted the following warning and Dave Chinner analyzed it: > >> > >>> == > >>> WARNING: possible circular locking dependency de

scsi/NCR5380: Suppress SDTR and WDTR message logging

2017-10-25 Thread Finn Thain
The 5380 drivers only support asynchronous transfers and the 5380 controllers only have narrow busses. Hence, the core driver will reject any SDTR and WDTR messages from target devices. Don't log this, it's expected behaviour. Also, fix the off-by-one array indices in the arguments to scmd_print

Re: [PATCH v5 9/9] block: Assign a lock_class per gendisk used for wait_for_completion()

2017-10-25 Thread Byungchul Park
On Thu, Oct 26, 2017 at 07:50:46AM +0200, Ingo Molnar wrote: > > * Jens Axboe wrote: > > > On 10/25/2017 03:13 AM, Ingo Molnar wrote: > > > > > > * Byungchul Park wrote: > > > > > >> Darrick posted the following warning and Dave Chinner analyzed it: > > >> > > >>>

[PATCH] iio/adc/palmas: Use common error handling code in palmas_gpadc_calibrate()

2017-10-25 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 26 Oct 2017 07:58:03 +0200 * Add a jump target so that a specific error message is stored only once at the end of this function implementation. * Adjust condition checks. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH] scripts: add leaking_addresses.pl

2017-10-25 Thread Tobin C. Harding
Currently we are leaking addresses from the kernel to user space. This script is an attempt to find those leakages. Script parses `dmesg` output and /proc and /sys files for hex strings that look like kernel addresses. Only works for 64 bit kernels, the reason being that kernel addresses on 64 bit

Re: [PATCH] kernel/kprobes: add check to avoid kprobe memory leak

2017-10-25 Thread Masami Hiramatsu
On Thu, 26 Oct 2017 09:22:23 +0800 chenjiankang wrote: > > On Tue, 24 Oct 2017 20:17:02 +0800 > > JianKang Chen wrote: > > > >> The function register_kretprobe is used to initialize a struct > >> kretprobe and allocate a list table for kprobe instance. > >> However,in this function, there is a

Re: 4.13 (and probably all recent) kernels refuse to boot on one Nokia N950, work or another

2017-10-25 Thread Filip Matijević
Hi Pavel, On 10/25/2017 10:34 PM, Pavel Machek wrote: > 3.5.3-nemo kernel seems to work I did take a quick look at the board-rm680.c since there are some conditional statements with regards to system revision but I didn't notice anything that might prevent it from booting. > If you have > any id

Re: [PATCH v2] x86/mm/64: Rename the register_page_bootmem_memmap() 'size' parameter to 'nr_pages'

2017-10-25 Thread Ingo Molnar
* Baoquan He wrote: > register_page_bootmem_memmap()'s 3rd 'size' parameter is named > in a somewhat misleading fashion - rename it to 'nr_pages' which > makes the units of it much clearer. > > And also rename the existing local variable 'nr_pages' to 'pages'. > Otherwise building error will be

Re: [PATCH V8 1/2] printk: remove tabular output for NULL pointer

2017-10-25 Thread Tobin C. Harding
Hi Joe, thanks for your review. On Wed, Oct 25, 2017 at 09:57:23PM -0700, Joe Perches wrote: > On Thu, 2017-10-26 at 13:53 +1100, Tobin C. Harding wrote: > > Currently pointer() checks for a NULL pointer argument and then if so > > attempts to print "(null)" with _some_ standard width. This width

Re: [PATCH V9 1/4] dma-mapping: Rework dma_get_cache_alignment()

2017-10-25 Thread 陈华才
Maybe my first version is suitable for stable. Huacai -- Original -- From: "Marek Szyprowski"; Date: Wed, Oct 25, 2017 03:21 PM To: "陈华才"; "Christoph Hellwig"; Cc: "Robin Murphy"; "Andrew Morton"; "Fuxin Zhang"; "linux-kernel"; "Ralf Baechle"; "JamesHoga

Re: [PATCH] x86/mm/64: Rename the argument 'size' as 'nr_pages' in register_page_bootmem_memmap

2017-10-25 Thread kbuild test robot
Hi Baoquan, [auto build test ERROR on tip/x86/core] [also build test ERROR on v4.14-rc6 next-20171018] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Baoquan-He/x86-mm-64-Rename-the-argument-siz

Re: [PATCH 1/3] [S390] vmur: convert urdev.ref_count from atomic_t to refcount_t

2017-10-25 Thread Martin Schwidefsky
On Fri, 20 Oct 2017 10:47:48 +0300 Elena Reshetova wrote: > atomic_t variables are currently used to implement reference > counters with the following properties: > - counter is initialized to 1 using atomic_set() > - a resource is freed upon counter reaching zero > - once counter reaches zero

Re: [PATCH] net: xfrm_user: use BUG_ON instead of if condition followed by BUG

2017-10-25 Thread Herbert Xu
On Wed, Oct 25, 2017 at 07:38:35PM -0500, Gustavo A. R. Silva wrote: > > I don't think we want something like: > > bool err; > > err = build_spdinfo(r_skb, net, sportid, seq, *flags) < 0 ? true : false; > BUG_ON(err); How about int err; err = build_spdinfo(r_skb, net, sportid,

Re: [PATCH] s390/ap_bus: Convert timers to use timer_setup()

2017-10-25 Thread Martin Schwidefsky
On Wed, 25 Oct 2017 03:27:37 -0700 Kees Cook wrote: > In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks, switch to using the new timer_setup() and from_timer() > to pass the timer pointer explicitly. > > Cc: Harald Freudenberger > Cc: Martin Schwi

Re: [PATCH v2] x86/mm/64: Rename the register_page_bootmem_memmap() 'size' parameter to 'nr_pages'

2017-10-25 Thread Baoquan He
On 10/26/17 at 08:25am, Ingo Molnar wrote: > > * Baoquan He wrote: > > > register_page_bootmem_memmap()'s 3rd 'size' parameter is named > > in a somewhat misleading fashion - rename it to 'nr_pages' which > > makes the units of it much clearer. > > > > And also rename the existing local variabl

Re: [PATCH 1/2] hwmon: (jc42) optionally try to disable the SMBUS timeout

2017-10-25 Thread Peter Rosin
On 2017-10-18 04:38, Guenter Roeck wrote: > On 10/17/2017 03:16 PM, Rob Herring wrote: >> On Fri, Oct 13, 2017 at 01:35:27PM -0700, Guenter Roeck wrote: >>> On Fri, Oct 13, 2017 at 04:26:57PM +0200, Peter Rosin wrote: On 2017-10-13 15:50, Guenter Roeck wrote: > On 10/13/2017 02:27 AM, Pete

[PATCH v9 1/2] media:imx274 device tree binding file

2017-10-25 Thread Leon Luo
The binding file for imx274 CMOS sensor V4l2 driver Signed-off-by: Leon Luo Acked-by: Sören Brinkmann Acked-by: Rob Herring --- v9: - add an entry in MAINTAINERS v8: - no changes v7: - no changes v6: - no changes v5: - add 'port' and 'endpoint' information v4: - no changes v3: - remove r

Re: [PATCH 1/4] usb: xhci: Fix build warning

2017-10-25 Thread Greg KH
On Wed, Oct 25, 2017 at 06:32:26PM +, Corentin Labbe wrote: > This patch fix the following build warnings: > drivers/usb/host/xhci.c:3378:6: warning: variable 'last_freed_endpoint' set > but not used [-Wunused-but-set-variable] > > Signed-off-by: Corentin Labbe > --- > drivers/usb/host/xhci

[PATCH v9 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor

2017-10-25 Thread Leon Luo
The imx274 is a Sony CMOS image sensor that has 1/2.5 image size. It supports up to 3840x2160 (4K) 60fps, 1080p 120fps. The interface is 4-lane MIPI CSI-2 running at 1.44Gbps each. This driver has been tested on Xilinx ZCU102 platform with a Leopard LI-IMX274MIPI-FMC camera board. Support for the

Re: [PATCH] Bluetooth: hci_ldisc: Fix another race when closing the tty.

2017-10-25 Thread Lukas Wunner
On Wed, Oct 25, 2017 at 10:15:19PM -0700, =?UTF-8?q?Ronald=20Tschal=C3=A4r?= wrote: > --- a/drivers/bluetooth/hci_ldisc.c > +++ b/drivers/bluetooth/hci_ldisc.c > @@ -523,13 +523,13 @@ static void hci_uart_tty_close(struct tty_struct *tty) > if (hdev) > hci_uart_close(hdev); >

Re: [PATCH 0/2] Fix s5p-mfc lock contention in request firmware paths

2017-10-25 Thread Shuah Khan
On 10/25/2017 04:28 PM, Marian Mihailescu wrote: > Hi Shuah, > > For MFC patch, you can delete the "dev" variable since it's not being > used anymore and results in a compile warning. > > - struct s5p_mfc_dev *dev = ctx->dev; > > Cheers, > Marian Oops. I thought I handled that. I will fix that

Re: [PATCH] x86/mm/64: Rename the argument 'size' as 'nr_pages' in register_page_bootmem_memmap

2017-10-25 Thread Baoquan He
On 10/26/17 at 02:33pm, kbuild test robot wrote: > Hi Baoquan, > > [auto build test ERROR on tip/x86/core] > [also build test ERROR on v4.14-rc6 next-20171018] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the system] > > url: > https://github.com

<    3   4   5   6   7   8