| Message ID | 20221228112456.31348-1-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | Rejected |
| Headers | show |
| Series | [1/2] arm64: dts: qcom: sm8350: add missing core_bi_pll_test_se GCC clock | expand |
On 28.12.2022 12:24, Krzysztof Kozlowski wrote: > The GCC bindings expect core_bi_pll_test_se clock input, even if it is > optional: > > sm8350-mtp.dtb: clock-controller@100000: clock-names:2: 'core_bi_pll_test_se' was expected > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- Is it even going to be used by anybody, or should we just drop it on the driver side as per usual? Konrad > arch/arm64/boot/dts/qcom/sm8350.dtsi | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi > index d473194c968d..d5747bb467e0 100644 > --- a/arch/arm64/boot/dts/qcom/sm8350.dtsi > +++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi > @@ -644,6 +644,7 @@ gcc: clock-controller@100000 { > #power-domain-cells = <1>; > clock-names = "bi_tcxo", > "sleep_clk", > + "core_bi_pll_test_se", > "pcie_0_pipe_clk", > "pcie_1_pipe_clk", > "ufs_card_rx_symbol_0_clk", > @@ -661,6 +662,7 @@ gcc: clock-controller@100000 { > <0>, > <0>, > <0>, > + <0>, > <&ufs_phy_rx_symbol_0_clk>, > <&ufs_phy_rx_symbol_1_clk>, > <&ufs_phy_tx_symbol_0_clk>,
On 28/12/2022 12:37, Konrad Dybcio wrote: > > > On 28.12.2022 12:24, Krzysztof Kozlowski wrote: >> The GCC bindings expect core_bi_pll_test_se clock input, even if it is >> optional: >> >> sm8350-mtp.dtb: clock-controller@100000: clock-names:2: 'core_bi_pll_test_se' was expected >> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> --- > Is it even going to be used by anybody, or should we just drop > it on the driver side as per usual? It's mentioned as possible parent, so there might be users somewhere... Or you want to say that other binding and DTS users cannot use that clock? Best regards, Krzysztof
On 28.12.2022 12:55, Krzysztof Kozlowski wrote: > On 28/12/2022 12:37, Konrad Dybcio wrote: >> >> >> On 28.12.2022 12:24, Krzysztof Kozlowski wrote: >>> The GCC bindings expect core_bi_pll_test_se clock input, even if it is >>> optional: >>> >>> sm8350-mtp.dtb: clock-controller@100000: clock-names:2: 'core_bi_pll_test_se' was expected >>> >>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>> --- >> Is it even going to be used by anybody, or should we just drop >> it on the driver side as per usual? > > It's mentioned as possible parent, so there might be users somewhere... > Or you want to say that other binding and DTS users cannot use that clock? There's no driver (even downstream) for a supplier of this clock and it's (probably) only used for early validation by qcom folks. What we're interested in, as far as debugging clocks goes, is handled by debugcc [1]. Konrad [1] https://github.com/andersson/debugcc/ > > Best regards, > Krzysztof >
On 28/12/2022 13:55, Krzysztof Kozlowski wrote: > On 28/12/2022 12:37, Konrad Dybcio wrote: >> >> >> On 28.12.2022 12:24, Krzysztof Kozlowski wrote: >>> The GCC bindings expect core_bi_pll_test_se clock input, even if it is >>> optional: >>> >>> sm8350-mtp.dtb: clock-controller@100000: clock-names:2: 'core_bi_pll_test_se' was expected >>> >>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>> --- >> Is it even going to be used by anybody, or should we just drop >> it on the driver side as per usual? > > It's mentioned as possible parent, so there might be users somewhere... > Or you want to say that other binding and DTS users cannot use that clock? Yes. In the past few months we have been removing the core_bi_pll_test from the old clock drivers (and new clock drivers mostly lack them). Let's remove it from the rest of clock drivers. > > Best regards, > Krzysztof >
On 28/12/2022 13:50, Dmitry Baryshkov wrote: > On 28/12/2022 13:55, Krzysztof Kozlowski wrote: >> On 28/12/2022 12:37, Konrad Dybcio wrote: >>> >>> >>> On 28.12.2022 12:24, Krzysztof Kozlowski wrote: >>>> The GCC bindings expect core_bi_pll_test_se clock input, even if it is >>>> optional: >>>> >>>> sm8350-mtp.dtb: clock-controller@100000: clock-names:2: 'core_bi_pll_test_se' was expected >>>> >>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>>> --- >>> Is it even going to be used by anybody, or should we just drop >>> it on the driver side as per usual? >> >> It's mentioned as possible parent, so there might be users somewhere... >> Or you want to say that other binding and DTS users cannot use that clock? > > Yes. In the past few months we have been removing the core_bi_pll_test > from the old clock drivers (and new clock drivers mostly lack them). > Let's remove it from the rest of clock drivers. If you are going to start doing the same work, please at least share it upfront. Best regards, Krzysztof
On 28/12/2022 16:25, Krzysztof Kozlowski wrote: > On 28/12/2022 13:50, Dmitry Baryshkov wrote: >> On 28/12/2022 13:55, Krzysztof Kozlowski wrote: >>> On 28/12/2022 12:37, Konrad Dybcio wrote: >>>> >>>> >>>> On 28.12.2022 12:24, Krzysztof Kozlowski wrote: >>>>> The GCC bindings expect core_bi_pll_test_se clock input, even if it is >>>>> optional: >>>>> >>>>> sm8350-mtp.dtb: clock-controller@100000: clock-names:2: 'core_bi_pll_test_se' was expected >>>>> >>>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>>>> --- >>>> Is it even going to be used by anybody, or should we just drop >>>> it on the driver side as per usual? >>> >>> It's mentioned as possible parent, so there might be users somewhere... >>> Or you want to say that other binding and DTS users cannot use that clock? >> >> Yes. In the past few months we have been removing the core_bi_pll_test >> from the old clock drivers (and new clock drivers mostly lack them). >> Let's remove it from the rest of clock drivers. > > If you are going to start doing the same work, please at least share it > upfront. Excuse me.
On Wed, 28 Dec 2022 12:24:55 +0100, Krzysztof Kozlowski wrote: > The GCC bindings expect core_bi_pll_test_se clock input, even if it is > optional: > > sm8350-mtp.dtb: clock-controller@100000: clock-names:2: 'core_bi_pll_test_se' was expected > > Applied, thanks! [2/2] arm64: dts: qcom: sm8350-sony-xperia-sagami: specify which LDO modes are allowed commit: 8ea261588fe98d171fcecf477a9f27aea8a06fd0 Best regards,
diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi index d473194c968d..d5747bb467e0 100644 --- a/arch/arm64/boot/dts/qcom/sm8350.dtsi +++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi @@ -644,6 +644,7 @@ gcc: clock-controller@100000 { #power-domain-cells = <1>; clock-names = "bi_tcxo", "sleep_clk", + "core_bi_pll_test_se", "pcie_0_pipe_clk", "pcie_1_pipe_clk", "ufs_card_rx_symbol_0_clk", @@ -661,6 +662,7 @@ gcc: clock-controller@100000 { <0>, <0>, <0>, + <0>, <&ufs_phy_rx_symbol_0_clk>, <&ufs_phy_rx_symbol_1_clk>, <&ufs_phy_tx_symbol_0_clk>,
The GCC bindings expect core_bi_pll_test_se clock input, even if it is optional: sm8350-mtp.dtb: clock-controller@100000: clock-names:2: 'core_bi_pll_test_se' was expected Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- arch/arm64/boot/dts/qcom/sm8350.dtsi | 2 ++ 1 file changed, 2 insertions(+)