Author: Bhaktipriya Shridhar <bhaktipriya96@gmail.com> Doubly linked lists which are iterated using list_empty and list_entry macros have been replaced with list_for_each_entry_safe macro. This makes the iteration simpler and more readable. This patch replaces the while loop containing list_empty and list_entry with list_for_each_entry_safe. This was done with Coccinelle. @@ expression E1; identifier I1, I2; type T; iterator …
Category: Linux
Mar 11 2016
Staging: android: ashmem.c: Redefine macros as static inline functions
Author: Bhumika Goyal <bhumirks@gmail.com> Convert macros page_in_range and range_before_page into static inline functions as static inline functions are preferred over macros. The change can be done as the arguments at all call sites have the same type. Also, both the macros have same type of arguments and return values. Done using coccinelle: @r@ identifier f; …
Mar 10 2016
staging: rtl8723au: hal: Remove unnecessary function and its call
Author: Bhaktipriya Shridhar <bhaktipriya96@gmail.com> The function odm_DynamicBBPowerSaving23a on being called, simply returns back. The function hasn’t been mentioned in the TODO and doesn’t have FIXME code around. Hence, odm_DynamicBBPowerSaving23a and its calls have been removed. This was done using Coccinelle. @@ identifier f; @@ void f(…) { -return; } Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman …
Mar 10 2016
staging: unisys: visornic: Remove unnecessary else after return
Author: Bhaktipriya Shridhar <bhaktipriya96@gmail.com> This patch fixes the checkpatch warning that else is not generally useful after a break or return. This was done using Coccinelle: @@ expression e2; statement s1; @@ if(e2) { … return …; } -else s Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman — drivers/staging/unisys/visornic/visornic_main.c | 5 ++— 1 file changed, 2 …
Mar 10 2016
staging: media: davinci_vpfe: Remove unnecessary else after return
Author: Bhaktipriya Shridhar <bhaktipriya96@gmail.com> This patch fixes the checkpatch warning that else is not generally useful after a break or return. This was done using Coccinelle: @@ expression e2; statement s1; @@ if(e2) { … return …; } -else s1 Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman — drivers/staging/media/davinci_vpfe/dm365_ipipeif.c | 2 +- 1 file changed, 1 …
Mar 10 2016
staging: media: davinci_vpfe: dm365_ipipe_hw: Remove unnecessary else after return
Author: Bhaktipriya Shridhar <bhaktipriya96@gmail.com> This patch fixes the checkpatch warning that else is not generally useful after a break or return. This was done using Coccinelle: @@ expression e2; statement s1; @@ if(e2) { … return …; } -else s1 Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman — drivers/staging/media/davinci_vpfe/dm365_ipipe_hw.c | 2 +- 1 file changed, 1 …
Mar 10 2016
staging: rtl8192u: ieee80211_crypt: Remove unnecessary else after return
Author: Bhaktipriya Shridhar <bhaktipriya96@gmail.com> This patch fixes the checkpatch warning that else is not generally useful after a break or return. This was done using Coccinelle: @@ expression e2; statement s1; @@ if(e2) { … return …; } -else s1 Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman — drivers/staging/rtl8192u/ieee80211/ieee80211_crypt.c | 3 +– 1 file changed, 1 …
Mar 10 2016
staging: rtl8192e: rtllib_crypt_tkip: Remove unnecessary else after return
Author: Bhaktipriya Shridhar <bhaktipriya96@gmail.com> This patch fixes the checkpatch warning that else is not generally useful after a break or return. This was done using Coccinelle: @@ expression e2; statement s1; @@ if(e2) { … return …; } -else s1 Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman — drivers/staging/rtl8192e/rtllib_crypt_tkip.c | 3 +– 1 file changed, 1 …
Mar 10 2016
staging: iio: addac: Remove unnecessary else after return
Author: Bhaktipriya Shridhar <bhaktipriya96@gmail.com> This patch fixes the checkpatch warning that else is not generally useful after a break or return. This was done using Coccinelle: @@ expression e2; statement s1; @@ if(e2) { … return …; } -else s1 Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman — drivers/staging/iio/addac/adt7316.c | 15 ++++++——— 1 file changed, 6 …
Mar 10 2016
staging: rtl8723au: core: Remove unnecessary else after return
Author: Bhaktipriya Shridhar <bhaktipriya96@gmail.com> This patch fixes the checkpatch warning that else is not generally useful after a break or return. This was done using Coccinelle: @@ expression e2; statement s1; @@ if(e2) { … return …; } -else s1 Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman — drivers/staging/rtl8723au/core/rtw_efuse.c | 4 ++– 1 file changed, 2 …