Re: [PATCH] staging/android: refactor SYNC_IOC_FILE_INFO

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 pointed by info->sync_fence_info.

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

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 valid (zero, in this case) > > Signed-

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

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 > --- > drivers/staging/android/uapi/sync.h | 6 +++--- > 1 file changed, 3 i

Re: [PATCH] staging/android: refactor SYNC_IOC_FILE_INFO

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 >> > >> > Change SYNC_IOC_FILE_INFO behaviour to avoid future API breaks and >> > opti

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

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 >> ; Olaf Hering ; Cathy Avery >> >> Subject: [PATCH] Drivers: hv: util

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

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 b/drivers/staging/wilc1

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

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: [...] I'll look into this more later today, but want

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

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 changed, 5 insertions(+), 14 deletions(

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

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 p

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

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 b/drivers/staging/slicos

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

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 b/drivers/staging/slicoss/slicos

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

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 > 5)Removed

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

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 b/drivers/staging/slicoss/slicos

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

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 b/drivers/staging/slicoss/sli

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

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

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

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 --- drivers/staging/android/lowmemorykiller.c | 2 ++ 1 file changed, 2 insertions(+) diff -

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

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

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

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

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

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 Die

[PATCH] staging: unisys: Update MAINTAINERS file

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 a/drivers/staging/unisys/MAINTAINERS b/drivers/stagin

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

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.

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 _

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

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

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

> -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 > ; de...@linuxdriverproject.org; linux- > p...@vger.kernel.org; linux-ker...@vger.kernel.org; ke

[PATCH v2] staging: unisys: Update MAINTAINERS file

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 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisy

[PATCH v2] staging: unisys: Update MAINTAINERS file

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

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

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 _

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

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

Re: [PATCH] staging/android: refactor SYNC_IOC_FILE_INFO

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 the new approach the ioctl needs to be called > > twice to retrieve t

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

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) > ... > $ ethtool -s eth0 sp

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

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 for eth0: > > ... > > 

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

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 the >> > hv_netvsc device via ethtool. >> > 

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

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 v4 1/5] staging/android: add num_fences field to struct sync_file_info

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 http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-de

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

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