- 27 9月, 2011 8 次提交
-
-
由 Franky Lin 提交于
remove unused module parameter brcmf_dongle_memsize and related code Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Use work queue to defer cfg80211 event handle jobs Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
Using a block statement to scope function variables is not common in linux kernel development. Browsed through the brcmfmac to remove those. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NFranky (Zhenhui) Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Annotated endianness on structures send/received to/from dongle. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Structure that is received from dongle has been annotated. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Structure that is send/received to/from dongle has been annotated. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Structures received from dongle have been annotated. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
- 21 9月, 2011 5 次提交
-
-
由 K. Y. Srinivasan 提交于
Now that the transformation of the util driver is complete, get rid of hv_cb_utils[] and other unneeded code. Signed-off-by: NK. Y. Srinivasan <kys@microsoft.com> Signed-off-by: NHaiyang Zhang <haiyangz@microsoft.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 K. Y. Srinivasan 提交于
Now, properly handle util services in the util driver and eliminate code that will not be necessary. In the current code, util services were all handled not as other vmbus devices (net, block) but rather through special handling (channel setup etc.). In this patch we handle all services using the standard Linux Driver Model. Signed-off-by: NK. Y. Srinivasan <kys@microsoft.com> Signed-off-by: NHaiyang Zhang <haiyangz@microsoft.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 K. Y. Srinivasan 提交于
In preparation for modifying the util driver to fully conform to the Linux Driver Model, perform some service specific init and de-init operations in util_probe()/util_remove() as opposed to in init_hyperv_utils()/exit_hyperv_utils() as is currently done. Signed-off-by: NK. Y. Srinivasan <kys@microsoft.com> Signed-off-by: NHaiyang Zhang <haiyangz@microsoft.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 David Kilroy 提交于
... instead of using IWEVCUSTOM. Use the defined events for michael mic failure, association request info and association response info. Signed-off-by: NDavid Kilroy <kilroyd@googlemail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 David Kilroy 提交于
Since the driver is now in-kernel remove support for building against ancient WIRELESS_EXT versions. Signed-off-by: NDavid Kilroy <kilroyd@googlemail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
- 20 9月, 2011 4 次提交
-
-
由 Kautuk Consul 提交于
If the usermode app does an ioctl over this serial device by using TIOCMIWAIT, then the code will wait by setting the current task state to TASK_INTERRUPTIBLE and then calling schedule(). This will be woken up by the qt2_process_modem_status on URB completion when the port_extra->shadowMSR is set to the new modem status. However, this could result in a lost wakeup scenario due to a race in the logic in the qt2_ioctl(TIOCMIWAIT) loop and the URB completion for new modem status in qt2_process_modem_status. Due to this, the usermode app's task will continue to sleep despite a change in the modem status. Signed-off-by: NKautuk Consul <consul.kautuk@gmail.com> Cc: stable <stable@kernel.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Kevin McKinney 提交于
In the first alteration, the MAX_CNTL_PKT_SIZE is the maximum size of the control packet in ->Adapter->txctlpacket[] which is defined in InitAdapter(). This caps the size of kmalloc memory allocation. In the second change, this max cap fixes a potential memory corruption bug when subsequent memset and memcpy calls are invoked. Signed-off-by: NKevin McKinney <klmckinney1@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Kevin McKinney 提交于
If IoBuffer.InputLength is zero then this will cause an Oops when we dereference the ZERO_SIZE_PTR. Or if it's smaller than sizeof(struct link_request) then we would get memory corruption when we set ->PLength in CopyBufferToControlPacket(). Signed-off-by: NKevin McKinney <klmckinney1@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alexander Thomas 提交于
Revert changes in definitions that were submitted on May 11 2011 and committed on June 7 2011. No reason for the change in these values was given in the patch comment, it broke compatibility with older versions, and was difficult to detect by the simultaneous move of the lines of code. Signed-off-by: NAlexander Thomas <alexander.thomas@esaturnus.com> Cc: stable <stable@kernel.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
- 17 9月, 2011 23 次提交
-
-
由 Roland Vossen 提交于
Structure that is received from dongle has been annotated. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
Several global variables that were set but never initialized have been removed. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NFranky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
The source file contained some static global variables that were needed per device handled by the driver. They have moved to the struct brcmf_sdio_dev for that reason. Reviewed-by: NFranky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
The macro was only used once in the code and has been removed for clarity reading the code. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NFranky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
The brcmfmac had a module parameter which could be used to override the name of the wireless net device name and it defaults to 'wlan'. This is considered rather useless, because what is in a name. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NFranky (Zhenhui) Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Use macro to replace global variable for ARP offload configuration Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
_brcmf_sysioc_thread is handling deferred jobs from 4 callers. Use work queue to handle set_mac_address/set_multicast_list requests. brcmf_add_if/brcmf_del_if never run in atomic context and can call brcmf_op_if directly. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Structure that is sent to the dongle has been annotated. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Structure that is sent to the dongle has been annotated. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Structures that are sent to the dongle have been annotated. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Three structures that are sent to the dongle have been annotated for correct endianness: struct brcmf_iscan_params, brcmf_scan_params, brcmf_cfg80211_scan_req. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
brcmf_dev_ioctl() does not provide a user space buffer to brcmf_netdev_ioctl_priv. Hence these calls are not necessary. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Removed a function parameter since it was redundant. Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Remove unnecessary global variable Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Remove unnecessary global variable Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Use constant values for packet filter configuration Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Remove brcmf_deferred_tx as it should be always 1 Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Replaced the global variable with macro. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Dead code removal. Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NFranky Lin <frankyl@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Julia Lawall 提交于
Sizeof a pointer-typed expression returns the size of the pointer, not that of the pointed data. The semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression *e; type T; identifier f; @@ f(...,(T)e,..., -sizeof(e) +sizeof(*e) ,...) // </smpl> Signed-off-by: NJulia Lawall <julia@diku.dk> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Leonid V. Fedorenchik 提交于
Fix long function prototypes and line up arguments nicely. Signed-off-by: NLeonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Leonid V. Fedorenchik 提交于
Fix indent of function arguments. Signed-off-by: NLeonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Leonid V. Fedorenchik 提交于
Line up integer constant with others. Signed-off-by: NLeonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-