From: Dexuan Cui
Without the patch, vmbus_hvsock_device_unregister() can destroy the device
prematurely when close() is called, and can cause NULl dereferencing or
potential data loss (the last portion of the data stream may be dropped
prematurely).
Signed-off-by: Dexuan Cui
Cc: Haiyang Zhang
Hi,
On 18-10-17 04:33, Chanwoo Choi wrote:
Hi Hans,
On 2017년 09월 23일 03:37, Hans de Goede wrote:
Cherry Trail SoCs have a built-in USB-role mux for switching between
the host and device controllers, rather then using an external mux
controller by a GPIO.
There is a driver using the mux-subsys
On 2017년 10월 18일 18:14, Hans de Goede wrote:
> Hi,
>
> On 18-10-17 04:33, Chanwoo Choi wrote:
>> Hi Hans,
>>
>> On 2017년 09월 23일 03:37, Hans de Goede wrote:
>>> Cherry Trail SoCs have a built-in USB-role mux for switching between
>>> the host and device controllers, rather then using an external m
-wlan-ng-Remove-unnecessary-parentheses/20171018-180433
config: x86_64-randconfig-x005-201742 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
All warnings (new ones prefixed by
From: Colin Ian King
The variable region_start is assigned twice, the first value is
never read and redundant, so can be removed. We can clean up the
code further by assigning rg_start directly rather than using the
temporary variable region_start which can then be removed. Cleans
up the clang w
On Mon, Oct 16, 2017 at 10:46:08AM +0200, Christian Gromm wrote:
> This patch renames and moves the source files of the driver. Additionally,
> it adapts the ABI, Makefiles and Kconfig files of the kernel's build
> system.
Ok, this says _what_ the patch does, but we know that already by looking
at
On Mon, Oct 16, 2017 at 10:46:09AM +0200, Christian Gromm wrote:
> The following patch adapts the driver to use the device model by:
>
> - adopting the MOST bus_type
> - registering the core as a busdriver
> - removing private kobject/kset usage
> - removing private lists a
On Mon, Oct 16, 2017 at 10:46:24AM +0200, Christian Gromm wrote:
> A module that is meant to make up the userspace experience of the driver
> is no longer referred to as AIM. Instead it is denoted as a component of
> the core. Further, it is understood as an integral part of the core, the
> purpose
On Mon, Oct 16, 2017 at 10:46:07AM +0200, Christian Gromm wrote:
> This patch set integrates the MOST driver to the Linux kernel's device
> model and provides bug fixes. Part of this process is moving and renaming
> files to get a clearer directory layout.
This needs to be reworked a bit better, s
On Thu, Oct 05, 2017 at 03:16:42PM +0530, Srishti Sharma wrote:
> This is a cleanup patch and doesn't change runtime behavior. It
> changes an open coded list traversal to use list_for_each_entry_safe.
> Done using the following semantic patch by coccinelle.
You have sent me two different patches
On Wed, Oct 18, 2017 at 12:45:57AM +0300, Andrii Vladyka wrote:
> check for skb->len in gdm_lte_emulate_arp
>
> Signed-off-by: Andrii Vladyka
> ---
> drivers/staging/gdm724x/gdm_lte.c | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/drivers/staging/gdm724x/gdm_lte.c
> b/drivers/st
fix return codes in gdm_lte in gdm724x driver
Signed-off-by: Andrii Vladyka
---
drivers/staging/gdm724x/gdm_lte.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/gdm724x/gdm_lte.c
b/drivers/staging/gdm724x/gdm_lte.c
index 066b14e..0527b0d 10064
check for skb->len in gdm_lte_emulate_arp in gdm724x driver
Signed-off-by: Andrii Vladyka
---
drivers/staging/gdm724x/gdm_lte.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/staging/gdm724x/gdm_lte.c
b/drivers/staging/gdm724x/gdm_lte.c
index 9ab6ce2..066b14e 100644
--- a/driv
On Thu, Oct 05, 2017 at 04:44:31PM -0700, Matthew Giassa wrote:
> Removing a comment that duplicates definitions for pci_power_t
> enumeration, and pointing to the relevant header file (current comment
> is also missing PCI_POWER_ERROR).
>
> Signed-off-by: Matthew Giassa
> ---
> drivers/staging/
On Sat, Oct 07, 2017 at 10:56:25PM +0530, Srishti Sharma wrote:
> Use setup_timer to combine initialization of a timer with the
> initialization of the timer's function and data fields. Done
> using the following semantic patch by coccinelle.
>
> @r@
> struct timer_list *l;
> expression f, d;
> @@
On Wed, Oct 04, 2017 at 07:21:10PM +0200, Srinivasan Shanmugam wrote:
> Removed all the unnecessary extern from rtl8723bs
That's not what this patch does :(
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailm
On 18.10.2017 14:12, Greg KH wrote:
On Mon, Oct 16, 2017 at 10:46:09AM +0200, Christian Gromm wrote:
The following patch adapts the driver to use the device model by:
- adopting the MOST bus_type
- registering the core as a busdriver
- removing private kobject/kset usage
On Thu, Oct 05, 2017 at 03:45:33PM +0200, Srinivasan Shanmugam wrote:
> Fix parenthesis alignment warning in validate_recv_mgnt_frame()
>
> Signed-off-by: Srinivasan Shanmugam
> ---
> drivers/staging/rtl8723bs/core/rtw_recv.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Patch does no
On Thu, Oct 05, 2017 at 04:38:23PM -0700, Matthew Giassa wrote:
> Explicitly casting pci_power_t types to resolve sparse warnings (shown
> below).
>
> Also fixing a related logging bug where pci_power_t is cast to unsigned
> (can be negative, i.e. PCI_POWER_ERROR).
>
> Original sparse report:
>
On Mon, Oct 16, 2017 at 09:48:21PM -0400, Frank A. Cancio Bello wrote:
> Remove unnecessary parentheses to comply with preferred coding style for
> the linux kernel and avoid the following checkpatch's message:
> 'CHECK: Unnecessary parentheses around'
>
> Credits to checkpatch.
>
> Signed-off-by
On Wed, Oct 18, 2017 at 04:02:33PM +0200, Christian Gromm wrote:
> On 18.10.2017 14:12, Greg KH wrote:
> > On Mon, Oct 16, 2017 at 10:46:09AM +0200, Christian Gromm wrote:
> > > The following patch adapts the driver to use the device model by:
> > >
> > > - adopting the MOST bus_type
> > > - r
On Wed, Oct 18, 2017 at 04:02:33PM +0200, Christian Gromm wrote:
> On 18.10.2017 14:12, Greg KH wrote:
> > On Mon, Oct 16, 2017 at 10:46:09AM +0200, Christian Gromm wrote:
> > > The following patch adapts the driver to use the device model by:
> > >
> > > - adopting the MOST bus_type
> > > - r
From: Boris Brezillon
The core currently send the READ0 and SEQIN+PAGEPROG commands in
nand_do_read/write_ops(). This is inconsistent with
->read/write_oob[_raw]() hooks behavior which are expected to send
these commands.
There's already a flag (NAND_ECC_CUSTOM_PAGE_ACCESS) to inform the core
th
Change the data_interface field from the nand_chip structure, convert
the pointer to a static structure.
Also remove the nand_get_default_data_interface() function that become
useless and rename the onfi_init_data_interface() by
nand_fill_data_interface(), which is a more appropriate name because
Hi,
This series adds the implementation of the NAND framework ->exec_op()
interface with all the related hooks and helpers. The reasons for adding
it are explained in details in the commit log "mtd: nand: add
->exec_op() implementation", but mostly it will ease later expansion of
the framework, a
Change the bindings to use the reworked Marvell NAND controller driver.
Also adapt the nand controller node organization to distinguish which
property is relevant for the controller, and which one is NAND chip
specific. Expose the partitions as a subnode of the NAND chip.
Remove the 'marvell,nand-
Introduce the new way to control the NAND controller drivers by
implementing the ->exec_op() core helpers and allowing new drivers to
use it instead of relying on ->cmd_ctrl(), ->cmdfunc() and
->read/write_byte/word/buf().
The logic is now to send to the controller driver a list of
instructions. T
Add marvell_nand driver which aims at replacing the existing pxa3xx_nand
driver.
The new driver intends to be easier to understand and follows the brand
new NAND framework rules by implementing hooks for every pattern the
controller might support and referencing them inside a parser object
that wi
Change the bindings to use the reworked Marvell NAND controller driver.
Also adapt the nand controller node organization to distinguish which
property is relevant for the controller, and which one is NAND chip
specific. Expose the partitions as a subnode of the NAND chip.
Remove the 'marvell,nand-
Change the bindings to use the reworked Marvell NAND controller driver.
Also adapt the nand controller node organization to distinguish which
property is relevant for the controller, and which one is NAND chip
specific. Expose the partitions as a subnode of the NAND chip.
Remove the 'marvell,nand-
From: Boris Brezillon
This is part of the process of removing direct calls to ->cmdfunc()
outside of the core in order to introduce a better interface to execute
NAND operations.
Here we provide several helpers and make use of them to remove all
direct calls to ->cmdfunc(). This way, we can easi
Change the bindings to use the reworked Marvell NAND controller driver.
Also adapt the nand controller node organization to distinguish which
property is relevant for the controller, and which one is NAND chip
specific. Expose the partitions as a subnode of the NAND chip.
Signed-off-by: Miquel Ray
Document the bindings for the legacy and the new bindings relative to
Marvell NAND controller driver rework.
Signed-off-by: Miquel Raynal
---
.../devicetree/bindings/mtd/marvell-nand.txt | 95 ++
1 file changed, 95 insertions(+)
create mode 100644 Documentation/devicet
Change the bindings to use the reworked Marvell NAND controller driver.
Also adapt the nand controller node organization to distinguish which
property is relevant for the controller, and which one is NAND chip
specific. Expose the partitions as a subnode of the NAND chip.
Remove the 'marvell,nand-
Change the bindings to use the reworked Marvell NAND controller driver.
Also adapt the nand controller node organization to distinguish which
property is relevant for the controller, and which one is NAND chip
specific. Expose the partitions as a subnode of the NAND chip.
Remove the 'marvell,nand-
On Wed, Oct 18, 2017 at 04:17:04PM +0200, Greg KH wrote:
> On Mon, Oct 16, 2017 at 09:48:21PM -0400, Frank A. Cancio Bello wrote:
> > Remove unnecessary parentheses to comply with preferred coding style for
> > the linux kernel and avoid the following checkpatch's message:
> > 'CHECK: Unnecessary p
On Wed, Oct 18, 2017 at 10:46:09AM -0400, Frank A. Cancio Bello wrote:
> On Wed, Oct 18, 2017 at 04:17:04PM +0200, Greg KH wrote:
> > On Mon, Oct 16, 2017 at 09:48:21PM -0400, Frank A. Cancio Bello wrote:
> > > Remove unnecessary parentheses to comply with preferred coding style for
> > > the linux
On Wed, 18 Oct 2017, Srishti Sharma wrote:
>
>
> On Wed, Oct 18, 2017 at 6:01 PM, Greg KH wrote:
> On Thu, Oct 05, 2017 at 03:16:42PM +0530, Srishti Sharma wrote:
> > This is a cleanup patch and doesn't change runtime behavior.
> It
> > changes an open coded list travers
On 18.10.2017 16:35, Dan Carpenter wrote:
On Wed, Oct 18, 2017 at 04:02:33PM +0200, Christian Gromm wrote:
On 18.10.2017 14:12, Greg KH wrote:
On Mon, Oct 16, 2017 at 10:46:09AM +0200, Christian Gromm wrote:
The following patch adapts the driver to use the device model by:
- adopting
On Wed, 18 Oct 2017, Srishti Sharma wrote:
>
>
> On Wed, Oct 18, 2017 at 8:29 PM, Julia Lawall wrote:
>
>
> On Wed, 18 Oct 2017, Srishti Sharma wrote:
>
> >
> >
> > On Wed, Oct 18, 2017 at 6:01 PM, Greg KH
>wrote:
> > On Thu, Oct 05, 2017 at 03:16:42P
Remove unnecessary parentheses to comply with preferred coding style for
the linux kernel and avoid the following checkpatch's messages:
'CHECK: Unnecessary parentheses around'
'CHECK: Logical continuations should be on the previous line'
Credits to checkpatch.
Signed-off-by: Frank A. Cancio Bell
On Fri, 2017-09-29 at 22:41 +0100, Russell King wrote:
> Setting a burst size of "8" doesn't work for IMX219 with 8-bit bayer,
> but a burst size of "16" does. Fix this.
>
> Signed-off-by: Russell King
Oh, well, this isn't for me to apply after all.
Acked-by: Philipp Zabel
regards
Philipp
>
On Wed, 18 Oct 2017 16:36:20 +0200
Miquel Raynal wrote:
> Change the data_interface field from the nand_chip structure, convert
> the pointer to a static structure.
You mean "embed a nand_data_interface object". Not sure the static
specifier can be used to describe what you do.
>
> Also remove
This patch series performs code cleanup like using list_for_each_entry_safe
to make code more compact and using list_entry instead of container_of for
struct list_head* variable types.
Srishti Sharma (4):
Staging: rtl8188eu: core: rtw_xmit: Use list_for_each_entry_safe
Staging: rtl8188eu: co
This is a cleanup patch and doesn't change runtime behaviour. It
changes an open coded list traversal to use list_for_each_entry_safe.
Done using the following semantic patch by coccinelle.
@r@
struct list_head* l;
expression e;
identifier m,list_del_init,f;
type T1;
T1* pos;
iterator name list_fo
This is a cleanup patch and doesn't change runtime behaviour. It
changes an open coded list traversal to use list_for_each_entry_safe.
Done using the following semantic patch by coccinelle.
@r@
struct list_head* l;
expression e;
identifier m,list_del_init,f;
type T1;
T1* pos;
iterator name list_fo
This is a cleanup patch and doesn't change runtime behaviour. It
changes an open coded list traversal to use list_for_each_entry_safe.
Done using the following semantic patch by coccinelle.
@r@
struct list_head* l;
expression e;
identifier m,list_del_init,f;
type T1;
T1* pos;
iterator name list_fo
For variables of the struct list_head* use list_entry to access
current list element instead of using container_of.
Done using the following semantic patch by coccinelle.
@r@
struct list_head* l;
@@
-container_of
+list_entry
(l,...)
Signed-off-by: Srishti Sharma
---
drivers/staging/rtl8188e
Hi,
I have seen a few randconfig builds where FSL_MC_BUS is enabled.
This selects GENERIC_MSI_IRQ_DOMAIN -- even if CONFIG_PCI and
CONFIG_PCI_MSI are not enabled. (This is with COMPILE_TEST enabled.)
I see in the README.txt file that the DPAA2/MC does use some kind of
message interrupts.
Severa
On 09/27/2017 06:20 AM, Benjamin Gaignard wrote:
> diff --git a/drivers/staging/android/ion/ion.c
> b/drivers/staging/android/ion/ion.c
> index 93e2c90..092b24c 100644
> --- a/drivers/staging/android/ion/ion.c
> +++ b/drivers/staging/android/ion/ion.c
> @@ -40,6 +40,8 @@
>
> #include "ion.h"
>
In preparation for unconditionally passing the struct timer_list pointer to
all timer callbacks, switch to using the new timer_setup() and from_timer()
to pass the timer pointer explicitly.
Cc: Greg Kroah-Hartman
Cc: Sergio Paracuellos
Cc: Adrien Descamps
Cc: Thibaut SAUTEREAU
Cc: de...@driver
Hi Suniel,
Well done with you continued versions. I am being particularly nit picky here
but since we are
striving for perfection I'm sure will humour me. If English is not your first
language please
forgive me for picking you up on language subtleties.
On Wed, Oct 18, 2017 at 12:11:55PM +0530,
On Wed, Oct 18, 2017 at 11:48:21AM -0400, Frank A. Cancio Bello wrote:
> Remove unnecessary parentheses to comply with preferred coding style for
> the linux kernel and avoid the following checkpatch's messages:
> 'CHECK: Unnecessary parentheses around'
> 'CHECK: Logical continuations should be on
On Wed, 18 Oct 2017 16:36:21 +0200
Miquel Raynal wrote:
> Introduce the new way to control the NAND controller drivers by
> implementing the ->exec_op() core helpers and allowing new drivers to
> use it instead of relying on ->cmd_ctrl(), ->cmdfunc() and
> ->read/write_byte/word/buf().
"
Introdu
On Wed, 18 Oct 2017 16:36:22 +0200
Miquel Raynal wrote:
> Document the bindings for the legacy and the new bindings relative to
> Marvell NAND controller driver rework.
>
> Signed-off-by: Miquel Raynal
> ---
> .../devicetree/bindings/mtd/marvell-nand.txt | 95
> ++
>
Hi Miquel,
On Wed, 18 Oct 2017 16:36:17 +0200
Miquel Raynal wrote:
> Hi,
>
> This series adds the implementation of the NAND framework ->exec_op()
> interface with all the related hooks and helpers. The reasons for adding
> it are explained in details in the commit log "mtd: nand: add
> ->exec
On Thu, Oct 19, 2017 at 08:40:08AM +1100, Tobin C. Harding wrote:
> On Wed, Oct 18, 2017 at 11:48:21AM -0400, Frank A. Cancio Bello wrote:
> > --- a/drivers/staging/wlan-ng/p80211req.c
> > +++ b/drivers/staging/wlan-ng/p80211req.c
> > @@ -124,7 +124,7 @@ int p80211req_dorequest(struct wlandevice *w
On Wed, Oct 18, 2017 at 06:50:52PM -0400, Frank A. Cancio Bello wrote:
> On Thu, Oct 19, 2017 at 08:40:08AM +1100, Tobin C. Harding wrote:
> > On Wed, Oct 18, 2017 at 11:48:21AM -0400, Frank A. Cancio Bello wrote:
> > > --- a/drivers/staging/wlan-ng/p80211req.c
> > > +++ b/drivers/staging/wlan-ng/p
Without the patch, when hvs_open_connection() hasn't completely established
a connection (e.g. it has changed sk->sk_state to SS_CONNECTED, but hasn't
inserted the sock into the connected queue), vsock_stream_connect() may see
the sk_state change and return the connection to the userspace, and nex
On Tue, 2017-10-10 at 15:05 +0100, Colin King wrote:
> From: Colin Ian King
>
> Shifting and masking strHostIfSetMulti->enabled is redundant since
> enabled is a bool and so all the shifted and masked values will be
> zero. Replace them with zero to simplify the code.
>
> Detected by CoveritySca
On Thursday 19 October 2017 02:24 AM, Tobin C. Harding wrote:
> Hi Suniel,
>
> Well done with you continued versions. I am being particularly nit picky here
> but since we are
> striving for perfection I'm sure will humour me. If English is not your first
> language please
> forgive me for picki
61 matches
Mail list logo
