提交 · 57538afb42e492eb6984f89663361bcfd80d9745 · openanolis / cloud-kernel

27 6月, 2016 3 次提交

staging: fsl-mc: clean up the device id struct · 57538afb

由 Stuart Yoder 提交于 6月 22, 2016

-rename the struct used for fsl-mc device ids to be more
 consistent with other busses
-remove the now obsolete and unused version fields
Signed-off-by: NStuart Yoder <stuart.yoder@nxp.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

57538afb

staging: fsl-mc: implement uevent callback and set the modalias · d568b767

由 Stuart Yoder 提交于 6月 22, 2016

Replace placeholder code in the uevent callback to properly
set the MODALIAS env variable.
Signed-off-by: NStuart Yoder <stuart.yoder@nxp.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

d568b767

staging: fsl-mc: add support for the modalias sysfs attribute · 3d579c35

由 Stuart Yoder 提交于 6月 22, 2016

In order to support uevent based module loading implement modalias support
for the fsl-mc bus driver. Aliases are based on vendor and object/device
id and are of the form "fsl-mc:vNdN".
Signed-off-by: NStuart Yoder <stuart.yoder@nxp.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

3d579c35

26 6月, 2016 18 次提交

staging: wilc1000: fix return value check in wlan_initialize_threads() · b3e6916d

由 Wei Yongjun 提交于 6月 17, 2016

In case of error, the function kthread_run() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check
should be replaced with IS_ERR().
Signed-off-by: NWei Yongjun <yongjun_wei@trendmicro.com.cn>
Reviewed-by: NJulian Calaby <julian.calaby@gmail.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

b3e6916d

staging: wilc1000: arrays can't be NULL · 4562d224

由 Luis de Bethencourt 提交于 6月 23, 2016

hif_drv->usr_scan_req.net.net_info[i] contains found_net_info structs
which have the following element:
u8 bssid[6];

pstrNetworkInfo, of type network_info, also contains an u8 array named
bssid.

request->ssids is an array of cfg80211_ssid structs. Making ssid:
u8 ssid[IEEE80211_MAX_SSID_LEN];

In these 3 cases the arrays are being checked against NULL, which can't
happen. Removing the checks since they will always be true.

Found with smatch:
drivers/staging/wilc1000/host_interface.c:1234 Handle_RcvdNtwrkInfo() warn: this array is probably non-NULL. 'hif_drv->usr_scan_req.net_info[i].bssid'
drivers/staging/wilc1000/host_interface.c:1235 Handle_RcvdNtwrkInfo() warn: this array is probably non-NULL. 'pstrNetworkInfo->bssid'
drivers/staging/wilc1000/host_interface.c:1253 Handle_RcvdNtwrkInfo() warn: this array is probably non-NULL. 'hif_drv->usr_scan_req.net_info[hif_drv->usr_scan_req.rcvd_ch_cnt].bssid'
drivers/staging/wilc1000/host_interface.c:1254 Handle_RcvdNtwrkInfo() warn: this array is probably non-NULL. 'pstrNetworkInfo->bssid'
Signed-off-by: NLuis de Bethencourt <luisbg@osg.samsung.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

4562d224

staging: wilc1000: Change interface wilc_mq_send to wilc_enqueue_cmd · a5c84b2f

由 Binoy Jayan 提交于 6月 23, 2016

Replace the interface 'wilc_mq_send' with 'wilc_enqueue_cmd'
and remove the now unused structures 'message' and 'message_queue'.
Restructure switch statement in the work queue helper function
host_if_work and remove unwanted indentation.
Signed-off-by: NBinoy Jayan <binoy.jayan@linaro.org>
Reviewed-by: NArnd Bergmann <arnd@arndb.de>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

a5c84b2f

staging: wilc1000: Replace kthread with workqueue for host interface · 2518ac59

由 Binoy Jayan 提交于 6月 23, 2016

