- 26 Feb, 2015 40 commits
-
-
Aya Mahfouz authored
This patch replaces init_timer and the 2 step initialization of function and data by setup_timer to make the code more concise. The issue was discovered using the following coccinelle script: @@ expression ds, e1, e2; @@ -init_timer (&ds); +setup_timer (&ds, e1, e2); ... ( -ds.function = e1; ... -ds.data = e2; | -ds.data = e2; ... -ds.function = e1; ) Signed-off-by:
Aya Mahfouz <mahfouz.saif.elyazal@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Aya Mahfouz authored
This patch replaces init_timer and the 2 step initialization of function and data by setup_timer to make the code more concise. The issue was discovered using the following coccinelle script: @@ expression ds, e1, e2; @@ -init_timer (&ds); +setup_timer (&ds, e1, e2); ... ( -ds.function = e1; ... -ds.data = e2; | -ds.data = e2; ... -ds.function = e1; ) Signed-off-by:
-
Aya Mahfouz authored
This patch replaces init_timer and the 2 step initialization of function and data by setup_timer to make the code more concise. The issue was discovered using the following coccinelle script: @@ expression ds, e1, e2; @@ -init_timer (&ds); +setup_timer (&ds, e1, e2); ... ( -ds.function = e1; ... -ds.data = e2; | -ds.data = e2; ... -ds.function = e1; ) Signed-off-by:
-
Aya Mahfouz authored
This patch replaces init_timer and the 2 step initialization of function and data by setup_timer to make the code more concise. The issue was discovered using the following coccinelle script: @@ expression ds, e1, e2; @@ -init_timer (&ds); +setup_timer (&ds, e1, e2); ... ( -ds.function = e1; ... -ds.data = e2; | -ds.data = e2; ... -ds.function = e1; ) Signed-off-by:
-
Yeliz Taneroglu authored
The following patch fixes the checkpatch.pl warning: Removed all trailing whitespace in fb_s6d1121.c Signed-off-by:
Yeliz Taneroglu <yeliztaneroglu@gmail.com> Signed-off-by:
-
Melike Yurtoglu authored
This patch fixes checkpatch.pl error ERROR: do not use C99 // comments Signed-off-by:
Melike Yurtoglu <aysemelikeyurtoglu@gmail.com> Signed-off-by:
-
Melike Yurtoglu authored
This patch fixes checkpatch.pl error trailing whitespace. Signed-off-by:
-
aybuke ozdemir authored
WARNING: Missing a blank line after declaration chackpatch.pl warning in fb_ssd1351.c Signed-off-by:
aybuke ozdemir <aybuke.147@gmail.com> Signed-off-by:
-
aybuke ozdemir authored
WARNING: Missing a blank line after declaration chackpatch.pl warning in fb_ili9320.c Signed-off-by:
-
Aya Mahfouz authored
This patch fixes the following checkpatch.pl errors: space prohibited before that ',' space required after that ',' spaces required around that '=' Given that the addition of spaces triggered the line over 80 characters warning, some lines were divided into two. Signed-off-by:
-
Aya Mahfouz authored
This patch fixes the following checkpatch.pl error: fix space prohibited before that ',' Signed-off-by:
-
Aya Mahfouz authored
This patch fixes the following checkpatch.pl error: space required after that ',' Signed-off-by:
-
Aya Mahfouz authored
This patch fixes the following checkpatch.pl warning: braces {} are not necessary for single statement blocks Signed-off-by: -
Aya Mahfouz authored
This patch fixes the following checkpatch.pl error: space prohibited before that ',' Signed-off-by:
-
Aya Mahfouz authored
This patch fixes the following checkpatch.pl warning: braces {} are not necessary for single statement blocks Signed-off-by: -
Aya Mahfouz authored
This patch fixes the following checkpatch.pl error: code indent should use tabs where possible Signed-off-by:
-
Melike Yurtoglu authored
WARNING: "Missing a blank line after declarations" Add blank line after declarations. That was found by running checkpatch Signed-off-by:
-
Melike Yurtoglu authored
WARNING: void function return statements are not generally useful Remove return in void function. That was found by running checkpatch. Signed-off-by:
Daniel Baluta <daniel.baluta@intel.com> Signed-off-by:
-
Melike Yurtoglu authored
WARNING: please, no spaces at the start of a line That was found by running checkpatch. Signed-off-by:
-
Melike Yurtoglu authored
WARNING: break is not useful after a goto or return. Remove unnecessary break after return. That was found by running checkpatch. Signed-off-by:
-
Melike Yurtoglu authored
WARNING: void function return statements are not generally useful Remove return in void function. That was found by running checkpatch. Signed-off-by:
-
Dilek Uzulmez authored
The following patch fixes the checkpatch.pl warning: drivers/staging/rtl8723au/core/rtw_ap.c WARNING: Missing a blank line after declarations Signed-off-by:
Dilek Uzulmez <dilekuzulmez@gmail.com> Signed-off-by:
-
Dilek Uzulmez authored
This patch fixes "quoted string split across lines warning" warning in rtw_ap.c Signed-off-by:
-
Vaishali Thakkar authored
Using byte ordering functions and then memcpy() is risky and prone to hide errors which are hard to track down. So, this patch introduces the use of function put_unaligned_le16 which makes the code clear. Here, use of variable tim_bitmap_le and variable itself is removed. Also, to be compatible with the changes header file is added too. Coccinelle is used to do this change and semantic patch used for this is as follows: @A@ typedef __le16; __le16 e16; identifier tmp; expression ptr; expression y,e; type T; @@ - tmp = cpu_to_le16(y); <+... when != tmp ( - memcpy(ptr, (T)&tmp, \(2\|sizeof(__le16)\|sizeof(e16)\)); + put_unaligned_le16(y,ptr); | - memcpy(ptr, (T)&tmp, ...); + put_unaligned_le16(y,ptr); ) ...+> ? tmp = e @@ type T; identifier a.tmp; @@ - T tmp; ...when != tmp Signed-off-by:
Vaishali Thakkar <vthakkar1994@gmail.com> Reviewed-by:
Arnd Bergmann <arnd@arndb.de> Signed-off-by:
-
Haneen Mohammed authored
This patch removes unneeded braces around a single if-statement. This problem was found using checkpatch.pl. Signed-off-by:
Haneen Mohammed <hamohammed.sa@gmail.com> Signed-off-by:
-
Vatika Harlalka authored
Replace if-else block with switch-case to make it more compact and increase readability. Signed-off-by:
Vatika Harlalka <vatikaharlalka@gmail.com> Acked-by:
-
Vatika Harlalka authored
Remove the nested conditionals. The if and else-if have the same code so they are combined to make the code more compact. Signed-off-by:
-
Vatika Harlalka authored
The first loop assigns values to ofdm_index array and the second checks for boundary conditions. They are combined and a comment is added to increase clarity. Signed-off-by:
-
Vatika Harlalka authored
Remove this branch as is2t is defined false and is not modified. The variable value32 is then unused. Signed-off-by:
-
Katie Dunne authored
Removes macro definitions that are unused elsewhere Signed-off-by:
Katie Dunne <kdunne@mail.ccsf.edu> Signed-off-by:
-
Vatika Harlalka authored
Remove unnecessary variable and replace its uses with previously defined variable. Signed-off-by:
-
Vatika Harlalka authored
Refactored the conditional code to make it more compact. Replaced the ternary operators with abs() macro to increase readability. Signed-off-by:
-
Vatika Harlalka authored
Replace ternary operator with existing abs() macro to increase code readability. Signed-off-by:
-
Vatika Harlalka authored
Remove redundant if condition as !result is always false. Signed-off-by:
-
Melike Yurtoglu authored
WARNING: "Missing a blank line after declarations" That was found by running checkpatch Signed-off-by:
-
Vatika Harlalka authored
This function only required the array from the 14th element onwards. Therefore, the array size is reduced and the loop counter is modified so as to start from 0. Also, the assignment of variable place is redundant as it is initialized again in the loop. Signed-off-by:
-
Vatika Harlalka authored
Replace switch-case block with single if statement to increase code readability. Signed-off-by:
-
Vatika Harlalka authored
Remove unneccessary variable and replace its uses with another variable which is previously defined. Signed-off-by:
-
Vatika Harlalka authored
Added a variable to reduce line size and enhance code readability. Signed-off-by:
-
Vaishali Thakkar authored
Using byte ordering functions and then memcpy() is risky and prone to hide errors which are hard to track down. So, this patch introduces the use of function put_unaligned_le16 which makes the code clear. Here, use of variable tim_bitmap_le and variable itself is removed. Also, to be compatible with the changes header file is added too. Coccinelle is used to do this change and semantic patch used for this is as follows: @A@ typedef __le16; __le16 e16; identifier tmp; expression ptr; expression y,e; type T; @@ - tmp = cpu_to_le16(y); <+... when != tmp ( - memcpy(ptr, (T)&tmp, \(2\|sizeof(__le16)\|sizeof(e16)\)); + put_unaligned_le16(y,ptr); | - memcpy(ptr, (T)&tmp, ...); + put_unaligned_le16(y,ptr); ) ...+> ? tmp = e @@ type T; identifier a.tmp; @@ - T tmp; ...when != tmp Signed-off-by:
Preeti U Murthy <preeti@linux.vnet.ibm.com> Signed-off-by:
-
