[PATCH] staging: vboxvideo: Fix reporting invalid suggested-offset-properties

The x and y hints receives from the host are unsigned 32 bit integers and they get set to -1 (0x) when invalid. Before this commit the vboxvideo driver was storing them in an u16 causing the -1 to be truncated to 65535 which, once reported to userspace, was breaking gnome 3.26+ in Wayland

Re: [PATCH 08/20] staging: vc04_services: vchiq_core: mark expected switch fall-through

> "Gustavo A. R. Silva" hat am 12. Oktober 2017 um > 18:16 geschrieben: > > > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. > > Signed-off-by: Gustavo A. R. Silva > --- >

[PATCH 19/20] staging: lustre: ptlrpc: mark expected switch fall-throughs

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/lustre/lustre/ptlrpc/pack_generic.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH 10/20] staging: rtl8188eu: usb_ops_linux: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 1077613 Signed-off-by: Gustavo A. R. Silva --- drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c | 1 + 1 file changed, 1 insertion(+)

[PATCH 04/20] staging: rtl8723bs: rtw_mlme_ext: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 06/20] staging: comedi: s526: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/comedi/drivers/s526.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git

[PATCH 18/20] staging: lustre: osc: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 1077598 Signed-off-by: Gustavo A. R. Silva --- drivers/staging/lustre/lustre/osc/osc_cache.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH 03/20] staging: rtl8192e: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/rtl8192e/rtllib_wx.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 08/20] staging: vc04_services: vchiq_core: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)

[PATCH 00/20] mark expected switch fall-throughs

In preparation to enabling -Wimplicit-fallthrough, this patchset aims to mark switch cases where we are expecting to fall through. In Kees Cook words: "This is an unfortunate omission in the C language, and thankfully both gcc and clang have stepped up to solve this the same way static analyzers

[PATCH 01/20] staging: ks7010: ks_wlan_net: mark expected switch fall-throughs

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 1364489 Signed-off-by: Gustavo A. R. Silva --- drivers/staging/ks7010/ks_wlan_net.c | 10 ++ 1 file changed, 10 insertions(+)

[PATCH 05/20] staging: lustre: lnet: net_fault: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/lustre/lnet/lnet/net_fault.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 12/20] staging: lustre: lnet: selftest: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/lustre/lnet/selftest/conctl.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 20/20] staging: lustre: rpc: mark expected switch fall-throughs

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 1077604 Addresses-Coverity-ID: 1077605 Signed-off-by: Gustavo A. R. Silva --- drivers/staging/lustre/lnet/selftest/rpc.c | 13

[PATCH 16/20] staging: lustre: lprocfs: mark expected switch fall-throughs

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 1271166 Addresses-Coverity-ID: 1271167 Addresses-Coverity-ID: 1271168 Addresses-Coverity-ID: 1271169 Signed-off-by: Gustavo A. R. Silva ---

[PATCH 14/20] staging: lustre: lnet: selftest: mark expected switch fall-throughs

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/lustre/lnet/selftest/module.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

[PATCH 02/20] staging: lustre: lnet: socklnd: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/lustre/lnet/klnds/socklnd/socklnd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 07/20] staging: rtl8188eu: usb_halinit: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 1373894 Signed-off-by: Gustavo A. R. Silva --- drivers/staging/rtl8188eu/hal/usb_halinit.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH 09/20] staging: vt6656: card: mark expected switch fall-throughs

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/vt6656/card.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/vt6656/card.c

[PATCH 11/20] staging: r8822be: mark expected switch fall-throughs

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/rtlwifi/halmac/halmac_88xx/halmac_api_88xx.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH 15/20] staging: lustre: llite: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/lustre/lustre/llite/namei.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH 17/20] staging: lustre: ldlm: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 13/20] staging: rtlwifi: halmac: mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/staging/rtlwifi/halmac/rtl_halmac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node

On 12.10.2017 16:45, Jon Hunter wrote: > > On 12/10/17 14:25, Thierry Reding wrote: >> * PGP Signed by an unknown key >> >> On Thu, Oct 12, 2017 at 03:06:17PM +0300, Dmitry Osipenko wrote: >>> Hello Vladimir, >>> >>> On 12.10.2017 10:43, Vladimir Zapolskiy wrote: Hello Dmitry, On

