summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Staging: hv: check return value of osd_PageAlloc()Bill Pemberton2010-05-11
| | | | | | | | | | The return value of osd_PageAlloc() was checked using an ASSERT(). Change that to more useful behaviour. Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: hv: remove ASSERT()s in ChannelMgt.cBill Pemberton2010-05-11
| | | | | | | Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: hv: Remove check for NULL before calling kfree()Bill Pemberton2010-05-11
| | | | | | | | | | kfree() knows how to deal with NULL, so there's no reason to check for NULL before passing something to it. Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: dt3155v4l: fix some sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | | | Mostly some __iomem markings and some static functions as well. Cc: Marin Mitov <mitov@issp.bas.bg Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: dt3155v4l: Adding the missing linux/delay.hMarin Mitov2010-05-11
| | | | | | | | | The prototypes of udelay() and msleep() are in linux/delay.h, so include it. Reported-by: Randy Dunlap <randy.dunlap@oracle.com> Signed-off-by: Marin Mitov <mitov@issp.bas.bg> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: vt6656: code cleanup, resolved sparse findingAndres More2010-05-11
| | | | | | | Cleared sparse warning 'Using plain integer as NULL pointer' Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: vt6656: code cleanup, removed OUT definitionAndres More2010-05-11
| | | | | | | Remoted empty OUT define in ttype.h and its usage across the code. Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: vt6656: removed SUCCESS define as it is not usedAndres More2010-05-11
| | | | | Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: arlan: fixed unnecessary whitespace style issue in arlan-main.cPrashant P. Shah2010-05-11
| | | | | | | | This is a patch to the arlan-main.c file that fixes the unnecessary whitespace issues found by the checkpatch.pl tool. Signed-off-by: Prashant P. Shah <pshah.mumbai@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: cxt1e1: fix up one remaining THIS_MODULE usagePeter Hüwe2010-05-11
| | | | | | | Copied from original Patch by Randy Dunlap Signed-off-by: Peter Huewe <peterhuewe@gmx.de> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: rtl8192u: Check kmalloc return value before use the buffer in ↵wzt.wzt@gmail.com2010-05-11
| | | | | | | | | ieee80211_softmac.c Check kmalloc return value before use the buffer. Signed-off-by: Zhitong Wang <zhitong.wangzt@alibaba-inc.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: cb_pcidas64: fix up build warningsGreg Kroah-Hartman2010-05-11
| | | | | | | | | | | Now that the 8255.h file is being included properly, we were using the incorrect function prototypes, which causes a build warning now. This fixes it up and preserves the __iomem markings that sparse wants to see. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: 8255: fix up previous static markings.Greg Kroah-Hartman2010-05-11
| | | | | | | | | | | | | Ian pointed out that exported symbols should not be marked as static :) Fixed this up by properly including the 8255.h file, and fixing the function prototypes there, as the CONFIG variables were not getting set so users of the header file were seeing the incorrect function prototypes. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: line6: driver: fix up sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | | | minor stuff. Cc: Markus Grabner <grabner@icg.tugraz.at> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: industrialio-trigger.c: minor fixupsGreg Kroah-Hartman2010-05-11
| | | | | | | | | | We needed to include a header file that declared the functions that are being exported in this file. Also fix up an indentation problem, and some sparse warnings. Cc: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: ring_sw.c: fix up sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | | | NULL usage, static stuff, etc. Cc: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: industrialio-ring.c: fix up sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | Cc: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: accel: fix up some sparse warnings.Greg Kroah-Hartman2010-05-11
| | | | | | | Minor stuff (static, NULL, etc.) Cc: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: trigger: fix up some global variablesGreg Kroah-Hartman2010-05-11
| | | | | | | These should be static. Cc: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: light: tsl2563: fix static sparse warningGreg Kroah-Hartman2010-05-11
| | | | | Cc: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: adc: max1363_ring.c: fix up sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | Cc: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: max1363_core: fix bug in kzalloc callGreg Kroah-Hartman2010-05-11
| | | | | | | The operands were switched around :( Cc: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: adc: max1363_core: fix up some sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | | | Also fix a minor coding style issue. Cc: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: light trivial whitespace fixJonathan Cameron2010-05-11
| | | | | Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: accelerometers trivial checkpatch related fixesJonathan Cameron2010-05-11
| | | | | Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: iio: Trivial - remove pointless semi colon (checkpatch found)Jonathan Cameron2010-05-11
| | | | | Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: hv: add basic ethtool supportStephen Hemminger2010-05-11
| | | | | | | | | Ethtool allows querying device information and controlling parameters. For now just add ability to turn on/off scatter/gather. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Acked-by: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: hv: add transmit flow controlStephen Hemminger2010-05-11
| | | | | | | | | Keep track of the number of pages sent over transmit and stop before going over. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Acked-by: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: hv: transmit scatter gather supportStephen Hemminger2010-05-11
| | | | | | | | | | | | The transmit management of pages was confusing for handling fragmented SKB's. (But since NETIF_F_SG was never set, the code was never hit). The parameter AdditionalRequestPageBufferCount is always one, (and leads to ugly code), so just inline and add comments. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Acked-by: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: hv: use existing Ethernet header sizeStephen Hemminger2010-05-11
| | | | | | | | | Use ETH_ALEN to indicate that MAC address is Ethernet. Also use Linux printk format for mac addresses. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Acked-by: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: hv: ring parameterStephen Hemminger2010-05-11
| | | | | | | | | The ring size parameter should be number of pages (not bytes). Add module parameter information as well. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Acked-by: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: hv: cleanup network driverStephen Hemminger2010-05-11
| | | | | | | | | | | Minor stuff: * Add module description * Remove variable set but never used. * Move variable inside conditional Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Acked-by: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: hwdrv_apci3xxx.c: loads of sparse cleanupsGreg Kroah-Hartman2010-05-11
| | | | | | | | __iomem pointer fixes, and static cleanups. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: hwdrv_apci3501.h: make a variable static.Greg Kroah-Hartman2010-05-11
| | | | | | | | Cleans up the sparse warning. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: hwdrv_apci2032.c: static sparse fixGreg Kroah-Hartman2010-05-11
| | | | | | Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: hwdrv_apci1564.c: static sparse cleanups.Greg Kroah-Hartman2010-05-11
| | | | | | Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: hwdrv_apci1500.c: sparse static cleanupsGreg Kroah-Hartman2010-05-11
| | | | | | | | Mark a bunch of variables and functions as static. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: hwdrv_apci1032.c: sparse warning fixupsGreg Kroah-Hartman2010-05-11
| | | | | | | | Mark a variable as static. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: hwdrv_apci035.h: fix sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | | | | This let us delete two variables and mark one as static. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: hwdrv_apci035.c: fix sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | | | | Some variables should be static. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: adl_pci9111.c: fix sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | | | | divisor and other problems fixed. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: adl_pci9118.c: fix unsigned problem with divisorsGreg Kroah-Hartman2010-05-11
| | | | | | Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: adv_pci1710.c: fix unsigned problem with divisorsGreg Kroah-Hartman2010-05-11
| | | | | | Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: amplc_dio200.c: fix NULL sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | | Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: cb_pcidas64.c: fix sparse warningsGreg Kroah-Hartman2010-05-11
| | | | | | | | This fixes a bunch of iomem and other sparse warnings. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: das1800.c: fix a locking error on the error path.Greg Kroah-Hartman2010-05-11
| | | | | | Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: plx9080.h: properly mark iomem variablesGreg Kroah-Hartman2010-05-11
| | | | | | | | It's a __iomem *, so mark it as such. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: quatech_daqp_cs.c: fix up the irqGreg Kroah-Hartman2010-05-11
| | | | | | | | | The irq needs to return the correct type. Also fix up some other sparse warnings that were found. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: addi_amcc_s5933.h: sparse cleanupGreg Kroah-Hartman2010-05-11
| | | | | | | | Mark a variable static that does not need to be global. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: comedi: addi_common.c: sparse cleanupsGreg Kroah-Hartman2010-05-11
| | | | | | | | | | | Now that we are properly marking the variable as __iomem, don't cast it. Also fix up some other sparse warnings. Cc: Ian Abbott <abbotti@mev.co.uk> Cc: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
generated by cgit v1.2.3 (git 2.25.1) at 2026-05-01 08:54:53 +0000