[PATCH 4.9 57/65] net: mpls: Pull common label check into helper

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: David Ahern commit b7b386f42f079b25b942c756820e36c6bd09b2ca upstream. mpls_route_add and mpls_route_del have the same checks on the label. Move to a helper. Avoid duplicate

[PATCH 4.9 57/65] net: mpls: Pull common label check into helper

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: David Ahern commit b7b386f42f079b25b942c756820e36c6bd09b2ca upstream. mpls_route_add and mpls_route_del have the same checks on the label. Move to a helper. Avoid duplicate extack messages in

[PATCH 4.9 40/65] ppp: prevent unregistered channels from connecting to PPP units

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Guillaume Nault [ Upstream commit 77f840e3e5f09c6d7d727e85e6e08276dd813d11 ] PPP units don't hold any reference on the channels connected to it. It is the channel's

[PATCH 4.9 38/65] net: ipv4: dont allow setting net.ipv4.route.min_pmtu below 68

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Sabrina Dubroca [ Upstream commit c7272c2f1229125f74f22dcdd59de9bbd804f1c8 ] According to RFC 1191 sections 3 and 4, ICMP frag-needed messages indicating an MTU below 68

[PATCH 4.9 35/65] hdlc_ppp: carrier detect ok, dont turn off negotiation

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Denis Du [ Upstream commit b6c3bad1ba83af1062a7ff6986d9edc4f3d7fc8e ] Sometimes when physical lines have a just good noise to make the protocol handshaking fail, but the

[PATCH 4.9 51/65] s390/qeth: fix overestimated count of buffer elements

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Julian Wiedmann [ Upstream commit 12472af89632beb1ed8dea29d4efe208ca05b06a ] qeth_get_elements_for_range() doesn't know how to handle a 0-length range (ie. start ==

[PATCH 4.9 35/65] hdlc_ppp: carrier detect ok, dont turn off negotiation

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Denis Du [ Upstream commit b6c3bad1ba83af1062a7ff6986d9edc4f3d7fc8e ] Sometimes when physical lines have a just good noise to make the protocol handshaking fail, but the carrier detect still

[PATCH 4.9 51/65] s390/qeth: fix overestimated count of buffer elements

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Julian Wiedmann [ Upstream commit 12472af89632beb1ed8dea29d4efe208ca05b06a ] qeth_get_elements_for_range() doesn't know how to handle a 0-length range (ie. start == end), and returns 1 when

[PATCH 4.9 40/65] ppp: prevent unregistered channels from connecting to PPP units

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Guillaume Nault [ Upstream commit 77f840e3e5f09c6d7d727e85e6e08276dd813d11 ] PPP units don't hold any reference on the channels connected to it. It is the channel's responsibility to ensure

[PATCH 4.9 38/65] net: ipv4: dont allow setting net.ipv4.route.min_pmtu below 68

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Sabrina Dubroca [ Upstream commit c7272c2f1229125f74f22dcdd59de9bbd804f1c8 ] According to RFC 1191 sections 3 and 4, ICMP frag-needed messages indicating an MTU below 68 should be rejected:

[PATCH 4.9 43/65] mlxsw: spectrum_switchdev: Check success of FDB add operation

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Shalom Toledo [ Upstream commit 0a8a1bf17e3af34f1f8d2368916a6327f8b3bfd5 ] Until now, we assumed that in case of error when adding FDB entries, the write operation will

[PATCH 4.9 46/65] rxrpc: Fix send in rxrpc_send_data_packet()

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: David Howells [ Upstream commit 93c62c45ed5fad1b87e3a45835b251cd68de9c46 ] All the kernel_sendmsg() calls in rxrpc_send_data_packet() need to send both parts of the iov[]

[PATCH 4.9 43/65] mlxsw: spectrum_switchdev: Check success of FDB add operation

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Shalom Toledo [ Upstream commit 0a8a1bf17e3af34f1f8d2368916a6327f8b3bfd5 ] Until now, we assumed that in case of error when adding FDB entries, the write operation will fail, but this is not

[PATCH 4.9 46/65] rxrpc: Fix send in rxrpc_send_data_packet()

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: David Howells [ Upstream commit 93c62c45ed5fad1b87e3a45835b251cd68de9c46 ] All the kernel_sendmsg() calls in rxrpc_send_data_packet() need to send both parts of the iov[] buffer, but one of

[PATCH 4.9 07/65] tpm-dev-common: Reject too short writes

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Alexander Steffen commit ee70bc1e7b63ac8023c9ff9475d8741e397316e7 upstream. tpm_transmit() does not offer an explicit interface to indicate the number of valid

