Re: [PATCH 4/5] input: touchscreen: support Allwinner SoCs' touchscreen

2016-09-25 Thread Quentin Schulz
On 24/07/2016 13:24, Jonathan Cameron wrote: > On 20/07/16 09:29, Quentin Schulz wrote: >> This adds support for Allwinner SoCs' (A10, A13 and A31) resistive >> touchscreen. This driver is probed by the MFD sunxi-gpadc-mfd. >> >> This driver uses ADC channels exposed through the IIO framework by

Re: [PATCH][V2] mlxsw: spectrum: remove redundant check if err is zero

2016-09-25 Thread Colin Ian King
On 24/09/16 22:08, Ido Schimmel wrote: > On Sat, Sep 24, 2016 at 06:03:38PM -0700, Colin King wrote: >> From: Colin Ian King >> >> There is an earlier check and return if err is non-zero, so >> the check to see if it is zero is redundant in every iteration >> of the loop

Re: [PATCH] mm: check VMA flags to avoid invalid PROT_NONE NUMA balancing

2016-09-25 Thread Rik van Riel
On Sun, 2016-09-25 at 13:52 -0700, Linus Torvalds wrote: > I was kind of assuming this would go through the normal channels for > THP patches, but it's been two weeks... > > Can I have an ACK from the involved people, and I'll apply it > directly.. Mel? Rik? Sorry about that, I was a little

[BUG, 4.8-rc7] perf: oops in intel_pmu_enable_all

2016-09-25 Thread Dave Chinner
Hi Folks, I just upgraded a test VM from 4.8-rc6 to 4.8-rc7, and went to run: # perf_4.7 top -g -U inside the VM - the kernel oops with the trace below. The perf binary was built from a 4.7 kernel, so it's not the latest but it still shouldn't oops the kernel. Reproduced multiple times simply

Re: [PATCH 4/3] g_NCR5380: Merge g_NCR5380 and g_NCR5380_mmio

2016-09-25 Thread Christoph Hellwig
On Sun, Sep 25, 2016 at 09:39:20PM +0200, Ondrej Zary wrote: > Merge the PIO and MMIO code (with the help of ioport_map) in g_NCR5380 and > delete g_NCR5380_mmio. Awesome! this looks great to me, but we probably should thrown in a MODULE_ALIAS for g_NCR5380_mmio so that existing module

Re: [PATCH V2] staging: rtl8172: usb_intf.c: Fixed block comment coding style

2016-09-25 Thread Larry Finger
On 09/25/2016 06:00 PM, Gonçalo Salazar wrote: Fixed a block comment indentation in the rtl8712 usb_intf.c file. Made this as a first commit. Resubmitted with updated subject. Please let me know of any feedback you have. I missed this last time, but everything above the --- line goes into the

Re: [PATCH] f2fs: remove dirty inode pages in error path

2016-09-25 Thread Chao Yu
On 2016/9/25 1:47, Jaegeuk Kim wrote: > On Sat, Sep 24, 2016 at 09:02:02AM +0800, Chao Yu wrote: >> On 2016/9/24 5:11, Jaegeuk Kim wrote: >>> When getting EIO while handling orphan inodes, we can get some dirty node >>> pages. Then, f2fs_write_node_pages() called by iput(node_inode) will try >>>

Re: [PATCH 2/3] f2fs: support checkpoint error injection

2016-09-25 Thread Chao Yu
On 2016/9/25 2:10, Jaegeuk Kim wrote: > On Sat, Sep 24, 2016 at 11:32:08AM +0800, Chao Yu wrote: >> On 2016/9/24 8:52, Jaegeuk Kim wrote: >>> On Sat, Sep 24, 2016 at 08:46:54AM +0800, Chao Yu wrote: Hi Jaegeuk, On 2016/9/24 7:53, Jaegeuk Kim wrote: > Hi Chao, > > The

RE: [PATCH v3] Input: elants_i2c - get product id on recovery mode for FW update

2016-09-25 Thread Johnny.Chuang
There is only one different which is adding a new empty line for coding style. > if (!error) > - error = elants_i2c_query_fw_id(ts); > + error = error2; > + > if (!error) > error = elants_i2c_query_fw_version(ts); -Original Message- From:

[PATCH 1/2] Revert "HID: dragonrise: fix HID Descriptor for 0x0006 PID"

2016-09-25 Thread Ioan-Adrian Ratiu
This reverts commit 18339f59c3a6 ("HID: dragonrise: fix HID...") because the "fix" is bogus. That report descriptor is different in hardware (see below) and it's the way the hardware works, it can't be fixed at this level because it reuses axes by design. What this change tried to fix is a

[PATCH 2/2] hid: input: add HID_QUIRK_REUSE_AXES and fix dragonrise

2016-09-25 Thread Ioan-Adrian Ratiu
Commit 79346d620e9d ("HID: input: force generic axis to be mapped to their user space axis") made mapping generic axes to their userspace equivalents mandatory and some lower end gamepads which were depending on the previous behaviour suffered severe regressions because they were reusing axes and

Re: iio: Adjust checks for null pointers in six functions

2016-09-25 Thread SF Markus Elfring
> Are you, by any chance, some kind of sociology grad? Not yet. ;-) - It looks also unlikely at the moment that my life will evolve in such direction. > You can't study development purely on the process level, I find that I am not studying this topic. Am I practically contributing small

