| Message ID | 20240614-fix-pcie-phy-compat-v3-5-730d1811acf4@linaro.org |
|---|---|
| State | Accepted |
| Commit | dc323623c3b87c48c99fe8dbbd1962f0129d3da9 |
| Headers | show |
| Series | phy: qcom: qmp-pcie: drop second clock-output-names entry | expand |
On 14/06/2024 12:18, Dmitry Baryshkov wrote: > There is no need to specify exact name for the second (AUX) output > clock. It has never been used for the lookups based on the system > clock name. The driver generates it on its own, in order to remain > compatible with the older DT. Drop the clock name. > > Fixes: d00b42f170df ("arm64: dts: qcom: sm8650: remove pcie-1-phy-aux-clk and add pcie1_phy pcie1_phy_aux_clk") > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- > arch/arm64/boot/dts/qcom/sm8650.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/qcom/sm8650.dtsi b/arch/arm64/boot/dts/qcom/sm8650.dtsi > index 5b8b1d581a13..5df2e00fdb5b 100644 > --- a/arch/arm64/boot/dts/qcom/sm8650.dtsi > +++ b/arch/arm64/boot/dts/qcom/sm8650.dtsi > @@ -2474,7 +2474,7 @@ pcie1_phy: phy@1c0e000 { > power-domains = <&gcc PCIE_1_PHY_GDSC>; > > #clock-cells = <1>; > - clock-output-names = "pcie1_pipe_clk", "pcie1_phy_aux_clk"; > + clock-output-names = "pcie1_pipe_clk"; > > #phy-cells = <0>; > > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
diff --git a/arch/arm64/boot/dts/qcom/sm8650.dtsi b/arch/arm64/boot/dts/qcom/sm8650.dtsi index 5b8b1d581a13..5df2e00fdb5b 100644 --- a/arch/arm64/boot/dts/qcom/sm8650.dtsi +++ b/arch/arm64/boot/dts/qcom/sm8650.dtsi @@ -2474,7 +2474,7 @@ pcie1_phy: phy@1c0e000 { power-domains = <&gcc PCIE_1_PHY_GDSC>; #clock-cells = <1>; - clock-output-names = "pcie1_pipe_clk", "pcie1_phy_aux_clk"; + clock-output-names = "pcie1_pipe_clk"; #phy-cells = <0>;
There is no need to specify exact name for the second (AUX) output clock. It has never been used for the lookups based on the system clock name. The driver generates it on its own, in order to remain compatible with the older DT. Drop the clock name. Fixes: d00b42f170df ("arm64: dts: qcom: sm8650: remove pcie-1-phy-aux-clk and add pcie1_phy pcie1_phy_aux_clk") Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- arch/arm64/boot/dts/qcom/sm8650.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)