Commit c1fc0050102150481ad4ba7e6449b3aa6e88d6e5
Committed by
Kishon Vijay Abraham I
1 parent
556186a02c
Exists in
ti-lsk-linux-4.1.y
and in
10 other branches
phy: Use PTR_ERR_OR_ZERO to fix warning raised by coccinelle
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR Generated by: coccinelle/api/ptr_ret.cocci Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com> Acked-by: Jason Cooper <jason@lakedaemon.net> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
Showing 4 changed files with 4 additions and 16 deletions Side-by-side Diff
drivers/phy/phy-berlin-sata.c
... | ... | @@ -268,10 +268,7 @@ |
268 | 268 | |
269 | 269 | phy_provider = |
270 | 270 | devm_of_phy_provider_register(dev, phy_berlin_sata_phy_xlate); |
271 | - if (IS_ERR(phy_provider)) | |
272 | - return PTR_ERR(phy_provider); | |
273 | - | |
274 | - return 0; | |
271 | + return PTR_ERR_OR_ZERO(phy_provider); | |
275 | 272 | } |
276 | 273 | |
277 | 274 | static const struct of_device_id phy_berlin_sata_of_match[] = { |
drivers/phy/phy-hix5hd2-sata.c
... | ... | @@ -164,10 +164,7 @@ |
164 | 164 | |
165 | 165 | phy_set_drvdata(phy, priv); |
166 | 166 | phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); |
167 | - if (IS_ERR(phy_provider)) | |
168 | - return PTR_ERR(phy_provider); | |
169 | - | |
170 | - return 0; | |
167 | + return PTR_ERR_OR_ZERO(phy_provider); | |
171 | 168 | } |
172 | 169 | |
173 | 170 | static const struct of_device_id hix5hd2_sata_phy_of_match[] = { |
drivers/phy/phy-miphy365x.c
... | ... | @@ -610,10 +610,7 @@ |
610 | 610 | } |
611 | 611 | |
612 | 612 | provider = devm_of_phy_provider_register(&pdev->dev, miphy365x_xlate); |
613 | - if (IS_ERR(provider)) | |
614 | - return PTR_ERR(provider); | |
615 | - | |
616 | - return 0; | |
613 | + return PTR_ERR_OR_ZERO(provider); | |
617 | 614 | } |
618 | 615 | |
619 | 616 | static const struct of_device_id miphy365x_of_match[] = { |
drivers/phy/phy-stih41x-usb.c
... | ... | @@ -160,10 +160,7 @@ |
160 | 160 | phy_set_drvdata(phy, phy_dev); |
161 | 161 | |
162 | 162 | phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); |
163 | - if (IS_ERR(phy_provider)) | |
164 | - return PTR_ERR(phy_provider); | |
165 | - | |
166 | - return 0; | |
163 | + return PTR_ERR_OR_ZERO(phy_provider); | |
167 | 164 | } |
168 | 165 | |
169 | 166 | static const struct of_device_id stih41x_usb_phy_of_match[] = { |