Message ID | 1465283488-30830-3-git-send-email-wenyou.yang@atmel.com |
---|---|
State | Superseded |
Delegated to: | Simon Glass |
Headers | show |
+Stephen Hi Wenyou, On 7 June 2016 at 01:11, Wenyou Yang <wenyou.yang@atmel.com> wrote: > The patch is referred to at91 clock driver of Linux, to make > the clock node descriptions in dt are aligned with the Linux. > > Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com> > --- > > Changes in v2: > - Remove the redundant log print. > > arch/arm/mach-at91/include/mach/at91_pmc.h | 11 ++- > drivers/clk/Kconfig | 1 + > drivers/clk/Makefile | 1 + > drivers/clk/at91/Kconfig | 14 +++ > drivers/clk/at91/Makefile | 11 +++ > drivers/clk/at91/clk-generated.c | 138 +++++++++++++++++++++++++++++ > drivers/clk/at91/clk-h32mx.c | 56 ++++++++++++ > drivers/clk/at91/clk-main.c | 55 ++++++++++++ > drivers/clk/at91/clk-master.c | 33 +++++++ > drivers/clk/at91/clk-peripheral.c | 68 ++++++++++++++ > drivers/clk/at91/clk-plla.c | 55 ++++++++++++ > drivers/clk/at91/clk-slow.c | 37 ++++++++ > drivers/clk/at91/clk-system.c | 65 ++++++++++++++ > drivers/clk/at91/clk-utmi.c | 67 ++++++++++++++ > drivers/clk/at91/pmc.c | 49 ++++++++++ > drivers/clk/at91/pmc.h | 17 ++++ > drivers/clk/at91/sckc.c | 30 +++++++ > 17 files changed, 705 insertions(+), 3 deletions(-) > create mode 100644 drivers/clk/at91/Kconfig > create mode 100644 drivers/clk/at91/Makefile > create mode 100644 drivers/clk/at91/clk-generated.c > create mode 100644 drivers/clk/at91/clk-h32mx.c > create mode 100644 drivers/clk/at91/clk-main.c > create mode 100644 drivers/clk/at91/clk-master.c > create mode 100644 drivers/clk/at91/clk-peripheral.c > create mode 100644 drivers/clk/at91/clk-plla.c > create mode 100644 drivers/clk/at91/clk-slow.c > create mode 100644 drivers/clk/at91/clk-system.c > create mode 100644 drivers/clk/at91/clk-utmi.c > create mode 100644 drivers/clk/at91/pmc.c > create mode 100644 drivers/clk/at91/pmc.h > create mode 100644 drivers/clk/at91/sckc.c > > diff --git a/arch/arm/mach-at91/include/mach/at91_pmc.h b/arch/arm/mach-at91/include/mach/at91_pmc.h > index 7684f09..d02f3e1 100644 > --- a/arch/arm/mach-at91/include/mach/at91_pmc.h > +++ b/arch/arm/mach-at91/include/mach/at91_pmc.h > @@ -149,6 +149,9 @@ typedef struct at91_pmc { > > #define AT91_PMC_PCR_PID_MASK (0x3f) > #define AT91_PMC_PCR_GCKCSS (0x7 << 8) > +#define AT91_PMC_PCR_GCKCSS_MASK 0x07 > +#define AT91_PMC_PCR_GCKCSS_OFFSET 8 > +#define AT91_PMC_PCR_GCKCSS_(x) ((x & 0x07) << 8) > #define AT91_PMC_PCR_GCKCSS_SLOW_CLK (0x0 << 8) > #define AT91_PMC_PCR_GCKCSS_MAIN_CLK (0x1 << 8) > #define AT91_PMC_PCR_GCKCSS_PLLA_CLK (0x2 << 8) > @@ -158,8 +161,9 @@ typedef struct at91_pmc { > #define AT91_PMC_PCR_CMD_WRITE (0x1 << 12) > #define AT91_PMC_PCR_DIV (0x3 << 16) > #define AT91_PMC_PCR_GCKDIV (0xff << 20) > -#define AT91_PMC_PCR_GCKDIV_(x) ((x & 0xff) << 20) > -#define AT91_PMC_PCR_GCKDIV_OFFSET 20 > +#define AT91_PMC_PCR_GCKDIV_MASK 0xff > +#define AT91_PMC_PCR_GCKDIV_OFFSET 20 > +#define AT91_PMC_PCR_GCKDIV_(x) ((x & 0xff) << 20) > #define AT91_PMC_PCR_EN (0x1 << 28) > #define AT91_PMC_PCR_GCKEN (0x1 << 29) > > @@ -243,8 +247,9 @@ typedef struct at91_pmc { > #define AT91_PMC_PCK1RDY (1 << 9) /* Programmable Clock 1 */ > #define AT91_PMC_PCK2RDY (1 << 10) /* Programmable Clock 2 */ > #define AT91_PMC_PCK3RDY (1 << 11) /* Programmable Clock 3 */ > +#define AT91_PMC_MOSCSELS BIT(16) /* Main Oscillator Selection Status */ > +#define AT91_PMC_MOSCRCS BIT(17) /* 12 MHz RC Oscillator Status */ > #define AT91_PMC_GCKRDY (1 << 24) > - > #define AT91_PMC_PROTKEY 0x504d4301 /* Activation Code */ > > /* PLL Charge Pump Current Register (PMC_PLLICPR) */ > diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig > index a98b74b..1fd1074 100644 > --- a/drivers/clk/Kconfig > +++ b/drivers/clk/Kconfig > @@ -21,5 +21,6 @@ config SPL_CLK > used as U-Boot proper. > > source "drivers/clk/uniphier/Kconfig" > +source "drivers/clk/at91/Kconfig" > > endmenu > diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile > index c51db15..2bb0f35 100644 > --- a/drivers/clk/Makefile > +++ b/drivers/clk/Makefile > @@ -11,3 +11,4 @@ obj-$(CONFIG_ROCKCHIP_RK3288) += clk_rk3288.o > obj-$(CONFIG_SANDBOX) += clk_sandbox.o > obj-$(CONFIG_MACH_PIC32) += clk_pic32.o > obj-$(CONFIG_CLK_UNIPHIER) += uniphier/ > +obj-$(CONFIG_CLK_AT91) += at91/ > diff --git a/drivers/clk/at91/Kconfig b/drivers/clk/at91/Kconfig > new file mode 100644 > index 0000000..f9cb937 > --- /dev/null > +++ b/drivers/clk/at91/Kconfig > @@ -0,0 +1,14 @@ > +config CLK_AT91 > + bool "AT91 clock drivers" > + depends on CLK > + help > + This option is used to enable the AT91 clock driver. > + > +config HAVE_AT91_UTMI > + bool > + > +config HAVE_AT91_H32MX > + bool > + > +config HAVE_AT91_GENERATED_CLK > + bool Please can you add help? Also we are changing the clock interface - please see this patch: http://patchwork.ozlabs.org/patch/625342/ I'm hoping to bring it in next week. > diff --git a/drivers/clk/at91/Makefile b/drivers/clk/at91/Makefile > new file mode 100644 > index 0000000..eb1326b > --- /dev/null > +++ b/drivers/clk/at91/Makefile > @@ -0,0 +1,11 @@ > +# > +# Makefile for at91 specific clk > +# > + > +obj-y += pmc.o sckc.o > +obj-y += clk-slow.o clk-main.o clk-plla.o clk-master.o > +obj-y += clk-system.o clk-peripheral.o > + > +obj-$(CONFIG_HAVE_AT91_UTMI) += clk-utmi.o > +obj-$(CONFIG_HAVE_AT91_H32MX) += clk-h32mx.o > +obj-$(CONFIG_HAVE_AT91_GENERATED_CLK) += clk-generated.o > diff --git a/drivers/clk/at91/clk-generated.c b/drivers/clk/at91/clk-generated.c > new file mode 100644 > index 0000000..08e7a2f > --- /dev/null > +++ b/drivers/clk/at91/clk-generated.c > @@ -0,0 +1,138 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > +#include <linux/io.h> > +#include <mach/at91_pmc.h> > +#include "pmc.h" > + > +DECLARE_GLOBAL_DATA_PTR; > + > +#define GENERATED_SOURCE_MAX 6 > +#define GENERATED_MAX_DIV 255 > + > +static ulong generated_clk_get_periph_rate(struct udevice *dev, int periph) > +{ > + struct pmc_platdata *plat = dev_get_platdata(dev); > + struct at91_pmc *pmc = plat->reg_base; > + struct udevice *parent; > + u32 tmp, gckdiv; > + u8 parent_id; > + int ret; > + > + writel(periph & AT91_PMC_PCR_PID_MASK, &pmc->pcr); > + tmp = readl(&pmc->pcr); > + parent_id = (tmp >> AT91_PMC_PCR_GCKCSS_OFFSET) & > + AT91_PMC_PCR_GCKCSS_MASK; > + gckdiv = (tmp >> AT91_PMC_PCR_GCKDIV_OFFSET) & AT91_PMC_PCR_GCKDIV_MASK; > + > + ret = clk_get_by_index(dev, parent_id, &parent); > + if (ret) > + return 0; > + > + return clk_get_rate(parent) / (gckdiv + 1); > +} > + > +static ulong generated_clk_set_periph_rate(struct udevice *dev, > + int periph, > + ulong rate) > +{ > + struct pmc_platdata *plat = dev_get_platdata(dev); > + struct at91_pmc *pmc = plat->reg_base; > + struct udevice *parent, *best_parent = NULL; > + ulong tmp_rate, best_rate = rate, parent_rate; > + int tmp_diff, best_diff = -1; > + u8 num_parents, i; > + u32 cells[GENERATED_SOURCE_MAX]; > + u32 div, best_div = 0; > + u8 best_parent_id = 0; > + u32 tmp; > + int ret; > + > + num_parents = fdtdec_get_int_array_count(gd->fdt_blob, dev->of_offset, > + "clocks", cells, > + GENERATED_SOURCE_MAX); Are you able to store this value in your priv data in your of_to_platdata() method? We should not be looking at the device tree outside that method. > + > + for (i = 0; i < num_parents; i++) { > + ret = clk_get_by_index(dev, i, &parent); > + if (ret) > + return ret; > + > + parent_rate = clk_get_rate(parent); > + if (!parent_rate) > + return -ENODEV; > + > + for (div = 1; div < GENERATED_MAX_DIV + 2; div++) { > + tmp_rate = DIV_ROUND_CLOSEST(parent_rate, div); > + if (rate < tmp_rate) > + continue; > + tmp_diff = rate - tmp_rate; > + > + if (best_diff < 0 || best_diff > tmp_diff) { > + best_rate = tmp_rate; > + best_diff = tmp_diff; > + > + best_div = div - 1; > + best_parent = parent; > + best_parent_id = i; > + } > + > + if (!best_diff || tmp_rate < rate) > + break; > + } > + > + if (!best_diff) > + break; > + } > + > + dev_dbg(dev, "GCK: best parent: %s, best_rate = %ld, best_div = %d\n", > + best_parent->name, best_rate, best_div); > + > + ret = clk_enable(best_parent, 0); > + if (ret) > + return -ENODEV; > + > + writel(periph & AT91_PMC_PCR_PID_MASK, &pmc->pcr); > + tmp = readl(&pmc->pcr); > + tmp &= ~(AT91_PMC_PCR_GCKDIV | AT91_PMC_PCR_GCKCSS); > + tmp |= AT91_PMC_PCR_GCKCSS_(best_parent_id) | > + AT91_PMC_PCR_CMD_WRITE | > + AT91_PMC_PCR_GCKDIV_(best_div) | > + AT91_PMC_PCR_GCKEN; > + writel(tmp, &pmc->pcr); > + > + while (!(readl(&pmc->sr) & AT91_PMC_GCKRDY)) > + ; > + > + return 0; > +} > + > +static struct clk_ops generated_clk_ops = { > + .get_periph_rate = generated_clk_get_periph_rate, > + .set_periph_rate = generated_clk_set_periph_rate, > +}; > + > +static int generated_clk_probe(struct udevice *dev) > +{ > + return at91_pmc_core_probe(dev); > +} > + > +static const struct udevice_id generated_clk_match[] = { > + { .compatible = "atmel,sama5d2-clk-generated" }, > + {} > +}; > + > +U_BOOT_DRIVER(generated_clk) = { > + .name = "generated-clk", > + .id = UCLASS_CLK, > + .of_match = generated_clk_match, > + .probe = generated_clk_probe, > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > + .ops = &generated_clk_ops, > +}; > diff --git a/drivers/clk/at91/clk-h32mx.c b/drivers/clk/at91/clk-h32mx.c > new file mode 100644 > index 0000000..035562a > --- /dev/null > +++ b/drivers/clk/at91/clk-h32mx.c > @@ -0,0 +1,56 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > +#include <dm/util.h> > +#include <linux/io.h> > +#include <mach/at91_pmc.h> > +#include "pmc.h" > + > +DECLARE_GLOBAL_DATA_PTR; > + > +#define H32MX_MAX_FREQ 90000000 > + > +static ulong sama5d4_h32mx_clk_get_rate(struct udevice *dev) > +{ > + struct pmc_platdata *plat = dev_get_platdata(dev); > + struct at91_pmc *pmc = plat->reg_base; > + ulong rate = gd->arch.mck_rate_hz; > + > + if (readl(&pmc->mckr) & AT91_PMC_MCKR_H32MXDIV) > + rate /= 2; > + > + if (rate > H32MX_MAX_FREQ) > + dm_warn("H32MX clock is too fast\n"); > + > + return rate; > +} > + > +static struct clk_ops sama5d4_h32mx_clk_ops = { > + .get_rate = sama5d4_h32mx_clk_get_rate, > +}; > + > +static int sama5d4_h32mx_clk_probe(struct udevice *dev) > +{ > + return at91_pmc_core_probe(dev); > +} > + > +static const struct udevice_id sama5d4_h32mx_clk_match[] = { > + { .compatible = "atmel,sama5d4-clk-h32mx" }, > + {} > +}; > + > +U_BOOT_DRIVER(sama5d4_h32mx_clk) = { > + .name = "sama5d4-h32mx-clk", > + .id = UCLASS_CLK, > + .of_match = sama5d4_h32mx_clk_match, > + .probe = sama5d4_h32mx_clk_probe, > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > + .ops = &sama5d4_h32mx_clk_ops, > +}; > diff --git a/drivers/clk/at91/clk-main.c b/drivers/clk/at91/clk-main.c > new file mode 100644 > index 0000000..10ab951 > --- /dev/null > +++ b/drivers/clk/at91/clk-main.c > @@ -0,0 +1,55 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > +#include <linux/io.h> > +#include <mach/at91_pmc.h> > +#include "pmc.h" > + > +DECLARE_GLOBAL_DATA_PTR; > + > +static int main_osc_clk_enable(struct udevice *dev, int periph) > +{ > + struct pmc_platdata *plat = dev_get_platdata(dev); > + struct at91_pmc *pmc = plat->reg_base; > + > + if (readl(&pmc->sr) & AT91_PMC_MOSCSELS) > + return 0; > + > + return -ENODEV; This error indicates that there is no device. But it looks like there is. What are you trying to represent here? > +} > + > +static ulong main_osc_clk_get_rate(struct udevice *dev) > +{ > + return gd->arch.main_clk_rate_hz; > +} > + > +static struct clk_ops main_osc_clk_ops = { > + .enable = main_osc_clk_enable, > + .get_rate = main_osc_clk_get_rate, > +}; > + > +static int main_osc_clk_probe(struct udevice *dev) > +{ > + return at91_pmc_core_probe(dev); > +} > + > +static const struct udevice_id main_osc_clk_match[] = { > + { .compatible = "atmel,at91sam9x5-clk-main" }, > + {} > +}; > + > +U_BOOT_DRIVER(at91sam9x5_main_osc_clk) = { > + .name = "at91sam9x5-main-osc-clk", > + .id = UCLASS_CLK, > + .of_match = main_osc_clk_match, > + .probe = main_osc_clk_probe, > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > + .ops = &main_osc_clk_ops, > +}; > diff --git a/drivers/clk/at91/clk-master.c b/drivers/clk/at91/clk-master.c > new file mode 100644 > index 0000000..6d844d0 > --- /dev/null > +++ b/drivers/clk/at91/clk-master.c > @@ -0,0 +1,33 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > + > +DECLARE_GLOBAL_DATA_PTR; > + > +static ulong at91_master_clk_get_rate(struct udevice *dev) > +{ > + return gd->arch.mck_rate_hz; > +} > + > +static struct clk_ops at91_master_clk_ops = { > + .get_rate = at91_master_clk_get_rate, > +}; > + > +static const struct udevice_id at91_master_clk_match[] = { > + { .compatible = "atmel,at91sam9x5-clk-master" }, > + {} > +}; > + > +U_BOOT_DRIVER(at91_master_clk) = { > + .name = "at91-master-clk", > + .id = UCLASS_CLK, > + .of_match = at91_master_clk_match, > + .ops = &at91_master_clk_ops, > +}; > diff --git a/drivers/clk/at91/clk-peripheral.c b/drivers/clk/at91/clk-peripheral.c > new file mode 100644 > index 0000000..4faeca5 > --- /dev/null > +++ b/drivers/clk/at91/clk-peripheral.c > @@ -0,0 +1,68 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > +#include <linux/io.h> > +#include <mach/at91_pmc.h> > +#include "pmc.h" > + > +#define PERIPHERAL_MAX 64 > + > +#define PERIPHERAL_AT91RM9200 0 > +#define PERIPHERAL_AT91SAM9X5 1 > + > +#define PERIPHERAL_ID_MIN 2 > +#define PERIPHERAL_ID_MAX 31 > +#define PERIPHERAL_MASK(id) (1 << ((id) & PERIPHERAL_ID_MAX)) > + > +#define PERIPHERAL_RSHIFT_MASK 0x3 > +#define PERIPHERAL_RSHIFT(val) (((val) >> 16) & PERIPHERAL_RSHIFT_MASK) > + > +#define PERIPHERAL_MAX_SHIFT 3 > + > +static int sam9x5_periph_clk_enable(struct udevice *dev, int periph) > +{ > + struct pmc_platdata *plat = dev_get_platdata(dev); > + struct at91_pmc *pmc = plat->reg_base; > + u32 tmp; > + > + if (periph < PERIPHERAL_ID_MIN) > + return -1; > + > + writel(periph & AT91_PMC_PCR_PID_MASK, &pmc->pcr); > + tmp = readl(&pmc->pcr); > + writel(tmp | > + AT91_PMC_PCR_CMD_WRITE | > + AT91_PMC_PCR_EN, &pmc->pcr); Or use setbits_le32(&pmc->pcr, AT91_PMC_PCR_CMD_WRITE | AT91_PMC_PCR_EN) > + > + return 0; > +} > + > +static struct clk_ops sam9x5_periph_clk_ops = { > + .enable = sam9x5_periph_clk_enable, > +}; > + > +static int sam9x5_periph_clk_probe(struct udevice *dev) > +{ > + return at91_pmc_core_probe(dev); > +} > + > +static const struct udevice_id sam9x5_periph_clk_match[] = { > + { .compatible = "atmel,at91sam9x5-clk-peripheral" }, > + {} > +}; > + > +U_BOOT_DRIVER(sam9x5_periph_clk) = { > + .name = "sam9x5-periph-clk", > + .id = UCLASS_CLK, > + .of_match = sam9x5_periph_clk_match, > + .probe = sam9x5_periph_clk_probe, > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > + .ops = &sam9x5_periph_clk_ops, > +}; > diff --git a/drivers/clk/at91/clk-plla.c b/drivers/clk/at91/clk-plla.c > new file mode 100644 > index 0000000..3a2168e > --- /dev/null > +++ b/drivers/clk/at91/clk-plla.c > @@ -0,0 +1,55 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > +#include <linux/io.h> > +#include <mach/at91_pmc.h> > +#include "pmc.h" > + > +DECLARE_GLOBAL_DATA_PTR; > + > +static int plla_clk_enable(struct udevice *dev, int periph) > +{ > + struct pmc_platdata *plat = dev_get_platdata(dev); > + struct at91_pmc *pmc = plat->reg_base; > + > + if (readl(&pmc->sr) & AT91_PMC_LOCKA) > + return 0; > + > + return -ENODEV; Same error question here. > +} > + > +static ulong plla_clk_get_rate(struct udevice *dev) > +{ > + return gd->arch.plla_rate_hz; > +} > + > +static struct clk_ops plla_clk_ops = { > + .enable = plla_clk_enable, > + .get_rate = plla_clk_get_rate, > +}; > + > +static int plla_clk_probe(struct udevice *dev) > +{ > + return at91_pmc_core_probe(dev); > +} > + > +static const struct udevice_id plla_clk_match[] = { > + { .compatible = "atmel,sama5d3-clk-pll" }, > + {} > +}; > + > +U_BOOT_DRIVER(at91_plla_clk) = { > + .name = "at91-plla-clk", > + .id = UCLASS_CLK, > + .of_match = plla_clk_match, > + .probe = plla_clk_probe, > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > + .ops = &plla_clk_ops, > +}; > diff --git a/drivers/clk/at91/clk-slow.c b/drivers/clk/at91/clk-slow.c > new file mode 100644 > index 0000000..f73a553 > --- /dev/null > +++ b/drivers/clk/at91/clk-slow.c > @@ -0,0 +1,37 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > + > +static int at91_slow_clk_enable(struct udevice *dev, int periph) > +{ > + return 0; > +} > + > +static ulong at91_slow_clk_get_rate(struct udevice *dev) > +{ > + return CONFIG_SYS_AT91_SLOW_CLOCK; > +} > + > +static struct clk_ops at91_slow_clk_ops = { > + .enable = at91_slow_clk_enable, > + .get_rate = at91_slow_clk_get_rate, > +}; > + > +static const struct udevice_id at91_slow_clk_match[] = { > + { .compatible = "atmel,at91sam9x5-clk-slow" }, > + {} > +}; > + > +U_BOOT_DRIVER(at91_slow_clk) = { > + .name = "at91-slow-clk", > + .id = UCLASS_CLK, > + .of_match = at91_slow_clk_match, > + .ops = &at91_slow_clk_ops, > +}; > diff --git a/drivers/clk/at91/clk-system.c b/drivers/clk/at91/clk-system.c > new file mode 100644 > index 0000000..5e50af6 > --- /dev/null > +++ b/drivers/clk/at91/clk-system.c > @@ -0,0 +1,65 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > +#include <linux/io.h> > +#include <mach/at91_pmc.h> > +#include "pmc.h" > + > +#define SYSTEM_MAX_ID 31 > + > +static inline int is_pck(int id) > +{ > + return (id >= 8) && (id <= 15); > +} > + > +static int at91_system_clk_enable(struct udevice *dev, int id) > +{ > + struct pmc_platdata *plat = dev_get_platdata(dev); > + struct at91_pmc *pmc = plat->reg_base; > + u32 mask; > + > + if (id > SYSTEM_MAX_ID) > + return -EINVAL; > + > + mask = BIT(id); > + > + writel(mask, &pmc->scer); > + > + if (!is_pck(id)) > + return 0; Please can you add a comment to explain the above two lines? > + > + while (!(readl(&pmc->sr) & mask)) > + ; > + > + return 0; > +} > + > +static struct clk_ops at91_system_clk_ops = { > + .enable = at91_system_clk_enable, > +}; > + > +static int at91_system_clk_probe(struct udevice *dev) > +{ > + return at91_pmc_core_probe(dev); > +} > + > +static const struct udevice_id at91_system_clk_match[] = { > + { .compatible = "atmel,at91rm9200-clk-system" }, > + {} > +}; > + > +U_BOOT_DRIVER(at91_system_clk) = { > + .name = "at91-system-clk", > + .id = UCLASS_CLK, > + .of_match = at91_system_clk_match, > + .probe = at91_system_clk_probe, > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > + .ops = &at91_system_clk_ops, > +}; > diff --git a/drivers/clk/at91/clk-utmi.c b/drivers/clk/at91/clk-utmi.c > new file mode 100644 > index 0000000..835c611 > --- /dev/null > +++ b/drivers/clk/at91/clk-utmi.c > @@ -0,0 +1,67 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > +#include <linux/io.h> > +#include <mach/at91_pmc.h> > +#include "pmc.h" > + > +DECLARE_GLOBAL_DATA_PTR; > + > +#define UTMI_FIXED_MUL 40 > + > +static int utmi_clk_enable(struct udevice *dev, int periph) > +{ > + struct pmc_platdata *plat = dev_get_platdata(dev); > + struct at91_pmc *pmc = plat->reg_base; > + u32 tmp; > + > + if (readl(&pmc->sr) & AT91_PMC_LOCKU) > + return 0; > + > + tmp = readl(&pmc->uckr); > + tmp |= AT91_PMC_UPLLEN | > + AT91_PMC_UPLLCOUNT | > + AT91_PMC_BIASEN; > + writel(tmp, &pmc->uckr); > + > + while (!(readl(&pmc->sr) & AT91_PMC_LOCKU)) > + ; > + > + return 0; > +} > + > +static ulong utmi_clk_get_rate(struct udevice *dev) > +{ > + return gd->arch.main_clk_rate_hz * UTMI_FIXED_MUL; > +} > + > +static struct clk_ops utmi_clk_ops = { > + .enable = utmi_clk_enable, > + .get_rate = utmi_clk_get_rate, > +}; > + > +static int utmi_clk_probe(struct udevice *dev) > +{ > + return at91_pmc_core_probe(dev); > +} > + > +static const struct udevice_id utmi_clk_match[] = { > + { .compatible = "atmel,at91sam9x5-clk-utmi" }, > + {} > +}; > + > +U_BOOT_DRIVER(at91sam9x5_utmi_clk) = { > + .name = "at91sam9x5-utmi-clk", > + .id = UCLASS_CLK, > + .of_match = utmi_clk_match, > + .probe = utmi_clk_probe, > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > + .ops = &utmi_clk_ops, > +}; > diff --git a/drivers/clk/at91/pmc.c b/drivers/clk/at91/pmc.c > new file mode 100644 > index 0000000..de802e8 > --- /dev/null > +++ b/drivers/clk/at91/pmc.c > @@ -0,0 +1,49 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > +#include <dm/root.h> > +#include "pmc.h" > + > +DECLARE_GLOBAL_DATA_PTR; > + > +static int at91_pmc_core_bind(struct udevice *dev) > +{ > + return dm_scan_fdt_node(dev, gd->fdt_blob, dev->of_offset, false); > +} > + > +static const struct udevice_id at91_pmc_core_match[] = { > + { .compatible = "atmel,sama5d2-pmc" }, > + {} > +}; > + > +U_BOOT_DRIVER(at91_pmc_core) = { > + .name = "at91_pmc_core", > + .id = UCLASS_CLK, > + .of_match = at91_pmc_core_match, > + .bind = at91_pmc_core_bind, > +}; > + > +int at91_pmc_core_probe(struct udevice *dev) > +{ > + struct pmc_platdata *plat = dev_get_platdata(dev); > + fdt_addr_t addr; > + > + dev = dev_get_parent(dev); > + if (!dev) > + return -EINVAL; There will always be a parent, so this test is redundant. > + > + addr = dev_get_addr(dev); > + if (addr == FDT_ADDR_T_NONE) > + return -EINVAL; You can use dev_get_addr_ptr() here. > + > + plat->reg_base = (struct at91_pmc *)addr; > + > + return 0; > +} > diff --git a/drivers/clk/at91/pmc.h b/drivers/clk/at91/pmc.h > new file mode 100644 > index 0000000..201dad8 > --- /dev/null > +++ b/drivers/clk/at91/pmc.h > @@ -0,0 +1,17 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#ifndef __AT91_PMC_H__ > +#define __AT91_PMC_H__ > + > +struct pmc_platdata { > + struct at91_pmc *reg_base; > +}; > + > +int at91_pmc_core_probe(struct udevice *dev); > + > +#endif > diff --git a/drivers/clk/at91/sckc.c b/drivers/clk/at91/sckc.c > new file mode 100644 > index 0000000..69ec636 > --- /dev/null > +++ b/drivers/clk/at91/sckc.c > @@ -0,0 +1,30 @@ > +/* > + * Copyright (C) 2016 Atmel Corporation > + * Wenyou.Yang <wenyou.yang@atmel.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <clk.h> > +#include <dm/device.h> > +#include <dm/root.h> > + > +DECLARE_GLOBAL_DATA_PTR; > + > +static int at91_sckc_clk_bind(struct udevice *dev) > +{ > + return dm_scan_fdt_node(dev, gd->fdt_blob, dev->of_offset, false); > +} > + > +static const struct udevice_id at91_sckc_clk_match[] = { > + { .compatible = "atmel,at91sam9x5-sckc" }, > + {} > +}; > + > +U_BOOT_DRIVER(at91_sckc_clk) = { > + .name = "at91_sckc_clk", > + .id = UCLASS_CLK, > + .of_match = at91_sckc_clk_match, > + .bind = at91_sckc_clk_bind, > +}; > -- > 2.7.4 > Regards, Simon
Hi Simon, > -----Original Message----- > From: sjg@google.com [mailto:sjg@google.com] On Behalf Of Simon Glass > Sent: 2016年6月10日 8:34 > To: Yang, Wenyou <Wenyou.Yang@atmel.com> > Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Andreas Bießmann > <andreas@biessmann.org>; Stephen Warren <swarren@wwwdotorg.org> > Subject: Re: [PATCH v2 2/2] clk: at91: Add new clock driver > > +Stephen > > Hi Wenyou, > > On 7 June 2016 at 01:11, Wenyou Yang <wenyou.yang@atmel.com> wrote: > > The patch is referred to at91 clock driver of Linux, to make the clock > > node descriptions in dt are aligned with the Linux. > > > > Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com> > > --- > > > > Changes in v2: > > - Remove the redundant log print. > > > > arch/arm/mach-at91/include/mach/at91_pmc.h | 11 ++- > > drivers/clk/Kconfig | 1 + > > drivers/clk/Makefile | 1 + > > drivers/clk/at91/Kconfig | 14 +++ > > drivers/clk/at91/Makefile | 11 +++ > > drivers/clk/at91/clk-generated.c | 138 > +++++++++++++++++++++++++++++ > > drivers/clk/at91/clk-h32mx.c | 56 ++++++++++++ > > drivers/clk/at91/clk-main.c | 55 ++++++++++++ > > drivers/clk/at91/clk-master.c | 33 +++++++ > > drivers/clk/at91/clk-peripheral.c | 68 ++++++++++++++ > > drivers/clk/at91/clk-plla.c | 55 ++++++++++++ > > drivers/clk/at91/clk-slow.c | 37 ++++++++ > > drivers/clk/at91/clk-system.c | 65 ++++++++++++++ > > drivers/clk/at91/clk-utmi.c | 67 ++++++++++++++ > > drivers/clk/at91/pmc.c | 49 ++++++++++ > > drivers/clk/at91/pmc.h | 17 ++++ > > drivers/clk/at91/sckc.c | 30 +++++++ > > 17 files changed, 705 insertions(+), 3 deletions(-) create mode > > 100644 drivers/clk/at91/Kconfig create mode 100644 > > drivers/clk/at91/Makefile create mode 100644 > > drivers/clk/at91/clk-generated.c create mode 100644 > > drivers/clk/at91/clk-h32mx.c create mode 100644 > > drivers/clk/at91/clk-main.c create mode 100644 > > drivers/clk/at91/clk-master.c create mode 100644 > > drivers/clk/at91/clk-peripheral.c create mode 100644 > > drivers/clk/at91/clk-plla.c create mode 100644 > > drivers/clk/at91/clk-slow.c create mode 100644 > > drivers/clk/at91/clk-system.c create mode 100644 > > drivers/clk/at91/clk-utmi.c create mode 100644 drivers/clk/at91/pmc.c > > create mode 100644 drivers/clk/at91/pmc.h create mode 100644 > > drivers/clk/at91/sckc.c > > > > diff --git a/arch/arm/mach-at91/include/mach/at91_pmc.h > > b/arch/arm/mach-at91/include/mach/at91_pmc.h > > index 7684f09..d02f3e1 100644 > > --- a/arch/arm/mach-at91/include/mach/at91_pmc.h > > +++ b/arch/arm/mach-at91/include/mach/at91_pmc.h > > @@ -149,6 +149,9 @@ typedef struct at91_pmc { > > > > #define AT91_PMC_PCR_PID_MASK (0x3f) > > #define AT91_PMC_PCR_GCKCSS (0x7 << 8) > > +#define AT91_PMC_PCR_GCKCSS_MASK 0x07 > > +#define AT91_PMC_PCR_GCKCSS_OFFSET 8 > > +#define AT91_PMC_PCR_GCKCSS_(x) ((x & 0x07) << 8) > > #define AT91_PMC_PCR_GCKCSS_SLOW_CLK (0x0 << 8) > > #define AT91_PMC_PCR_GCKCSS_MAIN_CLK (0x1 << 8) > > #define AT91_PMC_PCR_GCKCSS_PLLA_CLK (0x2 << 8) > > @@ -158,8 +161,9 @@ typedef struct at91_pmc { > > #define AT91_PMC_PCR_CMD_WRITE (0x1 << 12) > > #define AT91_PMC_PCR_DIV (0x3 << 16) > > #define AT91_PMC_PCR_GCKDIV (0xff << 20) > > -#define AT91_PMC_PCR_GCKDIV_(x) ((x & 0xff) << 20) > > -#define AT91_PMC_PCR_GCKDIV_OFFSET 20 > > +#define AT91_PMC_PCR_GCKDIV_MASK 0xff > > +#define AT91_PMC_PCR_GCKDIV_OFFSET 20 > > +#define AT91_PMC_PCR_GCKDIV_(x) ((x & 0xff) << 20) > > #define AT91_PMC_PCR_EN (0x1 << 28) > > #define AT91_PMC_PCR_GCKEN (0x1 << 29) > > > > @@ -243,8 +247,9 @@ typedef struct at91_pmc { > > #define AT91_PMC_PCK1RDY (1 << 9) /* > Programmable Clock 1 */ > > #define AT91_PMC_PCK2RDY (1 << 10) /* > Programmable Clock 2 */ > > #define AT91_PMC_PCK3RDY (1 << 11) /* > Programmable Clock 3 */ > > +#define AT91_PMC_MOSCSELS BIT(16) /* Main > Oscillator Selection Status */ > > +#define AT91_PMC_MOSCRCS BIT(17) /* 12 MHz > RC Oscillator Status */ > > #define AT91_PMC_GCKRDY (1 << 24) > > - > > #define AT91_PMC_PROTKEY 0x504d4301 /* Activation > Code */ > > > > /* PLL Charge Pump Current Register (PMC_PLLICPR) */ diff --git > > a/drivers/clk/Kconfig b/drivers/clk/Kconfig index a98b74b..1fd1074 > > 100644 > > --- a/drivers/clk/Kconfig > > +++ b/drivers/clk/Kconfig > > @@ -21,5 +21,6 @@ config SPL_CLK > > used as U-Boot proper. > > > > source "drivers/clk/uniphier/Kconfig" > > +source "drivers/clk/at91/Kconfig" > > > > endmenu > > diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile index > > c51db15..2bb0f35 100644 > > --- a/drivers/clk/Makefile > > +++ b/drivers/clk/Makefile > > @@ -11,3 +11,4 @@ obj-$(CONFIG_ROCKCHIP_RK3288) += clk_rk3288.o > > obj-$(CONFIG_SANDBOX) += clk_sandbox.o > > obj-$(CONFIG_MACH_PIC32) += clk_pic32.o > > obj-$(CONFIG_CLK_UNIPHIER) += uniphier/ > > +obj-$(CONFIG_CLK_AT91) += at91/ > > diff --git a/drivers/clk/at91/Kconfig b/drivers/clk/at91/Kconfig new > > file mode 100644 index 0000000..f9cb937 > > --- /dev/null > > +++ b/drivers/clk/at91/Kconfig > > @@ -0,0 +1,14 @@ > > +config CLK_AT91 > > + bool "AT91 clock drivers" > > + depends on CLK > > + help > > + This option is used to enable the AT91 clock driver. > > + > > +config HAVE_AT91_UTMI > > + bool > > + > > +config HAVE_AT91_H32MX > > + bool > > + > > +config HAVE_AT91_GENERATED_CLK > > + bool > > Please can you add help? Accepted. > > Also we are changing the clock interface - please see this patch: > > http://patchwork.ozlabs.org/patch/625342/ > > I'm hoping to bring it in next week. > > > diff --git a/drivers/clk/at91/Makefile b/drivers/clk/at91/Makefile new > > file mode 100644 index 0000000..eb1326b > > --- /dev/null > > +++ b/drivers/clk/at91/Makefile > > @@ -0,0 +1,11 @@ > > +# > > +# Makefile for at91 specific clk > > +# > > + > > +obj-y += pmc.o sckc.o > > +obj-y += clk-slow.o clk-main.o clk-plla.o clk-master.o obj-y += > > +clk-system.o clk-peripheral.o > > + > > +obj-$(CONFIG_HAVE_AT91_UTMI) += clk-utmi.o > > +obj-$(CONFIG_HAVE_AT91_H32MX) += clk-h32mx.o > > +obj-$(CONFIG_HAVE_AT91_GENERATED_CLK) += clk-generated.o > > diff --git a/drivers/clk/at91/clk-generated.c > > b/drivers/clk/at91/clk-generated.c > > new file mode 100644 > > index 0000000..08e7a2f > > --- /dev/null > > +++ b/drivers/clk/at91/clk-generated.c > > @@ -0,0 +1,138 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > +#include <linux/io.h> > > +#include <mach/at91_pmc.h> > > +#include "pmc.h" > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +#define GENERATED_SOURCE_MAX 6 > > +#define GENERATED_MAX_DIV 255 > > + > > +static ulong generated_clk_get_periph_rate(struct udevice *dev, int > > +periph) { > > + struct pmc_platdata *plat = dev_get_platdata(dev); > > + struct at91_pmc *pmc = plat->reg_base; > > + struct udevice *parent; > > + u32 tmp, gckdiv; > > + u8 parent_id; > > + int ret; > > + > > + writel(periph & AT91_PMC_PCR_PID_MASK, &pmc->pcr); > > + tmp = readl(&pmc->pcr); > > + parent_id = (tmp >> AT91_PMC_PCR_GCKCSS_OFFSET) & > > + AT91_PMC_PCR_GCKCSS_MASK; > > + gckdiv = (tmp >> AT91_PMC_PCR_GCKDIV_OFFSET) & > > + AT91_PMC_PCR_GCKDIV_MASK; > > + > > + ret = clk_get_by_index(dev, parent_id, &parent); > > + if (ret) > > + return 0; > > + > > + return clk_get_rate(parent) / (gckdiv + 1); } > > + > > +static ulong generated_clk_set_periph_rate(struct udevice *dev, > > + int periph, > > + ulong rate) { > > + struct pmc_platdata *plat = dev_get_platdata(dev); > > + struct at91_pmc *pmc = plat->reg_base; > > + struct udevice *parent, *best_parent = NULL; > > + ulong tmp_rate, best_rate = rate, parent_rate; > > + int tmp_diff, best_diff = -1; > > + u8 num_parents, i; > > + u32 cells[GENERATED_SOURCE_MAX]; > > + u32 div, best_div = 0; > > + u8 best_parent_id = 0; > > + u32 tmp; > > + int ret; > > + > > + num_parents = fdtdec_get_int_array_count(gd->fdt_blob, dev->of_offset, > > + "clocks", cells, > > + > > + GENERATED_SOURCE_MAX); > > Are you able to store this value in your priv data in your > of_to_platdata() method? We should not be looking at the device tree outside that > method. Okay, move it to ofdata_to_platdata(). > > > + > > + for (i = 0; i < num_parents; i++) { > > + ret = clk_get_by_index(dev, i, &parent); > > + if (ret) > > + return ret; > > + > > + parent_rate = clk_get_rate(parent); > > + if (!parent_rate) > > + return -ENODEV; > > + > > + for (div = 1; div < GENERATED_MAX_DIV + 2; div++) { > > + tmp_rate = DIV_ROUND_CLOSEST(parent_rate, div); > > + if (rate < tmp_rate) > > + continue; > > + tmp_diff = rate - tmp_rate; > > + > > + if (best_diff < 0 || best_diff > tmp_diff) { > > + best_rate = tmp_rate; > > + best_diff = tmp_diff; > > + > > + best_div = div - 1; > > + best_parent = parent; > > + best_parent_id = i; > > + } > > + > > + if (!best_diff || tmp_rate < rate) > > + break; > > + } > > + > > + if (!best_diff) > > + break; > > + } > > + > > + dev_dbg(dev, "GCK: best parent: %s, best_rate = %ld, best_div = %d\n", > > + best_parent->name, best_rate, best_div); > > + > > + ret = clk_enable(best_parent, 0); > > + if (ret) > > + return -ENODEV; > > + > > + writel(periph & AT91_PMC_PCR_PID_MASK, &pmc->pcr); > > + tmp = readl(&pmc->pcr); > > + tmp &= ~(AT91_PMC_PCR_GCKDIV | AT91_PMC_PCR_GCKCSS); > > + tmp |= AT91_PMC_PCR_GCKCSS_(best_parent_id) | > > + AT91_PMC_PCR_CMD_WRITE | > > + AT91_PMC_PCR_GCKDIV_(best_div) | > > + AT91_PMC_PCR_GCKEN; > > + writel(tmp, &pmc->pcr); > > + > > + while (!(readl(&pmc->sr) & AT91_PMC_GCKRDY)) > > + ; > > + > > + return 0; > > +} > > + > > +static struct clk_ops generated_clk_ops = { > > + .get_periph_rate = generated_clk_get_periph_rate, > > + .set_periph_rate = generated_clk_set_periph_rate, }; > > + > > +static int generated_clk_probe(struct udevice *dev) { > > + return at91_pmc_core_probe(dev); } > > + > > +static const struct udevice_id generated_clk_match[] = { > > + { .compatible = "atmel,sama5d2-clk-generated" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(generated_clk) = { > > + .name = "generated-clk", > > + .id = UCLASS_CLK, > > + .of_match = generated_clk_match, > > + .probe = generated_clk_probe, > > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > > + .ops = &generated_clk_ops, > > +}; > > diff --git a/drivers/clk/at91/clk-h32mx.c > > b/drivers/clk/at91/clk-h32mx.c new file mode 100644 index > > 0000000..035562a > > --- /dev/null > > +++ b/drivers/clk/at91/clk-h32mx.c > > @@ -0,0 +1,56 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > +#include <dm/util.h> > > +#include <linux/io.h> > > +#include <mach/at91_pmc.h> > > +#include "pmc.h" > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +#define H32MX_MAX_FREQ 90000000 > > + > > +static ulong sama5d4_h32mx_clk_get_rate(struct udevice *dev) { > > + struct pmc_platdata *plat = dev_get_platdata(dev); > > + struct at91_pmc *pmc = plat->reg_base; > > + ulong rate = gd->arch.mck_rate_hz; > > + > > + if (readl(&pmc->mckr) & AT91_PMC_MCKR_H32MXDIV) > > + rate /= 2; > > + > > + if (rate > H32MX_MAX_FREQ) > > + dm_warn("H32MX clock is too fast\n"); > > + > > + return rate; > > +} > > + > > +static struct clk_ops sama5d4_h32mx_clk_ops = { > > + .get_rate = sama5d4_h32mx_clk_get_rate, }; > > + > > +static int sama5d4_h32mx_clk_probe(struct udevice *dev) { > > + return at91_pmc_core_probe(dev); } > > + > > +static const struct udevice_id sama5d4_h32mx_clk_match[] = { > > + { .compatible = "atmel,sama5d4-clk-h32mx" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(sama5d4_h32mx_clk) = { > > + .name = "sama5d4-h32mx-clk", > > + .id = UCLASS_CLK, > > + .of_match = sama5d4_h32mx_clk_match, > > + .probe = sama5d4_h32mx_clk_probe, > > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > > + .ops = &sama5d4_h32mx_clk_ops, }; > > diff --git a/drivers/clk/at91/clk-main.c b/drivers/clk/at91/clk-main.c > > new file mode 100644 index 0000000..10ab951 > > --- /dev/null > > +++ b/drivers/clk/at91/clk-main.c > > @@ -0,0 +1,55 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > +#include <linux/io.h> > > +#include <mach/at91_pmc.h> > > +#include "pmc.h" > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +static int main_osc_clk_enable(struct udevice *dev, int periph) { > > + struct pmc_platdata *plat = dev_get_platdata(dev); > > + struct at91_pmc *pmc = plat->reg_base; > > + > > + if (readl(&pmc->sr) & AT91_PMC_MOSCSELS) > > + return 0; > > + > > + return -ENODEV; > > This error indicates that there is no device. But it looks like there is. What are you > trying to represent here? It is supposed that the main oscillator clock should be enabled, if it is not enabled, it is an error, return -EINVAL? > > > +} > > + > > +static ulong main_osc_clk_get_rate(struct udevice *dev) { > > + return gd->arch.main_clk_rate_hz; } > > + > > +static struct clk_ops main_osc_clk_ops = { > > + .enable = main_osc_clk_enable, > > + .get_rate = main_osc_clk_get_rate, }; > > + > > +static int main_osc_clk_probe(struct udevice *dev) { > > + return at91_pmc_core_probe(dev); } > > + > > +static const struct udevice_id main_osc_clk_match[] = { > > + { .compatible = "atmel,at91sam9x5-clk-main" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(at91sam9x5_main_osc_clk) = { > > + .name = "at91sam9x5-main-osc-clk", > > + .id = UCLASS_CLK, > > + .of_match = main_osc_clk_match, > > + .probe = main_osc_clk_probe, > > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > > + .ops = &main_osc_clk_ops, > > +}; > > diff --git a/drivers/clk/at91/clk-master.c > > b/drivers/clk/at91/clk-master.c new file mode 100644 index > > 0000000..6d844d0 > > --- /dev/null > > +++ b/drivers/clk/at91/clk-master.c > > @@ -0,0 +1,33 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +static ulong at91_master_clk_get_rate(struct udevice *dev) { > > + return gd->arch.mck_rate_hz; > > +} > > + > > +static struct clk_ops at91_master_clk_ops = { > > + .get_rate = at91_master_clk_get_rate, }; > > + > > +static const struct udevice_id at91_master_clk_match[] = { > > + { .compatible = "atmel,at91sam9x5-clk-master" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(at91_master_clk) = { > > + .name = "at91-master-clk", > > + .id = UCLASS_CLK, > > + .of_match = at91_master_clk_match, > > + .ops = &at91_master_clk_ops, > > +}; > > diff --git a/drivers/clk/at91/clk-peripheral.c > > b/drivers/clk/at91/clk-peripheral.c > > new file mode 100644 > > index 0000000..4faeca5 > > --- /dev/null > > +++ b/drivers/clk/at91/clk-peripheral.c > > @@ -0,0 +1,68 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > +#include <linux/io.h> > > +#include <mach/at91_pmc.h> > > +#include "pmc.h" > > + > > +#define PERIPHERAL_MAX 64 > > + > > +#define PERIPHERAL_AT91RM9200 0 > > +#define PERIPHERAL_AT91SAM9X5 1 > > + > > +#define PERIPHERAL_ID_MIN 2 > > +#define PERIPHERAL_ID_MAX 31 > > +#define PERIPHERAL_MASK(id) (1 << ((id) & PERIPHERAL_ID_MAX)) > > + > > +#define PERIPHERAL_RSHIFT_MASK 0x3 > > +#define PERIPHERAL_RSHIFT(val) (((val) >> 16) & > > +PERIPHERAL_RSHIFT_MASK) > > + > > +#define PERIPHERAL_MAX_SHIFT 3 > > + > > +static int sam9x5_periph_clk_enable(struct udevice *dev, int periph) > > +{ > > + struct pmc_platdata *plat = dev_get_platdata(dev); > > + struct at91_pmc *pmc = plat->reg_base; > > + u32 tmp; > > + > > + if (periph < PERIPHERAL_ID_MIN) > > + return -1; > > + > > + writel(periph & AT91_PMC_PCR_PID_MASK, &pmc->pcr); > > + tmp = readl(&pmc->pcr); > > + writel(tmp | > > + AT91_PMC_PCR_CMD_WRITE | > > + AT91_PMC_PCR_EN, &pmc->pcr); > > Or use setbits_le32(&pmc->pcr, AT91_PMC_PCR_CMD_WRITE | > AT91_PMC_PCR_EN) Accepted. > > > + > > + return 0; > > +} > > + > > +static struct clk_ops sam9x5_periph_clk_ops = { > > + .enable = sam9x5_periph_clk_enable, }; > > + > > +static int sam9x5_periph_clk_probe(struct udevice *dev) { > > + return at91_pmc_core_probe(dev); } > > + > > +static const struct udevice_id sam9x5_periph_clk_match[] = { > > + { .compatible = "atmel,at91sam9x5-clk-peripheral" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(sam9x5_periph_clk) = { > > + .name = "sam9x5-periph-clk", > > + .id = UCLASS_CLK, > > + .of_match = sam9x5_periph_clk_match, > > + .probe = sam9x5_periph_clk_probe, > > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > > + .ops = &sam9x5_periph_clk_ops, }; > > diff --git a/drivers/clk/at91/clk-plla.c b/drivers/clk/at91/clk-plla.c > > new file mode 100644 index 0000000..3a2168e > > --- /dev/null > > +++ b/drivers/clk/at91/clk-plla.c > > @@ -0,0 +1,55 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > +#include <linux/io.h> > > +#include <mach/at91_pmc.h> > > +#include "pmc.h" > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +static int plla_clk_enable(struct udevice *dev, int periph) { > > + struct pmc_platdata *plat = dev_get_platdata(dev); > > + struct at91_pmc *pmc = plat->reg_base; > > + > > + if (readl(&pmc->sr) & AT91_PMC_LOCKA) > > + return 0; > > + > > + return -ENODEV; > > Same error question here. > > > +} > > + > > +static ulong plla_clk_get_rate(struct udevice *dev) { > > + return gd->arch.plla_rate_hz; > > +} > > + > > +static struct clk_ops plla_clk_ops = { > > + .enable = plla_clk_enable, > > + .get_rate = plla_clk_get_rate, }; > > + > > +static int plla_clk_probe(struct udevice *dev) { > > + return at91_pmc_core_probe(dev); } > > + > > +static const struct udevice_id plla_clk_match[] = { > > + { .compatible = "atmel,sama5d3-clk-pll" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(at91_plla_clk) = { > > + .name = "at91-plla-clk", > > + .id = UCLASS_CLK, > > + .of_match = plla_clk_match, > > + .probe = plla_clk_probe, > > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > > + .ops = &plla_clk_ops, > > +}; > > diff --git a/drivers/clk/at91/clk-slow.c b/drivers/clk/at91/clk-slow.c > > new file mode 100644 index 0000000..f73a553 > > --- /dev/null > > +++ b/drivers/clk/at91/clk-slow.c > > @@ -0,0 +1,37 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > + > > +static int at91_slow_clk_enable(struct udevice *dev, int periph) { > > + return 0; > > +} > > + > > +static ulong at91_slow_clk_get_rate(struct udevice *dev) { > > + return CONFIG_SYS_AT91_SLOW_CLOCK; } > > + > > +static struct clk_ops at91_slow_clk_ops = { > > + .enable = at91_slow_clk_enable, > > + .get_rate = at91_slow_clk_get_rate, }; > > + > > +static const struct udevice_id at91_slow_clk_match[] = { > > + { .compatible = "atmel,at91sam9x5-clk-slow" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(at91_slow_clk) = { > > + .name = "at91-slow-clk", > > + .id = UCLASS_CLK, > > + .of_match = at91_slow_clk_match, > > + .ops = &at91_slow_clk_ops, > > +}; > > diff --git a/drivers/clk/at91/clk-system.c > > b/drivers/clk/at91/clk-system.c new file mode 100644 index > > 0000000..5e50af6 > > --- /dev/null > > +++ b/drivers/clk/at91/clk-system.c > > @@ -0,0 +1,65 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > +#include <linux/io.h> > > +#include <mach/at91_pmc.h> > > +#include "pmc.h" > > + > > +#define SYSTEM_MAX_ID 31 > > + > > +static inline int is_pck(int id) > > +{ > > + return (id >= 8) && (id <= 15); } > > + > > +static int at91_system_clk_enable(struct udevice *dev, int id) { > > + struct pmc_platdata *plat = dev_get_platdata(dev); > > + struct at91_pmc *pmc = plat->reg_base; > > + u32 mask; > > + > > + if (id > SYSTEM_MAX_ID) > > + return -EINVAL; > > + > > + mask = BIT(id); > > + > > + writel(mask, &pmc->scer); > > + > > + if (!is_pck(id)) > > + return 0; > > Please can you add a comment to explain the above two lines? Accepted. > > > + > > + while (!(readl(&pmc->sr) & mask)) > > + ; > > + > > + return 0; > > +} > > + > > +static struct clk_ops at91_system_clk_ops = { > > + .enable = at91_system_clk_enable, }; > > + > > +static int at91_system_clk_probe(struct udevice *dev) { > > + return at91_pmc_core_probe(dev); } > > + > > +static const struct udevice_id at91_system_clk_match[] = { > > + { .compatible = "atmel,at91rm9200-clk-system" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(at91_system_clk) = { > > + .name = "at91-system-clk", > > + .id = UCLASS_CLK, > > + .of_match = at91_system_clk_match, > > + .probe = at91_system_clk_probe, > > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > > + .ops = &at91_system_clk_ops, > > +}; > > diff --git a/drivers/clk/at91/clk-utmi.c b/drivers/clk/at91/clk-utmi.c > > new file mode 100644 index 0000000..835c611 > > --- /dev/null > > +++ b/drivers/clk/at91/clk-utmi.c > > @@ -0,0 +1,67 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > +#include <linux/io.h> > > +#include <mach/at91_pmc.h> > > +#include "pmc.h" > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +#define UTMI_FIXED_MUL 40 > > + > > +static int utmi_clk_enable(struct udevice *dev, int periph) { > > + struct pmc_platdata *plat = dev_get_platdata(dev); > > + struct at91_pmc *pmc = plat->reg_base; > > + u32 tmp; > > + > > + if (readl(&pmc->sr) & AT91_PMC_LOCKU) > > + return 0; > > + > > + tmp = readl(&pmc->uckr); > > + tmp |= AT91_PMC_UPLLEN | > > + AT91_PMC_UPLLCOUNT | > > + AT91_PMC_BIASEN; > > + writel(tmp, &pmc->uckr); > > + > > + while (!(readl(&pmc->sr) & AT91_PMC_LOCKU)) > > + ; > > + > > + return 0; > > +} > > + > > +static ulong utmi_clk_get_rate(struct udevice *dev) { > > + return gd->arch.main_clk_rate_hz * UTMI_FIXED_MUL; } > > + > > +static struct clk_ops utmi_clk_ops = { > > + .enable = utmi_clk_enable, > > + .get_rate = utmi_clk_get_rate, }; > > + > > +static int utmi_clk_probe(struct udevice *dev) { > > + return at91_pmc_core_probe(dev); } > > + > > +static const struct udevice_id utmi_clk_match[] = { > > + { .compatible = "atmel,at91sam9x5-clk-utmi" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(at91sam9x5_utmi_clk) = { > > + .name = "at91sam9x5-utmi-clk", > > + .id = UCLASS_CLK, > > + .of_match = utmi_clk_match, > > + .probe = utmi_clk_probe, > > + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), > > + .ops = &utmi_clk_ops, > > +}; > > diff --git a/drivers/clk/at91/pmc.c b/drivers/clk/at91/pmc.c new file > > mode 100644 index 0000000..de802e8 > > --- /dev/null > > +++ b/drivers/clk/at91/pmc.c > > @@ -0,0 +1,49 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > +#include <dm/root.h> > > +#include "pmc.h" > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +static int at91_pmc_core_bind(struct udevice *dev) { > > + return dm_scan_fdt_node(dev, gd->fdt_blob, dev->of_offset, > > +false); } > > + > > +static const struct udevice_id at91_pmc_core_match[] = { > > + { .compatible = "atmel,sama5d2-pmc" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(at91_pmc_core) = { > > + .name = "at91_pmc_core", > > + .id = UCLASS_CLK, > > + .of_match = at91_pmc_core_match, > > + .bind = at91_pmc_core_bind, > > +}; > > + > > +int at91_pmc_core_probe(struct udevice *dev) { > > + struct pmc_platdata *plat = dev_get_platdata(dev); > > + fdt_addr_t addr; > > + > > + dev = dev_get_parent(dev); > > + if (!dev) > > + return -EINVAL; > > There will always be a parent, so this test is redundant. Accepted. > > > + > > + addr = dev_get_addr(dev); > > + if (addr == FDT_ADDR_T_NONE) > > + return -EINVAL; > > You can use dev_get_addr_ptr() here. Accepted. > > > + > > + plat->reg_base = (struct at91_pmc *)addr; > > + > > + return 0; > > +} > > diff --git a/drivers/clk/at91/pmc.h b/drivers/clk/at91/pmc.h new file > > mode 100644 index 0000000..201dad8 > > --- /dev/null > > +++ b/drivers/clk/at91/pmc.h > > @@ -0,0 +1,17 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#ifndef __AT91_PMC_H__ > > +#define __AT91_PMC_H__ > > + > > +struct pmc_platdata { > > + struct at91_pmc *reg_base; > > +}; > > + > > +int at91_pmc_core_probe(struct udevice *dev); > > + > > +#endif > > diff --git a/drivers/clk/at91/sckc.c b/drivers/clk/at91/sckc.c new > > file mode 100644 index 0000000..69ec636 > > --- /dev/null > > +++ b/drivers/clk/at91/sckc.c > > @@ -0,0 +1,30 @@ > > +/* > > + * Copyright (C) 2016 Atmel Corporation > > + * Wenyou.Yang <wenyou.yang@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <clk.h> > > +#include <dm/device.h> > > +#include <dm/root.h> > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +static int at91_sckc_clk_bind(struct udevice *dev) { > > + return dm_scan_fdt_node(dev, gd->fdt_blob, dev->of_offset, > > +false); } > > + > > +static const struct udevice_id at91_sckc_clk_match[] = { > > + { .compatible = "atmel,at91sam9x5-sckc" }, > > + {} > > +}; > > + > > +U_BOOT_DRIVER(at91_sckc_clk) = { > > + .name = "at91_sckc_clk", > > + .id = UCLASS_CLK, > > + .of_match = at91_sckc_clk_match, > > + .bind = at91_sckc_clk_bind, > > +}; > > -- > > 2.7.4 > > > > Regards, > Simon
diff --git a/arch/arm/mach-at91/include/mach/at91_pmc.h b/arch/arm/mach-at91/include/mach/at91_pmc.h index 7684f09..d02f3e1 100644 --- a/arch/arm/mach-at91/include/mach/at91_pmc.h +++ b/arch/arm/mach-at91/include/mach/at91_pmc.h @@ -149,6 +149,9 @@ typedef struct at91_pmc { #define AT91_PMC_PCR_PID_MASK (0x3f) #define AT91_PMC_PCR_GCKCSS (0x7 << 8) +#define AT91_PMC_PCR_GCKCSS_MASK 0x07 +#define AT91_PMC_PCR_GCKCSS_OFFSET 8 +#define AT91_PMC_PCR_GCKCSS_(x) ((x & 0x07) << 8) #define AT91_PMC_PCR_GCKCSS_SLOW_CLK (0x0 << 8) #define AT91_PMC_PCR_GCKCSS_MAIN_CLK (0x1 << 8) #define AT91_PMC_PCR_GCKCSS_PLLA_CLK (0x2 << 8) @@ -158,8 +161,9 @@ typedef struct at91_pmc { #define AT91_PMC_PCR_CMD_WRITE (0x1 << 12) #define AT91_PMC_PCR_DIV (0x3 << 16) #define AT91_PMC_PCR_GCKDIV (0xff << 20) -#define AT91_PMC_PCR_GCKDIV_(x) ((x & 0xff) << 20) -#define AT91_PMC_PCR_GCKDIV_OFFSET 20 +#define AT91_PMC_PCR_GCKDIV_MASK 0xff +#define AT91_PMC_PCR_GCKDIV_OFFSET 20 +#define AT91_PMC_PCR_GCKDIV_(x) ((x & 0xff) << 20) #define AT91_PMC_PCR_EN (0x1 << 28) #define AT91_PMC_PCR_GCKEN (0x1 << 29) @@ -243,8 +247,9 @@ typedef struct at91_pmc { #define AT91_PMC_PCK1RDY (1 << 9) /* Programmable Clock 1 */ #define AT91_PMC_PCK2RDY (1 << 10) /* Programmable Clock 2 */ #define AT91_PMC_PCK3RDY (1 << 11) /* Programmable Clock 3 */ +#define AT91_PMC_MOSCSELS BIT(16) /* Main Oscillator Selection Status */ +#define AT91_PMC_MOSCRCS BIT(17) /* 12 MHz RC Oscillator Status */ #define AT91_PMC_GCKRDY (1 << 24) - #define AT91_PMC_PROTKEY 0x504d4301 /* Activation Code */ /* PLL Charge Pump Current Register (PMC_PLLICPR) */ diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig index a98b74b..1fd1074 100644 --- a/drivers/clk/Kconfig +++ b/drivers/clk/Kconfig @@ -21,5 +21,6 @@ config SPL_CLK used as U-Boot proper. source "drivers/clk/uniphier/Kconfig" +source "drivers/clk/at91/Kconfig" endmenu diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile index c51db15..2bb0f35 100644 --- a/drivers/clk/Makefile +++ b/drivers/clk/Makefile @@ -11,3 +11,4 @@ obj-$(CONFIG_ROCKCHIP_RK3288) += clk_rk3288.o obj-$(CONFIG_SANDBOX) += clk_sandbox.o obj-$(CONFIG_MACH_PIC32) += clk_pic32.o obj-$(CONFIG_CLK_UNIPHIER) += uniphier/ +obj-$(CONFIG_CLK_AT91) += at91/ diff --git a/drivers/clk/at91/Kconfig b/drivers/clk/at91/Kconfig new file mode 100644 index 0000000..f9cb937 --- /dev/null +++ b/drivers/clk/at91/Kconfig @@ -0,0 +1,14 @@ +config CLK_AT91 + bool "AT91 clock drivers" + depends on CLK + help + This option is used to enable the AT91 clock driver. + +config HAVE_AT91_UTMI + bool + +config HAVE_AT91_H32MX + bool + +config HAVE_AT91_GENERATED_CLK + bool diff --git a/drivers/clk/at91/Makefile b/drivers/clk/at91/Makefile new file mode 100644 index 0000000..eb1326b --- /dev/null +++ b/drivers/clk/at91/Makefile @@ -0,0 +1,11 @@ +# +# Makefile for at91 specific clk +# + +obj-y += pmc.o sckc.o +obj-y += clk-slow.o clk-main.o clk-plla.o clk-master.o +obj-y += clk-system.o clk-peripheral.o + +obj-$(CONFIG_HAVE_AT91_UTMI) += clk-utmi.o +obj-$(CONFIG_HAVE_AT91_H32MX) += clk-h32mx.o +obj-$(CONFIG_HAVE_AT91_GENERATED_CLK) += clk-generated.o diff --git a/drivers/clk/at91/clk-generated.c b/drivers/clk/at91/clk-generated.c new file mode 100644 index 0000000..08e7a2f --- /dev/null +++ b/drivers/clk/at91/clk-generated.c @@ -0,0 +1,138 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> +#include <linux/io.h> +#include <mach/at91_pmc.h> +#include "pmc.h" + +DECLARE_GLOBAL_DATA_PTR; + +#define GENERATED_SOURCE_MAX 6 +#define GENERATED_MAX_DIV 255 + +static ulong generated_clk_get_periph_rate(struct udevice *dev, int periph) +{ + struct pmc_platdata *plat = dev_get_platdata(dev); + struct at91_pmc *pmc = plat->reg_base; + struct udevice *parent; + u32 tmp, gckdiv; + u8 parent_id; + int ret; + + writel(periph & AT91_PMC_PCR_PID_MASK, &pmc->pcr); + tmp = readl(&pmc->pcr); + parent_id = (tmp >> AT91_PMC_PCR_GCKCSS_OFFSET) & + AT91_PMC_PCR_GCKCSS_MASK; + gckdiv = (tmp >> AT91_PMC_PCR_GCKDIV_OFFSET) & AT91_PMC_PCR_GCKDIV_MASK; + + ret = clk_get_by_index(dev, parent_id, &parent); + if (ret) + return 0; + + return clk_get_rate(parent) / (gckdiv + 1); +} + +static ulong generated_clk_set_periph_rate(struct udevice *dev, + int periph, + ulong rate) +{ + struct pmc_platdata *plat = dev_get_platdata(dev); + struct at91_pmc *pmc = plat->reg_base; + struct udevice *parent, *best_parent = NULL; + ulong tmp_rate, best_rate = rate, parent_rate; + int tmp_diff, best_diff = -1; + u8 num_parents, i; + u32 cells[GENERATED_SOURCE_MAX]; + u32 div, best_div = 0; + u8 best_parent_id = 0; + u32 tmp; + int ret; + + num_parents = fdtdec_get_int_array_count(gd->fdt_blob, dev->of_offset, + "clocks", cells, + GENERATED_SOURCE_MAX); + + for (i = 0; i < num_parents; i++) { + ret = clk_get_by_index(dev, i, &parent); + if (ret) + return ret; + + parent_rate = clk_get_rate(parent); + if (!parent_rate) + return -ENODEV; + + for (div = 1; div < GENERATED_MAX_DIV + 2; div++) { + tmp_rate = DIV_ROUND_CLOSEST(parent_rate, div); + if (rate < tmp_rate) + continue; + tmp_diff = rate - tmp_rate; + + if (best_diff < 0 || best_diff > tmp_diff) { + best_rate = tmp_rate; + best_diff = tmp_diff; + + best_div = div - 1; + best_parent = parent; + best_parent_id = i; + } + + if (!best_diff || tmp_rate < rate) + break; + } + + if (!best_diff) + break; + } + + dev_dbg(dev, "GCK: best parent: %s, best_rate = %ld, best_div = %d\n", + best_parent->name, best_rate, best_div); + + ret = clk_enable(best_parent, 0); + if (ret) + return -ENODEV; + + writel(periph & AT91_PMC_PCR_PID_MASK, &pmc->pcr); + tmp = readl(&pmc->pcr); + tmp &= ~(AT91_PMC_PCR_GCKDIV | AT91_PMC_PCR_GCKCSS); + tmp |= AT91_PMC_PCR_GCKCSS_(best_parent_id) | + AT91_PMC_PCR_CMD_WRITE | + AT91_PMC_PCR_GCKDIV_(best_div) | + AT91_PMC_PCR_GCKEN; + writel(tmp, &pmc->pcr); + + while (!(readl(&pmc->sr) & AT91_PMC_GCKRDY)) + ; + + return 0; +} + +static struct clk_ops generated_clk_ops = { + .get_periph_rate = generated_clk_get_periph_rate, + .set_periph_rate = generated_clk_set_periph_rate, +}; + +static int generated_clk_probe(struct udevice *dev) +{ + return at91_pmc_core_probe(dev); +} + +static const struct udevice_id generated_clk_match[] = { + { .compatible = "atmel,sama5d2-clk-generated" }, + {} +}; + +U_BOOT_DRIVER(generated_clk) = { + .name = "generated-clk", + .id = UCLASS_CLK, + .of_match = generated_clk_match, + .probe = generated_clk_probe, + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), + .ops = &generated_clk_ops, +}; diff --git a/drivers/clk/at91/clk-h32mx.c b/drivers/clk/at91/clk-h32mx.c new file mode 100644 index 0000000..035562a --- /dev/null +++ b/drivers/clk/at91/clk-h32mx.c @@ -0,0 +1,56 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> +#include <dm/util.h> +#include <linux/io.h> +#include <mach/at91_pmc.h> +#include "pmc.h" + +DECLARE_GLOBAL_DATA_PTR; + +#define H32MX_MAX_FREQ 90000000 + +static ulong sama5d4_h32mx_clk_get_rate(struct udevice *dev) +{ + struct pmc_platdata *plat = dev_get_platdata(dev); + struct at91_pmc *pmc = plat->reg_base; + ulong rate = gd->arch.mck_rate_hz; + + if (readl(&pmc->mckr) & AT91_PMC_MCKR_H32MXDIV) + rate /= 2; + + if (rate > H32MX_MAX_FREQ) + dm_warn("H32MX clock is too fast\n"); + + return rate; +} + +static struct clk_ops sama5d4_h32mx_clk_ops = { + .get_rate = sama5d4_h32mx_clk_get_rate, +}; + +static int sama5d4_h32mx_clk_probe(struct udevice *dev) +{ + return at91_pmc_core_probe(dev); +} + +static const struct udevice_id sama5d4_h32mx_clk_match[] = { + { .compatible = "atmel,sama5d4-clk-h32mx" }, + {} +}; + +U_BOOT_DRIVER(sama5d4_h32mx_clk) = { + .name = "sama5d4-h32mx-clk", + .id = UCLASS_CLK, + .of_match = sama5d4_h32mx_clk_match, + .probe = sama5d4_h32mx_clk_probe, + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), + .ops = &sama5d4_h32mx_clk_ops, +}; diff --git a/drivers/clk/at91/clk-main.c b/drivers/clk/at91/clk-main.c new file mode 100644 index 0000000..10ab951 --- /dev/null +++ b/drivers/clk/at91/clk-main.c @@ -0,0 +1,55 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> +#include <linux/io.h> +#include <mach/at91_pmc.h> +#include "pmc.h" + +DECLARE_GLOBAL_DATA_PTR; + +static int main_osc_clk_enable(struct udevice *dev, int periph) +{ + struct pmc_platdata *plat = dev_get_platdata(dev); + struct at91_pmc *pmc = plat->reg_base; + + if (readl(&pmc->sr) & AT91_PMC_MOSCSELS) + return 0; + + return -ENODEV; +} + +static ulong main_osc_clk_get_rate(struct udevice *dev) +{ + return gd->arch.main_clk_rate_hz; +} + +static struct clk_ops main_osc_clk_ops = { + .enable = main_osc_clk_enable, + .get_rate = main_osc_clk_get_rate, +}; + +static int main_osc_clk_probe(struct udevice *dev) +{ + return at91_pmc_core_probe(dev); +} + +static const struct udevice_id main_osc_clk_match[] = { + { .compatible = "atmel,at91sam9x5-clk-main" }, + {} +}; + +U_BOOT_DRIVER(at91sam9x5_main_osc_clk) = { + .name = "at91sam9x5-main-osc-clk", + .id = UCLASS_CLK, + .of_match = main_osc_clk_match, + .probe = main_osc_clk_probe, + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), + .ops = &main_osc_clk_ops, +}; diff --git a/drivers/clk/at91/clk-master.c b/drivers/clk/at91/clk-master.c new file mode 100644 index 0000000..6d844d0 --- /dev/null +++ b/drivers/clk/at91/clk-master.c @@ -0,0 +1,33 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> + +DECLARE_GLOBAL_DATA_PTR; + +static ulong at91_master_clk_get_rate(struct udevice *dev) +{ + return gd->arch.mck_rate_hz; +} + +static struct clk_ops at91_master_clk_ops = { + .get_rate = at91_master_clk_get_rate, +}; + +static const struct udevice_id at91_master_clk_match[] = { + { .compatible = "atmel,at91sam9x5-clk-master" }, + {} +}; + +U_BOOT_DRIVER(at91_master_clk) = { + .name = "at91-master-clk", + .id = UCLASS_CLK, + .of_match = at91_master_clk_match, + .ops = &at91_master_clk_ops, +}; diff --git a/drivers/clk/at91/clk-peripheral.c b/drivers/clk/at91/clk-peripheral.c new file mode 100644 index 0000000..4faeca5 --- /dev/null +++ b/drivers/clk/at91/clk-peripheral.c @@ -0,0 +1,68 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> +#include <linux/io.h> +#include <mach/at91_pmc.h> +#include "pmc.h" + +#define PERIPHERAL_MAX 64 + +#define PERIPHERAL_AT91RM9200 0 +#define PERIPHERAL_AT91SAM9X5 1 + +#define PERIPHERAL_ID_MIN 2 +#define PERIPHERAL_ID_MAX 31 +#define PERIPHERAL_MASK(id) (1 << ((id) & PERIPHERAL_ID_MAX)) + +#define PERIPHERAL_RSHIFT_MASK 0x3 +#define PERIPHERAL_RSHIFT(val) (((val) >> 16) & PERIPHERAL_RSHIFT_MASK) + +#define PERIPHERAL_MAX_SHIFT 3 + +static int sam9x5_periph_clk_enable(struct udevice *dev, int periph) +{ + struct pmc_platdata *plat = dev_get_platdata(dev); + struct at91_pmc *pmc = plat->reg_base; + u32 tmp; + + if (periph < PERIPHERAL_ID_MIN) + return -1; + + writel(periph & AT91_PMC_PCR_PID_MASK, &pmc->pcr); + tmp = readl(&pmc->pcr); + writel(tmp | + AT91_PMC_PCR_CMD_WRITE | + AT91_PMC_PCR_EN, &pmc->pcr); + + return 0; +} + +static struct clk_ops sam9x5_periph_clk_ops = { + .enable = sam9x5_periph_clk_enable, +}; + +static int sam9x5_periph_clk_probe(struct udevice *dev) +{ + return at91_pmc_core_probe(dev); +} + +static const struct udevice_id sam9x5_periph_clk_match[] = { + { .compatible = "atmel,at91sam9x5-clk-peripheral" }, + {} +}; + +U_BOOT_DRIVER(sam9x5_periph_clk) = { + .name = "sam9x5-periph-clk", + .id = UCLASS_CLK, + .of_match = sam9x5_periph_clk_match, + .probe = sam9x5_periph_clk_probe, + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), + .ops = &sam9x5_periph_clk_ops, +}; diff --git a/drivers/clk/at91/clk-plla.c b/drivers/clk/at91/clk-plla.c new file mode 100644 index 0000000..3a2168e --- /dev/null +++ b/drivers/clk/at91/clk-plla.c @@ -0,0 +1,55 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> +#include <linux/io.h> +#include <mach/at91_pmc.h> +#include "pmc.h" + +DECLARE_GLOBAL_DATA_PTR; + +static int plla_clk_enable(struct udevice *dev, int periph) +{ + struct pmc_platdata *plat = dev_get_platdata(dev); + struct at91_pmc *pmc = plat->reg_base; + + if (readl(&pmc->sr) & AT91_PMC_LOCKA) + return 0; + + return -ENODEV; +} + +static ulong plla_clk_get_rate(struct udevice *dev) +{ + return gd->arch.plla_rate_hz; +} + +static struct clk_ops plla_clk_ops = { + .enable = plla_clk_enable, + .get_rate = plla_clk_get_rate, +}; + +static int plla_clk_probe(struct udevice *dev) +{ + return at91_pmc_core_probe(dev); +} + +static const struct udevice_id plla_clk_match[] = { + { .compatible = "atmel,sama5d3-clk-pll" }, + {} +}; + +U_BOOT_DRIVER(at91_plla_clk) = { + .name = "at91-plla-clk", + .id = UCLASS_CLK, + .of_match = plla_clk_match, + .probe = plla_clk_probe, + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), + .ops = &plla_clk_ops, +}; diff --git a/drivers/clk/at91/clk-slow.c b/drivers/clk/at91/clk-slow.c new file mode 100644 index 0000000..f73a553 --- /dev/null +++ b/drivers/clk/at91/clk-slow.c @@ -0,0 +1,37 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> + +static int at91_slow_clk_enable(struct udevice *dev, int periph) +{ + return 0; +} + +static ulong at91_slow_clk_get_rate(struct udevice *dev) +{ + return CONFIG_SYS_AT91_SLOW_CLOCK; +} + +static struct clk_ops at91_slow_clk_ops = { + .enable = at91_slow_clk_enable, + .get_rate = at91_slow_clk_get_rate, +}; + +static const struct udevice_id at91_slow_clk_match[] = { + { .compatible = "atmel,at91sam9x5-clk-slow" }, + {} +}; + +U_BOOT_DRIVER(at91_slow_clk) = { + .name = "at91-slow-clk", + .id = UCLASS_CLK, + .of_match = at91_slow_clk_match, + .ops = &at91_slow_clk_ops, +}; diff --git a/drivers/clk/at91/clk-system.c b/drivers/clk/at91/clk-system.c new file mode 100644 index 0000000..5e50af6 --- /dev/null +++ b/drivers/clk/at91/clk-system.c @@ -0,0 +1,65 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> +#include <linux/io.h> +#include <mach/at91_pmc.h> +#include "pmc.h" + +#define SYSTEM_MAX_ID 31 + +static inline int is_pck(int id) +{ + return (id >= 8) && (id <= 15); +} + +static int at91_system_clk_enable(struct udevice *dev, int id) +{ + struct pmc_platdata *plat = dev_get_platdata(dev); + struct at91_pmc *pmc = plat->reg_base; + u32 mask; + + if (id > SYSTEM_MAX_ID) + return -EINVAL; + + mask = BIT(id); + + writel(mask, &pmc->scer); + + if (!is_pck(id)) + return 0; + + while (!(readl(&pmc->sr) & mask)) + ; + + return 0; +} + +static struct clk_ops at91_system_clk_ops = { + .enable = at91_system_clk_enable, +}; + +static int at91_system_clk_probe(struct udevice *dev) +{ + return at91_pmc_core_probe(dev); +} + +static const struct udevice_id at91_system_clk_match[] = { + { .compatible = "atmel,at91rm9200-clk-system" }, + {} +}; + +U_BOOT_DRIVER(at91_system_clk) = { + .name = "at91-system-clk", + .id = UCLASS_CLK, + .of_match = at91_system_clk_match, + .probe = at91_system_clk_probe, + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), + .ops = &at91_system_clk_ops, +}; diff --git a/drivers/clk/at91/clk-utmi.c b/drivers/clk/at91/clk-utmi.c new file mode 100644 index 0000000..835c611 --- /dev/null +++ b/drivers/clk/at91/clk-utmi.c @@ -0,0 +1,67 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> +#include <linux/io.h> +#include <mach/at91_pmc.h> +#include "pmc.h" + +DECLARE_GLOBAL_DATA_PTR; + +#define UTMI_FIXED_MUL 40 + +static int utmi_clk_enable(struct udevice *dev, int periph) +{ + struct pmc_platdata *plat = dev_get_platdata(dev); + struct at91_pmc *pmc = plat->reg_base; + u32 tmp; + + if (readl(&pmc->sr) & AT91_PMC_LOCKU) + return 0; + + tmp = readl(&pmc->uckr); + tmp |= AT91_PMC_UPLLEN | + AT91_PMC_UPLLCOUNT | + AT91_PMC_BIASEN; + writel(tmp, &pmc->uckr); + + while (!(readl(&pmc->sr) & AT91_PMC_LOCKU)) + ; + + return 0; +} + +static ulong utmi_clk_get_rate(struct udevice *dev) +{ + return gd->arch.main_clk_rate_hz * UTMI_FIXED_MUL; +} + +static struct clk_ops utmi_clk_ops = { + .enable = utmi_clk_enable, + .get_rate = utmi_clk_get_rate, +}; + +static int utmi_clk_probe(struct udevice *dev) +{ + return at91_pmc_core_probe(dev); +} + +static const struct udevice_id utmi_clk_match[] = { + { .compatible = "atmel,at91sam9x5-clk-utmi" }, + {} +}; + +U_BOOT_DRIVER(at91sam9x5_utmi_clk) = { + .name = "at91sam9x5-utmi-clk", + .id = UCLASS_CLK, + .of_match = utmi_clk_match, + .probe = utmi_clk_probe, + .platdata_auto_alloc_size = sizeof(struct pmc_platdata), + .ops = &utmi_clk_ops, +}; diff --git a/drivers/clk/at91/pmc.c b/drivers/clk/at91/pmc.c new file mode 100644 index 0000000..de802e8 --- /dev/null +++ b/drivers/clk/at91/pmc.c @@ -0,0 +1,49 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> +#include <dm/root.h> +#include "pmc.h" + +DECLARE_GLOBAL_DATA_PTR; + +static int at91_pmc_core_bind(struct udevice *dev) +{ + return dm_scan_fdt_node(dev, gd->fdt_blob, dev->of_offset, false); +} + +static const struct udevice_id at91_pmc_core_match[] = { + { .compatible = "atmel,sama5d2-pmc" }, + {} +}; + +U_BOOT_DRIVER(at91_pmc_core) = { + .name = "at91_pmc_core", + .id = UCLASS_CLK, + .of_match = at91_pmc_core_match, + .bind = at91_pmc_core_bind, +}; + +int at91_pmc_core_probe(struct udevice *dev) +{ + struct pmc_platdata *plat = dev_get_platdata(dev); + fdt_addr_t addr; + + dev = dev_get_parent(dev); + if (!dev) + return -EINVAL; + + addr = dev_get_addr(dev); + if (addr == FDT_ADDR_T_NONE) + return -EINVAL; + + plat->reg_base = (struct at91_pmc *)addr; + + return 0; +} diff --git a/drivers/clk/at91/pmc.h b/drivers/clk/at91/pmc.h new file mode 100644 index 0000000..201dad8 --- /dev/null +++ b/drivers/clk/at91/pmc.h @@ -0,0 +1,17 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#ifndef __AT91_PMC_H__ +#define __AT91_PMC_H__ + +struct pmc_platdata { + struct at91_pmc *reg_base; +}; + +int at91_pmc_core_probe(struct udevice *dev); + +#endif diff --git a/drivers/clk/at91/sckc.c b/drivers/clk/at91/sckc.c new file mode 100644 index 0000000..69ec636 --- /dev/null +++ b/drivers/clk/at91/sckc.c @@ -0,0 +1,30 @@ +/* + * Copyright (C) 2016 Atmel Corporation + * Wenyou.Yang <wenyou.yang@atmel.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <clk.h> +#include <dm/device.h> +#include <dm/root.h> + +DECLARE_GLOBAL_DATA_PTR; + +static int at91_sckc_clk_bind(struct udevice *dev) +{ + return dm_scan_fdt_node(dev, gd->fdt_blob, dev->of_offset, false); +} + +static const struct udevice_id at91_sckc_clk_match[] = { + { .compatible = "atmel,at91sam9x5-sckc" }, + {} +}; + +U_BOOT_DRIVER(at91_sckc_clk) = { + .name = "at91_sckc_clk", + .id = UCLASS_CLK, + .of_match = at91_sckc_clk_match, + .bind = at91_sckc_clk_bind, +};
The patch is referred to at91 clock driver of Linux, to make the clock node descriptions in dt are aligned with the Linux. Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com> --- Changes in v2: - Remove the redundant log print. arch/arm/mach-at91/include/mach/at91_pmc.h | 11 ++- drivers/clk/Kconfig | 1 + drivers/clk/Makefile | 1 + drivers/clk/at91/Kconfig | 14 +++ drivers/clk/at91/Makefile | 11 +++ drivers/clk/at91/clk-generated.c | 138 +++++++++++++++++++++++++++++ drivers/clk/at91/clk-h32mx.c | 56 ++++++++++++ drivers/clk/at91/clk-main.c | 55 ++++++++++++ drivers/clk/at91/clk-master.c | 33 +++++++ drivers/clk/at91/clk-peripheral.c | 68 ++++++++++++++ drivers/clk/at91/clk-plla.c | 55 ++++++++++++ drivers/clk/at91/clk-slow.c | 37 ++++++++ drivers/clk/at91/clk-system.c | 65 ++++++++++++++ drivers/clk/at91/clk-utmi.c | 67 ++++++++++++++ drivers/clk/at91/pmc.c | 49 ++++++++++ drivers/clk/at91/pmc.h | 17 ++++ drivers/clk/at91/sckc.c | 30 +++++++ 17 files changed, 705 insertions(+), 3 deletions(-) create mode 100644 drivers/clk/at91/Kconfig create mode 100644 drivers/clk/at91/Makefile create mode 100644 drivers/clk/at91/clk-generated.c create mode 100644 drivers/clk/at91/clk-h32mx.c create mode 100644 drivers/clk/at91/clk-main.c create mode 100644 drivers/clk/at91/clk-master.c create mode 100644 drivers/clk/at91/clk-peripheral.c create mode 100644 drivers/clk/at91/clk-plla.c create mode 100644 drivers/clk/at91/clk-slow.c create mode 100644 drivers/clk/at91/clk-system.c create mode 100644 drivers/clk/at91/clk-utmi.c create mode 100644 drivers/clk/at91/pmc.c create mode 100644 drivers/clk/at91/pmc.h create mode 100644 drivers/clk/at91/sckc.c