在 2017-05-29 15:34,Chen-Yu Tsai 写道:
Hi,

On Sat, May 27, 2017 at 06:23:06PM +0800, Icenowy Zheng wrote:
Allwinner R40 SoC have a clock controller module in the style of the
SoCs beyond sun6i, however, it's more rich and complex.

Add support for it.

Signed-off-by: Icenowy Zheng <icen...@aosc.io>
---
Changes in v3:
- Rebased on current linux-next.
Changes in v2:
- Fixes according to the SoC's user manual.

 drivers/clk/sunxi-ng/Kconfig              |   10 +
 drivers/clk/sunxi-ng/Makefile             |    1 +
drivers/clk/sunxi-ng/ccu-sun8i-r40.c | 1153 +++++++++++++++++++++++++++++
 drivers/clk/sunxi-ng/ccu-sun8i-r40.h      |   68 ++
 include/dt-bindings/clock/sun8i-r40-ccu.h |  191 +++++
 include/dt-bindings/reset/sun8i-r40-ccu.h |  129 ++++
 6 files changed, 1552 insertions(+)
 create mode 100644 drivers/clk/sunxi-ng/ccu-sun8i-r40.c
 create mode 100644 drivers/clk/sunxi-ng/ccu-sun8i-r40.h
 create mode 100644 include/dt-bindings/clock/sun8i-r40-ccu.h
 create mode 100644 include/dt-bindings/reset/sun8i-r40-ccu.h

diff --git a/drivers/clk/sunxi-ng/Kconfig b/drivers/clk/sunxi-ng/Kconfig
index 67acef3d2494..c11ad3375907 100644
--- a/drivers/clk/sunxi-ng/Kconfig
+++ b/drivers/clk/sunxi-ng/Kconfig
@@ -157,6 +157,16 @@ config SUN8I_DE2_CCU
        select SUNXI_CCU_DIV
        select SUNXI_CCU_GATE

+config SUN8I_R40_CCU
+       bool "Support for the Allwinner R40 CCU"
+       select SUNXI_CCU_DIV
+       select SUNXI_CCU_NK
+       select SUNXI_CCU_NKM
+       select SUNXI_CCU_NKMP
+       select SUNXI_CCU_NM
+       select SUNXI_CCU_MP

You seem to be missing a few, such as CCU_GATE, CCU_MUX, CCU_PHASE.
The list should at least be the same as the included headers.

+       default MACH_SUN8I
+
 config SUN9I_A80_CCU
        bool "Support for the Allwinner A80 CCU"
        select SUNXI_CCU_DIV
diff --git a/drivers/clk/sunxi-ng/Makefile b/drivers/clk/sunxi-ng/Makefile
index 0185c6ffadcb..217db667a994 100644
--- a/drivers/clk/sunxi-ng/Makefile
+++ b/drivers/clk/sunxi-ng/Makefile
@@ -28,6 +28,7 @@ obj-$(CONFIG_SUN8I_H3_CCU)    += ccu-sun8i-h3.o
 obj-$(CONFIG_SUN8I_V3S_CCU)    += ccu-sun8i-v3s.o
 obj-$(CONFIG_SUN8I_DE2_CCU)    += ccu-sun8i-de2.o
 obj-$(CONFIG_SUN8I_R_CCU)      += ccu-sun8i-r.o
+obj-$(CONFIG_SUN8I_R40_CCU)    += ccu-sun8i-r40.o
 obj-$(CONFIG_SUN9I_A80_CCU)    += ccu-sun9i-a80.o
 obj-$(CONFIG_SUN9I_A80_CCU)    += ccu-sun9i-a80-de.o
 obj-$(CONFIG_SUN9I_A80_CCU)    += ccu-sun9i-a80-usb.o
diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-r40.c b/drivers/clk/sunxi-ng/ccu-sun8i-r40.c
new file mode 100644
index 000000000000..484a8956f59c
--- /dev/null
+++ b/drivers/clk/sunxi-ng/ccu-sun8i-r40.c
@@ -0,0 +1,1153 @@
+/*
+ * Copyright (c) 2017 Icenowy Zheng <icen...@aosc.io>
+ *
+ * This software is licensed under the terms of the GNU General Public + * License version 2, as published by the Free Software Foundation, and
+ * may be copied, distributed, and modified under those terms.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include <linux/clk-provider.h>
+#include <linux/of_address.h>
+
+#include "ccu_common.h"
+#include "ccu_reset.h"
+
+#include "ccu_div.h"
+#include "ccu_gate.h"
+#include "ccu_mp.h"
+#include "ccu_mult.h"
+#include "ccu_nk.h"
+#include "ccu_nkm.h"
+#include "ccu_nkmp.h"
+#include "ccu_nm.h"
+#include "ccu_phase.h"
+
+#include "ccu-sun8i-r40.h"
+
+static SUNXI_CCU_NKMP_WITH_GATE_LOCK(pll_cpu_clk, "pll-cpu",
+                                    "osc24M", 0x000,
+                                    8, 5,      /* N */
+                                    4, 2,      /* K */
+                                    0, 2,      /* M */
+                                    16, 2,     /* P */

P does not do /8. Please add a limit.

+                                    BIT(31),   /* gate */
+                                    BIT(28),   /* lock */
+                                    0);
+
+/*
+ * The Audio PLL is supposed to have 4 outputs: 3 fixed factors from
+ * the base (2x, 4x and 8x), and one variable divider (the one true
+ * pll audio).
+ *
+ * We don't have any need for the variable divider for now, so we just
+ * hardcode it to match with the clock names
+ */
+#define SUN8I_R40_PLL_AUDIO_REG        0x008
+
+static SUNXI_CCU_NM_WITH_GATE_LOCK(pll_audio_base_clk, "pll-audio-base",
+                                  "osc24M", 0x008,
+                                  8, 7,        /* N */
+                                  0, 5,        /* M */
+                                  BIT(31),     /* gate */
+                                  BIT(28),     /* lock */
+                                  0);
+
+static SUNXI_CCU_NM_WITH_FRAC_GATE_LOCK(pll_video0_clk, "pll-video0",
+                                       "osc24M", 0x0010,
+                                       8, 7,           /* N */
+                                       0, 4,           /* M */
+                                       BIT(24),        /* frac enable */
+                                       BIT(25),        /* frac select */
+                                       270000000,      /* frac rate 0 */
+                                       297000000,      /* frac rate 1 */
+                                       BIT(31),        /* gate */
+                                       BIT(28),        /* lock */
+                                       0);
+
+static SUNXI_CCU_NM_WITH_FRAC_GATE_LOCK(pll_ve_clk, "pll-ve",
+                                       "osc24M", 0x0018,
+                                       8, 7,           /* N */
+                                       0, 4,           /* M */
+                                       BIT(24),        /* frac enable */
+                                       BIT(25),        /* frac select */
+                                       270000000,      /* frac rate 0 */
+                                       297000000,      /* frac rate 1 */
+                                       BIT(31),        /* gate */
+                                       BIT(28),        /* lock */
+                                       0);
+
+static SUNXI_CCU_NKM_WITH_GATE_LOCK(pll_ddr0_clk, "pll-ddr0",
+                                   "osc24M", 0x020,
+                                   8, 5,       /* N */
+                                   4, 2,       /* K */
+                                   0, 2,       /* M */
+                                   BIT(31),    /* gate */
+                                   BIT(28),    /* lock */
+                                   0);
+
+/* According to the BSP driver, pll-periph{0,1} have M at 0:1 */

This is also mentioned in the user manual. You can leave it out.

Also, please leave a note mentioning that support for the SATA clock
output of the peripheral PLL is not implemented here.

+static SUNXI_CCU_NKM_WITH_GATE_LOCK(pll_periph0_clk, "pll-periph0",
+                                   "osc24M", 0x028,
+                                   8, 5,       /* N */
+                                   4, 2,       /* K */
+                                   0, 2,       /* M */
+                                   BIT(31),    /* gate */
+                                   BIT(28),    /* lock */
+                                   0);
+
+static SUNXI_CCU_NKM_WITH_GATE_LOCK(pll_periph1_clk, "pll-periph1",
+                                   "osc24M", 0x02c,
+                                   8, 5,       /* N */
+                                   4, 2,       /* K */
+                                   0, 2,       /* M */
+                                   BIT(31),    /* gate */
+                                   BIT(28),    /* lock */
+                                   0);
+
+static SUNXI_CCU_NM_WITH_FRAC_GATE_LOCK(pll_video1_clk, "pll-video1",
+                                       "osc24M", 0x030,
+                                       8, 7,           /* N */
+                                       0, 4,           /* M */
+                                       BIT(24),        /* frac enable */
+                                       BIT(25),        /* frac select */
+                                       270000000,      /* frac rate 0 */
+                                       297000000,      /* frac rate 1 */
+                                       BIT(31),        /* gate */
+                                       BIT(28),        /* lock */
+                                       CLK_SET_RATE_UNGATE);

Why do you have this set for only some of the PLLs? Either you need it
for all of them, or you don't need it for any of them.

+
+/*
+ * For the special bit in gate part, please see the BSP source code at + * https://github.com/BPI-SINOVOIP/BPI-M2U-bsp/blob/master/linux-sunxi/drivers/clk/sunxi/clk-sun8iw11.c#L665
+ */
+static SUNXI_CCU_NKM_WITH_GATE_LOCK(pll_sata_clk, "pll-sata",
+                                   "osc24M", 0x034,
+                                   8, 5,       /* N */
+                                   4, 2,       /* K */
+                                   0, 2,       /* M */
+                                   BIT(31) | BIT(14),  /* gate */
+                                   BIT(28),    /* lock */
+                                   0);

I think this is a somewhat simplified approach. From what I understand
of the user manual, the SATA clock path look like:


[ PLL-PERIPH0-SATA ] -\
                      mux @ 0x34 bit 30 --- gate @ 0x34 bit 14 --- ...
[ PLL-SATA ] ---------/

... from above ... ------\
                         mux @ 0xc8 bit 24 --- gate @ 0xc8 bit 31
[ external oscillator ] -/

If you choose to simplify the implementation, please include a detailed
note as to why you chose to do so, and the validity of the simplification.

I think it can be fully implemented...

But how should I call the internal clock controlled by the mux @ 0x34 bit 30?


