- 11 Jul, 2012 14 commits
-
-
Miguel Gómez authored
Use a goto to remove one level of indentation, fixing style warnings. Signed-off-by:
Miguel Gómez <magomez@igalia.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Miguel Gómez authored
Rework some conditions to reduce indentation and fix style warnings. Signed-off-by:
-
Miguel Gómez authored
Refactor XGI_GetVBType() to reduce indentation and fix style warnings. Signed-off-by:
-
Miguel Gómez authored
Refactor some conditions to reduce indentation and fix style warnings. Signed-off-by:
-
Miguel Gómez authored
Rework some conditions to reduce indentation and fix style warnings. Signed-off-by:
-
Miguel Gómez authored
Signed-off-by:
-
Miguel Gómez authored
Signed-off-by:
-
Miguel Gómez authored
Simplify the code and make it more readable. Signed-off-by:
-
Miguel Gómez authored
The switch performs no action so it can be removed. Signed-off-by:
-
Miguel Gómez authored
Signed-off-by:
-
Miguel Gómez authored
Struct _chswtable and the variable mychswtable are not used anywhere, so remove them. Signed-off-by:
-
Miguel Gómez authored
The piece of code that checks for LCDA in xgifb_probe() just checks for some register values but doesn't really do anything in response to them (the actions that should be executed are commented). As nothing is really being done, the code can be safely removed. Signed-off-by:
-
Miguel Gómez authored
LCD_TYPEs are taken from drivers/video/sis/sis.h, so this enumeration is not needed. Signed-off-by:
-
Miguel Gómez authored
Remove commented code and useless comments. Leave only those with relevant hints to the code. Signed-off-by:
-
- 09 Jul, 2012 26 commits
-
-
Greg Kroah-Hartman authored
Merge tag 'togreg_8_7_2012' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio into staging-next Mixed back of updates and trivial examples fixes. Nothing terribly interesting in this one. 8 July 2012
-
Greg Kroah-Hartman authored
Merge tag 'fixes-togreg_8_7_2012' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio into staging-next Mixed bag of IIO fixes. Quite a few cases of interrupt handlers returning errors and a few more specific bits and bobs. Most of these only effect fairly obscure error paths. The IRQF_ONESHOT one may cause trouble given the requests will now be rejected. Jonathan 8_7_2012
-
Devendra Naga authored
if no case matches we are simply asserting and doing break. and i think we may need to return that -ENODEV , no device is present, rather assert'ing. Signed-off-by:
Devendra Naga <devendra.aaru@gmail.com> Signed-off-by:
-
Devendra Naga authored
As the private pointer is valid at the remove of driver, and remove wont' be called if probe fails, so no point for checking of ASSERT Signed-off-by:
-
Justin P. Mattock authored
Signed-off-by:
Justin P. Mattock <justinmattock@gmail.com> Signed-off-by:
-
Marcos Paulo de Souza authored
All these macros were reported by forgotten-macros tool (https://github.com/marcosps/forgotten_macros). Signed-off-by:
Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by:
-
Marcos Paulo de Souza authored
These macros were reported by forgotten-macros tool (https://github.com/marcosps/forgotten_macros). Signed-off-by:
-
Marcos Paulo de Souza authored
These macros were reported by forgotten-macros tool (https://github.com/marcosps/forgotten_macros). Signed-off-by:
-
Marcos Paulo de Souza authored
These macros were reported by forgotte-macros tool (https://github.com/marcosps/forgotten_macros). Signed-off-by:
-
Marcos Paulo de Souza authored
These macros were reported by forgotten-macros tool (https://github.com/marcosps/forgotten_macros). Signed-off-by:
-
Marcos Paulo de Souza authored
This commit removes code that will never be executed by vt6655 driver. Was the forgotten-macros tool(https://github.com/marcosps/forgotten_macros) who reported these blocks for us. Signed-off-by:
-
Marcos Paulo de Souza authored
These macros were reported by forgotten-macros tool (https://github.com/marcosps/forgotten_macros). Signed-off-by:
-
Marcos Paulo de Souza authored
These macros were reported by forgotten-macros tool (https://github.com/marcosps/forgotten_macros). Signed-off-by:
-
Marcos Paulo de Souza authored
Signed-off-by:
-
Marcos Paulo de Souza authored
The unused macros were reporteds by forgotten-macros tool(https://github.com/marcosps/forgotten_macros). Signed-off-by:
-
Marcos Paulo de Souza authored
The commented macros are reported by forgotten-macros tool(https://github.com/marcosps/forgotten_macros). Signed-off-by:
-
Cruz Julian Bishop authored
This fixes the following warnings: 1: Changes a printk(KERN_ERR) call to a pr_err call in line 785 2: Changes a printk(KERN_ERR) call to a pr_err call in line 791 3: Changes a printk(KERN_ERR) call to a pr_err call in line 798 4: Reduces line length below 80 at line 785 (Not intended) 5: Reduces line length below 80 at line 798 (Not intended) Signed-off-by:
Cruz Julian Bishop <cruz@massive-dynamics.biz> Signed-off-by:
-
Devendra Naga authored
as ret is assigned to the return of ft1000_poll, we dont need to initialise ret with STATUS_SUCCESS. Signed-off-by:
-
Devendra Naga authored
The below checkpatch warns fixed, drivers/staging/ft1000/ft1000-pcmcia/ft1000_proc.c:38: ERROR: that open brace { should be on the previous line drivers/staging/ft1000/ft1000-pcmcia/ft1000_proc.c:45: ERROR: that open brace { should be on the previous line drivers/staging/ft1000/ft1000-pcmcia/ft1000_proc.c:57: ERROR: that open brace { should be on the previous line Signed-off-by: -
Devendra Naga authored
Signed-off-by:
-
Devendra Naga authored
as the if else statements enclose only one line, so braces around them are not needed. The following warning is fixed drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c:100: WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: -
Devendra Naga authored
sparse throws warning about the ft1000_poll_thread as drivers/staging/ft1000/ft1000-usb/ft1000_usb.c:40:5: warning: symbol 'ft1000_poll_thread' was not declared. Should it be static? Signed-off-by:
-
Devendra Naga authored
sparse warns about having 0 assigned to a pointer, fix it up by using NULL. The following sparse warning is fixed drivers/staging/ft1000/ft1000-usb/ft1000_debug.c:170:52: warning: Using plain integer as NULL pointer Signed-off-by:
-
Devendra Naga authored
as the if statement encloses only one line braces around it are not needed. The following warn fixed, WARNING: braces {} are not necessary for single statement blocks Signed-off-by: -
Devendra Naga authored
The following warning is fixed up. drivers/staging/rts_pstor/sd.c:190: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: -
Devendra Naga authored
for if else statements having single block no braces are needed fixed the following checkpatch warning drivers/staging/rts_pstor/sd.c:140: WARNING: braces {} are not necessary for any arm of this statement Signed-off-by:
-