Re: [PATCH 2/2] radix-tree: Fix optimisation problem

2016-09-25 Thread Cedric Blancher
LGTM, except that #define is_sibling_entry should be IS_SIBLING_ENTRY Ced On 25 September 2016 at 21:04, Linus Torvalds wrote: > On Sun, Sep 25, 2016 at 11:04 AM, Linus Torvalds > wrote: >> >> The more I look at that particular

Re: [PATCH 3/5] iio: adc: sunxi-gpadc-iio: enable iio_buffers

2016-09-25 Thread Quentin Schulz
On 25/09/2016 11:10, Jonathan Cameron wrote: > On 24/09/16 18:40, Quentin Schulz wrote: >> Hi Jonathan, >> >> Sorry for the (long) delay, I did not have time to work on it. I'll >> mainly work in my free time now. >> >> Keep in mind this patch was proposed based on the v2 of the ADC patches. >>

Re: [PATCH 2/2] radix-tree: Fix optimisation problem

2016-09-25 Thread Linus Torvalds
On Sun, Sep 25, 2016 at 12:04 PM, Linus Torvalds wrote: >It gets rid of > the ad-hoc arithmetic in radix_tree_descend(), and just makes all that > be inside the is_sibling_entry() logic instead. Which got renamed and > made to actually return the main

Re: [PATCH] dma-buf/sw_sync: mark sync_timeline_create() static

2016-09-25 Thread Gustavo Padovan
Hi Baoyou, 2016-09-22 Greg Kroah-Hartman : > On Tue, Sep 20, 2016 at 06:23:33PM +0530, Sumit Semwal wrote: > > Hi Baoyou, > > > > On 20 September 2016 at 16:43, Gustavo Padovan wrote: > > > 2016-09-18 Baoyou Xie : > > > >

Re: [PATCH] drm/i2c: tda998x: don't register the connector

2016-09-25 Thread Daniel Vetter
On Fri, Sep 23, 2016 at 03:05:03PM +0100, Russell King - ARM Linux wrote: > On Fri, Sep 23, 2016 at 03:13:15PM +0200, Daniel Vetter wrote: > > Hm, maybe we should simply not call ->lastclose for kms drivers. That is > > kinda only a hack for ums/dri1 drivers. > > Are you sure about that - isn't

Re: [PATCH] drm: bridge: analogix/dp: mark symbols static where possible

2016-09-25 Thread Daniel Vetter
On Sun, Sep 25, 2016 at 03:32:44PM +0800, Baoyou Xie wrote: > We get 2 warnings when building kernel with W=1: > drivers/gpu/drm/bridge/analogix/analogix_dp_core.c:1053:5: warning: no > previous prototype for 'analogix_dp_get_modes' [-Wmissing-prototypes] >

[PATCH] cpufreq: st: add missing \n to end of dev_err message

2016-09-25 Thread Colin King
From: Colin Ian King Trival fix, dev_err message is missing a \n, so add it. Signed-off-by: Colin Ian King --- drivers/cpufreq/sti-cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpufreq/sti-cpufreq.c

[PATCH] nvme-rdma: add missing \n to end of dev_err message

2016-09-25 Thread Colin King
From: Colin Ian King Trival fix, dev_err message is missing a \n, so add it. Also break line as it was over 80 chars wide. Signed-off-by: Colin Ian King --- drivers/nvme/host/rdma.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

Re: [PATCH 3/3] g_NCR5380: Stop using scsi_module.c

