[PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style

Linux kernel coding style uses one space around most binary and ternary operators and spaces are prohibited at the start of a line. Concatenated strings should have space between elements and space is required after ','. In conditional statements, do not use unnecessary braces where a single statem

[PATCH 01/10] staging: rtl8723bs: Add spaces around '='

Use one space around (on each side of) '=' operator Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_

[PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*'

Use one space around most binary operators Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c inde

Re: [PATCH v6] staging: pi433: Fixes minor typo errors

On Fri, Sep 08, 2017 at 11:18:08PM +0530, Harsha Sharma wrote: > Hello, > I have tried to follow above given instructions but please correct me if I am > wrong somewhere. What "above given instructions"? And please fix your email client to not send html email, it is getting rejected by the mailin

Re: [PATCH 1/2] staging: typec: tcpm: Validate source and sink caps

On Sat, Sep 09, 2017 at 09:54:42AM -0700, Guenter Roeck wrote: > On 09/08/2017 12:13 PM, Greg Kroah-Hartman wrote: > > On Fri, Sep 08, 2017 at 10:29:52AM -0700, Badhri Jagan Sridharan wrote: > > > On Fri, Sep 8, 2017 at 2:45 AM, Greg Kroah-Hartman > > > wrote: > > > > On Thu, Sep 07, 2017 at 06:22

[PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!='

Use spaces around most binary operators Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c index 1

[PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line

No spaces at the start of a line Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c index 5ceccd7..97ff3ab 1006

[PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings

Use spaces between concatenated strings Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 50 ++--- 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep

[PATCH 06/10] staging: rtl8723bs: Add space after ','

Space required after ',' Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c index b320df8..ecb01cf

[PATCH 07/10] staging: rtl8723bs: Removed unneccesary space

Remove space between function name and open parenthesis '(' Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_in

[PATCH 08/10] staging: rtl8723bs: Remove unnecessary space

Remove space before ';', '++', ',' Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c index 8e13fb3..7a2d

[PATCH 09/10] staging: rtl8723bs: Remove unneccesary braces and change position of open brace

Follow linux-kernel code style for conditional statements Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 63 + 1 file changed, 19 insertions(+), 44 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/stagin

[PATCH 10/10] staging: rtl8723bs: Remove unnecessary blank lines

Blank lines aren't necessary after an open brace and before a close brace Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_in

[PATCH] staging:r8192u: replace request_module with try_then_request_module

Return value of request_module() does not handled, so it is possible to use shorter try_then_request_module(). Signed-off-by: Ivan Safonov --- .../staging/rtl8192u/ieee80211/ieee80211_softmac.c | 27 +++--- drivers/staging/rtl8192u/ieee80211/ieee80211_wx.c | 15 2 f

Re: [Outreachy kernel] [PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*'

On Sun, 10 Sep 2017, Harsha Sharma wrote: > Use one space around most binary operators > > Signed-off-by: Harsha Sharma Acked-by: Julia Lawall > --- > drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/st

Re: [Outreachy kernel] [PATCH 01/10] staging: rtl8723bs: Add spaces around '='

On Sun, 10 Sep 2017, Harsha Sharma wrote: > Use one space around (on each side of) '=' operator > > Signed-off-by: Harsha Sharma Acked-by: Julia Lawall > --- > drivers/staging/rtl8723bs/os_dep/os_intfs.c | 20 ++-- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff

Re: [Outreachy kernel] [PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!='

On Sun, 10 Sep 2017, Harsha Sharma wrote: > Use spaces around most binary operators > > Signed-off-by: Harsha Sharma Acked-by: Julia Lawall > --- > drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/stagi

Re: [Outreachy kernel] [PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line

On Sun, 10 Sep 2017, Harsha Sharma wrote: > No spaces at the start of a line > > Signed-off-by: Harsha Sharma Acked-by: Julia Lawall > --- > drivers/staging/rtl8723bs/os_dep/os_intfs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/rtl8723bs/os_dep/

[PATCH 1/5] staging: typec: tcpm: Drop commented out code

Commented out code can be added as needed. Drop it. Also drop TODO and an obsolete XXX comment. Signed-off-by: Guenter Roeck --- drivers/staging/typec/tcpm.c | 37 + 1 file changed, 1 insertion(+), 36 deletions(-) diff --git a/drivers/staging/typec/tcpm.c b/d

[PATCH 2/5] staging: typec: tcpm: Document data structures

Document struct tcpc_config and struct tcpc_dev. Drop unused TCPC_USB_SWITCH_RESTORE. Signed-off-by: Guenter Roeck --- drivers/staging/typec/tcpm.h | 57 ++-- 1 file changed, 50 insertions(+), 7 deletions(-) diff --git a/drivers/staging/typec/tcpm.h b/dri

[PATCH 3/5] staging: typec: pd: Document struct pd_message

struct pd_message is the format of a PD message as seen on the wire. Signed-off-by: Guenter Roeck --- drivers/staging/typec/pd.h | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/staging/typec/pd.h b/drivers/staging/typec/pd.h index 30b32ad72acd..42a10883a2cb 100644 --- a/drivers/

Re: [PATCH 4/5] typec: tcpm: Move out of staging

On 09/10/2017 01:46 PM, Joe Perches wrote: On Sun, 2017-09-10 at 13:37 -0700, Guenter Roeck wrote: Move tcpm (USB Type-C Port Manager) out of staging. git format-patch -M Thanks - I'll resend but wait a bit for additional comments before I do. Guenter _

Re: [PATCH v2 03/11] mux: core: Add usb.h header with MUX_USB_* and and MUX_TYPEC_* state constants

On 2017-09-08 19:07, Hans de Goede wrote: > Hi, > > On 08-09-17 17:47, Peter Rosin wrote: >> On 2017-09-05 18:42, Hans de Goede wrote: >>> Add MUX_USB_* and MUX_TYPEC_* state constant defines, which can be used by >>> USB device/host, resp. Type-C polarity/role/altmode mux drivers and >>> consumer

Re: [PATCH 2/5] android: binder: Add allocator selftest

Hi Geert, This selftest can't run until userspace has called mmap. Since it is hooked into ioctl, it's probably not worth the effort of making it a module, since ioctl will have to check at runtime whether the selftest module has been loaded, which would add overhead even when the test is not enab

Re: [PATCH v2 08/11] staging: typec: tcpm: Set mux to device mode when configured as such

On 09/05/2017 09:42 AM, Hans de Goede wrote: Setting the mux to TYPEC_MUX_NONE, TCPC_USB_SWITCH_DISCONNECT when the data-role is device is not correct. Plenty of devices support operating as USB device through a (separate) USB device controller. So this commit instead splits out TYPEC_MUX_USB in

[PATCH v2 1/5] staging: typec: tcpm: Drop commented out code

Commented out code can be added as needed. Drop it. Also drop TODO and an obsolete XXX comment. Signed-off-by: Guenter Roeck --- v2: No change drivers/staging/typec/tcpm.c | 37 + 1 file changed, 1 insertion(+), 36 deletions(-) diff --git a/drivers/staging/t

[PATCH v2 3/5] staging: typec: pd: Document struct pd_message

struct pd_message is the format of a PD message as seen on the wire. Signed-off-by: Guenter Roeck --- v2: No change drivers/staging/typec/pd.h | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/staging/typec/pd.h b/drivers/staging/typec/pd.h index 30b32ad72acd..42a10883a2cb 100644

[PATCH v2 2/5] staging: typec: tcpm: Document data structures

Document struct tcpc_config and struct tcpc_dev. Drop unused TCPC_USB_SWITCH_RESTORE. Signed-off-by: Guenter Roeck --- v2: No change drivers/staging/typec/tcpm.h | 57 ++-- 1 file changed, 50 insertions(+), 7 deletions(-) diff --git a/drivers/staging/typ

[PATCH v2 4/5] typec: tcpm: Move out of staging

Move tcpm (USB Type-C Port Manager) out of staging. Signed-off-by: Guenter Roeck --- v2: Use format-patch -M drivers/staging/typec/Kconfig | 8 drivers/staging/typec/Makefile| 1 - drivers/staging/typec/TODO|

[PATCH v2 5/5] usb: typec: fusb302: Move out of staging

The driver is in good enough shape to be moved out of staging. Do it. Signed-off-by: Guenter Roeck --- v2: Use format-patch -M drivers/staging/typec/Kconfig| 2 -- drivers/staging/typec/Makefile | 1 - drivers/staging/typec/fusb302/TODO

Re: [PATCH 4/5] typec: tcpm: Move out of staging

On Sun, 2017-09-10 at 13:37 -0700, Guenter Roeck wrote: > Move tcpm (USB Type-C Port Manager) out of staging. git format-patch -M ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-deve

Re: [PATCH 2/5] vmbus: suppress uevents for hv_sock devices

On Sat, Sep 09, 2017 at 11:08:46PM -0700, k...@exchange.microsoft.com wrote: > From: Dexuan Cui > > hv_sock driver is automatically loaded when an application creates an > AF_VSOCK socket, so we don't really need to trigger uevents to the user > space udevd. > > And hv_sock devices can appear an

[PATCH] Staging: xgifb: XGI_main_26.c: Fix alignment to match parentheses

Fix alignment so that it matches open parentheses. Signed-off-by: Srishti Sharma --- drivers/staging/xgifb/XGI_main_26.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/xgifb/XGI_main_26.c b/drivers/staging/xgifb/XGI_main_26.c index b450c74..b813f1d 10064

Re: [PATCH 2/5] android: binder: Add allocator selftest

Hi Sherry, On Wed, Aug 16, 2017 at 2:25 AM, Sherry Yang wrote: > binder_alloc_selftest tests that alloc_new_buf handles page allocation and > deallocation properly when allocate and free buffers. The test allocates 5 > buffers of various sizes to cover all possible page alignment cases, and > fre

[PATCH] staging: atomisp: add a driver for ov5648 camera sensor

The ov5680 5-megapixel camera sensor from OmniVision supports up to 2592x1944 resolution and MIPI CSI-2 interface. Output format is raw sRGB/Bayer with 10 bits per colour (SGRBG10_1X10). This patch is a port of ov5680 driver from following 01org/ProductionKernelQuilts patches: - 0004-ov2680-ov5648