Re: [PATCH 1/2] dt-bindings: trivial: add tfa9879 device

2017-12-02 Thread Peter Rosin
On 2017-12-02 22:17, Fabio Estevam wrote: > On Fri, Dec 1, 2017 at 8:44 PM, Peter Rosin wrote: >> Add record for NXP TFA9879 Mono BTL Class D audio amplifier. >> >> Signed-off-by: Peter Rosin >> --- >> Documentation/devicetree/bindings/trivial-devices.txt | 1 + >> 1 file changed, 1 insertion(+)

Re: [PATCH] cpu/hotplug: merge cpuhp_bp_states&cpuhp_ap_states as cpuhp_hp_states

2017-12-02 Thread Paul E. McKenney
On Fri, Dec 01, 2017 at 09:50:05PM +0800, Lai Jiangshan wrote: > cpuhp_bp_states&cpuhp_ap_states have diffent set of steps > without any conflicting configed steps, so that they can > be merged. > > The original `[CPUHP_BRINGUP_CPU] = { },` is removed, because > the new cpuhp_hp_states has CPUHP_O

Re: [PATCH] refcount_t: documentation for memory ordering differences

2017-12-02 Thread Andrea Parri
On Sun, Dec 03, 2017 at 07:20:03AM +0100, Andrea Parri wrote: > On Fri, Dec 01, 2017 at 12:34:23PM -0800, Randy Dunlap wrote: > > On 11/29/2017 04:36 AM, Elena Reshetova wrote: > > > Some functions from refcount_t API provide different > > > memory ordering guarantees that their atomic counterparts

Re: [PATCH] refcount_t: documentation for memory ordering differences

2017-12-02 Thread Andrea Parri
On Fri, Dec 01, 2017 at 12:34:23PM -0800, Randy Dunlap wrote: > On 11/29/2017 04:36 AM, Elena Reshetova wrote: > > Some functions from refcount_t API provide different > > memory ordering guarantees that their atomic counterparts. > > This adds a document outlining these differences. > > > > Signe

Re: [PATCH] ARM: dts: keystone-k2l: Add the second gpio bank node

2017-12-02 Thread santosh.shilim...@oracle.com
On 11/13/17 9:12 AM, Santosh Shilimkar wrote: On 11/9/2017 3:42 AM, Keerthy wrote: In case of k2l there are 2 more banks with 16 pins each. Adding the node as the da-vinci driver now supports multiple banks. Signed-off-by: Keerthy --- This is based on linux-next branch. Boot tested on keyst

Re: PROBLEM: Asus C201 video mode problems on HDMI hotplug (regression)

2017-12-02 Thread Nick Bowler
Hi Jose, On 2017-12-02 17:11 +, Jose Abreu wrote: > On 01-12-2017 00:11, Nick Bowler wrote: > > Another data point... the following patch appears sufficient to > > restore working behaviour. [...] > I don't think you can do this. The phy pll lock check is > recommended and can indicate hw fail

Re: netfilter: xt_bpf: Fix XT_BPF_MODE_FD_PINNED mode of 'xt_bpf_info_v1'

2017-12-02 Thread Willem de Bruijn
>> OK... See vfs.git#untested.mkobj; it really needs testing, though - >> mq_open(2) >> passes LTP tests, but that's not saying much, and BPF side is completely >> untested. > > ... and FWIW, completely untested patch for net/netfilter/xt_bpf.c follows: Thanks a lot for this fix. The tree inclu

Re: [PATCH] staging: pi433: fix coding style in function rf69_set_rx_cfg()

