Re: [PATCH 1/9] staging: ks7010: Replace manual array copy with ether_addr_copy().

On Wed, 2018-03-28 at 22:51 -0700, Quytelda Kahja wrote: > Copying the dummy HW address into the struct net_device doesn't need > to be done byte by byte; use ether_addr_copy() instead. > Additionally, dev->dev_addr is not eight bytes long. > ether_setup() sets the dev->addr_len to ETH_ALEN (define

Re: [PATCH 9/9] staging: ks7010: Remove extra blank line between functions.

On Wed, 2018-03-28 at 22:51 -0700, Quytelda Kahja wrote: > Remove an extra blank line indicated by checkpatch. The changelog doesn't match the actual changes made. > diff --git a/drivers/staging/ks7010/ks7010_sdio.c > b/drivers/staging/ks7010/ks7010_sdio.c [] > @@ -305,7 +305,8 @@ static void tx

[PATCH 9/9] staging: ks7010: Remove extra blank line between functions.

Remove an extra blank line indicated by checkpatch. Signed-off-by: Quytelda Kahja --- drivers/staging/ks7010/ks7010_sdio.c | 3 ++- drivers/staging/ks7010/ks_hostif.c | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ks7010/ks7010_sdio.c b/drivers/staging/k

[PATCH 7/9] staging: ks7010: Remove trailing "_t" from all structure names.

The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Remove the "_t" suffix from every structure defined in this driver. Signed-off-by: Quytelda Kahja --- drivers/staging/ks7010/ks7010_sdio.c | 2 +- drive

[PATCH 8/9] staging: ks7010: Remove 'eth_addr' field from 'struct ks_wlan_private'.

The ethernet address of the network device is already stored in the 'dev_addr' field of 'struct net_device'. Since 'struct ks_wlan_private' keeps a pointer to the driver's 'struct net_device', there is no reason to duplicate this information in 'struct ks_wlan_private'. Signed-off-by: Quytelda Ka

[PATCH 5/9] staging: ks7010: Change mac_address_valid to a bool instead of int.

'mac_address_valid' is only ever assigned 0 or 1, so it makes more sense to use a bool type for this variable. Signed-off-by: Quytelda Kahja --- drivers/staging/ks7010/ks_hostif.c | 4 ++-- drivers/staging/ks7010/ks_wlan.h | 2 +- drivers/staging/ks7010/ks_wlan_net.c | 4 ++-- 3 files chan

[PATCH 1/9] staging: ks7010: Replace manual array copy with ether_addr_copy().

Copying the dummy HW address into the struct net_device doesn't need to be done byte by byte; use ether_addr_copy() instead. Additionally, dev->dev_addr is not eight bytes long. ether_setup() sets the dev->addr_len to ETH_ALEN (defined as 6) in the net core code. Signed-off-by: Quytelda Kahja ---

[PATCH 6/9] staging: ks7010: Remove unused member 'reg_net' from 'ks_wlan_private'.

'reg_net' is never used in this driver. Signed-off-by: Quytelda Kahja x --- drivers/staging/ks7010/ks_wlan.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/ks7010/ks_wlan.h b/drivers/staging/ks7010/ks_wlan.h index 846bb1041850..1b7036c32d1c 100644 --- a/drivers/staging/ks7010/

[PATCH 4/9] staging: ks7010: Rename ks_wlan_set_multicast_list()

All of the net_device_ops callbacks are named after their counterparts in the kernel's 'struct net_device_ops', except ks_wlan_set_multicast_list(). Rename it to ks_wlan_set_rx_mode() for greater consistency. Signed-off-by: Quytelda Kahja --- drivers/staging/ks7010/ks_wlan_net.c | 6 +++--- 1 f

[PATCH 3/9] staging: ks7010: Reorder ks_wlan_netdev_ops members.

Reorder the members of 'ks_wlan_netdev_ops' to reflect the order of their counterparts in the kernel's 'struct net_device_ops'. Signed-off-by: Quytelda Kahja --- drivers/staging/ks7010/ks_wlan_net.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/stagin

