Re: [PATCH 1/3] clk: qcom: Add SM8550 clock driver

On 10/04/2024 11:27, Sumit Garg wrote:

On Wed, 10 Apr 2024 at 14:46, Neil Armstrong neil.armstrong@linaro.org wrote:

...

On 10/04/2024 11:13, Sumit Garg wrote:

...

Hi Neil,

On Thu, 4 Apr 2024 at 22:16, Neil Armstrong neil.armstrong@linaro.org wrote:

...

Add the GCC and TCSRCC clock driver for the SM8550 SoC.

The GCC driver uses the clk-qcom infrastructure to support GDSCs, Resets and gates. While the TCSRCC is a simpler clock driver which only supports gates.

The GCC enable and set_rate callbacks contains some tweaks to setup clocks for Debug UART, SDCard controller and USB.

Okay so these are the peripherals you intend to support to begin with.

...

The TCSRCC gates returns the XO frequency, which is used by the Synopsys eUSB2 driver to determine the PHY configuration.

Signed-off-by: Neil Armstrong neil.armstrong@linaro.org

drivers/clk/qcom/Kconfig | 8 + drivers/clk/qcom/Makefile | 1 + drivers/clk/qcom/clock-sm8550.c | 335 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 344 insertions(+)

diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig index 8dae635ac2c..c908a3d19c9 100644 --- a/drivers/clk/qcom/Kconfig +++ b/drivers/clk/qcom/Kconfig @@ -47,6 +47,14 @@ config CLK_QCOM_SDM845 on the Snapdragon 845 SoC. This driver supports the clocks and resets exposed by the GCC hardware block.

+config CLK_QCOM_SM8550

•   bool "Qualcomm SM8550 GCC"
  

•   select CLK_QCOM
  

•   help
  

•     Say Y here to enable support for the Global Clock Controller
  

•     on the Snapdragon SM8550 SoC. This driver supports the clocks
  

•     and resets exposed by the GCC hardware block.
  

• endmenu

  endif

diff --git a/drivers/clk/qcom/Makefile b/drivers/clk/qcom/Makefile index cb179fdac58..d9ac5719f49 100644 --- a/drivers/clk/qcom/Makefile +++ b/drivers/clk/qcom/Makefile @@ -8,3 +8,4 @@ obj-$(CONFIG_CLK_QCOM_APQ8016) += clock-apq8016.o obj-$(CONFIG_CLK_QCOM_APQ8096) += clock-apq8096.o obj-$(CONFIG_CLK_QCOM_IPQ4019) += clock-ipq4019.o obj-$(CONFIG_CLK_QCOM_QCS404) += clock-qcs404.o +obj-$(CONFIG_CLK_QCOM_SM8550) += clock-sm8550.o diff --git a/drivers/clk/qcom/clock-sm8550.c b/drivers/clk/qcom/clock-sm8550.c new file mode 100644 index 00000000000..c0249925cc7 --- /dev/null +++ b/drivers/clk/qcom/clock-sm8550.c @@ -0,0 +1,335 @@ +// SPDX-License-Identifier: BSD-3-Clause +/*

• • Clock drivers for Qualcomm sm8550
• • (C) Copyright 2024 Linaro Ltd.
• */

+#include <clk-uclass.h> +#include <dm.h> +#include <linux/delay.h> +#include <errno.h> +#include <asm/io.h> +#include <linux/bug.h> +#include <linux/bitops.h> +#include <dt-bindings/clock/qcom,sm8550-gcc.h> +#include <dt-bindings/clock/qcom,sm8550-tcsr.h>

+#include "clock-qcom.h"

+/* On-board TCXO, TOFIX get from DT */ +#define TCXO_RATE 38400000

+/* bi_tcxo_div2 divided after RPMh output */ +#define TCXO_DIV2_RATE (TCXO_RATE / 2)