2016-09-25 Thread Christoph Hellwig
> +static int generic_NCR5380_isa_match(struct device *pdev, unsigned int ndev) > +{ > + struct Scsi_Host *sh = NULL; > + > + sh = generic_NCR5380_hw_init(_template, pdev, base[ndev], > + irq[ndev], card[ndev]); > + if (!sh && base[ndev]) > +

[PATCH 4/5] irqtime: Consolidate accounting synchronization with u64_stats API

2016-09-25 Thread Frederic Weisbecker
The irqtime accounting currently implement its own ad hoc implementation of u64_stats API. Lets rather consolidate it with the appropriate library. Reviewed-by: Rik van Riel Cc: Paolo Bonzini Cc: Wanpeng Li Cc: Mike Galbraith

[PATCH 3/5] u64_stats: Introduce IRQs disabled helpers

2016-09-25 Thread Frederic Weisbecker
Introduce light versions of u64_stats helpers for context where either preempt or IRQs are disabled. This way we can make this library usable by scheduler irqtime accounting which currenty implement its ad-hoc version. Cc: Rik van Riel Cc: Paolo Bonzini Cc:

[PATCH 5/5] irqtime: Consolidate irqtime flushing code

2016-09-25 Thread Frederic Weisbecker
The code performing irqtime nsecs stats flushing to kcpustat is roughly the same for hardirq and softirq. So lets consolidate that common code. Reviewed-by: Rik van Riel Cc: Paolo Bonzini Cc: Wanpeng Li Cc: Mike Galbraith

[PATCH 1/5] irqtime: No need for preempt-safe accessors

2016-09-25 Thread Frederic Weisbecker
We can safely use the preempt-unsafe accessors for irqtime when we flush its counters to kcpustat as IRQs are disabled at this time. Reviewed-by: Rik van Riel Cc: Paolo Bonzini Cc: Wanpeng Li Cc: Mike Galbraith Cc:

[PATCH 2/5] irqtime: Remove needless IRQs disablement on kcpustat update

2016-09-25 Thread Frederic Weisbecker
The callers of the functions performing irqtime kcpustat updates have IRQS disabled, no need to disable them again. Reviewed-by: Rik van Riel Cc: Paolo Bonzini Cc: Wanpeng Li Cc: Mike Galbraith Cc: Peter Zijlstra

[PATCH 0/5] sched/cputime: irqtime cleanups v2

2016-09-25 Thread Frederic Weisbecker
This second version fixes the ifdeffery issues reported by Paolo Bonzini and includes Rick's reviewed-by tags. I also intended to include Peterz's patch for lockdep assertion on irqs disabled but unfortunately I get false positives as irqs_disabled() doesn't always imply

Re: [PATCH 4/3] g_NCR5380: Merge g_NCR5380 and g_NCR5380_mmio

2016-09-25 Thread Christoph Hellwig
On Mon, Sep 26, 2016 at 10:37:13AM +1000, Finn Thain wrote: > Let's not do that yet. I have a patch series that fixes the type casting > and related issues that I will post soon. I expect that Ondrej's patches > will be ready before mine are, so I'll rebase mine on this work before I > ask him

[PATCH 11/11 RESEND] staging: dgnc: introduce find_board_by_major()

2016-09-25 Thread Daeseok Youn
It was used to get a board structure with dgnc_BoardsByMajor array. But this driver already has the array for managing initialized board as dgap_board[]. It can be used for searching the board structure by major number. Signed-off-by: Daeseok Youn --- RESEND: send the

Re: [PATCH] mm: check VMA flags to avoid invalid PROT_NONE NUMA balancing

2016-09-25 Thread Rik van Riel
On Sun, 2016-09-25 at 15:50 -0700, Linus Torvalds wrote: > On Sun, Sep 25, 2016 at 3:34 PM, Rik van Riel > wrote: > > > > > > The patch looks good to me, too. > > > > Acked-by: Rik van Riel > > Thanks, amended the commit since I hadn't pushed out yet. > >

[PATCH 10/11 RESEND] staging: dgnc: remove useless variables

2016-09-25 Thread Daeseok Youn
The dgnc_major_serial_registered and dgnc_major_serial_registered do not need to use to check whether the tty driver is registered or not. These variables are used only in dgnc_cleanup_tty() function, This function will be called normally with initialized board structure. It means the

RE: [v12, 0/8] Fix eSDHC host version register bug

2016-09-25 Thread Y.B. Lu
Any comments about this version patchset ? :) > -Original Message- > From: Yangbo Lu [mailto:yangbo...@nxp.com] > Sent: Wednesday, September 21, 2016 2:57 PM > To: linux-...@vger.kernel.org; ulf.hans...@linaro.org; Scott Wood; Arnd > Bergmann > Cc: linuxppc-...@lists.ozlabs.org;

Re: thousands of kworker processes with 4.7.x and 4.8-rc*

2016-09-25 Thread Nikolay Borisov
On 25.09.2016 15:40, Tomasz Chmielewski wrote: > On 2016-09-25 18:29, Tomasz Chmielewski wrote: > >>> I'll try to bisect. >> >> OK, not a kernel regression, but some config change caused it. >> However, I'm not able to locate which change exactly. >> >> I'm attaching two configs which I've

[PATCH] net: smc91x: take into account register shift

2016-09-25 Thread Robert Jarzmik
This aligns smc91x with its cousin, namely smc911x.c. This also allows the driver to run also in a device-tree based lubbock board build, on which it was tested. Signed-off-by: Robert Jarzmik --- drivers/net/ethernet/smsc/smc91x.c | 3 +++ 1 file changed, 3 insertions(+)

Re: [PATCH 1/2] clk: rockchip: add I2S internal clock IDs for rk3288

2016-09-25 Thread Heiko Stuebner
Hi John, sorry this took so long, Am Mittwoch, 7. September 2016, 19:18:23 CEST schrieb John Keeping: > On Wed, 07 Sep 2016 19:58:31 +0200, Heiko Stuebner wrote: > > Am Mittwoch, 7. September 2016, 17:53:29 CEST schrieb John Keeping: > > > To minimize jitter on the I2S clocks, it is important

[PATCH] nvme: add missing \n to end of dev_warn message

2016-09-25 Thread Colin King
From: Colin Ian King Trival fix, dev_warn message is missing a \n, so add it. Signed-off-by: Colin Ian King --- drivers/nvme/host/pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nvme/host/pci.c

Re: [PATCH, TRIVIAL] Fixed block comment coding style

2016-09-25 Thread Larry Finger
On 09/25/2016 04:18 PM, Gonçalo Salazar wrote: Fixed a block comment indentation in the rtl8712 usb_intf.c file. Made this as a first commit. Please let me know of any feedback you have. Thanks, Gonçalo Salazar Your subject heeds to specify which driver is involved. In addition, fixes for

Linux 4.8-rc8