[PATCH 2/9] staging: ks7010: Remove unecessary cast.

The driver casts '&ks_wlan_handler_def' to 'struct iw_handler_def *', but it is already of that type. Signed-off-by: Quytelda Kahja --- drivers/staging/ks7010/ks_wlan_net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ks7010/ks_wlan_net.c b/drivers/staging

Re: [PATCH] staging: vt6655: check for memory allocation failures

On Wed, Mar 28, 2018 at 05:55:57PM +0800, Jia-Ju Bai wrote: > >@@ -646,7 +649,8 @@ static void device_init_td1_ring(struct vnt_private > >*priv) > > i++, curr += sizeof(struct vnt_tx_desc)) { > > desc = &priv->apTD1Rings[i]; > > desc->td_info = kzalloc(sizeof(*desc

RE: [PATCH 1/1] Drivers: hv: vmbus: Fix ring buffer signaling

Thanks, I hadn't updated to rc7 yet -Original Message- From: Joshua R. Poulson Sent: Wednesday, March 28, 2018 12:18 PM To: Stephen Hemminger Cc: K. Y. Srinivasan ; KY Srinivasan ; gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de;

RE: [Resend Patch 1/3] Vmbus: Add function to report available ring buffer to write in total ring size percentage

Maybe best through greg's char-misc tree since it has generic hv code and sometime updates both network and scsi. Alternatively, put common code through one tree, and hold off the network device change till next release. -Original Message- From: Long Li Sent: Wednesday, March 28, 2018

RE: [Resend Patch 1/3] Vmbus: Add function to report available ring buffer to write in total ring size percentage

> Subject: Re: [Resend Patch 1/3] Vmbus: Add function to report available ring > buffer to write in total ring size percentage > > > Long, > > > Netvsc has a function to calculate how much ring buffer in percentage > > is available to write. This function is also useful for storvsc and > > other

[PATCH 2/2] staging: rtl8723bs: Remove yield call, replace with cond_resched()

Remove yield() call. Yield does not guarantee progress, cond_resched() is a safer alternative Signed-off-by: Thomas Avery --- drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c b/drivers/s

[PATCH 1/2] staging: rtl8723bs: Replace yield() call with cond_resched()

Remove yield call(). yield does not guarantee progress, and should not be used. cond_resched() is a safe alternative. Signed-off-by: Thomas Avery --- drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/core/r

Re: [Resend Patch 1/3] Vmbus: Add function to report available ring buffer to write in total ring size percentage

Long, > Netvsc has a function to calculate how much ring buffer in percentage is > available to write. This function is also useful for storvsc and other > vmbus devices. What is the submission strategy for this series? Do you expect it to go through net or scsi? If the latter, I'll need an ack

[PATCH] staging: mt7621-eth: Fix sparse warning in ethtool.c

Include the local ethtool.h header file in mtk_eth_soc.h so implementation files have centralized access to it. This fixes the following sparse warning: drivers/staging/mt7621-eth/ethtool.c:213:6: warning: symbol 'mtk_set_ethtool_ops' was not declared. Should it be static? Signed-off-by: Chris C

Re: [PATCH 1/1] Drivers: hv: vmbus: Fix ring buffer signaling

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v4.16-rc7&id=655296c8bbeffcf020558c4455305d597a73bde1 On Wed, Mar 28, 2018 at 12:01 PM, Stephen Hemminger wrote: > On Sun, 4 Mar 2018 22:24:08 -0700 > k...@exchange.microsoft.com wrote: > >> From: Michael Kelley >> >>

Re: [PATCH 1/1] Drivers: hv: vmbus: Fix ring buffer signaling

On Sun, 4 Mar 2018 22:24:08 -0700 k...@exchange.microsoft.com wrote: > From: Michael Kelley > > Fix bugs in signaling the Hyper-V host when freeing space in the > host->guest ring buffer: > > 1. The interrupt_mask must not be used to determine whether to signal >on the host->guest ring buf

[PATCH v4 07/13] staging: typec: tcpci: register port before request irq

With that we can clear any pending events and the port is registered so driver can be ready to handle typec events once we request irq. Signed-off-by: Peter Chen Signed-off-by: Li Jun --- drivers/staging/typec/tcpci.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff -

[PATCH v4 12/13] staging: typec: tcpci: Only touch target bit when enable vconn

We need regmap_update_bits to avoid touch any other bits when enable or disable vconn. Reviewed-by: Guenter Roeck Signed-off-by: Li Jun --- drivers/staging/typec/tcpci.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/staging/typec/tcpci.c b/drivers/staging/

[PATCH v4 13/13] staging: typec: tcpci: move tcpci driver out of staging

Move TCPCI(Typec port controller interface) driver out of staging. Signed-off-by: Li Jun --- drivers/staging/Kconfig| 2 -- drivers/staging/Makefile | 1 - drivers/staging/typec/Kconfig | 14 -- drivers/staging/typec/Makefile | 1 - d

[PATCH v4 06/13] usb: typec: tcpm: support get typec and pd config from device properties

This patch adds support of get typec and power delivery config from firmware description. Signed-off-by: Li Jun --- drivers/usb/typec/tcpm.c | 140 ++- 1 file changed, 115 insertions(+), 25 deletions(-) diff --git a/drivers/usb/typec/tcpm.c b/drivers/

[PATCH v4 04/13] usb: typec: add fwnode to tcpc

Add fwnode handle to get the fwnode so we can get typec configs it contains. Suggested-by: Heikki Krogerus Signed-off-by: Li Jun --- drivers/staging/typec/tcpci.c | 14 +++--- drivers/usb/typec/tcpm.c | 1 + include/linux/usb/tcpm.h | 2 ++ 3 files changed, 10 insertions(+),

[PATCH v4 08/13] staging: typec: tcpci: enable vbus detection

TCPCI implementation may need SW to enable VBUS detection to generate power status events. Reviewed-by: Guenter Roeck Signed-off-by: Li Jun --- drivers/staging/typec/tcpci.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/staging/typec/tcpci.c b/drivers/staging/typec/tcpci.c i

[PATCH v4 09/13] typec: tcpm: add starting value for drp toggling

As DRP port autonomously toggles the Rp/Rd need a start value to begin with, so add one parameter for it in tcpm_start_drp_toggling. Reviewed-by: Guenter Roeck Signed-off-by: Li Jun --- drivers/usb/typec/tcpm.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drive

[PATCH v4 11/13] staging: typec: tcpci: keep the not connecting cc line open

While set polarity, we should keep the not connecting cc line to be open. Signed-off-by: Li Jun --- drivers/staging/typec/tcpci.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/drivers/staging/typec/tcpci.c b/drivers/staging/typec/tcpci.c index d5b4e4e..b

[PATCH v4 03/13] staging: typec: tcpci: add compatible string for nxp ptn5110

Add nxp ptn5110 typec controller compatible string: usb-tcpci,ptn5110, which is a standard tcpci chip with power delivery support. Signed-off-by: Li Jun --- drivers/staging/typec/tcpci.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/typec/tcpci.c b/drivers/staging/typec/tcp

[PATCH v4 10/13] usb: typec: tcpm: set cc for drp toggling attach

In case of drp toggling, we may need set correct cc value for role control after attach as it may never been set. Signed-off-by: Li Jun --- drivers/usb/typec/tcpm.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/usb/typec/tcpm.c b/drivers/usb/typec/tcpm.c index 218c230..72d4232

[PATCH v4 05/13] usb: typec: add API to get typec basic port power and data config

This patch adds 3 APIs to get the typec port power and data type, and preferred power role by its name string. Suggested-by: Heikki Krogerus Signed-off-by: Li Jun --- drivers/usb/typec/class.c | 52 +++ include/linux/usb/typec.h | 3 +++ 2 files chan

[PATCH v4 00/13] staging: typec: tcpci: move out of staging

This patch set attempts to move the tcpci driver out of staging by fix some tcpci driver issues and define typec and power delivery device properties, the changes are verified on NXP PTN5110, which is a standard tcpci typec port controller device with power delivery support, tested power source and

[PATCH v4 02/13] dt-bindings: usb: add documentation for typec port controller(TCPCI)

TCPCI stands for typec port controller interface, its implementation has full typec port control with power delivery support, it's a standard i2c slave with GPIO input as irq interface, detail see spec "Universal Serial Bus Type-C Port Controller Interface Specification Revision 1.0, Version 1.1"

[PATCH v4 01/13] dt-bindings: connector: add properties for typec

Add bingdings supported by current typec driver, so user can pass all those properties via dt. Signed-off-by: Li Jun --- .../bindings/connector/usb-connector.txt | 39 ++ 1 file changed, 39 insertions(+) diff --git a/Documentation/devicetree/bindings/connector/usb-

Re: [PATCH v2 02/12] staging: ks7010: avoid camel cases in MichaelMICFunction

On Wed, Mar 28, 2018 at 1:49 PM, Greg KH wrote: > On Fri, Mar 23, 2018 at 02:05:10PM +0100, Sergio Paracuellos wrote: >> This commit replace camel cases for name and params used in >> MichaelMICFunction. This improves a bit readability. >> >> Signed-off-by: Sergio Paracuellos >> --- >> drivers/s

[PATCH 07/12] staging: ks7010: replace GetUInt32 macro with get_unaligned_le32

This commit replaces custom GetUInt32 macro with get_unaligned_le32 which is included in the linux kernel. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/staging/ks7010/michael_mic

[PATCH 09/12] staging: ks7010: remove some dead code from ks_wlan_set_essid function

This commit removes death code which is not being used at all. The statements which are contained inside the else block of preprocessor #if 1 directive are no sense. Also remove #if 1 preprocessor stuff just because it is just true and being executed always. This change improves a bit readability o

[PATCH 06/12] staging: ks7010: replace PutUInt32 macro with put_unaligned_le32()

This commit replaces PutUInt32 custom macro with put_unaligned_le32 function included in the linux kernel. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 14 +++--- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/drivers/staging/ks7010/micha

[PATCH 02/12] staging: ks7010: avoid camel cases for MichaelInitFunction

This commit avoid camel cases in MichaelInitFunction signature and params renaming it to michael_init. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/staging/ks7010/michael_mic.c

[PATCH 12/12] staging: ks7010: replace uint8_t in favour of u8 in michael_get_mic

This commit replaces uint8_t parameter for preferred one u8 in michael_get_mic function. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ks7010/michael_mic.c b/drivers/staging/ks7010

[PATCH 08/12] staging: ks7010: replace MichaelBlockFunction macro with inline function

This commit replaces MichaelBlockFunction macro with similar inline function renaming it to michael_block. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 31 --- 1 file changed, 16 insertions(+), 15 deletions(-) diff --git a/drivers/stag

[PATCH 11/12] staging: ks7010: replace uint8_t in favour of u8 in michael_append

This commit replaces param which is uint8_t in michael_append function in favour of preferred one u8. It also removes no more needed casts when calling this function. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletio

[PATCH 10/12] staging: ks7010: replace uint8_t in favour of u8 in michael_init

This commit replaces uint8_t for preferred one u8 in parameter of michael_init function. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ks7010/michael_mic.c b/drivers/staging/ks7010/m

[PATCH 00/12] staging: ks7010: Some clean ups

This are some clean ups for this driver. Changes are as follows: - Some camel cases fixes - remove some macros to use inline functions or functions which exists in linux kernel itself - Use u8 types instead of uint8_t There was a previous patch series including some of these patches which wa

[PATCH 04/12] staging: ks7010: replace macro MichaelClear with inline function

This commit replaces MichaelClear macro with similar inline function renaming it to michael_clear. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/ks7010/michael_mic.c

[PATCH 05/12] staging: ks7010: avoid camel cases in MichaelGetMIC function

This commit avoid camel cases in MichaelGetMIC function and params renaming it to michael_get_mic. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/drivers/staging/ks7010/m

[PATCH 03/12] staging: ks7010: avoid camel cases in MichaelAppend function

This commit avoid camel cases in MichaelAppend function and params renaming it to michael_append. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/michael_mic.c | 45 ++-- 1 file changed, 22 insertions(+), 23 deletions(-) diff --git a/drivers/staging/

[PATCH 01/12] staging: ks7010: avoid camel cases in MichaelMICFunction

This commit replace camel cases for name and params used in MichaelMICFunction. This improves a bit readability. Signed-off-by: Sergio Paracuellos --- drivers/staging/ks7010/ks_hostif.c | 24 drivers/staging/ks7010/michael_mic.c | 15 +++ drivers/staging/ks

Re: [PATCH 12/18] media: staging: atomisp: avoid a warning if 32 bits build

Em Wed, 28 Mar 2018 17:13:29 +0300 Dan Carpenter escreveu: > On Mon, Mar 26, 2018 at 05:10:45PM -0400, Mauro Carvalho Chehab wrote: > > Checking if a size_t value is bigger than ULONG_INT only makes > > sense if building on 64 bits, as warned by: > > > > drivers/staging/media/atomisp/platfor

Re: [PATCH v2 02/12] staging: ks7010: avoid camel cases in MichaelMICFunction

On Wed, Mar 28, 2018 at 1:49 PM, Greg KH wrote: > On Fri, Mar 23, 2018 at 02:05:10PM +0100, Sergio Paracuellos wrote: >> This commit replace camel cases for name and params used in >> MichaelMICFunction. This improves a bit readability. >> >> Signed-off-by: Sergio Paracuellos >> --- >> drivers/s

Re: [PATCH 12/18] media: staging: atomisp: avoid a warning if 32 bits build

On Mon, Mar 26, 2018 at 05:10:45PM -0400, Mauro Carvalho Chehab wrote: > Checking if a size_t value is bigger than ULONG_INT only makes > sense if building on 64 bits, as warned by: > > drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c:697 > gmin_get_config_var() warn

Re: Mts & Bond

I have a very Important Information for you, can you please contact me, I look forward for your response Thank you. Sincerely Mr.Peter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-

Re: [PATCH v3 2/6] staging: ks7010: remove some dead code from ks_wlan_set_essid function

On Mon, Mar 26, 2018 at 06:52:48PM +0200, Sergio Paracuellos wrote: > This commit removes death code which is not being used at all because > it is inside a preprocessor if 0 directive. This change improves a bit > readability of ks_wlan_set_essid function. That's not what your patch does: > > S

Re: [PATCH 1/2] staging: ks7010: Remove trailing "_t" from all structure names.

On Fri, Mar 23, 2018 at 11:40:42PM -0700, Quytelda Kahja wrote: > The "_t" suffix is not needed for structure names in this driver, > and is a reflection of an older typedef system that is no longer > in place. Remove the "_t" suffix from every structure defined in this > driver. > > Signed-off-b

Re: [PATCH v2 02/12] staging: ks7010: avoid camel cases in MichaelMICFunction

On Fri, Mar 23, 2018 at 02:05:10PM +0100, Sergio Paracuellos wrote: > This commit replace camel cases for name and params used in > MichaelMICFunction. This improves a bit readability. > > Signed-off-by: Sergio Paracuellos > --- > drivers/staging/ks7010/ks_hostif.c | 24 ---

Re: [PATCH v2 09/12] staging: ks7010: replace MichaelBlockFunction macro with inline function

On Fri, Mar 23, 2018 at 02:05:17PM +0100, Sergio Paracuellos wrote: > This commit replaces MichaelBlockFunction macro with similar > inline function renaming it to michael_block. > > Signed-off-by: Sergio Paracuellos > --- > drivers/staging/ks7010/michael_mic.c | 30 +++-

Re: [PATCH v2 07/12] staging: ks7010: replace PutUInt32 macro with an inline function

On Fri, Mar 23, 2018 at 02:05:15PM +0100, Sergio Paracuellos wrote: > This commit replaces PutUInt32 macro in favour of inline function > renaming it to put_uint32. > > Signed-off-by: Sergio Paracuellos > --- > drivers/staging/ks7010/michael_mic.c | 18 +- > 1 file changed, 9 ins

Re: [PATCH v2 08/12] staging: ks7010: replace GetUInt32 macro with inline function

On Fri, Mar 23, 2018 at 02:05:16PM +0100, Sergio Paracuellos wrote: > This commit replaces GetUInt32 with inline function renaming > it to get_uint32. > > Signed-off-by: Sergio Paracuellos > --- > drivers/staging/ks7010/michael_mic.c | 15 +-- > 1 file changed, 9 insertions(+), 6 del

Re: [PATCH] staging: wilc1000: replace kmalloc + memcpy with kmemdup

On Mon, Mar 26, 2018 at 06:16:29PM +0100, Colin King wrote: > From: Colin Ian King > > Replace several allocation and memcpys with kmemdup and add in some > missing memory allocation failure checks. Also fix an incorrect > -EFAULT return with -ENOMEM. > > Signed-off-by: Colin Ian King > --- >

Re: [PATCH v2 1/9] staging: wilc1000: remove unused global variables related to p2p

On Mon, Mar 26, 2018 at 05:15:55PM +0530, Ajay Singh wrote: > Cleanup patch to remove the unused global variables defined for p2p. > > Signed-off-by: Ajay Singh > Reviewed-by: Claudiu Beznea > --- > drivers/staging/wilc1000/host_interface.c | 59 > --- > 1 file chan

Re: [PATCH] ANDROID: binder: prevent transactions into own process.

On Wed, Mar 28, 2018 at 1:29 PM, Greg KH wrote: > I can mark it for stable, and then when you get the "this did not apply > to this tree" email, you can send a backported patch to me so I know to > take that one then. Ack, thanks. > > thanks, > > greg k-h

Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

On Wed, Mar 28, 2018 at 1:28 PM, Greg KH wrote: > What is different from "v2" you sent before this? No change information > from v1? Sorry I messed this up - the first resend did not have "v2" in the subject but did contain v2 change information, the second resend had the v2 subject and did not

Re: [PATCH 00/11] staging: wilc1000: fix for checkpatch and handled malloc memory properly

On Tue, Mar 27, 2018 at 12:52:13PM +0530, Ajay Singh wrote: > > Please let me know, in case I have to rework and resubmit this patch > series to make them into staging branch. You already sent a v2 series for this, right? Why respond to the v1 set? confused, greg k-h __

Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

On Wed, Mar 28, 2018 at 11:14:50AM +0200, Martijn Coenen wrote: > This can't happen with normal nodes (because you can't get a ref > to a node you own), but it could happen with the context manager; > to make the behavior consistent with regular nodes, reject > transactions into the context manager

Re: [PATCH] ANDROID: binder: prevent transactions into own process.

On Wed, Mar 28, 2018 at 11:06:54AM +0200, Martijn Coenen wrote: > On Wed, Mar 28, 2018 at 10:19 AM, Greg KH wrote: > > Does this need to go to older kernels as well? > > Yes, this should apply cleanly to 4.14 as well. It won't apply to > pre-4.14 kernels because of the fine-grained locking change

[PATCH v3] staging: fsl-dpaa2/ethsw: Fix tag control information value overwrite

The tag control information (TCI) part of the VLAN header contains several fields, including PCP (priority code point) and PVID (port VLAN id). Current implementation uses function ethsw_port_set_tci() to set the PVID value and mistakenly overwrites the rest of the TCI fields with 0, including PCP

Re: [PATCH] staging: fsl-dpaa2/ethsw: Fix TCI values overwrite

://github.com/0day-ci/linux/commits/Razvan-Stefanescu/staging-fsl-dpaa2-ethsw-Fix-TCI-values-overwrite/20180328-154703 config: i386-allmodconfig (attached as .config) compiler: gcc-7 (Debian 7.3.0-1) 7.3.0 reproduce: # save the attached .config to linux build tree make ARCH=i386 All

Re: [PATCH] staging: vt6655: check for memory allocation failures

On 2018/3/28 14:31, Ji-Hun Kim wrote: There are no null pointer checking on rd_info and td_info values which are allocated by kzalloc. It has potential null pointer dereferencing issues. Add return when allocation is failed. Signed-off-by: Ji-Hun Kim --- drivers/staging/vt6655/device_main.c

Re: [PATCH 07/18] media: staging: atomisp: fix endianess issues

Em Tue, 27 Mar 2018 14:02:55 +0300 Andy Shevchenko escreveu: > On Mon, 2018-03-26 at 17:10 -0400, Mauro Carvalho Chehab wrote: > > There are lots of be-related warnings there, as it doesn't properly > > mark what data uses bigendian. > > > @@ -107,7 +107,7 @@ mt9m114_write_reg(struct i2c_clien

[PATCH v2] ANDROID: binder: prevent transactions into own process.

This can't happen with normal nodes (because you can't get a ref to a node you own), but it could happen with the context manager; to make the behavior consistent with regular nodes, reject transactions into the context manager by the process owning it. Reported-by: syzbot+09e05aba06723a94d...@syz

[v2] ANDROID: binder: prevent transactions into own process.

This can't happen with normal nodes (because you can't get a ref to a node you own), but it could happen with the context manager; to make the behavior consistent with regular nodes, reject transactions into the context manager by the process owning it. Reported-by: syzbot+09e05aba06723a94d...@syz