+static const struct freq_tbl ftbl_gcc_qupv3_wrap1_s2_clk_src[] = {

•   F(7372800, CFG_CLK_SRC_GPLL0_EVEN, 1, 384, 15625),
  

•   F(14745600, CFG_CLK_SRC_GPLL0_EVEN, 1, 768, 15625),
  

•   F(19200000, CFG_CLK_SRC_CXO, 1, 0, 0),
  

•   F(29491200, CFG_CLK_SRC_GPLL0_EVEN, 1, 1536, 15625),
  

•   F(32000000, CFG_CLK_SRC_GPLL0_EVEN, 1, 8, 75),
  

•   F(48000000, CFG_CLK_SRC_GPLL0_EVEN, 1, 4, 25),
  

•   F(51200000, CFG_CLK_SRC_GPLL0_EVEN, 1, 64, 375),
  

•   F(64000000, CFG_CLK_SRC_GPLL0_EVEN, 1, 16, 75),
  

•   F(75000000, CFG_CLK_SRC_GPLL0_EVEN, 4, 0, 0),
  

•   F(80000000, CFG_CLK_SRC_GPLL0_EVEN, 1, 4, 15),
  

•   F(96000000, CFG_CLK_SRC_GPLL0_EVEN, 1, 8, 25),
  

•   F(100000000, CFG_CLK_SRC_GPLL0, 6, 0, 0),
  

•   { }
  

+};

+static const struct freq_tbl ftbl_gcc_sdcc2_apps_clk_src[] = {

•   F(400000, CFG_CLK_SRC_CXO, 12, 1, 4),
  

•   F(25000000, CFG_CLK_SRC_GPLL0_EVEN, 12, 0, 0),
  

•   F(37500000, CFG_CLK_SRC_GPLL0_EVEN, 8, 0, 0),
  

•   F(50000000, CFG_CLK_SRC_GPLL0_EVEN, 6, 0, 0),
  

•   F(100000000, CFG_CLK_SRC_GPLL0_EVEN, 3, 0, 0),
  

•   /* TOFIX F(202000000, CFG_CLK_SRC_GPLL9, 4, 0, 0), */
  

•   { }
  

+};

+static const struct freq_tbl ftbl_gcc_usb30_prim_master_clk_src[] = {

•   F(66666667, CFG_CLK_SRC_GPLL0_EVEN, 4.5, 0, 0),
  

•   F(133333333, CFG_CLK_SRC_GPLL0, 4.5, 0, 0),
  

•   F(200000000, CFG_CLK_SRC_GPLL0, 3, 0, 0),
  

•   F(240000000, CFG_CLK_SRC_GPLL0, 2.5, 0, 0),
  

•   { }
  

+};

+static ulong sm8550_set_rate(struct clk *clk, ulong rate) +{

•   struct msm_clk_priv *priv = dev_get_priv(clk->dev);
  

•   const struct freq_tbl *freq;
  

•   switch (clk->id) {
  

•   case GCC_QUPV3_WRAP1_S7_CLK: /* UART7 */
  

•           freq = qcom_find_freq(ftbl_gcc_qupv3_wrap1_s2_clk_src, rate);
  

•           clk_rcg_set_rate_mnd(priv->base, 0x18898,
  

•                                freq->pre_div, freq->m, freq->n, freq->src, 16);
  

•           return freq->freq;
  

•   case GCC_SDCC2_APPS_CLK:
  

•           freq = qcom_find_freq(ftbl_gcc_sdcc2_apps_clk_src, rate);
  

•           clk_rcg_set_rate_mnd(priv->base, 0x14018,
  

•                                freq->pre_div, freq->m, freq->n, freq->src, 8);
  

•           return freq->freq;
  

•   case GCC_USB30_PRIM_MASTER_CLK:
  

•           freq = qcom_find_freq(ftbl_gcc_usb30_prim_master_clk_src, rate);
  

•           clk_rcg_set_rate_mnd(priv->base, 0x3902c,
  

•                                freq->pre_div, freq->m, freq->n, freq->src, 8);
  

•           return freq->freq;
  

•   case GCC_USB30_PRIM_MOCK_UTMI_CLK:
  

•           clk_rcg_set_rate(priv->base, 0x39044, 0, 0);
  

•           return TCXO_DIV2_RATE;
  

•   case GCC_USB3_PRIM_PHY_AUX_CLK_SRC:
  

•           clk_rcg_set_rate(priv->base, 0x39070, 0, 0);
  

•           return TCXO_DIV2_RATE;
  

•   default:
  

•           return 0;
  

•   }
  

+}

