aboutsummaryrefslogtreecommitdiff
path: root/drivers
AgeCommit message (Collapse)AuthorFilesLines
2010-03-03Staging: sep: include driver name in firmware filenamesBen Hutchings1-2/+2
The current names "cache.image.bin" and "resident.image.bin" are far too generic. Signed-off-by: Ben Hutchings <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03staging: declare MODULE_FIRMWARE in various driversBen Hutchings8-0/+19
Signed-off-by: Ben Hutchings <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: dream: HTC Dream camera, not need sizeofPavel Vasilyev1-2/+1
http://bugzilla.kernel.org/show_bug.cgi?id=14825 sizeof(extlen), always will be sizeof( unit32_t) or 4 It seems that something is wrong?!?! Signed-off-by: Pavel Vasilyev <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: vme/tsi148: Depend on VIRT_TO_BUSJeff Mahoney1-0/+1
This driver depends on virt_to_bus working correctly, but it doesn't exist on ppc64 (and probably other arches). Signed-off-by: Jeff Mahoney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: otus: fix assignment makes integer from pointer without a cast warningsJeff Mahoney2-10/+4
This patch fixes some %x pointer printing to %p. It also uses the skb_tail_pointer and skb_mac_header macros for accessing thos members. Signed-off-by: Jeff Mahoney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: rtl8192u: dont use -mhard-floatJeff Mahoney1-1/+1
rtl8192u uses -mhard-float, which doesn't exist on ia64. Since in-kernel floating-point isn't allowed, this is implied anyway. Signed-off-by: Jeff Mahoney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: rtl8192e: Use skb_tail_pointerJeff Mahoney2-4/+3
rtl8192e uses skb->tail directly. This patch uses the tail pointer macros instead. Signed-off-by: Jeff Mahoney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: rt2860: Use skb_tail_pointerJeff Mahoney2-7/+7
rt2860 uses skb->tail directly. This patch uses the tail pointer macros instead. Signed-off-by: Jeff Mahoney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: ramzswap: Free memory when create_device is failedMinchan Kim1-8/+13
If create_device is failed, it can't free gendisk and request_queue of preceding devices. It cause memory leak. This patch fixes it. Signed-off-by: Minchan Kim <[email protected]> Acked-by: Nitin Gupta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03staging: make i2c device id constantNémeth Márton8-8/+8
The id_table field of the struct i2c_driver is constant in <linux/i2c.h> so it is worth to make the initialization data also constant. The semantic match that finds this kind of pattern is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r@ disable decl_init,const_decl_init; identifier I1, I2, x; @@ struct I1 { ... const struct I2 *x; ... }; @s@ identifier r.I1, y; identifier r.x, E; @@ struct I1 y = { .x = E, }; @c@ identifier r.I2; identifier s.E; @@ const struct I2 E[] = ... ; @depends on !c@ identifier r.I2; identifier s.E; @@ + const struct I2 E[] = ...; // </smpl> Signed-off-by: Németh Márton <[email protected]> Cc: Julia Lawall <[email protected]> Cc: [email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03staging: make USB device id constantNémeth Márton16-16/+16
The id_table field of the struct usb_device_id is constant in <linux/usb.h> so it is worth to make the initialization data also constant. The semantic match that finds this kind of pattern is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r@ disable decl_init,const_decl_init; identifier I1, I2, x; @@ struct I1 { ... const struct I2 *x; ... }; @s@ identifier r.I1, y; identifier r.x, E; @@ struct I1 y = { .x = E, }; @c@ identifier r.I2; identifier s.E; @@ const struct I2 E[] = ... ; @depends on !c@ identifier r.I2; identifier s.E; @@ + const struct I2 E[] = ...; // </smpl> Signed-off-by: Németh Márton <[email protected]> Cc: Julia Lawall <[email protected]> Cc: [email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03staging: make PCI device id constantNémeth Márton6-6/+6
The id_table field of the struct pci_driver is constant in <linux/pci.h> so it is worth to make the initialization data also constant. The semantic match that finds this kind of pattern is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r@ disable decl_init,const_decl_init; identifier I1, I2, x; @@ struct I1 { ... const struct I2 *x; ... }; @s@ identifier r.I1, y; identifier r.x, E; @@ struct I1 y = { .x = E, }; @c@ identifier r.I2; identifier s.E; @@ const struct I2 E[] = ... ; @depends on !c@ identifier r.I2; identifier s.E; @@ + const struct I2 E[] = ...; // </smpl> Signed-off-by: Németh Márton <[email protected]> Cc: Julia Lawall <[email protected]> Cc: [email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03staging: iio: Fix incorrect existence check for a shared event pointer.Jonathan Cameron1-8/+8
A second smatch detected error. First part fixes in a typo in the comment directly above that I noticed whilst trying to remember what this code actually does. Second part is the actual fix. I'm fairly amazed this one never caused trouble in testing as it is in one of the most common paths. Signed-off-by: Jonathan Cameron <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03staging: iio: Ensure mutex is correctly unlocked in __iio_push_eventJonathan Cameron1-1/+4
This error was picked up by running the smatch static checker over all the IIO subsytem. Signed-off-by: Jonathan Cameron <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: otus: zfTkipInit(): increment another pointerSimon Horman1-2/+3
Increment the pointer rather than its value. These appear to be logic errors. Thanks to Dan Carpenter for the first hunk of this change. Cc: Dan Carpenter <[email protected]> Signed-off-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: otus: Fix ZM_SEQ_DEBUG macro for no-debug case[email protected]2-7/+9
tcp_seq is only initialised in case where ZM_ENABLE_PERFORMANCE_EVALUATION is defined. So move the call to ZM_SEQ_DEBUG() and the decleration of tcp_seq in there too. This allows ZM_SEQ_DEBUG() to be removed from the non-ZM_ENABLE_PERFORMANCE_EVALUATION case in the header file. This resolves several compile warnings for the non-ZM_ENABLE_PERFORMANCE_EVALUATION case. However, the ZM_ENABLE_PERFORMANCE_EVALUATION case seems to be completely broken. $ gcc (Debian 4.4.2-8) 4.4.2 Copyright (C) 2009 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. $ make ... drivers/staging/otus/80211core/cagg.c: In function 'zfAggRxEnabled': drivers/staging/otus/80211core/cagg.c:1872: warning: left-hand operand of comma expression has no effect drivers/staging/otus/80211core/cagg.c:1872: warning: left-hand operand of comma expression has no effect drivers/staging/otus/80211core/cagg.c:1872: warning: statement with no effect ... Signed-off-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: otus: remove unused constants[email protected]1-133/+0
Signed-off-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: otus: remove unused variables[email protected]10-28/+1
Signed-off-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: otus: dont mix declarations and code[email protected]1-3/+5
$ gcc (Debian 4.4.2-8) 4.4.2 Copyright (C) 2009 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. $ make ... drivers/staging/otus/ioctl.c: In function 'usbdrv_wpa_ioctl': drivers/staging/otus/ioctl.c:2253: warning: ISO C90 forbids mixed declarations and code ... Signed-off-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03staging: crystalhd: trim register definesJarod Wilson1-11542/+0
We don't actually need most of the register defines to build, and most of the ones we don't need aren't currently interesting. We'll leave a full copy of all of them in libcrystalhd's source, and only include what we need and/or think might be interesting in the driver. Signed-off-by: Jarod Wilson <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03staging: crystalhd: add missing fixes for userspace lib buildJarod Wilson1-0/+6
I somehow managed to not actually include these two fixes in the submission that was committed to the staging tree. libcrystalhd should eventually be built against the kernel-provided header, and needs the stdint.h include. The VOID bit is to keep things in sync with the Mac OS X driver and library that Scott Davilla is also working on. Signed-off-by: Scott Davilla <[email protected]> Signed-off-by: Jarod Wilson <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: wlan-ng: Add select WEXT_PRIV to Kconfig to prevent build failurePeter Huewe1-0/+1
Without WEXT_PRIV set the p80211wext.c fails to build due to unknown fields in the iw_handler_def struct. Those fields are enclosed in WEXT_PRIV conditionals in the prototype of iw_handler_def in include/net/iw_handler.h Signed-off-by: Peter Huewe <[email protected]> Acked-by: Simon Horman <[email protected]> Cc: stable <[email protected]> [2.6.33 only] Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: rtl8192u: Add select WEXT_PRIV to Kconfig to prevent build failurePeter Huewe1-1/+2
Without WEXT_PRIV set the driver fails to build due to unknown fields in the iw_handler_def struct. Those fields are enclosed in WEXT_PRIV conditionals in the prototype of iw_handler_def in include/net/iw_handler.h Signed-off-by: Peter Huewe <[email protected]> Acked-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: otus: Add select WEXT_PRIV to Kconfig to prevent build failurePeter Huewe1-0/+2
Without WEXT_PRIV set the driver fails to build due to unknown fields in the iw_handler_def struct. Those fields are enclosed in WEXT_PRIV conditionals in the prototype of iw_handler_def in include/net/iw_handler.h Signed-off-by: Peter Huewe <[email protected]> Acked-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: asus_oled: Add NULL test for kmallocPeter Huewe1-0/+5
This patch adds a NULL test to check wether kmalloc was successful or not. Signed-off-by: Peter Huewe <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03staging: cx25821: avoid NULL pointer dereferenceAlexander Beregalov2-2/+4
Assign dev only if fh is not NULL. Signed-off-by: Alexander Beregalov <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: v56655: add missing parentheses (in comment)Roel Kluin1-1/+1
`!' has a higher precedence than `&' so parentheses are required. Signed-off-by: Roel Kluin <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: batman-adv: Update the README for mainline usage.Andrew Lunn1-37/+13
The module now only compiles inside mainline, so remove the comments about different versions of the kernel it can be used with. Also update comments about building with debug enabled and how to use the VIS data now that it no longer natively outputs dot or JSON. Signed-off-by: Andrew Lunn <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: Fixed pohmelfs regression because of per-bdi writeback.Evgeniy Polyakov2-2/+21
Since pohmelfs isn't tied to a single block device, it needs to setup a backing dev like nfs/btrfs/etc do. Signed-off-by: Jens Axboe <[email protected]> Signed-off-by: Evgeniy Polyakov <[email protected]> Cc: stable <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: pohmelfs: avoid null dereferenceDan Carpenter1-1/+1
err_out_put is for when the netfs_trans_alloc() succeeded. Signed-off-by: Dan Carpenter <[email protected]> Acked-by: Evgeniy Polyakov <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: pohmelfs: Remove uses of NIPQUAD, use %pI4Joe Perches1-2/+1
Signed-off-by: Joe Perches <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: sep: Fix use of legacy ioctl fopAlan Cox1-2/+2
SEP doesn't need lock_kernel. Signed-off-by: Alan Cox <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: sep: Fix build problems from header changesAlan Cox1-3/+4
Signed-off-by: Alan Cox <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: otus: 80211core: Fix &&/|| confusionRoel Kluin1-1/+1
This always evaluates to true. Signed-off-by: Roel Kluin <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: rar: Fix all coding style issues.Benjamin Adolphi2-107/+93
This fixes all coding style issues and some spelling mistakes. Signed-off-by: Benjamin Adolphi <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: rtl8192su: buffer overflow in r8192U_core.cDan Carpenter1-84/+43
There was a buffer overflow in the original code. rf_path was 2 and it should have been only 0 or 1. I don't have the hardware for this, so I can't test it. Looking at the code, there are two almost identical sections for updating the hal variables. The first one was clearly wrong and had the array overflow as well. The second one looked correct. I decided to use the second section as is except for whitespace changes. The differences between the two original sections: 1) The second one had more debug output. 2) The second one looped over rf_path instead of corrupting data. 3) The second one had these additional assigments. if (rf_path == 0) { priv->TxPowerLevelOFDM24G[i] = priv->RfTxPwrLevelOfdm1T[rf_path][i] ; priv->TxPowerLevelCCK[i] = priv->RfTxPwrLevelCck[rf_path][i]; } Signed-off-by: Dan Carpenter <[email protected]> Cc: Jerry chuang <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: vt6655/device_main.c: use %pM to shown MAC addressH Hartley Sweeten1-4/+1
Trivial, use the %pM kernel extension to display the MAC address. Signed-off-by: H Hartley Sweeten <[email protected]> Cc: Forest Bond <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03r8192U_core.c: use %pM to shown MAC addressH Hartley Sweeten1-8/+6
Trivial, use the %pM kernel extension to display the MAC address. Signed-off-by: H Hartley Sweeten <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: r8192E_core.c: use %pM to shown MAC addressH Hartley Sweeten1-4/+1
Trivial, use the %pM kernel extension to display the MAC address. Signed-off-by: H Hartley Sweeten <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: et131x_netdev.c: use %pM to shown MAC addressH Hartley Sweeten1-6/+2
Trivial, use the %pM kernel extension to display the MAC address. Signed-off-by: H Hartley Sweeten <[email protected]> Cc: Olaf Hartmann <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: otus: off by one in usbdrvwext_siwessid()Dan Carpenter1-1/+1
A 33 char ESSID is too long and it could cause a buffer overflow a couple lines below when we put a NULL terminator on the end. Signed-off-by: Dan Carpenter <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: cx25821: off by one in cx25821_enum_input()Dan Carpenter1-1/+1
The INPUT(n) macro indexes an array of size 2. Signed-off-by: Dan Carpenter <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: cx25821: fix double unlock in medusa_video_init()Dan Carpenter1-3/+1
medusa_set_videostandard() takes the lock but it always drops it before returning. This was found with a static checker and compile tested only. :/ Signed-off-by: Dan Carpenter <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: comedi: reorder check in pcmmio_attach()Dan Carpenter1-1/+1
The check for dio_num_asics is used to determine if there is more than 1 irq. If it is false then irq[1] is past the end of the array. Signed-off-by: Dan Carpenter <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: comedi: off by one in doDevConfig()Dan Carpenter1-1/+1
Changed > to >= to avoid array overflow. Signed-off-by: Dan Carpenter <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: ramzswap: Minor spelling fixesC yp7-13/+13
Also removed an extra semicolon. Signed-off-by: Cyp <[email protected]> Cc: Nitin Gupta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: dt3155: remove compat codeSimon Horman6-1073/+8
Remove compatibility code as this is not an older version of the kernel. Signed-off-by: Simon Horman <[email protected]> Cc: Scott Smedley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: rtl8192su, rtl8192u: use min_t() in store_debug_level()Simon Horman2-2/+2
sizeof() returns a size_t but the other types involved are unsigned long, so using min() results in a warning. As sizeof() is called on an 11 character buffer defined immediately above unsigned long is obviously wide enough for the result. Signed-off-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: rtl8192e: remove some functions from the __exit sectionSimon Horman3-3/+3
ieee80211_crypto_tkip_exit(), ieee80211_crypto_deinit() and ieee80211_crypto_ccmp_exit() are called by ieee80211_rtl_init() which are in section __init, so they can't be in section __exit. Signed-off-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2010-03-03Staging: rtl8192e: print the elements of tx_pn and rx_pn not the arrays ↵Simon Horman1-6/+13
themselves Signed-off-by: Simon Horman <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>