| Message ID | 20240216-drop-sc7180-mss-v1-2-0a8dc8d71c0c@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | clk: qcom: drop the SC7180 Modem subsystem clock driver | expand |
On 16.02.2024 12:14, Dmitry Baryshkov wrote: > This clock controller has never been used in the DT files merged to the > kernel. According to Sibi, it only worked on the pre-production devices. > For the production devices this functionality has been moved to the > firmware. > > Drop the bindings now after dropping the driver itself. > > Cc: Sibi Sankar <quic_sibis@quicinc.com> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- I'd say it's fine to keep the bindings (for hw) Konrad
On 16/02/2024 12:14, Dmitry Baryshkov wrote: > This clock controller has never been used in the DT files merged to the > kernel. According to Sibi, it only worked on the pre-production devices. > For the production devices this functionality has been moved to the > firmware. > > Drop the bindings now after dropping the driver itself. > > Cc: Sibi Sankar <quic_sibis@quicinc.com> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- > .../devicetree/bindings/clock/qcom,sc7180-mss.yaml | 61 ---------------------- > 1 file changed, 61 deletions(-) Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On 2/16/24 16:44, Dmitry Baryshkov wrote: > This clock controller has never been used in the DT files merged to the > kernel. According to Sibi, it only worked on the pre-production devices. > For the production devices this functionality has been moved to the > firmware. > > Drop the bindings now after dropping the driver itself. > > Cc: Sibi Sankar <quic_sibis@quicinc.com> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Reviewed-by: Sibi Sankar <quic_sibis@quicinc.com> > --- > .../devicetree/bindings/clock/qcom,sc7180-mss.yaml | 61 ---------------------- > 1 file changed, 61 deletions(-) > > diff --git a/Documentation/devicetree/bindings/clock/qcom,sc7180-mss.yaml b/Documentation/devicetree/bindings/clock/qcom,sc7180-mss.yaml > deleted file mode 100644 > index 873a2f918bac..000000000000 > --- a/Documentation/devicetree/bindings/clock/qcom,sc7180-mss.yaml > +++ /dev/null > @@ -1,61 +0,0 @@ > -# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) [...] > -... >
diff --git a/Documentation/devicetree/bindings/clock/qcom,sc7180-mss.yaml b/Documentation/devicetree/bindings/clock/qcom,sc7180-mss.yaml deleted file mode 100644 index 873a2f918bac..000000000000 --- a/Documentation/devicetree/bindings/clock/qcom,sc7180-mss.yaml +++ /dev/null @@ -1,61 +0,0 @@ -# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) -%YAML 1.2 ---- -$id: http://devicetree.org/schemas/clock/qcom,sc7180-mss.yaml# -$schema: http://devicetree.org/meta-schemas/core.yaml# - -title: Qualcomm Modem Clock Controller on SC7180 - -maintainers: - - Taniya Das <quic_tdas@quicinc.com> - -description: | - Qualcomm modem clock control module provides the clocks on SC7180. - - See also:: include/dt-bindings/clock/qcom,mss-sc7180.h - -properties: - compatible: - const: qcom,sc7180-mss - - clocks: - items: - - description: gcc_mss_mfab_axi clock from GCC - - description: gcc_mss_nav_axi clock from GCC - - description: gcc_mss_cfg_ahb clock from GCC - - clock-names: - items: - - const: gcc_mss_mfab_axis - - const: gcc_mss_nav_axi - - const: cfg_ahb - - '#clock-cells': - const: 1 - - reg: - maxItems: 1 - -required: - - compatible - - reg - - clocks - - '#clock-cells' - -additionalProperties: false - -examples: - - | - #include <dt-bindings/clock/qcom,gcc-sc7180.h> - clock-controller@41a8000 { - compatible = "qcom,sc7180-mss"; - reg = <0x041a8000 0x8000>; - clocks = <&gcc GCC_MSS_MFAB_AXIS_CLK>, - <&gcc GCC_MSS_NAV_AXI_CLK>, - <&gcc GCC_MSS_CFG_AHB_CLK>; - clock-names = "gcc_mss_mfab_axis", - "gcc_mss_nav_axi", - "cfg_ahb"; - #clock-cells = <1>; - }; -...
This clock controller has never been used in the DT files merged to the kernel. According to Sibi, it only worked on the pre-production devices. For the production devices this functionality has been moved to the firmware. Drop the bindings now after dropping the driver itself. Cc: Sibi Sankar <quic_sibis@quicinc.com> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- .../devicetree/bindings/clock/qcom,sc7180-mss.yaml | 61 ---------------------- 1 file changed, 61 deletions(-)