Catégorie : Linux

drm/vc4: fix platform_no_drv_owner.cocci warnings

Author: Julia Lawall <julia.lawall@lip6.fr> drivers/gpu/drm/vc4/vc4_drv.c:248:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Signed-off-by: Fengguang Wu Signed-off-by: Julia Lawall Signed-off-by: Eric Anholt — drivers/gpu/drm/vc4/vc4_drv.c | 1 – 1 file changed, 1 deletion(-)   diff –git a/drivers/gpu/drm/vc4/vc4_drv.c b/drivers/gpu/drm/vc4/vc4_drv.c index …

Lire la suite

drm/vc4: fix itnull.cocci warnings

Author: Julia Lawall <julia.lawall@lip6.fr> Connector cannot be null because it is a list entry, ie accessed at an offset from the positions of the list structure pointers themselves. Generated by: scripts/coccinelle/iterators/itnull.cocci Signed-off-by: Fengguang Wu Signed-off-by: Julia Lawall Signed-off-by: Eric Anholt — drivers/gpu/drm/vc4/vc4_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)   diff –git …

Lire la suite

of/irq: add missing of_node_put

Author: Julia Lawall <Julia.Lawall@lip6.fr> for_each_matching_node performs an of_node_get on each iteration, so a break out of the loop requires an of_node_put. A simplified version of the semantic patch that fixes this problem is as follows (http://coccinelle.lip6.fr): // @@ local idexpression n; expression e; identifier l; @@ for_each_matching_node(n,…) { … ( of_node_put(n); | e = n …

Lire la suite

of/unittest: add missing of_node_put

Author: Julia Lawall <Julia.Lawall@lip6.fr> for_each_child_of_node performs an of_node_get on each iteration, so a break out of the loop requires an of_node_put. A simplified version of the semantic patch that fixes this problem is as follows (http://coccinelle.lip6.fr): // @@ expression root,e; local idexpression child; @@ for_each_child_of_node(root, child) { … when != of_node_put(child) when != e = …

Lire la suite

of/platform: add missing of_node_put

Author: Julia Lawall <Julia.Lawall@lip6.fr> for_each_child_of_node performs an of_node_get on each iteration, so a break out of the loop requires an of_node_put. A simplified version of the semantic patch that fixes this problem is as follows (http://coccinelle.lip6.fr): // @@ local idexpression n; expression root,e; @@ for_each_child_of_node(root,n) { … ( of_node_put(n); | e = n | + …

Lire la suite

of/overlay: add missing of_node_put

Author: Julia Lawall <Julia.Lawall@lip6.fr> for_each_child_of_node performs an of_node_get on each iteration, so a break out of the loop requires an of_node_put. A simplified version of the semantic patch that fixes this problem is as follows (http://coccinelle.lip6.fr): // @@ expression root,e; local idexpression child; @@ for_each_child_of_node(root, child) { … when != of_node_put(child) when != e = …

Lire la suite

clk: scpi: add missing of_node_put

Author: Julia Lawall <Julia.Lawall@lip6.fr> for_each_available_child_of_node performs an of_node_get on each iteration, so a break out of the loop requires an of_node_put. The semantic patch that fixes this problem is as follows (http://coccinelle.lip6.fr): // @@ expression root,e; local idexpression child; @@ for_each_available_child_of_node(root, child) { … when != of_node_put(child) when != e = child ( return child; …

Lire la suite

clk: imx31: add missing of_node_put

Author: Julia Lawall <Julia.Lawall@lip6.fr> for_each_compatible_node performs an of_node_get on each iteration, so a break out of the loop requires an of_node_put. The semantic patch that fixes this problem is as follows (http://coccinelle.lip6.fr): // @@ local idexpression n; expression e; @@ for_each_compatible_node(n,…) { … ( of_node_put(n); | e = n | + of_node_put(n); ? break; ) …

Lire la suite

clk: imx27: add missing of_node_put

Author: Julia Lawall <Julia.Lawall@lip6.fr> for_each_compatible_node performs an of_node_get on each iteration, so a break out of the loop requires an of_node_put. The semantic patch that fixes this problem is as follows (http://coccinelle.lip6.fr): // @@ local idexpression n; expression e; @@ for_each_compatible_node(n,…) { … ( of_node_put(n); | e = n | + of_node_put(n); ? break; ) …

Lire la suite

clk: si5351: add missing of_node_put

Author: Julia Lawall <Julia.Lawall@lip6.fr> for_each_child_of_node performs an of_node_get on each iteration, so a break out of the loop requires an of_node_put. A simplified version of the semantic patch that fixes this problem is as follows (http://coccinelle.lip6.fr): // @@ expression root,e; local idexpression child; @@ for_each_child_of_node(root, child) { … when != of_node_put(child) when != e = …

Lire la suite