Re: [PATCH 00/27] staging: add driver for KS7010 based SDIO cards

2016-05-10 Thread Wolfram Sang
On Tue, May 10, 2016 at 11:05:34PM +0200, Greg KH wrote: > On Tue, May 10, 2016 at 03:52:13PM +0200, Wolfram Sang wrote: > > This driver can be used with later Spectec SDW-821 and all SDW-823 cards. > > See patch 1 and README for more details about its origin. To have the card > > working, you need

Re: [PATCH] Staging: wlan-ng: fix comments style

2016-05-10 Thread Kroah-Hartman
On Tue, May 10, 2016 at 09:05:15PM -0400, YU Bo wrote: > Hi Greg, > On Mon, May 09, 2016 at 02:26:13PM +0200, Kroah-Hartman wrote: > > On Sun, May 08, 2016 at 10:45:16AM -0400, YU Bo wrote: > > > The patch fixed warning reported by checkpatch.pl: Block comments use a > > > trailing */ on a separate

Re: [PATCH v1] ION: Sys_heap: Add cached pool to spead up cached buffer alloc

2016-05-10 Thread Chen Feng
On 2016/5/11 6:48, Laura Abbott wrote: > On 05/09/2016 04:35 AM, Chen Feng wrote: >> Add ion cached pool in system heap. This patch add a cached pool >> in system heap. It has a great improvement of alloc for cached >> buffer. >> > > Can you give some benchmark numbers here? > ok, My test is u

Re: [PATCH] Staging: wlan-ng: fix comments style

2016-05-10 Thread YU Bo
Hi Greg, On Mon, May 09, 2016 at 02:26:13PM +0200, Kroah-Hartman wrote: On Sun, May 08, 2016 at 10:45:16AM -0400, YU Bo wrote: The patch fixed warning reported by checkpatch.pl: Block comments use a trailing */ on a separate line. Signed-off-by: YU Bo --- drivers/staging/wlan-ng/prism2mgmt.h

Re: [PATCH v1] ION: Sys_heap: Add cached pool to spead up cached buffer alloc

2016-05-10 Thread Laura Abbott
On 05/09/2016 04:35 AM, Chen Feng wrote: Add ion cached pool in system heap. This patch add a cached pool in system heap. It has a great improvement of alloc for cached buffer. Can you give some benchmark numbers here? v1: Makes the cached buffer zeroed before going to pool Signed-off-by: C

Re: [PATCH 00/27] staging: add driver for KS7010 based SDIO cards

2016-05-10 Thread Greg KH
On Tue, May 10, 2016 at 03:52:13PM +0200, Wolfram Sang wrote: > This driver can be used with later Spectec SDW-821 and all SDW-823 cards. > See patch 1 and README for more details about its origin. To have the card > working, you need another patch which is already in mmc/next: > > https://patchwo

流程与其他管理要素的关系

