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

[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 -

[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 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

[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

[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

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

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

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 >>>

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

[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

[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

[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

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(-)

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

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

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

[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

[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

[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

[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

[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

[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

[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

[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

[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

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: > > > >

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

[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

[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

[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

[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: -

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

[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

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

[PATCH] staging: ccree: Add missing newlines

Logging without newlines are still prone to interleaving. Add newlines where necessary. Miscellanea: o Coalesce formats o Realign arguments o Fix a couple misindented lines with the above logging changes Signed-off-by: Joe Perches --- drivers/staging/ccree/ssi_aead.c

[PATCH 1/5] vmbus: don't acquire the mutex in vmbus_hvsock_device_unregister()

From: Dexuan Cui Due to commit 54a66265d675 ("Drivers: hv: vmbus: Fix rescind handling"), we need this patch to resolve the below deadlock: after we get the mutex in vmbus_hvsock_device_unregister() and call vmbus_device_unregister() -> device_unregister() -> ... ->

[PATCH 4/5] vmbus: add per-channel sysfs info

From: Stephen Hemminger This extends existing vmbus related sysfs structure to provide per-channel state information. This is useful when diagnosing issues with multiple queues in networking and storage. The existing sysfs only displayed information about the primary

[PATCH 5/5] Drivers: hv: vmbus: Expose per-channel event counters

From: Stephen Hemminger When investigating performance, it is useful to be able to look at the number of host and guest events per-channel. This is equivalent to per-device interrupt statistics. Signed-off-by: Stephen Hemminger Signed-off-by:

[PATCH 3/5] Drivers: hv: fcopy: restore correct transfer length

From: Olaf Hering Till recently the expected length of bytes read by the daemon did depend on the context. It was either hv_start_fcopy or hv_do_fcopy. The daemon had a buffer size of two pages, which was much larger than needed. Now the expected length of bytes read by the

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

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 and disappear frequency, e.g. 100 per second, so triggering the