Re: [PATCH v4 4/5] staging/android: refactor SYNC_IOC_FILE_INFO

2016-02-29 Thread Dan Carpenter
On Fri, Feb 26, 2016 at 03:31:46PM -0300, Gustavo Padovan wrote: > +no_fences: > + info->num_fences = sync_file->num_fences; > > - if (copy_to_user((void __user *)arg, info, len)) > + if (copy_to_user((void __user *)arg, info, sizeof(*info))) > ret = -EFAULT; >

Re: [PATCH v4 1/5] staging/android: add num_fences field to struct sync_file_info

2016-02-29 Thread Dan Carpenter
This breaks userspace. You used to be able to figure it out from info->len - sizeof(struct sync_file_info). regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH] Staging: slicoss: change memory allocation style in slicoss.c

2016-02-29 Thread Dan Carpenter
You can't send two patches with the same subject. Just combine them into one patch anyway. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH net-next] hv_netvsc: add ethtool support for set and get of settings

2016-02-29 Thread David Miller
From: Ben Hutchings Date: Mon, 29 Feb 2016 22:34:38 + > On Mon, 2016-02-29 at 17:09 -0500, David Miller wrote: >> From: Simon Xiao >> Date: Thu, 25 Feb 2016 15:24:08 -0800 >> >> > This patch allows the user to set and retrieve speed and duplex of

Re: [PATCH net-next] hv_netvsc: add ethtool support for set and get of settings

2016-02-29 Thread Ben Hutchings
On Mon, 2016-02-29 at 17:09 -0500, David Miller wrote: > From: Simon Xiao > Date: Thu, 25 Feb 2016 15:24:08 -0800 > > > This patch allows the user to set and retrieve speed and duplex of the > > hv_netvsc device via ethtool. > >  > > Example: > > $ ethtool eth0 > > Settings

Re: [PATCH net-next] hv_netvsc: add ethtool support for set and get of settings

2016-02-29 Thread David Miller
From: Simon Xiao Date: Thu, 25 Feb 2016 15:24:08 -0800 > This patch allows the user to set and retrieve speed and duplex of the > hv_netvsc device via ethtool. > > Example: > $ ethtool eth0 > Settings for eth0: > ... > Speed: Unknown! > Duplex: Unknown! (255) > ...

Re: [PATCH] staging/android: refactor SYNC_IOC_FILE_INFO

2016-02-29 Thread Gustavo Padovan
Hi Maarten, 2016-02-29 Maarten Lankhorst : > Op 26-02-16 om 22:00 schreef Gustavo Padovan: > > From: Gustavo Padovan > > > > Change SYNC_IOC_FILE_INFO behaviour to avoid future API breaks and > > optimize buffer allocation. In

Re: [patch] PCI: hv: potential use after free

2016-02-29 Thread Dan Carpenter
You're right. My bad. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

INFORMAÇÃO IMPORTANTE RE: Consultoria de Investimento em sua Localidade.

2016-02-29 Thread Alecssandro
Oi Amigo, INFORMAÇÃO IMPORTANTE Nossa família está interessada em investir fundos em sua localidade. Mais informações para você se interessou. Saudações, Sir Henry Neville Lindley Keswick Presidente da Jardine Matheson Holdings Ltd https://en.wikipedia.org/wiki/Keswick_family

[PATCH v2] staging: unisys: Update MAINTAINERS file

2016-02-29 Thread David Kershner
Benjamin Romer is no longer an s-Par maintainer. Changes from v1: - Fixed spelling in comment of my name. - Obtained signoff from Benjamin Romer (currently at bro...@comcast.net) David Kershner (1): staging: unisys: Update MAINTAINERS file drivers/staging/unisys/MAINTAINERS | 2 +- 1 file

[PATCH v2] staging: unisys: Update MAINTAINERS file

2016-02-29 Thread David Kershner
Benjamin Romer is no longer a maintainer for the s-Par drivers. David Kershner will now be the maintainer. Signed-off-by: David Kershner Signed-off-by: Benjamin Romer --- drivers/staging/unisys/MAINTAINERS | 2 +- 1 file changed, 1

RE: [patch] PCI: hv: potential use after free

2016-02-29 Thread Jake Oshins
> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Saturday, February 27, 2016 2:44 AM > To: KY Srinivasan ; Jake Oshins > > Cc: Haiyang Zhang ; Bjorn Helgaas > ;

Re: FW: [PATCH] staging: unisys: Update MAINTAINERS file

