Re: [PATCH 00/10] Code style patches for staging/irda

2017-12-02 Thread Greg KH
On Sun, Nov 26, 2017 at 01:45:55AM +0300, Mike wrote: > > This is my task from Little Penguin :) What does that mean? > > Mike (10): > trailing whitespace fixed > spaces before tabs are fixed > trailing */ on a separate line fixed > if-else code style fixed > space prohibited before t

Re: [PATCH 01/10] trailing whitespace fixed

2017-12-02 Thread Greg KH
On Sun, Nov 26, 2017 at 01:45:56AM +0300, Mike wrote: > Signed-off-by: Mike > --- > drivers/staging/irda/drivers/ali-ircc.c | 1002 > +++ > 1 file changed, 501 insertions(+), 501 deletions(-) > Hi, This is the friendly patch-bot of Greg Kroah-Hartman. You have sen

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

2017-12-02 Thread Ard Biesheuvel
On 1 December 2017 at 16:33, Kees Cook wrote: > On Fri, Dec 1, 2017 at 7:34 AM, Greg Kroah-Hartman > wrote: > >> And isn't there a specific %p modifier you should use for a kernel >> pointer. I've lost the thread here for what should, or should not, be >> done for kernel pointers these days base

Re: [PATCH tip/core/rcu 10/10] rcu: Account for rcu_all_qs() in cond_resched()

2017-12-02 Thread Peter Zijlstra
On Fri, Dec 01, 2017 at 11:21:44AM -0800, Paul E. McKenney wrote: > If cond_resched() returns false, then it has already invoked > rcu_all_qs(). This is also invoked (now redundantly) by > rcu_note_voluntary_context_switch(). This commit therefore changes > cond_resched_rcu_qs() to invoke rcu_not

Re: [PATCH 0/2] Fix boot regression for s390 and remove break_lock

2017-12-02 Thread Heiko Carstens
On Tue, Nov 28, 2017 at 06:42:17PM +, Will Deacon wrote: > Hi all, > > The following two patches do the following: > > 1. Fix boot breakage reported on s390 caused by a8a217c22116 > 2. Kill off the break_lock field entirely, since it's not actually > that useful > > I didn't go the

[PATCH v2 net-next] net/tcp: trace all TCP/IP state transition with tcp_set_state tracepoint

2017-12-02 Thread Yafang Shao
The TCP/IP transition from TCP_LISTEN to TCP_SYN_RECV and some other transitions are not traced with tcp_set_state tracepoint. In order to trace the whole tcp lifespans, two helpers are introduced, void __tcp_set_state(struct sock *sk, int state) void __sk_state_store(struct sock *sk, int newstate

[PATCH] staging: pi433: Bugfix for shift operation in rf69_get_modulation()

2017-12-02 Thread Marcus Wolf
From: root Defines used in cases contain already shifted bits, so currentValue must not be shifted. Signed-off-by: Marcus Wolf --- drivers/staging/pi433/rf69.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c ind

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

2017-12-02 Thread Paolo Valente
> 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 blkio stats behind >> CONFIG_DEBUG_BLK_CGROUP") introduced two batches of confusing ifdefs: >> one reported in [1], plus a similar o

Re: [PATCH] staging: pi433: Bugfix for shift operation in rf69_get_modulation()

2017-12-02 Thread Greg KH
On Sat, Dec 02, 2017 at 11:58:12AM +0200, Marcus Wolf wrote: > From: root I think something went wrong here :) Also, you should never need to do kernel development as root... greg k-h

Re: [PATCH] staging: pi433: Bugfix for shift operation in rf69_get_modulation()

2017-12-02 Thread Marcus Wolf
Hi Greg, yes, of course. Just got the wrong console to enter the commit command - but when I recognized, it was already too late :-/ The patch already was generated in user console. I am wondering myself, where this From: comes from. I used 'git send-email -1 --annotate' How to prevent the

linux-next on Nokia N900: hang on boot

2017-12-02 Thread Pavel Machek
Hi! next from 2017-12-01 was tested. MMC is detected, but then I see wtl4030_bci: .. battery temperature out of range, and machine halts. This is just before userspace would be started. 4.15-rc1 seems to work ok on the box. Any ideas?

[PATCH] firmware: ti_sci: Use %zu for size_t print format

2017-12-02 Thread Nishanth Menon
mbox_msg->len is of type size_t and %d is incorrect format. Instead use %zu for handling size_t correctly. Signed-off-by: Nishanth Menon --- drivers/firmware/ti_sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c ind

[PATCH] mailbox: ti-msgmgr: Switch to SPDX Licensing

2017-12-02 Thread Nishanth Menon
Switch to SPDX licensing and drop the GPL text which comes redundant. Signed-off-by: Nishanth Menon --- Jassi, I am not sure if you are planning on doing mailbox enmasse, if you are, then feel free to drop this to take the overall changes. Regards, Nishanth Menon drivers/mailbox/ti-msgmgr.c |

[PATCH] mailbox: ti-msgmgr: Use %zu for size_t print format

2017-12-02 Thread Nishanth Menon
message->len is of type size_t and %d is incorrect format usage. Instead use %zu for handling size_t correctly. Signed-off-by: Nishanth Menon --- drivers/mailbox/ti-msgmgr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mailbox/ti-msgmgr.c b/drivers/mailbox/ti-msgmg

