Re: [PATCH v3 06/12] phy: qualcomm: qcom-uniphy-pcie-usb3-28lp: Improve error handling in power_on

From: Konrad Dybcio

Date: Tue Sep 01 2026 - 10:44:41 EST


On 8/25/26 2:37 PM, George Moussalem via B4 Relay wrote:
> From: George Moussalem <george.moussalem@xxxxxxxxxxx>
>
> When the PHY fails the enable clocks, it leaves the PHY out of reset.
> Fix this by asserting the reset before returning the error.
>
> In addition, language correct the error messages.
>
> Signed-off-by: George Moussalem <george.moussalem@xxxxxxxxxxx>
> ---

[...]

> ret = clk_bulk_prepare_enable(phy->num_clks, phy->clks);
> if (ret) {
> - dev_err(phy->dev, "clk prepare and enable failed %d\n", ret);
> - return ret;
> + dev_err(phy->dev, "failed to enable clocks: %d\n", ret);
> + goto err_assert_resets;

This is a single-use goto for a single line, just inline it here

Konrad