+
+static SUNXI_CCU_NM_WITH_FRAC_GATE_LOCK(pll_gpu_clk, "pll-gpu",
+                                       "osc24M", 0x038,
+                                       8, 7,           /* N */
+                                       0, 4,           /* M */
+                                       BIT(24),        /* frac enable */
+                                       BIT(25),        /* frac select */
+                                       270000000,      /* frac rate 0 */
+                                       297000000,      /* frac rate 1 */
+                                       BIT(31),        /* gate */
+                                       BIT(28),        /* lock */
+                                       0);
+
+static const char * const pll_mipi_parents[] = { "pll-video0", "pll-video1" };

pll-video1 is not listed in the manual.

+static SUNXI_CCU_NKM_WITH_MUX_GATE_LOCK(pll_mipi_clk, "pll-mipi",
+                                       pll_mipi_parents, 0x040,
+                                       8, 4,   /* N */
+                                       4, 2,   /* K */
+                                       0, 4,   /* M */
+                                       21, 0,  /* mux */
+                                       BIT(31) | BIT(23) | BIT(22), /* gate */
+                                       BIT(28),        /* lock */
+                                       CLK_SET_RATE_UNGATE);

Please leave a note mentioning that you only implement support for the
"MIPI" mode.
You should also force it at probe time. See the driver for the A31.

+
+static SUNXI_CCU_NM_WITH_FRAC_GATE_LOCK(pll_de_clk, "pll-de",
+                                       "osc24M", 0x0048,
+                                       8, 7,           /* N */
+                                       0, 4,           /* M */
+                                       BIT(24),        /* frac enable */
+                                       BIT(25),        /* frac select */
+                                       270000000,      /* frac rate 0 */
+                                       297000000,      /* frac rate 1 */
+                                       BIT(31),        /* gate */
+                                       BIT(28),        /* lock */
+                                       0);
+
+static SUNXI_CCU_NM_WITH_GATE_LOCK(pll_ddr1_clk, "pll-ddr1",
+                                  "osc24M", 0x04c,
+                                  8, 7,        /* N */

N has minimum and maximum limits.

+                                  0, 2,        /* M */
+                                  BIT(31),     /* gate */
+                                  BIT(28),     /* lock */
+                                  CLK_SET_RATE_UNGATE);
+
+static const char * const cpu_parents[] = { "osc32k", "osc24M",
+                                            "pll-cpu", "pll-cpu" };
+static SUNXI_CCU_MUX(cpu_clk, "cpu", cpu_parents,
+                    0x050, 16, 2, CLK_IS_CRITICAL);
+
+static SUNXI_CCU_M(axi_clk, "axi", "cpu", 0x050, 0, 2, 0);
+
+static const char * const ahb1_parents[] = { "osc32k", "osc24M",
+                                            "axi", "pll-periph0" };
+static const struct ccu_mux_var_prediv ahb1_predivs[] = {
+       { .index = 3, .shift = 6, .width = 2 },
+};
+static struct ccu_div ahb1_clk = {
+       .div            = _SUNXI_CCU_DIV_FLAGS(4, 2, CLK_DIVIDER_POWER_OF_TWO),
+
+       .mux            = {
+               .shift  = 12,
+               .width  = 2,
+
+               .var_predivs    = ahb1_predivs,
+               .n_var_predivs  = ARRAY_SIZE(ahb1_predivs),
+       },
+
+       .common         = {
+               .reg            = 0x054,
+               .features       = CCU_FEATURE_VARIABLE_PREDIV,
+               .hw.init        = CLK_HW_INIT_PARENTS("ahb1",
+                                                     ahb1_parents,
+                                                     &ccu_div_ops,
+                                                     0),
+       },
+};
+
+static struct clk_div_table apb1_div_table[] = {
+       { .val = 0, .div = 2 },
+       { .val = 1, .div = 2 },
+       { .val = 2, .div = 4 },
+       { .val = 3, .div = 8 },
+       { /* Sentinel */ },
+};
+static SUNXI_CCU_DIV_TABLE(apb1_clk, "apb1", "ahb1",
+                          0x054, 8, 2, apb1_div_table, 0);
+
+static const char * const apb2_parents[] = { "osc32k", "osc24M",
+                                            "pll-periph0-2x",
+                                            "pll-periph0-2x" };
+static SUNXI_CCU_MP_WITH_MUX(apb2_clk, "apb2", apb2_parents, 0x058,
+                            0, 5,      /* M */
+                            16, 2,     /* P */
+                            24, 2,     /* mux */
+                            0);
+
+static SUNXI_CCU_GATE(bus_mipi_dsi_clk,        "bus-mipi-dsi",       "ahb1",
+                     0x060, BIT(1), 0);
+static SUNXI_CCU_GATE(bus_ce_clk,      "bus-ce",     "ahb1",
+                     0x060, BIT(5), 0);
+static SUNXI_CCU_GATE(bus_dma_clk,     "bus-dma",    "ahb1",
+                     0x060, BIT(6), 0);
+static SUNXI_CCU_GATE(bus_mmc0_clk,    "bus-mmc0",   "ahb1",
+                     0x060, BIT(8), 0);
+static SUNXI_CCU_GATE(bus_mmc1_clk,    "bus-mmc1",   "ahb1",
+                     0x060, BIT(9), 0);
+static SUNXI_CCU_GATE(bus_mmc2_clk,    "bus-mmc2",   "ahb1",
+                     0x060, BIT(10), 0);
+static SUNXI_CCU_GATE(bus_mmc3_clk,    "bus-mmc3",   "ahb1",
+                     0x060, BIT(11), 0);
+static SUNXI_CCU_GATE(bus_nand_clk,    "bus-nand",   "ahb1",
+                     0x060, BIT(13), 0);
+static SUNXI_CCU_GATE(bus_dram_clk,    "bus-dram",   "ahb1",
+                     0x060, BIT(14), 0);
+static SUNXI_CCU_GATE(bus_emac_clk,    "bus-emac",   "ahb1",
+                     0x060, BIT(17), 0);
+static SUNXI_CCU_GATE(bus_ts_clk,      "bus-ts",     "ahb1",
+                     0x060, BIT(18), 0);
+static SUNXI_CCU_GATE(bus_hstimer_clk, "bus-hstimer",        "ahb1",
+                     0x060, BIT(19), 0);
+static SUNXI_CCU_GATE(bus_spi0_clk,    "bus-spi0",   "ahb1",
+                     0x060, BIT(20), 0);
+static SUNXI_CCU_GATE(bus_spi1_clk,    "bus-spi1",   "ahb1",
+                     0x060, BIT(21), 0);
+static SUNXI_CCU_GATE(bus_spi2_clk,    "bus-spi2",   "ahb1",
+                     0x060, BIT(22), 0);
+static SUNXI_CCU_GATE(bus_spi3_clk,    "bus-spi3",   "ahb1",
+                     0x060, BIT(23), 0);
+static SUNXI_CCU_GATE(bus_sata_clk,    "bus-sata",   "ahb1",
+                     0x060, BIT(24), 0);
+static SUNXI_CCU_GATE(bus_otg_clk,     "bus-otg",    "ahb1",
+                     0x060, BIT(25), 0);
+static SUNXI_CCU_GATE(bus_ehci0_clk,   "bus-ehci0",  "ahb1",
+                     0x060, BIT(26), 0);
+static SUNXI_CCU_GATE(bus_ehci1_clk,   "bus-ehci1",  "ahb1",
+                     0x060, BIT(27), 0);
+static SUNXI_CCU_GATE(bus_ehci2_clk,   "bus-ehci2",  "ahb1",
+                     0x060, BIT(28), 0);
+static SUNXI_CCU_GATE(bus_ohci0_clk,   "bus-ohci0",  "ahb1",
+                     0x060, BIT(29), 0);
+static SUNXI_CCU_GATE(bus_ohci1_clk,   "bus-ohci1",  "ahb1",
+                     0x060, BIT(30), 0);
+static SUNXI_CCU_GATE(bus_ohci2_clk,   "bus-ohci2",  "ahb1",
+                     0x060, BIT(31), 0);
+
+static SUNXI_CCU_GATE(bus_ve_clk,      "bus-ve",     "ahb1",
+                     0x064, BIT(0), 0);
+static SUNXI_CCU_GATE(bus_de_mp_clk,   "bus-de-mp",  "ahb1",
+                     0x064, BIT(2), 0);

The mixer processor is unrelated to the display engine.
Please drop the "de" part of the name.

+static SUNXI_CCU_GATE(bus_deinterlace_clk,     "bus-deinterlace",    "ahb1",
+                     0x064, BIT(5), 0);
+static SUNXI_CCU_GATE(bus_csi0_clk,    "bus-csi0",   "ahb1",
+                     0x064, BIT(8), 0);
+static SUNXI_CCU_GATE(bus_csi1_clk,    "bus-csi1",   "ahb1",
+                     0x064, BIT(9), 0);
+static SUNXI_CCU_GATE(bus_hdmi_slow_clk,       "bus-hdmi-slow",      "ahb1",
+                     0x064, BIT(10), 0);
+static SUNXI_CCU_GATE(bus_hdmi_clk,    "bus-hdmi",   "ahb1",
+                     0x064, BIT(11), 0);

Please stick to the names in the user manual.
They are hdmi0 and hdmi1.

