Re: [PATCH v2 2/3] dt-bindings: clk: qcom: Add YAML schemas for the GCC clock bindings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Rob,

Thank you for your review comments.

On 8/20/2019 12:44 AM, Rob Herring wrote:
On Mon, Aug 19, 2019 at 11:38 AM Taniya Das <tdas@xxxxxxxxxxxxxx> wrote:

The GCC clock provider have a bunch of generic properties that
are needed in a device tree. Add a YAML schemas for those. Also update
the compatible for SC7180 along with example for clocks & clock-names.

Signed-off-by: Taniya Das <tdas@xxxxxxxxxxxxxx>
---
  .../devicetree/bindings/clock/qcom,gcc.yaml   | 141 ++++++++++++++++
  include/dt-bindings/clock/qcom,gcc-sc7180.h   | 155 ++++++++++++++++++
  2 files changed, 296 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/clock/qcom,gcc.yaml
  create mode 100644 include/dt-bindings/clock/qcom,gcc-sc7180.h

You need to remove the old doc.


Would remove it in the next patch.


diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc.yaml
new file mode 100644
index 000000000000..17c563a036c7
--- /dev/null
+++ b/Documentation/devicetree/bindings/clock/qcom,gcc.yaml
@@ -0,0 +1,141 @@
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/clock/qcom,gcc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Qualcomm Global Clock & Reset Controller Binding
+
+maintainers:
+  - Stephen Boyd <sboyd@xxxxxxxxxx>
+
+properties:
+  "#clock-cells":
+    const: 1
+
+  "#reset-cells":
+    const: 1
+
+  reg:
+    maxItems: 1
+    description: |
+      shall contain the base register location and length

Don't need a description if there's only 1 entry.


Thanks, would remove.

+
+  compatible :
+     enum:
+       - qcom,gcc-apq8064
+       - qcom,gcc-apq8084
+       - qcom,gcc-ipq8064
+       - qcom,gcc-ipq4019
+       - qcom,gcc-ipq8074
+       - qcom,gcc-msm8660
+       - qcom,gcc-msm8916
+       - qcom,gcc-msm8960
+       - qcom,gcc-msm8974
+       - qcom,gcc-msm8974pro
+       - qcom,gcc-msm8974pro-ac
+       - qcom,gcc-msm8994
+       - qcom,gcc-msm8996
+       - qcom,gcc-msm8998
+       - qcom,gcc-mdm9615
+       - qcom,gcc-qcs404
+       - qcom,gcc-sdm630
+       - qcom,gcc-sdm660
+       - qcom,gcc-sdm845
+       - qcom,gcc-sc7180
+
+  clocks:
+    minItems: 1
+    maxItems: 2
+    items:
+      - description: Board XO source
+      - description: Board active XO source
+
+  clock-names:
+    minItems: 1
+    maxItems: 2
+    items:
+      - const: bi_tcxo
+      - const: bi_tcxo_ao
+
+  nvmem-cells:
+    $ref: /schemas/types.yaml#/definitions/phandle-array

Standard property, you don't need the type here. What's needed is the size.


Would take care of it in the next patch.

+    description:
+      Qualcomm TSENS (thermal sensor device) on some devices can
+      be part of GCC and hence the TSENS properties can also be part
+      of the GCC/clock-controller node.
+      For more details on the TSENS properties please refer
+      Documentation/devicetree/bindings/thermal/qcom-tsens.txt
+
+  nvmem-cell-names:
+    $ref: /schemas/types.yaml#/definitions/string-array

Don't need the type (and this would have to be under an 'allOf' to
actually work).


Would take care of it in the next patch.

