- 04 Feb, 2011 34 commits
-
-
Alan Cox authored
Signed-off-by:
Alan Cox <alan@linux.intel.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Alan Cox authored
Signed-off-by:
-
Alan Cox authored
Signed-off-by:
-
Tomas Winkler authored
Add first level indentation before revamping the functions This of course breaks 80 characters limit but it will be fixed through the revamp Signed-off-by:
Tomas Winkler <tomas.winkler@intel.com> Signed-off-by:
-
Tomas Winkler authored
easyoss_fops are only accessed from within easycap_sound_oss file Signed-off-by:
-
Tomas Winkler authored
Signed-off-by:
-
Tomas Winkler authored
rc is used extensively in code as return code variable so it is better not shadowing it in macros Signed-off-by:
-
Tomas Winkler authored
the underscored types should be used in user space headers only Signed-off-by:
-
Tomas Winkler authored
naming variable s16 is confusing since it is also a type name. Signed-off-by:
-
Tomas Winkler authored
naming variable u8 is confusing since it is also a type name. Signed-off-by:
-
Tomas Winkler authored
1. naming variable s32 is confusing since it is also a type name. 2. use s32 instead of __s32, the later is for user space Signed-off-by:
-
Tomas Winkler authored
remove intializations to NULL where not needed and let the compiler find flows with unitilized variables. Fix one such flow in easycap_vma_fault function Signed-off-by:
-
Tomas Winkler authored
use idiom 'if (rc)' for checking return value instead of if (0 != rc) Signed-off-by:
-
Tomas Winkler authored
fix style issue: if (NULL != pdata_urb->purb) { created by the patch: 'staging/easycap: don't cast NULL pointer' After dropping the casting there is no longer 80 columns limitation Reported-by:Dan Carpenter <error27@gmail.com> Signed-off-by:
-
Tomas Winkler authored
for sake of readability replace #if defined with #ifdef and #if (!defined with #ifndef Signed-off-by:
-
Tomas Winkler authored
we can kill this option for in-kernel driver Signed-off-by:
-
Tomas Winkler authored
Signed-off-by:
-
Tomas Winkler authored
Signed-off-by:
-
Marek Belisko authored
dsp_img_info->version and requested_version have same type so additional temporary variable creation could be omitted because variables could be compares directly. Signed-off-by:
Marek Belisko <marek.belisko@open-nandra.com> Signed-off-by:
-
Marek Belisko authored
Same check was done on three places which make code unreadable. Put repeat routine to separate function. Signed-off-by:
-
Marek Belisko authored
Remove variables which was defined and assigned but never used in function. Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Remove code which was under #if 0. Signed-off-by:
-
- 02 Feb, 2011 6 commits
-
-
Joe Perches authored
Just use 0. Signed-off-by:
Joe Perches <joe@perches.com> Signed-off-by:
-
Joe Perches authored
Just use the status variable as a test not a comparison. Signed-off-by:
-
Joe Perches authored
Signed-off-by:
-
Joe Perches authored
Signed-off-by:
-
Joe Perches authored
Signed-off-by:
-
Joe Perches authored
Signed-off-by:
-