Re: [PATCH] ANDROID: binder: prevent transactions into own process.

On Wed, Mar 28, 2018 at 10:19 AM, Greg KH wrote: > Does this need to go to older kernels as well? Yes, this should apply cleanly to 4.14 as well. It won't apply to pre-4.14 kernels because of the fine-grained locking changes, but the same issue exists there and I suspect it would cause the same c

Re: [PATCH] staging: vt6655: check for memory allocation failures

On Wed, Mar 28, 2018 at 03:31:31PM +0900, Ji-Hun Kim wrote: > There are no null pointer checking on rd_info and td_info values which > are allocated by kzalloc. It has potential null pointer dereferencing > issues. Add return when allocation is failed. > > Signed-off-by: Ji-Hun Kim > --- > drive

Re: [PATCH] ANDROID: binder: prevent transactions into own process.

On Wed, Mar 28, 2018 at 09:29:03AM +0200, Martijn Coenen wrote: > This can't happen with normal nodes (because you can't get a ref > to a node you own), but it could happen with the context manager; > to make the behavior consistent with regular nodes, reject > transactions into the context manager

[PATCH] ANDROID: binder: prevent transactions into own process.

This can't happen with normal nodes (because you can't get a ref to a node you own), but it could happen with the context manager; to make the behavior consistent with regular nodes, reject transactions into the context manager by the process owning it. Reported-by: syzbot+09e05aba06723a94d...@syz

RE: [PATCH] staging: fsl-dpaa2/ethsw: Fix TCI values overwrite

> -Original Message- > From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- > ow...@vger.kernel.org] On Behalf Of Andrew Lunn > Sent: Tuesday, March 27, 2018 4:38 PM > To: Razvan Stefanescu > Cc: gre...@linuxfoundation.org; de...@driverdev.osuosl.org; linux- > ker...@vger.kerne