| 9fc86028fa21f8831c0fdc701732cf491da1202c |
|
10-Apr-2011 |
Joe Perches <joe@perches.com> |
staging: Remove unnecessary semicolons when if (foo) {...};
Done via perl script:
$ cat remove_semi_if.pl
my $match_balanced_parentheses = qr/(\((?:[^\(\)]++|(?-1))*\))/;
my $match_balanced_braces = qr/(\{(?:[^\{\}]++|(?-1))*\})/;
foreach my $file (@ARGV) {
my $f;
my $text;
my $oldtext;
next if ((-d $file));
open($f, '<', $file)
or die "$P: Can't open $file for read\n";
$oldtext = do { local($/) ; <$f> };
close($f);
next if ($oldtext eq "");
$text = $oldtext;
my $count = 0;
do {
$count = 0;
$count += $text =~ s@\b(if\s*${match_balanced_parentheses}\s*)${match_balanced_braces}\s*;@"$1$3"@egx;
} while ($count > 0);
if ($text ne $oldtext) {
my $newfile = $file;
open($f, '>', $newfile)
or die "$P: Can't open $newfile for write\n";
print $f $text;
close($f);
}
}
$
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 25985edcedea6396277003854657b5f3cb31a628 |
|
31-Mar-2011 |
Lucas De Marchi <lucas.demarchi@profusion.mobi> |
Fix common misspellings
Fixes generated by 'codespell' and manually reviewed.
Signed-off-by: Lucas De Marchi <lucas.demarchi@profusion.mobi>
|
| 4a499de2d13786de626c3f689022d7ec25230911 |
|
30-Oct-2010 |
Mariano Reingart <reingart@gmail.com> |
staging: vt6656: resolved checkpatch finding
removed a C99 '//' comment and added a space around '='
Signed-off-by: Mariano Reingart <reingart@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 203e4615ee9d9fa8d3506b9d0ef30095e4d5bc90 |
|
05-Aug-2010 |
Andres More <more.andres@gmail.com> |
staging: vt6656: removed custom definitions of Ethernet packet types
Removed some unused definitions of Ethernet packet types, also replaced
two of them with in-kernel counterparts from include/linux/if_ether.h
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| bd2bc4c763db8a9c6eeef0d2cf6f384514b6cb6c |
|
03-Aug-2010 |
Andres More <more.andres@gmail.com> |
staging: vt6656: added spaces around '||'
Cleared checkpatch ERROR: spaces required around that '||'
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 6487c49e8a739ae6020ccda9470f5837e792ab53 |
|
03-Aug-2010 |
Andres More <more.andres@gmail.com> |
staging: vt6656: removed NTSTATUS definition
Replaced NTSTATUS with int, as defined in a couple of places.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 22040bbf84ae5cc39b2b97f064c97592aa425d06 |
|
03-Aug-2010 |
Andres More <more.andres@gmail.com> |
staging: vt6656: simplified tests involving both multi/broad-casts
By definition the broadcast address is also a multicast address.
Lindented the modified section as required by checkpatch.pl.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 020c0a9a78f8d5215f09164f7e5c5252dea098f9 |
|
12-Jul-2010 |
Andres More <more.andres@gmail.com> |
staging: vt6656: replaced MAX_DATA_LEN with ETH_DATA_LEN
Removed custom macro with one from include/linux/if_ether.h.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| abad19d07627a8b67b441452b0450e178416d2d0 |
|
12-Jul-2010 |
Andres More <more.andres@gmail.com> |
staging: vt6656: replaced U_CRC_LEN with ETH_FCS_LEN
Removed custom macro with one from include/linux/if_ether.h.
FCS is an Ethernet field that holds the CRC value.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 24fe608a2d4d78cc1a5f5424a4db5c13e3a00983 |
|
12-Jul-2010 |
Andres More <more.andres@gmail.com> |
staging: vt6656: removed WPA_SM_Transtatus definition
Removed an always defined macro, perhaps used to patch the driver
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 4b50fb4042a9e39cd24b411546020d6b56ff519b |
|
23-Jun-2010 |
Andres More <more.andres@gmail.com> |
Staging: vt6656: replaced IS_BROADCAST_ADDRESS with is_broadcast_ether_addr
Replaced and removed the custom definition.
Minor checkpatch warnings on long lines and indentation were not cleared.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 2a1a17498bcaea1580a4fa051286bc97d4c5e730 |
|
21-Jun-2010 |
Andres More <more.andres@gmail.com> |
Staging: vt6656: removed '#if 0' definitions
Removed code in .c files that was being skipped by the preprocessor
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 21ec51f3df9fd3c83ce0a4ac6a18e5083984b8ae |
|
18-May-2010 |
Charles Clément <caratorn@gmail.com> |
Staging: vt6656: use ETH_HLEN macro instead of custom one
Replace custom header length definition U_HEADER_LEN by ETH_HLEN
from <linux/if_ether.h>. Also remove unused U_TYPE_LEN.
Signed-off-by: Charles Clément <caratorn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| cc856e61ee4ffb150ff352e3d6940978a2f819e8 |
|
18-May-2010 |
Andres More <more.andres@gmail.com> |
Staging: vt6656: removed custom UCHAR/USHORT/UINT/ULONG/ULONGLONG typedefs
Cleared all checkpatch warnings but 'do not add new typedefs' ones.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 0cbd8d9854284d3ff38d04aaa3ae726fb1c4a958 |
|
07-May-2010 |
Andres More <more.andres@gmail.com> |
staging: vt6656: code cleanup, removed HANDLE definition in ttype.h
Checkpatch warnings about using externs in .c files were not resolved,
neither some long lines on deeply nested code.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 6f8c13c7dbe0d51e62bdb1aa11c1c38a8f8e3133 |
|
05-May-2010 |
Andres More <more.andres@gmail.com> |
Staging: vt6656: code cleanup, removed OUT definition
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>
|
| 8611a29ab967bc197494db19d31994d1b5a26fdc |
|
01-May-2010 |
Andres More <more.andres@gmail.com> |
Staging: vt6656: removed VOID/PVOID definitions
Warnings about the usage of externs in .c files were not resolved here.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 592ccfebb3d7ae6d2fa367b97f080790befa3c6c |
|
17-Apr-2010 |
Andres More <more.andres@gmail.com> |
Staging: vt6656: Removed IN definition
Code cleanup, removed empty IN definition used to denote input parameters.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 9a0e756c5280750c23bd44d2b855a1f5442ea7b4 |
|
14-Apr-2010 |
Andres More <more.andres@gmail.com> |
Staging: vt6656: incorporated ETH_ALEN macro instead of custom one
Replaced custom U_ETHER_ADDR_LEN by ETH_ALEN from <linux/if_ether.h>.
Resolved checkpatch findings on the changed lines, mostly indentation.
Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| f77f13e22df4a40d237697df496152c8c37f3f2b |
|
29-Mar-2010 |
Gilles Espinasse <g.esp@free.fr> |
Fix comment and Kconfig typos for 'require' and 'fragment'
Signed-off-by: Gilles Espinasse <g.esp@free.fr>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
|
| ee93e1971dc2dbc3133962b3d914b18d7c93a681 |
|
16-Oct-2009 |
Roel Kluin <roel.kluin@gmail.com> |
Staging: vt6655: Correct unsigned bound issue
uNodeIndex is unsigned, check whether it is within bounds instead.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 8a3d91b02830dd7c58d27e5de65222219ac2163f |
|
12-Aug-2009 |
Jim Lieb <lieb@canonical.com> |
Staging: vt665x: remove tbit.h
Remove use of tbit macros and remove the header file.
Signed-off-by: Jim Lieb <lieb@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 3e362598fd3c7b8729afd0383a4cdf08849b020e |
|
12-Aug-2009 |
Jim Lieb <lieb@canonical.com> |
Staging: vt665x: Remove umem.h Part 2
Remove references to umem.h macros and refer directly to memcpy
functions. Delete the include file.
Signed-off-by: Jim Lieb <lieb@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 9d26d60f99855056a20ac7c0b68eb5ed04619831 |
|
12-Aug-2009 |
Jim Lieb <lieb@canonical.com> |
Staging: vt665x: Clean up include files, Part 1
Remove cplusplus lines from include files
Remove needless ifdefs on includes to conform with C
conventions. Remove misc commented code/includes
Update TODO
Signed-off-by: Jim Lieb <lieb@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 193a823caaf0e2a79a447014be00a6b70ed216a2 |
|
12-Aug-2009 |
Jim Lieb <lieb@canonical.com> |
Staging: vt665x: Typedef and macro cleanup Part 1
Clean up unused typedefs and macros to remove Win32'isms and
misc non-linux constructs. Text edits to referencing
source for less frequently used macros.
Signed-off-by: Jim Lieb <lieb@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|
| 92b96797118e5836294a6d42a5a8e10b86f50e3f |
|
13-Jun-2009 |
Forest Bond <forest@alittletooquiet.net> |
Staging: Add pristine upstream vt6656 driver sources to drivers/staging/vt6656.
Add pristine upstream vt6656 driver sources to drivers/staging/vt6656. These
files were copied from the driver directory in the upstream source archive,
available here:
http://www.viaarena.com/Driver/VT6656_Linux_src_v1.19_12_x86.zip
After copying, trailing whitespace was stripped. This is GPL-licensed code.
Signed-off-by: Forest Bond <forest@alittletooquiet.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
|