Catégorie : Linux

staging: lustre: obdclass: Use list_for_each_entry_safe

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 …

Lire la suite

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; …

Lire la suite

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 …

Lire la suite

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 …

Lire la suite

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 …

Lire la suite

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 …

Lire la suite

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 …

Lire la suite

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 …

Lire la suite

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 …

Lire la suite

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 …

Lire la suite