2016-09-25 Thread Linus Torvalds
So as already mentioned last week (and hinted at as a possibility), here's rc8. Things actually did start to calm down this week, but I didn't get the feeling that there was no point in doing one final rc, so here we are. I expect the final 4.8 release next weekend, unless something really

[PATCH] sched: Change READ_ONCE(jiffies) into jiffies

2016-09-25 Thread Bart Van Assche
A quote from Documentation/memory_barriers.txt: For example, because 'jiffies' is marked volatile, it is never necessary to say READ_ONCE(jiffies). The reason for this is that READ_ONCE() and WRITE_ONCE() are implemented as volatile casts, which has no effect when its argument is already marked

Re: [PATCH 3/3] g_NCR5380: Stop using scsi_module.c

2016-09-25 Thread Finn Thain
On Sat, 24 Sep 2016, Ondrej Zary wrote: > Convert g_NCR5380 to use scsi_add_host instead of scsi_module.c > Use pnp_driver and isa_driver to manage cards. > > Signed-off-by: Ondrej Zary > --- > drivers/scsi/g_NCR5380.c | 310 >

Re: [RFC/PATCH] usb: misc: Add a driver for TC7USB40MU

2016-09-25 Thread Peter Chen
On Thu, Sep 22, 2016 at 11:51:02AM -0700, Stephen Boyd wrote: > Quoting Peter Chen (2016-09-16 18:16:05) > > On Wed, Sep 14, 2016 at 01:55:02AM -0700, Stephen Boyd wrote: > > > Quoting Stephen Boyd (2016-09-13 18:42:46) > > > > On the db410c 96boards platform we have a TC7USB40MU[1] on the > > > >

[PULL] mtd: nand: material for 4.9

2016-09-25 Thread Boris Brezillon
Hi Brian, Here is my PR for 4.9. Let me know if you see any problem. Also, please let us know ahead of time if you can't take this PR (I know you are busy lately, and I guess I could send this PR directly to Linus or ask Richard to take it in its UBIFS/UBI tree). Thanks, Boris The following

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

2016-09-25 Thread Chase Metzger
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 b/drivers/staging/greybus/gpio.c index ea8234a..5e06e42

Re: RK3288 USB Issues

