Re: [PATCH 1/5] mm/vmalloc.c: correct a few logic error for __insert_vmap_area()

2016-09-27 Thread zijun_hu
On 09/22/2016 07:15 AM, David Rientjes wrote: > On Thu, 22 Sep 2016, zijun_hu wrote: > >>> We don't support inserting when va->va_start == tmp_va->va_end, plain and >>> simple. There's no reason to do so. NACK to the patch. >>> >> i am sorry i disagree with you because >> 1) in almost all

[PATCH] staging: ks7010: Use __packed over __attribute__((packed))

2016-09-27 Thread Matt Kilgore
This replaces uses of __attribute__((packed)) with __packed, which is recommended to be used over the direct __attribute__. This patch then includes as necessary to use __packed. Signed-off-by: Matthew Kilgore --- drivers/staging/ks7010/eap_packet.h | 10 ++--

Re: [PATCH] kcov: properly check if we are in an interrupt

2016-09-27 Thread Dmitry Vyukov
On Tue, Sep 27, 2016 at 1:32 AM, Andrew Morton wrote: > On Fri, 23 Sep 2016 16:51:13 +0200 Andrey Konovalov > wrote: > >> in_interrupt() returns a nonzero value when we are either in an >> interrupt or have bh disabled via local_bh_disable(). Since we are >> interested in only ignoring coverage

Re: [PATCH] arm64, numa: Add cpu_to_node() implementation.

2016-09-27 Thread Hanjun Guo
On 09/20/2016 09:21 PM, Robert Richter wrote: On 20.09.16 19:32:34, Hanjun Guo wrote: On 09/20/2016 06:43 PM, Robert Richter wrote: Unfortunately either your nor my code does fix the BUG_ON() I see with the numa kernel: kernel BUG at mm/page_alloc.c:1848! See below for the core dump. It

Re: Kernel Oops on 4.8.0-rc8 while running trinity tests

2016-09-27 Thread Abdul Haleem
The kernel oops is still reproducible on 4.8.0-rc8 on PowerPC bare metal While running trinity system call fuzzer, I see these kernel oops messages: Unable to handle kernel paging request for data at address 0xe45f7702 Faulting instruction address: 0xc0055380 Oops: Kernel

linux-next: Tree for Sep 27

2016-09-27 Thread Stephen Rothwell
py_from_user faults) Merging btrfs-kdave/for-next (819d18cf0d4a Merge branch 'for-next-next-4.9-20160927' into for-next-20160927) Merging ceph/master (0276dca6c1ec rbd: add force close option) Merging cifs/for-next (659f6f19d311 SMB3: Add mount parameter to allow user to override max credits) Merging

Re: [PATCH] i2c: imx: defer probe if bus recovery GPIOs are not ready

2016-09-27 Thread Uwe Kleine-König
Hello Stefan, On Mon, Sep 26, 2016 at 05:18:58PM -0700, Stefan Agner wrote: > Some SoC might load the GPIO driver after the I2C driver and > using the I2C bus recovery mechanism via GPIOs. In this case > it is crucial to defer probing if the GPIO request functions > do so, otherwise the I2C

Re: [PATCH V2 2/2] fs/super.c: don't fool lockdep in freeze_super() and thaw_super() paths

2016-09-27 Thread Jan Kara
On Mon 26-09-16 18:55:25, Oleg Nesterov wrote: > On 09/26, Jan Kara wrote: > > > > On Mon 26-09-16 18:08:06, Oleg Nesterov wrote: > > > +/* > > > + * Tell lockdep we are holding these locks before we call > > > ->unfreeze_fs(sb). > > > + */ > > > +static void sb_freeze_acquire(struct super_block

[PATCH v4] i2c: hibvt: add Hisilicon BVT I2C controller driver

2016-09-27 Thread Pan Wen
add Hisilicon BVT I2C controller driver support. Signed-off-by: Pan Wen --- change log v4: Modify the default frequency to 100KHz. v3: Add a SoC specific compatible string. v2: 1)Fixed a compile error. 2)Dropped the clock-names property. .../devicetree/bindings/i2c/i2c-hibvt.txt | 24

[PATCH] VSOCK: Don't dec ack backlog twice for rejected connections

2016-09-27 Thread Jorgen Hansen
If a pending socket is marked as rejected, we will decrease the sk_ack_backlog twice. So don't decrement it for rejected sockets in vsock_pending_work(). Testing of the rejected socket path was done through code modifications. Reported-by: Stefan Hajnoczi Signed-off-by: Jorgen Hansen

Re: [PATCH] ARM: dts: imx6: Add support for Toradex Colibri iMX6 module

2016-09-27 Thread Marcel Ziswiler
On Wed, 2016-09-21 at 16:54 +0530, Sanchayan Maity wrote: > Add support for Toradex Colibri iMX6 module. > > Signed-off-by: Sanchayan Maity Acked-by: Marcel Ziswiler > --- >  arch/arm/boot/dts/Makefile   |   1 + >  arch/arm/boot/dts/imx6dl-colibri-eval-v3.dts | 253 >  

Re: [PATCH v2 1/3] ARM: dts: imx6qdl-apalis: Do not rely on DDC I2C bus bitbang for HDMI

2016-09-27 Thread Marcel Ziswiler
On Mon, 2016-09-19 at 10:41 +0530, Sanchayan Maity wrote: > Remove the use of DDC I2C bus bitbang to support reading of EDID > and rely on support from internal HDMI I2C master controller instead. > As a result remove the device tree property ddc-i2c-bus. > > Signed-off-by: Sanchayan Maity

[PATCH v3 2/2] drm/panel: simple: add support for Sharp LQ150X1LG11 panels

2016-09-27 Thread Peter Rosin
From: Gustaf Lindström The Sharp 15" LQ150X1LG11 panel is an XGA TFT LCD panel. The simple-panel driver is used to get support for essential functionality of the panel. Signed-off-by: Gustaf Lindström Signed-off-by: Peter Rosin --- drivers/gpu/drm/panel/panel-simple.c | 27

[PATCH] ipmi: fix crash on reading version from proc after unregisted bmc

2016-09-27 Thread Xie XiuQi
I meet a crash, which could be reproduce: 1) while true; do cat /proc/ipmi/0/version; done 2) modprobe -rv ipmi_si ipmi_msghandler ipmi_devintf [82761.021137] IPMI BT: req2rsp=5 secs retries=2 [82761.034524] ipmi device interface [82761.18] ipmi_si ipmi_si.0: Found new BMC (man_id: 0x0007db,

Re: [PATCH 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-27 Thread Dan Carpenter
On Mon, Sep 26, 2016 at 08:38:14PM +0300, Sergei Shtylyov wrote: > >@@ -53,7 +53,7 @@ void gigaset_dbg_buffer(enum debuglevel level, const > >unsigned char *msg, > > { > > unsigned char outbuf[80]; > > unsigned char c; > >-size_t space = sizeof outbuf - 1; > >+size_t space =

Re: [PATCH 4/5] ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread Dan Carpenter
This patch creates new bugs. I have a policy of not telling Markus where the bug is, because otherwise he'll just resend the patch and I have told him many times to stop sending these cleanup patches that just introduce bugs and waste maintainer time. regards, dan carpenter

Re: [PATCH V2 2/2] fs/super.c: don't fool lockdep in freeze_super() and thaw_super() paths

2016-09-27 Thread Dave Chinner
On Tue, Sep 27, 2016 at 08:51:35AM +0200, Jan Kara wrote: > On Mon 26-09-16 18:55:25, Oleg Nesterov wrote: > > On 09/26, Jan Kara wrote: > > > > > > On Mon 26-09-16 18:08:06, Oleg Nesterov wrote: > > > > +/* > > > > + * Tell lockdep we are holding these locks before we call > > > >

Re: [PATCH] staging: skein: Remove extra blank lines

2016-09-27 Thread Dan Carpenter
On Mon, Sep 26, 2016 at 11:56:27AM -0300, Martin wrote: > Fix coding style issue "multiple blank lines" > detected by checkpatch.pl in threefish_block.c file. > > Signed-off-by: Martin This Signed-off-by are like a legal document. Use first and last name. Also fix your from header as well.

Re: [PATCH 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-27 Thread Dan Carpenter
On Tue, Sep 27, 2016 at 10:08:37AM +0300, Dan Carpenter wrote: > On Mon, Sep 26, 2016 at 08:38:14PM +0300, Sergei Shtylyov wrote: > > >@@ -53,7 +53,7 @@ void gigaset_dbg_buffer(enum debuglevel level, const > > >unsigned char *msg, > > > { > > > unsigned char outbuf[80]; > > > unsigned char c;

Re: ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread SF Markus Elfring
> This patch creates new bugs. Thanks for your information. > I have a policy of not telling Markus where the bug is, I find this kind of response strange. > because otherwise he'll just resend the patch This can also happen when the other contributors request it. > and I have told him

Re: [PATCH 4/5] ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread Dan Carpenter
Ah well... Someone else discovered the double free bug first and gave it away. Reassuring, I guess. regards, dan carpenter

Re: [PATCH net v2] L2TP:Adjust intf MTU,factor underlay L3,overlay L2

2016-09-27 Thread David Miller
From: "R. Parameswaran" Date: Thu, 22 Sep 2016 13:52:43 -0700 (PDT) > From ed585bdd6d3d2b3dec58d414f514cd764d89159d Mon Sep 17 00:00:00 2001 > From: "R. Parameswaran" > Date: Thu, 22 Sep 2016 13:19:25 -0700 > Subject: [PATCH] L2TP:Adjust intf MTU,factor underlay L3,overlay L2 > > Take into

Re: [PATCH] kcov: properly check if we are in an interrupt

2016-09-27 Thread Peter Zijlstra
On Tue, Sep 27, 2016 at 08:21:32AM +0200, Dmitry Vyukov wrote: > > I suspect there is a bunch of places that use in_interrupt(), but mean > the same as KCOV wants -- am I in interrupt? and not am I in interrupt > context or in normal task context but inside local_bh_disable(). For > example, why

Re: [PATCH 06/11] pipe: no need to confirm page cache buf

2016-09-27 Thread Miklos Szeredi
On Tue, Sep 27, 2016 at 5:40 AM, Al Viro wrote: > On Wed, Sep 14, 2016 at 10:37:11AM +0200, Miklos Szeredi wrote: > >> Things could happen to that page that make it not uptodate while sitting in >> the pipe, but it's questionable whether we should care about that. >> Checking for being uptodate

[PATCHv4 3/4] staging:android:ion: Fix blank line issues

2016-09-27 Thread Yannis Damigos
This patch fixes the following blank line issues: CHECK: Please don't use multiple blank lines CHECK: Please use a blank line after function/struct/union/enum declarations +} +RESERVEDMEM_OF_DECLARE(ion, "ion-region", rmem_ion_setup); Signed-off-by: Yannis Damigos ---

[PATCHv4 1/4] staging:android:ion: Fix comparison with NULL

2016-09-27 Thread Yannis Damigos
This patch fixes the following comparison with NULL issues: CHECK: Comparison to NULL could be written "compatible[i].name" + for (i = 0; compatible[i].name != NULL; i++) { CHECK: Comparison to NULL could be written "!compatible[i].name" + if (compatible[i].name == NULL)

[PATCHv4 4/4] staging:android:ion: Fix whitespace issue

2016-09-27 Thread Yannis Damigos
This patch fixes the following whitespace issue: CHECK: spaces preferred around that '*' (ctx:VxV) FILE: drivers/staging/android/ion/ion_of.c:91: +sizeof(struct ion_platform_heap)*num_heaps, Signed-off-by: Yannis Damigos --- drivers/staging/android/ion/ion_of.c | 2

Re: solo6010 modprobe lockup since e1ceb25a (v4.3 regression)

2016-09-27 Thread Andrey Utkin
On Tue, Sep 27, 2016 at 07:27:53AM +0200, Krzysztof Hałasa wrote: > Andrey Utkin writes: > > >> Does (only) adding the > >> > >>pci_read_config_word(solo_dev->pdev, PCI_STATUS, ); > >> > >> in solo_reg_write() help? > > > > Yes. > > I have posted a patch with this change few days ago, I

[PATCHv4 2/4] staging:android:ion: Fix alignment issues

2016-09-27 Thread Yannis Damigos
This patch fixes the following whitespace issues: CHECK: Alignment should match open parenthesis + pr_info("%s: id %d type %d name %s align %lx\n", __func__, + heap->id, heap->type, heap->name, heap->align); CHECK: Alignment should match open parenthesis +struct

Re: linux-next: manual merge of the vfs-jk tree with the fuse tree

2016-09-27 Thread Jan Kara
Hi Stephen, On Tue 27-09-16 10:12:04, Stephen Rothwell wrote: > Today's linux-next merge of the vfs-jk tree got a conflict in: > > fs/fuse/dir.c > > between commits: > > 3b3fb74935ce ("fuse: invalidate dir dentry after chmod") > 5f69e628df75 ("fuse: fix killing s[ug]id in setattr") > >

Re: [PATCH 0/2] NFS: Use complete() instead complete_all()

2016-09-27 Thread Daniel Wagner
Hi Anna, On 09/26/2016 03:31 PM, Anna Schumaker wrote: > No, there are a handful that we know don't pass. When I run on NFS v4.1, I > get: > > Failures: generic/035 generic/071 generic/128 generic/294 > generic/306 generic/309 generic/310 generic/313 generic/314 > generic/315 generic/316

[PATCH V2 0/6] ARM64: Uprobe support added

2016-09-27 Thread Pratyush Anand
is_tarp_insn() * Changes for rebase conflict resolution V1 was here: https://lkml.org/lkml/2016/8/2/29 Patches have been rebased on next-20160927, so that there would be no conflicts with other arm64/for-next/core patches. Patches have been tested for following: 1. Step-able instructions, like sub

Re: [PATCH -v2 7/9] sched,tracing: Update trace_sched_pi_setprio()

2016-09-27 Thread Peter Zijlstra
On Mon, Sep 26, 2016 at 01:04:30PM -0400, Steven Rostedt wrote: > On Mon, 26 Sep 2016 14:32:20 +0200 > Peter Zijlstra wrote: > > > Pass the PI donor task, instead of a numerical priority. > > > > Numerical priorities are not sufficient to describe state ever since > > SCHED_DEADLINE. > > > >

Re: [PATCHv2 2/3] tty/serial: at91: fix hardware handshake with GPIOs

2016-09-27 Thread Richard Genoud
2016-09-22 15:20 GMT+02:00 Uwe Kleine-König : > On Mon, Sep 12, 2016 at 11:47:32AM +0200, Richard Genoud wrote: >> Commit 1cf6e8fc8341 ("tty/serial: at91: fix RTS line management when >> hardware handshake is enabled") broke the hardware handshake when GPIOs >> where used. > > s/where/were/ ok.

[PATCH V2 4/6] arm64: Handle TRAP_BRKPT for user mode as well

2016-09-27 Thread Pratyush Anand
uprobe is registered at break_hook with a unique ESR code. So, when a TRAP_BRKPT occurs, call_break_hook checks if it was for uprobe. If not, then send a SIGTRAP to user. Signed-off-by: Pratyush Anand --- arch/arm64/kernel/debug-monitors.c | 18 +++--- 1 file changed, 11

[PATCH V2 5/6] arm64: introduce mm context flag to keep 32 bit task information

2016-09-27 Thread Pratyush Anand
We need to decide in some cases like uprobe instruction analysis that whether the current mm context belongs to a 32 bit task or 64 bit. This patch has introduced an unsigned flag variable in mm_context_t. Currently, we set and clear TIF_32BIT depending on the condition that whether an elf binary

[PATCH V2 2/6] arm64: kgdb_step_brk_fn: ignore other's exception

2016-09-27 Thread Pratyush Anand
ARM64 step exception does not have any syndrome information. So, it is responsibility of exception handler to take care that they handle it only if exception was raised for them. Since kgdb_step_brk_fn() always returns 0, therefore we might have problem when we will have other step handler

[PATCH V2 1/6] arm64: kprobe: protect/rename few definitions to be reused by uprobe

2016-09-27 Thread Pratyush Anand
decode-insn code has to be reused by arm64 uprobe implementation as well. Therefore, this patch protects some portion of kprobe code and renames few other, so that decode-insn functionality can be reused by uprobe even when CONFIG_KPROBES is not defined. kprobe_opcode_t and struct

Re: [PATCH 2/2] bpf samples: update tracex5 sample to use __seccomp_filter

2016-09-27 Thread David Miller
From: "Naveen N. Rao" Date: Sat, 24 Sep 2016 02:10:05 +0530 > seccomp_phase1() does not exist anymore. Instead, update sample to use > __seccomp_filter(). While at it, set max locked memory to unlimited. > > Signed-off-by: Naveen N. Rao Also applied to net-next, thanks.

Re: [PATCH v3] net: ip, diag -- Add diag interface for raw sockets

2016-09-27 Thread Cyrill Gorcunov
On Mon, Sep 26, 2016 at 07:54:37PM -0600, David Ahern wrote: > On 9/26/16 4:38 PM, Cyrill Gorcunov wrote: > > Something like > > > > Index: linux-ml.git/include/uapi/linux/inet_diag.h > > === > > ---

Re: [PATCH 1/2] bpf samples: fix compiler errors with sockex2 and sockex3

2016-09-27 Thread David Miller
From: "Naveen N. Rao" Date: Sat, 24 Sep 2016 02:10:04 +0530 > These samples fail to compile as 'struct flow_keys' conflicts with > definition in net/flow_dissector.h. Fix the same by renaming the > structure used in the sample. > > Signed-off-by: Naveen N. Rao Applied to net-next.

[PATCH V2 6/6] arm64: Add uprobe support

2016-09-27 Thread Pratyush Anand
This patch adds support for uprobe on ARM64 architecture. Unit tests for following have been done so far and they have been found working 1. Step-able instructions, like sub, ldr, add etc. 2. Simulation-able like ret, cbnz, cbz etc. 3. uretprobe 4. Reject-able instructions like

[PATCH V2 3/6] arm64: Handle TRAP_TRACE for user mode as well

2016-09-27 Thread Pratyush Anand
uprobe registers a handler at step_hook. So, single_step_handler now checks for user mode as well if there is a valid hook. Signed-off-by: Pratyush Anand --- arch/arm64/kernel/debug-monitors.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git

Re: [PATCH] kcov: properly check if we are in an interrupt

2016-09-27 Thread Dmitry Vyukov
On Tue, Sep 27, 2016 at 9:34 AM, Peter Zijlstra wrote: > On Tue, Sep 27, 2016 at 08:21:32AM +0200, Dmitry Vyukov wrote: >> >> I suspect there is a bunch of places that use in_interrupt(), but mean >> the same as KCOV wants -- am I in interrupt? and not am I in interrupt >> context or in normal

Re: [PATCH v3] bpf: Set register type according to is_valid_access()

2016-09-27 Thread David Miller
From: Mickaël Salaün Date: Sat, 24 Sep 2016 20:01:50 +0200 > This prevent future potential pointer leaks when an unprivileged eBPF > program will read a pointer value from its context. Even if > is_valid_access() returns a pointer type, the eBPF verifier replace it > with UNKNOWN_VALUE. The

[PATCH] MAINTAINERS: update fbdev entries

2016-09-27 Thread Tomi Valkeinen
Remove Jean-Christophe from the maintainers, and remove links to old unmaintained web pages and git trees. Signed-off-by: Tomi Valkeinen --- Jean-Christophe hasn't done any fbdev work for years, hasn't responded to any emails, and now emails to him bounce back. MAINTAINERS | 3 --- 1 file

[PATCH v3 0/2] drm/panel: simple: add support for Sharp LQ150X1LG11 panels

2016-09-27 Thread Peter Rosin
Hi! I'm back with a new attempt at getting this panel upstream. This time with the bindings in a separate commit, as I hear that is preferred. I think I covered all the pins on the panel... Cheers, Peter Gustaf Lindström (1): drm/panel: simple: add support for Sharp LQ150X1LG11 panels Peter

Re: [PATCH 15/19] reset: sti: Remove STiH415/6 reset support

2016-09-27 Thread Philipp Zabel
Hi Peter, Am Mittwoch, den 14.09.2016, 14:27 +0100 schrieb Peter Griffin: > Support for STiH415/6 SoCs is being removed from the > kernel because the platforms are obsolete. This patch removes > the reset drivers for these SoC's. > > Signed-off-by: Peter Griffin > Cc: > --- >

Re: [PATCH 2/3] regulator: max77693: Also manipulate the fast charge current

2016-09-27 Thread Krzysztof Kozlowski
On Tue, Sep 27, 2016 at 01:31:09AM +0200, Wolfgang Wiedmeyer wrote: > For MAX77693, the fast charge current also needs to be manipulated for > proper charging. The fast charge current is only set in the case of > the MAX77693 type, as the MAX77843 properly manipulates the fast > charge current.

Re: [PATCH v2] fs/select: add vmalloc fallback for select(2)

2016-09-27 Thread Vlastimil Babka
On 09/27/2016 02:01 AM, Andrew Morton wrote: On Thu, 22 Sep 2016 18:43:59 +0200 Vlastimil Babka wrote: The select(2) syscall performs a kmalloc(size, GFP_KERNEL) where size grows with the number of fds passed. We had a customer report page allocation failures of order-4 for this allocation.

Re: [PATCH 1/3] mfd: max77693: Add defines for charger current control

2016-09-27 Thread Krzysztof Kozlowski
On Tue, Sep 27, 2016 at 01:31:08AM +0200, Wolfgang Wiedmeyer wrote: > This prepares for an updated regulator and charger driver. The defines > are needed to set the maximum input current and the fast charge > current. > > Signed-off-by: Wolfgang Wiedmeyer This shouldn't be separate patch,

Re: [PATCH] video: fbdev: i810: add in missing white space in error message text

2016-09-27 Thread Tomi Valkeinen
On 12/09/16 15:14, Colin King wrote: > From: Colin Ian King > > A printk message spans two lines and the literal string is missing > a white space between words. Add the white space. > > Signed-off-by: Colin Ian King > --- > drivers/video/fbdev/i810/i810_main.c | 2 +- > 1 file changed, 1

Re: [PATCH] video: fbdev: add missing \n at end of printk error message

2016-09-27 Thread Tomi Valkeinen
On 12/09/16 13:24, Colin King wrote: > From: Colin Ian King > > The message is missing a \n, add it. > > Signed-off-by: Colin Ian King > --- > drivers/video/fbdev/s1d13xxxfb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/fbdev/s1d13xxxfb.c >

Re: [PATCH v3 2/2] ARM: exynos_defconfig: Remove old non-working MIPI driver

2016-09-27 Thread Tomi Valkeinen
On 16/09/16 11:32, Krzysztof Kozlowski wrote: > The Exynos MIPI driver does not work anymore (it is board file only) so > it is removed. Remove also config options. > > Cc: Inki Dae > Cc: Donghwa Lee > Cc: Kyungmin Park > Cc: Tomi Valkeinen > Signed-off-by: Krzysztof Kozlowski > > --- > >

Re: [PATCH v2] fs/select: add vmalloc fallback for select(2)

2016-09-27 Thread Vlastimil Babka
On 09/27/2016 03:38 AM, Eric Dumazet wrote: On Mon, 2016-09-26 at 17:01 -0700, Andrew Morton wrote: I don't share Eric's concerns about performance here. If the vmalloc() is called, we're about to write to that quite large amount of memory which we just allocated, and the vmalloc() overhead

Re: [PATCH 3/3] power_supply: max77693: Listen for cable events and enable charging

2016-09-27 Thread Krzysztof Kozlowski
On Tue, Sep 27, 2016 at 01:31:10AM +0200, Wolfgang Wiedmeyer wrote: > This patch adds a listener for extcon cable events and enables > charging if an USB cable is connected. It recognizes SDP and DCP cable > types and treats them the same (same input current and fast charge > current). The maximum

Re: [PATCH 04/26] matroxfb: constify local structures

2016-09-27 Thread Tomi Valkeinen
On 11/09/16 16:05, Julia Lawall wrote: > For structure types defined in the same file or local header files, find > top-level static structure declarations that have the following > properties: > 1. Never reassigned. > 2. Address never taken > 3. Not passed to a top-level macro call > 4. No

Re: [PATCH RESEND 2/2] mfd: twl6040: Register child device for twl6040-pdmclk

2016-09-27 Thread Peter Ujfalusi
| 2 +- >> 3 files changed, 7 insertions(+), 1 deletion(-) > > Applied, thanks. Lee, for some reason this patch is not found in linux-next (20160927). I can see patch 1, but this one is missing. -- Péter >> diff --git a/Documentation/devicetree/bindings/mfd/twl60

Re: [PATCH] video: fbdev: constify fb_fix_screeninfo and fb_var_screeninfo structures

2016-09-27 Thread Tomi Valkeinen
On 11/09/16 18:17, Julia Lawall wrote: > These structures are only used to copy into other structures, so declare > them as const. > > The semantic patch that makes this change in the fb_fix_screeninfo case is > as follows (http://coccinelle.lip6.fr/). The fb_var_screeninfo case is > analogous.

Re: [PATCH] simplefb: Disable and release clocks and regulators in destroy callback

2016-09-27 Thread Tomi Valkeinen
On 07/09/16 12:09, Chen-Yu Tsai wrote: > simplefb gets unregister when a proper framebuffer driver comes in and > kicks it out. However the claimed clocks and regulators stay enabled > as they are only released in the platform device remove function, which > in theory would never get called. > >

[PATCH] soc/fsl: dpaa_sys.h: fix ARM32 typo

2016-09-27 Thread Valentin Rothberg
The Kconfig symbol for 32bit ARM is 'ARM', not 'ARM32'. Signed-off-by: Valentin Rothberg --- drivers/soc/fsl/qbman/dpaa_sys.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/fsl/qbman/dpaa_sys.h b/drivers/soc/fsl/qbman/dpaa_sys.h index b63fd72295c6..608b1f608e13

Re: [PATCH v2] sysfs print name of undiscoverable attribute group

2016-09-27 Thread Johannes Thumshirn
On Mon, Sep 19, 2016 at 10:14:54AM +0200, Johannes Thumshirn wrote: > Print the name of an undiscoverable attribute group and not the > pointer's address. > > Signed-off-by: Johannes Thumshirn > --- > fs/sysfs/group.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git

[PATCH] ASoC: tlv320aic31xx: do not declare support for mono DAI

2016-09-27 Thread Nikita Yushchenko
This hardware supports only 2-channel DAI, even mono ADC digital output has two channels with the same data. Having min_channels=1 results in broken playback of mono files in setups where CPU DAI supports mono. Signed-off-by: Nikita Yushchenko --- sound/soc/codecs/tlv320aic31xx.c | 6 +++--- 1

Re: [PATCH 0/2] MCB patches for v4.9

2016-09-27 Thread Johannes Thumshirn
On Wed, Sep 14, 2016 at 12:05:22PM +0200, Johannes Thumshirn wrote: > Hi Greg, > > Here are the additional MCB patches for v4.9. > > They're around enabling of DMA for devices on an MCBus, are tested and > apply cleanly against your char-misc-next tree. > > Thanks, > Johannes > > Michael

Re: [PATCH 0/2] NFS: Use complete() instead complete_all()

2016-09-27 Thread Daniel Wagner
On 09/27/2016 09:42 AM, Daniel Wagner wrote: > Failures: generic/035 generic/087 generic/088 generic/089 generic/126 > generic/184 generic/285 generic/347 generic/350 > Failed 9 of 62 tests > > Hmm, not sure if this good or bad. Does it help? FWIW, running the same tests on non patched kernel

Re: [PATCH 1/2] fbdev: ssd1307fb: constify the device_info pointer

2016-09-27 Thread Tomi Valkeinen
On 16/08/16 12:27, LABBE Corentin wrote: > of_match_device return const data, so instead of casting its return value > this patch constify the device_info pointer. > > Signed-off-by: LABBE Corentin > --- > drivers/video/fbdev/ssd1307fb.c | 6 +++--- > 1 file changed, 3 insertions(+), 3

Re: [PATCH] fs/select: add vmalloc fallback for select(2)

2016-09-27 Thread Vlastimil Babka
On 09/23/2016 06:47 PM, Jason Baron wrote: Hi, On 09/23/2016 03:24 AM, Nicholas Piggin wrote: On Fri, 23 Sep 2016 14:42:53 +0800 "Hillf Danton" wrote: The select(2) syscall performs a kmalloc(size, GFP_KERNEL) where size grows with the number of fds passed. We had a customer report page

Re: [PATCH] matroxfb: fix size of memcpy

2016-09-27 Thread Tomi Valkeinen
On 25/08/16 20:44, Sudip Mukherjee wrote: > hw->DACreg has a size of 80 bytes and MGADACbpp32 has 21. So when > memcpy copies MGADACbpp32 to hw->DACreg it copies 80 bytes but > only 21 bytes are valid. > > Signed-off-by: Sudip Mukherjee > --- > drivers/video/fbdev/matrox/matroxfb_Ti3026.c | 2

Re: [PATCH v2 4/4] PM / AVS: rockchip-cpu-avs: add driver handling Rockchip cpu avs

2016-09-27 Thread Finley Xiao
在 2016/9/26 16:05, Heiko Stuebner 写道: Am Montag, 26. September 2016, 09:25:11 CEST schrieb Viresh Kumar: On 12-09-16, 14:55, Stephen Boyd wrote: On 08/29, Viresh Kumar wrote: On 18-08-16, 16:52, Finlye Xiao wrote: +static int rockchip_adjust_opp_table(struct device *cpu_dev, +

[PATCH v3] fs/select: add vmalloc fallback for select(2)

2016-09-27 Thread Vlastimil Babka
The select(2) syscall performs a kmalloc(size, GFP_KERNEL) where size grows with the number of fds passed. We had a customer report page allocation failures of order-4 for this allocation. This is a costly order, so it might easily fail, as the VM expects such allocation to have a lower-order

Re: [PATCH] video: smscufx: remove unused variable

2016-09-27 Thread Tomi Valkeinen
On 25/05/16 15:19, Sudip Mukherjee wrote: > The variable was only assigned with the pointer but was never used. > We can safely remove it. > > Signed-off-by: Sudip Mukherjee > --- > drivers/video/fbdev/smscufx.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git

Re: Regression in 4.8 - CPU speed set very low

2016-09-27 Thread Larry Finger
On 09/26/2016 10:12 PM, Doug Smythies wrote: On 2016.09.26 18:31 Srinivas Pandruvada wrote: On Mon, 2016-09-26 at 19:48 -0500, Larry Finger wrote: On 09/26/2016 07:21 PM, Rafael J. Wysocki wrote: On Tue, Sep 27, 2016 at 1:53 AM, Larry Finger wrote: But for both we need a reproducer anyway. I

Re: ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread Paul Bolle
On Tue, 2016-09-27 at 07:20 +0200, SF Markus Elfring wrote: > Will it matter here if the function "kfree" will be called for the > data structure members "bcs" and "inbuf" after a later function call > failed within the implementation of "gigaset_initcs"? My translation of this question is: could

Re: [Update][RFC/RFT][PATCH v3 2/5] driver core: Functional dependencies tracking support

2016-09-27 Thread Lukas Wunner
On Fri, Sep 16, 2016 at 02:33:55PM +0200, Rafael J. Wysocki wrote: > +void device_links_unbind_consumers(struct device *dev) > +{ > + struct device_link *link; > + int idx; > + > + start: > + idx = device_links_read_lock(); > + > + list_for_each_entry_rcu(link, >links_to_consumers,

Re: [bug] crypto/vmx/p8_ghash memory corruption in 4.8-rc7

2016-09-27 Thread Jan Stancek
- Original Message - > From: "Herbert Xu" > To: "Marcelo Cerri" > Cc: "Jan Stancek" , "rui y wang" , > mhce...@linux.vnet.ibm.com, > leosi...@linux.vnet.ibm.com, pfsmor...@linux.vnet.ibm.com, > linux-cry...@vger.kernel.org, > linuxppc-...@lists.ozlabs.org,

Re: [PATCH] padata: remove recently added module usage from bool code

2016-09-27 Thread Sebastian Andrzej Siewior
On 2016-09-26 21:18:21 [-0400], Paul Gortmaker wrote: > > ...and so it currently is not being built as a module by anyone. that is correct. Acked-by: Sebastian Andrzej Siewior Sebastian

Re: [PATCH] brcmfmac: implement more accurate skb tracking

2016-09-27 Thread Arend Van Spriel
On 26-9-2016 16:59, Dan Williams wrote: > On Mon, 2016-09-26 at 14:13 +0200, Rafał Miłecki wrote: >> On 26 September 2016 at 13:46, Arend Van Spriel >> wrote: >>> >>> On 26-9-2016 12:23, Rafał Miłecki wrote: From: Rafał Miłecki We need to track 802.1x packets to know if there

[GIT PULL] tpmdd reverts for Linux 4.9

2016-09-27 Thread Jarkko Sakkinen
git://git.infradead.org/users/pcmoore/selinux into next (2016-09-21 11:54:19 +1000) are available in the git repository at: git://git.infradead.org/users/jjs/linux-tpmdd.git tags/tpmdd-next-20160927 for you to fetch changes up to 324152502b0ef2f65a3fe039b7dcd27b223d14ac: Revert "tpm/tpm_crb: imple

[PATCH 4.9] brcmfmac: use correct skb freeing helper when deleting flowring

2016-09-27 Thread Rafał Miłecki
From: Rafał Miłecki Flowrings contain skbs waiting for transmission that were passed to us by netif. It means we checked every one of them looking for 802.1x Ethernet type. When deleting flowring we have to use freeing function that will check for 802.1x type as well. Freeing skbs without a

[PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Quentin Lambert
Checkpatch printed a style ERROR concerning a missing space befire '('. This patch fix this issue. Signed-off-by: Quentin Lambert --- drivers/staging/greybus/manifest.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/staging/greybus/manifest.c +++

Re: [GIT PULL] tpmdd reverts for Linux 4.9

2016-09-27 Thread James Morris
/Jarkko > > The following changes since commit 8a17ef9d8543744a0c6a6f5fdc65b5770e7eaa64: > > Merge branch 'stable-4.9' of git://git.infradead.org/users/pcmoore/selinux > into next (2016-09-21 11:54:19 +1000) > > are available in the git repository at: > > git://git.infradea

Re: [PATCH] brcmfmac: implement more accurate skb tracking

2016-09-27 Thread Arend Van Spriel
On 26-9-2016 14:38, Rafał Miłecki wrote: > On 26 September 2016 at 14:13, Rafał Miłecki wrote: >> On 26 September 2016 at 13:46, Arend Van Spriel >> wrote: >>> On 26-9-2016 12:23, Rafał Miłecki wrote: From: Rafał Miłecki We need to track 802.1x packets to know if there are any

Re: [PATCH v3 2/4] tmp/tpm_crb: fix Intel PTT hw bug during idle state

2016-09-27 Thread Jarkko Sakkinen
On Thu, Sep 15, 2016 at 09:23:29AM +0300, Jarkko Sakkinen wrote: > On Mon, Sep 12, 2016 at 04:04:19PM +0300, Tomas Winkler wrote: > > There is a HW bug in Skylake, and Broxton PCH Intel PTT device, where > > most of the registers in the control area except START, REQUEST, CANCEL, > > and LOC_CTRL

Re: [PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Viresh Kumar
On Tue, Sep 27, 2016 at 2:50 PM, Quentin Lambert wrote: > Checkpatch printed a style ERROR concerning a missing space befire '('. > This patch fix this issue. > > Signed-off-by: Quentin Lambert > --- > drivers/staging/greybus/manifest.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH] greybus: audio: fix uninitialized variable errors found by cppcheck

2016-09-27 Thread Viresh Kumar
On Sat, Sep 24, 2016 at 11:06 PM, Vaibhav Agarwal wrote: > Currently, if info is null, the dev_err message is dereferencing an > uninitialized module pointer. Instead, it should use codec->dev pointer > in dev_err call and better align with other err msg in this function. > > Also, ret variable

Re: [PATCH] drivers/staging/greybus: gpio.c - Fixed a checkpatch generated warning

2016-09-27 Thread Viresh Kumar
On Mon, Sep 26, 2016 at 2:05 AM, Chase Metzger wrote: > Removed braces for single line if statement. > > Signed-off-by: Chase Metzger > --- > drivers/staging/greybus/gpio.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/staging/greybus/gpio.c

Re: [PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Quentin Lambert
On 27/09/2016 11:31, Viresh Kumar wrote: On Tue, Sep 27, 2016 at 2:50 PM, Quentin Lambert wrote: Checkpatch printed a style ERROR concerning a missing space befire '('. This patch fix this issue. Signed-off-by: Quentin Lambert --- drivers/staging/greybus/manifest.c |2 +- 1 file

Re: ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread SF Markus Elfring
>> Will it matter here if the function "kfree" will be called for the >> data structure members "bcs" and "inbuf" after a later function call >> failed within the implementation of "gigaset_initcs"? > > My translation of this question is: could you please hold my hand while > I read the code of a

Re: [PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Viresh Kumar
On 27 September 2016 at 15:03, Quentin Lambert wrote: > > > On 27/09/2016 11:31, Viresh Kumar wrote: >> >> On Tue, Sep 27, 2016 at 2:50 PM, Quentin Lambert >> wrote: >>> >>> Checkpatch printed a style ERROR concerning a missing space befire '('. >>> This patch fix this issue. >>> >>>

Re: [PATCH 00/17] clean up readlinks

2016-09-27 Thread Miklos Szeredi
On Tue, Sep 27, 2016 at 5:10 AM, Al Viro wrote: > On Mon, Sep 12, 2016 at 09:29:02PM +0200, Miklos Szeredi wrote: >> The first patch is actually a bug fix, but I put it into this bunch for >> simplicity... >> >> The rest are really cleanups as well as minor bugfixes that are byproducts >> of the

[PATCH v2] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Quentin Lambert
Checkpatch printed a style ERROR concerning a missing space before '('. This patch fixes this issue. Signed-off-by: Quentin Lambert Acked-by: Viresh Kumar --- v2: fixes typos in commit message drivers/staging/greybus/manifest.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---

BUG: scheduling while atomic in f_fs when gadget remove driver

2016-09-27 Thread Chen Yu
Hi All, I'm working on Hikey board based around the HiSilicon Kirin 620, with linaro kernel version 4.8.rc1 and I get below BUG error while extracting USB cable from PC. The funtion using f_fs is adb and usb_gadget_unregister_driver will be called after extracting USB cable from PC. [

Re: [f2fs-dev] [PATCH v2] f2fs: introduce get_checkpoint_version for cleanup

2016-09-27 Thread Chao Yu
On 2016/9/27 10:05, Tiezhu Yang wrote: > There exists almost same codes when get the value of pre_version > and cur_version in function validate_checkpoint, this patch adds > get_checkpoint_version to clean up redundant codes. > > Signed-off-by: Tiezhu Yang > --- > fs/f2fs/checkpoint.c | 63 >

Re: [PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Dan Carpenter
On Tue, Sep 27, 2016 at 11:33:49AM +0200, Quentin Lambert wrote: > > > On 27/09/2016 11:31, Viresh Kumar wrote: > >On Tue, Sep 27, 2016 at 2:50 PM, Quentin Lambert > > wrote: > >>Checkpatch printed a style ERROR concerning a missing space befire '('. > >>This patch fix this issue. > >> >

Re: ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread Paul Bolle
You're in Eliza mode again. (Hat tip to Björn Mork, https://lkml.org/lkml/2016/1/4/259). Paul Bolle

RE: [PATCH v3 2/4] tmp/tpm_crb: fix Intel PTT hw bug during idle state

2016-09-27 Thread Winkler, Tomas
> > On Thu, Sep 15, 2016 at 09:23:29AM +0300, Jarkko Sakkinen wrote: > > On Mon, Sep 12, 2016 at 04:04:19PM +0300, Tomas Winkler wrote: > > > There is a HW bug in Skylake, and Broxton PCH Intel PTT device, > > > where most of the registers in the control area except START, > > > REQUEST, CANCEL,

Re: BUG: scheduling while atomic in f_fs when gadget remove driver

2016-09-27 Thread Felipe Balbi
Hi, Chen Yu writes: > Hi All, > > I'm working on Hikey board based around the HiSilicon Kirin 620, with > linaro kernel version 4.8.rc1 and I get below BUG error while > extracting USB cable from PC. which peripheral controller does this one have? Is it dwc3? I'm very interested in knowing

[PATCH] MIPS: loongson32: Remove several RTC-related macros

2016-09-27 Thread Yang Ling
Add regs-rtc.h to replace the macros of redundancy. Signed-off-by: Yang Ling --- arch/mips/include/asm/mach-loongson32/regs-rtc.h | 23 +++ arch/mips/loongson32/common/platform.c | 23 ++- 2 files changed, 33 insertions(+), 13 deletions(-)

Re: linux-next: Tree for Sep 27

2016-09-27 Thread Sergey Senozhatsky
Hello, On (09/27/16 16:40), Stephen Rothwell wrote: > > Changes since 20160923: > seems that commit e3b37f11e6e4e6b6 ("netfilter: replace list_head with single linked list") breaks the build on !CONFIG_NETFILTER_INGRESS systems accessing ->nf_hooks_ingress static void nf_set_hooks_head(struct

Re: [PATCH 4.9] brcmfmac: use correct skb freeing helper when deleting flowring

2016-09-27 Thread Arend Van Spriel
On 27-9-2016 11:14, Rafał Miłecki wrote: > From: Rafał Miłecki > > Flowrings contain skbs waiting for transmission that were passed to us > by netif. It means we checked every one of them looking for 802.1x > Ethernet type. When deleting flowring we have to use freeing function > that will check

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