* [PATCH] PCI: rockchip: Handle return value of clk_prepare_enable
@ 2017-06-01 10:21 Arvind Yadav
2017-06-02 7:28 ` Shawn Lin
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Arvind Yadav @ 2017-06-01 10:21 UTC (permalink / raw)
To: shawn.lin, wenrui.li, heiko
Cc: linux-pci, linux-rockchip, linux-arm-kernel, linux-kernel
clk_prepare_enable() can fail here and we must check its return value.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
drivers/pci/host/pcie-rockchip.c | 34 ++++++++++++++++++++++++++++------
1 file changed, 28 insertions(+), 6 deletions(-)
diff --git a/drivers/pci/host/pcie-rockchip.c b/drivers/pci/host/pcie-rockchip.c
index 0e020b6..0a248cd 100644
--- a/drivers/pci/host/pcie-rockchip.c
+++ b/drivers/pci/host/pcie-rockchip.c
@@ -1259,24 +1259,46 @@ static int __maybe_unused rockchip_pcie_resume_noirq(struct device *dev)
struct rockchip_pcie *rockchip = dev_get_drvdata(dev);
int err;
- clk_prepare_enable(rockchip->clk_pcie_pm);
- clk_prepare_enable(rockchip->hclk_pcie);
- clk_prepare_enable(rockchip->aclk_perf_pcie);
- clk_prepare_enable(rockchip->aclk_pcie);
+ err = clk_prepare_enable(rockchip->clk_pcie_pm);
+ if (err)
+ goto err_pcie_pm;
+
+ err = clk_prepare_enable(rockchip->hclk_pcie);
+ if (err)
+ goto err_hclk_pcie;
+
+ err = clk_prepare_enable(rockchip->aclk_perf_pcie);
+ if (err)
+ goto err_aclk_perf_pcie;
+
+ err = clk_prepare_enable(rockchip->aclk_pcie);
+ if (err)
+ goto err_aclk_pcie;
err = rockchip_pcie_init_port(rockchip);
if (err)
- return err;
+ goto err_pcie_resume;
err = rockchip_cfg_atu(rockchip);
if (err)
- return err;
+ goto err_pcie_resume;
/* Need this to enter L1 again */
rockchip_pcie_update_txcredit_mui(rockchip);
rockchip_pcie_enable_interrupts(rockchip);
return 0;
+
+err_pcie_resume:
+ clk_disable_unprepare(rockchip->aclk_pcie);
+err_aclk_pcie:
+ clk_disable_unprepare(rockchip->aclk_perf_pcie);
+err_aclk_perf_pcie:
+ clk_disable_unprepare(rockchip->hclk_pcie);
+err_hclk_pcie:
+ clk_disable_unprepare(rockchip->clk_pcie_pm);
+err_pcie_pm:
+ return err;
}
static int rockchip_pcie_probe(struct platform_device *pdev)
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] PCI: rockchip: Handle return value of clk_prepare_enable
2017-06-01 10:21 [PATCH] PCI: rockchip: Handle return value of clk_prepare_enable Arvind Yadav
@ 2017-06-02 7:28 ` Shawn Lin
2017-06-02 8:34 ` Heiko Stuebner
2017-06-15 21:48 ` Bjorn Helgaas
2 siblings, 0 replies; 4+ messages in thread
From: Shawn Lin @ 2017-06-02 7:28 UTC (permalink / raw)
To: Arvind Yadav
Cc: heiko, linux-pci, shawn.lin, linux-kernel, linux-rockchip,
linux-arm-kernel
SGksCgrU2iAyMDE3LzYvMSAxODoyMSwgQXJ2aW5kIFlhZGF2INC0tcA6Cj4gY2xrX3ByZXBhcmVf
ZW5hYmxlKCkgY2FuIGZhaWwgaGVyZSBhbmQgd2UgbXVzdCBjaGVjayBpdHMgcmV0dXJuIHZhbHVl
Lgo+IAo+IFNpZ25lZC1vZmYtYnk6IEFydmluZCBZYWRhdiA8YXJ2aW5kLnlhZGF2LmNzQGdtYWls
LmNvbT4KCkFja2VkLWJ5OiBTaGF3biBMaW4gPHNoYXduLmxpbkByb2NrLWNoaXBzLmNvbT4KCj4g
LS0tCj4gICBkcml2ZXJzL3BjaS9ob3N0L3BjaWUtcm9ja2NoaXAuYyB8IDM0ICsrKysrKysrKysr
KysrKysrKysrKysrKysrKystLS0tLS0KPiAgIDEgZmlsZSBjaGFuZ2VkLCAyOCBpbnNlcnRpb25z
KCspLCA2IGRlbGV0aW9ucygtKQo+IAo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL3BjaS9ob3N0L3Bj
aWUtcm9ja2NoaXAuYyBiL2RyaXZlcnMvcGNpL2hvc3QvcGNpZS1yb2NrY2hpcC5jCj4gaW5kZXgg
MGUwMjBiNi4uMGEyNDhjZCAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL3BjaS9ob3N0L3BjaWUtcm9j
a2NoaXAuYwo+ICsrKyBiL2RyaXZlcnMvcGNpL2hvc3QvcGNpZS1yb2NrY2hpcC5jCj4gQEAgLTEy
NTksMjQgKzEyNTksNDYgQEAgc3RhdGljIGludCBfX21heWJlX3VudXNlZCByb2NrY2hpcF9wY2ll
X3Jlc3VtZV9ub2lycShzdHJ1Y3QgZGV2aWNlICpkZXYpCj4gICAJc3RydWN0IHJvY2tjaGlwX3Bj
aWUgKnJvY2tjaGlwID0gZGV2X2dldF9kcnZkYXRhKGRldik7Cj4gICAJaW50IGVycjsKPiAgIAo+
IC0JY2xrX3ByZXBhcmVfZW5hYmxlKHJvY2tjaGlwLT5jbGtfcGNpZV9wbSk7Cj4gLQljbGtfcHJl
cGFyZV9lbmFibGUocm9ja2NoaXAtPmhjbGtfcGNpZSk7Cj4gLQljbGtfcHJlcGFyZV9lbmFibGUo
cm9ja2NoaXAtPmFjbGtfcGVyZl9wY2llKTsKPiAtCWNsa19wcmVwYXJlX2VuYWJsZShyb2NrY2hp
cC0+YWNsa19wY2llKTsKPiArCWVyciA9IGNsa19wcmVwYXJlX2VuYWJsZShyb2NrY2hpcC0+Y2xr
X3BjaWVfcG0pOwo+ICsJaWYgKGVycikKPiArCQlnb3RvIGVycl9wY2llX3BtOwo+ICsKPiArCWVy
ciA9IGNsa19wcmVwYXJlX2VuYWJsZShyb2NrY2hpcC0+aGNsa19wY2llKTsKPiArCWlmIChlcnIp
Cj4gKwkJZ290byBlcnJfaGNsa19wY2llOwo+ICsKPiArCWVyciA9IGNsa19wcmVwYXJlX2VuYWJs
ZShyb2NrY2hpcC0+YWNsa19wZXJmX3BjaWUpOwo+ICsJaWYgKGVycikKPiArCQlnb3RvIGVycl9h
Y2xrX3BlcmZfcGNpZTsKPiArCj4gKwllcnIgPSBjbGtfcHJlcGFyZV9lbmFibGUocm9ja2NoaXAt
PmFjbGtfcGNpZSk7Cj4gKwlpZiAoZXJyKQo+ICsJCWdvdG8gZXJyX2FjbGtfcGNpZTsKPiAgIAo+
ICAgCWVyciA9IHJvY2tjaGlwX3BjaWVfaW5pdF9wb3J0KHJvY2tjaGlwKTsKPiAgIAlpZiAoZXJy
KQo+IC0JCXJldHVybiBlcnI7Cj4gKwkJZ290byBlcnJfcGNpZV9yZXN1bWU7Cj4gICAKPiAgIAll
cnIgPSByb2NrY2hpcF9jZmdfYXR1KHJvY2tjaGlwKTsKPiAgIAlpZiAoZXJyKQo+IC0JCXJldHVy
biBlcnI7Cj4gKwkJZ290byBlcnJfcGNpZV9yZXN1bWU7Cj4gICAKPiAgIAkvKiBOZWVkIHRoaXMg
dG8gZW50ZXIgTDEgYWdhaW4gKi8KPiAgIAlyb2NrY2hpcF9wY2llX3VwZGF0ZV90eGNyZWRpdF9t
dWkocm9ja2NoaXApOwo+ICAgCXJvY2tjaGlwX3BjaWVfZW5hYmxlX2ludGVycnVwdHMocm9ja2No
aXApOwo+ICAgCj4gICAJcmV0dXJuIDA7Cj4gKwo+ICtlcnJfcGNpZV9yZXN1bWU6Cj4gKwljbGtf
ZGlzYWJsZV91bnByZXBhcmUocm9ja2NoaXAtPmFjbGtfcGNpZSk7Cj4gK2Vycl9hY2xrX3BjaWU6
Cj4gKwljbGtfZGlzYWJsZV91bnByZXBhcmUocm9ja2NoaXAtPmFjbGtfcGVyZl9wY2llKTsKPiAr
ZXJyX2FjbGtfcGVyZl9wY2llOgo+ICsJY2xrX2Rpc2FibGVfdW5wcmVwYXJlKHJvY2tjaGlwLT5o
Y2xrX3BjaWUpOwo+ICtlcnJfaGNsa19wY2llOgo+ICsJY2xrX2Rpc2FibGVfdW5wcmVwYXJlKHJv
Y2tjaGlwLT5jbGtfcGNpZV9wbSk7Cj4gK2Vycl9wY2llX3BtOgo+ICsJcmV0dXJuIGVycjsKPiAg
IH0KPiAgIAo+ICAgc3RhdGljIGludCByb2NrY2hpcF9wY2llX3Byb2JlKHN0cnVjdCBwbGF0Zm9y
bV9kZXZpY2UgKnBkZXYpCj4gCgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX18KbGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5l
bEBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4v
bGlzdGluZm8vbGludXgtYXJtLWtlcm5lbAo=
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] PCI: rockchip: Handle return value of clk_prepare_enable
2017-06-01 10:21 [PATCH] PCI: rockchip: Handle return value of clk_prepare_enable Arvind Yadav
2017-06-02 7:28 ` Shawn Lin
@ 2017-06-02 8:34 ` Heiko Stuebner
2017-06-15 21:48 ` Bjorn Helgaas
2 siblings, 0 replies; 4+ messages in thread
From: Heiko Stuebner @ 2017-06-02 8:34 UTC (permalink / raw)
To: Arvind Yadav
Cc: linux-pci, shawn.lin, wenrui.li, linux-kernel, linux-rockchip,
linux-arm-kernel
Am Donnerstag, 1. Juni 2017, 15:51:45 CEST schrieb Arvind Yadav:
> clk_prepare_enable() can fail here and we must check its return value.
>
> Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
looks good, but you may want to include
Bjorn Helgaas <bhelgaas@google.com>
who is the maintainer of the pci subsystem and would be the one
to apply your patch.
Aynway, the patch itself looks good to me
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
Cheers
Heiko
> ---
> drivers/pci/host/pcie-rockchip.c | 34 ++++++++++++++++++++++++++++------
> 1 file changed, 28 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/pci/host/pcie-rockchip.c b/drivers/pci/host/pcie-rockchip.c
> index 0e020b6..0a248cd 100644
> --- a/drivers/pci/host/pcie-rockchip.c
> +++ b/drivers/pci/host/pcie-rockchip.c
> @@ -1259,24 +1259,46 @@ static int __maybe_unused rockchip_pcie_resume_noirq(struct device *dev)
> struct rockchip_pcie *rockchip = dev_get_drvdata(dev);
> int err;
>
> - clk_prepare_enable(rockchip->clk_pcie_pm);
> - clk_prepare_enable(rockchip->hclk_pcie);
> - clk_prepare_enable(rockchip->aclk_perf_pcie);
> - clk_prepare_enable(rockchip->aclk_pcie);
> + err = clk_prepare_enable(rockchip->clk_pcie_pm);
> + if (err)
> + goto err_pcie_pm;
> +
> + err = clk_prepare_enable(rockchip->hclk_pcie);
> + if (err)
> + goto err_hclk_pcie;
> +
> + err = clk_prepare_enable(rockchip->aclk_perf_pcie);
> + if (err)
> + goto err_aclk_perf_pcie;
> +
> + err = clk_prepare_enable(rockchip->aclk_pcie);
> + if (err)
> + goto err_aclk_pcie;
>
> err = rockchip_pcie_init_port(rockchip);
> if (err)
> - return err;
> + goto err_pcie_resume;
>
> err = rockchip_cfg_atu(rockchip);
> if (err)
> - return err;
> + goto err_pcie_resume;
>
> /* Need this to enter L1 again */
> rockchip_pcie_update_txcredit_mui(rockchip);
> rockchip_pcie_enable_interrupts(rockchip);
>
> return 0;
> +
> +err_pcie_resume:
> + clk_disable_unprepare(rockchip->aclk_pcie);
> +err_aclk_pcie:
> + clk_disable_unprepare(rockchip->aclk_perf_pcie);
> +err_aclk_perf_pcie:
> + clk_disable_unprepare(rockchip->hclk_pcie);
> +err_hclk_pcie:
> + clk_disable_unprepare(rockchip->clk_pcie_pm);
> +err_pcie_pm:
> + return err;
> }
>
> static int rockchip_pcie_probe(struct platform_device *pdev)
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] PCI: rockchip: Handle return value of clk_prepare_enable
2017-06-01 10:21 [PATCH] PCI: rockchip: Handle return value of clk_prepare_enable Arvind Yadav
2017-06-02 7:28 ` Shawn Lin
2017-06-02 8:34 ` Heiko Stuebner
@ 2017-06-15 21:48 ` Bjorn Helgaas
2 siblings, 0 replies; 4+ messages in thread
From: Bjorn Helgaas @ 2017-06-15 21:48 UTC (permalink / raw)
To: Arvind Yadav
Cc: shawn.lin, wenrui.li, heiko, linux-pci, linux-rockchip,
linux-arm-kernel, linux-kernel
On Thu, Jun 01, 2017 at 03:51:45PM +0530, Arvind Yadav wrote:
> clk_prepare_enable() can fail here and we must check its return value.
>
> Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Applied with ack/reviewed-by from Shawn and Heiko to pci/host-rockchip for
v4.13, thanks!
I had to apply it manually because of other changes in the same area, so it
would be good if you could check it over.
https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git/commit/?h=pci/host-rockchip&id=45f8ce937bc2e06e040f1118f64783b54ade7363
> ---
> drivers/pci/host/pcie-rockchip.c | 34 ++++++++++++++++++++++++++++------
> 1 file changed, 28 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/pci/host/pcie-rockchip.c b/drivers/pci/host/pcie-rockchip.c
> index 0e020b6..0a248cd 100644
> --- a/drivers/pci/host/pcie-rockchip.c
> +++ b/drivers/pci/host/pcie-rockchip.c
> @@ -1259,24 +1259,46 @@ static int __maybe_unused rockchip_pcie_resume_noirq(struct device *dev)
> struct rockchip_pcie *rockchip = dev_get_drvdata(dev);
> int err;
>
> - clk_prepare_enable(rockchip->clk_pcie_pm);
> - clk_prepare_enable(rockchip->hclk_pcie);
> - clk_prepare_enable(rockchip->aclk_perf_pcie);
> - clk_prepare_enable(rockchip->aclk_pcie);
> + err = clk_prepare_enable(rockchip->clk_pcie_pm);
> + if (err)
> + goto err_pcie_pm;
> +
> + err = clk_prepare_enable(rockchip->hclk_pcie);
> + if (err)
> + goto err_hclk_pcie;
> +
> + err = clk_prepare_enable(rockchip->aclk_perf_pcie);
> + if (err)
> + goto err_aclk_perf_pcie;
> +
> + err = clk_prepare_enable(rockchip->aclk_pcie);
> + if (err)
> + goto err_aclk_pcie;
>
> err = rockchip_pcie_init_port(rockchip);
> if (err)
> - return err;
> + goto err_pcie_resume;
>
> err = rockchip_cfg_atu(rockchip);
> if (err)
> - return err;
> + goto err_pcie_resume;
>
> /* Need this to enter L1 again */
> rockchip_pcie_update_txcredit_mui(rockchip);
> rockchip_pcie_enable_interrupts(rockchip);
>
> return 0;
> +
> +err_pcie_resume:
> + clk_disable_unprepare(rockchip->aclk_pcie);
> +err_aclk_pcie:
> + clk_disable_unprepare(rockchip->aclk_perf_pcie);
> +err_aclk_perf_pcie:
> + clk_disable_unprepare(rockchip->hclk_pcie);
> +err_hclk_pcie:
> + clk_disable_unprepare(rockchip->clk_pcie_pm);
> +err_pcie_pm:
> + return err;
> }
>
> static int rockchip_pcie_probe(struct platform_device *pdev)
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-06-15 21:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-01 10:21 [PATCH] PCI: rockchip: Handle return value of clk_prepare_enable Arvind Yadav
2017-06-02 7:28 ` Shawn Lin
2017-06-02 8:34 ` Heiko Stuebner
2017-06-15 21:48 ` Bjorn Helgaas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).