2016-02-29 Thread Ben Romer
On Mon, 2016-02-29 at 17:02 +, Kershner, David A wrote: > > On Mon, Feb 29, 2016 at 10:24:06AM -0500, David Kershner wrote: > > > Benjamin Romer is no longer a mainta.iner for the s-Par drivers. David > > > Kershenr will now be the maintainer > > > Signed-off-by: David Kershner

INFORMAÇÃO IMPORTANTE RE: Consultoria de Investimento em sua Localidade.

2016-02-29 Thread Alecssandro
Oi Amigo, INFORMAÇÃO IMPORTANTE Nossa família está interessada em investir fundos em sua localidade. Mais informações para você se interessou. Saudações, Sir Henry Neville Lindley Keswick Presidente da Jardine Matheson Holdings Ltd https://en.wikipedia.org/wiki/Keswick_family

INFORMAÇÃO IMPORTANTE RE: Consultoria de Investimento em sua Localidade.

2016-02-29 Thread Alecssandro
Oi Amigo, INFORMAÇÃO IMPORTANTE Nossa família está interessada em investir fundos em sua localidade. Mais informações para você se interessou. Saudações, Sir Henry Neville Lindley Keswick Presidente da Jardine Matheson Holdings Ltd https://en.wikipedia.org/wiki/Keswick_family

[PATCH] staging: unisys: Update MAINTAINERS file

2016-02-29 Thread David Kershner
Benjamin Romer is no longer a maintainer for the s-Par drivers. David Kershenr will now be the maintainer. Signed-off-by: David Kershner --- drivers/staging/unisys/MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] staging: nvec: Fixed 12 code style checks and changed udelay() to usleep_range() on 2 lines

2016-02-29 Thread Parth Sane
Hi, Sure thanks for pointing that out. I’ll send in a small patch for the problems in the following order. 1) Null Comparison 2) License declaration Hopefully maybe you guys can tell me what else is needed. Thanks for being so supportive! Regards, Parth Sane > On 29-Feb-2016, at 5:00 PM, Marc

Re: [PATCH] staging: nvec: Fixed 12 code style checks and changed udelay() to usleep_range() on 2 lines

2016-02-29 Thread Marc Dietrich
Hi, Am Montag, 29. Februar 2016, 17:21:14 CET schrieb Parth Sane: > Hi, > As per the previous email, I understand that some patches were > unnecessary(For example the fix for the spaces ). not sure what you mean here. I was referring to the BUG_ON removal not the space changes. It's common (or

INFORMAÇÃO IMPORTANTE RE: Consultoria de Investimento em sua Localidade.

2016-02-29 Thread Mr. DOUGLAS
Oi, Temos planejado investir alguns fundos no seu país. Responder se você estiver interessado. Saudações, Sir Henry Neville Lindley Keswick Presidente da Jardine Matheson Holdings Ltd https://en.wikipedia.org/wiki/Keswick_family == Hi friend,

[PATCH] staging: lowmemorykiller: fix 2 checks that checkpatch complained

2016-02-29 Thread sandeepjain . linux
From: Sandeep Jain Specifically: lowmemorykiller.c:53: CHECK: use a blank line after enum declarations lowmemorykiller.c:60: CHECK: use a blank line after enum declarations Signed-off-by: Sandeep Jain ---

Re: [PATCH] staging: nvec: Fixed 12 code style checks and changed udelay() to usleep_range() on 2 lines

2016-02-29 Thread Parth Sane
Hi, As per the previous email, I understand that some patches were unnecessary(For example the fix for the spaces ). But it was a bit confusing. Can someone please let me know which patches are needed and which are not? Regards, Parth Sane > On 29-Feb-2016, at 5:00 PM, Marc Dietrich

[PATCH] Staging: slicoss: changes memory allocation style in slicoss.c

2016-02-29 Thread Ben Marsh
This is a patch to slicoss.c that changes the memory allocation style as identified by checkpatch.pl Signed-off-by: Ben Marsh --- drivers/staging/slicoss/slicoss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/slicoss/slicoss.c

[PATCH] Staging: slicoss: memory allocation style change in slicoss.c

2016-02-29 Thread Ben Marsh
This is a patch to slicoss.c to change the memory allocation style as identified by checkpatch.pl Signed-off-by: Ben Marsh --- drivers/staging/slicoss/slicoss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/slicoss/slicoss.c

Re: [PATCH] staging: nvec: Fixed 12 code style checks and changed udelay() to usleep_range() on 2 lines

