2015-10-01 오후 2:31에 PINTU KUMAR 이(가) 쓴 글:
-Original Message-
From: Laura Abbott [mailto:labb...@redhat.com]
Sent: Wednesday, September 30, 2015 11:42 PM
To: Rohit kumar; gre...@linuxfoundation.org; a...@android.com;
riandr...@android.com; dan.carpen...@oracle.com;
sumit.sem...@linaro.
> -Original Message-
> From: Laura Abbott [mailto:labb...@redhat.com]
> Sent: Wednesday, September 30, 2015 11:42 PM
> To: Rohit kumar; gre...@linuxfoundation.org; a...@android.com;
> riandr...@android.com; dan.carpen...@oracle.com;
> sumit.sem...@linaro.org; mitch...@codeaurora.org; akpm@
From: Oleg Drokin
This is server-side code that cannot work on the client
(vfs would do this check on the local node).
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 132
1 file changed, 132 deletions(-)
diff --git a/drivers/staging/l
From: Oleg Drokin
Since staging tree code is just the client, ns_is_client is always
true, so change all callers as such and drop all the
dead code for when it's false.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 13 --
drivers/staging/lustre/lus
From: Oleg Drokin
This rather large patch prunes all unused EXPORT_SYMBOLS and
marks functions only used locally as static lustre ldlm module.
The only two remaining nonstatic functions that should be static
now are:
ldlm_cancel_lru_local
ldlm_resource_putref_locked
But some bigger code shuffli
From: Oleg Drokin
Server-side scanning is not really used in the client code,
so it's ok to drop it.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 3 -
drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 348 +
2 files changed, 13 in
From: Oleg Drokin
Since there are no server namespaces, just replace it with
ldlm_cli_inactive_namespace_list pointer.
Also make ldlm_cli_inactive_namespace_list static as it's only
used in ldlm_resource.c
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 7 --
From: Oleg Drokin
Now that ldlm_flock code no longer uses it.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 2 --
drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 5 +++--
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/l
From: Oleg Drokin
This includes ldlm_register_intent(), ns_policy field in the namespace
and all of it's users, as this could only happen on the server.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 8 --
drivers/staging/lustre/lustre/ldlm/ldlm_lock.c
From: Oleg Drokin
Only used on the server.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 1 -
drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 28 --
2 files changed, 29 deletions(-)
diff --git a/drivers/staging/lustre/lustre/incl
From: Oleg Drokin
All users are gone, and they were used on the server anyway.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 3 -
drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 70 --
2 files changed, 73 deletions(-)
diff --git
From: Oleg Drokin
It was directly used only on the server. Client side part
was split into smaller chunks to avoid deadlocks.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 2 --
drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 27 -
From: Oleg Drokin
It's not used anywhere.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 1 -
drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 8
2 files changed, 9 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lustre_dlm.h
From: Oleg Drokin
They are only used on the server.
Also remove helper functions and cleanup callsites.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 2 -
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c| 1 -
drivers/staging/lustre/lustre/ldlm/ldlm_l
From: Oleg Drokin
Since the code we have is Lustre-client only, this function
always returns 0, so drop it and amend all the callsites to
drop dead code.
One of the places also sets LDLM_FL_NS_SRV to indicate a lock
is in a server namespace. This too cannot happen in this code,
so drop all such c
From: Oleg Drokin
This is only used on the server.
Also while we are at it, remove unused prototypes for
ldlm_server_ast and ldlm_handle_enqueue0() that are not
defined anywhere
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 15 -
drivers/staging/lustre
From: Oleg Drokin
It was only used on the server.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 2 --
drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 26 --
2 files changed, 28 deletions(-)
diff --git a/drivers/staging/lustre/lus
From: Oleg Drokin
Now that we removed all the lock conversion functions,
also get rid of the converson queue since nothing could ever
get there anyway.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 2 --
drivers/staging/lustre/lustre/ldlm/ldlm_lock.c
From: Oleg Drokin
And all supporting export infrastructure. There's no use for it
all on client.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c| 102 -
drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 2 -
2 files changed, 104 delet
From: Oleg Drokin
Also ll_ra_read_get_locked that was only used by it.
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/llite/llite_internal.h | 1 -
drivers/staging/lustre/lustre/llite/rw.c | 24 --
2 files changed, 25 deletions(-)
diff --git a/driver
From: Oleg Drokin
Nothing adds locks into waiting list on the client, so no
point in retaining those.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 2 --
drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 8
drivers/staging/lustre/lustre/ldlm/
From: Oleg Drokin
Also remove the lmv and mdc infrastructure to query these keys.
In fact this whole "unlink cookies" thing becomes irrelevant
with newer servers, but since we still retain 2.[123] servers
commpatibility, we cannot completely remove all traces of it yet.
Signed-off-by: Oleg Droki
From: Oleg Drokin
This was used for a function that's no longer important, but
is no longer used anywhere.
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/llite/llite_internal.h | 2 --
drivers/staging/lustre/lustre/llite/llite_lib.c| 26 --
2 files changed,
From: Oleg Drokin
Lock conversion is not really implemented, so let's stop
pretending here.
This removes ldlm_lock_convert, ldlm_cli_lock_convert
and ldlm_lock_downgrade.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 4 -
drivers/staging/lustre/lustre/ld
From: Oleg Drokin
This is only used on the server in case a client gets disconnected.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 2 -
drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 50 --
2 files changed, 52 deletions(-)
diff
From: Oleg Drokin
This particular incarnation is only used on the server.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 3 +-
drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 36 --
2 files changed, 1 insertion(+), 38 deletions(-)
From: Oleg Drokin
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 1 -
drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 13 -
2 files changed, 14 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lustre_dlm.h
b/drivers/staging
From: Oleg Drokin
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 1 -
drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 16
2 files changed, 17 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lustre_dlm.h
b/drivers/stag
From: Oleg Drokin
Not used anywhere
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 1 -
drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 21 -
2 files changed, 22 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lust
From: Oleg Drokin
These generic switch functions appear to be unused as all the
individual ones are called directly.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/lov/lov_internal.h | 2 --
drivers/staging/lustre/lustre/lov/lov_pack.c | 20
drivers/stag
From: Oleg Drokin
Also lov_lsm_addref is only used in the file it is defined,
so make it static
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/lov/lov_cl_internal.h | 3 ---
drivers/staging/lustre/lustre/lov/lov_object.c | 13 +
2 files changed, 1 insertion(+),
From: Oleg Drokin
This continues the quest for killing unused code in the Lustre client.
Main focus in this set is on Lustre LDLM code - it kills both
outright unused code and code that looks like it's used, but in fact
also unused as various conditions that would lead to it would only
happen on
From: Oleg Drokin
On client side interval-tree code operations are pretty basic,
so get rid of the code that is only used on the server to figure
out how much to extend the locks and such.
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/include/interval_tree.h | 36 ---
drivers/stag
From: Oleg Drokin
These functions are used on the server-only, so get rid of them.
Also get rid of ldlm export hash operations and the struct.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_dlm.h | 2 -
drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c| 105 ---
From: Oleg Drokin
The ioctl for this function was removed, but the function was
forgotten, so kill it now.
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/llite/llite_internal.h | 1 -
drivers/staging/lustre/lustre/llite/namei.c| 28 --
2 files changed,
The existing dgap_parsefile() rejects invalid config file.
But before we know that config file is invalid a lot of memory leak
can happen. Removing the chances of memory leak won't heart anyone.
regards,
Ronit Halder
___
devel mailing list
de...@linuxdri
On Tue, Sep 29, 2015 at 5:55 AM, Greg KH wrote:
> On Mon, Sep 21, 2015 at 11:29:50PM +0530, Punit Vara wrote:
>> This patch is to rtsx_chip.h that fixes up following warning reported
>> by checkpatch.pl :
>>
>> -Block comments use * on subsequent lines
>> -Block comments use a trailing */ on a sep
On Wednesday, September 30, 2015 9:37 AM, Ian Abbott wrote:
> Expand the descriptions of the functions and document the return values.
>
> Signed-off-by: Ian Abbott
Reviewed-by: H Hartley Sweeten
___
devel mailing list
de...@linuxdriverproject.org
htt
On Wednesday, September 30, 2015 9:11 AM, Ian Abbott wrote:
> Expand the descriptions of the functions and document the return values.
>
> Signed-off-by: Ian Abbott
Reviewed-by: H Hartley Sweeten
___
devel mailing list
de...@linuxdriverproject.org
htt
On Wednesday, September 30, 2015 8:44 AM, Ian Abbott wrote:
> Fix the 'Return' tags in the kernel-doc comments as they currently say
> 'Returns', which is not recognized by kernel-doc.
>
> Signed-off-by: Ian Abbott
Reviewed-by: H Hartley Sweeten
___
d
On 09/29/2015 10:37 PM, Rohit kumar wrote:
This patch fixes error handling case when buffer->pages allocation
fails. Also, it removes unreachable code of checking ret variable
although it is not updated.
Reviewed-by: Laura Abbott
Signed-off-by: Rohit kumar
---
drivers/staging/android/io
Зqpaвcтвyйтe! Bac интepecyюm бaзы данныx нoвыx kлиентoв qля Baшeгo Бизнeсa?
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Expand the descriptions of the functions and document the return values.
Signed-off-by: Ian Abbott
---
drivers/staging/comedi/comedi_usb.c | 75 +
1 file changed, 52 insertions(+), 23 deletions(-)
diff --git a/drivers/staging/comedi/comedi_usb.c
b/drivers/st
Expand the descriptions of the functions and document the return values.
Signed-off-by: Ian Abbott
---
drivers/staging/comedi/comedi_pcmcia.c | 104 -
1 file changed, 76 insertions(+), 28 deletions(-)
diff --git a/drivers/staging/comedi/comedi_pcmcia.c
b/drivers
Fix the 'Return' tags in the kernel-doc comments as they currently say
'Returns', which is not recognized by kernel-doc.
Signed-off-by: Ian Abbott
---
drivers/staging/comedi/comedi_pci.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/comedi/comedi_
In commit 0a0796eb16f3, there is no include to mon.h, a newly created
header file. This omission leads to the following Sparse warnings:
CHECK drivers/staging/rtl8188eu/os_dep/mon.c
drivers/staging/rtl8188eu/os_dep/mon.c:81:6: warning: symbol
'rtl88eu_mon_recv_hook' was not declared. Should i
This patch fixes a potential buffer overflow detected by smatch.
pda16 has length 512, while processing an element with index < 512 we
are checking for an end marker in the next element. This poses a
potential buffer overflow if no end marker is present.
Signed-off-by: Tillmann Heidsieck
---
v2:
On Wed, Sep 30, 2015 at 06:12:50PM +0530, Chandra Gorentla wrote:
> > > while (1) {
> > > - wilc_mq_recv(&gMsgQHostIF, &msg, sizeof(struct host_if_msg),
> > > &u32Ret);
> > > + ret = wilc_mq_recv(&gMsgQHostIF, &msg,
> > > + sizeof(struct host_if_ms
Two lines of code that were over 80 characters long is fixed
by splitting them across multiple lines.
The lines of code are now easier to comprehend.
Issue found by checkpatch.
Signed-off-by: Arjun Krishna Babu
---
drivers/staging/vt6655/baseband.c | 14 --
1 file changed, 12 inser
On Tue, Sep 29, 2015 at 05:31:53AM +0300, Dan Carpenter wrote:
> On Mon, Sep 28, 2015 at 11:43:55PM +0530, Chandra S Gorentla wrote:
> > - The message queue is replaced with standard Linux linked list
> > - kmem_cache is used for list members
> > - A check for return value of receive method is a
On Tue, Sep 29, 2015 at 02:44:02AM +0200, Greg KH wrote:
> On Mon, Sep 28, 2015 at 11:43:55PM +0530, Chandra S Gorentla wrote:
> > - The message queue is replaced with standard Linux linked list
> > - kmem_cache is used for list members
>
> Why?
The list entires are of fixed type and repeatedly
Dear Dan,
Thank you!
Always helpful your advice listen carefully.
I will send in the morning.
Thanks, BR
Leo
-Original Message-
From: Dan Carpenter [mailto:dan.carpen...@oracle.com]
Sent: Wednesday, September 30, 2015 7:05 PM
To: Cho, Tony
Cc: gre...@linuxfoundation.org; de...@drive
wstats->qual.noise is of type uint8, so it shouldn't be assigned a
negative number. Assigning it 0x100 - 100, which is the equivalent
to -100 dBm when IW_QUAL_DBM is set.
Signed-off-by: Luis de Bethencourt
---
drivers/staging/rtl8192e/rtl8192e/rtl_wx.c | 2 +-
1 file changed, 1 insertion(+), 1 d
On Wed, Sep 30, 2015 at 11:39:45AM +0530, Ronit Halder wrote:
> In dgap_parsefile() char pointers are set with kstrdup()
> without checking that some string is allocated to that
> char pointer before.
Why would this happen? Wouldn't it be better to reject the invalid
config file?
regards,
dan ca
On Wed, Sep 30, 2015 at 06:55:11PM +0900, Tony Cho wrote:
> This patch renames strHostIfRegisterFrame to reg_f to avoid CamelCase
> naming convention.
Could we go with "reg_frame"? f is very ambigous.
regards,
dan carpenter
___
devel mailing list
de..
On Wed, Sep 30, 2015 at 7:20 PM, Greg KH wrote:
> On Wed, Sep 30, 2015 at 05:52:14PM +0900, Chaehyun Lim wrote:
>> The kbuild test robot reports the following build error for
>> openrisc-allmodconfig.
>>
>> >> drivers/staging/wilc1000/wilc_sdio.c:584:2: error: implicit
>> >> declaration of functio
On Wed, Sep 30, 2015 at 05:52:14PM +0900, Chaehyun Lim wrote:
> The kbuild test robot reports the following build error for
> openrisc-allmodconfig.
>
> >> drivers/staging/wilc1000/wilc_sdio.c:584:2: error: implicit
> >> declaration of function 'memset'
>
> The error occurs due to missing a stand
Ugh... No. Just do this:
diff --git a/drivers/staging/wilc1000/wilc_msgqueue.c
b/drivers/staging/wilc1000/wilc_msgqueue.c
index 869736a..52d8f95 100644
--- a/drivers/staging/wilc1000/wilc_msgqueue.c
+++ b/drivers/staging/wilc1000/wilc_msgqueue.c
@@ -102,6 +102,8 @@ int wilc_mq_send(WILC_MsgQueu
This patch renames strHostIfRemainOnChan to remain_on_ch to avoid
CamelCase naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interf
From: Leo Kim
This patch fix kernel fail after ifconfig wlan0 up.
- if normal operation, kfree functions don't execute.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_msgqueue.c | 16 ++--
1 file changed, 10 insertions(+), 6 deletions(-)
diff --
This patch renames strHostIfBASessionInfo to session_info to avoid
CamelCase naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/driv
This patch renames strHostIfRegisterFrame to reg_f to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/sta
This patch renames strHostIfSetOperationMode to mode to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/
This patch renames pUserData to data to avoid CamelCase naming
convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wilc1000/host_in
This patch renames strHostIfGetMacAddress to get_mac_info to avoid
CamelCase naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/sta
This patch renames strHostIFDelAllSta to del_all_sta_info to avoid
CamelCase naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/sta
This patch renames strHostIfSetMulti to multicast_info to avoid
CamelCase naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/stagin
This patch renames strHostIfSetMacAddress to set_mac_info to avoid
CamelCase naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/sta
This patch renames strHostIfSetIP to ip_info to avoid CamelCase naming
convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging
This patch renames strHostIfSetDrvHandler to drv to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wilc
This patch renames strHostIfStaInactiveT to mac_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/
This patch renames strPowerMgmtparam to pwr_mgmt_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging
This patch renames strAddStaParam to add_sta_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/w
This patch removes struct timer_cb which is not used.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 15 ---
1 file changed, 15 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wilc1000/host_interface.c
index da8e589..7d
This patch renames strDelStaParam to del_sta_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wil
This patch removes struct del_beacon which is not necessary. This patch
also changes Handle_DelBeacon function by removing 2nd parameter, struct
del_beacon because it is not used inside.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 24 +++-
1 file c
This patch renames strEditStaParam to edit_sta_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wil
This patch removes struct get_channel from the driver because it is not
used anywhere.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 15 ---
1 file changed, 15 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wilc1000/h
This patch renames struct set_beacon to beacon_attr to keep the naming
convention as scan_attr, connect_attr and channel_attr.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/wi
This patch renames strHostIFSetBeacon to beacon_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/
This patch renames strHostIFSetChan to channel_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/w
This patch renames struct set_channel to channel_attr to keep the naming
convention such as scan_attr and connect_attr.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/wilc1000/ho
This patch renames strHostIFCfgParamAttr to cfg_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/
This patch renames strHostIFkeyAttr to key_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 100 +++---
1 file changed, 50 insertions(+), 50 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interfac
This patch renames strRcvdGnrlAsyncInfo to async_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/st
This patch renames strRcvdNetworkInfo to net_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/stagin
This patch renames strHostIFscanAttr to scan_info in order to avoid
CamelCase naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/staging/wilc1000/hos
This patch removes AGING_ALG from the Makefile because it is always in
use.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/Makefile | 1 -
drivers/staging/wilc1000/coreconfigurator.h | 2 --
2 files changed, 3 deletions(-)
diff --git a/drivers/staging/wilc1000/Makefile
b/driver
This patch renames strHostIFconnectAttr to con_info to avoid CamelCase
naming convention.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 32 +++
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/drivers/staging/wilc1000/host_inte
This patch removes unnecessary definition, PLL_WORKAROUND from the
Makefile. There is no use throughout the driver.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wilc1000/Makefile
b/drivers/stagi
On Tue, Sep 29, 2015 at 10:24:59PM +0200, Tillmann Heidsieck wrote:
> On Mon, Sep 28, 2015 at 09:56:19AM -0500, Larry Finger wrote:
> > The primary requirement for moving a driver out of staging is that it use
> > mac80211. No amount of cosmetic fixing will ever make that change for
> > rtl8712u!
>
The kbuild test robot reports the following build error for
openrisc-allmodconfig.
>> drivers/staging/wilc1000/wilc_sdio.c:584:2: error: implicit
>> declaration of function 'memset'
The error occurs due to missing a standard header file as
so that three .c files are included it.
Signed-off-by:
On Wed, Sep 30, 2015 at 05:13:43PM +0900, Chaehyun Lim wrote:
> On Wed, Sep 30, 2015 at 3:01 PM, Greg Kroah-Hartman
> wrote:
> > On Wed, Sep 30, 2015 at 01:15:51PM +0800, kbuild test robot wrote:
> >> tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
> >> staging-testing
On Wed, Sep 30, 2015 at 3:01 PM, Greg Kroah-Hartman
wrote:
> On Wed, Sep 30, 2015 at 01:15:51PM +0800, kbuild test robot wrote:
>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
>> staging-testing
>> head: 7477929566b65f692b79974b784ee59e8b2b9f18
>> commit: e215a8712
94 matches
Mail list logo