[PATCH 4.9 03/65] tpm_i2c_nuvoton: fix potential buffer overruns caused by bit glitches on the bus

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Jeremy Boone commit f9d4d9b5a5ef2f017bc344fb65a58a902517173b upstream. Discrete TPMs are often connected over slow serial buses which, on some platforms, can have

[PATCH 4.9 07/65] tpm-dev-common: Reject too short writes

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Alexander Steffen commit ee70bc1e7b63ac8023c9ff9475d8741e397316e7 upstream. tpm_transmit() does not offer an explicit interface to indicate the number of valid bytes in the communication

[PATCH 4.9 03/65] tpm_i2c_nuvoton: fix potential buffer overruns caused by bit glitches on the bus

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Jeremy Boone commit f9d4d9b5a5ef2f017bc344fb65a58a902517173b upstream. Discrete TPMs are often connected over slow serial buses which, on some platforms, can have glitches causing bit flips.

[PATCH 4.9 17/65] media: m88ds3103: dont call a non-initalized function

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Mauro Carvalho Chehab commit b9c97c67fd19262c002d94ced2bfb513083e161e upstream. If m88d3103 chip ID is not recognized, the device is not initialized. However, it

[PATCH 4.9 17/65] media: m88ds3103: dont call a non-initalized function

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Mauro Carvalho Chehab commit b9c97c67fd19262c002d94ced2bfb513083e161e upstream. If m88d3103 chip ID is not recognized, the device is not initialized. However, it returns from probe without

[PATCH 4.9 00/65] 4.9.87-stable review

2018-03-09 Thread Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.9.87 release. There are 65 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made by Mon Mar 12 00:18:06 UTC 2018. Anything

[PATCH 4.9 00/65] 4.9.87-stable review

2018-03-09 Thread Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.9.87 release. There are 65 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made by Mon Mar 12 00:18:06 UTC 2018. Anything

[PATCH 4.9 13/65] cpufreq: s3c24xx: Fix broken s3c_cpufreq_init()

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Viresh Kumar commit 0373ca74831b0f93cd4cdbf7ad3aec3c33a479a5 upstream. commit a307a1e6bc0d "cpufreq: s3c: use cpufreq_generic_init()" accidentally broke cpufreq on

[PATCH 4.9 13/65] cpufreq: s3c24xx: Fix broken s3c_cpufreq_init()

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Viresh Kumar commit 0373ca74831b0f93cd4cdbf7ad3aec3c33a479a5 upstream. commit a307a1e6bc0d "cpufreq: s3c: use cpufreq_generic_init()" accidentally broke cpufreq on s3c2410 and s3c2412. These

[PATCH 4.9 11/65] timers: Forward timer base before migrating timers

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Lingutla Chandrasekhar commit c52232a49e203a65a6e1a670cd5262f59e9364a0 upstream. On CPU hotunplug the enqueued timers of the unplugged CPU are migrated to a live CPU.

[PATCH 4.9 19/65] ARM: mvebu: Fix broken PL310_ERRATA_753970 selects

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Ulf Magnusson commit 8aa36a8dcde3183d84db7b0d622ffddcebb61077 upstream. The MACH_ARMADA_375 and MACH_ARMADA_38X boards select ARM_ERRATA_753970, but it was renamed to

[PATCH 4.9 19/65] ARM: mvebu: Fix broken PL310_ERRATA_753970 selects

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Ulf Magnusson commit 8aa36a8dcde3183d84db7b0d622ffddcebb61077 upstream. The MACH_ARMADA_375 and MACH_ARMADA_38X boards select ARM_ERRATA_753970, but it was renamed to PL310_ERRATA_753970 by

[PATCH 4.9 11/65] timers: Forward timer base before migrating timers

2018-03-09 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Lingutla Chandrasekhar commit c52232a49e203a65a6e1a670cd5262f59e9364a0 upstream. On CPU hotunplug the enqueued timers of the unplugged CPU are migrated to a live CPU. This happens from the

[PATCH 4.4 27/36] sctp: fix dst refcnt leak in sctp_v4_get_dst

