[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a7677638-8108-0bd7-19ca-0cfff5fcf534@canonical.com>
Date: Tue, 18 Jan 2022 17:18:28 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com>
To: Alim Akhtar <alim.akhtar@...sung.com>,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Cc: soc@...nel.org, linux-clk@...r.kernel.org,
devicetree@...r.kernel.org, olof@...om.net, arnd@...db.de,
linus.walleij@...aro.org, catalin.marinas@....com,
robh+dt@...nel.org, s.nawrocki@...sung.com,
linux-samsung-soc@...r.kernel.org, pankaj.dubey@...sung.com,
linux-fsd@...la.com
Subject: Re: [PATCH v2 02/16] dt-bindings: clock: Add bindings definitions for
FSD CMU blocks
On 18/01/2022 15:48, Alim Akhtar wrote:
> Clock controller driver of FSD platform is designed to have separate
> instances for each particular CMU. So clock IDs in this bindings header
> also start from 1 for each CMU block.
>
> Cc: linux-fsd@...la.com
> Reported-by: kernel test robot <lkp@...el.com>
> [robot: reported missing #endif]
> Signed-off-by: Pankaj Dubey <pankaj.dubey@...sung.com>
> Signed-off-by: Alim Akhtar <alim.akhtar@...sung.com>
> ---
> include/dt-bindings/clock/fsd-clk.h | 149 ++++++++++++++++++++++++++++
> 1 file changed, 149 insertions(+)
> create mode 100644 include/dt-bindings/clock/fsd-clk.h
>
> diff --git a/include/dt-bindings/clock/fsd-clk.h b/include/dt-bindings/clock/fsd-clk.h
> new file mode 100644
> index 000000000000..8cf45d2430b0
> --- /dev/null
> +++ b/include/dt-bindings/clock/fsd-clk.h
> @@ -0,0 +1,149 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Copyright (c) 2017 - 2022: Samsung Electronics Co., Ltd.
> + * https://www.samsung.com
> + * Copyright (c) 2017-2022 Tesla, Inc.
> + * https://www.tesla.com
> + *
> + * The constants defined in this header are being used in dts
> + * and fsd platform driver.
> + */
> +
> +#ifndef _DT_BINDINGS_CLOCK_FSD_H
> +#define _DT_BINDINGS_CLOCK_FSD_H
> +
> +/* CMU */
> +#define DOUT_CMU_PLL_SHARED0_DIV4 1
> +#define DOUT_CMU_PERIC_SHARED1DIV36 2
> +#define DOUT_CMU_PERIC_SHARED0DIV3_TBUCLK 3
> +#define DOUT_CMU_PERIC_SHARED0DIV20 4
> +#define DOUT_CMU_PERIC_SHARED1DIV4_DMACLK 5
> +#define DOUT_CMU_PLL_SHARED0_DIV6 6
> +#define DOUT_CMU_FSYS0_SHARED1DIV4 7
> +#define DOUT_CMU_FSYS0_SHARED0DIV4 8
> +#define DOUT_CMU_FSYS1_SHARED0DIV8 9
> +#define DOUT_CMU_FSYS1_SHARED0DIV4 10
> +#define CMU_CPUCL_SWITCH_GATE 11
> +#define DOUT_CMU_IMEM_TCUCLK 12
> +#define DOUT_CMU_IMEM_ACLK 13
> +#define DOUT_CMU_IMEM_DMACLK 14
> +#define GAT_CMU_FSYS0_SHARED0DIV4 15
> +#define CMU_NR_CLK 16
> +
> +/* PERIC */
> +#define PERIC_SCLK_UART0 1
> +#define PERIC_PCLK_UART0 2
> +#define PERIC_SCLK_UART1 3
> +#define PERIC_PCLK_UART1 4
> +#define PERIC_DMA0_IPCLKPORT_ACLK 5
> +#define PERIC_DMA1_IPCLKPORT_ACLK 6
> +#define PERIC_PWM0_IPCLKPORT_I_PCLK_S0 7
> +#define PERIC_PWM1_IPCLKPORT_I_PCLK_S0 8
> +#define PERIC_PCLK_SPI0 9
> +#define PERIC_SCLK_SPI0 10
> +#define PERIC_PCLK_SPI1 11
> +#define PERIC_SCLK_SPI1 12
> +#define PERIC_PCLK_SPI2 13
> +#define PERIC_SCLK_SPI2 14
> +#define PERIC_PCLK_TDM0 15
> +#define PERIC_PCLK_HSI2C0 16
> +#define PERIC_PCLK_HSI2C1 17
> +#define PERIC_PCLK_HSI2C2 18
> +#define PERIC_PCLK_HSI2C3 19
> +#define PERIC_PCLK_HSI2C4 20
> +#define PERIC_PCLK_HSI2C5 21
> +#define PERIC_PCLK_HSI2C6 22
> +#define PERIC_PCLK_HSI2C7 23
> +#define PERIC_MCAN0_IPCLKPORT_CCLK 24
> +#define PERIC_MCAN0_IPCLKPORT_PCLK 25
> +#define PERIC_MCAN1_IPCLKPORT_CCLK 26
> +#define PERIC_MCAN1_IPCLKPORT_PCLK 27
> +#define PERIC_MCAN2_IPCLKPORT_CCLK 28
> +#define PERIC_MCAN2_IPCLKPORT_PCLK 29
> +#define PERIC_MCAN3_IPCLKPORT_CCLK 30
> +#define PERIC_MCAN3_IPCLKPORT_PCLK 31
> +#define PERIC_PCLK_ADCIF 32
> +#define PERIC_EQOS_TOP_IPCLKPORT_CLK_PTP_REF_I 33
> +#define PERIC_EQOS_TOP_IPCLKPORT_ACLK_I 34
> +#define PERIC_EQOS_TOP_IPCLKPORT_HCLK_I 35
> +#define PERIC_EQOS_TOP_IPCLKPORT_RGMII_CLK_I 36
> +#define PERIC_EQOS_TOP_IPCLKPORT_CLK_RX_I 37
> +#define PERIC_BUS_D_PERIC_IPCLKPORT_EQOSCLK 38
> +#define PERIC_BUS_P_PERIC_IPCLKPORT_EQOSCLK 39
> +#define PERIC_HCLK_TDM0 40
> +#define PERIC_PCLK_TDM1 41
> +#define PERIC_HCLK_TDM1 42
> +#define PERIC_EQOS_PHYRXCLK_MUX 43
> +#define PERIC_EQOS_PHYRXCLK 44
> +#define PERIC_DOUT_RGMII_CLK 45
> +#define PERIC_NR_CLK 46
> +
> +/* FSYS0 */
> +#define UFS0_MPHY_REFCLK_IXTAL24 1
> +#define UFS0_MPHY_REFCLK_IXTAL26 2
> +#define UFS1_MPHY_REFCLK_IXTAL24 3
> +#define UFS1_MPHY_REFCLK_IXTAL26 4
> +#define UFS0_TOP0_HCLK_BUS 5
> +#define UFS0_TOP0_ACLK 6
> +#define UFS0_TOP0_CLK_UNIPRO 7
> +#define UFS0_TOP0_FMP_CLK 8
> +#define UFS1_TOP1_HCLK_BUS 9
> +#define UFS1_TOP1_ACLK 10
> +#define UFS1_TOP1_CLK_UNIPRO 11
> +#define UFS1_TOP1_FMP_CLK 12
> +#define PCIE_SUBCTRL_INST0_DBI_ACLK_SOC 13
> +#define PCIE_SUBCTRL_INST0_AUX_CLK_SOC 14
> +#define PCIE_SUBCTRL_INST0_MSTR_ACLK_SOC 15
> +#define PCIE_SUBCTRL_INST0_SLV_ACLK_SOC 16
> +#define FSYS0_EQOS_TOP0_IPCLKPORT_CLK_PTP_REF_I 17
> +#define FSYS0_EQOS_TOP0_IPCLKPORT_ACLK_I 18
> +#define FSYS0_EQOS_TOP0_IPCLKPORT_HCLK_I 19
> +#define FSYS0_EQOS_TOP0_IPCLKPORT_RGMII_CLK_I 20
> +#define FSYS0_EQOS_TOP0_IPCLKPORT_CLK_RX_I 21
> +#define FSYS0_DOUT_FSYS0_PERIBUS_GRP 22
> +#define FSYS0_NR_CLK 23
> +
> +/* FSYS1 */
> +#define PCIE_LINK0_IPCLKPORT_DBI_ACLK 1
> +#define PCIE_LINK0_IPCLKPORT_AUX_ACLK 2
> +#define PCIE_LINK0_IPCLKPORT_MSTR_ACLK 3
> +#define PCIE_LINK0_IPCLKPORT_SLV_ACLK 4
> +#define PCIE_LINK1_IPCLKPORT_DBI_ACLK 5
> +#define PCIE_LINK1_IPCLKPORT_AUX_ACLK 6
> +#define PCIE_LINK1_IPCLKPORT_MSTR_ACLK 7
> +#define PCIE_LINK1_IPCLKPORT_SLV_ACLK 8
> +#define FSYS1_NR_CLK 9
> +
> +/* IMEM */
> +#define IMEM_DMA0_IPCLKPORT_ACLK 1
> +#define IMEM_DMA1_IPCLKPORT_ACLK 2
> +#define IMEM_WDT0_IPCLKPORT_PCLK 3
> +#define IMEM_WDT1_IPCLKPORT_PCLK 4
> +#define IMEM_WDT2_IPCLKPORT_PCLK 5
> +#define IMEM_MCT_PCLK 6
> +#define IMEM_TMU_CPU0_IPCLKPORT_I_CLK_TS 7
> +#define IMEM_TMU_CPU2_IPCLKPORT_I_CLK_TS 8
> +#define IMEM_TMU_TOP_IPCLKPORT_I_CLK_TS 9
> +#define IMEM_TMU_GPU_IPCLKPORT_I_CLK_TS 10
> +#define IMEM_TMU_GT_IPCLKPORT_I_CLK_TS 11
> +#define IMEM_NR_CLK 12
> +
> +/* MFC */
> +#define MFC_MFC_IPCLKPORT_ACLK 1
> +#define MFC_NR_CLK 2
> +
> +/* CAM_CSI */
> +#define CAM_CSI0_0_IPCLKPORT_I_ACLK 1
> +#define CAM_CSI0_1_IPCLKPORT_I_ACLK 2
> +#define CAM_CSI0_2_IPCLKPORT_I_ACLK 3
> +#define CAM_CSI0_3_IPCLKPORT_I_ACLK 4
> +#define CAM_CSI1_0_IPCLKPORT_I_ACLK 5
> +#define CAM_CSI1_1_IPCLKPORT_I_ACLK 6
> +#define CAM_CSI1_2_IPCLKPORT_I_ACLK 7
> +#define CAM_CSI1_3_IPCLKPORT_I_ACLK 8
> +#define CAM_CSI2_0_IPCLKPORT_I_ACLK 9
> +#define CAM_CSI2_1_IPCLKPORT_I_ACLK 10
> +#define CAM_CSI2_2_IPCLKPORT_I_ACLK 11
> +#define CAM_CSI2_3_IPCLKPORT_I_ACLK 12
> +#define CAM_CSI_NR_CLK 13
One empty line would be nice here.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com>
> +#endif /*_DT_BINDINGS_CLOCK_FSD_H */
>
Best regards,
Krzysztof
Powered by blists - more mailing lists