* [PATCH RESEND 0/4] phy: Use dev_err_probe() to simplify code
@ 2022-09-24 7:02 ` Yuan Can
0 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:02 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
This series simplify the error handling in probe function by
switching from dev_err() to dev_err_probe().
Yuan Can (4):
phy: qcom-qmp-combo: Use dev_err_probe() to simplify code
phy: qcom-qmp-pcie-msm8996: Use dev_err_probe() to simplify code
phy: qcom-qmp-ufs: Use dev_err_probe() to simplify code
phy: qcom-qmp-usb: Use dev_err_probe() to simplify code
drivers/phy/qualcomm/phy-qcom-qmp-combo.c | 21 +++++++------------
.../phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c | 9 +++-----
drivers/phy/qualcomm/phy-qcom-qmp-ufs.c | 9 +++-----
drivers/phy/qualcomm/phy-qcom-qmp-usb.c | 9 +++-----
4 files changed, 16 insertions(+), 32 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH RESEND 0/4] phy: Use dev_err_probe() to simplify code
@ 2022-09-24 7:02 ` Yuan Can
0 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:02 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
This series simplify the error handling in probe function by
switching from dev_err() to dev_err_probe().
Yuan Can (4):
phy: qcom-qmp-combo: Use dev_err_probe() to simplify code
phy: qcom-qmp-pcie-msm8996: Use dev_err_probe() to simplify code
phy: qcom-qmp-ufs: Use dev_err_probe() to simplify code
phy: qcom-qmp-usb: Use dev_err_probe() to simplify code
drivers/phy/qualcomm/phy-qcom-qmp-combo.c | 21 +++++++------------
.../phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c | 9 +++-----
drivers/phy/qualcomm/phy-qcom-qmp-ufs.c | 9 +++-----
drivers/phy/qualcomm/phy-qcom-qmp-usb.c | 9 +++-----
4 files changed, 16 insertions(+), 32 deletions(-)
--
2.17.1
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH RESEND 1/4] phy: qcom-qmp-combo: Use dev_err_probe() to simplify code
2022-09-24 7:02 ` Yuan Can
@ 2022-09-24 7:02 ` Yuan Can
-1 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:02 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs.
Signed-off-by: Yuan Can <yuancan@huawei.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Andrew Halaney <ahalaney@redhat.com>
---
drivers/phy/qualcomm/phy-qcom-qmp-combo.c | 21 +++++++--------------
1 file changed, 7 insertions(+), 14 deletions(-)
diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-combo.c b/drivers/phy/qualcomm/phy-qcom-qmp-combo.c
index 41f938126ff1..da85fbf2cb0c 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-combo.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-combo.c
@@ -2755,14 +2755,10 @@ static int qmp_combo_create(struct device *dev, struct device_node *np, int id,
*/
qphy->pipe_clk = devm_get_clk_from_child(dev, np, NULL);
if (IS_ERR(qphy->pipe_clk)) {
- if (cfg->type == PHY_TYPE_USB3) {
- ret = PTR_ERR(qphy->pipe_clk);
- if (ret != -EPROBE_DEFER)
- dev_err(dev,
- "failed to get lane%d pipe_clk, %d\n",
- id, ret);
- return ret;
- }
+ if (cfg->type == PHY_TYPE_USB3)
+ return dev_err_probe(dev, PTR_ERR(qphy->pipe_clk),
+ "failed to get lane%d pipe_clk\n",
+ id);
qphy->pipe_clk = NULL;
}
@@ -2879,12 +2875,9 @@ static int qmp_combo_probe(struct platform_device *pdev)
return ret;
ret = qmp_combo_vreg_init(dev, cfg);
- if (ret) {
- if (ret != -EPROBE_DEFER)
- dev_err(dev, "failed to get regulator supplies: %d\n",
- ret);
- return ret;
- }
+ if (ret)
+ return dev_err_probe(dev, ret,
+ "failed to get regulator supplies\n");
num = of_get_available_child_count(dev->of_node);
/* do we have a rogue child node ? */
--
2.17.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH RESEND 1/4] phy: qcom-qmp-combo: Use dev_err_probe() to simplify code
@ 2022-09-24 7:02 ` Yuan Can
0 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:02 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs.
Signed-off-by: Yuan Can <yuancan@huawei.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Andrew Halaney <ahalaney@redhat.com>
---
drivers/phy/qualcomm/phy-qcom-qmp-combo.c | 21 +++++++--------------
1 file changed, 7 insertions(+), 14 deletions(-)
diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-combo.c b/drivers/phy/qualcomm/phy-qcom-qmp-combo.c
index 41f938126ff1..da85fbf2cb0c 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-combo.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-combo.c
@@ -2755,14 +2755,10 @@ static int qmp_combo_create(struct device *dev, struct device_node *np, int id,
*/
qphy->pipe_clk = devm_get_clk_from_child(dev, np, NULL);
if (IS_ERR(qphy->pipe_clk)) {
- if (cfg->type == PHY_TYPE_USB3) {
- ret = PTR_ERR(qphy->pipe_clk);
- if (ret != -EPROBE_DEFER)
- dev_err(dev,
- "failed to get lane%d pipe_clk, %d\n",
- id, ret);
- return ret;
- }
+ if (cfg->type == PHY_TYPE_USB3)
+ return dev_err_probe(dev, PTR_ERR(qphy->pipe_clk),
+ "failed to get lane%d pipe_clk\n",
+ id);
qphy->pipe_clk = NULL;
}
@@ -2879,12 +2875,9 @@ static int qmp_combo_probe(struct platform_device *pdev)
return ret;
ret = qmp_combo_vreg_init(dev, cfg);
- if (ret) {
- if (ret != -EPROBE_DEFER)
- dev_err(dev, "failed to get regulator supplies: %d\n",
- ret);
- return ret;
- }
+ if (ret)
+ return dev_err_probe(dev, ret,
+ "failed to get regulator supplies\n");
num = of_get_available_child_count(dev->of_node);
/* do we have a rogue child node ? */
--
2.17.1
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH RESEND 2/4] phy: qcom-qmp-pcie-msm8996: Use dev_err_probe() to simplify code
2022-09-24 7:02 ` Yuan Can
@ 2022-09-24 7:02 ` Yuan Can
-1 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:02 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs.
Signed-off-by: Yuan Can <yuancan@huawei.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Andrew Halaney <ahalaney@redhat.com>
---
drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c b/drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c
index 245f6dc1710e..690e0825e3b1 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c
@@ -880,12 +880,9 @@ static int qmp_pcie_msm8996_probe(struct platform_device *pdev)
return ret;
ret = qmp_pcie_msm8996_vreg_init(dev, cfg);
- if (ret) {
- if (ret != -EPROBE_DEFER)
- dev_err(dev, "failed to get regulator supplies: %d\n",
- ret);
- return ret;
- }
+ if (ret)
+ return dev_err_probe(dev, ret,
+ "failed to get regulator supplies\n");
num = of_get_available_child_count(dev->of_node);
/* do we have a rogue child node ? */
--
2.17.1
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH RESEND 2/4] phy: qcom-qmp-pcie-msm8996: Use dev_err_probe() to simplify code
@ 2022-09-24 7:02 ` Yuan Can
0 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:02 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs.
Signed-off-by: Yuan Can <yuancan@huawei.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Andrew Halaney <ahalaney@redhat.com>
---
drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c b/drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c
index 245f6dc1710e..690e0825e3b1 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-pcie-msm8996.c
@@ -880,12 +880,9 @@ static int qmp_pcie_msm8996_probe(struct platform_device *pdev)
return ret;
ret = qmp_pcie_msm8996_vreg_init(dev, cfg);
- if (ret) {
- if (ret != -EPROBE_DEFER)
- dev_err(dev, "failed to get regulator supplies: %d\n",
- ret);
- return ret;
- }
+ if (ret)
+ return dev_err_probe(dev, ret,
+ "failed to get regulator supplies\n");
num = of_get_available_child_count(dev->of_node);
/* do we have a rogue child node ? */
--
2.17.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH RESEND 3/4] phy: qcom-qmp-ufs: Use dev_err_probe() to simplify code
2022-09-24 7:02 ` Yuan Can
@ 2022-09-24 7:02 ` Yuan Can
-1 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:02 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs.
Signed-off-by: Yuan Can <yuancan@huawei.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Andrew Halaney <ahalaney@redhat.com>
---
drivers/phy/qualcomm/phy-qcom-qmp-ufs.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c b/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c
index 7b335b50b4a1..672ef4bbada7 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c
@@ -1239,12 +1239,9 @@ static int qmp_ufs_probe(struct platform_device *pdev)
return ret;
ret = qmp_ufs_vreg_init(dev, cfg);
- if (ret) {
- if (ret != -EPROBE_DEFER)
- dev_err(dev, "failed to get regulator supplies: %d\n",
- ret);
- return ret;
- }
+ if (ret)
+ return dev_err_probe(dev, ret,
+ "failed to get regulator supplies\n");
num = of_get_available_child_count(dev->of_node);
/* do we have a rogue child node ? */
--
2.17.1
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH RESEND 3/4] phy: qcom-qmp-ufs: Use dev_err_probe() to simplify code
@ 2022-09-24 7:02 ` Yuan Can
0 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:02 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs.
Signed-off-by: Yuan Can <yuancan@huawei.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Andrew Halaney <ahalaney@redhat.com>
---
drivers/phy/qualcomm/phy-qcom-qmp-ufs.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c b/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c
index 7b335b50b4a1..672ef4bbada7 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c
@@ -1239,12 +1239,9 @@ static int qmp_ufs_probe(struct platform_device *pdev)
return ret;
ret = qmp_ufs_vreg_init(dev, cfg);
- if (ret) {
- if (ret != -EPROBE_DEFER)
- dev_err(dev, "failed to get regulator supplies: %d\n",
- ret);
- return ret;
- }
+ if (ret)
+ return dev_err_probe(dev, ret,
+ "failed to get regulator supplies\n");
num = of_get_available_child_count(dev->of_node);
/* do we have a rogue child node ? */
--
2.17.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH RESEND 4/4] phy: qcom-qmp-usb: Use dev_err_probe() to simplify code
2022-09-24 7:02 ` Yuan Can
@ 2022-09-24 7:03 ` Yuan Can
-1 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:03 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs.
Signed-off-by: Yuan Can <yuancan@huawei.com>
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Andrew Halaney <ahalaney@redhat.com>
---
drivers/phy/qualcomm/phy-qcom-qmp-usb.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-usb.c b/drivers/phy/qualcomm/phy-qcom-qmp-usb.c
index 93994f1a46e2..2ff2e798d6e2 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-usb.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-usb.c
@@ -2817,12 +2817,9 @@ static int qmp_usb_probe(struct platform_device *pdev)
return ret;
ret = qmp_usb_vreg_init(dev, cfg);
- if (ret) {
- if (ret != -EPROBE_DEFER)
- dev_err(dev, "failed to get regulator supplies: %d\n",
- ret);
- return ret;
- }
+ if (ret)
+ return dev_err_probe(dev, ret,
+ "failed to get regulator supplies\n");
num = of_get_available_child_count(dev->of_node);
/* do we have a rogue child node ? */
--
2.17.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH RESEND 4/4] phy: qcom-qmp-usb: Use dev_err_probe() to simplify code
@ 2022-09-24 7:03 ` Yuan Can
0 siblings, 0 replies; 12+ messages in thread
From: Yuan Can @ 2022-09-24 7:03 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, kishon, vkoul, linux-arm-msm,
linux-phy
Cc: yuancan
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs.
Signed-off-by: Yuan Can <yuancan@huawei.com>
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Andrew Halaney <ahalaney@redhat.com>
---
drivers/phy/qualcomm/phy-qcom-qmp-usb.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-usb.c b/drivers/phy/qualcomm/phy-qcom-qmp-usb.c
index 93994f1a46e2..2ff2e798d6e2 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-usb.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-usb.c
@@ -2817,12 +2817,9 @@ static int qmp_usb_probe(struct platform_device *pdev)
return ret;
ret = qmp_usb_vreg_init(dev, cfg);
- if (ret) {
- if (ret != -EPROBE_DEFER)
- dev_err(dev, "failed to get regulator supplies: %d\n",
- ret);
- return ret;
- }
+ if (ret)
+ return dev_err_probe(dev, ret,
+ "failed to get regulator supplies\n");
num = of_get_available_child_count(dev->of_node);
/* do we have a rogue child node ? */
--
2.17.1
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH RESEND 0/4] phy: Use dev_err_probe() to simplify code
2022-09-24 7:02 ` Yuan Can
@ 2022-09-24 7:33 ` Vinod Koul
-1 siblings, 0 replies; 12+ messages in thread
From: Vinod Koul @ 2022-09-24 7:33 UTC (permalink / raw)
To: Yuan Can
Cc: agross, andersson, konrad.dybcio, kishon, linux-arm-msm, linux-phy
On 24-09-22, 07:02, Yuan Can wrote:
> This series simplify the error handling in probe function by
> switching from dev_err() to dev_err_probe().
Applied, thanks
--
~Vinod
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH RESEND 0/4] phy: Use dev_err_probe() to simplify code
@ 2022-09-24 7:33 ` Vinod Koul
0 siblings, 0 replies; 12+ messages in thread
From: Vinod Koul @ 2022-09-24 7:33 UTC (permalink / raw)
To: Yuan Can
Cc: agross, andersson, konrad.dybcio, kishon, linux-arm-msm, linux-phy
On 24-09-22, 07:02, Yuan Can wrote:
> This series simplify the error handling in probe function by
> switching from dev_err() to dev_err_probe().
Applied, thanks
--
~Vinod
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2022-09-24 7:34 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-24 7:02 [PATCH RESEND 0/4] phy: Use dev_err_probe() to simplify code Yuan Can
2022-09-24 7:02 ` Yuan Can
2022-09-24 7:02 ` [PATCH RESEND 1/4] phy: qcom-qmp-combo: " Yuan Can
2022-09-24 7:02 ` Yuan Can
2022-09-24 7:02 ` [PATCH RESEND 2/4] phy: qcom-qmp-pcie-msm8996: " Yuan Can
2022-09-24 7:02 ` Yuan Can
2022-09-24 7:02 ` [PATCH RESEND 3/4] phy: qcom-qmp-ufs: " Yuan Can
2022-09-24 7:02 ` Yuan Can
2022-09-24 7:03 ` [PATCH RESEND 4/4] phy: qcom-qmp-usb: " Yuan Can
2022-09-24 7:03 ` Yuan Can
2022-09-24 7:33 ` [PATCH RESEND 0/4] phy: " Vinod Koul
2022-09-24 7:33 ` Vinod Koul
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.