2016-09-25 Thread Norbert Lange
Hi, I built a kernel with the sources from your branch, and the problem is gone. Looking at the differences I had added the reset to the wrong usb port in the dtb (phy1 instead of phy 2) Thanks for your help figuring this out usbphy1: usb-phy@334 { #phy-cells = <0>; reg = <0x334>; clocks = <

Re: [PATCH] ixgbe: mark symbols static where possible

2016-09-25 Thread Jeff Kirsher
On Sun, 2016-09-25 at 14:03 +0800, Baoyou Xie wrote: > We get 2 warnings when building kernel with W=1: > drivers/net/ethernet/intel/ixgbe/ixgbe_x550.c:2128:5: warning: no > previous prototype for 'ixgbe_led_on_t_x550em' [-Wmissing-prototypes] >

Re: [PATCH 01/11 V2] staging: dgnc: remove redundant initialization for channel array

2016-09-25 Thread DaeSeok Youn
2016-09-23 21:15 GMT+09:00 Greg KH : > On Fri, Sep 23, 2016 at 10:25:02AM +0900, Daeseok Youn wrote: >> The channel array in board_t was initialized in dgnc_found_board() >> with NULL. But the channel is going to initialize in dgnc_tty_init() >> again. So the channel

Re: [PATCH] net: smc91x: take into account register shift

2016-09-25 Thread David Miller
From: Robert Jarzmik Date: Sun, 25 Sep 2016 23:00:45 +0200 > This aligns smc91x with its cousin, namely smc911x.c. > This also allows the driver to run also in a device-tree based lubbock > board build, on which it was tested. > > Signed-off-by: Robert Jarzmik

[PATCH 4/3] g_NCR5380: Merge g_NCR5380 and g_NCR5380_mmio

2016-09-25 Thread Ondrej Zary
Merge the PIO and MMIO code (with the help of ioport_map) in g_NCR5380 and delete g_NCR5380_mmio. Signed-off-by: Ondrej Zary --- drivers/scsi/Kconfig | 32 ++--- drivers/scsi/g_NCR5380.c | 257 +++--

[PATCH] net: tg3: use new api ethtool_{get|set}_link_ksettings

2016-09-25 Thread Philippe Reynes
The ethtool api {get|set}_settings is deprecated. We move this driver to new api {get|set}_link_ksettings. Signed-off-by: Philippe Reynes --- drivers/net/ethernet/broadcom/tg3.c | 112 +++ 1 files changed, 62 insertions(+), 50 deletions(-)

Re: [PATCH 1/2] mfd: ab8500-debugfs: add function declaration

2016-09-25 Thread Arnd Bergmann
On Sunday 25 September 2016, Baoyou Xie wrote: > On 24 September 2016 at 16:30, Arnd Bergmann wrote: > > > > This can't be right for two reasons: > > > > - the suspend_test_wake_cause_interrupt_is_mine function again > > is only used in the file it is defined in > > > > but, it

Re: [PATCH 2/3] g_NCR5380: Reduce overrides[] from array to struct

2016-09-25 Thread Christoph Hellwig
On Sat, Sep 24, 2016 at 08:59:04PM +0200, Ondrej Zary wrote: > Remove compile-time card type definition GENERIC_NCR5380_OVERRIDE. > Then remove all code iterating the overrides[] array and reduce it to > struct card. > > Signed-off-by: Ondrej Zary Looks great!

[PATCH] [4.8-rc7, regression] fault_in_multipages_readable() throws set-but-unused error

2016-09-25 Thread Dave Chinner
From: Dave Chinner When building XFS with -Werror, it now fails with: ./include/linux/pagemap.h: In function ¿fault_in_multipages_readable¿: ./include/linux/pagemap.h:602:16: error: variable ¿c¿ set but not used [-Werror=unused-but-set-variable] volatile char c;

Re: [PATCH 4/3] g_NCR5380: Merge g_NCR5380 and g_NCR5380_mmio

2016-09-25 Thread Finn Thain
On Sun, 25 Sep 2016, Christoph Hellwig wrote: > On Sun, Sep 25, 2016 at 09:39:20PM +0200, Ondrej Zary wrote: > > > #define NCR5380_read(reg) \ > > + ioread8(((struct NCR5380_hostdata *)shost_priv(instance))->iomem + \ > > + ((struct NCR5380_hostdata *)shost_priv(instance))->offset

Re: [PATCH] [4.8-rc7, regression] fault_in_multipages_readable() throws set-but-unused error

2016-09-25 Thread Linus Torvalds
Thanks, applied. I did happen to notice: On Sun, Sep 25, 2016 at 4:57 PM, Dave Chinner wrote: > > ./include/linux/pagemap.h: In function ¿fault_in_multipages_readable¿: > ./include/linux/pagemap.h:602:16: error: variable ¿c¿ set but not used >

[PATCH 1/2] drm: Simplify logging macros, convert DRM_NOTE to DRM_NOTICE

2016-09-25 Thread Joe Perches
Use a bit more consistent style with kernel loglevels without using macro argument concatenation. Miscellanea: o Single statement macros don't need do {} while (0) Signed-off-by: Joe Perches --- drivers/gpu/drm/i915/intel_guc_loader.c | 22 --

[PATCH 0/2] drm: Neaten and reduce object size

2016-09-25 Thread Joe Perches
Joe Perches (2): drm: Simplify logging macros, convert DRM_NOTE to DRM_NOTICE drm: Simplify drm_printk to reduce object size quite a bit drivers/gpu/drm/drm_drv.c | 5 +-- drivers/gpu/drm/i915/intel_guc_loader.c | 22 +++-- include/drm/drmP.h | 56

[PATCH 2/2] drm: Simplify drm_printk to reduce object size quite a bit

2016-09-25 Thread Joe Perches
Remove function name and special " *ERROR*" from argument list $ size drivers/gpu/drm/built-in.o* (x86-32 defconfig, most drm selected) textdata bss dec hex filename 5635366 182579 14328 5832273 58fe51 drivers/gpu/drm/built-in.o.new 5779552 182579 14328 5976459 5b318b

RE: [v6,2/2] QE: remove PPCisms for QE

2016-09-25 Thread Qiang Zhao
On Sun, Sep 25, 2016 at 12:19PM -0500, Scott Wood wrote: > -Original Message- > From: Scott Wood [mailto:o...@buserror.net] > Sent: Sunday, September 25, 2016 12:19 PM > To: Qiang Zhao > Cc: linuxppc-...@lists.ozlabs.org; pku@gmail.com; X.B. Xie >

Re: [PATCH] Staging: android: fixed permissions style issue

2016-09-25 Thread Joe Perches
On Sun, 2016-09-25 at 21:47 -0500, Eric Salem wrote: > Fixed a coding style issue. Changed symbolic permissions to octal. If you do these, please shorten the lines where possible. > diff --git a/drivers/staging/android/lowmemorykiller.c > b/drivers/staging/android/lowmemorykiller.c [] > @@

Re: [PATCH 2/2] radix-tree: Fix optimisation problem

2016-09-25 Thread Linus Torvalds
On Sun, Sep 25, 2016 at 11:04 AM, Linus Torvalds wrote: > > The more I look at that particular piece of code, the less I like it. It's > buggy shit. It needs to be rewritten entirely too actually check for sibling > entries, not that ad-hoc arithmetic crap. Here's

Re: [PATCH] drm/i2c: tda998x: don't register the connector

2016-09-25 Thread Daniel Vetter
On Fri, Sep 23, 2016 at 03:42:52PM +0100, Brian Starkey wrote: > On Fri, Sep 23, 2016 at 03:13:15PM +0200, Daniel Vetter wrote: > > On Fri, Sep 23, 2016 at 01:52:49PM +0100, Brian Starkey wrote: > > > On Fri, Sep 23, 2016 at 12:58:46PM +0200, Daniel Vetter wrote: > > > > On Fri, Sep 23, 2016 at

[PATCH, TRIVIAL] Fixed block comment coding style

2016-09-25 Thread Gonçalo Salazar
Fixed a block comment indentation in the rtl8712 usb_intf.c file. Made this as a first commit. Please let me know of any feedback you have. Thanks, Gonçalo Salazar Signed-off-by: Gonçalo Salazar --- drivers/staging/rtl8712/usb_intf.c | 2 +- 1 file changed, 1

Re: [PATCH] mm: check VMA flags to avoid invalid PROT_NONE NUMA balancing

2016-09-25 Thread Linus Torvalds
On Sun, Sep 25, 2016 at 3:34 PM, Rik van Riel wrote: > > The patch looks good to me, too. > > Acked-by: Rik van Riel Thanks, amended the commit since I hadn't pushed out yet. Btw, the only reason this bug could happen is that we do that "force=1" for remote vm

Re: [PATCH] mm: check VMA flags to avoid invalid PROT_NONE NUMA balancing

2016-09-25 Thread Hugh Dickins
On Sun, 25 Sep 2016, Linus Torvalds wrote: > On Sun, Sep 25, 2016 at 3:34 PM, Rik van Riel wrote: > > > > The patch looks good to me, too. > > > > Acked-by: Rik van Riel > > Thanks, amended the commit since I hadn't pushed out yet. > > Btw, the only reason

[PATCH 05/11 V2 RESEND] staging: dgnc: move functions unrelated with dgnc_found_board()

2016-09-25 Thread Daeseok Youn
The functions related with tty device initialization are needed to be moved from dgnc_found_board() to dgnc_init_one(). Signed-off-by: Daeseok Youn --- RESEND: send the whole series again. Because I missed some patches when the V2 patches sent. V2: The subject line was

Re: [PATCH 1/3] g_NCR5380: Remove deprecated __setup

2016-09-25 Thread Finn Thain
On Sat, 24 Sep 2016, Ondrej Zary wrote: > Remove deprecated __setup for parsing command line parameters. > g_NCR5380.* parameters could be used instead. Removing useful features is deprecated too. The best reason I can see for removing the __setup parameters is that the g_NCR5380 module

[PATCH 04/11] staging: dgnc: kfree for board structure in dgnc_found_board()

2016-09-25 Thread Daeseok Youn
The board structure should be freed when any function was failed in dgnc_found_board(). And the board strucure will be stored into dgnc_board array when the dgnc_found_board() function has no error. Signed-off-by: Daeseok Youn --- RESEND: send the whole series again.

[PATCH v2] Staging: android: fixed permissions style issue

2016-09-25 Thread Eric Salem
Fixed a coding style issue. Changed symbolic permissions to octal. Signed-off-by: Eric Salem --- drivers/staging/android/lowmemorykiller.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/staging/android/lowmemorykiller.c

[PATCH 2/2] power: supply: max17042_battery: add technology property support

2016-09-25 Thread Wolfgang Wiedmeyer
This patch reports the battery technology as Li-ion. Signed-off-by: Wolfgang Wiedmeyer --- drivers/power/max17042_battery.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/power/max17042_battery.c b/drivers/power/max17042_battery.c index 20cb1fd..43cb5df

[PATCH 0/2] power: supply: max17042_battery: Improve properties

2016-09-25 Thread Wolfgang Wiedmeyer
These two patches fix the capacity property and add the technology property. Wolfgang Wiedmeyer (2): power: supply: max17042_battery: use VF SOC register for capacity property power: supply: max17042_battery: add technology property support drivers/power/max17042_battery.c | 6 +- 1

[PATCH 1/2] power: supply: max17042_battery: use VF SOC register for capacity property

2016-09-25 Thread Wolfgang Wiedmeyer
The capacity property uses the RepSOC register to report the current state of charge. This register did not provide a reliable SOC value during my testing with the max17047 variant on a Galaxy S3 (Trats2/GT-I9300). The reported value did not change or even stayed zero in some cases. However, the

[PATCH] lightnvm: add missing \n to end of dev_err message

2016-09-25 Thread Colin King
From: Colin Ian King Trival fix, dev_err message is missing a \n, so add it. Signed-off-by: Colin Ian King --- drivers/nvme/host/lightnvm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nvme/host/lightnvm.c

[PATCH V2] staging: rtl8172: usb_intf.c: Fixed block comment coding style

2016-09-25 Thread Gonçalo Salazar
Fixed a block comment indentation in the rtl8712 usb_intf.c file. Made this as a first commit. Resubmitted with updated subject. Please let me know of any feedback you have. Thanks, Gonçalo Salazar Signed-off-by: Gonçalo Salazar --- drivers/staging/rtl8712/usb_intf.c | 2

Re: [PATCH] igb: mark igb_rxnfc_write_vlan_prio_filter() static

2016-09-25 Thread Arnd Bergmann
On Monday 26 September 2016, Jeff Kirsher wrote: > On Sun, 2016-09-25 at 14:05 +0800, Baoyou Xie wrote: > > We get 1 warning when building kernel with W=1: > > drivers/net/ethernet/intel/igb/igb_ethtool.c:2707:5: warning: no previous > > prototype for 'igb_rxnfc_write_vlan_prio_filter'

[PATCH 02/11 RESEND] staging: dgnc: remove useless message buffer

2016-09-25 Thread Daeseok Youn
There is a temporary message buffer for the boot message in dgnc_found_board() but the buffer was not used anywhere in dgnc driver. Signed-off-by: Daeseok Youn --- RESEND: send the whole series again. Because I missed some patches when the V2 patches sent.

[PATCH 01/11 V2 RESEND] staging: dgnc: remove redundant initialization for channel array

2016-09-25 Thread Daeseok Youn
The channel array in board_t was initialized in dgnc_found_board() with NULL. But the channel is going to initialize in dgnc_tty_init(). So the channel array doesn't need to set NULL for initailization. Signed-off-by: Daeseok Youn --- RESEND: send the whole series again.

RE: [PATCH] pm8001: mark symbols static where possible

2016-09-25 Thread lindar_liu
Thanks for fix. Acked-by: Lindar Liu > -Original Message- > From: Baoyou Xie [mailto:baoyou@linaro.org] > Sent: Friday, September 23, 2016 9:54 PM > To: jinpu.w...@profitbricks.com; lindar_...@usish.com; > j...@linux.vnet.ibm.com; martin.peter...@oracle.com >

Re: [PATCH] [4.8-rc7, regression] fault_in_multipages_readable() throws set-but-unused error

2016-09-25 Thread Dave Chinner
On Sun, Sep 25, 2016 at 06:21:05PM -0700, Linus Torvalds wrote: > Thanks, applied. > > I did happen to notice: > > On Sun, Sep 25, 2016 at 4:57 PM, Dave Chinner wrote: > > > > ./include/linux/pagemap.h: In function ¿fault_in_multipages_readable¿: > >

Re: [PATCH -v3 00/10] THP swap: Delay splitting THP during swapping out

2016-09-25 Thread Huang, Ying
Shaohua Li writes: > On Fri, Sep 23, 2016 at 10:32:39AM +0800, Huang, Ying wrote: >> Rik van Riel writes: >> >> > On Thu, 2016-09-22 at 15:56 -0700, Shaohua Li wrote: >> >> On Wed, Sep 07, 2016 at 09:45:59AM -0700, Huang, Ying wrote: >> >> >. >> >> > - It will

Re: [PATCH -v3 00/10] THP swap: Delay splitting THP during swapping out

2016-09-25 Thread Shaohua Li
On Fri, Sep 23, 2016 at 10:32:39AM +0800, Huang, Ying wrote: > Rik van Riel writes: > > > On Thu, 2016-09-22 at 15:56 -0700, Shaohua Li wrote: > >> On Wed, Sep 07, 2016 at 09:45:59AM -0700, Huang, Ying wrote: > >> >  > >> > - It will help the memory fragmentation, especially

Re: [PATCH] mm: check VMA flags to avoid invalid PROT_NONE NUMA balancing

2016-09-25 Thread Linus Torvalds
I was kind of assuming this would go through the normal channels for THP patches, but it's been two weeks... Can I have an ACK from the involved people, and I'll apply it directly.. Mel? Rik? Linus On Sun, Sep 25, 2016 at 11:47 AM, Lorenzo Stoakes wrote:

[tip:irq/core] genirq: Make function __irq_do_set_handler() static

2016-09-25 Thread tip-bot for Wei Yongjun
Commit-ID: b8129a1f6aaaca02d92186acf19ceb545b4b489a Gitweb: http://git.kernel.org/tip/b8129a1f6aaaca02d92186acf19ceb545b4b489a Author: Wei Yongjun AuthorDate: Sun, 25 Sep 2016 15:36:39 + Committer: Thomas Gleixner CommitDate: Sun, 25 Sep

[PATCH] cpufreq: kirkwood: add missing \n to end of dev_err messages

2016-09-25 Thread Colin King
From: Colin Ian King Trival fix, dev_err messages are missing a \n, so add it. Signed-off-by: Colin Ian King --- drivers/cpufreq/kirkwood-cpufreq.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

Re: [PATCH] mm: check VMA flags to avoid invalid PROT_NONE NUMA balancing

2016-09-25 Thread Linus Torvalds
On Sun, Sep 25, 2016 at 1:52 PM, Linus Torvalds wrote: > > Can I have an ACK from the involved people, and I'll apply it > directly.. Mel? Rik? Oh well. The patch looks fine to me and I want to include it in the rc8 release, so I'll apply it. Worst comes to worst

Re: [PATCH] igb: mark igb_rxnfc_write_vlan_prio_filter() static

2016-09-25 Thread Jeff Kirsher
On Sun, 2016-09-25 at 14:05 +0800, Baoyou Xie wrote: > We get 1 warning when building kernel with W=1: > drivers/net/ethernet/intel/igb/igb_ethtool.c:2707:5: warning: no previous > prototype for 'igb_rxnfc_write_vlan_prio_filter' [-Wmissing-prototypes] > > In fact, this function is only used in

[PATCH 2/2] fix memory leaks in tracing_buffers_splice_read()

2016-09-25 Thread Steven Rostedt
From: Al Viro Cc: sta...@vger.kernel.org Signed-off-by: Al Viro --- kernel/trace/trace.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index

[PATCH 0/2] [GIT PULL] tracing: A couple of fixes with tracefs

2016-09-25 Thread Steven Rostedt
Linus, Al Viro has been looking at the tracefs code, and has pointed out some issues. This contains one fix by me and one by Al. I'm sure that he'll come up with more but for now I tested these patches and they don't appear to have any negative impact on tracing. Please pull the latest

[PATCH 1/2] tracing: Move mutex to protect against resetting of seq data

2016-09-25 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" The iter->seq can be reset outside the protection of the mutex. So can reading of user data. Move the mutex up to the beginning of the function. Fixes: d7350c3f45694 ("tracing/core: make the read callbacks reentrants") Cc:

Re: [PATCH 3/6] isdn/hisax: add function declarations

2016-09-25 Thread Arnd Bergmann
On Sunday 25 September 2016, Baoyou Xie wrote: > > > @@ -1350,3 +1350,63 @@ static inline struct pci_dev > > *hisax_find_pci_device(unsigned int vendor, > > > } > > > > > > #endif > > > + > > > +#if CARD_TELES3 > > > +int setup_teles3(struct IsdnCard *card); > > > +#endif > > > + > > > +#if

Re: [BUG, 4.8-rc7] perf: oops in intel_pmu_enable_all

2016-09-25 Thread Dave Chinner
On Mon, Sep 26, 2016 at 09:22:45AM +1000, Dave Chinner wrote: > Hi Folks, > > I just upgraded a test VM from 4.8-rc6 to 4.8-rc7, and went to run: > > # perf_4.7 top -g -U > > inside the VM - the kernel oops with the trace below. The perf > binary was built from a 4.7 kernel, so it's not the

Re: [PATCH 1/3] g_NCR5380: Remove deprecated __setup

2016-09-25 Thread Christoph Hellwig
On Sat, Sep 24, 2016 at 08:59:03PM +0200, Ondrej Zary wrote: > Remove deprecated __setup for parsing command line parameters. > g_NCR5380.* parameters could be used instead. This assumes everyone has moved over to it. But I suspect the remaining hand full of users could adjust..

[PATCH 03/11 V2 RESEND] staging: dgnc: missing NULL check for ioremap in dgnc_do_remap()

2016-09-25 Thread Daeseok Youn
The ioremap() function can be failed, so it need to have error handling in dgnc_do_remap(). And also the return type of dgnc_do_remap() should be changed from "void" to "int" Signed-off-by: Daeseok Youn --- RESEND: send the whole series again. Because I missed some

Re: [PATCH 00/14] perf clang: Support compiling BPF script use builtin clang

2016-09-25 Thread Wangnan (F)
On 2016/9/24 23:16, Alexei Starovoitov wrote: On Fri, Sep 23, 2016 at 12:49:47PM +, Wang Nan wrote: This patch set is the first step to implement features I announced in LinuxCon NA 2016. See page 31 of:

Aw

2016-09-25 Thread info
Benötigen Sie eine Finanzierung? Haben Sie einen Kredit für geschäftliche oder persönliche Bedürfnisse und Projektfinanzierung benötigen? Möchten Sie Ihr Unternehmen zu refinanzieren? Unser Unternehmen ist in den Vereinigten Staaten und Europa. Wir bieten Ihnen ein Darlehen für jede Person oder

Re: [PATCH v2 1/1] mm/hugetlb: fix memory offline with hugepage size > memory block size

2016-09-25 Thread Rui Teng
On 9/23/16 7:03 PM, Gerald Schaefer wrote: On Fri, 23 Sep 2016 14:40:33 +0800 Rui Teng wrote: On 9/22/16 5:51 PM, Michal Hocko wrote: On Wed 21-09-16 14:35:34, Gerald Schaefer wrote: dissolve_free_huge_pages() will either run into the VM_BUG_ON() or a list

[PATCH] Staging: android: fixed permissions style issue

2016-09-25 Thread Eric Salem
Fixed a coding style issue. Changed symbolic permissions to octal. Signed-off-by: Eric Salem --- drivers/staging/android/lowmemorykiller.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/android/lowmemorykiller.c

Re: [PATCH] [4.8-rc7, regression] fault_in_multipages_readable() throws set-but-unused error

2016-09-25 Thread Linus Torvalds
On Sun, Sep 25, 2016 at 7:25 PM, Dave Chinner wrote: >> You have some nasty unicode corruption. The email is marked as being >> >> Content-Type: text/plain; charset=UTF-8 >> Content-Transfer-Encoding: 8bit > > It's whatever git-send-email spat out. I was under the

Re: [PATCH] cpufreq: kirkwood: add missing \n to end of dev_err messages

2016-09-25 Thread Viresh Kumar
On 25-09-16, 14:35, Colin King wrote: > From: Colin Ian King > > Trival fix, dev_err messages are missing a \n, so add it. > > Signed-off-by: Colin Ian King > --- > drivers/cpufreq/kirkwood-cpufreq.c | 8 > 1 file changed, 4

Re: [PATCH] cpufreq: st: add missing \n to end of dev_err message

2016-09-25 Thread Viresh Kumar
On 25-09-16, 14:40, Colin King wrote: > From: Colin Ian King > > Trival fix, dev_err message is missing a \n, so add it. > > Signed-off-by: Colin Ian King > --- > drivers/cpufreq/sti-cpufreq.c | 2 +- > 1 file changed, 1 insertion(+), 1

[PATCH] staging: sm750fb: rename getChipType to sm750_get_chip_type

2016-09-25 Thread Moshe Green
Rename CamelCased function getChipType to sm750_get_chip_type (prefex with sm750 in order to make the context of the function clear). This issue was found by checkpatch.pl Signed-off-by: Moshe Green --- drivers/staging/sm750fb/ddk750_chip.c | 16

  1   2   3   4   5   6   >