2016-02-29 Thread Marc Dietrich
Hi Parth, please also cc de...@linuxdriverproject.org next time. Some comments below. Am Freitag, 26. Februar 2016, 15:24:21 CET schrieb Parth Sane: > 1)Null comparison > 2)Extra Line after curly braces > 3)Changed udelay to udelay_range > 4)BUG_ON to WARN_ON to avoid crashing kernel >

[PATCH] Staging: slicoss: change memory allocation style in slicoss.c

2016-02-29 Thread Ben Marsh
This is a patch to slicoss.c to change the memory allocation style as identified by checkpatch.pl Signed-off-by: Ben Marsh --- drivers/staging/slicoss/slicoss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/slicoss/slicoss.c

[PATCH] Staging: slicoss: change memory allocation style in slicoss.c

2016-02-29 Thread Ben Marsh
This is a patch to slicoss.c to change the memory allocation style in slicoss.c as found by checkpatch.pl Signed-off-by: Ben Marsh --- drivers/staging/slicoss/slicoss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/slicoss/slicoss.c

Re: [PATCH] staging: rtl8192e: rtllib_softmac.c: Remove multiple blank lines

2016-02-29 Thread Vaishali Thakkar
On Sunday 28 February 2016 08:18 PM, Tapan Prakash T wrote: > Fixed blank lines issues reported by checkpatch.pl Hi, I can see that you have sent 3 patches for the same file. But they are sent as separate patches. So, this might be confusing for the maintainer. So, in this case these are some

Re: [PATCH v3 39/52] mtd: nand: jz4780: switch to mtd_ooblayout_ops

2016-02-29 Thread Harvey Hunt
Hi Boris, On 26/02/16 00:57, Boris Brezillon wrote: Implementing the mtd_ooblayout_ops interface is the new way of exposing ECC/OOB layout to MTD users. Signed-off-by: Boris Brezillon --- drivers/mtd/nand/jz4780_nand.c | 19 +-- 1 file

Re: [PATCH v3 07/52] mtd: nand: core: use mtd_ooblayout_xxx() helpers where appropriate

2016-02-29 Thread Harvey Hunt
Hi Boris, On 26/02/16 18:33, Boris Brezillon wrote: Hi Harvey, On Fri, 26 Feb 2016 15:17:28 + Harvey Hunt wrote: Hi Boris, On 26/02/16 15:10, Boris Brezillon wrote: Hi Harvey, On Fri, 26 Feb 2016 14:35:08 + Harvey Hunt wrote:

[PATCH] staging: wilc1000: wilc_wlan_if.h: remove unused functions

2016-02-29 Thread Tony Cho
This patch removes the unused functions anymore related to the configuration of the bus interface clock speed. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wlan_if.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan_if.h

Re: [PATCH] Drivers: hv: util: fix a race with daemons startup

2016-02-29 Thread Vitaly Kuznetsov
KY Srinivasan writes: >> -Original Message- >> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] >> Sent: Thursday, February 18, 2016 5:35 AM >> To: de...@linuxdriverproject.org >> Cc: KY Srinivasan ; Haiyang Zhang >> ;

Re: [PATCH] staging/android: refactor SYNC_IOC_FILE_INFO

2016-02-29 Thread Emil Velikov
Hi Gustavo, On 27 February 2016 at 15:25, Gustavo Padovan wrote: > Hi Emil, > > 2016-02-27 Emil Velikov : > >> Hi Gustavo, >> >> On 26 February 2016 at 21:00, Gustavo Padovan wrote: >> > From: Gustavo Padovan

Re: [PATCH v4 3/5] staging/android: remove redundant comments on sync_merge_data

2016-02-29 Thread Maarten Lankhorst
Op 26-02-16 om 19:31 schreef Gustavo Padovan: > From: Gustavo Padovan > > struct sync_merge_data already have documentation on top of the > struct definition. No need to duplicate it. > > Signed-off-by: Gustavo Padovan > --- >

Re: [PATCH v4 5/5] staging/android: add flags member to sync ioctl structs

2016-02-29 Thread Maarten Lankhorst
Op 26-02-16 om 19:31 schreef Gustavo Padovan: > From: Gustavo Padovan > > Play safe and add flags member to all structs. So we don't need to > break API or create new IOCTL in the future if new features that requires > flags arises. > > v2: check if flags are

Re: [PATCH] staging/android: refactor SYNC_IOC_FILE_INFO

2016-02-29 Thread Maarten Lankhorst
Op 26-02-16 om 22:00 schreef Gustavo Padovan: > From: Gustavo Padovan > > Change SYNC_IOC_FILE_INFO behaviour to avoid future API breaks and > optimize buffer allocation. In the new approach the ioctl needs to be called > twice to retrieve the array of fence_infos