[PATCH] staging: pi433: Bugfix for shift operation in rf69_get_modulation()

2017-12-02 Thread Marcus Wolf
Defines used in cases contain already shifted bits, so currentValue must not be shifted. signed_of_by: Marcus Wolf --- drivers/staging/pi433/rf69.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c index e69a215..12

Re: [PATCH 1/2] Undo the split of setup_cpu_entry_area

2017-12-02 Thread Thomas Gleixner
On Fri, 1 Dec 2017, Andy Lutomirski wrote: > This is obviously a hack. Either the patch should be adjusted back to > the version I sent or trap_init should forcibly initialize all PMDs > by something like __set_fixmap(..., __mkpte(0)); or however it's spelled. I split it because the whole thing

Re: [PATCH 2/2] x86/kpti: Reference all cpu_entry_area pagetables in the usermode tables

2017-12-02 Thread Thomas Gleixner
On Fri, 1 Dec 2017, Andy Lutomirski wrote: > We were manually configuring cpu_entry_area in the usermode tables. > This was error-prone and wasted memory. (Not much memory, but > still.) Instead, just reference the same pagetables. > > This avoids needing to keep the KPTI code and the normal >

Re: [PATCH 1/2] Undo the split of setup_cpu_entry_area

2017-12-02 Thread Thomas Gleixner
On Sat, 2 Dec 2017, Thomas Gleixner wrote: > On Fri, 1 Dec 2017, Andy Lutomirski wrote: > > > This is obviously a hack. Either the patch should be adjusted back to > > the version I sent or trap_init should forcibly initialize all PMDs > > by something like __set_fixmap(..., __mkpte(0)); or howe

[PATCH] staging: pi433: Bugfix for shift operation in rf69_get_modulation()

2017-12-02 Thread Marcus Wolf
Defines used in cases contain already shifted bits, so currentValue must not be shifted. Signed-off-by: Marcus Wolf --- drivers/staging/pi433/rf69.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c index e69a215..1

RE: [EXT] Re: [PATCH net] net: phylink: fix link state on phy-connect

2017-12-02 Thread Yan Markman
Hi Russel The Grygorii has raised one Additional point (about netif_carrier_off) I just didn't want to start before finishing the previous one. On ifconfig-down the mac_config() called but with LINK=0. The config has no any knowledge what is intention -- up or down and should be done under disab

Re: linux-next on Nokia N900: hang on boot

2017-12-02 Thread Pali Rohár
On Saturday 02 December 2017 11:14:56 Pavel Machek wrote: > Hi! > > next from 2017-12-01 was tested. > > MMC is detected, but then I see wtl4030_bci: .. battery temperature > out of range, and machine halts. This is just before userspace would > be started. > > 4.15-rc1 seems to work ok on the b

Re: [PATCH] iio: stm32: fix adc/trigger link error

2017-12-02 Thread Jonathan Cameron
On Mon, 27 Nov 2017 10:09:47 +0100 Fabrice Gasnier wrote: > On 09/25/2017 07:26 PM, Jonathan Cameron wrote: > > On Mon, 25 Sep 2017 11:16:04 +0100 > > Jonathan Cameron wrote: > > > >> On Mon, 25 Sep 2017 11:54:06 +0200 > >> Fabrice Gasnier wrote: > >> > >>> On 09/10/2017 06:00 PM, Jonathan

[PATCH] drm/tilcdc: make tilcdc_mode_hvtotal() static