+    description:
+      Names for each nvmem-cells specified.
+    items:
+      - const: calib
+      - const: calib_backup
+
+  "#thermal-sensor-cells":
+    const: 1
+
+  "#power-domain-cells":
+    const: 1
+
+  protected-clocks:
+    description:
+       Protected clock specifier list as per common clock binding
+
+required:
+  - "#clock-cells"
+  - "#reset-cells"
+  - compatible
+  - reg
+  - clocks
+  - clock-names
+
+examples:
+  - |
+    clock-controller@900000 {
+      compatible = "qcom,gcc-msm8960";
+      reg = <0x900000 0x4000>;
+      #clock-cells = <1>;
+      #reset-cells = <1>;
+      #power-domain-cells = <1>;
+    };
+
+
+  - |
+    // Example of GCC with TSENS properties:
+    clock-controller@900000 {
+      compatible = "qcom,gcc-apq8064";
+      reg = <0x00900000 0x4000>;
+      nvmem-cells = <&tsens_calib>, <&tsens_backup>;
+      nvmem-cell-names = "calib", "calib_backup";
+      #clock-cells = <1>;
+      #reset-cells = <1>;
+      #thermal-sensor-cells = <1>;
+    };
+
+  - |
+    //Example of GCC with protected-clocks properties:
+    clock-controller@100000 {
+      compatible = "qcom,gcc-sdm845";
+      reg = <0x100000 0x1f0000>;
+      #clock-cells = <1>;
+      #reset-cells = <1>;
+      #power-domain-cells = <1>;
+      protected-clocks = <187>, <188>, <189>, <190>, <191>;
+    };
+
+  - |
+    //Example of GCC with clock nodes properties:
+    clock-controller@100000 {
+      compatible = "qcom,gcc-sc7180";
+      reg = <0x100000 0x1f0000>;
+      clocks = <&rpmhcc 0>, <&rpmhcc 1>;
+      clock-names = "bi_tcxo", "bi_tcxo_ao";
+      #clock-cells = <1>;
+      #reset-cells = <1>;
+      #power-domain-cells = <1>;
+    };
+...
diff --git a/include/dt-bindings/clock/qcom,gcc-sc7180.h b/include/dt-bindings/clock/qcom,gcc-sc7180.h
new file mode 100644
index 000000000000..d76b061f6a4e
--- /dev/null
+++ b/include/dt-bindings/clock/qcom,gcc-sc7180.h
@@ -0,0 +1,155 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Copyright (c) 2019, The Linux Foundation. All rights reserved.
+ */
+
+#ifndef _DT_BINDINGS_CLK_QCOM_GCC_SC7180_H
+#define _DT_BINDINGS_CLK_QCOM_GCC_SC7180_H
+
+/* GCC clocks */
+#define GCC_GPLL0_MAIN_DIV_CDIV                                        0
+#define GPLL0                                                  1
+#define GPLL0_OUT_EVEN                                         2
+#define GPLL1                                                  3
+#define GPLL4                                                  4
+#define GPLL6                                                  5
+#define GPLL7                                                  6
+#define GCC_AGGRE_UFS_PHY_AXI_CLK                              7
+#define GCC_AGGRE_USB3_PRIM_AXI_CLK                            8
+#define GCC_BOOT_ROM_AHB_CLK                                   9
+#define GCC_CAMERA_AHB_CLK                                     10
+#define GCC_CAMERA_HF_AXI_CLK                                  11
+#define GCC_CAMERA_THROTTLE_HF_AXI_CLK                         12
+#define GCC_CAMERA_XO_CLK                                      13
+#define GCC_CE1_AHB_CLK                                                14
+#define GCC_CE1_AXI_CLK                                                15
+#define GCC_CE1_CLK                                            16
+#define GCC_CFG_NOC_USB3_PRIM_AXI_CLK                          17
+#define GCC_CPUSS_AHB_CLK                                      18
+#define GCC_CPUSS_AHB_CLK_SRC                                  19
+#define GCC_CPUSS_GNOC_CLK                                     20
+#define GCC_CPUSS_RBCPR_CLK                                    21
+#define GCC_DDRSS_GPU_AXI_CLK                                  22
+#define GCC_DISP_AHB_CLK                                       23
+#define GCC_DISP_GPLL0_CLK_SRC                                 24
+#define GCC_DISP_GPLL0_DIV_CLK_SRC                             25
+#define GCC_DISP_HF_AXI_CLK                                    26
+#define GCC_DISP_THROTTLE_HF_AXI_CLK                           27
+#define GCC_DISP_XO_CLK                                                28
+#define GCC_GP1_CLK                                            29
+#define GCC_GP1_CLK_SRC                                                30
+#define GCC_GP2_CLK                                            31
+#define GCC_GP2_CLK_SRC                                                32
+#define GCC_GP3_CLK                                            33
+#define GCC_GP3_CLK_SRC                                                34
+#define GCC_GPU_CFG_AHB_CLK                                    35
+#define GCC_GPU_GPLL0_CLK_SRC                                  36
+#define GCC_GPU_GPLL0_DIV_CLK_SRC                              37
+#define GCC_GPU_MEMNOC_GFX_CLK                                 38
+#define GCC_GPU_SNOC_DVM_GFX_CLK                               39
+#define GCC_NPU_AXI_CLK                                                40
+#define GCC_NPU_BWMON_AXI_CLK                                  41
+#define GCC_NPU_BWMON_DMA_CFG_AHB_CLK                          42
+#define GCC_NPU_BWMON_DSP_CFG_AHB_CLK                          43
+#define GCC_NPU_CFG_AHB_CLK                                    44
+#define GCC_NPU_DMA_CLK                                                45
+#define GCC_NPU_GPLL0_CLK_SRC                                  46
+#define GCC_NPU_GPLL0_DIV_CLK_SRC                              47
+#define GCC_PDM2_CLK                                           48
+#define GCC_PDM2_CLK_SRC                                       49
+#define GCC_PDM_AHB_CLK                                                50
+#define GCC_PDM_XO4_CLK                                                51
+#define GCC_PRNG_AHB_CLK                                       52
+#define GCC_QSPI_CNOC_PERIPH_AHB_CLK                           53
+#define GCC_QSPI_CORE_CLK                                      54
+#define GCC_QSPI_CORE_CLK_SRC                                  55
+#define GCC_QUPV3_WRAP0_CORE_2X_CLK                            56
+#define GCC_QUPV3_WRAP0_CORE_CLK                               57
+#define GCC_QUPV3_WRAP0_S0_CLK                                 58
+#define GCC_QUPV3_WRAP0_S0_CLK_SRC                             59
+#define GCC_QUPV3_WRAP0_S1_CLK                                 60
+#define GCC_QUPV3_WRAP0_S1_CLK_SRC                             61
+#define GCC_QUPV3_WRAP0_S2_CLK                                 62
+#define GCC_QUPV3_WRAP0_S2_CLK_SRC                             63
+#define GCC_QUPV3_WRAP0_S3_CLK                                 64
+#define GCC_QUPV3_WRAP0_S3_CLK_SRC                             65
+#define GCC_QUPV3_WRAP0_S4_CLK                                 66
+#define GCC_QUPV3_WRAP0_S4_CLK_SRC                             67
+#define GCC_QUPV3_WRAP0_S5_CLK                                 68
+#define GCC_QUPV3_WRAP0_S5_CLK_SRC                             69
+#define GCC_QUPV3_WRAP1_CORE_2X_CLK                            70
+#define GCC_QUPV3_WRAP1_CORE_CLK                               71
+#define GCC_QUPV3_WRAP1_S0_CLK                                 72
+#define GCC_QUPV3_WRAP1_S0_CLK_SRC                             73
+#define GCC_QUPV3_WRAP1_S1_CLK                                 74
+#define GCC_QUPV3_WRAP1_S1_CLK_SRC                             75
+#define GCC_QUPV3_WRAP1_S2_CLK                                 76
+#define GCC_QUPV3_WRAP1_S2_CLK_SRC                             77
+#define GCC_QUPV3_WRAP1_S3_CLK                                 78
+#define GCC_QUPV3_WRAP1_S3_CLK_SRC                             79
+#define GCC_QUPV3_WRAP1_S4_CLK                                 80
+#define GCC_QUPV3_WRAP1_S4_CLK_SRC                             81
+#define GCC_QUPV3_WRAP1_S5_CLK                                 82
+#define GCC_QUPV3_WRAP1_S5_CLK_SRC                             83
+#define GCC_QUPV3_WRAP_0_M_AHB_CLK                             84
+#define GCC_QUPV3_WRAP_0_S_AHB_CLK                             85
+#define GCC_QUPV3_WRAP_1_M_AHB_CLK                             86
+#define GCC_QUPV3_WRAP_1_S_AHB_CLK                             87
+#define GCC_SDCC1_AHB_CLK                                      88
+#define GCC_SDCC1_APPS_CLK                                     89
+#define GCC_SDCC1_APPS_CLK_SRC                                 90
+#define GCC_SDCC1_ICE_CORE_CLK                                 91
+#define GCC_SDCC1_ICE_CORE_CLK_SRC                             92
+#define GCC_SDCC2_AHB_CLK                                      93
+#define GCC_SDCC2_APPS_CLK                                     94
+#define GCC_SDCC2_APPS_CLK_SRC                                 95
+#define GCC_SYS_NOC_CPUSS_AHB_CLK                              96
+#define GCC_UFS_MEM_CLKREF_CLK                                 97
+#define GCC_UFS_PHY_AHB_CLK                                    98
+#define GCC_UFS_PHY_AXI_CLK                                    99
+#define GCC_UFS_PHY_AXI_CLK_SRC                                        100
+#define GCC_UFS_PHY_ICE_CORE_CLK                               101
+#define GCC_UFS_PHY_ICE_CORE_CLK_SRC                           102
+#define GCC_UFS_PHY_PHY_AUX_CLK                                        103
+#define GCC_UFS_PHY_PHY_AUX_CLK_SRC                            104
+#define GCC_UFS_PHY_RX_SYMBOL_0_CLK                            105
+#define GCC_UFS_PHY_TX_SYMBOL_0_CLK                            106
+#define GCC_UFS_PHY_UNIPRO_CORE_CLK                            107
+#define GCC_UFS_PHY_UNIPRO_CORE_CLK_SRC                                108
+#define GCC_USB30_PRIM_MASTER_CLK                              109
+#define GCC_USB30_PRIM_MASTER_CLK_SRC                          110
+#define GCC_USB30_PRIM_MOCK_UTMI_CLK                           111
+#define GCC_USB30_PRIM_MOCK_UTMI_CLK_SRC                       112
+#define GCC_USB30_PRIM_SLEEP_CLK                               113
+#define GCC_USB3_PRIM_CLKREF_CLK                               114
+#define GCC_USB3_PRIM_PHY_AUX_CLK                              115
+#define GCC_USB3_PRIM_PHY_AUX_CLK_SRC                          116
+#define GCC_USB3_PRIM_PHY_COM_AUX_CLK                          117
+#define GCC_USB3_PRIM_PHY_PIPE_CLK                             118
+#define GCC_USB_PHY_CFG_AHB2PHY_CLK                            119
+#define GCC_VIDEO_AHB_CLK                                      120
+#define GCC_VIDEO_AXI_CLK                                      121
+#define GCC_VIDEO_GPLL0_DIV_CLK_SRC                            122
+#define GCC_VIDEO_THROTTLE_AXI_CLK                             123
+#define GCC_VIDEO_XO_CLK                                       124
+
+/* GCC resets */
+#define GCC_QUSB2PHY_PRIM_BCR                                  0
+#define GCC_QUSB2PHY_SEC_BCR                                   1
+#define GCC_UFS_PHY_BCR                                                2
+#define GCC_USB30_PRIM_BCR                                     3
+#define GCC_USB3_DP_PHY_PRIM_BCR                               4
+#define GCC_USB3_DP_PHY_SEC_BCR                                        5
+#define GCC_USB3_PHY_PRIM_BCR                                  6
+#define GCC_USB3_PHY_SEC_BCR                                   7
+#define GCC_USB3PHY_PHY_PRIM_BCR                               8
+#define GCC_USB3PHY_PHY_SEC_BCR                                        9
+#define GCC_USB_PHY_CFG_AHB2PHY_BCR                            10
+
+/* GCC GDSCRs */
+#define UFS_PHY_GDSC                                           0
+#define USB30_PRIM_GDSC                                                1
+#define HLOS1_VOTE_MMNOC_MMU_TBU_HF0_GDSC                      2
+#define HLOS1_VOTE_MMNOC_MMU_TBU_SF_GDSC                       3
+
+#endif
--
Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member
of the Code Aurora Forum, hosted by the  Linux Foundation.


--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation.

--



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux