- 04 Feb, 2011 27 commits
-
-
Tomas Winkler authored
the underscored types should be used in user space headers only Signed-off-by:
Tomas Winkler <tomas.winkler@intel.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
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 12 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:
-
Joe Perches authored
Signed-off-by:
-
Joe Perches authored
Signed-off-by:
-
Joe Perches authored
Signed-off-by:
-
Joe Perches authored
Remove obfuscating A_CHAR uses, just use char. Signed-off-by:
-
Joe Perches authored
Use kernel bool and true/false. Signed-off-by:
-
Greg Kroah-Hartman authored
This was done to resolve conflicts in the following files due to patches in Linus's tree and in the staging-next tree: drivers/staging/brcm80211/brcmsmac/wl_mac80211.c drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c Signed-off-by:
-
- 01 Feb, 2011 1 commit
-
-
Alan Cox authored
More debug printk pruning and turn down some that the user can cause intentionally to debug level. Tidy up the ioctl code a bit more Signed-off-by:
Alan Cox <alan@linux.intel.com> Signed-off-by:
-