+static SUNXI_CCU_GATE(bus_de_clk,      "bus-de",     "ahb1",
+                     0x064, BIT(12), 0);
+static SUNXI_CCU_GATE(bus_tve0_clk,    "bus-tve0",   "ahb1",
+                     0x064, BIT(13), 0);
+static SUNXI_CCU_GATE(bus_tve1_clk,    "bus-tve1",   "ahb1",
+                     0x064, BIT(14), 0);
+static SUNXI_CCU_GATE(bus_tve_top_clk, "bus-tve-top",        "ahb1",
+                     0x064, BIT(15), 0);
+static SUNXI_CCU_GATE(bus_gmac_clk,    "bus-gmac",   "ahb1",
+                     0x064, BIT(17), 0);
+static SUNXI_CCU_GATE(bus_gpu_clk,     "bus-gpu",    "ahb1",
+                     0x064, BIT(20), 0);
+static SUNXI_CCU_GATE(bus_tvd0_clk,    "bus-tvd0",   "ahb1",
+                     0x064, BIT(21), 0);
+static SUNXI_CCU_GATE(bus_tvd1_clk,    "bus-tvd1",   "ahb1",
+                     0x064, BIT(22), 0);
+static SUNXI_CCU_GATE(bus_tvd2_clk,    "bus-tvd2",   "ahb1",
+                     0x064, BIT(23), 0);
+static SUNXI_CCU_GATE(bus_tvd3_clk,    "bus-tvd3",   "ahb1",
+                     0x064, BIT(24), 0);
+static SUNXI_CCU_GATE(bus_tvd_top_clk, "bus-tvd-top",        "ahb1",
+                     0x064, BIT(25), 0);
+static SUNXI_CCU_GATE(bus_tcon0_clk,   "bus-tcon0",  "ahb1",
+                     0x064, BIT(26), 0);
+static SUNXI_CCU_GATE(bus_tcon1_clk,   "bus-tcon1",  "ahb1",
+                     0x064, BIT(27), 0);
+static SUNXI_CCU_GATE(bus_tve0_tcon_clk,       "bus-tve0-tcon",      "ahb1",
+                     0x064, BIT(28), 0);
+static SUNXI_CCU_GATE(bus_tve1_tcon_clk,       "bus-tve1-tcon",      "ahb1",
+                     0x064, BIT(29), 0);

Please stick to the names used in the user manual.
They are "tcon-lcdN" and "tcon-tvN".

+static SUNXI_CCU_GATE(bus_tcon_top_clk,        "bus-tcon-top",       "ahb1",
+                     0x064, BIT(30), 0);
+
+static SUNXI_CCU_GATE(bus_codec_clk,   "bus-codec",  "apb1",
+                     0x068, BIT(0), 0);
+static SUNXI_CCU_GATE(bus_spdif_clk,   "bus-spdif",  "apb1",
+                     0x068, BIT(1), 0);
+static SUNXI_CCU_GATE(bus_ac97_clk,    "bus-ac97",   "apb1",
+                     0x068, BIT(2), 0);
+static SUNXI_CCU_GATE(bus_pio_clk,     "bus-pio",    "apb1",
+                     0x068, BIT(5), 0);
+static SUNXI_CCU_GATE(bus_ir0_clk,     "bus-ir0",    "apb1",
+                     0x068, BIT(6), 0);
+static SUNXI_CCU_GATE(bus_ir1_clk,     "bus-ir1",    "apb1",
+                     0x068, BIT(7), 0);
+static SUNXI_CCU_GATE(bus_ths_clk,     "bus-ths",    "apb1",
+                     0x068, BIT(8), 0);
+static SUNXI_CCU_GATE(bus_keypad_clk,  "bus-keypad", "apb1",
+                     0x068, BIT(10), 0);
+static SUNXI_CCU_GATE(bus_i2s0_clk,    "bus-i2s0",   "apb1",
+                     0x068, BIT(12), 0);
+static SUNXI_CCU_GATE(bus_i2s1_clk,    "bus-i2s1",   "apb1",
+                     0x068, BIT(13), 0);
+static SUNXI_CCU_GATE(bus_i2s2_clk,    "bus-i2s2",   "apb1",
+                     0x068, BIT(14), 0);
+
+static SUNXI_CCU_GATE(bus_i2c0_clk,    "bus-i2c0",   "apb2",
+                     0x06c, BIT(0), 0);
+static SUNXI_CCU_GATE(bus_i2c1_clk,    "bus-i2c1",   "apb2",
+                     0x06c, BIT(1), 0);
+static SUNXI_CCU_GATE(bus_i2c2_clk,    "bus-i2c2",   "apb2",
+                     0x06c, BIT(2), 0);
+static SUNXI_CCU_GATE(bus_i2c3_clk,    "bus-i2c3",   "apb2",
+                     0x06c, BIT(3), 0);
+/*
+ * In datasheet here's "Reserved", however the gate exists in BSP soucre
+ * code.
+ */
+static SUNXI_CCU_GATE(bus_can_clk,     "bus-can",    "apb2",
+                     0x06c, BIT(4), 0);
+static SUNXI_CCU_GATE(bus_scr_clk,     "bus-scr",    "apb2",
+                     0x06c, BIT(5), 0);
+static SUNXI_CCU_GATE(bus_ps20_clk,    "bus-ps20",   "apb2",
+                     0x06c, BIT(6), 0);
+static SUNXI_CCU_GATE(bus_ps21_clk,    "bus-ps21",   "apb2",
+                     0x06c, BIT(7), 0);
+static SUNXI_CCU_GATE(bus_i2c4_clk,    "bus-i2c4",   "apb2",
+                     0x06c, BIT(15), 0);
+static SUNXI_CCU_GATE(bus_uart0_clk,   "bus-uart0",  "apb2",
+                     0x06c, BIT(16), 0);
+static SUNXI_CCU_GATE(bus_uart1_clk,   "bus-uart1",  "apb2",
+                     0x06c, BIT(17), 0);
+static SUNXI_CCU_GATE(bus_uart2_clk,   "bus-uart2",  "apb2",
+                     0x06c, BIT(18), 0);
+static SUNXI_CCU_GATE(bus_uart3_clk,   "bus-uart3",  "apb2",
+                     0x06c, BIT(19), 0);
+static SUNXI_CCU_GATE(bus_uart4_clk,   "bus-uart4",  "apb2",
+                     0x06c, BIT(20), 0);
+static SUNXI_CCU_GATE(bus_uart5_clk,   "bus-uart5",  "apb2",
+                     0x06c, BIT(21), 0);
+static SUNXI_CCU_GATE(bus_uart6_clk,   "bus-uart6",  "apb2",
+                     0x06c, BIT(22), 0);
+static SUNXI_CCU_GATE(bus_uart7_clk,   "bus-uart7",  "apb2",
+                     0x06c, BIT(23), 0);
+
+/* Educated guess: the BSP clock driver do not have this clock */

The gate is documented in the user manual. What part is a guess?

+static SUNXI_CCU_GATE(bus_dbg_clk,     "bus-dbg",    "ahb1",
+                     0x070, BIT(7), 0);
+
+static SUNXI_CCU_M_WITH_GATE(ths_clk, "ths", "osc24M",
+                            0x074, 0, 2, BIT(31), 0);

This clock has a mux. Even with only one parent, you should still
implement the mux, so the kernel can recover from cases where some
other piece of software left it in a bad state.

Also the divider is of the power of two type.