RE: [PATCH 3/3] Drivers: hv: vmbus: Make pannic reporting to be more useful

> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Thursday, October 12, 2017 6:40 AM > To: k...@exchange.microsoft.com > Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; >

Re: staging/irda/net: Adjustments for several function implementations

> Did you read drivers/staging/irda/TODO ? Yes. How do recent contributions (by other software developers) fit to information that is provided in this file? Regards, Markus ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node

On 12/10/17 14:25, Thierry Reding wrote: > * PGP Signed by an unknown key > > On Thu, Oct 12, 2017 at 03:06:17PM +0300, Dmitry Osipenko wrote: >> Hello Vladimir, >> >> On 12.10.2017 10:43, Vladimir Zapolskiy wrote: >>> Hello Dmitry, >>> >>> On 10/11/2017 11:08 PM, Dmitry Osipenko wrote: Add

Re: [PATCH 3/3] Drivers: hv: vmbus: Make pannic reporting to be more useful

k...@exchange.microsoft.com writes: > From: "K. Y. Srinivasan" > > Hyper-V allows the guest to report panic and the guest can pass additional > information. All this is logged on the host. Currently Linux is passing back > information that is not particularly useful. Make the

Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node

On Thu, Oct 12, 2017 at 03:06:17PM +0300, Dmitry Osipenko wrote: > Hello Vladimir, > > On 12.10.2017 10:43, Vladimir Zapolskiy wrote: > > Hello Dmitry, > > > > On 10/11/2017 11:08 PM, Dmitry Osipenko wrote: > >> Add a device node for the video decoder engine found on Tegra20. > >> > >>

[PATCH] staging/ccree: Declare compiled out functions static inline

Sparse was giving out a warning for symbols 'cc_set_ree_fips_status' and 'fips_handler' that they were not declared and need to be made static. This patch makes both the symbols static inline, to remove the warnings. Signed-off-by: Rishabh Hardas ---

Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node

Hello Vladimir, On 12.10.2017 10:43, Vladimir Zapolskiy wrote: > Hello Dmitry, > > On 10/11/2017 11:08 PM, Dmitry Osipenko wrote: >> Add a device node for the video decoder engine found on Tegra20. >> >> Signed-off-by: Dmitry Osipenko >> --- >> arch/arm/boot/dts/tegra20.dtsi

Re: [PATCH 00/10] staging/irda/net: Adjustments for several function implementations

Did you read drivers/staging/irda/TODO ? There is no need to answer that. You already have. Thanks a lot for your ever lasting invaluable contributions to /dev/null. Please continue. Bjørn ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node

On 12.10.2017 13:57, Jon Hunter wrote: > > On 12/10/17 11:51, Dmitry Osipenko wrote: >> On 12.10.2017 11:49, Jon Hunter wrote: >>> >>> On 11/10/17 21:08, Dmitry Osipenko wrote: Add a device node for the video decoder engine found on Tegra20. Signed-off-by: Dmitry Osipenko

Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node

On 12/10/17 11:51, Dmitry Osipenko wrote: > On 12.10.2017 11:49, Jon Hunter wrote: >> >> On 11/10/17 21:08, Dmitry Osipenko wrote: >>> Add a device node for the video decoder engine found on Tegra20. >>> >>> Signed-off-by: Dmitry Osipenko >>> --- >>>

[PATCH 10/10] staging/irda/net: Use seq_puts() in four functions

From: Markus Elfring Date: Thu, 12 Oct 2017 11:08:36 +0200 Strings which did not contain a data format specification should be put into a sequence. Thus use the corresponding function "seq_puts". This issue was detected by using the Coccinelle software.

[PATCH 09/10] staging/irda/net: Combine some seq_printf() calls in two functions

From: Markus Elfring Date: Thu, 12 Oct 2017 08:58:38 +0200 Some data were printed into a sequence by separate function calls. Print the same data by a single function call at each place instead. This issue was detected by using the Coccinelle software.

Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node

