linux-stable.git/drivers/staging/crystalhd, branch linux-3.2.y

staging: crystalhd/bc_dts_types.h: typedef cleanup

2011-09-16T18:12:56+00:00

Not much left out of this header file. All these typedefs can be found in stdint.h

Signed-off-by: Arvydas Sidorenko 
Signed-off-by: Greg Kroah-Hartman

drivers/staging/crystalhd/: Fix a coding style issues

2011-09-12T14:51:25+00:00

crystalhd_fw_if.h: indentation fix (spaces to tabs)
The rest are brackets.

NOTE: there are quite some 80 character warnings, but they look in place,
comments mostly on the right next to the constants and stuff like that. I
haven't touched them, since this rule is 'going away', but in case you would
like it to be fixed, let me know.

I'm sending 2 patches, but it they aren't connected in any way, so doesn't
matter the apply order. It's just a cleanup.

P.S sorry for the first lame patches a couple of days ago

Signed-off-by: Arvydas Sidorenko 
Signed-off-by: Greg Kroah-Hartman

staging: Remove unnecessary semicolons when switch (foo) {...};

2011-04-25T23:58:35+00:00

Done via perl script:

$ cat remove_semi_switch.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(switch\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 
Signed-off-by: Greg Kroah-Hartman

staging: crystalhd: remove unused typedef bc_bool_t

2011-04-20T20:48:24+00:00

Remove unused typedef bc_bool_t.

Signed-off-by: Nicolas Kaiser 
Reviewed-by: Jesper Juhl 
Signed-off-by: Greg Kroah-Hartman

Fix common misspellings

2011-03-31T14:26:23+00:00

Fixes generated by 'codespell' and manually reviewed.

Signed-off-by: Lucas De Marchi

staging: crystalhd: fix memory leaks

2011-03-14T19:23:34+00:00

Free resources before exit.

Signed-off-by: Alexander Beregalov 
Signed-off-by: Greg Kroah-Hartman

Staging: crystalhd: don't waste the last char of buffer

2011-03-14T18:47:59+00:00

pinfo->name is a 32 char buffer.  In the original code, the last char
wasn't fully utilized.

Signed-off-by: Dan Carpenter 
Signed-off-by: Greg Kroah-Hartman

Staging: crystalhd: change GFP_ATOMIC to GFP_KERNEL

2011-03-14T18:47:08+00:00

These two allocations are only called from the probe() path and there
aren't any locks held for probe().

Signed-off-by: Dan Carpenter 
Signed-off-by: Greg Kroah-Hartman

drivers/staging: Remove unnecessary casts of pci_get_drvdata

2010-11-16T20:06:47+00:00

Signed-off-by: Joe Perches 
Signed-off-by: Greg Kroah-Hartman

drivers/staging: Remove unnecessary semicolons

2010-11-16T20:06:47+00:00

Signed-off-by: Joe Perches 
Signed-off-by: Greg Kroah-Hartman