2017-12-02 Thread Xiongwei Song
The function tilcdc_mode_hvtotal is local to the source and does not need to be in global scope, so make it static. drivers/gpu/drm/tilcdc/tilcdc_crtc.c:297:6: warning: no previous prototype for 'tilcdc_mode_hvtotal' [-Wmissing-prototypes] uint tilcdc_mode_hvtotal(const struct drm_display_mode *

[PATCH] pinctrl: gemini: Support drive strength setting

2017-12-02 Thread Linus Walleij
The Gemini pin controller can set drive strength for a few select groups of pins (not individually). Implement this for GMAC0 and 1 (ethernet ports), IDE and PCI. Cc: devicet...@vger.kernel.org Signed-off-by: Linus Walleij --- The DT binding part is just using generic bindings, so should be prett

Re: [PATCH v4 0/4] fix the clock setting for SAR ADC

2017-12-02 Thread Jonathan Cameron
On Tue, 28 Nov 2017 21:11:29 +0800 Yixun Lan wrote: > On 11/11/17 08:37, Jonathan Cameron wrote: > > On Tue, 7 Nov 2017 22:36:00 +0100 > > Martin Blumenstingl wrote: > > > >> Hi Yixun, > >> > >> On Tue, Nov 7, 2017 at 3:09 PM, Yixun Lan wrote: > >>> patch [1/4]: > >>> Fix wrong SARADC/SA

Re: [PATCH] iio: dummy_evgen: modify the return value check for irq_sim_init()

2017-12-02 Thread Jonathan Cameron
On Mon, 27 Nov 2017 11:54:49 +0100 Bartosz Golaszewski wrote: > As discussed with Marc Zyngier: irq_sim_init() and its devres variant > should return the base of the allocated interrupt range on success > rather than 0. This will be modified later - first, change the way > users handle the return

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

2017-12-02 Thread Marcus Wolf
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 is possible to switch of the function entry debug lines even while debug is switched on. Signed-off-by:

Re: [PATCH] iio: common: ssp_sensors: use ktime_get_real_ns() timestamps

2017-12-02 Thread Jonathan Cameron
On Mon, 27 Nov 2017 12:51:48 +0100 Arnd Bergmann wrote: > getnstimeofday() suffers from the overflow in y2038 on 32-bit > architectures and requires a conversion into the nanosecond format that > we want here. > > This changes ssp_parse_dataframe() to use ktime_get_real_ns() directly, > which do

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

2017-12-02 Thread Jonathan Cameron
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 > I2C device registered via OF will always match a legacy I2C device ID and >

Re: [PATCH] gpio: gpio-stmpe: make various char arrays static const, shrinks object size

2017-12-02 Thread Linus Walleij
On Tue, Nov 28, 2017 at 7:23 PM, Colin King wrote: > From: Colin Ian King > > Don't populate the read-only arrays edge_det_values, rise_values and > fall_values on the stack but instead make them static and constify them. > Makes the object code smaller by over 240 bytes: > > Before: >text

Re: [PATCH v6 03/17] mfd: madera: Add common support for Cirrus Logic Madera codecs

2017-12-02 Thread Linus Walleij
On Wed, Nov 29, 2017 at 12:36 PM, Richard Fitzgerald wrote: > On 29/11/17 10:18, Linus Walleij wrote: >> >> On Thu, Nov 23, 2017 at 6:13 PM, Richard Fitzgerald >> wrote: >> >>> +config MFD_MADERA_I2C >>> + bool "Cirrus Logic Madera codecs with I2C" >>> + select MFD_MADERA >>> +

Re: [PATCH v2] pinctrl: intel: Initialize GPIO properly when used through irqchip

2017-12-02 Thread Linus Walleij
On Wed, Nov 29, 2017 at 2:25 PM, Mika Westerberg wrote: > When a GPIO is requested using gpiod_get_* APIs the intel pinctrl driver > switches the pin to GPIO mode and makes sure interrupts are routed to > the GPIO hardware instead of IOAPIC. However, if the GPIO is used > directly through irqchip

Re: [PATCH] pinctrl: stm32: Fix copyright

2017-12-02 Thread Linus Walleij
On Wed, Nov 29, 2017 at 3:25 PM, Benjamin Gaignard wrote: > Uniformize STMicroelectronics copyrights header > > Signed-off-by: Benjamin Gaignard Patch applied. Yours, Linus Walleij

Re: Shutting down a VM with Kernel 4.14 will sometime hang and a reboot is the only way to recover.

2017-12-02 Thread Harald Moeller
Hello, my name is Harry and this is my first post here, hope I'm doing this the right way, sorry if not ... I'm not a subscriber to the full list yet so I understand I shall ask you to be personally CCed. I am following this as I do experience the same (or sort-a same) issue with 4.14.2. M

Re: [PATCH 2/2] iio: accel: bmc150: Check for a second ACPI device for BOSC0200

2017-12-02 Thread Jonathan Cameron
On Wed, 29 Nov 2017 22:31:12 + Jeremy Cline wrote: > Some BOSC0200 acpi_device-s describe two accelerometers in a single ACPI > device. Check for a companion device and handle a second i2c_client > if it is present. + Mika and Wolfram - please cc them on anything odd and i2c / ACPI related.

[PATCH] staging: pi433: rf69.c: Replace macros READ_REG and WRITE_REG with inline functions setBit rstBit and rmwBit

2017-12-02 Thread Marcus Wolf
To increase the readability of the register accesses, the abstraction of the helpers was increased from simple read and write to set bit, reset bit and read modify write bit. In addition - according to the proposal from Walter Harms from 20.07.2017 - instead of marcros inline functions were used.

Re: [PATCH 1/3] iio: trigger: Fix platform_get_irq's error checking

2017-12-02 Thread Jonathan Cameron
On Thu, 30 Nov 2017 21:13:34 +0530 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 Applied to the togreg branch of iio.gi

Re: [PATCH tip/core/rcu 10/10] rcu: Account for rcu_all_qs() in cond_resched()

2017-12-02 Thread Paul E. McKenney
On Sat, Dec 02, 2017 at 09:56:26AM +0100, Peter Zijlstra wrote: > On Fri, Dec 01, 2017 at 11:21:44AM -0800, Paul E. McKenney wrote: > > If cond_resched() returns false, then it has already invoked > > rcu_all_qs(). This is also invoked (now redundantly) by > > rcu_note_voluntary_context_switch().

Re: [PATCH] rbtree.txt: fix typo in cached rbtree section

2017-12-02 Thread Davidlohr Bueso
I'm happy to ack your patch but you have to send it correctly. You are missing a changelog (although it ought to be small due to the trivial change) as well as you SoB tag. Please consult Documentation/process/submitting-patches.rst. You also need to Cc akpm as he routes such areas to Linus. Tha

Re: [PATCH v2 2/2] pinctrl: Allow indicating loss of pin states during low-power

2017-12-02 Thread Linus Walleij
On Wed, Nov 29, 2017 at 6:37 PM, Florian Fainelli wrote: > On 11/29/2017 09:02 AM, Tony Lindgren wrote: >> Hmm well typically a device driver that loses it's context just does >> save and restore of the registers in runtime PM suspend/resume >> as needed. In this case it would mean duplicating th

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

2017-12-02 Thread Sebastian Andrzej Siewior
On 2017-12-01 13:32:22 [-0500], Steven Rostedt wrote: > > 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 b

Wireless regressions in v4.15-rc1

2017-12-02 Thread Kalle Valo
Hi, just a heads up to everyone that there are multiple regressions in v4.15-rc1. For starters hostapd (=AP mode) doesn't work because of: net: netlink: Update attr validation to require exact length for some types https://git.kernel.org/linus/28033ae4e0f5 Jouni fixed this already in hostapd but

Re: [PATCH v2 1/2] iio: hx711: add triggered buffer support

2017-12-02 Thread Jonathan Cameron
On Thu, 30 Nov 2017 22:43:42 +0100 Andreas Klinger wrote: > Add buffer to device data struct and add trigger function > > Data format is quite simple: > voltage - channel 0 32 Bit > voltage - channel 1 32 Bit > timestamp 64 Bit > > Using both channels at the sa

Re: [PATCH v2 2/2] iio: hx711: fix bug in reset functionality

2017-12-02 Thread Jonathan Cameron
On Thu, 30 Nov 2017 22:44:45 +0100 Andreas Klinger wrote: > return value in hx711_reset() should indicate status of dout > otherwise the calling function is reporting an error als false positive > > wait up to 1 second until the device is in normal state > by changing the channel on every trigge

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

2017-12-02 Thread Heiko Carstens
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 to read from kernel memory iff the kern_addr_valid() check pass

Re: [PATCH 1/2] Undo the split of setup_cpu_entry_area

2017-12-02 Thread Andy Lutomirski
> On Dec 2, 2017, at 2:47 AM, Thomas Gleixner wrote: > >> On Sat, 2 Dec 2017, Thomas Gleixner wrote: >> >>> On Fri, 1 Dec 2017, Andy Lutomirski wrote: >>> >>> This is obviously a hack. Either the patch should be adjusted back to >>> the version I sent or trap_init should forcibly initialize

[PATCH 1/4] platform/x86: ideapad-laptop: Remove unnesscary else

2017-12-02 Thread Jiaxun Yang
To deal with checkpatch warning: WARNING: else is not generally useful after a break or return Signed-off-by: Jiaxun Yang --- drivers/platform/x86/ideapad-laptop.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/platform/x86/ideapad-laptop.c b/drivers/pl

[PATCH 2/4] platform/x86: ideapad-laptop: Use __func__ instead of write_ec_cmd in pr_err

2017-12-02 Thread Jiaxun Yang
To deal with checkpatch warning: WARNING: Prefer using '"%s...", __func__' to using 'write_ec_cmd', this function's name, in a string Signed-off-by: Jiaxun Yang --- drivers/platform/x86/ideapad-laptop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/platform/x86/idea

[PATCH 3/4] platform/x86: ideapad-laptop: use kstrto instead of sscanf and do clean up

2017-12-02 Thread Jiaxun Yang
To deal with checkpatch warnings: WARNING: Prefer kstrto to single variable sscanf WARNING: Missing a blank line after declarations WARNING: Block comments use a trailing */ on a separate line Signed-off-by: Jiaxun Yang --- drivers/platform/x86/ideapad-laptop.c | 20 ++-- 1 fil

[PATCH 4/4] platform/x86: ideapad-laptop: add lenovo RESCUER R720-15IKBN to no_hw_rfkill_list

2017-12-02 Thread Jiaxun Yang
Since Roger Jargoyhen reported that this device doesn't have Hardware rfkill switch, this patch add it to no_hw_rfkill_list to prevent radio always be blocked Signed-off-by: Jiaxun Yang --- drivers/platform/x86/ideapad-laptop.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/driver

Re: [PATCH tip/core/rcu 10/10] rcu: Account for rcu_all_qs() in cond_resched()

2017-12-02 Thread Peter Zijlstra
On Sat, Dec 02, 2017 at 04:22:20AM -0800, Paul E. McKenney wrote: > Because there are a few key places within RCU and rcutorture that need it. > Without it, there are scenarios where the new cond_resched() never gets > activated, and thus doesn't take effect. Ah, I missed that interaction. > The

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

2017-12-02 Thread Peter Zijlstra
On Sat, Dec 02, 2017 at 01:53:31PM +0100, Sebastian Andrzej Siewior wrote: > On 2017-12-01 13:32:22 [-0500], Steven Rostedt wrote: > > > > 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_sing

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

2017-12-02 Thread Frederic Weisbecker
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 with CONFIG_NO_HZ_FULL_ALL=y > and CONFIG_CPU_ISOLATION=n. These warnings ar

Re: ath9k: dfs: use swap macro in ath9k_check_chirping

2017-12-02 Thread Kalle Valo
"Gustavo A. R. Silva" wrote: > Make use of the swap macro and remove unnecessary variable temp. > This makes the code easier to read and maintain. > > This code was detected with the help of Coccinelle. > > Signed-off-by: Gustavo A. R. Silva > Signed-off-by: Kalle Valo Patch applied to ath-n

Re: [PATCH 1/2] Undo the split of setup_cpu_entry_area

2017-12-02 Thread Thomas Gleixner
On Sat, 2 Dec 2017, Andy Lutomirski wrote: > > > > On Dec 2, 2017, at 2:47 AM, Thomas Gleixner wrote: > > > >> On Sat, 2 Dec 2017, Thomas Gleixner wrote: > >> > >>> On Fri, 1 Dec 2017, Andy Lutomirski wrote: > >>> > >>> This is obviously a hack. Either the patch should be adjusted back to >

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

2017-12-02 Thread Borislav Petkov
On Fri, Dec 01, 2017 at 09:50:33AM -0600, Yazen Ghannam wrote: > From: Yazen Ghannam > > Scalable MCA systems have various types of banks. The bank's type can > determine how we handle errors from it. For example, if a bank represents > a UMC then we will need to convert its address from a normal

Re: [PATCH 1/1] at24: Fix I²C device selection for runtime PM

2017-12-02 Thread Sakari Ailus
On Fri, Dec 01, 2017 at 05:29:27PM +0100, Bartosz Golaszewski wrote: > 2017-12-01 16:35 GMT+01:00 Sakari Ailus : > > Hi Sven, > > > > On Fri, Dec 01, 2017 at 10:20:41AM -0500, Sven Van Asbroeck wrote: > >> Thank you, it fixes the issue on the multi-address eeprom that I have > >> access to. > >> >

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

2017-12-02 Thread Philippe Mikoyan
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 mechanisms. Thank you for noting the typo, 'll se

Re: [PATCH v2] eeprom: at24: fix I2C device selection for runtime PM

2017-12-02 Thread Sakari Ailus
Hi, On Fri, Dec 01, 2017 at 01:37:12PM -0500, Sven Van Asbroeck wrote: > From: Sakari Ailus > > The at24 driver creates dummy I2C devices to access offsets in the chip > that are outside the area supported using a single I2C address. It is not > meaningful to use runtime PM to such devices; the

Re: [EXT] Re: [PATCH net] net: phylink: fix link state on phy-connect

2017-12-02 Thread Russell King - ARM Linux
On Sat, Dec 02, 2017 at 11:08:45AM +, Yan Markman wrote: > Hi Russel > > The Grygorii has raised one Additional point (about netif_carrier_off) > I just didn't want to start before finishing the previous one. > > On ifconfig-down the mac_config() called but with LINK=0. > The config has no any

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

2017-12-02 Thread Matthew Wilcox
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 discourage the use of this option, but I'm going to include som

[PATCH] staging: pi433: Removed some obsolete or duplicated defines; moved two defines to better locations

2017-12-02 Thread Marcus Wolf
The define FIFO_SIZE was moved to rf69_registers.h. Although it is not a register, it is a value, that is given by hardware (like the registers). The define FIFO_THRESHOLD was moved to pi433_if.c, since it is a value, that is freely choosen by the interface implementation. The better the response

Re: [PATCH 4/6] Unsuck "x86/entry/64: Create a percpu SYSCALL entry trampoline"

2017-12-02 Thread Josh Poimboeuf
On Thu, Nov 30, 2017 at 10:29:44PM -0800, Andy Lutomirski wrote: > diff --git a/arch/x86/entry/entry_64.S b/arch/x86/entry/entry_64.S > index caf74a1bb3de..28f4e7553c26 100644 > --- a/arch/x86/entry/entry_64.S > +++ b/arch/x86/entry/entry_64.S > @@ -180,14 +180,24 @@ ENTRY(entry_SYSCALL_64_trampoli

Re: [PATCH v4 1/2] gpio: gpiolib: Generalise state persistence beyond sleep

2017-12-02 Thread Linus Walleij
On Thu, Nov 30, 2017 at 4:55 AM, Andrew Jeffery wrote: > General support for state persistence is added to gpiolib with the > introduction of a new pinconf parameter to propagate the request to > hardware. The existing persistence support for sleep is adapted to > include hardware support if the

[PATCH] staging: pi433: Make rf69_set_dc_cut_off_frequency_intern static

2017-12-02 Thread Marcus Wolf
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 +- drivers/staging/pi433/rf69.h |1 - 2 files changed, 1 insertion(

Re: [PATCH v4 2/2] gpio: aspeed: Add support for reset tolerance

2017-12-02 Thread Linus Walleij
On Thu, Nov 30, 2017 at 4:55 AM, Andrew Jeffery wrote: > Use the new pinconf parameter for state persistence to expose the > associated capability of the Aspeed GPIO controller. > > Signed-off-by: Andrew Jeffery > Reviewed-by: Joel Stanley Patch applied. Yours, Linus Walleij

Re: [PATCH v2] pinctrl: stm32: Fix copyright

2017-12-02 Thread Linus Walleij
On Thu, Nov 30, 2017 at 9:46 AM, Benjamin Gaignard wrote: > Uniformize STMicroelectronics copyrights header > Add SPDX identifier > > Signed-off-by: Benjamin Gaignard > Acked-by: Alexandre TORGUE > CC: Maxime Coquelin Pulled out the v1 I applied and applied this instead. Yours, Linus Walleij

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

2017-12-02 Thread Linus Walleij
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 checking > for zero is not correct. > > Signed-off-by: Arvind Yadav (...)

Re: [PATCH] dt-bindings: pinctrl: uniphier: add UniPhier pinctrl binding

2017-12-02 Thread Linus Walleij
On Tue, Nov 28, 2017 at 8:49 AM, Masahiro Yamada wrote: > The driver has been in the tree for a while, but its binding document > is missing. Hence, here it is. > > Signed-off-by: Masahiro Yamada Patch applied with Rob's ACK. Yours, Linus Walleij

Re: [PATCH] iio: dummy_evgen: modify the return value check for irq_sim_init()

2017-12-02 Thread Bartosz Golaszewski
2017-12-02 12:43 GMT+01:00 Jonathan Cameron : > On Mon, 27 Nov 2017 11:54:49 +0100 > Bartosz Golaszewski wrote: > >> As discussed with Marc Zyngier: irq_sim_init() and its devres variant >> should return the base of the allocated interrupt range on success >> rather than 0. This will be modified l

[PATCH v2] ipc: Fix ipc data structures inconsistency

2017-12-02 Thread Philippe Mikoyan
As described in the title, this patch fixes id_ds inconsistency when ctl_stat executes concurrently with some ds-changing function, e.g. shmat, msgsnd or whatever. For instance, if shmctl(IPC_STAT) is running concurrently with shmat, following data structure can be returned: {... shm_lpid = 0, shm

Re: [PATCH v4 10/10] ARM: dtsi: axp81x: set pinmux for GPIO0/1 when used as LDOs

2017-12-02 Thread Maxime Ripard
On Fri, Dec 01, 2017 at 02:44:51PM +0100, Quentin Schulz wrote: > On AXP813/818, GPIO0 and GPIO1 can be used as LDO as (respectively) > ldo_io0 and ldo_io1. > > Let's add the pinctrl properties to the said regulators. > > Signed-off-by: Quentin Schulz Acked-by: Maxime Ripard Maxime -- Maxim

Re: [PATCH v4 06/10] pinctrl: axp209: add programmable ADC muxing value

2017-12-02 Thread Maxime Ripard
On Fri, Dec 01, 2017 at 02:44:47PM +0100, Quentin Schulz wrote: > To prepare for patches that will add support for a new PMIC that has a > different GPIO adc muxing value, add an adc_mux within axp20x_pctl > structure and use it. > > Signed-off-by: Quentin Schulz Acked-by: Maxime Ripard Maxime

Re: [PATCH 1/2] arm64: dts: a64-olinuxino: Enable RTL8723BS WiFi

2017-12-02 Thread Maxime Ripard
On Fri, Dec 01, 2017 at 02:36:04PM +0530, Jagan Teki wrote: > Enable RTL8723BS WiFi chip on a64-olinuxino board: > - WiFi SDIO interface is connected to MMC1 > - WiFi REG_ON pin connected to gpio PL2: attach to mmc-pwrseq > - WiFi HOST_WAKE pin connected to gpio PL3 > > To make rtl8723bs chip to w

Re: [PATCH v4 02/10] pinctrl: axp209: add pinctrl features

2017-12-02 Thread Maxime Ripard
On Fri, Dec 01, 2017 at 02:44:43PM +0100, Quentin Schulz wrote: > +static void axp20x_gpio_set(struct gpio_chip *chip, unsigned offset, > + int value) > +{ checkpatch output: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' > +static int axp20x_pmx_set_mux(struct p

Re: [PATCH 2/2] arm64: allwinner: a64-sopine: Use dcdc1 regulator instead of vcc3v3

2017-12-02 Thread Maxime Ripard
Hi On Fri, Dec 01, 2017 at 02:36:05PM +0530, Jagan Teki wrote: > Since current tree support AXP803 regulators, > replace fixed regulator vcc3v3 with AXP803 dcdc1 regulator. > > Tested mmc0 on sopine baseboard. > > Signed-off-by: Jagan Teki Is it supposed to be a fix? Maxime -- Maxime Ripard

Re: [PATCH] dmaengine: doc : Fix warning "Title underline too short" while make xmldocs

2017-12-02 Thread Jonathan Corbet
On Wed, 29 Nov 2017 00:16:56 +0900 Masanari Iida wrote: > This patch fix following warning during 'make xmldocs' > > Documentation/driver-api/dmaengine/client.rst:188: > WARNING: Title underline too short. > > Further APIs: > Applied to the docs tree, thanks. jon

Re: [PATCH 4/5] staging: pi433: rf69.c style fix - code indent should use tabs

2017-12-02 Thread Marcus Wolf
Revieved-by: Marcus Wolf Am 11.10.2017 um 21:24 schrieb Marcin Ciupak: This patch fixes the following checkpatch.pl error: ERROR: code indent should use tabs where possible in rf69.c file as requested by TODO file. Additionally some style warnings remain valid here and could be fixed by anot

Re: [PATCH] kernel-docs.rst: update broken linuxjournal links

2017-12-02 Thread Jonathan Corbet
On Thu, 30 Nov 2017 17:47:25 +0700 Nguyễn Thái Ngọc Duy wrote: > Signed-off-by: Nguyễn Thái Ngọc Duy > --- > Documentation/process/kernel-docs.rst | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) So I'm reluctant to take patches without a changelog. One would hel

Re: [PATCH] genericirq.rst: Remove :c:func:`...` in code blocks

2017-12-02 Thread Jonathan Corbet
On Tue, 28 Nov 2017 06:16:21 +0100 Jonathan Neuschäfer wrote: > In code blocks, :c:func:`...` annotations don't result in > cross-references. Instead, they are rendered verbatim. Remove these > broken annotations, and mark function calls with parentheses() again. > > Fixes: 76d40fae1351 ("gener

[PATCH] staging: pi433: Style fix - spaces required

2017-12-02 Thread Marcus Wolf
Leftover changes from patch [PATCH 2/5] staging: pi433: rf69.c style fix - spaces required around from Marcin Ciupak (11.10.2017) Signed-of-by: Marcus Wolf --- drivers/staging/pi433/rf69.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433/rf69.c b/driv

Re: [PATCH] staging: pi433: Bugfix for shift operation in rf69_get_modulation()

2017-12-02 Thread Greg KH
On Sat, Dec 02, 2017 at 12:32:20PM +0200, Marcus Wolf wrote: > Defines used in cases contain already shifted bits, so currentValue must not > be shifted. Please properly wrap your changelog text. And you need a blank line before your signed-off-by line. > signed_of_by: Marcus Wolf "Signed-off

Re: [PATCH] staging: pi433: Bugfix for shift operation in rf69_get_modulation()

2017-12-02 Thread Greg KH
On Sat, Dec 02, 2017 at 12:51:54PM +0200, Marcus Wolf wrote: > Defines used in cases contain already shifted bits, so currentValue must not > be shifted. > Signed-off-by: Marcus Wolf Odd extra ' ' in your subject :( And again, linewrap please. thanks, greg k-h

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

2017-12-02 Thread 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 is > possible to switch of the funct

Re: [PATCH] staging: pi433: rf69.c: Replace macros READ_REG and WRITE_REG with inline functions setBit rstBit and rmwBit

2017-12-02 Thread Greg KH
On Sat, Dec 02, 2017 at 02:21:32PM +0200, Marcus Wolf wrote: > To increase the readability of the register accesses, the abstraction of the > helpers was increased from simple read and write to set bit, reset bit and > read modify write bit. In addition - according to the proposal from Walter > Har

GREETINGS BELOVED

2017-12-02 Thread MISS BORTE OGOTAI
-- GREETINGS BELOVED I AM BORTE ,I WAS DIAGNOSE WITH OVARIAN CANCER,WHICH DOCTORS HAVE CONFIRMED THAT I HAVE ONLY FEW WEEKS TO LIVE, SO I HAVE DECIDED TO DONATE EVERYTHING I HAVE TO THE ORPHANAGE AND THE POOR WIDOWS THROUGH YOU AND YOUR HELP .PLEASE KINDLY REPLY ME ONLY ON MY EMAIL ADDRES

[PATCH AUTOSEL for 4.14 03/64] net/smc: use sk_rcvbuf as start for rmb creation

2017-12-02 Thread alexander . levin
From: Ursula Braun [ Upstream commit 4e1061f4a2bba1669c7297455c73ddafbebf2b12 ] Commit 3e034725c0d8 ("net/smc: common functions for RMBs and send buffers") merged handling of SMC receive and send buffers. It introduced sk_buf_size as merged start value for size determination. But since sk_buf_si

[PATCH AUTOSEL for 4.14 20/64] dynamic-debug-howto: fix optional/omitted ending line number to be LARGE instead of 0

2017-12-02 Thread alexander . levin
From: Randy Dunlap [ Upstream commit 1f3c790bd5989fcfec9e53ad8fa09f5b740c958f ] line-range is supposed to treat "1-" as "1-endoffile", so handle the special case by setting last_lineno to UINT_MAX. Fixes this error: dynamic_debug:ddebug_parse_query: last-line:0 < 1st-line:1 dynamic_debug:d

[PATCH AUTOSEL for 4.14 35/64] bpf: fix lockdep splat

2017-12-02 Thread alexander . levin
From: Eric Dumazet [ Upstream commit 89ad2fa3f043a1e8daae193bcb5fe34d5f8caf28 ] pcpu_freelist_pop() needs the same lockdep awareness than pcpu_freelist_populate() to avoid a false positive. [ INFO: SOFTIRQ-safe -> SOFTIRQ-unsafe lock order detected ] switchto-defaul/12508 [HC0[0]:SC0[6]:HE0:

[PATCH AUTOSEL for 4.14 44/64] atm: horizon: Fix irq release error

2017-12-02 Thread alexander . levin
From: Arvind Yadav [ Upstream commit bde533f2ea607cbbbe76ef8738b36243939a7bc2 ] atm_dev_register() can fail here and passed parameters to free irq which is not initialised. Initialization of 'dev->irq' happened after the 'goto out_free_irq'. So using 'irq' insted of 'dev->irq' in free_irq(). Si

[PATCH AUTOSEL for 4.14 51/64] IB/mlx5: Assign send CQ and recv CQ of UMR QP

2017-12-02 Thread alexander . levin
From: Majd Dibbiny [ Upstream commit 31fde034a8bd964a5c7c1a5663fc87a913158db2 ] The UMR's QP is created by calling mlx5_ib_create_qp directly, and therefore the send CQ and the recv CQ on the ibqp weren't assigned. Assign them right after calling the mlx5_ib_create_qp to assure that any access

Re: [PATCH v6 2/2] staging: ion: create one device entry per heap

2017-12-02 Thread Greg KH
On Wed, Nov 29, 2017 at 03:00:56PM +0100, Benjamin Gaignard wrote: > 2017-11-28 14:32 GMT+01:00 Greg KH : > > On Mon, Nov 06, 2017 at 04:59:45PM +0100, Benjamin Gaignard wrote: > >> Instead a getting only one common device "/dev/ion" for > >> all the heaps this patch allow to create one device > >>

[PATCH AUTOSEL for 4.14 53/64] afs: Connect up the CB.ProbeUuid

2017-12-02 Thread alexander . levin
From: David Howells [ Upstream commit f4b3526d83c40dd8bf5948b9d7a1b2c340f0dcc8 ] The handler for the CB.ProbeUuid operation in the cache manager is implemented, but isn't listed in the switch-statement of operation selection, so won't be used. Fix this by adding it. Signed-off-by: David Howell

[PATCH AUTOSEL for 4.9 21/33] bpf: fix lockdep splat

2017-12-02 Thread alexander . levin
From: Eric Dumazet [ Upstream commit 89ad2fa3f043a1e8daae193bcb5fe34d5f8caf28 ] pcpu_freelist_pop() needs the same lockdep awareness than pcpu_freelist_populate() to avoid a false positive. [ INFO: SOFTIRQ-safe -> SOFTIRQ-unsafe lock order detected ] switchto-defaul/12508 [HC0[0]:SC0[6]:HE0:

[PATCH AUTOSEL for 4.9 33/33] RDMA/cxgb4: Annotate r2 and stag as __be32

2017-12-02 Thread alexander . levin
From: Leon Romanovsky [ Upstream commit 7d7d065a5eec7e218174d5c64a9f53f99ffdb119 ] Chelsio cxgb4 HW is big-endian, hence there is need to properly annotate r2 and stag fields as __be32 and not __u32 to fix the following sparse warnings. drivers/infiniband/hw/cxgb4/qp.c:614:16: warning: in

[PATCH AUTOSEL for 4.14 64/64] RDMA/cxgb4: Annotate r2 and stag as __be32

2017-12-02 Thread alexander . levin
From: Leon Romanovsky [ Upstream commit 7d7d065a5eec7e218174d5c64a9f53f99ffdb119 ] Chelsio cxgb4 HW is big-endian, hence there is need to properly annotate r2 and stag fields as __be32 and not __u32 to fix the following sparse warnings. drivers/infiniband/hw/cxgb4/qp.c:614:16: warning: in

[PATCH AUTOSEL for 3.18 12/15] IB/mlx4: Increase maximal message size under UD QP

2017-12-02 Thread alexander . levin
From: Mark Bloch [ Upstream commit 5f22a1d87c5315a98981ecf93cd8de226cffe6ca ] Maximal message should be used as a limit to the max message payload allowed, without the headers. The ConnectX-3 check is done against this value includes the headers. When the payload is 4K this will cause the NIC to

[PATCH AUTOSEL for 4.9 30/33] ipvlan: fix ipv6 outbound device

2017-12-02 Thread alexander . levin
From: Keefe Liu [ Upstream commit ca29fd7cce5a6444d57fb86517589a1a31c759e1 ] When process the outbound packet of ipv6, we should assign the master device to output device other than input device. Signed-off-by: Keefe Liu Acked-by: Mahesh Bandewar Signed-off-by: David S. Miller Signed-off-by:

[PATCH AUTOSEL for 4.9 27/33] IB/mlx5: Assign send CQ and recv CQ of UMR QP

2017-12-02 Thread alexander . levin
From: Majd Dibbiny [ Upstream commit 31fde034a8bd964a5c7c1a5663fc87a913158db2 ] The UMR's QP is created by calling mlx5_ib_create_qp directly, and therefore the send CQ and the recv CQ on the ibqp weren't assigned. Assign them right after calling the mlx5_ib_create_qp to assure that any access

[PATCH AUTOSEL for 3.18 10/15] atm: horizon: Fix irq release error

2017-12-02 Thread alexander . levin
From: Arvind Yadav [ Upstream commit bde533f2ea607cbbbe76ef8738b36243939a7bc2 ] atm_dev_register() can fail here and passed parameters to free irq which is not initialised. Initialization of 'dev->irq' happened after the 'goto out_free_irq'. So using 'irq' insted of 'dev->irq' in free_irq(). Si

  1   2   3   4   >