On 12.10.2017 11:49, Jon Hunter wrote: > > On 11/10/17 21:08, Dmitry Osipenko wrote: >> Add a device node for the video decoder engine found on Tegra20. >> >> Signed-off-by: Dmitry Osipenko >> --- >> arch/arm/boot/dts/tegra20.dtsi | 17 + >> 1 file changed, 17

[PATCH 08/10] staging/irda/net: Use common error handling code in irias_new_object()

From: Markus Elfring Date: Thu, 12 Oct 2017 08:52:53 +0200 Add a jump target so that a bit of exception handling can be better reused at the end of this function. Signed-off-by: Markus Elfring ---

[PATCH 07/10] staging/irda/net: Delete an unnecessary variable initialisation in four functions

From: Markus Elfring Date: Wed, 11 Oct 2017 22:26:00 +0200 The variable "tx_skb" will eventually be set to an appropriate pointer a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring ---

[PATCH 06/10] staging/irda/net: Delete an unnecessary variable initialisation in two functions

From: Markus Elfring Date: Wed, 11 Oct 2017 22:22:13 +0200 The local variable "tx_skb" will only be used in a single if branch of these functions. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring

[PATCH 05/10] staging/irda/net: Delete an unnecessary variable initialisation in irlap_recv_discovery_xid_rsp()

From: Markus Elfring Date: Wed, 11 Oct 2017 22:20:22 +0200 The variable "discovery" will eventually be set to an appropriate pointer a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring

[PATCH 04/10] staging/irda/net: Delete an unnecessary variable initialisation in irlap_recv_discovery_xid_cmd()

From: Markus Elfring Date: Wed, 11 Oct 2017 22:18:34 +0200 The local variable "discovery" will only be used in a single if branch of this function. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring

[PATCH 03/10] staging/irda/net: Adjust 385 checks for null pointers

From: Markus Elfring Date: Wed, 11 Oct 2017 22:10:26 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written … Thus

[PATCH 02/10] staging: irda: Delete ten error messages for a failed memory allocation

From: Markus Elfring Date: Tue, 10 Oct 2017 21:10:43 +0200 Omit extra messages for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 01/10] staging: irda: Improve a size determination in 20 functions

From: Markus Elfring Date: Tue, 10 Oct 2017 19:35:56 +0200 * Replace the specification of data types by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 00/10] staging/irda/net: Adjustments for several function implementations

From: Markus Elfring Date: Thu, 12 Oct 2017 11:25:43 +0200 Several update suggestions were taken into account from static source code analysis. Markus Elfring (10): Improve a size determination in 20 functions Delete ten error messages for a failed memory

Re: Two rtlwifi drivers?

On Thu, Oct 12, 2017 at 11:38:06AM +0300, Kalle Valo wrote: > > So what to do? Any ideas? What makes your life easier? You can just > > ignore the staging tree, as it should not affect your portion of the > > kernel at all, right? > > Yes, I automatically ignore anything staging related. But

Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node

Hello Dmitry, On 10/11/2017 11:08 PM, Dmitry Osipenko wrote: > Add a device node for the video decoder engine found on Tegra20. > > Signed-off-by: Dmitry Osipenko > --- > arch/arm/boot/dts/tegra20.dtsi | 17 + > 1 file changed, 17 insertions(+) > > diff --git

Re: Two rtlwifi drivers?

Larry Finger writes: > On 10/11/2017 08:13 AM, Greg Kroah-Hartman wrote: > >> On Wed, Oct 11, 2017 at 12:06:00PM +0300, Kalle Valo wrote: >> I think it's horrid too. But, if no one is able to do the real work >> here, we hurt users who just need to use their hardware

Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node

On 11/10/17 21:08, Dmitry Osipenko wrote: > Add a device node for the video decoder engine found on Tegra20. > > Signed-off-by: Dmitry Osipenko > --- > arch/arm/boot/dts/tegra20.dtsi | 17 + > 1 file changed, 17 insertions(+) > > diff --git

Re: Two rtlwifi drivers?

Greg Kroah-Hartman writes: >> >> I'm getting slightly annoyed that we now apparently have two duplicate >> >> rtlwifi drivers (with the same name!) and I'm being spammed by staging >> >> patches. Was this really a smart thing to do? And what will be the >> >> future