Format: Quote

usb: gadget: lpc32xx_udc: fix wrong clk_put() sequence

Author: Felipe Balbi <balbi@ti.com> This patch fixes the following Coccinelle error: drivers/usb/gadget/lpc32xx_udc.c:3313:1-7: ERROR: \ missing clk_put; clk_get on line 3139 and \ execution via conditional on line 3146 Signed-off-by: Felipe Balbi Signed-off-by: Greg Kroah-Hartman — drivers/usb/gadget/lpc32xx_udc.c | 4 ++– 1 file changed, 2 insertions(+), 2 deletions(-)   diff –git a/drivers/usb/gadget/lpc32xx_udc.c b/drivers/usb/gadget/lpc32xx_udc.c index 049ebab..a139894 100644 — …

Continue reading

staging:lustre: Removed assignments from if statements.

Author: Chi Pham <fempsci@gmail.com> Fixed some minor checkpatch warnings such as whitespace. Coccinelle was used for this patch (NOTE: some of the changes were made by hand). The script is not complete (semantically) and might raise some checkpatch warnings in terms of indentation depending on existing code. *** IFASSIGNMENT.COCCI START *** /* Coccinelle script to …

Continue reading

staging:dgnc: Removed assignments from if statements.

Author: Chi Pham <fempsci@gmail.com> Coccinelle was used for this patch. The script is not complete (semantically) and might raise some checkpatch warnings in terms of indentation depending on existing code. *** IFASSIGNMENT.COCCI START *** /* Coccinelle script to handle assignments in if statements * For compound statements, can so far only handle statements with the …

Continue reading

cfg80211: move regulatory flags to their own variable

Author: Luis R. Rodriguez <mcgrof@do-not-panic.com> We’ll expand this later, this will make it easier to classify and review what things are related to regulatory or not. Coccinelle only missed 4 hits, which I had to do manually, supplying the SmPL in case of merge conflicts. @@ struct wiphy *wiphy; @@ -wiphy->flags |= WIPHY_FLAG_CUSTOM_REGULATORY +wiphy->regulatory_flags |= …

Continue reading

rtlwifi: fix usage of freq_reg_info()

Author: Luis R. Rodriguez <mcgrof@do-not-panic.com> freq_reg_info() expects KHz and not MHz, fix this. In this case we’ll now be getting the no-ir flags cleared on channels for any channel when the country IE trusts that channel. @@ struct ieee80211_channel *ch; struct wiphy *wiphy; const struct ieee80211_reg_rule *rule; @@ -rule = freq_reg_info(wiphy, ch->center_freq); +rule = freq_reg_info(wiphy, …

Continue reading

brcm80211: fix usage of freq_reg_info()

Author: Luis R. Rodriguez <mcgrof@do-not-panic.com> freq_reg_info() expects KHz and not MHz, fix this. In this case we’ll now be getting the no-ir flags cleared on channels for any channel when the country IE trusts that channel. @@ struct ieee80211_channel *ch; struct wiphy *wiphy; const struct ieee80211_reg_rule *rule; @@ -rule = freq_reg_info(wiphy, ch->center_freq); +rule = freq_reg_info(wiphy, …

Continue reading

ath: fix usage of freq_reg_info()

Author: Luis R. Rodriguez <mcgrof@do-not-panic.com> freq_reg_info() expects KHz and not MHz, fix this. In this case we’ll now be getting the no-ir flags cleared on channels for any channel when the country IE trusts that channel. @@ struct ieee80211_channel *ch; struct wiphy *wiphy; const struct ieee80211_reg_rule *rule; @@ -rule = freq_reg_info(wiphy, ch->center_freq); +rule = freq_reg_info(wiphy, …

Continue reading

rtc: simplify use of devm_ioremap_resource

Author: Julia Lawall <Julia.Lawall@lip6.fr> Remove unneeded error handling on the result of a call to platform_get_resource when the value is passed to devm_ioremap_resource. Move the call to platform_get_resource adjacent to the call to devm_ioremap_resource to make the connection between them more clear. A simplified version of the semantic patch that makes this change is as …

Continue reading

staging: lustre: fix return type of lo_release.

Author: Cyril Roelandt <tipecaml@gmail.com> The return type of block_device_operations.release() changed to void in commit db2a144b. Found with the following Coccinelle patch: @has_release_func@ identifier i; identifier release_func; @@ struct block_device_operations i = { .release = release_func }; @depends on has_release_func@ identifier has_release_func.release_func; @@ – int + void release_func(…) { … – return …; } Signed-off-by: Cyril …

Continue reading

[SCSI] pm80xx: remove unneeded NULL check

Author: Dan Carpenter <dan.carpenter@oracle.com> Coccinelle complains about the inconsistent NULL checking on “t”. It turns out the check isn’t needed because we verified that “t” is non-NULL at the start of the function. Signed-off-by: Dan Carpenter Acked-by: Anand Kumar Santhanam Signed-off-by: James Bottomley — drivers/scsi/pm8001/pm8001_hwi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) …

Continue reading