2018-03-09 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Tommi Rantala [ Upstream commit 4a31a6b19f9ddf498c81f5c9b089742b7472a6f8 ] Fix dst reference count leak in sctp_v4_get_dst() introduced in commit 410f03831 ("sctp:

[PATCH 4.4 27/36] sctp: fix dst refcnt leak in sctp_v4_get_dst

2018-03-09 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Tommi Rantala [ Upstream commit 4a31a6b19f9ddf498c81f5c9b089742b7472a6f8 ] Fix dst reference count leak in sctp_v4_get_dst() introduced in commit 410f03831 ("sctp: add routing output

[PATCH 4.4 25/36] ppp: prevent unregistered channels from connecting to PPP units

2018-03-09 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Guillaume Nault [ Upstream commit 77f840e3e5f09c6d7d727e85e6e08276dd813d11 ] PPP units don't hold any reference on the channels connected to it. It is the channel's

[PATCH 4.4 25/36] ppp: prevent unregistered channels from connecting to PPP units

2018-03-09 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Guillaume Nault [ Upstream commit 77f840e3e5f09c6d7d727e85e6e08276dd813d11 ] PPP units don't hold any reference on the channels connected to it. It is the channel's responsibility to ensure

[PATCH 4.4 15/36] leds: do not overflow sysfs buffer in led_trigger_show

2018-03-09 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Nathan Sullivan commit 3b9b95363c45365d606ad4bbba16acca75fdf6d3 upstream. Per the documentation, use scnprintf instead of sprintf to ensure there is never more than

[PATCH 4.4 15/36] leds: do not overflow sysfs buffer in led_trigger_show

2018-03-09 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Nathan Sullivan commit 3b9b95363c45365d606ad4bbba16acca75fdf6d3 upstream. Per the documentation, use scnprintf instead of sprintf to ensure there is never more than PAGE_SIZE bytes of trigger

[PATCH 3.18 06/21] net: fec: introduce fec_ptp_stop and use in probe fail path

2018-03-09 Thread Greg Kroah-Hartman
3.18-stable review patch. If anyone has any objections, please let me know. -- From: Lucas Stach commit 32cba57ba74be58589aeb4cb6496183e46a5e3e5 upstream. This function frees resources and cancels delayed work item that have been initialized in

[PATCH 3.18 06/21] net: fec: introduce fec_ptp_stop and use in probe fail path

2018-03-09 Thread Greg Kroah-Hartman
3.18-stable review patch. If anyone has any objections, please let me know. -- From: Lucas Stach commit 32cba57ba74be58589aeb4cb6496183e46a5e3e5 upstream. This function frees resources and cancels delayed work item that have been initialized in fec_ptp_init(). Use this to do

[PATCH 3.18 03/21] ALSA: usb-audio: Add a quirck for B PX headphones

2018-03-09 Thread Greg Kroah-Hartman
3.18-stable review patch. If anyone has any objections, please let me know. -- From: Erik Veijola commit 240a8af929c7c57dcde28682725b29cf8474e8e5 upstream. The capture interface doesn't work and the playback interface only supports 48 kHz sampling rate

[PATCH 4.4 13/36] ARM: dts: LogicPD Torpedo: Fix I2C1 pinmux

2018-03-09 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Adam Ford commit 74402055a2d3ec998a1ded599e86185a27d9bbf4 upstream. The pinmuxing was missing for I2C1 which was causing intermittent issues with the PMIC which is

[PATCH 3.18 03/21] ALSA: usb-audio: Add a quirck for B PX headphones

2018-03-09 Thread Greg Kroah-Hartman
3.18-stable review patch. If anyone has any objections, please let me know. -- From: Erik Veijola commit 240a8af929c7c57dcde28682725b29cf8474e8e5 upstream. The capture interface doesn't work and the playback interface only supports 48 kHz sampling rate even though it

[PATCH 4.4 13/36] ARM: dts: LogicPD Torpedo: Fix I2C1 pinmux

2018-03-09 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Adam Ford commit 74402055a2d3ec998a1ded599e86185a27d9bbf4 upstream. The pinmuxing was missing for I2C1 which was causing intermittent issues with the PMIC which is connected to I2C1. The

[PATCH 3.18 00/21] 3.18.99-stable review

2018-03-09 Thread Greg Kroah-Hartman
This is the start of the stable review cycle for the 3.18.99 release. There are 21 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made by Mon Mar 12 00:17:44 UTC 2018. Anything

[PATCH 3.18 00/21] 3.18.99-stable review

2018-03-09 Thread Greg Kroah-Hartman
This is the start of the stable review cycle for the 3.18.99 release. There are 21 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made by Mon Mar 12 00:17:44 UTC 2018. Anything

[PATCH] proc: reject "." and ".." as filenames

2018-03-09 Thread Alexey Dobriyan
Various subsystems can create files and directories in /proc with names directly controlled by userspace. Which means "/", "." and ".." are no-no. "/" split is already taken care of, do the other 2 prohibited names. Signed-off-by: Alexey Dobriyan --- fs/proc/generic.c |

[PATCH] proc: reject "." and ".." as filenames

2018-03-09 Thread Alexey Dobriyan
Various subsystems can create files and directories in /proc with names directly controlled by userspace. Which means "/", "." and ".." are no-no. "/" split is already taken care of, do the other 2 prohibited names. Signed-off-by: Alexey Dobriyan --- fs/proc/generic.c |8 1 file

[PATCH 4/4] gpio: Remove VLA from stmpe driver

2018-03-09 Thread Laura Abbott
The new challenge is to remove VLAs from the kernel (see https://lkml.org/lkml/2018/3/7/621) This patch replaces a VLA with an appropriate call to kmalloc_array. Signed-off-by: Laura Abbott --- drivers/gpio/gpio-stmpe.c | 7 ++- 1 file changed, 6 insertions(+), 1

[PATCH 0/4] VLA removal from the GPIO subsystem

2018-03-09 Thread Laura Abbott
Hi, For those who haven't seen it, there's an effort to remove VLAs from the kernel so we can turn on -Wvla in the name of security. See https://lkml.org/lkml/2018/3/7/621 for more details and discussion. This is a series to remove a few VLAs from the gpio subsystem. These are compile tested

[PATCH 4/4] gpio: Remove VLA from stmpe driver

2018-03-09 Thread Laura Abbott
The new challenge is to remove VLAs from the kernel (see https://lkml.org/lkml/2018/3/7/621) This patch replaces a VLA with an appropriate call to kmalloc_array. Signed-off-by: Laura Abbott --- drivers/gpio/gpio-stmpe.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git

[PATCH 0/4] VLA removal from the GPIO subsystem

2018-03-09 Thread Laura Abbott
Hi, For those who haven't seen it, there's an effort to remove VLAs from the kernel so we can turn on -Wvla in the name of security. See https://lkml.org/lkml/2018/3/7/621 for more details and discussion. This is a series to remove a few VLAs from the gpio subsystem. These are compile tested

[PATCH 2/4] gpio: Remove VLA from MAX3191X driver

2018-03-09 Thread Laura Abbott
The new challenge is to remove VLAs from the kernel (see https://lkml.org/lkml/2018/3/7/621) This patch replaces several a VLA with an appropriate call to kmalloc_array. Signed-off-by: Laura Abbott --- drivers/gpio/gpio-max3191x.c | 7 ++- 1 file changed, 6

[PATCH 2/4] gpio: Remove VLA from MAX3191X driver

2018-03-09 Thread Laura Abbott
The new challenge is to remove VLAs from the kernel (see https://lkml.org/lkml/2018/3/7/621) This patch replaces several a VLA with an appropriate call to kmalloc_array. Signed-off-by: Laura Abbott --- drivers/gpio/gpio-max3191x.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-)

[PATCH 3/4] gpio: Remove VLA from xra1403 driver

2018-03-09 Thread Laura Abbott
The new challenge is to remove VLAs from the kernel (see https://lkml.org/lkml/2018/3/7/621) This patch replaces a VLA with an appropriate call to kmalloc_array. Signed-off-by: Laura Abbott --- drivers/gpio/gpio-xra1403.c | 8 +++- 1 file changed, 7 insertions(+), 1

[PATCH 3/4] gpio: Remove VLA from xra1403 driver

2018-03-09 Thread Laura Abbott
The new challenge is to remove VLAs from the kernel (see https://lkml.org/lkml/2018/3/7/621) This patch replaces a VLA with an appropriate call to kmalloc_array. Signed-off-by: Laura Abbott --- drivers/gpio/gpio-xra1403.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff

Re: [PATCH 4.14 1/4] powerpc/mm/slice: Remove intermediate bitmap copy

2018-03-09 Thread Greg Kroah-Hartman
On Fri, Mar 09, 2018 at 04:48:59PM +0100, Christophe Leroy wrote: > Upstream 326691ad4f179e6edc7eb1271e618dd673e4736d There is no such git commit id in Linus's tree :( Please fix up and resend the series. thanks, greg k-h

[PATCH 1/4] gpio: Remove VLA from gpiolib

2018-03-09 Thread Laura Abbott
The new challenge is to remove VLAs from the kernel (see https://lkml.org/lkml/2018/3/7/621) This patch replaces several VLAs with an appropriate call to kmalloc_array. Signed-off-by: Laura Abbott --- drivers/gpio/gpiolib.c | 55

Re: [PATCH 4.14 1/4] powerpc/mm/slice: Remove intermediate bitmap copy

2018-03-09 Thread Greg Kroah-Hartman
On Fri, Mar 09, 2018 at 04:48:59PM +0100, Christophe Leroy wrote: > Upstream 326691ad4f179e6edc7eb1271e618dd673e4736d There is no such git commit id in Linus's tree :( Please fix up and resend the series. thanks, greg k-h

[PATCH 1/4] gpio: Remove VLA from gpiolib

2018-03-09 Thread Laura Abbott
The new challenge is to remove VLAs from the kernel (see https://lkml.org/lkml/2018/3/7/621) This patch replaces several VLAs with an appropriate call to kmalloc_array. Signed-off-by: Laura Abbott --- drivers/gpio/gpiolib.c | 55 +++--- 1 file

Re: [PATCH v3] kernel.h: Skip single-eval logic on literals in min()/max()

2018-03-09 Thread Andrew Morton
On Fri, 9 Mar 2018 12:05:36 -0800 Kees Cook wrote: > When max() is used in stack array size calculations from literal values > (e.g. "char foo[max(sizeof(struct1), sizeof(struct2))]", the compiler > thinks this is a dynamic calculation due to the single-eval logic, which >

Re: [PATCH v3] kernel.h: Skip single-eval logic on literals in min()/max()

2018-03-09 Thread Andrew Morton
On Fri, 9 Mar 2018 12:05:36 -0800 Kees Cook wrote: > When max() is used in stack array size calculations from literal values > (e.g. "char foo[max(sizeof(struct1), sizeof(struct2))]", the compiler > thinks this is a dynamic calculation due to the single-eval logic, which > is not needed in the

Re: [PATCH v2] kconfig: make unmet dependency warnings readable

2018-03-09 Thread Eugeniu Rosca
Hi Masahiro, Thanks for constantly improving the user experience of Kconfig. Just one small remark below. On Tue, Mar 06, 2018 at 07:51:39PM +0900, Masahiro Yamada wrote: > Currently, the unmet dependency warnings end up with endlessly long > expressions, most of which are false positives. > >

Re: [PATCH v2] kconfig: make unmet dependency warnings readable

2018-03-09 Thread Eugeniu Rosca
Hi Masahiro, Thanks for constantly improving the user experience of Kconfig. Just one small remark below. On Tue, Mar 06, 2018 at 07:51:39PM +0900, Masahiro Yamada wrote: > Currently, the unmet dependency warnings end up with endlessly long > expressions, most of which are false positives. > >

Re: [PATCH] proc: register filesystem last

2018-03-09 Thread Alexey Dobriyan
On Fri, Mar 09, 2018 at 02:49:38PM -0800, Andrew Morton wrote: > On Sat, 10 Mar 2018 01:27:09 +0300 Alexey Dobriyan > wrote: > > > As soon as register_filesystem() exits, filesystem can be mounted. > > It is better to present fully operational /proc. > > > > Of course it

Re: [PATCH] proc: register filesystem last

2018-03-09 Thread Alexey Dobriyan
On Fri, Mar 09, 2018 at 02:49:38PM -0800, Andrew Morton wrote: > On Sat, 10 Mar 2018 01:27:09 +0300 Alexey Dobriyan > wrote: > > > As soon as register_filesystem() exits, filesystem can be mounted. > > It is better to present fully operational /proc. > > > > Of course it doesn't matter because

Re: [PATCH v2 0/2] check I2C device id for pca984x chips

2018-03-09 Thread Peter Rosin
On 2018-03-05 21:58, Peter Rosin wrote: > Hi! > > This series tries to check the I2C device id, but instead of open > coding the check in the pca954x driver, I have a new function in > the core doing the work. > > Changes since v1: > - Added Tested-by tag from Adrian > - Added Reviewed-by tag

Re: [PATCH v2 0/2] check I2C device id for pca984x chips

2018-03-09 Thread Peter Rosin
On 2018-03-05 21:58, Peter Rosin wrote: > Hi! > > This series tries to check the I2C device id, but instead of open > coding the check in the pca954x driver, I have a new function in > the core doing the work. > > Changes since v1: > - Added Tested-by tag from Adrian > - Added Reviewed-by tag

Re: [PATCH 1/2] of: unittest: remove unneeded local return value variables

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 02:39:04PM -0800, frowand.l...@gmail.com wrote: > From: Frank Rowand > > A common pattern in many unittest functions is to save the return > value of a function in a local variable, then test the value of > the local variable, without using that

Re: [PATCH 1/2] of: unittest: remove unneeded local return value variables

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 02:39:04PM -0800, frowand.l...@gmail.com wrote: > From: Frank Rowand > > A common pattern in many unittest functions is to save the return > value of a function in a local variable, then test the value of > the local variable, without using that return value for any

Re: [PATCH 1/2] dt-bindings: trivial: add various mcp4017/18/19 potentiometers

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 08:58:29PM +0100, Peter Rosin wrote: > They come in 5 kOhm, 10 kOhm, 50 kOhm and 100 kOhm variations. > > Signed-off-by: Peter Rosin > --- > Documentation/devicetree/bindings/trivial-devices.txt | 12 > 1 file changed, 12 insertions(+) I've

Re: [PATCH 1/2] dt-bindings: trivial: add various mcp4017/18/19 potentiometers

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 08:58:29PM +0100, Peter Rosin wrote: > They come in 5 kOhm, 10 kOhm, 50 kOhm and 100 kOhm variations. > > Signed-off-by: Peter Rosin > --- > Documentation/devicetree/bindings/trivial-devices.txt | 12 > 1 file changed, 12 insertions(+) I've applied to avoid

Re: [PATCH v2 2/2] dt-bindings: backlight: Add binding for RAVE SP backlight driver

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 11:20:46AM -0800, Andrey Smirnov wrote: > Add Device Tree bindings for RAVE SP backlight driver - an MFD cell of > parent RAVE SP driver (documented in > Documentation/devicetree/bindings/mfd/zii,rave-sp.txt). > > Cc: Lee Jones > Cc: Daniel Thompson

Re: [PATCH v2 2/2] dt-bindings: backlight: Add binding for RAVE SP backlight driver

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 11:20:46AM -0800, Andrey Smirnov wrote: > Add Device Tree bindings for RAVE SP backlight driver - an MFD cell of > parent RAVE SP driver (documented in > Documentation/devicetree/bindings/mfd/zii,rave-sp.txt). > > Cc: Lee Jones > Cc: Daniel Thompson > Cc: Jingoo Han >

Re: [PATCH v2 02/12] clk: stm32mp1: Introduce STM32MP1 clock driver

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 05:53:55PM +0100, gabriel.fernan...@st.com wrote: > From: Gabriel Fernandez > > This patch introduces the mechanism to probe stm32mp1 driver. > It also defines registers definition. > This patch also introduces the generic mechanism to register >

Re: [PATCH v2 02/12] clk: stm32mp1: Introduce STM32MP1 clock driver

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 05:53:55PM +0100, gabriel.fernan...@st.com wrote: > From: Gabriel Fernandez > > This patch introduces the mechanism to probe stm32mp1 driver. > It also defines registers definition. > This patch also introduces the generic mechanism to register > a clock (a simple gate,

Re: [PATCH v2 01/12] dt-bindings: Document STM32MP1 Reset Clock Controller (RCC) bindings

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 05:53:54PM +0100, gabriel.fernan...@st.com wrote: > From: Gabriel Fernandez > > The RCC block is responsible of the management of the clock and reset > generation for the complete circuit. > > Signed-off-by: Gabriel Fernandez

Re: [PATCH v2 01/12] dt-bindings: Document STM32MP1 Reset Clock Controller (RCC) bindings

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 05:53:54PM +0100, gabriel.fernan...@st.com wrote: > From: Gabriel Fernandez > > The RCC block is responsible of the management of the clock and reset > generation for the complete circuit. > > Signed-off-by: Gabriel Fernandez > --- >

Re: [RFC PATCH -tip 0/9] kprobes: Cleanup jprobe implementation

2018-03-09 Thread Steven Rostedt
On Fri, 9 Mar 2018 21:35:17 +0900 Masami Hiramatsu wrote: > Hello, > > Since we decided to remove jprobe from kernel last year, > its APIs are disabled and we worked on moving in-kernel > jprobe users to kprobes or trace-events. And now no jprobe > users are here anymore.

Re: [RFC PATCH -tip 0/9] kprobes: Cleanup jprobe implementation

2018-03-09 Thread Steven Rostedt
On Fri, 9 Mar 2018 21:35:17 +0900 Masami Hiramatsu wrote: > Hello, > > Since we decided to remove jprobe from kernel last year, > its APIs are disabled and we worked on moving in-kernel > jprobe users to kprobes or trace-events. And now no jprobe > users are here anymore. > > I think it is

ip6tables - last base chain position %u doesn't match underflow %u (hook %u

2018-03-09 Thread valdis . kletnieks
Am hitting an issue with this commit: commit 0d7df906a0e78079a02108b06d32c3ef2238ad25 Author: Florian Westphal Date: Tue Feb 27 19:42:37 2018 +0100 netfilter: x_tables: ensure last rule in base chain matches underflow/policy This trips on my system: [ 64.402790]

ip6tables - last base chain position %u doesn't match underflow %u (hook %u

2018-03-09 Thread valdis . kletnieks
Am hitting an issue with this commit: commit 0d7df906a0e78079a02108b06d32c3ef2238ad25 Author: Florian Westphal Date: Tue Feb 27 19:42:37 2018 +0100 netfilter: x_tables: ensure last rule in base chain matches underflow/policy This trips on my system: [ 64.402790] ip6_tables: last

Re: [PATCH v4 03/10] drivers: qcom: rpmh-rsc: log RPMH requests in FTRACE

2018-03-09 Thread Steven Rostedt
On Fri, 9 Mar 2018 16:25:36 -0700 Lina Iyer wrote: > Log sent RPMH requests and interrupt responses in FTRACE. > > Cc: Steven Rostedt > Signed-off-by: Lina Iyer > --- > > Changes in v4: > - fix compilation issues, use

Re: [PATCH v4 03/10] drivers: qcom: rpmh-rsc: log RPMH requests in FTRACE

2018-03-09 Thread Steven Rostedt
On Fri, 9 Mar 2018 16:25:36 -0700 Lina Iyer wrote: > Log sent RPMH requests and interrupt responses in FTRACE. > > Cc: Steven Rostedt > Signed-off-by: Lina Iyer > --- > > Changes in v4: > - fix compilation issues, use __assign_str > - use %#x instead of 0x%08x Hmm, I don't

Re: [PATCH 2/4] ARM: sunxi: mc-smp: Use DT enable-method for sun9i A80 SMP

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 11:00:10PM +0800, Chen-Yu Tsai wrote: > Instead of having an early init function check the machine compatible > and installing multi-cluster SMP support for the A80 if it matches, > use a new cpu enable-method string. This makes the platform support > future proof in case

Re: [PATCH 2/4] ARM: sunxi: mc-smp: Use DT enable-method for sun9i A80 SMP

2018-03-09 Thread Rob Herring
On Thu, Mar 08, 2018 at 11:00:10PM +0800, Chen-Yu Tsai wrote: > Instead of having an early init function check the machine compatible > and installing multi-cluster SMP support for the A80 if it matches, > use a new cpu enable-method string. This makes the platform support > future proof in case

[PATCH 3/4] staging:iio:ad2s1210: Remove old_data from ad2s1210_state

2018-03-09 Thread Rodrigo Siqueira
The variable old_data is a bool type, which only receives the value 'true' in the function ad2s1210_config_write and ad2s1210_config_read. There is no other use for this variable. This patch removes old_data from the ad2s1210_state and from all the function that use it. Signed-off-by: Rodrigo

[PATCH 3/4] staging:iio:ad2s1210: Remove old_data from ad2s1210_state

2018-03-09 Thread Rodrigo Siqueira
The variable old_data is a bool type, which only receives the value 'true' in the function ad2s1210_config_write and ad2s1210_config_read. There is no other use for this variable. This patch removes old_data from the ad2s1210_state and from all the function that use it. Signed-off-by: Rodrigo

[PATCH 2/4] staging:iio:ad2s1210: Remove unused #define directive

2018-03-09 Thread Rodrigo Siqueira
This patch removes some #define directives not used in the code. Signed-off-by: Rodrigo Siqueira --- drivers/staging/iio/resolver/ad2s1210.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/iio/resolver/ad2s1210.c

Re: [PATCH v2 2/8] [PATCH 2/8] Documentations: dt-bindings: Add a document of PECI adapter driver for Aspeed AST24xx/25xx SoCs

2018-03-09 Thread Jae Hyun Yoo
Hi Milton, Thanks for sharing your time to review this patch. Please see my answer inline. Jae On 3/9/2018 3:41 PM, Milton Miller II wrote: About 03/07/2018 04:12PM in some time zone, Pavel Machek wrote: Subject: Re: [PATCH v2 2/8] [PATCH 2/8] Documentations: dt-bindings: Add a document

[PATCH 2/4] staging:iio:ad2s1210: Remove unused #define directive

2018-03-09 Thread Rodrigo Siqueira
This patch removes some #define directives not used in the code. Signed-off-by: Rodrigo Siqueira --- drivers/staging/iio/resolver/ad2s1210.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/iio/resolver/ad2s1210.c b/drivers/staging/iio/resolver/ad2s1210.c index

Re: [PATCH v2 2/8] [PATCH 2/8] Documentations: dt-bindings: Add a document of PECI adapter driver for Aspeed AST24xx/25xx SoCs

2018-03-09 Thread Jae Hyun Yoo
Hi Milton, Thanks for sharing your time to review this patch. Please see my answer inline. Jae On 3/9/2018 3:41 PM, Milton Miller II wrote: About 03/07/2018 04:12PM in some time zone, Pavel Machek wrote: Subject: Re: [PATCH v2 2/8] [PATCH 2/8] Documentations: dt-bindings: Add a document

[PATCH 4/4] staging:iio:ad2s1210: Add comments/documentation

2018-03-09 Thread Rodrigo Siqueira
The original code of AD2S1210 does not have documentation for structs and register configurations; this difficult the code comprehension. This patch adds structs documentation, briefly comments some register settings and acronyms, and adds little explanations of some calculation found in the code.

[PATCH 4/4] staging:iio:ad2s1210: Add comments/documentation

2018-03-09 Thread Rodrigo Siqueira
The original code of AD2S1210 does not have documentation for structs and register configurations; this difficult the code comprehension. This patch adds structs documentation, briefly comments some register settings and acronyms, and adds little explanations of some calculation found in the code.

[PATCH 0/4] staging:iio:ad2s1210: Driver cleanup

2018-03-09 Thread Rodrigo Siqueira
This patchset removes some unused #define directive and variables. Additionally, it fixes two checkpatch problems. Finally, the last patch adds struct documentation and comments to some part of the code. All of the changes have the intention to improve the readability of new updates. Rodrigo

[PATCH 1/4] staging:iio:ad2s1210: Remove end of line with '['

2018-03-09 Thread Rodrigo Siqueira
This patch fixes the checkpatch.pl check: iio/resolver/ad2s1210.c:202: CHECK: Lines should not end with a '[' Signed-off-by: Rodrigo Siqueira --- drivers/staging/iio/resolver/ad2s1210.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

[PATCH 1/4] staging:iio:ad2s1210: Remove end of line with '['

2018-03-09 Thread Rodrigo Siqueira
This patch fixes the checkpatch.pl check: iio/resolver/ad2s1210.c:202: CHECK: Lines should not end with a '[' Signed-off-by: Rodrigo Siqueira --- drivers/staging/iio/resolver/ad2s1210.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

[PATCH 0/4] staging:iio:ad2s1210: Driver cleanup

2018-03-09 Thread Rodrigo Siqueira
This patchset removes some unused #define directive and variables. Additionally, it fixes two checkpatch problems. Finally, the last patch adds struct documentation and comments to some part of the code. All of the changes have the intention to improve the readability of new updates. Rodrigo

Re: WARNING in __proc_create

2018-03-09 Thread Eric Dumazet
On 03/09/2018 03:32 PM, Cong Wang wrote: On Fri, Mar 9, 2018 at 3:21 PM, Eric Dumazet wrote: On 03/09/2018 03:05 PM, Cong Wang wrote: BTW, the warning itself is all about empty names, so perhaps it's better to fix them separately. Huh ? You want more syzbot

Re: WARNING in __proc_create

2018-03-09 Thread Eric Dumazet
On 03/09/2018 03:32 PM, Cong Wang wrote: On Fri, Mar 9, 2018 at 3:21 PM, Eric Dumazet wrote: On 03/09/2018 03:05 PM, Cong Wang wrote: BTW, the warning itself is all about empty names, so perhaps it's better to fix them separately. Huh ? You want more syzbot reports ? I do not. I

Re: [PATCH v2 2/8] [PATCH 2/8] Documentations: dt-bindings: Add a document of PECI adapter driver for Aspeed AST24xx/25xx SoCs

2018-03-09 Thread Milton Miller II
About 03/07/2018 04:12PM in some time zone, Pavel Machek wrote: >Subject: Re: [PATCH v2 2/8] [PATCH 2/8] Documentations: dt-bindings: >Add a document of PECI adapter driver for Aspeed AST24xx/25xx SoCs > >Hi! > >> >Are these SoCs x86-based? >> >> Yes, these are ARM SoCs. Please see Andrew's

Re: [PATCH v2 2/8] [PATCH 2/8] Documentations: dt-bindings: Add a document of PECI adapter driver for Aspeed AST24xx/25xx SoCs

2018-03-09 Thread Milton Miller II
About 03/07/2018 04:12PM in some time zone, Pavel Machek wrote: >Subject: Re: [PATCH v2 2/8] [PATCH 2/8] Documentations: dt-bindings: >Add a document of PECI adapter driver for Aspeed AST24xx/25xx SoCs > >Hi! > >> >Are these SoCs x86-based? >> >> Yes, these are ARM SoCs. Please see Andrew's

[PATCH] ARM: bcm2835: Add the DPI hardware to the device tree.

2018-03-09 Thread Eric Anholt
It's currently marked disabled, as it's not useful without a panel associated with it and the GPIO pins routed to ALT2. Signed-off-by: Eric Anholt --- This patch got misplaced as part of the original VC4 DPI work (which included the binding). I've got a full branch demoing a

[PATCH] ARM: bcm2835: Add the DPI hardware to the device tree.

2018-03-09 Thread Eric Anholt
It's currently marked disabled, as it's not useful without a panel associated with it and the GPIO pins routed to ALT2. Signed-off-by: Eric Anholt --- This patch got misplaced as part of the original VC4 DPI work (which included the binding). I've got a full branch demoing a panel at

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