2017-12-02 Thread kbuild test robot
Hi Victor, Thank you for the patch! Yet something to improve: [auto build test ERROR on staging/staging-testing] [also build test ERROR on v4.15-rc1 next-20171201] [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

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread John Hubbard
On 12/02/2017 02:19 PM, Matthew Wilcox wrote: > On Sat, Dec 02, 2017 at 07:49:20PM +0100, Jann Horn wrote: >> On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: >>> On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: [...] > > Maybe that should be up front rather than burie

Re: [PATCH 6/6] ARM: configs: keystone_defconfig: Enable few peripheral drivers

2017-12-02 Thread santosh.shilim...@oracle.com
On 11/22/17 11:51 PM, Vignesh R wrote: Enable drivers for QSPI, LEDS, gpio-decoder that are present on 66AK2G EVM and 66AK2G ICE boards. Signed-off-by: Vignesh R --- Please submit a patch also to enable all these peripherals in multi-v7 config. Just enable all remainder options enabled in keys

Re: [PATCH net-next v4 2/2] net: ethernet: socionext: add AVE ethernet driver

2017-12-02 Thread kbuild test robot
Hi Kunihiko, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Kunihiko-Hayashi/dt-bindings-net-add-DT-bindings-for-Socionext-UniPhier-AVE/20171203-095248 config: alpha-allyesconfig (attached as .

Re: New ORC unwinder in 4.14 broke cross-compilation?

2017-12-02 Thread Randy Dunlap
On 11/13/2017 10:33 AM, Andrew Randrianasulu wrote: > Hello! > > I was building my new 4.14 kernel, and on first instance I got working kernel > + > modules, booted ok, found my fb console was missing, recompiled kernel with > framebuffer console built-in (it resetted itself from M to N, becaus

Re: [GIT PULL] RISC-V Cleanups and ABI Fixes for 4.15-rc2

2017-12-02 Thread Andrea Parri
Hi Palmer, On Fri, Dec 01, 2017 at 01:39:12PM -0800, Palmer Dabbelt wrote: > The following changes since commit 4fbd8d194f06c8a3fd2af1ce560ddb31f7ec8323: > > Linux 4.15-rc1 (2017-11-26 16:01:47 -0800) > > are available in the git repository at: > > ssh://g...@gitolite.kernel.org/pub/scm/lin

Re: [PATCH RFC] stat.2: Document that stat can fail with EINTR

2017-12-02 Thread Keno Fischer
Resending as plain text (apologies for those receiving it twice, and those that got an HTML copy, I'm used to my mail client switching that over automatically, which for some reason didn't happen here). This is exactly the discussion I want to generate, so thank you. I should point out that I'm n

[PATCH] bluetooth: hci_ll: remove \n from kernel messages

2017-12-02 Thread David Lechner
The bt_* printk macros include a \n already, so we don't need extra ones here. Signed-off-by: David Lechner --- drivers/bluetooth/hci_ll.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/bluetooth/hci_ll.c b/drivers/bluetooth/hci_ll.c index 42142be..0f0a3a2 100644

[PATCH] bluetooth: serdev: hci_ll: Wait for CTS instead of using msleep

2017-12-02 Thread David Lechner
When a TI Bluetooth chip is reset, it takes about 100ms for the RTS line of the chip to deassert. For my use case with a TI CC2560A chip, this delay was not long enough and caused the local UART to never transmit at all (TI AM1808 SoC UART2). We can wait for the CTS signal using serdev_device_wait

Re: [PATCH net,stable v4 0/3] vhost: fix a few skb leaks

2017-12-02 Thread David Miller
From: w...@redhat.com Date: Fri, 1 Dec 2017 05:10:35 -0500 > Matthew found a roughly 40% tcp throughput regression with commit > c67df11f(vhost_net: try batch dequing from skb array) as discussed > in the following thread: > https://www.mail-archive.com/netdev@vger.kernel.org/msg187936.html Seri

Re: [kernel-hardening][PATCH v2 3/3] arm: mm: dump: add checking for writable and executable pages

2017-12-02 Thread park jinbum
I agree with your opinion, Laura. I'll make a new version to take advantage of the existing pg_level and bits arrays. Thanks, Jinbum Park. 2017-12-02 6:59 GMT+09:00 Laura Abbott : > On 12/01/2017 03:34 AM, Jinbum Park wrote: >> >> +static inline bool is_prot_ro(struct pg_state *st) >> +{ >> +

Re: [PATCH RFC] stat.2: Document that stat can fail with EINTR

2017-12-02 Thread Matthew Wilcox
On Sat, Dec 02, 2017 at 07:23:59PM -0500, Keno Fischer wrote: > The catalyst for this patch was me experiencing EINTR errors when > using the 9p file system. In linux commit 9523feac, the 9p file > system was changed to use wait_event_killable instead of > wait_event_interruptible, which does indee

Re: [PATCH net-next 0/2] net: dsa: cross-chip FDB support

2017-12-02 Thread David Miller
From: Vivien Didelot Date: Thu, 30 Nov 2017 12:56:41 -0500 > DSA can have interconnected switches. For instance, the ZII Dev Rev B > board described in arch/arm/boot/dts/vf610-zii-dev-rev-b.dts has a > switch fabric composed of 3 switch devices like this: > > lan4

Re: [PATCH net-next 0/4] net: dsa: simplify switchdev prepare phase

2017-12-02 Thread David Miller
From: Vivien Didelot Date: Thu, 30 Nov 2017 11:23:56 -0500 > This patch series brings no functional changes. > > It removes the unused switchdev_trans arguments from the dsa_switch_ops > for both MDB and VLAN operations, and provides functions to prepare and > add these objects for a given bitma

[PATCH v2 0/8] perf tools: perf tools: Clarify overwrite and backward, bugfix

2017-12-02 Thread Wang Nan
THe final result of this patchset is removing the concept of 'forward/backward', merge them into the concept of 'overwrite'. Patch 1 to 5 clear arguments lists of many functions, remove the 'overwrite'. Because all callers of these functions doesn't need the overwrite be set, we can simply remove

[PATCH v2 6/8] perf mmap: Fix perf backward recording

2017-12-02 Thread Wang Nan
perf record backward recording doesn't work as we expected: it never overwrite when ring buffer full. Test: (Run a busy printing task background like this: while True: print 123 send SIGUSR2 to perf to capture snapshot.) # ./perf record --overwrite -e raw_syscalls:sys_enter -e raw_sysca

[PATCH v2 7/8] perf tools: Don't discard prev in backward mode

2017-12-02 Thread Wang Nan
Perf record can switch output. The new output should only store the data after switching. However, in overwrite backward mode, the new output still have the data from old output. That also brings extra overhead. At the end of mmap_read, the position of processed ring buffer is saved in md->prev. N

[PATCH v2 5/8] perf tools: Remove overwrite and check_messup from mmap read

2017-12-02 Thread Wang Nan
All perf_mmap__read_forward() read from read-write ring buffer, so no need check_messup. Reading from backward ring buffer doesn't require check_messup because it never mess up. Cleanup arguments lists. Signed-off-by: Wang Nan --- tools/perf/util/evlist.c | 2 +- tools/perf/util/mmap.c | 28 +

[PATCH v2 4/8] perf tools: Remove overwrite from arguments list of perf_mmap__push

2017-12-02 Thread Wang Nan
'overwrite' argument is always 'false'. Revmove it from arguments list of perf_mmap__push. Signed-off-by: Wang Nan --- tools/perf/builtin-record.c | 2 +- tools/perf/util/mmap.c | 6 +++--- tools/perf/util/mmap.h | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/to

[PATCH v2 2/8] perf tools: Remove 'overwrite' parameter from perf_evlist__mmap_ex

2017-12-02 Thread Wang Nan
All users of perf_evlist__mmap_ex set !overwrite. Remove it from its arguments list. Signed-off-by: Wang Nan --- tools/perf/builtin-record.c | 2 +- tools/perf/util/evlist.c| 8 tools/perf/util/evlist.h| 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/tools/

[PATCH v2 8/8] perf tools: Replace 'backward' to 'overwrite' in evlist, mmap and record

2017-12-02 Thread Wang Nan
Remove the backward/forward concept to make it uniform with user interface (the '--overwrite' option). Signed-off-by: Wang Nan --- tools/perf/builtin-record.c | 14 +++--- tools/perf/tests/backward-ring-buffer.c | 4 ++-- tools/perf/util/evlist.c| 30

[PATCH v2 1/8] perf tools: Remove 'overwrite' parameter from perf_evlist__mmap

2017-12-02 Thread Wang Nan
Now all perf_evlist__mmap's users doesn't set 'overwrite'. Remove it from arguments list. Signed-off-by: Wang Nan --- tools/perf/arch/x86/tests/perf-time-to-tsc.c | 2 +- tools/perf/builtin-kvm.c | 2 +- tools/perf/builtin-top.c | 2 +- tools/perf/builtin-

[PATCH v2 3/8] perf tools: Remove evlist->overwrite

2017-12-02 Thread Wang Nan
evlist->overwrite is set to false in all users. It can be removed. Signed-off-by: Wang Nan --- tools/perf/builtin-record.c | 2 +- tools/perf/util/evlist.c| 5 ++--- tools/perf/util/evlist.h| 1 - 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/tools/perf/builtin-record.c

Re: [PATCH v18 05/10] xbitmap: add more operations

2017-12-02 Thread Tetsuo Handa
Matthew Wilcox wrote: > On Fri, Dec 01, 2017 at 03:09:08PM +, Wang, Wei W wrote: > > On Friday, December 1, 2017 9:02 PM, Tetsuo Handa wrote: > > > If start == end is legal, > > > > > >for (; start < end; start = (start | (IDA_BITMAP_BITS - 1)) + 1) { > > > > > > makes this loop do nothin

Re: [PATCH v18 05/10] xbitmap: add more operations

2017-12-02 Thread Tetsuo Handa
Matthew Wilcox wrote: > On Thu, Nov 30, 2017 at 10:35:03PM +0900, Tetsuo Handa wrote: > > According to xb_set_bit(), it seems to me that we are trying to avoid > > memory allocation > > for "struct ida_bitmap" when all set bits within a 1024-bits bitmap reside > > in the first > > 61 bits. > > >

Re: [PATCH 2/2] ipc: Fix ipc data structures inconsistency

2017-12-02 Thread Davidlohr Bueso
On Sat, 02 Dec 2017, Philippe Mikoyan wrote: On Fri, 1 Dec 2017 09:20:07 -0800 Davidlohr Bueso wrote: Hmm yeah that's pretty fishy, also shm_atime = 0, no? Yeah, definitely, other data structure fields can also be inconsistent, and applying not only to shmem, but also to other ipc mechani

Re: [GIT PULL] hash addresses printed with %p

2017-12-02 Thread Dave Young
On 12/02/17 at 10:22pm, Matt Fleming wrote: > (Cc'ing Dave since this is used for kexec on EFI) > > On Fri, 01 Dec, at 09:54:43AM, Ard Biesheuvel wrote: > > On 1 December 2017 at 09:48, Greg Kroah-Hartman > > wrote: > > > On Thu, Nov 30, 2017 at 05:18:42PM +, Ard Biesheuvel wrote: > > >> On 3

Re: [GIT PULL] revert ARM SCPI changes for v4.15-rc1

2017-12-02 Thread Olof Johansson
On Fri, Dec 01, 2017 at 11:53:05AM -0800, Kevin Hilman wrote: > Arnd, Olof, > > These ARM SCPI changes caused SCPI regressions resulting in CPUfreq > failures on most Amlogic SoCs (found by kernelci.org.) > > Unfortunately, this was not caught in linux-next due to other > bugs/panics on these pla

Re: [GIT PULL] ARM: uniphier: fixes for v4.15

2017-12-02 Thread Olof Johansson
On Wed, Nov 29, 2017 at 11:01:57PM +0900, Masahiro Yamada wrote: > Hi Arnd, Olof, > > Here are some fixes of ARM UniPhier SoC family. > There is a mistake in the IRQ number. Otherwise, I wouldn't have done > this early pull request. One more trivial patch to avoid conflict. > Also, MAINTAINERS u

Re: [GIT PULL] tee fix for v4.15

2017-12-02 Thread Olof Johansson
On Wed, Nov 29, 2017 at 11:16:50AM +0100, Jens Wiklander wrote: > Hi arm-soc maiantainers, > > Please pull this OP-TEE driver init fix for v4.15 > > Thanks, > Jens > > The following changes since commit 4fbd8d194f06c8a3fd2af1ce560ddb31f7ec8323: > > Linux 4.15-rc1 (2017-11-26 16:01:47 -0800) >

Re: [PATCH] arm64: dts: sort vendor subdirectories in Makefile alphabetically

2017-12-02 Thread Olof Johansson
On Sat, Nov 18, 2017 at 12:10:03PM +0900, Masahiro Yamada wrote: > The list is almost sorted. Move "lg" up to complete it. > > Signed-off-by: Masahiro Yamada Applied to fixes, thanks. -Olof

Re: [PATCH] iio: accel: bmc150: Add OF device ID table

2017-12-02 Thread Javier Martinez Canillas
Hello Jonathan, On 12/02/2017 01:02 PM, Jonathan Cameron wrote: > On Fri, 1 Dec 2017 12:10:58 +0100 > Javier Martinez Canillas wrote: > >> The driver doesn't have a struct of_device_id table but supported devices >> are registered via Device Trees. This is working on the assumption that a >> I2

RE: [PATCH 1/2] x86/mce/AMD: Define function to get SMCA bank type

2017-12-02 Thread Ghannam, Yazen
> -Original Message- > From: linux-edac-ow...@vger.kernel.org [mailto:linux-edac- > ow...@vger.kernel.org] On Behalf Of Borislav Petkov > Sent: Saturday, December 2, 2017 9:22 AM > To: Ghannam, Yazen > Cc: linux-e...@vger.kernel.org; Tony Luck ; > x...@kernel.org; linux-kernel@vger.kernel.

Re: [PATCH] fix system_state checking in early_ioremap

2017-12-02 Thread Dave Young
On 12/02/17 at 11:34am, Dave Young wrote: > Since below commit earlyprintk=efi,keep does not work any more with a warning > in mm/early_ioremap.c: WARN_ON(system_state >= SYSTEM_RUNNING): Should be WARN_ON(system_state != SYSTEM_BOOTING) in original code, copy paste wrongly, if need a resend pleas

[PATCH RFC] stat.2: Document that stat can fail with EINTR

2017-12-02 Thread Keno Fischer
Particularly on network file systems, a stat call may require submitting a message over the network and waiting interruptably for a reply. Signed-off-by: Keno Fischer --- The catalyst for this patch was me experiencing EINTR errors when using the 9p file system. In linux commit 9523feac, the 9p

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread John Hubbard
On 12/02/2017 02:19 PM, Matthew Wilcox wrote: > On Sat, Dec 02, 2017 at 07:49:20PM +0100, Jann Horn wrote: >> On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: >>> On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: MAP_FIXED has been widely used for a very long time,

[PATCHv2] staging: pi433: pi433_if.c remove SET_CHECKED macro

2017-12-02 Thread Nguyen Phan Quang Minh
The macro calls its argument -a function- twice, makes the calling function return prematurely -skipping resource cleanup code- and hurts understandability. Signed-off-by: Nguyen Phan Quang Minh --- Base on Dan's feedback, I checked and the macro return indeed skips over some cleanup code, hence

Re: [GIT PULL] hash addresses printed with %p

2017-12-02 Thread Matt Fleming
(Cc'ing Dave since this is used for kexec on EFI) On Fri, 01 Dec, at 09:54:43AM, Ard Biesheuvel wrote: > On 1 December 2017 at 09:48, Greg Kroah-Hartman > wrote: > > On Thu, Nov 30, 2017 at 05:18:42PM +, Ard Biesheuvel wrote: > >> On 30 November 2017 at 17:10, Greg Kroah-Hartman > >> wrote:

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread Matthew Wilcox
On Sat, Dec 02, 2017 at 07:49:20PM +0100, Jann Horn wrote: > On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: > > On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: > >> MAP_FIXED has been widely used for a very long time, yet the man > >> page still claims that "the use

[PATCH] objtool: Fix 64-bit build on 32-bit host

2017-12-02 Thread Josh Poimboeuf
From: Mikulas Patocka The new ORC unwinder breaks the build of a 64-bit kernel on a 32-bit host. Building the kernel on a i386 or x32 host fails with: orc_dump.c: In function 'orc_dump': orc_dump.c:105:26: error: passing argument 2 of 'elf_getshdrnum' from incompatible pointer type [-Werro

Re: netfilter: xt_bpf: Fix XT_BPF_MODE_FD_PINNED mode of 'xt_bpf_info_v1'

2017-12-02 Thread Al Viro
On Sat, Dec 02, 2017 at 06:48:50PM +, Al Viro wrote: > On Fri, Dec 01, 2017 at 09:47:00PM +0100, Daniel Borkmann wrote: > > > > Might want to replace security_path_mknod() with something saner, while > > > we are > > > at it. > > > > > > Objections? > > > > No, thanks for looking into this,

Re: [PATCH v2 11/18] drm/sun4i: Add A83T support

2017-12-02 Thread Jernej Škrabec
Dne četrtek, 30. november 2017 ob 16:33:12 CET je Maxime Ripard napisal(a): > On Tue, Nov 28, 2017 at 11:33:44PM +0100, Jernej Škrabec wrote: > > Hi! > > > > Dne torek, 28. november 2017 ob 23:00:14 CET je Maxime Ripard napisal(a): > > > On Tue, Nov 28, 2017 at 04:48:55PM +0100, Jernej Škrabec wro

[PATCH/RFC] DT: leds: Fix 'label' property description and add 'colour' property

2017-12-02 Thread Jacek Anaszewski
Label property was imposed a uniqueness requirement, which was erroneous, since ePAPR defines it to "a human readable string describing a device". Also the binding description misleadingly suggested direct usage of label for LED class device name, whereas it should only define a LED function. The

Re: [PATCH] Staging: pi433: fix brace coding style issues in pi433_if.c

2017-12-02 Thread Joe Perches
On Sat, 2017-12-02 at 22:05 +0100, Tomas Marek wrote: > This patch fix several brace on next line, braces not necessary, space > around =/<, and space before/after open/close parenthesis coding style > errors find by checkpatch in pi433_if.c. > > Signed-off-by: Tomas Marek > --- > drivers/stagin

[PATCH v1 2/2] PCI/ASPM: Enable Latency Tolerance Reporting when supported

2017-12-02 Thread Bjorn Helgaas
From: Bjorn Helgaas Enable Latency Tolerance Reporting (LTR). Note that LTR must be enabled in the Root Port first, and must not be enabled in any downstream device unless the Root Port and all intermediate Switches also support LTR. See PCIe r3.1, sec 6.18. Signed-off-by: Bjorn Helgaas --- d

[PATCH v1 1/2] PCI/ASPM: Calculate LTR_L1.2_THRESHOLD from device characteristics

2017-12-02 Thread Bjorn Helgaas
From: Bjorn Helgaas Per PCIe r3.1, sec 5.5.1, LTR_L1.2_THRESHOLD determines whether we enter the L1.2 Link state: if L1.2 is enabled and downstream devices have reported that they can tolerate latency of at least LTR_L1.2_THRESHOLD, we must enter L1.2 when CLKREQ# is de-asserted. The implication

[PATCH v1 0/2] PCI/ASPM: Refine L1 PM Substates support

2017-12-02 Thread Bjorn Helgaas
The PCIe active link power state is L0. ASPM defines two low-power states: L0s and L1. The L1 PM Substates feature defines two additional low-power states: L1.1 and L2.2. The L1.2 state may have substantial entry/exit latency. Downstream devices can use the Latency Tolerance Reporting (LTR) fea

Re: [alsa-devel] [PATCH 1/2] dt-bindings: trivial: add tfa9879 device

2017-12-02 Thread Fabio Estevam
On Fri, Dec 1, 2017 at 8:44 PM, Peter Rosin wrote: > Add record for NXP TFA9879 Mono BTL Class D audio amplifier. > > Signed-off-by: Peter Rosin > --- > Documentation/devicetree/bindings/trivial-devices.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/binding

[PATCH] Staging: pi433: fix brace coding style issues in pi433_if.c

2017-12-02 Thread Tomas Marek
This patch fix several brace on next line, braces not necessary, space around =/<, and space before/after open/close parenthesis coding style errors find by checkpatch in pi433_if.c. Signed-off-by: Tomas Marek --- drivers/staging/pi433/pi433_if.c | 130 --- 1

[ANNOUNCE] v4.14.3-rt5

2017-12-02 Thread Sebastian Andrzej Siewior
Dear RT folks! I'm pleased to announce the v4.14.3-rt5 patch set. Changes since v4.14.3-rt4: - More FPU loop unbreaks (enable/disable) in the SSE/AVX accelerate functions on x86. - Disabling NEON in kernel mode on ARM/ARM64. This is only avoid unbreaking the loops in NEON code as i

Re: [PATCH] fs/proc/kcore.c: use probe_kernel_read() instead of memcpy()

2017-12-02 Thread Kees Cook
On Sat, Dec 2, 2017 at 5:27 AM, Heiko Carstens wrote: > git commit df04abfd181a ("fs/proc/kcore.c: Add bounce buffer for ktext > data") added a bounce buffer to avoid hardened usercopy > checks. Copying to the bounce buffer was implemented with a simple > memcpy() assuming that it is always valid

Re: [PATCH 08/10] net: fjes: Handle return value of platform_get_irq and platform_get_resource

2017-12-02 Thread arvindY
Hi Sergei, On Sunday 03 December 2017 01:36 AM, Sergei Shtylyov wrote: Hello! On 12/02/2017 10:26 PM, Arvind Yadav wrote: platform_get_irq() and platform_get_resource() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/net/fjes/fjes_main.c | 10

Re: [PATCH v2] gpio: ftgpio010: Fix platform_get_irq's error checking

2017-12-02 Thread Linus Walleij
On Sat, Dec 2, 2017 at 6:01 PM, Arvind Yadav wrote: > The platform_get_irq() function returns negative if an error occurs. > zero or positive number on success. platform_get_irq() error checking > for zero is not correct. > > Signed-off-by: Arvind Yadav > --- > changes in v2: > It's

Re: [PATCH 05/10] net: ethernet: i825xx: Fix platform_get_irq's error checking

2017-12-02 Thread arvindY
Hi Sergei, On Sunday 03 December 2017 01:38 AM, Sergei Shtylyov wrote: Hello. On 12/02/2017 10:26 PM, Arvind Yadav wrote: The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Th

Re: [PATCH 05/10] net: ethernet: i825xx: Fix platform_get_irq's error checking

2017-12-02 Thread Sergei Shtylyov
Hello. On 12/02/2017 10:26 PM, Arvind Yadav wrote: The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. The why you consider returning 0 a sign of failure? Signed-off-by: Arvind

Re: [PATCH 07/10] net: ethernet: smsc: Handle return value of platform_get_irq

2017-12-02 Thread Sergei Shtylyov
On 12/02/2017 10:26 PM, Arvind Yadav wrote: platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/net/ethernet/smsc/smc911x.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/ethernet/smsc/smc911x.c b/drivers/net/et

Re: [PATCH 08/10] net: fjes: Handle return value of platform_get_irq and platform_get_resource

2017-12-02 Thread Sergei Shtylyov
Hello! On 12/02/2017 10:26 PM, Arvind Yadav wrote: platform_get_irq() and platform_get_resource() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/net/fjes/fjes_main.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/net/fjes/

Re: [PATCH v3] staging: xgifb: remove macros with hidden variable

2017-12-02 Thread Dan Carpenter
Much nicer. Thanks! regards, dan carpenter

[PATCH 01/10] net: bcmgenet: Fix platform_get_irq's error checking

2017-12-02 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 05/10] net: ethernet: i825xx: Fix platform_get_irq's error checking

2017-12-02 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/net/ethernet/i825xx/sni_82596.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) dif

[PATCH 03/10] net: ezchip: nps_enet: Fix platform_get_irq's error checking

2017-12-02 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. And remove unnecessary check for free_netdev(). Signed-off-by: Arvind Yadav --- drivers/net/ethernet/ezchip/nps_enet.c | 7 +++

[PATCH 08/10] net: fjes: Handle return value of platform_get_irq and platform_get_resource

2017-12-02 Thread Arvind Yadav
platform_get_irq() and platform_get_resource() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/net/fjes/fjes_main.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/net/fjes/fjes_main.c b/drivers/net/fjes/fjes_main.c index 750954b.

[PATCH 10/10] net: ethernet: cpmac: Handle return value of platform_get_irq_byname

2017-12-02 Thread Arvind Yadav
platform_get_irq_byname() can fail here and we must check its return value Signed-off-by: Arvind Yadav --- drivers/net/ethernet/ti/cpmac.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/net/ethernet/ti/cpmac.c b/drivers/net/ethernet/ti/cpmac.c index 9b8a30b..f3acfc0 100644 --- a

[PATCH 07/10] net: ethernet: smsc: Handle return value of platform_get_irq

2017-12-02 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/net/ethernet/smsc/smc911x.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/ethernet/smsc/smc911x.c b/drivers/net/ethernet/smsc/smc911x.c index 0515744..a1cf18c 1006

[PATCH 09/10] net: ethernet: korina: Handle return value of platform_get_irq_byname

2017-12-02 Thread Arvind Yadav
platform_get_irq_byname() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/net/ethernet/korina.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/net/ethernet/korina.c b/drivers/net/ethernet/korina.c index ae195f8..e778504 100644 --- a

[PATCH 06/10] net: ethernet: natsemi: Handle return value of platform_get_irq

2017-12-02 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/net/ethernet/natsemi/jazzsonic.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/ethernet/natsemi/jazzsonic.c b/drivers/net/ethernet/natsemi/jazzsonic.c index d5b288

[PATCH 02/10] net: bcmgenet: free netdev on of_match_node() error

2017-12-02 Thread Arvind Yadav
The change is to call free_netdev(), If of_match_node() will fail. Signed-off-by: Arvind Yadav --- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.c b/drivers/net/ethernet/broad

[PATCH 04/10] can: xilinx: Handle return value of platform_get_irq

2017-12-02 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/net/can/xilinx_can.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/net/can/xilinx_can.c b/drivers/net/can/xilinx_can.c index 89aec07..e36e2a2 100644 --- a/drivers/net/ca

[PATCH 00/10] Handle return value of platform_get_*

2017-12-02 Thread Arvind Yadav
- The platform_get_*_*() function returns negative if an error occurs. zero or positive number on success. platform_get_irq_byname() error checking for zero is not correct. - The change is to call free_netdev(), If of_match_node() will fail. - Handle return value of platform_get_resource()

Re: [PATCH] sched/isolation: Make NO_HZ_FULL select CPU_ISOLATION

2017-12-02 Thread Paul E. McKenney
On Sat, Dec 02, 2017 at 02:59:12PM +0100, Frederic Weisbecker wrote: > 2017-11-30 21:20 UTC+01:00, Paul E. McKenney : > > Commit 5c4991e24c69 ("sched/isolation: Split out new > > CONFIG_CPU_ISOLATION=y config from CONFIG_NO_HZ_FULL") can result in RCU > > CPU stall warnings when running rcutorture

Re: [PATCH v1] PCI: Make PCI_SCAN_ALL_PCIE_DEVS work for Root as well as Downstream Ports

2017-12-02 Thread Bjorn Helgaas
On Fri, Dec 01, 2017 at 06:27:10PM -0600, Bjorn Helgaas wrote: > From: Bjorn Helgaas > > PCIe Downstream Ports normally have only a Device 0 below them. To > optimize enumeration, we don't scan for other devices *unless* the > PCI_SCAN_ALL_PCIE_DEVS flag is set by set by quirks or the > "pci=pci

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread Jann Horn
On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: > On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: >> MAP_FIXED has been widely used for a very long time, yet the man >> page still claims that "the use of this option is discouraged". > > I think we should continue to d

Re: netfilter: xt_bpf: Fix XT_BPF_MODE_FD_PINNED mode of 'xt_bpf_info_v1'

2017-12-02 Thread Al Viro
On Fri, Dec 01, 2017 at 09:47:00PM +0100, Daniel Borkmann wrote: > > Might want to replace security_path_mknod() with something saner, while we > > are > > at it. > > > > Objections? > > No, thanks for looking into this, and sorry for this fugly hack! :( Not > that this doesn't make it any bett

RE: [PATCH net-next] net: hns3: Refactors "reset" handling code in HCLGE layer of HNS3 driver

2017-12-02 Thread Salil Mehta
Hi Andrew, > -Original Message- > From: Andrew Lunn [mailto:and...@lunn.ch] > Sent: Friday, December 01, 2017 1:44 PM > To: Salil Mehta > Cc: da...@davemloft.net; Zhuangyuzeng (Yisen) > ; lipeng (Y) ; > mehta.salil@gmail.com; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; li

Re: [PATCH] sched/rt: Do not do push/pull when there is only one CPU

2017-12-02 Thread Steven Rostedt
On Sat, 2 Dec 2017 12:12:52 -0500 Steven Rostedt wrote: > v2 coming up. Not really a v2, but a different approach. Look for the patch with the subject: "[PATCH] sched/rt: Do not pull from current CPU if only one cpu to pull" -- Steve

Re: [PATCH] staging: pi433: Make rf69_set_dc_cut_off_frequency_intern static

2017-12-02 Thread Joe Perches
On Sat, 2017-12-02 at 19:15 +0200, Marcus Wolf wrote: > Can you give me a hint, how to invoke the checkpatch.pl. There are lots of ways. I suggest reading the output of $ ./scripts/checkpatch.pl --help For instance: [ after the fact checking ] $ git checkout -b [ commit one or more changes

Re: [PATCH 1/1] timecounter: Make cyclecounter struct part of timecounter struct

2017-12-02 Thread Richard Cochran
On Sat, Dec 02, 2017 at 10:01:35AM +0530, Sagar Arun Kamble wrote: > There is no real need for the users of timecounters to define cyclecounter > and timecounter variables separately. Since timecounter will always be > based on cyclecounter, have cyclecounter struct as member of timecounter > struc

[PATCH] sched/rt: Do not pull from current CPU if only one cpu to pull

2017-12-02 Thread Steven Rostedt
Daniel Wagner reported a crash on the beaglebone black. This is a single CPU architecture, and does not have a functional: arch_send_call_function_single_ipi() and can crash if that is called. As it only has one CPU, it shouldn't be called, but if the kernel is compiled for SMP, the push/pull RT s

[PATCH] staging: android: add identifiers to function parameters

2017-12-02 Thread Ashish Kalra
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by: Ashish Kalra --- drivers/staging/android/ion/ion.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion.h b/drivers/st

Re: [PATCH] block, bfq: remove batches of confusing ifdefs

2017-12-02 Thread Paolo Valente
> Il giorno 02 dic 2017, alle ore 17:06, Jens Axboe ha > scritto: > > On 12/02/2017 03:04 AM, Paolo Valente wrote: >> >>> Il giorno 30 nov 2017, alle ore 22:21, Jens Axboe ha >>> scritto: >>> >>> On 11/28/2017 02:37 AM, Paolo Valente wrote: Commit a33801e8b473 ("block, bfq: move debug

[PATCH] staging: android: Fix checkpatch.pl error

2017-12-02 Thread Ashish Kalra
fixed CHECK: Lines should not end with a '(' reported by checkpatch.pl Signed-off-by: Ashish Kalra --- drivers/staging/android/ion/ion.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index fa4b75f4

Re: [PATCH] staging: pi433: Make rf69_set_dc_cut_off_frequency_intern static

2017-12-02 Thread Marcus Wolf
Am 02.12.2017 um 18:46 schrieb Joe Perches: On Sat, 2017-12-02 at 17:20 +0200, Marcus Wolf wrote: rf69_set_dc_cut_off_frequency_intern is used by rf69.c internally only. Therefore removed the function from header and declared it staic in the implemtation. Signed-off-by: Marcus Wolf --- driv

[PATCH v2] irq: Guard handle_bad_irq log messages

2017-12-02 Thread Guenter Roeck
An interrupt storm on a bad interrupt will cause the kernel log to be clogged. [ 60.089234] ->handle_irq(): be2f803f, [ 60.090455] 0xbf2af380 [ 60.090510] handle_bad_irq+0x0/0x2e5 [ 60.090522] ->irq_data.chip(): bf2af380, [ 60.090553]IRQ_NOPROBE set [ 60.09

Re: [PATCH] sched/rt: Do not do push/pull when there is only one CPU

2017-12-02 Thread Steven Rostedt
On Sat, 2 Dec 2017 13:53:31 +0100 Sebastian Andrzej Siewior wrote: > what about a check next to sched_feat(RT_PUSH_IPI)? I don't know if this > is a hot path or not (due to bitmap_weight). If it is, then I would > suggest something like a jump-label which is enabled if more than one > CPU has bee

Re: [PATCH] irq: Guard handle_bad_irq log messages

2017-12-02 Thread Guenter Roeck
On 12/02/2017 08:52 AM, Joe Perches wrote: On Sat, 2017-12-02 at 08:29 -0800, Guenter Roeck wrote: An interrupt storm on a bad interrupt will cause the kernel log to be clogged. [ 60.089234] ->handle_irq(): be2f803f, [ 60.090455] 0xbf2af380 [ 60.090510] handle_bad_irq+0x0

Re: PROBLEM: Asus C201 video mode problems on HDMI hotplug (regression)

2017-12-02 Thread Jose Abreu
Hi Nick, On 01-12-2017 00:11, Nick Bowler wrote: > Hi, > > On 2017-11-27 22:30 -0500, Nick Bowler wrote: >> A note about the test setup: I had to remove the test equipment so I >> no longer have any information about the video mode from the sink side >> (like in the photos). Thus, with the curren

[PATCH] kernel: irq: Extend lockdep class for request mutex

2017-12-02 Thread Andrew Lunn
The IRQ code already has support for lockdep class for the lock mutex in an interrupt descriptor. Extend this to add a second class for the request mutex in the descriptor. Not having a class is resulting in false positive splats in some code paths. Signed-off-by: Andrew Lunn --- arch/powerpc/sy

[PATCH v2] gpio: ftgpio010: Fix platform_get_irq's error checking

2017-12-02 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v2: It's should be 'irq <= 0' instead of 'irq < 0'. drivers/gpio/gpio-ftgpi

Re: [PATCH] staging: pi433: rf69.c: Introduced define DEBUG_FUNC_ENTRY

2017-12-02 Thread Marcus Wolf
Am 02.12.2017 um 17:00 schrieb Greg KH: On Sat, Dec 02, 2017 at 01:45:50PM +0200, Marcus Wolf wrote: Since dev_dbg already depends on define DEBUG, there was no sense, to enclose dev_dbg lines with #ifdef DEBUG. Instead of removing #ifdef DEBUG, I introduced define DEBUG_FUNC_ENTRY. So now it i

Re: [PATCH] irq: Guard handle_bad_irq log messages

2017-12-02 Thread Joe Perches
On Sat, 2017-12-02 at 08:29 -0800, Guenter Roeck wrote: > An interrupt storm on a bad interrupt will cause the kernel > log to be clogged. > > [ 60.089234] ->handle_irq(): be2f803f, > [ 60.090455] 0xbf2af380 > [ 60.090510] handle_bad_irq+0x0/0x2e5 > [ 60.090522] ->irq_data

Re: [PATCH] gpio: ftgpio010: Fix platform_get_irq's error checking

2017-12-02 Thread arvindY
Hi Linus, On Saturday 02 December 2017 09:00 PM, Linus Walleij wrote: Hi Arvind, thanks for the patch! On Thu, Nov 30, 2017 at 3:12 PM, Arvind Yadav wrote: The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error check

Re: [PATCH v2 22/35] nds32: Device tree support

2017-12-02 Thread Greentime Hu
2017-11-28 3:07 GMT+08:00 Rob Herring : > On Mon, Nov 27, 2017 at 6:28 AM, Greentime Hu wrote: >> From: Greentime Hu >> >> This patch adds support for device tree. >> >> Signed-off-by: Vincent Chen >> Signed-off-by: Greentime Hu >> --- >> arch/nds32/boot/dts/Makefile |8 ++ >> arch/n

Re: [PATCH] staging: pi433: Make rf69_set_dc_cut_off_frequency_intern static

2017-12-02 Thread Joe Perches
On Sat, 2017-12-02 at 17:20 +0200, Marcus Wolf wrote: > rf69_set_dc_cut_off_frequency_intern is used by rf69.c internally only. > Therefore removed the function from header and declared it staic in > the implemtation. > Signed-off-by: Marcus Wolf > --- > drivers/staging/pi433/rf69.c |2 +- >

  1   2   3   4   >