| Message ID | 20240614-fix-pcie-phy-compat-v3-3-730d1811acf4@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| 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: e76862840660 ("arm64: dts: qcom: sm8450: correct pcie1 phy clocks inputs to gcc") > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- > arch/arm64/boot/dts/qcom/sm8450.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/qcom/sm8450.dtsi b/arch/arm64/boot/dts/qcom/sm8450.dtsi > index 1e762cc8085a..9bafb3b350ff 100644 > --- a/arch/arm64/boot/dts/qcom/sm8450.dtsi > +++ b/arch/arm64/boot/dts/qcom/sm8450.dtsi > @@ -2087,7 +2087,7 @@ pcie1_phy: phy@1c0e000 { > "rchng", > "pipe"; > > - clock-output-names = "pcie_1_pipe_clk", "pcie_1_phy_aux_clk"; > + clock-output-names = "pcie_1_pipe_clk"; > #clock-cells = <1>; > > #phy-cells = <0>; > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
diff --git a/arch/arm64/boot/dts/qcom/sm8450.dtsi b/arch/arm64/boot/dts/qcom/sm8450.dtsi index 1e762cc8085a..9bafb3b350ff 100644 --- a/arch/arm64/boot/dts/qcom/sm8450.dtsi +++ b/arch/arm64/boot/dts/qcom/sm8450.dtsi @@ -2087,7 +2087,7 @@ pcie1_phy: phy@1c0e000 { "rchng", "pipe"; - clock-output-names = "pcie_1_pipe_clk", "pcie_1_phy_aux_clk"; + clock-output-names = "pcie_1_pipe_clk"; #clock-cells = <1>; #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: e76862840660 ("arm64: dts: qcom: sm8450: correct pcie1 phy clocks inputs to gcc") Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- arch/arm64/boot/dts/qcom/sm8450.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)