Hi Sumit,
On Thu, 07 Jan 2016 19:43:09 +0530
Sumit Semwal wrote:
> Hi Rohit,
>
> On 6 January 2016 at 12:41, Rohit kumar wrote:
> > Currently we can only import dma buf fd's to get ion_handle.
> > Adding support to import dma buf handles to support kernel
> > specific use cases.
> >
> > An exa
On 16-01-07 07:49 PM, KY Srinivasan wrote:
>
>
>> -Original Message-
>> From: John Fastabend [mailto:john.fastab...@gmail.com]
>> Sent: Thursday, January 7, 2016 5:02 PM
>> To: Vitaly Kuznetsov ; Simon Xiao
>> ; Eric Dumazet
>> Cc: Tom Herbert ; net...@vger.kernel.org; KY
>> Srinivasan ;
> -Original Message-
> From: John Fastabend [mailto:john.fastab...@gmail.com]
> Sent: Thursday, January 7, 2016 5:02 PM
> To: Vitaly Kuznetsov ; Simon Xiao
> ; Eric Dumazet
> Cc: Tom Herbert ; net...@vger.kernel.org; KY
> Srinivasan ; Haiyang Zhang ;
> de...@linuxdriverproject.org; linux
On 16-01-07 05:28 AM, Vitaly Kuznetsov wrote:
> Eric Dumazet writes:
>
>> On Thu, 2016-01-07 at 10:33 +0100, Vitaly Kuznetsov wrote:
>>> Recent changes to 'struct flow_keys' (e.g commit d34af823ff40 ("net: Add
>>> VLAN ID to flow_keys")) introduced a performance regression in netvsc
>>> driver.
On Thu, 2016-01-07 at 16:40 -0800, K. Y. Srinivasan wrote:
> The macro startget_to_rport() can return NULL; handle that case
> properly.
OK, can we unwind why you think you could possibly need this? It would
mean that fc_timed_out was called for a non-FC device, which was
thought to be an impossi
The macro startget_to_rport() can return NULL; handle that case
properly.
Signed-off-by: K. Y. Srinivasan
Cc:
---
drivers/scsi/scsi_transport_fc.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/scsi_transport_fc.c
index
You can't have 40 patches with the same subject. Also that is too
generic a subject.
Also don't break it up into one line per patch, break it up so it's
patch 1: remove blank lines
patch 2: no spaces after a cast
patch 3: fix all NULL compares
etc...
regards,
dan carpenter
Removed space after casting operator
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index 7369a1b..45411d9 100644
--- a/drivers/s
Removed space after casting operator
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index 45411d9..a0f45d2 100644
--- a/drivers/s
Logical continuation fix for 'if' block
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index 28d7fd8..8769e47 100644
--- a/d
Logical continuation fix for 'if' block
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index d3ee4f2..28d7fd8 100644
--- a/dr
Removed blank line after open braces
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index 2c251e3..27342f8 100644
--- a/drivers/staging/wlan-ng/p8
Logical continuation fixing for 'if' block
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index 32039ae..d3ee4f2 100644
--- a
Removed blank line after braces
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index d799701..c9bbf89 100644
--- a/drivers/staging/wlan-ng/p80211c
Alignment matched with open parenthesis
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index 44bf164..2c251e3 100644
--- a/dr
Fixed a logical continuation
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index 4bcc444..ff5a2cc 100644
--- a/drivers/stagi
Logical continuation is fixed
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index c9bbf89..44bf164 100644
--- a/drivers/stag
Removed space after casting
Signed-off-by: Pranjal Bhor
---
drivers/staging/wlan-ng/p80211conv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wlan-ng/p80211conv.c
b/drivers/staging/wlan-ng/p80211conv.c
index 7e3d4b5..4bcc444 100644
--- a/drivers/staging/wl
Fix checkpatch warning of adding space around + in accordance with
kernel coding style.
Signed-off-by: Shraddha Barke
---
Changes in v2-
No change. Added lkml and other maintainers.
drivers/staging/goldfish/goldfish_audio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dr
These patches remove the last few checkpatch warnings in staging/goldfish
directory
Shraddha Barke (2):
Staging: goldfish: Add space around +
Staging: goldfish: Remove explicit NULL comparison
drivers/staging/goldfish/goldfish_audio.c | 8
1 file changed, 4 insertions(+), 4 deletio
On Thu, Jan 07, 2016 at 12:34:13PM +0300, Dan Carpenter wrote:
> queue_delayed_work() returns bool, not negative error codes. It returns
> false if the work has already been queued or true otherwise. Since
> we don't care about that, we can just remove the test.
>
> Signed-off-by: Dan Carpenter
Hi Rohit,
On 6 January 2016 at 12:41, Rohit kumar wrote:
> Currently we can only import dma buf fd's to get ion_handle.
> Adding support to import dma buf handles to support kernel
> specific use cases.
>
> An example use case is in linux platforms such as Tizen, in which
> DRM-GEM is used for bu
On Thu, 2016-01-07 at 10:33 +0100, Vitaly Kuznetsov wrote:
> Recent changes to 'struct flow_keys' (e.g commit d34af823ff40 ("net: Add
> VLAN ID to flow_keys")) introduced a performance regression in netvsc
> driver. Is problem is, however, not the above mentioned commit but the
> fact that netvsc_
queue_delayed_work() returns bool, not negative error codes. It returns
false if the work has already been queued or true otherwise. Since
we don't care about that, we can just remove the test.
Signed-off-by: Dan Carpenter
diff --git a/drivers/staging/unisys/visorbus/visorchipset.c
b/drivers/
You're going to have to split this into multiple patches that fix one
type of bug at a time.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
25 matches
Mail list logo