+
+static const char * const mod0_default_parents[] = { "osc24M", "pll-periph0",
+                                                    "pll-periph1" };
+static SUNXI_CCU_MP_WITH_MUX_GATE(nand_clk, "nand", mod0_default_parents, 0x080,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static SUNXI_CCU_MP_WITH_MUX_GATE(mmc0_clk, "mmc0", mod0_default_parents, 0x088,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static SUNXI_CCU_MP_WITH_MUX_GATE(mmc1_clk, "mmc1", mod0_default_parents, 0x08c,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static SUNXI_CCU_MP_WITH_MUX_GATE(mmc2_clk, "mmc2", mod0_default_parents, 0x090,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static SUNXI_CCU_MP_WITH_MUX_GATE(mmc3_clk, "mmc3", mod0_default_parents, 0x094,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static const char * const ts_parents[] = { "osc24M", "pll-periph0", };
+static SUNXI_CCU_MP_WITH_MUX_GATE(ts_clk, "ts", ts_parents, 0x098,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */

The mux is 4 bits wide.

+                                 BIT(31),      /* gate */
+                                 0);
+
+static const char * const ce_parents[] = { "osc24M", "pll-periph0-2x",
+                                          "pll-periph1-2x" };
+static SUNXI_CCU_MP_WITH_MUX_GATE(ce_clk, "ce", ce_parents, 0x09c,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static SUNXI_CCU_MP_WITH_MUX_GATE(spi0_clk, "spi0", mod0_default_parents, 0x0a0,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static SUNXI_CCU_MP_WITH_MUX_GATE(spi1_clk, "spi1", mod0_default_parents, 0x0a4,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static SUNXI_CCU_MP_WITH_MUX_GATE(spi2_clk, "spi2", mod0_default_parents, 0x0a8,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static SUNXI_CCU_MP_WITH_MUX_GATE(spi3_clk, "spi3", mod0_default_parents, 0x0ac,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static const char * const i2s_parents[] = { "pll-audio-8x", "pll-audio-4x",
+                                           "pll-audio-2x", "pll-audio" };
+static SUNXI_CCU_MUX_WITH_GATE(i2s0_clk, "i2s0", i2s_parents,
+                              0x0b0, 16, 2, BIT(31), CLK_SET_RATE_PARENT);
+
+static SUNXI_CCU_MUX_WITH_GATE(i2s1_clk, "i2s1", i2s_parents,
+                              0x0b4, 16, 2, BIT(31), CLK_SET_RATE_PARENT);
+
+static SUNXI_CCU_MUX_WITH_GATE(i2s2_clk, "i2s2", i2s_parents,
+                              0x0b8, 16, 2, BIT(31), CLK_SET_RATE_PARENT);
+
+static SUNXI_CCU_MUX_WITH_GATE(ac97_clk, "ac97", i2s_parents,
+                              0x0bc, 16, 2, BIT(31), CLK_SET_RATE_PARENT);
+
+static SUNXI_CCU_MUX_WITH_GATE(spdif_clk, "spdif", i2s_parents,
+                              0x0c0, 16, 2, BIT(31), CLK_SET_RATE_PARENT);
+
+static const char * const keypad_parents[] = { "osc24M", "osc32k", };

The parents are not sequential. You need a mux table here.

+static SUNXI_CCU_MP_WITH_MUX_GATE(keypad_clk, "keypad", keypad_parents, 0x0c4,
+                                 0, 5,         /* M */
+                                 16, 2,        /* P */
+                                 25, 1,        /* mux */

Mux is 2 bits wide.

+                                 BIT(31),      /* gate */
+                                 0);
+
+static const char * const sata_parents[] = { "pll-sata", "sata-ext", };
+static SUNXI_CCU_MUX_WITH_GATE(sata_clk, "sata", sata_parents,
+                              0x0c8, 24, 1, BIT(31), CLK_SET_RATE_PARENT);
+
+static SUNXI_CCU_GATE(usb_phy0_clk,    "usb-phy0",   "osc24M",
+                     0x0cc, BIT(8), 0);
+static SUNXI_CCU_GATE(usb_phy1_clk,    "usb-phy1",   "osc24M",
+                     0x0cc, BIT(9), 0);
+static SUNXI_CCU_GATE(usb_phy2_clk,    "usb-phy2",   "osc24M",
+                     0x0cc, BIT(10), 0);
+static SUNXI_CCU_GATE(usb_ohci0_clk,   "usb-ohci0",  "osc24M",
+                     0x0cc, BIT(16), 0);
+static SUNXI_CCU_GATE(usb_ohci1_clk,   "usb-ohci1",  "osc24M",
+                     0x0cc, BIT(17), 0);
+static SUNXI_CCU_GATE(usb_ohci2_clk,   "usb-ohci2",  "osc24M",
+                     0x0cc, BIT(18), 0);

Why bother with the 3 clocks below, if you aren't going to use them as parents for the 3 usb-ohci gates above? The user manual states the gates are for both
48M and 12M clocks.

+
+static const char * const usb_ohci12m_parents[] = { "osc24M-2x", "osc24M",
+                                                   "osc32k" };

You should have pre-dividers here so the clk subsystem can report correct
clk rates.

+static SUNXI_CCU_MUX(usb_ohci0_12m_clk, "usb-ohci0-12m", usb_ohci12m_parents,
+                    0x0cc, 20, 2, 0);
+static SUNXI_CCU_MUX(usb_ohci1_12m_clk, "usb-ohci1-12m", usb_ohci12m_parents,
+                    0x0cc, 22, 2, 0);
+static SUNXI_CCU_MUX(usb_ohci2_12m_clk, "usb-ohci2-12m", usb_ohci12m_parents,
+                    0x0cc, 24, 2, 0);
+
+static const char * const ir_parents[] = { "osc24M", "pll-periph0",
+                                          "pll-periph1", "osc32k" };
+static SUNXI_CCU_MP_WITH_MUX_GATE(ir0_clk, "ir0", ir_parents, 0x0d0,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static SUNXI_CCU_MP_WITH_MUX_GATE(ir1_clk, "ir1", ir_parents, 0x0d4,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static const char * const dram_parents[] = { "pll-ddr0", "pll-ddr1" };
+static SUNXI_CCU_M_WITH_MUX(dram_clk, "dram", dram_parents,
+                           0x0f4, 0, 2, 20, 2, CLK_IS_CRITICAL);
+
+static SUNXI_CCU_GATE(dram_ve_clk,     "dram-ve",    "dram",
+                     0x100, BIT(0), 0);
+static SUNXI_CCU_GATE(dram_csi0_clk,   "dram-csi0",  "dram",
+                     0x100, BIT(1), 0);
+static SUNXI_CCU_GATE(dram_csi1_clk,   "dram-csi1",  "dram",
+                     0x100, BIT(2), 0);
+static SUNXI_CCU_GATE(dram_ts_clk,     "dram-ts",    "dram",
+                     0x100, BIT(3), 0);
+static SUNXI_CCU_GATE(dram_tvd_clk,    "dram-tvd",   "dram",
+                     0x100, BIT(4), 0);
+static SUNXI_CCU_GATE(dram_de_mp_clk,  "dram-de-mp", "dram",
+                     0x100, BIT(5), 0);

Same thing about the mixer processor as before.

+static SUNXI_CCU_GATE(dram_deinterlace_clk, "dram-deinterlace", "dram",
+                     0x100, BIT(6), 0);
+
+static const char * const de_parents[] = { "pll-periph0-2x", "pll-de" };
+static SUNXI_CCU_M_WITH_MUX_GATE(de_clk, "de", de_parents,
+                                0x104, 0, 4, 24, 3, BIT(31), 0);
+static SUNXI_CCU_M_WITH_MUX_GATE(de_mp_clk, "de-mp", de_parents,
+                                0x108, 0, 4, 24, 3, BIT(31), 0);
+
+static const char * const tcon_parents[] = { "pll-video0", "pll-video1",
+                                            "pll-video0-2x", "pll-video1-2x",
+                                            "pll-mipi" };
+static SUNXI_CCU_MUX_WITH_GATE(tcon0_clk, "tcon0", tcon_parents,
+                              0x110, 24, 3, BIT(31), CLK_SET_RATE_PARENT);
+static SUNXI_CCU_MUX_WITH_GATE(tcon1_clk, "tcon1", tcon_parents,
+                              0x114, 24, 3, BIT(31), CLK_SET_RATE_PARENT);
+static SUNXI_CCU_M_WITH_MUX_GATE(tcon_tve0_clk, "tcon-tve0", tcon_parents,
+                                0x118, 0, 4, 24, 3, BIT(31), 0);
+static SUNXI_CCU_M_WITH_MUX_GATE(tcon_tve1_clk, "tcon-tve1", tcon_parents,
+                                0x11c, 0, 4, 24, 3, BIT(31), 0);

Again, please stick to the names used by the user manual.

+
+static const char * const deinterlace_parents[] = { "pll-periph0",
+                                                   "pll-periph1" };
+static SUNXI_CCU_M_WITH_MUX_GATE(deinterlace_clk, "deinterlace",
+                                deinterlace_parents, 0x124, 0, 4, 24, 3,
+                                BIT(31), 0);
+
+static const char * const csi_mclk_parents[] = { "osc24M", "pll-video",
+                                                "pll-periph1" };
+static SUNXI_CCU_M_WITH_MUX_GATE(csi1_mclk_clk, "csi1-mclk", csi_mclk_parents,
+                                0x130, 0, 5, 8, 3, BIT(15), 0);
+
+static const char * const csi_sclk_parents[] = { "pll-periph0", "pll-periph1" }; +static SUNXI_CCU_M_WITH_MUX_GATE(csi_sclk_clk, "csi-sclk", csi_sclk_parents,
+                                0x134, 16, 4, 24, 3, BIT(31), 0);
+
+static SUNXI_CCU_M_WITH_MUX_GATE(csi0_mclk_clk, "csi0-mclk", csi_mclk_parents,
+                                0x134, 0, 5, 8, 3, BIT(15), 0);
+
+static SUNXI_CCU_M_WITH_GATE(ve_clk, "ve", "pll-ve",
+                            0x13c, 16, 3, BIT(31), 0);
+
+static SUNXI_CCU_GATE(adda_clk,                "adda",               
"pll-audio",
+                     0x140, BIT(31), CLK_SET_RATE_PARENT);
+static SUNXI_CCU_GATE(adda_4x_clk,     "adda-4x",    "pll-audio-4x",
+                     0x140, BIT(30), CLK_SET_RATE_PARENT);

adda-4x is not listed in the user manual. Does it really exist?
Is it used anywhere?

+static SUNXI_CCU_GATE(avs_clk,         "avs",                "osc24M",
+                     0x144, BIT(31), 0);
+
+static const char * const hdmi_parents[] = { "pll-video0", "pll-video1" };
+static SUNXI_CCU_M_WITH_MUX_GATE(hdmi_clk, "hdmi", hdmi_parents,
+                                0x150, 0, 4, 24, 2, BIT(31), 0);
+
+static SUNXI_CCU_GATE(hdmi_slow_clk,   "hdmi-slow",  "osc24M",
+                     0x154, BIT(31), 0);
+
+static const char * const mbus_parents[] = { "osc24M", "pll-periph0-2x",
+                                            "pll-ddr0" };
+static SUNXI_CCU_MP_WITH_MUX_GATE(mbus_clk, "mbus", mbus_parents, 0x15c,
+                                 0, 4,         /* M */
+                                 16, 2,        /* P */

Curiously, the formula for clk rate calculation does not include this
factor.

+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 CLK_IS_CRITICAL);
+
+static const char * const mipi_dsi_parents[] = { "pll-video0", "pll-video1",
+                                                "pll-periph0" };
+static SUNXI_CCU_M_WITH_MUX_GATE(mipi_dsi_clk, "mipi-dsi", mipi_dsi_parents,
+                                0x168, 0, 4, 8, 2, BIT(15), 0);

This is called "dsi-dphy" in the user manual. Looking at the A31,
it had "dsi-sclk" in the upper 16 bits. So please use "dsi-dphy"
or "mipi-dsi-dphy" here.

+
+static SUNXI_CCU_M_WITH_MUX_GATE(tve0_clk, "tve0", tcon_parents,
+                                0x180, 0, 4, 24, 3, BIT(31), 0);
+static SUNXI_CCU_M_WITH_MUX_GATE(tve1_clk, "tve1", tcon_parents,
+                                0x184, 0, 4, 24, 3, BIT(31), 0);
+
+static const char * const tvd_parents[] = { "pll-video0", "pll-video1",
+                                           "pll-video0-2x", "pll-video1-2x" };
+static SUNXI_CCU_M_WITH_MUX_GATE(tvd0_clk, "tvd0", tvd_parents,
+                                0x188, 0, 4, 24, 2, BIT(31), 0);
+static SUNXI_CCU_M_WITH_MUX_GATE(tvd1_clk, "tvd1", tvd_parents,
+                                0x18c, 0, 4, 24, 2, BIT(31), 0);
+static SUNXI_CCU_M_WITH_MUX_GATE(tvd2_clk, "tvd2", tvd_parents,
+                                0x190, 0, 4, 24, 2, BIT(31), 0);
+static SUNXI_CCU_M_WITH_MUX_GATE(tvd3_clk, "tvd3", tvd_parents,
+                                0x194, 0, 4, 24, 2, BIT(31), 0);

The mux is 3 bits wide for the tvd clocks.

+
+static SUNXI_CCU_M_WITH_GATE(gpu_clk, "gpu", "pll-gpu",
+                            0x1a0, 0, 3, BIT(31), 0);

You should probably have CLK_SET_RATE_PARENT here.

+
+static const char * const out_parents[] = { "osc24M-32k", "osc32k", "osc24M" };

Please use pre-dividers instead of "osc24M-32k".

+static SUNXI_CCU_MP_WITH_MUX_GATE(outa_clk, "outa", out_parents, 0x1f0,
+                                 8, 5,         /* M */
+                                 20, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+static SUNXI_CCU_MP_WITH_MUX_GATE(outb_clk, "outb", out_parents, 0x1f4,
+                                 8, 5,         /* M */
+                                 20, 2,        /* P */
+                                 24, 2,        /* mux */
+                                 BIT(31),      /* gate */
+                                 0);
+
+static struct ccu_common *sun8i_r40_ccu_clks[] = {
+       &pll_cpu_clk.common,
+       &pll_audio_base_clk.common,
+       &pll_video0_clk.common,
+       &pll_ve_clk.common,
+       &pll_ddr0_clk.common,
+       &pll_periph0_clk.common,
+       &pll_periph1_clk.common,
+       &pll_video1_clk.common,
+       &pll_sata_clk.common,
+       &pll_gpu_clk.common,
+       &pll_mipi_clk.common,
+       &pll_de_clk.common,
+       &pll_ddr1_clk.common,
+       &cpu_clk.common,
+       &axi_clk.common,
+       &ahb1_clk.common,
+       &apb1_clk.common,
+       &apb2_clk.common,
+       &bus_mipi_dsi_clk.common,
+       &bus_ce_clk.common,
+       &bus_dma_clk.common,
+       &bus_mmc0_clk.common,
+       &bus_mmc1_clk.common,
+       &bus_mmc2_clk.common,
+       &bus_mmc3_clk.common,
+       &bus_nand_clk.common,
+       &bus_dram_clk.common,
+       &bus_emac_clk.common,
+       &bus_ts_clk.common,
+       &bus_hstimer_clk.common,
+       &bus_spi0_clk.common,
+       &bus_spi1_clk.common,
+       &bus_spi2_clk.common,
+       &bus_spi3_clk.common,
+       &bus_sata_clk.common,
+       &bus_otg_clk.common,
+       &bus_ehci0_clk.common,
+       &bus_ehci1_clk.common,
+       &bus_ehci2_clk.common,
+       &bus_ohci0_clk.common,
+       &bus_ohci1_clk.common,
+       &bus_ohci2_clk.common,
+       &bus_ve_clk.common,
+       &bus_de_mp_clk.common,
+       &bus_deinterlace_clk.common,
+       &bus_csi0_clk.common,
+       &bus_csi1_clk.common,
+       &bus_hdmi_slow_clk.common,
+       &bus_hdmi_clk.common,
+       &bus_de_clk.common,
+       &bus_tve0_clk.common,
+       &bus_tve1_clk.common,
+       &bus_tve_top_clk.common,
+       &bus_gmac_clk.common,
+       &bus_gpu_clk.common,
+       &bus_tvd0_clk.common,
+       &bus_tvd1_clk.common,
+       &bus_tvd2_clk.common,
+       &bus_tvd3_clk.common,
+       &bus_tvd_top_clk.common,
+       &bus_tcon0_clk.common,
+       &bus_tcon1_clk.common,
+       &bus_tve0_tcon_clk.common,
+       &bus_tve1_tcon_clk.common,
+       &bus_tcon_top_clk.common,
+       &bus_codec_clk.common,
+       &bus_spdif_clk.common,
+       &bus_ac97_clk.common,
+       &bus_pio_clk.common,
+       &bus_ir0_clk.common,
+       &bus_ir1_clk.common,
+       &bus_ths_clk.common,
+       &bus_keypad_clk.common,
+       &bus_i2s0_clk.common,
+       &bus_i2s1_clk.common,
+       &bus_i2s2_clk.common,
+       &bus_i2c0_clk.common,
+       &bus_i2c1_clk.common,
+       &bus_i2c2_clk.common,
+       &bus_i2c3_clk.common,
+       &bus_can_clk.common,
+       &bus_scr_clk.common,
+       &bus_ps20_clk.common,
+       &bus_ps21_clk.common,
+       &bus_i2c4_clk.common,
+       &bus_uart0_clk.common,
+       &bus_uart1_clk.common,
+       &bus_uart2_clk.common,
+       &bus_uart3_clk.common,
+       &bus_uart4_clk.common,
+       &bus_uart5_clk.common,
+       &bus_uart6_clk.common,
+       &bus_uart7_clk.common,
+       &bus_dbg_clk.common,
+       &ths_clk.common,
+       &nand_clk.common,
+       &mmc0_clk.common,
+       &mmc1_clk.common,
+       &mmc2_clk.common,
+       &mmc3_clk.common,
+       &ts_clk.common,
+       &ce_clk.common,
+       &spi0_clk.common,
+       &spi1_clk.common,
+       &spi2_clk.common,
+       &spi3_clk.common,
+       &i2s0_clk.common,
+       &i2s1_clk.common,
+       &i2s2_clk.common,
+       &ac97_clk.common,
+       &spdif_clk.common,
+       &keypad_clk.common,
+       &sata_clk.common,
+       &usb_phy0_clk.common,
+       &usb_phy1_clk.common,
+       &usb_phy2_clk.common,
+       &usb_ohci0_clk.common,
+       &usb_ohci1_clk.common,
+       &usb_ohci2_clk.common,
+       &usb_ohci0_12m_clk.common,
+       &usb_ohci1_12m_clk.common,
+       &usb_ohci2_12m_clk.common,
+       &ir0_clk.common,
+       &ir1_clk.common,
+       &dram_clk.common,
+       &dram_ve_clk.common,
+       &dram_csi0_clk.common,
+       &dram_csi1_clk.common,
+       &dram_ts_clk.common,
+       &dram_tvd_clk.common,
+       &dram_de_mp_clk.common,
+       &dram_deinterlace_clk.common,
+       &de_clk.common,
+       &de_mp_clk.common,
+       &tcon0_clk.common,
+       &tcon1_clk.common,
+       &tcon_tve0_clk.common,
+       &tcon_tve1_clk.common,
+       &deinterlace_clk.common,
+       &csi1_mclk_clk.common,
+       &csi_sclk_clk.common,
+       &csi0_mclk_clk.common,
+       &ve_clk.common,
+       &adda_clk.common,
+       &adda_4x_clk.common,
+       &avs_clk.common,
+       &hdmi_clk.common,
+       &hdmi_slow_clk.common,
+       &mbus_clk.common,
+       &mipi_dsi_clk.common,
+       &tve0_clk.common,
+       &tve1_clk.common,
+       &tvd0_clk.common,
+       &tvd1_clk.common,
+       &tvd2_clk.common,
+       &tvd3_clk.common,
+       &gpu_clk.common,
+       &outa_clk.common,
+       &outb_clk.common,
+};
+
+/* We hardcode the divider to 4 for now */
+static CLK_FIXED_FACTOR(pll_audio_clk, "pll-audio",
+                       "pll-audio-base", 4, 1, CLK_SET_RATE_PARENT);
+static CLK_FIXED_FACTOR(pll_audio_2x_clk, "pll-audio-2x",
+                       "pll-audio-base", 2, 1, CLK_SET_RATE_PARENT);
+static CLK_FIXED_FACTOR(pll_audio_4x_clk, "pll-audio-4x",
+                       "pll-audio-base", 1, 1, CLK_SET_RATE_PARENT);
+static CLK_FIXED_FACTOR(pll_audio_8x_clk, "pll-audio-8x",
+                       "pll-audio-base", 1, 2, CLK_SET_RATE_PARENT);
+static CLK_FIXED_FACTOR(pll_periph0_2x_clk, "pll-periph0-2x",
+                       "pll-periph0", 1, 2, 0);
+static CLK_FIXED_FACTOR(pll_periph1_2x_clk, "pll-periph1-2x",
+                       "pll-periph1", 1, 2, 0);
+static CLK_FIXED_FACTOR(pll_video0_2x_clk, "pll-video0-2x",
+                       "pll-video0", 1, 2, 0);
+static CLK_FIXED_FACTOR(pll_video1_2x_clk, "pll-video1-2x",
+                       "pll-video1", 1, 2, 0);
+static CLK_FIXED_FACTOR(osc24m_2x_clk, "osc24M-2x",
+                       "osc24M", 1, 2, 0);
+static CLK_FIXED_FACTOR(osc24m_32k_clk, "osc24M-32k",
+                       "osc24M", 750, 1, 0);

As mentioned above, please drop the last one.

+
+static struct clk_hw_onecell_data sun8i_r40_hw_clks = {
+       .hws    = {
+               [CLK_OSC24M_2X]         = &osc24m_2x_clk.hw,
+               [CLK_OSC24M_32K]        = &osc24m_32k_clk.hw,
+               [CLK_PLL_CPU]           = &pll_cpu_clk.common.hw,
+               [CLK_PLL_AUDIO_BASE]    = &pll_audio_base_clk.common.hw,
+               [CLK_PLL_AUDIO]         = &pll_audio_clk.hw,
+               [CLK_PLL_AUDIO_2X]      = &pll_audio_2x_clk.hw,
+               [CLK_PLL_AUDIO_4X]      = &pll_audio_4x_clk.hw,
+               [CLK_PLL_AUDIO_8X]      = &pll_audio_8x_clk.hw,
+               [CLK_PLL_VIDEO0]        = &pll_video0_clk.common.hw,
+               [CLK_PLL_VIDEO0_2X]     = &pll_video0_2x_clk.hw,
+               [CLK_PLL_VE]            = &pll_ve_clk.common.hw,
+               [CLK_PLL_DDR0]          = &pll_ddr0_clk.common.hw,
+               [CLK_PLL_PERIPH0]       = &pll_periph0_clk.common.hw,
+               [CLK_PLL_PERIPH0_2X]    = &pll_periph0_2x_clk.hw,
+               [CLK_PLL_PERIPH1]       = &pll_periph1_clk.common.hw,
+               [CLK_PLL_PERIPH1_2X]    = &pll_periph1_2x_clk.hw,
+               [CLK_PLL_VIDEO1]        = &pll_video1_clk.common.hw,
+               [CLK_PLL_VIDEO1_2X]     = &pll_video1_2x_clk.hw,
+               [CLK_PLL_SATA]          = &pll_sata_clk.common.hw,
+               [CLK_PLL_GPU]           = &pll_gpu_clk.common.hw,
+               [CLK_PLL_MIPI]          = &pll_mipi_clk.common.hw,
+               [CLK_PLL_DE]            = &pll_de_clk.common.hw,
+               [CLK_PLL_DDR1]          = &pll_ddr1_clk.common.hw,
+               [CLK_CPU]               = &cpu_clk.common.hw,
+               [CLK_AXI]               = &axi_clk.common.hw,
+               [CLK_AHB1]              = &ahb1_clk.common.hw,
+               [CLK_APB1]              = &apb1_clk.common.hw,
+               [CLK_APB2]              = &apb2_clk.common.hw,
+               [CLK_BUS_MIPI_DSI]      = &bus_mipi_dsi_clk.common.hw,
+               [CLK_BUS_CE]            = &bus_ce_clk.common.hw,
+               [CLK_BUS_DMA]           = &bus_dma_clk.common.hw,
+               [CLK_BUS_MMC0]          = &bus_mmc0_clk.common.hw,
+               [CLK_BUS_MMC1]          = &bus_mmc1_clk.common.hw,
+               [CLK_BUS_MMC2]          = &bus_mmc2_clk.common.hw,
+               [CLK_BUS_MMC3]          = &bus_mmc3_clk.common.hw,
+               [CLK_BUS_NAND]          = &bus_nand_clk.common.hw,
+               [CLK_BUS_DRAM]          = &bus_dram_clk.common.hw,
+               [CLK_BUS_EMAC]          = &bus_emac_clk.common.hw,
+               [CLK_BUS_TS]            = &bus_ts_clk.common.hw,
+               [CLK_BUS_HSTIMER]       = &bus_hstimer_clk.common.hw,
+               [CLK_BUS_SPI0]          = &bus_spi0_clk.common.hw,
+               [CLK_BUS_SPI1]          = &bus_spi1_clk.common.hw,
+               [CLK_BUS_SPI2]          = &bus_spi2_clk.common.hw,
+               [CLK_BUS_SPI3]          = &bus_spi3_clk.common.hw,
+               [CLK_BUS_SATA]          = &bus_sata_clk.common.hw,
+               [CLK_BUS_OTG]           = &bus_otg_clk.common.hw,
+               [CLK_BUS_EHCI0]         = &bus_ehci0_clk.common.hw,
+               [CLK_BUS_EHCI1]         = &bus_ehci1_clk.common.hw,
+               [CLK_BUS_EHCI2]         = &bus_ehci2_clk.common.hw,
+               [CLK_BUS_OHCI0]         = &bus_ohci0_clk.common.hw,
+               [CLK_BUS_OHCI1]         = &bus_ohci1_clk.common.hw,
+               [CLK_BUS_OHCI2]         = &bus_ohci2_clk.common.hw,
+               [CLK_BUS_VE]            = &bus_ve_clk.common.hw,
+               [CLK_BUS_DE_MP]         = &bus_de_mp_clk.common.hw,
+               [CLK_BUS_DEINTERLACE]   = &bus_deinterlace_clk.common.hw,
+               [CLK_BUS_CSI0]          = &bus_csi0_clk.common.hw,
+               [CLK_BUS_CSI1]          = &bus_csi1_clk.common.hw,
+               [CLK_BUS_HDMI_SLOW]     = &bus_hdmi_slow_clk.common.hw,
+               [CLK_BUS_HDMI]          = &bus_hdmi_clk.common.hw,
+               [CLK_BUS_DE]            = &bus_de_clk.common.hw,
+               [CLK_BUS_TVE0]          = &bus_tve0_clk.common.hw,
+               [CLK_BUS_TVE1]          = &bus_tve1_clk.common.hw,
+               [CLK_BUS_TVE_TOP]       = &bus_tve_top_clk.common.hw,
+               [CLK_BUS_GMAC]          = &bus_gmac_clk.common.hw,
+               [CLK_BUS_GPU]           = &bus_gpu_clk.common.hw,
+               [CLK_BUS_TVD0]          = &bus_tvd0_clk.common.hw,
+               [CLK_BUS_TVD1]          = &bus_tvd1_clk.common.hw,
+               [CLK_BUS_TVD2]          = &bus_tvd2_clk.common.hw,
+               [CLK_BUS_TVD3]          = &bus_tvd3_clk.common.hw,
+               [CLK_BUS_TVD_TOP]       = &bus_tvd_top_clk.common.hw,
+               [CLK_BUS_TCON0]         = &bus_tcon0_clk.common.hw,
+               [CLK_BUS_TCON1]         = &bus_tcon1_clk.common.hw,
+               [CLK_BUS_TVE0_TCON]     = &bus_tve0_tcon_clk.common.hw,
+               [CLK_BUS_TVE1_TCON]     = &bus_tve1_tcon_clk.common.hw,
+               [CLK_BUS_TCON_TOP]      = &bus_tcon_top_clk.common.hw,
+               [CLK_BUS_CODEC]         = &bus_codec_clk.common.hw,
+               [CLK_BUS_SPDIF]         = &bus_spdif_clk.common.hw,
+               [CLK_BUS_AC97]          = &bus_ac97_clk.common.hw,
+               [CLK_BUS_PIO]           = &bus_pio_clk.common.hw,
+               [CLK_BUS_IR0]           = &bus_ir0_clk.common.hw,
+               [CLK_BUS_IR1]           = &bus_ir1_clk.common.hw,
+               [CLK_BUS_THS]           = &bus_ths_clk.common.hw,
+               [CLK_BUS_KEYPAD]        = &bus_keypad_clk.common.hw,
+               [CLK_BUS_I2S0]          = &bus_i2s0_clk.common.hw,
+               [CLK_BUS_I2S1]          = &bus_i2s1_clk.common.hw,
+               [CLK_BUS_I2S2]          = &bus_i2s2_clk.common.hw,
+               [CLK_BUS_I2C0]          = &bus_i2c0_clk.common.hw,
+               [CLK_BUS_I2C1]          = &bus_i2c1_clk.common.hw,
+               [CLK_BUS_I2C2]          = &bus_i2c2_clk.common.hw,
+               [CLK_BUS_I2C3]          = &bus_i2c3_clk.common.hw,
+               [CLK_BUS_CAN]           = &bus_can_clk.common.hw,
+               [CLK_BUS_SCR]           = &bus_scr_clk.common.hw,
+               [CLK_BUS_PS20]          = &bus_ps20_clk.common.hw,
+               [CLK_BUS_PS21]          = &bus_ps21_clk.common.hw,
+               [CLK_BUS_I2C4]          = &bus_i2c4_clk.common.hw,
+               [CLK_BUS_UART0]         = &bus_uart0_clk.common.hw,
+               [CLK_BUS_UART1]         = &bus_uart1_clk.common.hw,
+               [CLK_BUS_UART2]         = &bus_uart2_clk.common.hw,
+               [CLK_BUS_UART3]         = &bus_uart3_clk.common.hw,
+               [CLK_BUS_UART4]         = &bus_uart4_clk.common.hw,
+               [CLK_BUS_UART5]         = &bus_uart5_clk.common.hw,
+               [CLK_BUS_UART6]         = &bus_uart6_clk.common.hw,
+               [CLK_BUS_UART7]         = &bus_uart7_clk.common.hw,
+               [CLK_BUS_DBG]           = &bus_dbg_clk.common.hw,
+               [CLK_THS]               = &ths_clk.common.hw,
+               [CLK_NAND]              = &nand_clk.common.hw,
+               [CLK_MMC0]              = &mmc0_clk.common.hw,
+               [CLK_MMC1]              = &mmc1_clk.common.hw,
+               [CLK_MMC2]              = &mmc2_clk.common.hw,
+               [CLK_MMC3]              = &mmc3_clk.common.hw,
+               [CLK_TS]                = &ts_clk.common.hw,
+               [CLK_CE]                = &ce_clk.common.hw,
+               [CLK_SPI0]              = &spi0_clk.common.hw,
+               [CLK_SPI1]              = &spi1_clk.common.hw,
+               [CLK_SPI2]              = &spi2_clk.common.hw,
+               [CLK_SPI3]              = &spi3_clk.common.hw,
+               [CLK_I2S0]              = &i2s0_clk.common.hw,
+               [CLK_I2S1]              = &i2s1_clk.common.hw,
+               [CLK_I2S2]              = &i2s2_clk.common.hw,
+               [CLK_AC97]              = &ac97_clk.common.hw,
+               [CLK_SPDIF]             = &spdif_clk.common.hw,
+               [CLK_KEYPAD]            = &keypad_clk.common.hw,
+               [CLK_SATA]              = &sata_clk.common.hw,
+               [CLK_USB_PHY0]          = &usb_phy0_clk.common.hw,
+               [CLK_USB_PHY1]          = &usb_phy1_clk.common.hw,
+               [CLK_USB_PHY2]          = &usb_phy2_clk.common.hw,
+               [CLK_USB_OHCI0]         = &usb_ohci0_clk.common.hw,
+               [CLK_USB_OHCI1]         = &usb_ohci1_clk.common.hw,
+               [CLK_USB_OHCI2]         = &usb_ohci2_clk.common.hw,
+               [CLK_USB_OHCI0_12M]     = &usb_ohci0_12m_clk.common.hw,
+               [CLK_USB_OHCI1_12M]     = &usb_ohci1_12m_clk.common.hw,
+               [CLK_USB_OHCI2_12M]     = &usb_ohci2_12m_clk.common.hw,
+               [CLK_IR0]               = &ir0_clk.common.hw,
+               [CLK_IR1]               = &ir1_clk.common.hw,
+               [CLK_DRAM]              = &dram_clk.common.hw,
+               [CLK_DRAM_VE]           = &dram_ve_clk.common.hw,
+               [CLK_DRAM_CSI0]         = &dram_csi0_clk.common.hw,
+               [CLK_DRAM_CSI1]         = &dram_csi1_clk.common.hw,
+               [CLK_DRAM_TS]           = &dram_ts_clk.common.hw,
+               [CLK_DRAM_TVD]          = &dram_tvd_clk.common.hw,
+               [CLK_DRAM_DE_MP]        = &dram_de_mp_clk.common.hw,
+               [CLK_DRAM_DEINTERLACE]  = &dram_deinterlace_clk.common.hw,
+               [CLK_DE]                = &de_clk.common.hw,
+               [CLK_DE_MP]             = &de_mp_clk.common.hw,
+               [CLK_TCON0]             = &tcon0_clk.common.hw,
+               [CLK_TCON1]             = &tcon1_clk.common.hw,
+               [CLK_TCON_TVE0]         = &tcon_tve0_clk.common.hw,
+               [CLK_TCON_TVE1]         = &tcon_tve1_clk.common.hw,
+               [CLK_DEINTERLACE]       = &deinterlace_clk.common.hw,
+               [CLK_CSI1_MCLK]         = &csi1_mclk_clk.common.hw,
+               [CLK_CSI_SCLK]          = &csi_sclk_clk.common.hw,
+               [CLK_CSI0_MCLK]         = &csi0_mclk_clk.common.hw,
+               [CLK_VE]                = &ve_clk.common.hw,
+               [CLK_ADDA]              = &adda_clk.common.hw,
+               [CLK_ADDA_4X]           = &adda_4x_clk.common.hw,
+               [CLK_AVS]               = &avs_clk.common.hw,
+               [CLK_HDMI]              = &hdmi_clk.common.hw,
+               [CLK_HDMI_SLOW]         = &hdmi_slow_clk.common.hw,
+               [CLK_MBUS]              = &mbus_clk.common.hw,
+               [CLK_MIPI_DSI]          = &mipi_dsi_clk.common.hw,
+               [CLK_TVE0]              = &tve0_clk.common.hw,
+               [CLK_TVE1]              = &tve1_clk.common.hw,
+               [CLK_TVD0]              = &tvd0_clk.common.hw,
+               [CLK_TVD1]              = &tvd1_clk.common.hw,
+               [CLK_TVD2]              = &tvd2_clk.common.hw,
+               [CLK_TVD3]              = &tvd3_clk.common.hw,
+               [CLK_GPU]               = &gpu_clk.common.hw,
+               [CLK_OUTA]              = &outa_clk.common.hw,
+               [CLK_OUTB]              = &outb_clk.common.hw,
+       },
+       .num    = CLK_NUMBER,
+};
+
+static struct ccu_reset_map sun8i_r40_ccu_resets[] = {
+       [RST_USB_PHY0]          =  { 0x0cc, BIT(0) },
+       [RST_USB_PHY1]          =  { 0x0cc, BIT(1) },
+       [RST_USB_PHY2]          =  { 0x0cc, BIT(2) },
+

There is also a reset control for the DRAM controller at bit 31
of register 0xf4.

+       [RST_MBUS]              =  { 0x0fc, BIT(31) },
+
+       [RST_BUS_MIPI_DSI]      =  { 0x2c0, BIT(1) },
+       [RST_BUS_CE]            =  { 0x2c0, BIT(5) },
+       [RST_BUS_DMA]           =  { 0x2c0, BIT(6) },
+       [RST_BUS_MMC0]          =  { 0x2c0, BIT(8) },
+       [RST_BUS_MMC1]          =  { 0x2c0, BIT(9) },
+       [RST_BUS_MMC2]          =  { 0x2c0, BIT(10) },
+       [RST_BUS_MMC3]          =  { 0x2c0, BIT(11) },
+       [RST_BUS_NAND]          =  { 0x2c0, BIT(13) },
+       [RST_BUS_DRAM]          =  { 0x2c0, BIT(14) },
+       [RST_BUS_EMAC]          =  { 0x2c0, BIT(17) },
+       [RST_BUS_TS]            =  { 0x2c0, BIT(18) },
+       [RST_BUS_HSTIMER]       =  { 0x2c0, BIT(19) },
+       [RST_BUS_SPI0]          =  { 0x2c0, BIT(20) },
+       [RST_BUS_SPI1]          =  { 0x2c0, BIT(21) },
+       [RST_BUS_SPI2]          =  { 0x2c0, BIT(22) },
+       [RST_BUS_SPI3]          =  { 0x2c0, BIT(23) },
+       [RST_BUS_SATA]          =  { 0x2c0, BIT(24) },
+       [RST_BUS_OTG]           =  { 0x2c0, BIT(25) },
+       [RST_BUS_EHCI0]         =  { 0x2c0, BIT(26) },
+       [RST_BUS_EHCI1]         =  { 0x2c0, BIT(27) },
+       [RST_BUS_EHCI2]         =  { 0x2c0, BIT(28) },
+       [RST_BUS_OHCI0]         =  { 0x2c0, BIT(29) },
+       [RST_BUS_OHCI1]         =  { 0x2c0, BIT(30) },
+       [RST_BUS_OHCI2]         =  { 0x2c0, BIT(31) },
+
+       [RST_BUS_VE]            =  { 0x2c4, BIT(0) },
+       [RST_BUS_DE_MP]         =  { 0x2c4, BIT(2) },

Same thing about the MP's name.

+       [RST_BUS_DEINTERLACE]   =  { 0x2c4, BIT(5) },
+       [RST_BUS_CSI0]          =  { 0x2c4, BIT(8) },
+       [RST_BUS_CSI1]          =  { 0x2c4, BIT(9) },
+       [RST_BUS_HDMI_SLOW]     =  { 0x2c4, BIT(10) },
+       [RST_BUS_HDMI]          =  { 0x2c4, BIT(11) },

Same thing about the HDMI's names.

+       [RST_BUS_DE]            =  { 0x2c4, BIT(12) },
+       [RST_BUS_TVE0]          =  { 0x2c4, BIT(13) },
+       [RST_BUS_TVE1]          =  { 0x2c4, BIT(14) },
+       [RST_BUS_TVE_TOP]       =  { 0x2c4, BIT(15) },
+       [RST_BUS_GMAC]          =  { 0x2c4, BIT(17) },
+       [RST_BUS_GPU]           =  { 0x2c4, BIT(20) },
+       [RST_BUS_TVD0]          =  { 0x2c4, BIT(21) },
+       [RST_BUS_TVD1]          =  { 0x2c4, BIT(22) },
+       [RST_BUS_TVD2]          =  { 0x2c4, BIT(23) },
+       [RST_BUS_TVD3]          =  { 0x2c4, BIT(24) },
+       [RST_BUS_TVD_TOP]       =  { 0x2c4, BIT(25) },
+       [RST_BUS_TCON0]         =  { 0x2c4, BIT(26) },
+       [RST_BUS_TCON1]         =  { 0x2c4, BIT(27) },
+       [RST_BUS_TCON_TVE0]     =  { 0x2c4, BIT(28) },
+       [RST_BUS_TCON_TVE1]     =  { 0x2c4, BIT(29) },

Same thing about the TCON names.

+       [RST_BUS_TCON_TOP]      =  { 0x2c4, BIT(30) },
+       [RST_BUS_DBG]           =  { 0x2c4, BIT(31) },
+
+       [RST_BUS_LVDS]          =  { 0x2c8, BIT(0) },
+
+       [RST_BUS_CODEC]         =  { 0x2d0, BIT(0) },
+       [RST_BUS_SPDIF]         =  { 0x2d0, BIT(1) },
+       [RST_BUS_AC97]          =  { 0x2d0, BIT(2) },
+       [RST_BUS_IR0]           =  { 0x2d0, BIT(6) },
+       [RST_BUS_IR1]           =  { 0x2d0, BIT(7) },
+       [RST_BUS_THS]           =  { 0x2d0, BIT(8) },
+       [RST_BUS_KEYPAD]        =  { 0x2d0, BIT(10) },
+       [RST_BUS_I2S0]          =  { 0x2d0, BIT(12) },
+       [RST_BUS_I2S1]          =  { 0x2d0, BIT(13) },
+       [RST_BUS_I2S2]          =  { 0x2d0, BIT(14) },
+
+       [RST_BUS_I2C0]          =  { 0x2d8, BIT(0) },
+       [RST_BUS_I2C1]          =  { 0x2d8, BIT(1) },
+       [RST_BUS_I2C2]          =  { 0x2d8, BIT(2) },
+       [RST_BUS_I2C3]          =  { 0x2d8, BIT(3) },
+       [RST_BUS_CAN]           =  { 0x2d8, BIT(4) },

Please add a note for CAN like you did with the bus gate.

+       [RST_BUS_SCR]           =  { 0x2d8, BIT(5) },
+       [RST_BUS_PS20]          =  { 0x2d8, BIT(6) },
+       [RST_BUS_PS21]          =  { 0x2d8, BIT(7) },
+       [RST_BUS_I2C4]          =  { 0x2d8, BIT(15) },
+       [RST_BUS_UART0]         =  { 0x2d8, BIT(16) },
+       [RST_BUS_UART1]         =  { 0x2d8, BIT(17) },
+       [RST_BUS_UART2]         =  { 0x2d8, BIT(18) },
+       [RST_BUS_UART3]         =  { 0x2d8, BIT(19) },
+       [RST_BUS_UART4]         =  { 0x2d8, BIT(20) },
+       [RST_BUS_UART5]         =  { 0x2d8, BIT(21) },
+       [RST_BUS_UART6]         =  { 0x2d8, BIT(22) },
+       [RST_BUS_UART7]         =  { 0x2d8, BIT(23) },
+};
+
+static const struct sunxi_ccu_desc sun8i_r40_ccu_desc = {
+       .ccu_clks       = sun8i_r40_ccu_clks,
+       .num_ccu_clks   = ARRAY_SIZE(sun8i_r40_ccu_clks),
+
+       .hw_clks        = &sun8i_r40_hw_clks,
+
+       .resets         = sun8i_r40_ccu_resets,
+       .num_resets     = ARRAY_SIZE(sun8i_r40_ccu_resets),
+};
+
+static struct ccu_mux_nb sun8i_r40_cpu_nb = {
+       .common         = &cpu_clk.common,
+       .cm             = &cpu_clk.mux,
+       .delay_us       = 1, /* > 8 clock cycles at 24 MHz */
+       .bypass_index   = 1, /* index of 24 MHz oscillator */
+};
+
+static void __init sun8i_r40_ccu_setup(struct device_node *node)
+{
+       void __iomem *reg;
+       u32 val;
+
+       reg = of_io_request_and_map(node, 0, of_node_full_name(node));
+       if (IS_ERR(reg)) {
+               pr_err("%s: Could not map the clock registers\n",
+                      of_node_full_name(node));
+               return;
+       }
+
+       /* Force the PLL-Audio-1x divider to 4 */
+       val = readl(reg + SUN8I_R40_PLL_AUDIO_REG);
+       val &= ~GENMASK(19, 16);
+       writel(val | (3 << 16), reg + SUN8I_R40_PLL_AUDIO_REG);
+
+       sunxi_ccu_probe(node, reg, &sun8i_r40_ccu_desc);
+
+       ccu_mux_notifier_register(pll_cpu_clk.common.hw.clk,
+                                 &sun8i_r40_cpu_nb);
+}
+CLK_OF_DECLARE(sun8i_r40_ccu, "allwinner,sun8i-r40-ccu",
+              sun8i_r40_ccu_setup);
diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-r40.h b/drivers/clk/sunxi-ng/ccu-sun8i-r40.h
new file mode 100644
index 000000000000..be30eb662d78
--- /dev/null
+++ b/drivers/clk/sunxi-ng/ccu-sun8i-r40.h
@@ -0,0 +1,68 @@
+/*
+ * Copyright 2017 Icenowy Zheng <icen...@aosc.io>
+ *
+ * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _CCU_SUN8I_R40_H_
+#define _CCU_SUN8I_R40_H_
+
+#include <dt-bindings/clock/sun8i-r40-ccu.h>
+#include <dt-bindings/reset/sun8i-r40-ccu.h>
+
+#define CLK_OSC24M_2X          0

As mentioned, please drop this one.

+#define CLK_OSC24M_32K         1
+#define CLK_PLL_CPU            2
+#define CLK_PLL_AUDIO_BASE     3
+#define CLK_PLL_AUDIO          4
+#define CLK_PLL_AUDIO_2X       5
+#define CLK_PLL_AUDIO_4X       6
+#define CLK_PLL_AUDIO_8X       7
+#define CLK_PLL_VIDEO0         8
+#define CLK_PLL_VIDEO0_2X      9
+#define CLK_PLL_VE             10
+#define CLK_PLL_DDR0           11
+#define CLK_PLL_PERIPH0                12
+#define CLK_PLL_PERIPH0_2X     13
+#define CLK_PLL_PERIPH1                14
+#define CLK_PLL_PERIPH1_2X     15
+#define CLK_PLL_VIDEO1         16
+#define CLK_PLL_VIDEO1_2X      17
+#define CLK_PLL_SATA           18
+#define CLK_PLL_GPU            19
+#define CLK_PLL_MIPI           20
+#define CLK_PLL_DE             21
+#define CLK_PLL_DDR1           22
+
+/* The CPU clock is exported */
+
+#define CLK_AXI                        24
+#define CLK_AHB1               25
+#define CLK_APB1               26
+#define CLK_APB2               27
+
+/* All the bus gates are exported */
+
+/* The first bunch of module clocks are exported */
+
+#define CLK_DRAM               134
+
+/* All the DRAM gates are exported */
+
+/* Some more module clocks are exported */
+
+#define CLK_MBUS               158
+
+/* Another bunch of module clocks are exported */
+
+#define CLK_NUMBER             (CLK_OUTB + 1)
+
+#endif /* _CCU_SUN8I_R40_H_ */
diff --git a/include/dt-bindings/clock/sun8i-r40-ccu.h b/include/dt-bindings/clock/sun8i-r40-ccu.h
new file mode 100644
index 000000000000..a907d841496d
--- /dev/null
+++ b/include/dt-bindings/clock/sun8i-r40-ccu.h
@@ -0,0 +1,191 @@
+/*
+ * Copyright (C) 2017 Icenowy Zheng <icen...@aosc.io>
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This file is free software; you can redistribute it and/or
+ *     modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ *     License, or (at your option) any later version.
+ *
+ *     This file is distributed in the hope that it will be useful,
+ *     but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *     GNU General Public License for more details.
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ *     obtaining a copy of this software and associated documentation
+ *     files (the "Software"), to deal in the Software without
+ *     restriction, including without limitation the rights to use,
+ *     copy, modify, merge, publish, distribute, sublicense, and/or
+ *     sell copies of the Software, and to permit persons to whom the
+ *     Software is furnished to do so, subject to the following
+ *     conditions:
+ *
+ *     The above copyright notice and this permission notice shall be
+ * included in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+ *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+ *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+ *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ *     OTHER DEALINGS IN THE SOFTWARE.
+ */
+
+#ifndef _DT_BINDINGS_CLK_SUN8I_R40_H_
+#define _DT_BINDINGS_CLK_SUN8I_R40_H_
+
+#define CLK_CPU                        23
+
+#define CLK_BUS_MIPI_DSI       28
+#define CLK_BUS_CE             29
+#define CLK_BUS_DMA            30
+#define CLK_BUS_MMC0           31
+#define CLK_BUS_MMC1           32
+#define CLK_BUS_MMC2           33
+#define CLK_BUS_MMC3           34
+#define CLK_BUS_NAND           35
+#define CLK_BUS_DRAM           36
+#define CLK_BUS_EMAC           37
+#define CLK_BUS_TS             38
+#define CLK_BUS_HSTIMER                39
+#define CLK_BUS_SPI0           40
+#define CLK_BUS_SPI1           41
+#define CLK_BUS_SPI2           42
+#define CLK_BUS_SPI3           43
+#define CLK_BUS_SATA           44
+#define CLK_BUS_OTG            45
+#define CLK_BUS_EHCI0          46
+#define CLK_BUS_EHCI1          47
+#define CLK_BUS_EHCI2          48
+#define CLK_BUS_OHCI0          49
+#define CLK_BUS_OHCI1          50
+#define CLK_BUS_OHCI2          51
+#define CLK_BUS_VE             52
+#define CLK_BUS_DE_MP          53

MP name...

+#define CLK_BUS_DEINTERLACE    54
+#define CLK_BUS_CSI0           55
+#define CLK_BUS_CSI1           56
+#define CLK_BUS_HDMI_SLOW      57
+#define CLK_BUS_HDMI           58

Names...

+#define CLK_BUS_DE             59
+#define CLK_BUS_TVE0           60
+#define CLK_BUS_TVE1           61
+#define CLK_BUS_TVE_TOP                62
+#define CLK_BUS_GMAC           63
+#define CLK_BUS_GPU            64
+#define CLK_BUS_TVD0           65
+#define CLK_BUS_TVD1           66
+#define CLK_BUS_TVD2           67
+#define CLK_BUS_TVD3           68
+#define CLK_BUS_TVD_TOP                69

+#define CLK_BUS_TCON0          70
+#define CLK_BUS_TCON1          71
+#define CLK_BUS_TVE0_TCON      72
+#define CLK_BUS_TVE1_TCON      73

Names...

+#define CLK_BUS_TCON_TOP       74
+#define CLK_BUS_CODEC          75
+#define CLK_BUS_SPDIF          76
+#define CLK_BUS_AC97           77
+#define CLK_BUS_PIO            78
+#define CLK_BUS_IR0            79
+#define CLK_BUS_IR1            80
+#define CLK_BUS_THS            81
+#define CLK_BUS_KEYPAD         82
+#define CLK_BUS_I2S0           83
+#define CLK_BUS_I2S1           84
+#define CLK_BUS_I2S2           85
+#define CLK_BUS_I2C0           86
+#define CLK_BUS_I2C1           87
+#define CLK_BUS_I2C2           88
+#define CLK_BUS_I2C3           89
+#define CLK_BUS_CAN            90
+#define CLK_BUS_SCR            91
+#define CLK_BUS_PS20           92
+#define CLK_BUS_PS21           93
+#define CLK_BUS_I2C4           94
+#define CLK_BUS_UART0          95
+#define CLK_BUS_UART1          96
+#define CLK_BUS_UART2          97
+#define CLK_BUS_UART3          98
+#define CLK_BUS_UART4          99
+#define CLK_BUS_UART5          100
+#define CLK_BUS_UART6          101
+#define CLK_BUS_UART7          102
+#define CLK_BUS_DBG            103
+
+#define CLK_THS                        104
+#define CLK_NAND               105
+#define CLK_MMC0               106
+#define CLK_MMC1               107
+#define CLK_MMC2               108
+#define CLK_MMC3               109
+#define CLK_TS                 110
+#define CLK_CE                 111
+#define CLK_SPI0               112
+#define CLK_SPI1               113
+#define CLK_SPI2               114
+#define CLK_SPI3               115
+#define CLK_I2S0               116
+#define CLK_I2S1               117
+#define CLK_I2S2               118
+#define CLK_AC97               119
+#define CLK_SPDIF              120
+#define CLK_KEYPAD             121
+#define CLK_SATA               122
+#define CLK_USB_PHY0           123
+#define CLK_USB_PHY1           124
+#define CLK_USB_PHY2           125
+#define CLK_USB_OHCI0          126
+#define CLK_USB_OHCI1          127
+#define CLK_USB_OHCI2          128
+#define CLK_USB_OHCI0_12M      129
+#define CLK_USB_OHCI1_12M      130
+#define CLK_USB_OHCI2_12M      131

These 3 should not be exported.

+#define CLK_IR0                        132
+#define CLK_IR1                        133
+
+#define CLK_DRAM_VE            135
+#define CLK_DRAM_CSI0          136
+#define CLK_DRAM_CSI1          137
+#define CLK_DRAM_TS            138
+#define CLK_DRAM_TVD           139
+#define CLK_DRAM_DE_MP         140

MP name...

+#define CLK_DRAM_DEINTERLACE   141
+#define CLK_DE                 142
+#define CLK_DE_MP              143

MP name...

+#define CLK_TCON0              144
+#define CLK_TCON1              145
+#define CLK_TCON_TVE0          146
+#define CLK_TCON_TVE1          147

Names...

+#define CLK_DEINTERLACE                148
+#define CLK_CSI1_MCLK          149
+#define CLK_CSI_SCLK           150
+#define CLK_CSI0_MCLK          151
+#define CLK_VE                 152
+#define CLK_ADDA               153

+#define CLK_ADDA_4X            154

Nonexistent.

+#define CLK_AVS                        155
+#define CLK_HDMI               156
+#define CLK_HDMI_SLOW          157
+
+#define CLK_MIPI_DSI           159
+#define CLK_TVE0               160
+#define CLK_TVE1               161
+#define CLK_TVD0               162
+#define CLK_TVD1               163
+#define CLK_TVD2               164
+#define CLK_TVD3               165
+#define CLK_GPU                        166
+#define CLK_OUTA               167
+#define CLK_OUTB               168
+
+#endif /* _DT_BINDINGS_CLK_SUN8I_R40_H_ */
diff --git a/include/dt-bindings/reset/sun8i-r40-ccu.h b/include/dt-bindings/reset/sun8i-r40-ccu.h
new file mode 100644
index 000000000000..a7001f34b0b9
--- /dev/null
+++ b/include/dt-bindings/reset/sun8i-r40-ccu.h

Name corrections apply to this file as well.

Regards
ChenYu

--
2.12.2


_______________________________________________
linux-arm-kernel mailing list
linux-arm-ker...@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to