summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Staging: et131x: Kill off MMC_tAlan Cox2010-03-031-4/+4
| | | | | | | | The contents of MMC_t are clean so kill off the MMC_t typedef Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Kill off TXDMA_tAlan Cox2010-03-032-5/+5
| | | | | | | All the contents are clean so kill off the top level typedefs Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: GLOBAL_t is now sane so kill the GLOBAL_t itselfAlan Cox2010-03-032-7/+5
| | | | | | Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: kill EXP_ROMAlan Cox2010-03-031-21/+0
| | | | | | | This is another unused type for the bin Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Kill off the rxdma typeAlan Cox2010-03-032-6/+6
| | | | | | | | | We have a clean struct of this now so turn the top level typedefs into a struct Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Clean up rxdma_csrAlan Cox2010-03-032-63/+41
| | | | | | | | This is another set of flags as typedef that can be cleaned up. Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: fbr_desc is now only sane typesAlan Cox2010-03-032-20/+20
| | | | | | | | Clean up the typdef for fbr_desc itself so we know it is done Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: kill of FBR_WORD2Alan Cox2010-03-032-18/+5
| | | | | | | | | This is only used meaningfully as a definition, we never mask and fetch the bits apart Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: kill off the FBR_NUM_DES typeAlan Cox2010-03-031-12/+0
| | | | | | | | | This is another 10 bit value with the high bits clear, and where the type doesn't get used anywhere properly anyway Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: crystalhd: fix build on powerpcDave Airlie2010-03-031-0/+1
| | | | | | | | | Add necessary include to fix build on PowerPC Signed-off-by: Dave Airlie <airlied@redhat.com> Signed-off-by: Jarod Wilson <jarod@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: rtl8187se: fix if statementDan Carpenter2010-03-031-1/+1
| | | | | | | | | | I removed a misplace semicolon. It is clear from the indentation that TxPwrTracking87SE() was only supposed to be called if CheckTxPwrTracking() returned true. Signed-off-by: Dan Carpenter <error27@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: rt{2860,2870}sta: Use request_firmware() to load firmwareBen Hutchings2010-03-039-2210/+113
| | | | | | | | | | | | | | | | | When originally introduced into staging, these drivers had custom firmware-loading code which checked a version number and CRC at the end of each blob. This reintroduces those checks, using crc-ccitt instead of custom code. The removed firmware will be added to the linux-firmware.git repository. Based on work by Darren Salt <linux@youmustbejoking.demon.co.uk>. Signed-off-by: Ben Hutchings <ben@decadent.org.uk> Tested-By: Darren Salt <linux@youmustbejoking.demon.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: rt2870sta: constify RTUSBMultiWrite(), RTUSBFirmwareWrite()Ben Hutchings2010-03-032-8/+8
| | | | | | | | These functions do not modify the data they are passed. Signed-off-by: Ben Hutchings <ben@decadent.org.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: serqt_usb2: Two branches the same in qt_set_termios()Roel Kluin2010-03-031-1/+1
| | | | | | | | Regardless of the condition, the branches executed the same code Signed-off-by: Roel Kluin <roel.kluin@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: rt2860/sta_ioctl.c: Two branches the same in rt_ioctl_giwscan()Roel Kluin2010-03-031-4/+1
| | | | | | | | Regardless of the condition, the branches execute the same code Signed-off-by: Roel Kluin <roel.kluin@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl818: Remove unneeded chanlist_len checkIan Abbott2010-03-031-8/+0
| | | | | | | | | For AI command testing, the driver does not need to range-check the length of the channel list as the comedi core has already checked it. Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl816: Check channel list in AI command testIan Abbott2010-03-031-19/+36
| | | | | | | | | | | Check the channel list is valid in step 5 of the AI command test. Split function check_and_setup_channel_list() in two. Also, remove unnecessary chanlist_len tests in step 3 of the AI command test as the comedi core has already checked it. Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl812: Update MUX for multi-channel AI commandIan Abbott2010-03-031-3/+11
| | | | | | | | | For multi-channel AI commands, the interrupt handler is missing code to switch to the next channel. Add some (untested) code to handle this. Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: adv_pci1710: Remove unneeded chanlist_len checkIan Abbott2010-03-031-8/+0
| | | | | | | | | | For AI command testing, the driver does not need to range-check the length of the channel list as the comedi core has already checked it. Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: adv_pci1710: Correct AI scan count and channel checksIan Abbott2010-03-031-0/+7
| | | | | | | | | | | | For AI commands, the internal channel list used to check incoming data was only set up partially if the channel list contained repeats. Set it up fully. Also the current scan position was not updated properly when moving a block of data from the FIFO unless a whole number of scans was moved. Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl818: Correct AI scan counting and channel checksIan Abbott2010-03-031-4/+27
| | | | | | | | | | | | | | | | | | | For AI commands, the scan counter should be updated after every scan. It was being updated after every sample except for DMA mode where it was being updated after every repeated segment of the channel list. Also AI commands with multiple channels were being terminated with an error prematurely except in DMA mode. This was because the driver was comparing channel numbers received from the hardware (combined with the sample value) with the expected channel numbers to check for a "channel dropout". This test was failing incorrectly because the driver was not keeping the current position within the (repeated segment of the) channel list up to date. Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl816: Correct AI scan countingIan Abbott2010-03-031-1/+10
| | | | | | | | | | | For AI commands, the scan counter should be updated after every scan. In DMA mode, it was being updated after every repeated segment of the channel list. In non-DMA mode, it was being updated after every sample. Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl812: Correct AI scan countingIan Abbott2010-03-031-2/+6
| | | | | | | | | The AI scan counter should be updated after every completed scan, not after every channel. Keep track of current channel. Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: Comedi: G1-3 status registers not read in ni_gpct_to_660x_register()Roel Kluin2010-03-031-3/+3
| | | | | | | | Read from the right status register Signed-off-by: Roel Kluin <roel.kluin@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl730: Checkpatch cleanupsBenjamin Adolphi2010-03-031-3/+3
| | | | | | | | This fixes all checkpatch issues in the pcl730 comedi driver. Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl725: Checkpatch cleanupsBenjamin Adolphi2010-03-031-3/+3
| | | | | | | | This fixes all checkpatch issues in the pcl725 comedi driver. Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl724: Checkpatch cleanupsBenjamin Adolphi2010-03-031-14/+12
| | | | | | | | This fixes all checkpatch issues in the pcl724 comedi driver. Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: pcl711: Checkpatch cleanupsBenjamin Adolphi2010-03-031-17/+12
| | | | | | | | This fixes some checkpatch issues in the pcl711 comedi driver. Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: go7007: Eliminate useless codeJulia Lawall2010-03-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | The variable s is initialized twice to the same (side effect-free) expression. Drop one initialization. A simplified version of the semantic match that finds this problem is: (http://coccinelle.lip6.fr/) // <smpl> @forall@ idexpression *x; identifier f!=ERR_PTR; @@ x = f(...) ... when != x ( x = f(...,<+...x...+>,...) | * x = f(...) ) // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: rti800: Checkpatch cleanupsBenjamin Adolphi2010-03-031-21/+20
| | | | | | | | This fixes some checkpatch issues in the rti800 comedi driver. Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: c6xdigio: Checkpatch cleanupsBenjamin Adolphi2010-03-031-29/+28
| | | | | | | | | This fixes some checkpatch issues and some spelling mistakes in the c6xdigio comedi driver. Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: adq12b: Checkpatch cleanupsBenjamin Adolphi2010-03-031-24/+24
| | | | | | | | This fixes some checkpatch issues in the adq12b comedi driver. Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: rtl8192u: ieee80211: CurrentMPDUDensity assigned to itself in ↵Roel Kluin2010-03-031-1/+1
| | | | | | | | | | HTUseDefaultSetting() CurrentMPDUDensity was designed to itself. Signed-off-by: Roel Kluin <roel.kluin@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: sm7xx: remove the buggy 2D acceleration supportWu Zhangjin2010-03-035-1635/+11
| | | | | | | | | | | | | | When scrolling the screen on the console with the 2D acceleration support, the whole system may hang. for example, when copying some files from another machine to YeeLoong netbook with the sm7xx video driver via scp and when the screen output is enabled, the system may hang. Before the bug is fixed, remove the 2D acceleration! Signed-off-by: Wu Zhangjin <wuzhangjin@gmail.com> Cc: Teddy Wang <teddy.wang@siliconmotion.com.cn> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: fix typos "couter" -> "counter"Uwe Kleine-König2010-03-032-6/+6
| | | | | | | | | | This patch was generated by git grep -l 'couter' drivers/staging | xargs -r perl -p -i -e 's/couter/counter/' Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: fix typos "aquire" -> "acquire"Uwe Kleine-König2010-03-035-5/+5
| | | | | | | | | This patch was generated by git grep -E -i -l '[Aa]quire' drivers/staging | xargs -r perl -p -i -e 's/([Aa])quire/$1cquire/' Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: fix typos "selct" -> "select"Uwe Kleine-König2010-03-031-2/+2
| | | | | | | | | This patch was generated by git grep -E -i -l 's(le|el)ct' drivers/staging | xargs -r perl -p -i -e 's/([Ss])(le|el)ct/$1elect/' Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: fix typos "enalbe" -> "enable"Uwe Kleine-König2010-03-034-4/+4
| | | | | | | | | | This patch was generated by git grep -E -l 'enalbe' drivers/staging | xargs -r perl -p -i -e 's/enalbe/enable/g' Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: wlags49: don't use custom implementation of atoi()Andy Shevchenko2010-03-034-90/+39
| | | | | | | | | | | Kernel has its own method called simple_strtoul() to do such things. Here we are using simple_strtoul(value, NULL, 0) because in original function the recognized base is 10 or 16 and input data is assumed to be unsigned. Signed-off-by: Andy Shevchenko <ext-andriy.shevchenko@nokia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: sep: declare MODULE_FIRMWAREBen Hutchings2010-03-031-0/+3
| | | | | | Signed-off-by: Ben Hutchings <ben@decadent.org.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: sep: include driver name in firmware filenamesBen Hutchings2010-03-031-2/+2
| | | | | | | | | The current names "cache.image.bin" and "resident.image.bin" are far too generic. Signed-off-by: Ben Hutchings <ben@decadent.org.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: declare MODULE_FIRMWARE in various driversBen Hutchings2010-03-038-0/+19
| | | | | | Signed-off-by: Ben Hutchings <ben@decadent.org.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: dream: HTC Dream camera, not need sizeofPavel Vasilyev2010-03-031-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 <pavel@pavlinux.ru> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: vme/tsi148: Depend on VIRT_TO_BUSJeff Mahoney2010-03-031-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 <jeffm@suse.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: otus: fix assignment makes integer from pointer without a cast warningsJeff Mahoney2010-03-032-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 <jeffm@suse.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: rtl8192u: dont use -mhard-floatJeff Mahoney2010-03-031-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 <jeffm@suse.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: rtl8192e: Use skb_tail_pointerJeff Mahoney2010-03-032-4/+3
| | | | | | | | | rtl8192e uses skb->tail directly. This patch uses the tail pointer macros instead. Signed-off-by: Jeff Mahoney <jeffm@suse.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: rt2860: Use skb_tail_pointerJeff Mahoney2010-03-032-7/+7
| | | | | | | | | rt2860 uses skb->tail directly. This patch uses the tail pointer macros instead. Signed-off-by: Jeff Mahoney <jeffm@suse.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: ramzswap: Free memory when create_device is failedMinchan Kim2010-03-031-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 <minchan.kim@gmail.com> Acked-by: Nitin Gupta <ngupta@vflare.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: make i2c device id constantNémeth Márton2010-03-038-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 <nm127@freemail.hu> Cc: Julia Lawall <julia@diku.dk> Cc: cocci@diku.dk Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
OpenPOWER on IntegriCloud