- 27 Oct, 2015 40 commits
-
-
Mike Rapoport authored
There are several occurencies of '#ifdef CAP_EXP<something>', spelled differently each time. None of these is ever defined and therefore they enclose dead code that can be removed. Signed-off-by:
Mike Rapoport <mike.rapoport@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Mike Rapoport authored
Both struct lynx_share and struct sm750_dev reprsent some parts of the SM750 graphics adapter. There is no point to keep these parts in different structures. Signed-off-by:
-
Mike Rapoport authored
The lynx_share field in lynx_cursor structure is never used and can be removed. Signed-off-by:
-
Mike Rapoport authored
Fix wrong indentation of a line in lynxfb_set_fbinfo Signed-off-by:
-
Mike Rapoport authored
Use sm750_dev as private driver data for the PCI device Signed-off-by:
-
Mike Rapoport authored
Use proper 'struct sm750_dev *' type instead of 'void *' Signed-off-by:
-
Mike Rapoport authored
Use sm750_dev rather than lynx_share as parameter for hw_sm750_map, hw_sm750_inithw and hw_sm750_initAccel functions. Signed-off-by:
-
Mike Rapoport authored
The hw_sm750_getVMSize wraps ddk750_getVMSize without any added functionality. Call ddk750_getVMSize directly and remove hw_sm750_getVMSize. Signed-off-by:
-
Mike Rapoport authored
Use sm750_dev instead of lynx_share in the private data of the frambuffer device. Signed-off-by:
-
Mike Rapoport authored
The PROC_SPEC* typedefs are not used and can be removed Signed-off-by:
-
Mike Rapoport authored
The struct sm750_state is never used on its own. Merge it into the struct sm750_dev. Signed-off-by:
-
Mike Rapoport authored
Use consistent naming for 'struct sm750_dev' variables Signed-off-by:
-
Mike Rapoport authored
The struct sm750_share represents the device state and name sm750_dev seems more appropriate. Signed-off-by:
-
Mike Rapoport authored
There is no actual need to split type definitions and global function declarations between lots of small headers. Signed-off-by:
-
Mike Rapoport authored
The smi_indent variable is only declared, but never used. Remove it. Signed-off-by:
-
Mike Rapoport authored
The custom macro PADDING differs from standard kernel ALIGN only in parameters order. Replace PADDING with ALIGN and remove it's definition. Signed-off-by:
-
Mike Rapoport authored
The SPC_SM750 define is not used and can be removed Signed-off-by:
-
Mike Rapoport authored
The methods in lynx_cursor always use the same implementation and there is no point to use redirection rather than direct function call. Signed-off-by:
-
Alison Schofield authored
Remove unneeded forward function declarations. Addresses checkpatch.pl: WARNING: externs should be avoided in .c files Signed-off-by:
Alison Schofield <amsfield22@gmail.com> Signed-off-by:
-
Alison Schofield authored
Move function types to be adjacent to the function names in declarations to improve readability. Signed-off-by:
-
Alison Schofield authored
Adds required spacing around operands to improve readability. Corrects checkpatch.pl ERROR: spaces required * Signed-off-by:
-
Alison Schofield authored
Reworked comments per kernel coding style to improve readability. Corrects checkpatch.pl: WARNING: Block comments use * on subsequent lines WARNING: line over 80 characters Signed-off-by:
-
Amitoj Kaur Chawla authored
Remove unnecessary variable that can be replaced with a single line of code. The semantic patch used to find this is: // <smpl> @@ expression ret; @@ - if (ret) return ret; - return 0; + return ret; @@ local idexpression ret; expression e; @@ - ret = e; - return ret; + return e; @@ type T; identifier i; expression E; @@ - T i = E; ... when != i @@ type T; identifier i; @@ - T i; ... when != i // </smpl> Signed-off-by:
Amitoj Kaur Chawla <amitoj1606@gmail.com> Signed-off-by:
-
Amitoj Kaur Chawla authored
Drop unused function that is defined but is not called anywhere. Signed-off-by:
-
Amitoj Kaur Chawla authored
Remove unnecessary variables that can be replaced with a single line of code. The semantic patch used to find this is: // <smpl> @@ expression ret; @@ - if (ret) return ret; - return 0; + return ret; @@ local idexpression ret; expression e; @@ - ret = e; - return ret; + return e; @@ type T; identifier i; expression E; @@ - T i = E; ... when != i @@ type T; identifier i; @@ - T i; ... when != i // </smpl> Signed-off-by:
-
Shraddha Barke authored
Remove declaration of function lprocfs_wr_timeouts from header file since it is not used Signed-off-by:
Shraddha Barke <shraddha.6596@gmail.com> Signed-off-by:
-
Shraddha Barke authored
Remove the functions which are defined but not used anywhere Signed-off-by:
-
Eva Rachel Retuya authored
Fix alignment issues by properly indenting function parameters in accordance to the kernel coding style. Checkpatch pointed out this issue. CHECK: Alignment should match open parenthesis Signed-off-by:
Eva Rachel Retuya <eraretuya@gmail.com> Signed-off-by:
-
Amitoj Kaur Chawla authored
Remove the export symbol for static function. The semantic patch used to find this is: // <smpl> @r@ type T; identifier f; @@ static T f (...) { ... } @@ identifier r.f; declarer name EXPORT_SYMBOL; @@ -EXPORT_SYMBOL(f); // </smpl> Signed-off-by: -
Amarjargal Gundjalam authored
This patch fixes the checkpatch issue: CHECK: Logical continuations should be on the previous line Signed-off-by:
Amarjargal Gundjalam <amarjargal.gundjalam@gmail.com> Signed-off-by:
-
Amarjargal Gundjalam authored
This patch fixes some of the checkpatch issue: CHECK: Alignment should match open parenthesis Signed-off-by:
-
Amarjargal Gundjalam authored
This patch fixes checkpatch issues: CHECK: Blank lines aren't necessary before a close brace '}' CHECK: Please don't use multiple blank lines Signed-off-by:
-
Amarjargal Gundjalam authored
This patch fixes some of the checkpatch issue: CHECK: Prefer using the BIT macro Signed-off-by:
-
Amarjargal Gundjalam authored
This patch fixes the checkpatch issue: CHECK: 'splitted' may be misspelled - perhaps 'split'? Signed-off-by:
-
Amarjargal Gundjalam authored
This patch fixes the checkpatch issue: CHECK: Comparison to NULL could be written Signed-off-by:
-
Burcin Akalin authored
Remove multiple blank lines. Problem found using checkpatch.pl "CHECK: Please don't use multiple blank lines" Signed-off-by:
Burcin Akalin <brcnakalin@gmail.com> Signed-off-by:
-
Burcin Akalin authored
Add space around operator '|'. Problem found using checkpatch.pl CHECK: spaces preferred around that '|' (ctx:VxV) Signed-off-by:
-
Alison Schofield authored
Replace one block comment with a single line comment. Follow style of previous comment and omit word channel, since it's clear the topic is channels. Addresses checkpatch.pl: WARNING: Block comments use * on subsequent lines Signed-off-by:
-
Alison Schofield authored
Replace lower case names in the enum dummy_scan_elements with uppercase names and a descriptive prefix: DUMMY_INDEX_ Signed-off-by:
-
Cristina Moraru authored
We need to check adis16060_spi_read return code to avoid feeding user space with bogus data. While, at it introduce out_unlock label in order to simplify locking on error path. Signed-off-by:
Cristina Moraru <cristina.moraru09@gmail.com> Signed-off-by:
-
