| Message ID | 20240404-topic-sm8x50-clock-v1-1-0c8ea7fad003@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | qcom: add clock driver support for SM8550 and SM8650 SoCc | expand |
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? > + > +static int sm8550_enable(struct clk *clk) > +{ > + struct msm_clk_priv *priv = dev_get_priv(clk->dev); > + > + switch (clk->id) { > + case GCC_AGGRE_USB3_PRIM_AXI_CLK: > + qcom_gate_clk_en(priv, GCC_USB30_PRIM_MASTER_CLK); > + fallthrough; > + case GCC_USB30_PRIM_MASTER_CLK: > + qcom_gate_clk_en(priv, GCC_USB3_PRIM_PHY_AUX_CLK); > + qcom_gate_clk_en(priv, GCC_USB3_PRIM_PHY_COM_AUX_CLK); > + break; > + } > + > + qcom_gate_clk_en(priv, clk->id); > + > + return 0; > +} > + > +static const struct qcom_reset_map sm8550_gcc_resets[] = { > + [GCC_CAMERA_BCR] = { 0x26000 }, > + [GCC_DISPLAY_BCR] = { 0x27000 }, > + [GCC_GPU_BCR] = { 0x71000 }, > + [GCC_PCIE_0_BCR] = { 0x6b000 }, > + [GCC_PCIE_0_LINK_DOWN_BCR] = { 0x6c014 }, > + [GCC_PCIE_0_NOCSR_COM_PHY_BCR] = { 0x6c020 }, > + [GCC_PCIE_0_PHY_BCR] = { 0x6c01c }, > + [GCC_PCIE_0_PHY_NOCSR_COM_PHY_BCR] = { 0x6c028 }, > + [GCC_PCIE_1_BCR] = { 0x8d000 }, > + [GCC_PCIE_1_LINK_DOWN_BCR] = { 0x8e014 }, > + [GCC_PCIE_1_NOCSR_COM_PHY_BCR] = { 0x8e020 }, > + [GCC_PCIE_1_PHY_BCR] = { 0x8e01c }, > + [GCC_PCIE_1_PHY_NOCSR_COM_PHY_BCR] = { 0x8e024 }, > + [GCC_PCIE_PHY_BCR] = { 0x6f000 }, > + [GCC_PCIE_PHY_CFG_AHB_BCR] = { 0x6f00c }, > + [GCC_PCIE_PHY_COM_BCR] = { 0x6f010 }, > + [GCC_PDM_BCR] = { 0x33000 }, > + [GCC_QUPV3_WRAPPER_1_BCR] = { 0x18000 }, > + [GCC_QUPV3_WRAPPER_2_BCR] = { 0x1e000 }, > + [GCC_QUPV3_WRAPPER_I2C_BCR] = { 0x17000 }, > + [GCC_QUSB2PHY_PRIM_BCR] = { 0x12000 }, > + [GCC_QUSB2PHY_SEC_BCR] = { 0x12004 }, > + [GCC_SDCC2_BCR] = { 0x14000 }, > + [GCC_SDCC4_BCR] = { 0x16000 }, > + [GCC_UFS_PHY_BCR] = { 0x77000 }, > + [GCC_USB30_PRIM_BCR] = { 0x39000 }, > + [GCC_USB3_DP_PHY_PRIM_BCR] = { 0x50008 }, > + [GCC_USB3_DP_PHY_SEC_BCR] = { 0x50014 }, > + [GCC_USB3_PHY_PRIM_BCR] = { 0x50000 }, > + [GCC_USB3_PHY_SEC_BCR] = { 0x5000c }, > + [GCC_USB3PHY_PHY_PRIM_BCR] = { 0x50004 }, > + [GCC_USB3PHY_PHY_SEC_BCR] = { 0x50010 }, > + [GCC_USB_PHY_CFG_AHB2PHY_BCR] = { 0x6a000 }, > + [GCC_VIDEO_AXI0_CLK_ARES] = { 0x32018, 2 }, > + [GCC_VIDEO_AXI1_CLK_ARES] = { 0x32024, 2 }, > + [GCC_VIDEO_BCR] = { 0x32000 }, > +}; Ditto. > + > +static const struct qcom_power_map sm8550_gdscs[] = { > + [PCIE_0_GDSC] = { 0x6b004 }, > + [PCIE_0_PHY_GDSC] = { 0x6c000 }, > + [PCIE_1_GDSC] = { 0x8d004 }, > + [PCIE_1_PHY_GDSC] = { 0x8e000 }, > + [UFS_PHY_GDSC] = { 0x77004 }, > + [UFS_MEM_PHY_GDSC] = { 0x9e000 }, > + [USB30_PRIM_GDSC] = { 0x39004 }, > + [USB3_PHY_GDSC] = { 0x50018 }, > +}; Ditto. -Sumit > + > +static struct msm_clk_data sm8550_gcc_data = { > + .resets = sm8550_gcc_resets, > + .num_resets = ARRAY_SIZE(sm8550_gcc_resets), > + .clks = sm8550_clks, > + .num_clks = ARRAY_SIZE(sm8550_clks), > + .power_domains = sm8550_gdscs, > + .num_power_domains = ARRAY_SIZE(sm8550_gdscs), > + > + .enable = sm8550_enable, > + .set_rate = sm8550_set_rate, > +}; > + > +static const struct udevice_id gcc_sm8550_of_match[] = { > + { > + .compatible = "qcom,sm8550-gcc", > + .data = (ulong)&sm8550_gcc_data, > + }, > + { } > +}; > + > +U_BOOT_DRIVER(gcc_sm8550) = { > + .name = "gcc_sm8550", > + .id = UCLASS_NOP, > + .of_match = gcc_sm8550_of_match, > + .bind = qcom_cc_bind, > + .flags = DM_FLAG_PRE_RELOC | DM_FLAG_DEFAULT_PD_CTRL_OFF, > +}; > + > +/* TCSRCC */ > + > +static const struct gate_clk sm8550_tcsr_clks[] = { > + GATE_CLK(TCSR_PCIE_0_CLKREF_EN, 0x15100, BIT(0)), > + GATE_CLK(TCSR_PCIE_1_CLKREF_EN, 0x15114, BIT(0)), > + GATE_CLK(TCSR_UFS_CLKREF_EN, 0x15110, BIT(0)), > + GATE_CLK(TCSR_UFS_PAD_CLKREF_EN, 0x15104, BIT(0)), > + GATE_CLK(TCSR_USB2_CLKREF_EN, 0x15118, BIT(0)), > + GATE_CLK(TCSR_USB3_CLKREF_EN, 0x15108, BIT(0)), > +}; > + > +static struct msm_clk_data sm8550_tcsrcc_data = { > + .clks = sm8550_tcsr_clks, > + .num_clks = ARRAY_SIZE(sm8550_tcsr_clks), > +}; > + > +static int tcsrcc_sm8550_clk_enable(struct clk *clk) > +{ > + struct msm_clk_priv *priv = dev_get_priv(clk->dev); > + > + qcom_gate_clk_en(priv, clk->id); > + > + return 0; > +} > + > +static ulong tcsrcc_sm8550_clk_get_rate(struct clk *clk) > +{ > + return TCXO_RATE; > +} > + > +static int tcsrcc_sm8550_clk_probe(struct udevice *dev) > +{ > + struct msm_clk_data *data = (struct msm_clk_data *)dev_get_driver_data(dev); > + struct msm_clk_priv *priv = dev_get_priv(dev); > + > + priv->base = dev_read_addr(dev); > + if (priv->base == FDT_ADDR_T_NONE) > + return -EINVAL; > + > + priv->data = data; > + > + return 0; > +} > + > +static struct clk_ops tcsrcc_sm8550_clk_ops = { > + .enable = tcsrcc_sm8550_clk_enable, > + .get_rate = tcsrcc_sm8550_clk_get_rate, > +}; > + > +static const struct udevice_id tcsrcc_sm8550_of_match[] = { > + { > + .compatible = "qcom,sm8550-tcsr", > + .data = (ulong)&sm8550_tcsrcc_data, > + }, > + { } > +}; > + > +U_BOOT_DRIVER(tcsrcc_sm8550) = { > + .name = "tcsrcc_sm8550", > + .id = UCLASS_CLK, > + .of_match = tcsrcc_sm8550_of_match, > + .ops = &tcsrcc_sm8550_clk_ops, > + .priv_auto = sizeof(struct msm_clk_priv), > + .probe = tcsrcc_sm8550_clk_probe, > + .flags = DM_FLAG_PRE_RELOC | DM_FLAG_DEFAULT_PD_CTRL_OFF, > +}; > > -- > 2.34.1 >
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. > >> + >> +static int sm8550_enable(struct clk *clk) >> +{ >> + struct msm_clk_priv *priv = dev_get_priv(clk->dev); >> + >> + switch (clk->id) { >> + case GCC_AGGRE_USB3_PRIM_AXI_CLK: >> + qcom_gate_clk_en(priv, GCC_USB30_PRIM_MASTER_CLK); >> + fallthrough; >> + case GCC_USB30_PRIM_MASTER_CLK: >> + qcom_gate_clk_en(priv, GCC_USB3_PRIM_PHY_AUX_CLK); >> + qcom_gate_clk_en(priv, GCC_USB3_PRIM_PHY_COM_AUX_CLK); >> + break; >> + } >> + >> + qcom_gate_clk_en(priv, clk->id); >> + >> + return 0; >> +} >> + >> +static const struct qcom_reset_map sm8550_gcc_resets[] = { >> + [GCC_CAMERA_BCR] = { 0x26000 }, >> + [GCC_DISPLAY_BCR] = { 0x27000 }, >> + [GCC_GPU_BCR] = { 0x71000 }, >> + [GCC_PCIE_0_BCR] = { 0x6b000 }, >> + [GCC_PCIE_0_LINK_DOWN_BCR] = { 0x6c014 }, >> + [GCC_PCIE_0_NOCSR_COM_PHY_BCR] = { 0x6c020 }, >> + [GCC_PCIE_0_PHY_BCR] = { 0x6c01c }, >> + [GCC_PCIE_0_PHY_NOCSR_COM_PHY_BCR] = { 0x6c028 }, >> + [GCC_PCIE_1_BCR] = { 0x8d000 }, >> + [GCC_PCIE_1_LINK_DOWN_BCR] = { 0x8e014 }, >> + [GCC_PCIE_1_NOCSR_COM_PHY_BCR] = { 0x8e020 }, >> + [GCC_PCIE_1_PHY_BCR] = { 0x8e01c }, >> + [GCC_PCIE_1_PHY_NOCSR_COM_PHY_BCR] = { 0x8e024 }, >> + [GCC_PCIE_PHY_BCR] = { 0x6f000 }, >> + [GCC_PCIE_PHY_CFG_AHB_BCR] = { 0x6f00c }, >> + [GCC_PCIE_PHY_COM_BCR] = { 0x6f010 }, >> + [GCC_PDM_BCR] = { 0x33000 }, >> + [GCC_QUPV3_WRAPPER_1_BCR] = { 0x18000 }, >> + [GCC_QUPV3_WRAPPER_2_BCR] = { 0x1e000 }, >> + [GCC_QUPV3_WRAPPER_I2C_BCR] = { 0x17000 }, >> + [GCC_QUSB2PHY_PRIM_BCR] = { 0x12000 }, >> + [GCC_QUSB2PHY_SEC_BCR] = { 0x12004 }, >> + [GCC_SDCC2_BCR] = { 0x14000 }, >> + [GCC_SDCC4_BCR] = { 0x16000 }, >> + [GCC_UFS_PHY_BCR] = { 0x77000 }, >> + [GCC_USB30_PRIM_BCR] = { 0x39000 }, >> + [GCC_USB3_DP_PHY_PRIM_BCR] = { 0x50008 }, >> + [GCC_USB3_DP_PHY_SEC_BCR] = { 0x50014 }, >> + [GCC_USB3_PHY_PRIM_BCR] = { 0x50000 }, >> + [GCC_USB3_PHY_SEC_BCR] = { 0x5000c }, >> + [GCC_USB3PHY_PHY_PRIM_BCR] = { 0x50004 }, >> + [GCC_USB3PHY_PHY_SEC_BCR] = { 0x50010 }, >> + [GCC_USB_PHY_CFG_AHB2PHY_BCR] = { 0x6a000 }, >> + [GCC_VIDEO_AXI0_CLK_ARES] = { 0x32018, 2 }, >> + [GCC_VIDEO_AXI1_CLK_ARES] = { 0x32024, 2 }, >> + [GCC_VIDEO_BCR] = { 0x32000 }, >> +}; > > Ditto. > >> + >> +static const struct qcom_power_map sm8550_gdscs[] = { >> + [PCIE_0_GDSC] = { 0x6b004 }, >> + [PCIE_0_PHY_GDSC] = { 0x6c000 }, >> + [PCIE_1_GDSC] = { 0x8d004 }, >> + [PCIE_1_PHY_GDSC] = { 0x8e000 }, >> + [UFS_PHY_GDSC] = { 0x77004 }, >> + [UFS_MEM_PHY_GDSC] = { 0x9e000 }, >> + [USB30_PRIM_GDSC] = { 0x39004 }, >> + [USB3_PHY_GDSC] = { 0x50018 }, >> +}; No need to trim > > Ditto. > > -Sumit > >> + >> +static struct msm_clk_data sm8550_gcc_data = { >> + .resets = sm8550_gcc_resets, >> + .num_resets = ARRAY_SIZE(sm8550_gcc_resets), >> + .clks = sm8550_clks, >> + .num_clks = ARRAY_SIZE(sm8550_clks), >> + .power_domains = sm8550_gdscs, >> + .num_power_domains = ARRAY_SIZE(sm8550_gdscs), >> + >> + .enable = sm8550_enable, >> + .set_rate = sm8550_set_rate, >> +}; >> + >> +static const struct udevice_id gcc_sm8550_of_match[] = { >> + { >> + .compatible = "qcom,sm8550-gcc", >> + .data = (ulong)&sm8550_gcc_data, >> + }, >> + { } >> +}; >> + >> +U_BOOT_DRIVER(gcc_sm8550) = { >> + .name = "gcc_sm8550", >> + .id = UCLASS_NOP, >> + .of_match = gcc_sm8550_of_match, >> + .bind = qcom_cc_bind, >> + .flags = DM_FLAG_PRE_RELOC | DM_FLAG_DEFAULT_PD_CTRL_OFF, >> +}; >> + >> +/* TCSRCC */ >> + >> +static const struct gate_clk sm8550_tcsr_clks[] = { >> + GATE_CLK(TCSR_PCIE_0_CLKREF_EN, 0x15100, BIT(0)), >> + GATE_CLK(TCSR_PCIE_1_CLKREF_EN, 0x15114, BIT(0)), >> + GATE_CLK(TCSR_UFS_CLKREF_EN, 0x15110, BIT(0)), >> + GATE_CLK(TCSR_UFS_PAD_CLKREF_EN, 0x15104, BIT(0)), >> + GATE_CLK(TCSR_USB2_CLKREF_EN, 0x15118, BIT(0)), >> + GATE_CLK(TCSR_USB3_CLKREF_EN, 0x15108, BIT(0)), >> +}; >> + >> +static struct msm_clk_data sm8550_tcsrcc_data = { >> + .clks = sm8550_tcsr_clks, >> + .num_clks = ARRAY_SIZE(sm8550_tcsr_clks), >> +}; >> + >> +static int tcsrcc_sm8550_clk_enable(struct clk *clk) >> +{ >> + struct msm_clk_priv *priv = dev_get_priv(clk->dev); >> + >> + qcom_gate_clk_en(priv, clk->id); >> + >> + return 0; >> +} >> + >> +static ulong tcsrcc_sm8550_clk_get_rate(struct clk *clk) >> +{ >> + return TCXO_RATE; >> +} >> + >> +static int tcsrcc_sm8550_clk_probe(struct udevice *dev) >> +{ >> + struct msm_clk_data *data = (struct msm_clk_data *)dev_get_driver_data(dev); >> + struct msm_clk_priv *priv = dev_get_priv(dev); >> + >> + priv->base = dev_read_addr(dev); >> + if (priv->base == FDT_ADDR_T_NONE) >> + return -EINVAL; >> + >> + priv->data = data; >> + >> + return 0; >> +} >> + >> +static struct clk_ops tcsrcc_sm8550_clk_ops = { >> + .enable = tcsrcc_sm8550_clk_enable, >> + .get_rate = tcsrcc_sm8550_clk_get_rate, >> +}; >> + >> +static const struct udevice_id tcsrcc_sm8550_of_match[] = { >> + { >> + .compatible = "qcom,sm8550-tcsr", >> + .data = (ulong)&sm8550_tcsrcc_data, >> + }, >> + { } >> +}; >> + >> +U_BOOT_DRIVER(tcsrcc_sm8550) = { >> + .name = "tcsrcc_sm8550", >> + .id = UCLASS_CLK, >> + .of_match = tcsrcc_sm8550_of_match, >> + .ops = &tcsrcc_sm8550_clk_ops, >> + .priv_auto = sizeof(struct msm_clk_priv), >> + .probe = tcsrcc_sm8550_clk_probe, >> + .flags = DM_FLAG_PRE_RELOC | DM_FLAG_DEFAULT_PD_CTRL_OFF, >> +}; >> >> -- >> 2.34.1 >>
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. -Sumit
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
On Wed, 10 Apr 2024 at 15:00, <neil.armstrong@linaro.org> wrote: > > 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. > Let me try to rephrase it. Can we test these GDSCs, resets and gates without actual driver support in U-Boot? -Sumit
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)), +}; + +static int sm8550_enable(struct clk *clk) +{ + struct msm_clk_priv *priv = dev_get_priv(clk->dev); + + switch (clk->id) { + case GCC_AGGRE_USB3_PRIM_AXI_CLK: + qcom_gate_clk_en(priv, GCC_USB30_PRIM_MASTER_CLK); + fallthrough; + case GCC_USB30_PRIM_MASTER_CLK: + qcom_gate_clk_en(priv, GCC_USB3_PRIM_PHY_AUX_CLK); + qcom_gate_clk_en(priv, GCC_USB3_PRIM_PHY_COM_AUX_CLK); + break; + } + + qcom_gate_clk_en(priv, clk->id); + + return 0; +} + +static const struct qcom_reset_map sm8550_gcc_resets[] = { + [GCC_CAMERA_BCR] = { 0x26000 }, + [GCC_DISPLAY_BCR] = { 0x27000 }, + [GCC_GPU_BCR] = { 0x71000 }, + [GCC_PCIE_0_BCR] = { 0x6b000 }, + [GCC_PCIE_0_LINK_DOWN_BCR] = { 0x6c014 }, + [GCC_PCIE_0_NOCSR_COM_PHY_BCR] = { 0x6c020 }, + [GCC_PCIE_0_PHY_BCR] = { 0x6c01c }, + [GCC_PCIE_0_PHY_NOCSR_COM_PHY_BCR] = { 0x6c028 }, + [GCC_PCIE_1_BCR] = { 0x8d000 }, + [GCC_PCIE_1_LINK_DOWN_BCR] = { 0x8e014 }, + [GCC_PCIE_1_NOCSR_COM_PHY_BCR] = { 0x8e020 }, + [GCC_PCIE_1_PHY_BCR] = { 0x8e01c }, + [GCC_PCIE_1_PHY_NOCSR_COM_PHY_BCR] = { 0x8e024 }, + [GCC_PCIE_PHY_BCR] = { 0x6f000 }, + [GCC_PCIE_PHY_CFG_AHB_BCR] = { 0x6f00c }, + [GCC_PCIE_PHY_COM_BCR] = { 0x6f010 }, + [GCC_PDM_BCR] = { 0x33000 }, + [GCC_QUPV3_WRAPPER_1_BCR] = { 0x18000 }, + [GCC_QUPV3_WRAPPER_2_BCR] = { 0x1e000 }, + [GCC_QUPV3_WRAPPER_I2C_BCR] = { 0x17000 }, + [GCC_QUSB2PHY_PRIM_BCR] = { 0x12000 }, + [GCC_QUSB2PHY_SEC_BCR] = { 0x12004 }, + [GCC_SDCC2_BCR] = { 0x14000 }, + [GCC_SDCC4_BCR] = { 0x16000 }, + [GCC_UFS_PHY_BCR] = { 0x77000 }, + [GCC_USB30_PRIM_BCR] = { 0x39000 }, + [GCC_USB3_DP_PHY_PRIM_BCR] = { 0x50008 }, + [GCC_USB3_DP_PHY_SEC_BCR] = { 0x50014 }, + [GCC_USB3_PHY_PRIM_BCR] = { 0x50000 }, + [GCC_USB3_PHY_SEC_BCR] = { 0x5000c }, + [GCC_USB3PHY_PHY_PRIM_BCR] = { 0x50004 }, + [GCC_USB3PHY_PHY_SEC_BCR] = { 0x50010 }, + [GCC_USB_PHY_CFG_AHB2PHY_BCR] = { 0x6a000 }, + [GCC_VIDEO_AXI0_CLK_ARES] = { 0x32018, 2 }, + [GCC_VIDEO_AXI1_CLK_ARES] = { 0x32024, 2 }, + [GCC_VIDEO_BCR] = { 0x32000 }, +}; + +static const struct qcom_power_map sm8550_gdscs[] = { + [PCIE_0_GDSC] = { 0x6b004 }, + [PCIE_0_PHY_GDSC] = { 0x6c000 }, + [PCIE_1_GDSC] = { 0x8d004 }, + [PCIE_1_PHY_GDSC] = { 0x8e000 }, + [UFS_PHY_GDSC] = { 0x77004 }, + [UFS_MEM_PHY_GDSC] = { 0x9e000 }, + [USB30_PRIM_GDSC] = { 0x39004 }, + [USB3_PHY_GDSC] = { 0x50018 }, +}; + +static struct msm_clk_data sm8550_gcc_data = { + .resets = sm8550_gcc_resets, + .num_resets = ARRAY_SIZE(sm8550_gcc_resets), + .clks = sm8550_clks, + .num_clks = ARRAY_SIZE(sm8550_clks), + .power_domains = sm8550_gdscs, + .num_power_domains = ARRAY_SIZE(sm8550_gdscs), + + .enable = sm8550_enable, + .set_rate = sm8550_set_rate, +}; + +static const struct udevice_id gcc_sm8550_of_match[] = { + { + .compatible = "qcom,sm8550-gcc", + .data = (ulong)&sm8550_gcc_data, + }, + { } +}; + +U_BOOT_DRIVER(gcc_sm8550) = { + .name = "gcc_sm8550", + .id = UCLASS_NOP, + .of_match = gcc_sm8550_of_match, + .bind = qcom_cc_bind, + .flags = DM_FLAG_PRE_RELOC | DM_FLAG_DEFAULT_PD_CTRL_OFF, +}; + +/* TCSRCC */ + +static const struct gate_clk sm8550_tcsr_clks[] = { + GATE_CLK(TCSR_PCIE_0_CLKREF_EN, 0x15100, BIT(0)), + GATE_CLK(TCSR_PCIE_1_CLKREF_EN, 0x15114, BIT(0)), + GATE_CLK(TCSR_UFS_CLKREF_EN, 0x15110, BIT(0)), + GATE_CLK(TCSR_UFS_PAD_CLKREF_EN, 0x15104, BIT(0)), + GATE_CLK(TCSR_USB2_CLKREF_EN, 0x15118, BIT(0)), + GATE_CLK(TCSR_USB3_CLKREF_EN, 0x15108, BIT(0)), +}; + +static struct msm_clk_data sm8550_tcsrcc_data = { + .clks = sm8550_tcsr_clks, + .num_clks = ARRAY_SIZE(sm8550_tcsr_clks), +}; + +static int tcsrcc_sm8550_clk_enable(struct clk *clk) +{ + struct msm_clk_priv *priv = dev_get_priv(clk->dev); + + qcom_gate_clk_en(priv, clk->id); + + return 0; +} + +static ulong tcsrcc_sm8550_clk_get_rate(struct clk *clk) +{ + return TCXO_RATE; +} + +static int tcsrcc_sm8550_clk_probe(struct udevice *dev) +{ + struct msm_clk_data *data = (struct msm_clk_data *)dev_get_driver_data(dev); + struct msm_clk_priv *priv = dev_get_priv(dev); + + priv->base = dev_read_addr(dev); + if (priv->base == FDT_ADDR_T_NONE) + return -EINVAL; + + priv->data = data; + + return 0; +} + +static struct clk_ops tcsrcc_sm8550_clk_ops = { + .enable = tcsrcc_sm8550_clk_enable, + .get_rate = tcsrcc_sm8550_clk_get_rate, +}; + +static const struct udevice_id tcsrcc_sm8550_of_match[] = { + { + .compatible = "qcom,sm8550-tcsr", + .data = (ulong)&sm8550_tcsrcc_data, + }, + { } +}; + +U_BOOT_DRIVER(tcsrcc_sm8550) = { + .name = "tcsrcc_sm8550", + .id = UCLASS_CLK, + .of_match = tcsrcc_sm8550_of_match, + .ops = &tcsrcc_sm8550_clk_ops, + .priv_auto = sizeof(struct msm_clk_priv), + .probe = tcsrcc_sm8550_clk_probe, + .flags = DM_FLAG_PRE_RELOC | DM_FLAG_DEFAULT_PD_CTRL_OFF, +};
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. 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(+)