| Age | Commit message (Collapse) | Author | Files | Lines |
|---|
|
BLOCKABLE() doesn't do what it's supposed to.
Signed-off-by: Jason Cooper <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
wl_event_thread.
Replace kernel_thread() with kthread_run(). Replace pid with tsk,
and exited with kthread_stop()/kthread_should_stop().
event_tsk, and tsk are NULL when their respective threads are not running.
Signed-off-by: Jason Cooper <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Replace kernel_thread() with kthread_run, kthread_stop() and
kthread_should_stop(). Remove sysioc_pid, sysioc_exit, and DAEMONIZE.
sysioc_tsk is NULL when not running.
Signed-off-by: Jason Cooper <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Replaced kernel_thread() with kthread_run(), kthread_stop(), and
kthread_should_stop(). Also removed all references to sysioc_pid and
sysioc_exit. DAEMONIZE removed because not used in dhd_linux.c.
sysioc_tsk is NULL when not running.
Signed-off-by: Jason Cooper <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
use kthread_run(), kthread_stop(), and kthread_should_stop(). Removes
dpc_pid, dpc_exited, and the call to DAEMONIZE.
dpc_tsk is NULL when not running.
Signed-off-by: Jason Cooper <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Replaced kernel_thread() with kthread_run(). Used kthread_should_stop()
in place of watchdog_exited completion. Replaced watchdog_pid with
struct task_struct.
watchdog_tsk is NULL when the task is not running.
Signed-off-by: Jason Cooper <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
remove unnecessary cflag, LINUX
Signed-off-by: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
remove unnecessary cflag, CONFIG_CFG80211
Signed-off-by: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
remove unnecessary cflag, BCMWPA2.
BCMDRIVER is already not used.
Signed-off-by: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Remove duplicated file, bcmutils.c under brcm80211/brcmfmac.
brcm80211/util/bcmutils.c will be shared between softmac
& fullmac. Makefile, and some other files are also modified
accordingly
Signed-off-by: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
This driver uses ioremap on regular memory to get an uncached mapping,
which causes problems on ARMv6 and higher due to aliasing with the
cached linar kernel mapping.
Make sure this gets fixed before the driver graduates from staging.
Cc: Felipe Contreras <[email protected]>
Signed-off-by: Arnd Bergmann <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
In the original code the interface name was IFNAMSIZ + 1, but that
caused problems in dhd_ifname2idx() which does:
strncmp(dhd->iflist[i]->name, name, IFNAMSIZ)
The wl_event_msg_t struct can only store 16 character names as well.
And thirdly there is a potential buffer overflow in dhd_op_if() because
if->net->name is IFNAMSIZ and we do:
strcpy(ifp->net->name, ifp->name);
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
The libata core changed this function so it needed to call a different
one.
See https://bugzilla.kernel.org/show_bug.cgi?id=19872 for details.
Reported-by: Heinz Wiesinger <[email protected]>
Tested-by: Heinz Wiesinger <[email protected]>
Cc: stable <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Purge unused wlc_id_name_entry
Signed-off-by: Brett Rudley <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
We're not using the packet engine anymore - goodbye.
Signed-off-by: Brett Rudley <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Bug fix for dual band problem. In particular it had an issue
to connect to 5G band AP.
Signed-off-by: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
On line 40 printk() needed an KERN_* facility level, so I gave it
INFO. Also, fixed a C99 comment error.
Signed-off-by: Tracey Dent <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
softmac
Combine dma_alloc routine for full and softmac
Signed-off-by: Brett Rudley <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Purge unused memlist debugging
Signed-off-by: Brett Rudley <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
It's not doing anything and is a bit silly.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
It's not even used for anything, not to mention, it is pretty silly.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use min_t() instead of min() in some places to make the
comparison explicit and resolve some compiler warnings.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use min_t() instead of min() in some places to make the
comparison explicit and resolve some compiler warnings.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use min_t() instead of min() in some places to make the comparison
explicit and resolve some compiler warnings.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use min_t() instead of min() in some places to make the comparison
explicit and resolve some compiler warnings.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]
|
|
Change the field types of dma_info_t to make comparing values easier
(and correct.) No need to keep rxbufsize as a u16, it can be an
unsigned int to make things easier.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
No one is using it.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
And use the kernel provided 'roundup' instead.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
And use the kernel provided IS_ALIGNED one instead.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
It's not used (and there's a kernel provided one if it's ever needed in
the future), so remove it.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use the real 'ARRAY_SIZE' definition instead.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use the real 'offsetof' definition instead.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use the kernel-provided version, this one is broken.
Note, there are more compiler warnings now, that's due to different
types being compared, which shows how the original macro was wrong in at
least one way. They need to be fixed up.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use the kernel-provided version, this one is broken.
Note, there are more compiler warnings now, that's due to different
types being compared, which shows how the original macro was wrong in at
least one way. They need to be fixed up.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Double casting is pretty pointless, don't do that.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
uint is already defined somewhere else, so just remove this version
of it.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use the kernel types, don't invent your own.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Use the kernel types, don't invent your own.
Cc: Brett Rudley <[email protected]>
Cc: Henry Ptasinski <[email protected]>
Cc: Nohee Ko <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
"status" is used as an index into the Adapter->PackInfo[] array, which
has NO_OF_QUEUES elements.
This code actually works OK. The SearchSfid() function always returns
a valid index or it returns NO_OF_QUEUES + 1. But it looks sloppy and
it makes the static checkers complain.
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
The error handling here is wrong. If psIntfAdapter were NULL then we
would have a NULL dereference in the debug output on the error path.
But this function is only called from usbbcm_device_probe() when
psIntfAdapter is non-NULL.
Since the check isn't needed and I removed it instead of fixing it.
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
bcm/InterfaceDld.c had a couple places which returned the number of
bytes remaining instead of -EFAULT.
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
I moved the check to see if "Adapter" was null in front of the
dereference.
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
The WIMAX_MAX_MTU macro is used in drivers/staging/bcm/CmHost.c like
this:
if (Adapter->PackInfo[uiSearchRuleIndex].uiMaxBucketSize < WIMAX_MAX_MTU * 8)
The multiplication by eight has precedence over the addition so the
macro needs parenthesis to work.
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
We assume that major is signed in register_control_device_interface().
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
There were a number of places in the bcm_char_ioctl() which returned the
number of bytes remaining to be copied instead of returning -EFAULT.
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Fixed file_operation struct, so that it is const.
Signed-off-by: Tracey Dent <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Put { on previous line, on line 34. Checkpatch.pl caught this.
Signed-off-by: Tracey Dent <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Replaced <modules>-objs with <modules>-y in Makefile.
Signed-off-by: Tracey Dent <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Changed <module>-objs to <module>-y n Makefile.
Signed-off-by: Tracey Dent <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|
|
Changed <module>-objs to <module>-y in Makefile.
Signed-off-by: Tracey Dent <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
|