[U-Boot] [PATCH v2 1/3] arm: bcm281xx: Add Ethernet Clock support

Steve Rae srae at broadcom.com
Thu Aug 28 20:09:28 CEST 2014



On 14-08-01 08:37 PM, Steve Rae wrote:
> From: Jiandong Zheng <jdzheng at broadcom.com>
>
> Enable Ethernet clock when Broadcom StarFighter2 Ethernet block
> (CONFIG_BCM_SF2_ETH) is enabled.
>
> Signed-off-by: Jiandong Zheng <jdzheng at broadcom.com>
> Signed-off-by: Steve Rae <srae at broadcom.com>
> ---
> - with the changes to boards.cfg, the board with this feature will be added separately
>
> Changes in v2:
> - removed 'boards.cfg' from this commit
> - (no other changes)
>
>   arch/arm/cpu/armv7/bcm281xx/Makefile        |   1 +
>   arch/arm/cpu/armv7/bcm281xx/clk-bcm281xx.c  |  34 +++++++
>   arch/arm/cpu/armv7/bcm281xx/clk-eth.c       | 143 ++++++++++++++++++++++++++++
>   arch/arm/include/asm/arch-bcm281xx/sysmap.h |   3 +
>   4 files changed, 181 insertions(+)
>   create mode 100644 arch/arm/cpu/armv7/bcm281xx/clk-eth.c
>
> diff --git a/arch/arm/cpu/armv7/bcm281xx/Makefile b/arch/arm/cpu/armv7/bcm281xx/Makefile
> index 98f5aa5..bd867a2 100644
> --- a/arch/arm/cpu/armv7/bcm281xx/Makefile
> +++ b/arch/arm/cpu/armv7/bcm281xx/Makefile
> @@ -9,3 +9,4 @@ obj-y	+= clk-core.o
>   obj-y	+= clk-bcm281xx.o
>   obj-y	+= clk-sdio.o
>   obj-y	+= clk-bsc.o
> +obj-$(CONFIG_BCM_SF2_ETH) += clk-eth.o
> diff --git a/arch/arm/cpu/armv7/bcm281xx/clk-bcm281xx.c b/arch/arm/cpu/armv7/bcm281xx/clk-bcm281xx.c
> index bc8a170..d16b99f 100644
> --- a/arch/arm/cpu/armv7/bcm281xx/clk-bcm281xx.c
> +++ b/arch/arm/cpu/armv7/bcm281xx/clk-bcm281xx.c
> @@ -118,6 +118,16 @@ unsigned long slave_apb_freq_tbl[8] = {
>   	78 * CLOCK_1M
>   };
>
> +unsigned long esub_freq_tbl[8] = {
> +	78 * CLOCK_1M,
> +	156 * CLOCK_1M,
> +	156 * CLOCK_1M,
> +	156 * CLOCK_1M,
> +	208 * CLOCK_1M,
> +	208 * CLOCK_1M,
> +	208 * CLOCK_1M
> +};
> +
>   static struct bus_clk_data bsc1_apb_data = {
>   	.gate = HW_SW_GATE_AUTO(0x0458, 16, 0, 1),
>   };
> @@ -295,6 +305,27 @@ static struct ccu_clock kps_ccu_clk = {
>   	.freq_tbl = slave_axi_freq_tbl,
>   };
>
> +#ifdef CONFIG_BCM_SF2_ETH
> +static struct ccu_clock esub_ccu_clk = {
> +	.clk = {
> +		.name = "esub_ccu_clk",
> +		.ops = &ccu_clk_ops,
> +		.ccu_clk_mgr_base = ESUB_CLK_BASE_ADDR,
> +	},
> +	.num_policy_masks = 1,
> +	.policy_freq_offset = 0x00000008,
> +	.freq_bit_shift = 8,
> +	.policy_ctl_offset = 0x0000000c,
> +	.policy0_mask_offset = 0x00000010,
> +	.policy1_mask_offset = 0x00000014,
> +	.policy2_mask_offset = 0x00000018,
> +	.policy3_mask_offset = 0x0000001c,
> +	.lvm_en_offset = 0x00000034,
> +	.freq_id = 2,
> +	.freq_tbl = esub_freq_tbl,
> +};
> +#endif
> +
>   /*
>    * Bus clocks
>    */
> @@ -517,6 +548,9 @@ struct clk_lookup arch_clk_tbl[] = {
>   	CLK_LK(bsc1_apb),
>   	CLK_LK(bsc2_apb),
>   	CLK_LK(bsc3_apb),
> +#ifdef CONFIG_BCM_SF2_ETH
> +	CLK_LK(esub_ccu),
> +#endif
>   };
>
>   /* public array size */
> diff --git a/arch/arm/cpu/armv7/bcm281xx/clk-eth.c b/arch/arm/cpu/armv7/bcm281xx/clk-eth.c
> new file mode 100644
> index 0000000..b0b92b9
> --- /dev/null
> +++ b/arch/arm/cpu/armv7/bcm281xx/clk-eth.c
> @@ -0,0 +1,143 @@
> +/*
> + * Copyright 2014 Broadcom Corporation.
> + *
> + * SPDX-License-Identifier:	GPL-2.0+
> + */
> +
> +#include <common.h>
> +#include <asm/io.h>
> +#include <asm/errno.h>
> +#include <asm/arch/sysmap.h>
> +#include <asm/kona-common/clk.h>
> +#include "clk-core.h"
> +
> +#define WR_ACCESS_ADDR			ESUB_CLK_BASE_ADDR
> +#define WR_ACCESS_PASSWORD				0xA5A500
> +
> +#define PLLE_POST_RESETB_ADDR		(ESUB_CLK_BASE_ADDR + 0x00000C00)
> +
> +#define PLLE_RESETB_ADDR		(ESUB_CLK_BASE_ADDR + 0x00000C58)
> +#define PLLE_RESETB_I_PLL_RESETB_PLLE_MASK		0x00010000
> +#define PLLE_POST_RESETB_I_POST_RESETB_PLLE_MASK	0x00000001
> +
> +#define PLL_LOCK_ADDR			(ESUB_CLK_BASE_ADDR + 0x00000C38)
> +#define PLL_LOCK_PLL_LOCK_PLLE_MASK			0x00000001
> +
> +#define ESW_SYS_DIV_ADDR		(ESUB_CLK_BASE_ADDR + 0x00000A04)
> +#define ESW_SYS_DIV_PLL_SELECT_MASK			0x00000300
> +#define ESW_SYS_DIV_DIV_MASK				0x0000001C
> +#define ESW_SYS_DIV_PLL_VAR_208M_CLK_SELECT		0x00000100
> +#define ESW_SYS_DIV_DIV_SELECT				0x4
> +#define ESW_SYS_DIV_TRIGGER_MASK			0x00000001
> +
> +#define ESUB_AXI_DIV_DEBUG_ADDR		(ESUB_CLK_BASE_ADDR + 0x00000E04)
> +#define ESUB_AXI_DIV_DEBUG_PLL_SELECT_MASK		0x0000001C
> +#define ESUB_AXI_DIV_DEBUG_PLL_SELECT_OVERRIDE_MASK	0x00000040
> +#define ESUB_AXI_DIV_DEBUG_PLL_VAR_208M_CLK_SELECT	0x0
> +#define ESUB_AXI_DIV_DEBUG_TRIGGER_MASK			0x00000001
> +
> +#define PLL_MAX_RETRY	100
> +
> +/* Enable appropriate clocks for Ethernet */
> +int clk_eth_enable(void)
> +{
> +	int rc = -1;
> +	int retry_count = 0;
> +	rc = clk_get_and_enable("esub_ccu_clk");
> +
> +	/* Enable Access to CCU registers */
> +	writel((1 | WR_ACCESS_PASSWORD), WR_ACCESS_ADDR);
> +
> +	writel(readl(PLLE_POST_RESETB_ADDR) &
> +	       ~PLLE_POST_RESETB_I_POST_RESETB_PLLE_MASK,
> +	       PLLE_POST_RESETB_ADDR);
> +
> +	/* Take PLL out of reset and put into normal mode */
> +	writel(readl(PLLE_RESETB_ADDR) | PLLE_RESETB_I_PLL_RESETB_PLLE_MASK,
> +	       PLLE_RESETB_ADDR);
> +
> +	/* Wait for PLL lock */
> +	rc = -1;
> +	while (retry_count < PLL_MAX_RETRY) {
> +		udelay(100);
> +		if (readl(PLL_LOCK_ADDR) & PLL_LOCK_PLL_LOCK_PLLE_MASK) {
> +			rc = 0;
> +			break;
> +		}
> +		retry_count++;
> +	}
> +
> +	if (rc == -1) {
> +		printf("%s: ETH-PLL lock timeout, Ethernet is not enabled!\n",
> +		       __func__);
> +		return -1;
> +	}
> +
> +	writel(readl(PLLE_POST_RESETB_ADDR) |
> +	       PLLE_POST_RESETB_I_POST_RESETB_PLLE_MASK,
> +	       PLLE_POST_RESETB_ADDR);
> +
> +	/* Switch esw_sys_clk to use 104MHz(208MHz/2) clock */
> +	writel((readl(ESW_SYS_DIV_ADDR) &
> +		~(ESW_SYS_DIV_PLL_SELECT_MASK | ESW_SYS_DIV_DIV_MASK)) |
> +	       ESW_SYS_DIV_PLL_VAR_208M_CLK_SELECT | ESW_SYS_DIV_DIV_SELECT,
> +	       ESW_SYS_DIV_ADDR);
> +
> +	writel(readl(ESW_SYS_DIV_ADDR) | ESW_SYS_DIV_TRIGGER_MASK,
> +	       ESW_SYS_DIV_ADDR);
> +
> +	/* Wait for trigger complete */
> +	rc = -1;
> +	retry_count = 0;
> +	while (retry_count < PLL_MAX_RETRY) {
> +		udelay(100);
> +		if (!(readl(ESW_SYS_DIV_ADDR) & ESW_SYS_DIV_TRIGGER_MASK)) {
> +			rc = 0;
> +			break;
> +		}
> +		retry_count++;
> +	}
> +
> +	if (rc == -1) {
> +		printf("%s: SYS CLK Trigger timeout, Ethernet is not enabled!\n",
> +		       __func__);
> +		return -1;
> +	}
> +
> +	/* switch Esub AXI clock to 208MHz */
> +	writel((readl(ESUB_AXI_DIV_DEBUG_ADDR) &
> +		~(ESUB_AXI_DIV_DEBUG_PLL_SELECT_MASK |
> +		  ESUB_AXI_DIV_DEBUG_PLL_SELECT_OVERRIDE_MASK |
> +		  ESUB_AXI_DIV_DEBUG_TRIGGER_MASK)) |
> +	       ESUB_AXI_DIV_DEBUG_PLL_VAR_208M_CLK_SELECT |
> +	       ESUB_AXI_DIV_DEBUG_PLL_SELECT_OVERRIDE_MASK,
> +	       ESUB_AXI_DIV_DEBUG_ADDR);
> +
> +	writel(readl(ESUB_AXI_DIV_DEBUG_ADDR) |
> +	       ESUB_AXI_DIV_DEBUG_TRIGGER_MASK,
> +	       ESUB_AXI_DIV_DEBUG_ADDR);
> +
> +	/* Wait for trigger complete */
> +	rc = -1;
> +	retry_count = 0;
> +	while (retry_count < PLL_MAX_RETRY) {
> +		udelay(100);
> +		if (!(readl(ESUB_AXI_DIV_DEBUG_ADDR) &
> +		      ESUB_AXI_DIV_DEBUG_TRIGGER_MASK)) {
> +			rc = 0;
> +			break;
> +		}
> +		retry_count++;
> +	}
> +
> +	if (rc == -1) {
> +		printf("%s: AXI CLK Trigger timeout, Ethernet is not enabled!\n",
> +		       __func__);
> +		return -1;
> +	}
> +
> +	/* Disable Access to CCU registers */
> +	writel(WR_ACCESS_PASSWORD, WR_ACCESS_ADDR);
> +
> +	return rc;
> +}
> diff --git a/arch/arm/include/asm/arch-bcm281xx/sysmap.h b/arch/arm/include/asm/arch-bcm281xx/sysmap.h
> index 880b4e0..350e7f6 100644
> --- a/arch/arm/include/asm/arch-bcm281xx/sysmap.h
> +++ b/arch/arm/include/asm/arch-bcm281xx/sysmap.h
> @@ -9,6 +9,9 @@
>   #define BSC1_BASE_ADDR		0x3e016000
>   #define BSC2_BASE_ADDR		0x3e017000
>   #define BSC3_BASE_ADDR		0x3e018000
> +#define DWDMA_AHB_BASE_ADDR	0x38100000
> +#define ESUB_CLK_BASE_ADDR	0x38000000
> +#define ESW_CONTRL_BASE_ADDR	0x38200000
>   #define GPIO2_BASE_ADDR		0x35003000
>   #define KONA_MST_CLK_BASE_ADDR	0x3f001000
>   #define KONA_SLV_CLK_BASE_ADDR	0x3e011000
>

Any chance of getting this series in soon?
( we have an upcoming release.... )
Thanks, Steve


More information about the U-Boot mailing list