Re: [PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long

On Thu, Jun 08, 2017 at 11:25:55PM -0700, Elias Carter wrote: > On Fri, Jun 09, 2017 at 07:41:12AM +0200, Greg KH wrote: > > On Thu, Jun 08, 2017 at 06:38:30PM -0700, Elias Carter wrote: > > > From: edcarter > > > > > > Fixed a coding style issue where a line was longer than 80 characters. > > >

Re: [PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long

On Fri, Jun 09, 2017 at 07:41:12AM +0200, Greg KH wrote: > On Thu, Jun 08, 2017 at 06:38:30PM -0700, Elias Carter wrote: > > From: edcarter > > > > Fixed a coding style issue where a line was longer than 80 characters. > > > > Signed-off-by: Elias Carter > > Your from and signed-off-by names h

[PATCH] Drivers: rtl8192u - style fix

Fixed checkpatch error of "no spaces at the start of a line" Signed-off-by: Derek Robson --- drivers/staging/rtl8192u/r8192U.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U.h b/drivers/staging/rtl8192u/r8192U.h index 4c7a5e3d3e5e..2959

[PATCH] Drivers: ccree - style fix

Align block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/ccree/cc_hw_queue_defs.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/ccree/cc_hw_queue_defs.h b/drivers/staging/ccree/cc_hw_queue_defs.h index aaa56c85bda2

Re: [PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long

On Thu, Jun 08, 2017 at 06:38:30PM -0700, Elias Carter wrote: > From: edcarter > > Fixed a coding style issue where a line was longer than 80 characters. > > Signed-off-by: Elias Carter Your from and signed-off-by names have to match :( ___ devel mai

[PATCH] Drivers: unisys: visorhba - style fix

Fixed style of permissions to octal. Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/unisys/visorhba/visorhba_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys/visorhba/visorhba_main.c b/drivers/staging/unisys/visorhba/visor

[PATCH] staging: ks7010: use little-endian types

This patch fixes a number of sparse warnings of the form: drivers/staging/ks7010/ks_hostif.c:2187:29: warning: incorrect type in assignment (different base types) generated when storing little-endian data in variables that do not have a specified endianness. Signed-off-by: Perry Hooker --

[PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long

From: edcarter Fixed a coding style issue where a line was longer than 80 characters. Signed-off-by: Elias Carter --- drivers/staging/wlan-ng/cfg80211.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/wlan-ng/cfg80211.c b/drivers/staging/wlan-ng/cfg80211.

[driver-core:driver_attr 10/11] drivers/s390/char/sclp.c:1104:23: error: 'con_pages_show' undeclared here (not in a function)

tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git driver_attr head: c50b7be3ac64d5322e8f84f2e78f2f4b518b36ef commit: 17d260decb4203d5be1b2ebdbf093897553bfeaa [10/11] s390: drivers: convert to use DRIVER_ATTR_RO/WO config: s390-default_defconfig (attached as .config)

[PATCH net-next 3/6] netvsc: use typed pointer for internal state

The element netvsc_device:extension is always a point to RNDIS information. Signed-off-by: Stephen Hemminger --- drivers/net/hyperv/hyperv_net.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/hyperv/hyperv_net.h b/drivers/net/hyperv/hyperv_net.h index 262b2ea57

[PATCH net-next 4/6] netvsc: mark error cases as unlikely

Mark if() statements used for error handling only as unlikely() Signed-off-by: Stephen Hemminger --- drivers/net/hyperv/netvsc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/hyperv/netvsc.c b/drivers/net/hyperv/netvsc.c index 652453d9fb08..caf89a245ba6 1006

[PATCH net-next 1/6] netvsc: optimize calculation of number of slots

Speed up transmit check for fragmented packets by using existing macros to compute number of pages, and eliminate loop since skb fragments each take a page. Number of slots is also unsigned. Signed-off-by: Stephen Hemminger --- drivers/net/hyperv/netvsc_drv.c | 43 ++-

[PATCH net-next 5/6] netvsc: pass net_device to netvsc_init_buf and netvsc_connect_vsp

Don't need to find netvsc_device structure, caller already had it. Also rearrange declarations. Signed-off-by: Stephen Hemminger --- drivers/net/hyperv/netvsc.c | 31 +++ 1 file changed, 11 insertions(+), 20 deletions(-) diff --git a/drivers/net/hyperv/netvsc.c b/dri

[PATCH net-next 2/6] netvsc: use hv_get_bytes_to_read

Don't need need to look at write space in netvsc_close. Signed-off-by: Stephen Hemminger --- drivers/net/hyperv/netvsc_drv.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv/netvsc_drv.c index df6d8e28949e..436a3a

[PATCH net-next 6/6] netvsc: fold in get_outbound_net_device

No longer need common code to find get_outbound_net_device. Signed-off-by: Stephen Hemminger --- drivers/net/hyperv/netvsc.c | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/drivers/net/hyperv/netvsc.c b/drivers/net/hyperv/netvsc.c index 4d4fde0c7974..7c5ed8f

[PATCH net-next 0/6] netvsc: small cleanups

These are all small optimizations found during development of later features. Stephen Hemminger (6): netvsc: optimize calculation of number of slots netvsc: use hv_get_bytes_to_read netvsc: use typed pointer for internal state netvsc: mark error cases as unlikely netvsc: pass net_device

Re: [PATCH v8 14/34] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder

On 06/08/2017 01:25 PM, Tim Harvey wrote: Steve, You need to remove the fim node now that you've moved this to V4L2 controls. Yep, I caught this just after sending the v8 patchset. I'll send a v9 of this patch. Steve ___ devel mailing list de..

Re: [PATCH v8 14/34] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder

On Wed, Jun 7, 2017 at 11:33 AM, Steve Longerbeam wrote: > Enables the ADV7180 decoder sensor. The ADV7180 connects to the > parallel-bus mux input on ipu1_csi0_mux. > > The ADV7180 power pin is via max7310_b port expander. > > Signed-off-by: Steve Longerbeam > > - Use IRQ_TYPE_LEVEL_LOW instead

[driver-core:driver_attr 8/11] drivers/pcmcia/ds.c:136:65: error: macro "DRIVER_ATTR_WO" passed 4 arguments, but takes just 1

tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git driver_attr head: c50b7be3ac64d5322e8f84f2e78f2f4b518b36ef commit: 9a7a2d0c98fd773bf315f298db91e01ffec8ac39 [8/11] pcmcia: ds: convert to use DRIVER_ATTR_RO config: x86_64-randconfig-x019-201723 (attached as .config)

[PATCH] atomisp: use correct dialect to disable warnings

There's a Macro that checks if gcc supports a warning before disabling it. Use it, in order to avoid warnings when building with older gcc versions. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/media/atomisp/i2c/Makefile| 6 -- drivers/staging/media/atomisp/i2c/imx/Makefi

The Director Of Promotions BMW Lottery Department

Dear BMW Enthusiast, This is to inform you that you have been selected for a prize of a brand new 2015 Model BMW 7 Series Car and a Check of $500,000.00usd from international programs was held Jan 29th of this first section of 2017 in the New Iberial Louisiana, UNITED STATES OF AMERICA. Mode

Re: [PATCH net v2 0/3] netvsc: bug fixes

From: Stephen Hemminger Date: Wed, 7 Jun 2017 15:53:46 -0700 > These are bugfixes for netvsc driver in 4.12. Looks good, series applied, thanks. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listin

[PATCH] [media] staging: css2400/Makefile: don't include makefiles

The atomisp css2400/Makefile includes a Makefile.common: include $(srctree)/$(src)/../Makefile.common Well, this file doesn't exist at the Kernel tree :-) So, don't include it. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/media/atomisp/pci/atomisp2/css2400/Makefile | 2 --

[PATCH v7 10/10] staging: fsl-mc: move bus driver out of staging

From: Stuart Yoder Move the source files out of staging into their final locations: -include files in drivers/staging/fsl-mc/include go to include/linux/fsl -irq-gic-v3-its-fsl-mc-msi.c goes to drivers/irqchip -source in drivers/staging/fsl-mc/bus goes to drivers/bus/fsl-mc -README.txt, p

[PATCH v7 07/10] staging: fsl-mc: add binding path to MAINTAINERS

From: Laurentiu Tudor The mc bus device tree binding is maintained but not mentioned in the MAINTAINERS file. This patch adds it. Signed-off-by: Laurentiu Tudor --- Notes: -v7 -rewrapped commit message -v2 -added commit description MAINTAINERS | 1 + 1 file changed, 1 ins

[PATCH v7 06/10] staging: fsl-mc: drop unused forward declaration

From: Laurentiu Tudor This forward declaration of "struct fsl_mc_resource" is of no use so drop it. Signed-off-by: Laurentiu Tudor --- Notes: -v7 -rewrapped commit message drivers/staging/fsl-mc/include/mc-sys.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/fsl

[PATCH v7 05/10] staging: fsl-mc: remove extra blank line

From: Laurentiu Tudor Remove extra blank line reported by checkpatch.pl. Signed-off-by: Laurentiu Tudor --- Notes: -v7 -no changes drivers/staging/fsl-mc/include/dprc.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/fsl-mc/include/dprc.h b/drivers/staging/fsl-m

[PATCH v7 09/10] staging: fsl-mc: add reference to mc-bus DT binding

From: Laurentiu Tudor Update README to reference the mc-bus device tree node binding. Signed-off-by: Laurentiu Tudor --- Notes: -v7 -rewrapped commit message drivers/staging/fsl-mc/README.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/fsl-mc/README.txt b

[PATCH v7 08/10] staging: fsl-mc: drop reference to restool

From: Laurentiu Tudor Drop reference to user space restool utility from the README. It will be added back together with the actual support in the bus driver. Signed-off-by: Laurentiu Tudor --- Notes: -v7 -no changes drivers/staging/fsl-mc/README.txt | 4 +--- 1 file changed, 1 inse

[PATCH v7 04/10] staging: fsl-mc: drop a few useless #includes

From: Laurentiu Tudor Some #includes were needlessly done from header files. Drop them from there and update the only .c file that implicitly needed one of those #includes. Signed-off-by: Laurentiu Tudor --- Notes: -v7 -rewrapped commit message drivers/staging/fsl-mc/bus/fsl-mc-msi

[PATCH v7 02/10] staging: fsl-mc: fix macros with possible side effects

From: Laurentiu Tudor Several macros were triggering this checkpatch.pl warning: "Macro argument reuse '$arg' - possible side-effects?" Fix the warning by avoiding multiple macro argument use. Signed-off-by: Laurentiu Tudor --- Notes: -v7 -no changes drivers/staging/fsl-mc/bus/dp

[PATCH v7 03/10] staging: fsl-mc: simplify couple of deallocations

From: Laurentiu Tudor Simplify a couple of deallocations code paths. This also fixes these checkpatch.pl false positives: "WARNING: kfree(NULL) is safe and this check is probably not required" Signed-off-by: Laurentiu Tudor --- Notes: -v7 -rewrapped commit message drivers/staging/

[PATCH v7 01/10] staging: fsl-mc: enclose macro params in parens

From: Laurentiu Tudor Several macros didn't had macro params enclosed in parens. Fix them to avoid precedence issues. Found with checkpatch.pl who was issuing this message: "Macro argument 'id' may be better as '(id)' to avoid precedence issues" Signed-off-by: Laurentiu Tudor --- Note

[PATCH v7 00/10] staging: fsl-mc: move bus driver out of staging

From: Laurentiu Tudor Now that there is a functional ethernet driver probed on the mc bus this patch series requests that the fsl-mc bus driver be moved out of staging. The proposed destination for the bus driver is drivers/bus. Proposed location for global header files for fsl-mc and dpaa2 is t

[PATCH] Drivers: sm750fb - style fix

Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name" in header files. Signed-off-by: Derek Robson --- drivers/staging/sm750fb/sm750.h | 24 1 file changed, 16 insertions(+), 8 deletions(-) diff --git a/drivers/s

[PATCH V2] Drivers: vt6655 - style fix

Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name" in header files. Signed-off-by: Derek Robson --- Changes in v2: - Make the commit message more clear. drivers/staging/vt6655/card.h| 30 ++--- drivers/staging/