+static const struct gate_clk sm8550_clks[] = {

•   GATE_CLK(GCC_AGGRE_NOC_PCIE_AXI_CLK,            0x52000, BIT(12)),
  

•   GATE_CLK(GCC_AGGRE_UFS_PHY_AXI_CLK,             0x770e4, BIT(0)),
  

•   GATE_CLK(GCC_AGGRE_UFS_PHY_AXI_HW_CTL_CLK,      0x770e4, BIT(1)),
  

•   GATE_CLK(GCC_AGGRE_USB3_PRIM_AXI_CLK,           0x3908c, BIT(0)),
  

•   GATE_CLK(GCC_CNOC_PCIE_SF_AXI_CLK,              0x52008, BIT(6)),
  

•   GATE_CLK(GCC_DDRSS_GPU_AXI_CLK,                 0x71154, BIT(0)),
  

•   GATE_CLK(GCC_DDRSS_PCIE_SF_QTB_CLK,             0x52000, BIT(19)),
  

•   GATE_CLK(GCC_PCIE_0_AUX_CLK,                    0x52008, BIT(3)),
  

•   GATE_CLK(GCC_PCIE_0_CFG_AHB_CLK,                0x52008, BIT(2)),
  

•   GATE_CLK(GCC_PCIE_0_MSTR_AXI_CLK,               0x52008, BIT(1)),
  

•   GATE_CLK(GCC_PCIE_0_PHY_RCHNG_CLK,              0x52000, BIT(22)),
  

•   GATE_CLK(GCC_PCIE_0_PIPE_CLK,                   0x52008, BIT(4)),
  

•   GATE_CLK(GCC_PCIE_0_SLV_AXI_CLK,                0x52008, BIT(0)),
  

•   GATE_CLK(GCC_PCIE_0_SLV_Q2A_AXI_CLK,            0x52008, BIT(5)),
  

•   GATE_CLK(GCC_PCIE_1_AUX_CLK,                    0x52000, BIT(29)),
  

•   GATE_CLK(GCC_PCIE_1_CFG_AHB_CLK,                0x52000, BIT(28)),
  

•   GATE_CLK(GCC_PCIE_1_MSTR_AXI_CLK,               0x52000, BIT(27)),
  

•   GATE_CLK(GCC_PCIE_1_PHY_AUX_CLK,                0x52000, BIT(24)),
  

•   GATE_CLK(GCC_PCIE_1_PHY_RCHNG_CLK,              0x52000, BIT(23)),
  

•   GATE_CLK(GCC_PCIE_1_PIPE_CLK,                   0x52000, BIT(30)),
  

•   GATE_CLK(GCC_PCIE_1_SLV_AXI_CLK,                0x52000, BIT(26)),
  

•   GATE_CLK(GCC_PCIE_1_SLV_Q2A_AXI_CLK,            0x52000, BIT(25)),
  

•   GATE_CLK(GCC_QUPV3_I2C_CORE_CLK,                0x52008, BIT(8)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S0_CLK,                  0x52008, BIT(10)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S1_CLK,                  0x52008, BIT(11)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S2_CLK,                  0x52008, BIT(12)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S3_CLK,                  0x52008, BIT(13)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S4_CLK,                  0x52008, BIT(14)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S5_CLK,                  0x52008, BIT(15)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S6_CLK,                  0x52008, BIT(16)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S7_CLK,                  0x52008, BIT(17)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S8_CLK,                  0x52010, BIT(14)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S9_CLK,                  0x52010, BIT(15)),
  

•   GATE_CLK(GCC_QUPV3_I2C_S_AHB_CLK,               0x52008, BIT(7)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_CORE_2X_CLK,           0x52008, BIT(18)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_CORE_CLK,              0x52008, BIT(19)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_S0_CLK,                0x52008, BIT(22)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_S1_CLK,                0x52008, BIT(23)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_S2_CLK,                0x52008, BIT(24)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_S3_CLK,                0x52008, BIT(25)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_S4_CLK,                0x52008, BIT(26)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_S5_CLK,                0x52008, BIT(27)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_S6_CLK,                0x52008, BIT(28)),
  

•   GATE_CLK(GCC_QUPV3_WRAP1_S7_CLK,                0x52010, BIT(16)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_CORE_2X_CLK,           0x52010, BIT(3)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_CORE_CLK,              0x52010, BIT(0)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_S0_CLK,                0x52010, BIT(4)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_S1_CLK,                0x52010, BIT(5)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_S2_CLK,                0x52010, BIT(6)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_S3_CLK,                0x52010, BIT(7)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_S4_CLK,                0x52010, BIT(8)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_S5_CLK,                0x52010, BIT(9)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_S6_CLK,                0x52010, BIT(10)),
  

•   GATE_CLK(GCC_QUPV3_WRAP2_S7_CLK,                0x52010, BIT(17)),
  

•   GATE_CLK(GCC_QUPV3_WRAP_1_M_AHB_CLK,            0x52008, BIT(20)),
  

•   GATE_CLK(GCC_QUPV3_WRAP_1_S_AHB_CLK,            0x52008, BIT(21)),
  

•   GATE_CLK(GCC_QUPV3_WRAP_2_M_AHB_CLK,            0x52010, BIT(2)),
  

•   GATE_CLK(GCC_QUPV3_WRAP_2_S_AHB_CLK,            0x52010, BIT(1)),
  

•   GATE_CLK(GCC_SDCC2_AHB_CLK,                     0x14010, BIT(0)),
  

•   GATE_CLK(GCC_SDCC2_APPS_CLK,                    0x14004, BIT(0)),
  

•   GATE_CLK(GCC_UFS_PHY_AHB_CLK,                   0x77024, BIT(0)),
  

•   GATE_CLK(GCC_UFS_PHY_AXI_CLK,                   0x77018, BIT(0)),
  

•   GATE_CLK(GCC_UFS_PHY_AXI_HW_CTL_CLK,            0x77018, BIT(1)),
  

•   GATE_CLK(GCC_UFS_PHY_ICE_CORE_CLK,              0x77074, BIT(0)),
  

•   GATE_CLK(GCC_UFS_PHY_ICE_CORE_HW_CTL_CLK,       0x77074, BIT(1)),
  

•   GATE_CLK(GCC_UFS_PHY_PHY_AUX_CLK,               0x770b0, BIT(0)),
  

•   GATE_CLK(GCC_UFS_PHY_PHY_AUX_HW_CTL_CLK,        0x770b0, BIT(1)),
  

•   GATE_CLK(GCC_UFS_PHY_RX_SYMBOL_0_CLK,           0x7702c, BIT(0)),
  

•   GATE_CLK(GCC_UFS_PHY_RX_SYMBOL_1_CLK,           0x770cc, BIT(0)),
  

•   GATE_CLK(GCC_UFS_PHY_TX_SYMBOL_0_CLK,           0x77028, BIT(0)),
  

•   GATE_CLK(GCC_UFS_PHY_UNIPRO_CORE_CLK,           0x77068, BIT(0)),
  

•   GATE_CLK(GCC_UFS_PHY_UNIPRO_CORE_HW_CTL_CLK,    0x77068, BIT(1)),
  

•   GATE_CLK(GCC_USB30_PRIM_MASTER_CLK,             0x39018, BIT(0)),
  

•   GATE_CLK(GCC_USB30_PRIM_MOCK_UTMI_CLK,          0x39028, BIT(0)),
  

•   GATE_CLK(GCC_USB30_PRIM_SLEEP_CLK,              0x39024, BIT(0)),
  

•   GATE_CLK(GCC_USB3_PRIM_PHY_AUX_CLK,             0x39060, BIT(0)),
  

•   GATE_CLK(GCC_USB3_PRIM_PHY_COM_AUX_CLK,         0x39064, BIT(0)),
  

•   GATE_CLK(GCC_USB3_PRIM_PHY_PIPE_CLK,            0x39068, BIT(0)),
  

+};

Shouldn't this list be trimmed down to supported peripherals only?

It's already the case, I also intend to support PCIe so I added the gates already, it would need some proper support in sm8550_set_rate/sm8550_enable when I'll figure out what's needed.

Is that all you need? Do we have the Qcom PCIe driver in U-Boot to start with? So please don't add unsupported code.

Saying it's unsupported code is quite exagerated, not sure I'm aligned on your position.

Neil

-Sumit