2016-05-10 Thread 周蓓媛
_《可_否单独签(订试用期协(议.xls Description: Binary data ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 1/1] Staging: comedi: comedi_fops.c: Fixed coding style issue

2016-05-10 Thread Ian Abbott
On 10/05/16 16:51, Shyam Saini wrote: Fixed following checkpatch.pl warnings WARNING: Prefer WRITE_ONCE(, ) over ACCESS_ONCE() = WARNING: Prefer READ_ONCE() over ACCESS_ONCE() Signed-off-by: Shyam Saini --- drivers/staging/comedi/comedi_fops.c | 12 ++-- 1 file changed, 6 insertion

[PATCH 1/1] Staging: comedi: comedi_fops.c: Fixed coding style issue

2016-05-10 Thread Shyam Saini
Fixed following checkpatch.pl warnings WARNING: Prefer WRITE_ONCE(, ) over ACCESS_ONCE() = WARNING: Prefer READ_ONCE() over ACCESS_ONCE() Signed-off-by: Shyam Saini --- drivers/staging/comedi/comedi_fops.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers

[PATCH 09/27] staging: ks7010: delete seperate debug header

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Move the one debug macro to the generic wlan header. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_config.c | 1 - drivers/staging/ks7010/ks7010_sdio.c | 1 - drivers/staging/ks7010/ks_debug.h | 28 drivers/staging/ks7010

[PATCH 00/27] staging: add driver for KS7010 based SDIO cards

2016-05-10 Thread Wolfram Sang
This driver can be used with later Spectec SDW-821 and all SDW-823 cards. See patch 1 and README for more details about its origin. To have the card working, you need another patch which is already in mmc/next: https://patchwork.kernel.org/patch/9043341/ A working topic branch for the Renesas H3

[PATCH 10/27] staging: ks7010: really iterate over multicast addresses

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang The loop variable was defined but not really used. Fix this. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks_hostif.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c inde

[PATCH 18/27] staging: ks7010: indent ks_hostif.c

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks_hostif.c | 1960 1 file changed, 1080 insertions(+), 880 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index 497b8aaf6daef8..f2

[PATCH 15/27] staging: ks7010: indent eap_packet.h

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/eap_packet.h | 49 ++--- 1 file changed, 24 insertions(+), 25 deletions(-) diff --git a/drivers/staging/ks7010/eap_packet.h b/drivers/staging/ks7010/eap_packet.h index b664bdd8c961db..883

[PATCH 25/27] staging: ks7010: indent ks7010_sdio.c

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Unlike the previous patches which are plain indent outcomes, this has some manual fixups to be not overly strict with the 80 char limit. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_sdio.c | 807 --- 1 file changed, 460 insert

[PATCH 03/27] staging: ks7010: remove custom firmware loader

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang FW_LOADER works fine, no need for a open coded fallback. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/Kconfig | 1 + drivers/staging/ks7010/ks7010_config.c | 62 -- drivers/staging/ks7010/ks7010_sdio.c | 56 ---

[PATCH 23/27] staging: ks7010: indent michael_mic.c

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/michael_mic.c | 43 ++-- 1 file changed, 21 insertions(+), 22 deletions(-) diff --git a/drivers/staging/ks7010/michael_mic.c b/drivers/staging/ks7010/michael_mic.c index ec8769a974427a..a

[PATCH 24/27] staging: ks7010: indent michael_mic.h

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/michael_mic.h | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/ks7010/michael_mic.h b/drivers/staging/ks7010/michael_mic.h index f14f160ad46549..57ecbe34153202 100644 -

[PATCH 05/27] staging: ks7010: remove code for old kernel versions

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang No need to be backwards compatible. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks_hostif.c | 9 - drivers/staging/ks7010/ks_wlan.h | 28 +--- drivers/staging/ks7010/ks_wlan_net.c | 3 --- 3 files changed, 1 insertion(+), 39

[PATCH 14/27] staging: ks7010: remove unecessary typedef

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Let's simply specify the struct to keep in sync with kernel coding style. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_config.c | 4 +- drivers/staging/ks7010/ks7010_sdio.c | 44 - drivers/staging/ks7010/ks7010_sdio.h | 2 +- drivers/staging

[PATCH 20/27] staging: ks7010: indent ks_wlan.h

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks_wlan.h | 246 +++ 1 file changed, 121 insertions(+), 125 deletions(-) diff --git a/drivers/staging/ks7010/ks_wlan.h b/drivers/staging/ks7010/ks_wlan.h index 58e3a5a6b44472..f0f9f8ef

[PATCH 06/27] staging: ks7010: avoid workqueue races

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang My Spectec SDW823 card oopsed when it was already inserted during boot. When debugging this, I noticed that the card init was done in a seperate workqueue which was only activated once in probe. After removing the workqueue and calling the card init directly from probe, the OOP

[PATCH 26/27] staging: ks7010: remove supported card table with one element

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang There is only this card supported, no need to iterate over the table. The resulting firmware filename wasn't used anyway, but came from the config file or hardcoded default. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_sdio.c | 31 ++-

[PATCH 21/27] staging: ks7010: indent ks_wlan_ioctl.h

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks_wlan_ioctl.h | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_wlan_ioctl.h b/drivers/staging/ks7010/ks_wlan_ioctl.h index e7469f7338f7af..cc4669eddcc1f7 100644

[PATCH 27/27] staging: ks7010: fix module annotations

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang List all authors, beautify description, match license to what is stated in file headers, add firmware information. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_sdio.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/stag

[PATCH 12/27] staging: ks7010: fix printk format warnings

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Use proper type for size_t. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_config.c | 2 +- drivers/staging/ks7010/ks7010_sdio.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ks7010/ks7010_config.c b/drivers/staging/

[PATCH 02/27] staging: ks7010: remove non-SDIO code and #ifdefs

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang I couldn't find any trace of code or even products using ks7010 with something else than SDIO. So, remove the conditionals. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/Makefile| 2 +- drivers/staging/ks7010/ks7010_config.c | 6 --- drivers/staging/ks7010

[PATCH 17/27] staging: ks7010: indent ks7010_sdio.h

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_sdio.h | 64 +--- 1 file changed, 30 insertions(+), 34 deletions(-) diff --git a/drivers/staging/ks7010/ks7010_sdio.h b/drivers/staging/ks7010/ks7010_sdio.h index 1d16673e5dae02..a

[PATCH 19/27] staging: ks7010: indent ks_hostif.h

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks_hostif.h | 525 ++--- 1 file changed, 262 insertions(+), 263 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h b/drivers/staging/ks7010/ks_hostif.h index c333ce1a4baa6e..01d

[PATCH 07/27] staging: ks7010: use long preamble as default

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang I had a problem connecting to a network with a short preamble, so let's make the safer option the default. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_config.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ks7010/ks

[PATCH 16/27] staging: ks7010: indent ks7010_config.c

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_config.c | 353 + 1 file changed, 184 insertions(+), 169 deletions(-) diff --git a/drivers/staging/ks7010/ks7010_config.c b/drivers/staging/ks7010/ks7010_config.c index 48809bcc779

[PATCH 11/27] staging: ks7010: make loading config file optional

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang We have sane defaults, so we don't need to bail out if there is no config file. Note that the config file should go away completely in favour of configuration mechanisms already upstream. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks7010_config.c | 8 +++- 1

[PATCH 13/27] staging: ks7010: add example cfg file as a reference

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang We want to remove it, but to do so properly, it is good to have a working example. Needs to be copied to /lib/firmware in order to be used. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks79xx.cfg | 116 ++ 1 file changed, 116 ins

[PATCH 08/27] staging: ks7010: use kernel helper to print buffer

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang No need for an open coded one. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/Makefile | 2 +- drivers/staging/ks7010/ks7010_sdio.c | 9 + drivers/staging/ks7010/ks_debug.c| 30 -- drivers/staging/ks7010/ks_debug.h|

[PATCH 04/27] staging: ks7010: remove checks for WIRELESS_EXT version

2016-05-10 Thread Wolfram Sang
From: Wolfram Sang We are by far newer than that anyhow. Signed-off-by: Wolfram Sang --- drivers/staging/ks7010/ks_hostif.c | 23 --- drivers/staging/ks7010/ks_wlan_net.c | 14 -- 2 files changed, 37 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.

Re: [PATCH 5/9] staging: lustre: fid: packing ost_idx in IDIF

2016-05-10 Thread Dan Carpenter
On Wed, May 04, 2016 at 10:28:56AM -0400, James Simmons wrote: > From: Fan Yong > > For a normal FID, we can know on which target the related object > is allocated via querying FLDB; but it is not true for an IDIF. > > To locate the OST via the given IDIF, when the IDIF is generated, > we pack t