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
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
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
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
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
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 ==
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
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
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
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:
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
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[]
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
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
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
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
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
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.
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
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
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
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
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
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
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.
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
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
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
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:
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
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
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
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
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
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
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
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
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
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
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
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
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
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 |
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
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
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
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
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
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
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(-)
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
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
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
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
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
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
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
>
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
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.
>
>
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.
>
>
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
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
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
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
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
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
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
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
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
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
>
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
>
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,
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
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
> ---
>
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.
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
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]
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
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
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
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
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
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
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
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
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
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
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
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.
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.
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
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
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
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
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
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
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
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
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
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
501 - 600 of 2416 matches
Mail list logo