Catégorie : Linux

chelsio: constify cmac_ops structures

Author: Julia Lawall <julia.lawall@lip6.fr> The cmac_ops structures are never modified, so declare them as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: David S. Miller — drivers/net/ethernet/chelsio/cxgb/pm3393.c | 2 +- drivers/net/ethernet/chelsio/cxgb/vsc7326.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)   diff –git a/drivers/net/ethernet/chelsio/cxgb/pm3393.c b/drivers/net/ethernet/chelsio/cxgb/pm3393.c index ec5e050..eb462d7 100644 — a/drivers/net/ethernet/chelsio/cxgb/pm3393.c +++ …

Lire la suite

drivers: net: xgene: constify xgene_mac_ops and xgene_port_ops structures

Author: Julia Lawall <julia.lawall@lip6.fr> The xgene_mac_ops and xgene_port_ops structures are never modified, so declare them as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: David S. Miller — drivers/net/ethernet/apm/xgene/xgene_enet_hw.c | 4 ++– drivers/net/ethernet/apm/xgene/xgene_enet_hw.h | 4 ++– drivers/net/ethernet/apm/xgene/xgene_enet_main.c | 8 ++++—- drivers/net/ethernet/apm/xgene/xgene_enet_main.h | 4 ++– drivers/net/ethernet/apm/xgene/xgene_enet_sgmac.c | 4 ++– drivers/net/ethernet/apm/xgene/xgene_enet_sgmac.h | 4 ++– …

Lire la suite

Staging: rdma:Delete unnecessary NULL check before calling function « kmem_cache_destroy »

Author: Yash Shah <yshah1@visteon.com> The kmem_cache_destroy() function tests whether its argument is NULL and then returns immediately. Thus the NULL check before calling this function is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Yash Shah Signed-off-by: Greg Kroah-Hartman — drivers/staging/rdma/ehca/ehca_av.c | 3 +– drivers/staging/rdma/ehca/ehca_cq.c | 3 +– drivers/staging/rdma/ehca/ehca_main.c | 3 …

Lire la suite

crypto: drbg – constify drbg_state_ops structures

Author: Julia Lawall <Julia.Lawall@lip6.fr> The drbg_state_ops structures are never modified, so declare them as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: Herbert Xu — crypto/drbg.c | 6 +++— 1 file changed, 3 insertions(+), 3 deletions(-)   diff –git a/crypto/drbg.c b/crypto/drbg.c index a7c2314..ab6ef1d 100644 — a/crypto/drbg.c +++ b/crypto/drbg.c @@ -626,7 +626,7 @@ …

Lire la suite

prism54: fix compare_const_fl.cocci warnings

Author: Julia Lawall <Julia.Lawall@lip6.fr> Move constants to the right of binary operators. Generated by: scripts/coccinelle/misc/compare_const_fl.cocci Signed-off-by: Fengguang Wu Signed-off-by: Julia Lawall Signed-off-by: Kalle Valo — drivers/net/wireless/intersil/prism54/oid_mgt.c | 10 +++++—– 1 file changed, 5 insertions(+), 5 deletions(-)   diff –git a/drivers/net/wireless/intersil/prism54/oid_mgt.c b/drivers/net/wireless/intersil/prism54/oid_mgt.c index 3a8d2db..6528ed5 100644 — a/drivers/net/wireless/intersil/prism54/oid_mgt.c +++ b/drivers/net/wireless/intersil/prism54/oid_mgt.c @@ -424,7 +424,7 @@ mgt_set_request(islpci_private *priv, enum …

Lire la suite

mwifiex: fix semicolon.cocci warnings

Author: Julia Lawall <Julia.Lawall@lip6.fr> Remove unneeded semicolon. Generated by: scripts/coccinelle/misc/semicolon.cocci Signed-off-by: Fengguang Wu Signed-off-by: Julia Lawall Signed-off-by: Kalle Valo — drivers/net/wireless/marvell/mwifiex/init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)   diff –git a/drivers/net/wireless/marvell/mwifiex/init.c b/drivers/net/wireless/marvell/mwifiex/init.c index de74a77..6f7876e 100644 — a/drivers/net/wireless/marvell/mwifiex/init.c +++ b/drivers/net/wireless/marvell/mwifiex/init.c @@ -95,7 +95,7 @@ int mwifiex_init_priv(struct mwifiex_private *priv) priv->curr_pkt_filter = HostCmd_ACT_MAC_RX_ON | …

Lire la suite

PCI / PM: constify pci_platform_pm_ops structure

Author: Julia Lawall <Julia.Lawall@lip6.fr> The pci_platform_pm_ops structure is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Acked-by: Bjorn Helgaas Signed-off-by: Rafael J. Wysocki — drivers/pci/pci-acpi.c | 2 +- drivers/pci/pci.c | 4 ++– drivers/pci/pci.h | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-)   diff –git a/drivers/pci/pci-acpi.c …

Lire la suite

[media] drivers/media/usb/as102: constify as102_priv_ops_t structure

Author: Julia Lawall <Julia.Lawall@lip6.fr> The as102_priv_ops_t structure is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab — drivers/media/usb/as102/as102_drv.h | 2 +- drivers/media/usb/as102/as102_usb_drv.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)   diff –git a/drivers/media/usb/as102/as102_drv.h b/drivers/media/usb/as102/as102_drv.h index aee2d76..8def19d 100644 …

Lire la suite

[media] constify stv6110x_devctl structure

Author: Julia Lawall <Julia.Lawall@lip6.fr> The stv6110x_devctl structure is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab — drivers/media/dvb-frontends/stv6110x.c | 4 ++– drivers/media/dvb-frontends/stv6110x.h | 4 ++– drivers/media/dvb-frontends/stv6110x_priv.h | 2 +- drivers/media/pci/ddbridge/ddbridge-core.c | 2 +- drivers/media/pci/ngene/ngene-cards.c | 2 +- drivers/media/pci/ttpci/budget.c | 4 …

Lire la suite

brcm80211: fix compare_const_fl.cocci warnings

Author: Julia Lawall <Julia.Lawall@lip6.fr> Move constants to the right of binary operators. Generated by: scripts/coccinelle/misc/compare_const_fl.cocci Signed-off-by: Fengguang Wu Signed-off-by: Julia Lawall Signed-off-by: Kalle Valo — drivers/net/wireless/broadcom/brcm80211/brcmsmac/channel.c | 4 ++– 1 file changed, 2 insertions(+), 2 deletions(-)   diff –git a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/channel.c b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/channel.c index 635ae03..38bd589 100644 — a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/channel.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/channel.c @@ -177,8 +177,8 @@ static bool brcms_c_country_valid(const …

Lire la suite