Deconstruct the kthread / message_queue logic, replacing it with
create_singlethread_workqueue() / queue_work() setup, by adding a
'struct work_struct' to 'struct host_if_msg'. The current kthread
hostIFthread() is converted to a work queue helper with the name
'host_if_work'.
Signed-off-by: NBinoy Jayan <binoy.jayan@linaro.org>
Reviewed-by: NArnd Bergmann <arnd@arndb.de>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

2518ac59

staging: wilc1000: message_queue: Move code to host interface · c6bb38a5

由 Binoy Jayan 提交于 6月 23, 2016

Move the contents of wilc_msgqueue.c and wilc_msgqueue.h into
host_interface.c, remove 'wilc_msgqueue.c' and 'wilc_msgqueue.h'.
This is done so as to restructure the implementation of the kthread
'hostIFthread' using a work queue.
Signed-off-by: NBinoy Jayan <binoy.jayan@linaro.org>
Reviewed-by: NArnd Bergmann <arnd@arndb.de>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

c6bb38a5

staging: wilc1000: Remove semaphore close_exit_sync · 77eebe82

由 Binoy Jayan 提交于 6月 15, 2016

The semaphore 'close_exit_sync' does not serve any purpose other
than delaying the deregistration of the device which it is trying
to protect from shared access. 'up' is called only when a subdevice
is closed and not when it is opened. So, the semaphore count only
goes up when the device is used.
Signed-off-by: NBinoy Jayan <binoy.jayan@linaro.org>
Reviewed-by: NArnd Bergmann <arnd@arndb.de>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

77eebe82

staging: wilc1000: Replace semaphore sync_event with completion · 04247e7d

由 Binoy Jayan 提交于 6月 15, 2016

The semaphore 'sync_event' is used as completion, so convert
it to a struct completion type. Also, return -ETIME if the return
value of wait_for_completion_timeout is 0.
Signed-off-by: NBinoy Jayan <binoy.jayan@linaro.org>
Reviewed-by: NArnd Bergmann <arnd@arndb.de>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

04247e7d

staging: wilc1000: Replace semaphore cfg_event with completion · fa659698

由 Binoy Jayan 提交于 6月 15, 2016

The semaphore 'cfg_event' is used as completion, so convert
it to a struct completion type.
Signed-off-by: NBinoy Jayan <binoy.jayan@linaro.org>
Reviewed-by: NArnd Bergmann <arnd@arndb.de>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

fa659698

staging: wilc1000: Replace semaphore txq_add_to_head_cs with mutex · 334bed08

由 Binoy Jayan 提交于 6月 15, 2016

The semaphore 'txq_add_to_head_cs' is a simple mutex, so it should be
written as one. Semaphores are going away in the future. Also, removing
the timeout scenario as the error handling code does not propagate the
timeout properly.
Signed-off-by: NBinoy Jayan <binoy.jayan@linaro.org>
Reviewed-by: NArnd Bergmann <arnd@arndb.de>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

334bed08

staging: wilc1000: Replace semaphore txq_event with completion · b27a6d5e

由 Binoy Jayan 提交于 6月 15, 2016

The semaphore 'txq_event' is used as completion, so convert it
to a struct completion type.
Signed-off-by: NBinoy Jayan <binoy.jayan@linaro.org>
Reviewed-by: NArnd Bergmann <arnd@arndb.de>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

b27a6d5e

staging: wilc1000: fix typo · b9811891

由 Julia Lawall 提交于 5月 17, 2016

firmare -> firmware
Signed-off-by: NJulia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

b9811891

staging: wilc1000: fix spelling mistake: "interupts" -> "interrupts" · 95b8cb89

由 Colin Ian King 提交于 6月 23, 2016

trivial fix to spelling mistake in dev_err messages
Signed-off-by: NColin Ian King <colin.king@canonical.com>
Reviewed-by: NJulian Calaby <julian.calaby@gmail.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

95b8cb89

staging: wilc1000: remove unnecesary type cast of bss_type · 2938f208