From: Lokesh Vutla <lokeshvutla@ti.com>
To: u-boot@lists.denx.de
Subject: [PATCH v4 13/27] clk: ti: move drivers to 'ti' directory
Date: Tue, 20 Oct 2020 12:05:21 +0530 [thread overview]
Message-ID: <e0ec2136-84dc-2d14-e740-e70824f33058@ti.com> (raw)
In-Reply-To: <20201019164658.13749-14-dariobin@libero.it>
On 19/10/20 10:16 pm, Dario Binacchi wrote:
> Add drivers/clk/ti/ folder and move all TI's code in this folder for
> better maintenance.
>
> Signed-off-by: Dario Binacchi <dariobin@libero.it>
> ---
>
> (no changes since v1)
>
> drivers/clk/Kconfig | 40 +-----------------
> drivers/clk/Makefile | 8 +---
> drivers/clk/ti/Kconfig | 42 +++++++++++++++++++
> drivers/clk/ti/Makefile | 12 ++++++
> .../clk-am3-dpll-x2.c} | 0
> .../{clk-ti-am3-dpll.c => ti/clk-am3-dpll.c} | 0
> drivers/clk/{clk-ti-ctrl.c => ti/clk-ctrl.c} | 0
> .../{clk-ti-divider.c => ti/clk-divider.c} | 2 +-
> drivers/clk/{clk-ti-gate.c => ti/clk-gate.c} | 0
> drivers/clk/{clk-ti-mux.c => ti/clk-mux.c} | 2 +-
> drivers/clk/{clk-ti-sci.c => ti/clk-sci.c} | 0
> drivers/clk/{clk-ti.c => ti/clk.c} | 2 +-
> drivers/clk/{clk-ti.h => ti/clk.h} | 0
> 13 files changed, 59 insertions(+), 49 deletions(-)
> create mode 100644 drivers/clk/ti/Kconfig
> create mode 100644 drivers/clk/ti/Makefile
> rename drivers/clk/{clk-ti-am3-dpll-x2.c => ti/clk-am3-dpll-x2.c} (100%)
> rename drivers/clk/{clk-ti-am3-dpll.c => ti/clk-am3-dpll.c} (100%)
> rename drivers/clk/{clk-ti-ctrl.c => ti/clk-ctrl.c} (100%)
> rename drivers/clk/{clk-ti-divider.c => ti/clk-divider.c} (99%)
> rename drivers/clk/{clk-ti-gate.c => ti/clk-gate.c} (100%)
> rename drivers/clk/{clk-ti-mux.c => ti/clk-mux.c} (99%)
> rename drivers/clk/{clk-ti-sci.c => ti/clk-sci.c} (100%)
> rename drivers/clk/{clk-ti.c => ti/clk.c} (96%)
> rename drivers/clk/{clk-ti.h => ti/clk.h} (100%)
Why don't we start adding the drivers directly unders drivers/clk/ti folder.
IMHO, it did not make sense to add drivers in drivers/clk and then move them to
drivers/clk/ti in the same series. I understand clk-to-sci.c. Movement of
clk-ti-sci.c can be a separate patch.
Thanks and regards,
Lokesh
>
> diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig
> index f383e295d3..99b1254de6 100644
> --- a/drivers/clk/Kconfig
> +++ b/drivers/clk/Kconfig
> @@ -98,45 +98,6 @@ config CLK_STM32F
> This clock driver adds support for RCC clock management
> for STM32F4 and STM32F7 SoCs.
>
> -config CLK_TI_AM3_DPLL
> - bool "TI AM33XX Digital Phase-Locked Loop (DPLL) clock drivers"
> - depends on CLK && OF_CONTROL
> - help
> - This enables the DPLL clock drivers support on AM33XX SoCs. The DPLL
> - provides all interface clocks and functional clocks to the processor.
> -
> -config CLK_TI_CTRL
> - bool "TI OMAP4 clock controller"
> - depends on CLK && OF_CONTROL
> - help
> - This enables the clock controller driver support on TI's SoCs.
> -
> -config CLK_TI_DIVIDER
> - bool "TI divider clock driver"
> - depends on CLK && OF_CONTROL && CLK_CCF
> - help
> - This enables the divider clock driver support on TI's SoCs.
> -
> -config CLK_TI_GATE
> - bool "TI gate clock driver"
> - depends on CLK && OF_CONTROL
> - help
> - This enables the gate clock driver support on TI's SoCs.
> -
> -config CLK_TI_MUX
> - bool "TI mux clock driver"
> - depends on CLK && OF_CONTROL && CLK_CCF
> - help
> - This enables the mux clock driver support on TI's SoCs.
> -
> -config CLK_TI_SCI
> - bool "TI System Control Interface (TI SCI) clock driver"
> - depends on CLK && TI_SCI_PROTOCOL && OF_CONTROL
> - help
> - This enables the clock driver support over TI System Control Interface
> - available on some new TI's SoCs. If you wish to use clock resources
> - managed by the TI System Controller, say Y here. Otherwise, say N.
> -
> config CLK_HSDK
> bool "Enable cgu clock driver for HSDK boards"
> depends on CLK && TARGET_HSDK
> @@ -209,6 +170,7 @@ source "drivers/clk/owl/Kconfig"
> source "drivers/clk/renesas/Kconfig"
> source "drivers/clk/sunxi/Kconfig"
> source "drivers/clk/sifive/Kconfig"
> +source "drivers/clk/ti/Kconfig"
> source "drivers/clk/tegra/Kconfig"
> source "drivers/clk/uniphier/Kconfig"
>
> diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile
> index c98aa27e71..848cfd46f5 100644
> --- a/drivers/clk/Makefile
> +++ b/drivers/clk/Makefile
> @@ -13,12 +13,12 @@ obj-$(CONFIG_$(SPL_TPL_)CLK_COMPOSITE_CCF) += clk-composite.o
>
> obj-y += analogbits/
> obj-y += imx/
> +obj-y += ti/
> obj-y += tegra/
> obj-$(CONFIG_ARCH_ASPEED) += aspeed/
> obj-$(CONFIG_ARCH_MEDIATEK) += mediatek/
> obj-$(CONFIG_ARCH_MTMIPS) += mtmips/
> obj-$(CONFIG_ARCH_MESON) += meson/
> -obj-$(CONFIG_ARCH_OMAP2PLUS) += clk-ti.o
> obj-$(CONFIG_ARCH_ROCKCHIP) += rockchip/
> obj-$(CONFIG_ARCH_SOCFPGA) += altera/
> obj-$(CONFIG_CLK_AT91) += at91/
> @@ -48,11 +48,5 @@ obj-$(CONFIG_SANDBOX) += clk_sandbox.o
> obj-$(CONFIG_SANDBOX) += clk_sandbox_test.o
> obj-$(CONFIG_SANDBOX_CLK_CCF) += clk_sandbox_ccf.o
> obj-$(CONFIG_STM32H7) += clk_stm32h7.o
> -obj-$(CONFIG_CLK_TI_AM3_DPLL) += clk-ti-am3-dpll.o clk-ti-am3-dpll-x2.o
> -obj-$(CONFIG_CLK_TI_CTRL) += clk-ti-ctrl.o
> -obj-$(CONFIG_CLK_TI_DIVIDER) += clk-ti-divider.o
> -obj-$(CONFIG_CLK_TI_GATE) += clk-ti-gate.o
> -obj-$(CONFIG_CLK_TI_MUX) += clk-ti-mux.o
> -obj-$(CONFIG_CLK_TI_SCI) += clk-ti-sci.o
> obj-$(CONFIG_CLK_VERSAL) += clk_versal.o
> obj-$(CONFIG_CLK_CDCE9XX) += clk-cdce9xx.o
> diff --git a/drivers/clk/ti/Kconfig b/drivers/clk/ti/Kconfig
> new file mode 100644
> index 0000000000..3c61046728
> --- /dev/null
> +++ b/drivers/clk/ti/Kconfig
> @@ -0,0 +1,42 @@
> +# SPDX-License-Identifier: GPL-2.0+
> +#
> +# Copyright (C) 2020 Dario Binacchi <dariobin@libero.it>
> +#
> +config CLK_TI_AM3_DPLL
> + bool "TI AM33XX Digital Phase-Locked Loop (DPLL) clock drivers"
> + depends on CLK && OF_CONTROL
> + help
> + This enables the DPLL clock drivers support on AM33XX SoCs. The DPLL
> + provides all interface clocks and functional clocks to the processor.
> +
> +config CLK_TI_CTRL
> + bool "TI OMAP4 clock controller"
> + depends on CLK && OF_CONTROL
> + help
> + This enables the clock controller driver support on TI's SoCs.
> +
> +config CLK_TI_DIVIDER
> + bool "TI divider clock driver"
> + depends on CLK && OF_CONTROL && CLK_CCF
> + help
> + This enables the divider clock driver support on TI's SoCs.
> +
> +config CLK_TI_GATE
> + bool "TI gate clock driver"
> + depends on CLK && OF_CONTROL
> + help
> + This enables the gate clock driver support on TI's SoCs.
> +
> +config CLK_TI_MUX
> + bool "TI mux clock driver"
> + depends on CLK && OF_CONTROL && CLK_CCF
> + help
> + This enables the mux clock driver support on TI's SoCs.
> +
> +config CLK_TI_SCI
> + bool "TI System Control Interface (TI SCI) clock driver"
> + depends on CLK && TI_SCI_PROTOCOL && OF_CONTROL
> + help
> + This enables the clock driver support over TI System Control Interface
> + available on some new TI's SoCs. If you wish to use clock resources
> + managed by the TI System Controller, say Y here. Otherwise, say N.
> diff --git a/drivers/clk/ti/Makefile b/drivers/clk/ti/Makefile
> new file mode 100644
> index 0000000000..a279a2838d
> --- /dev/null
> +++ b/drivers/clk/ti/Makefile
> @@ -0,0 +1,12 @@
> +# SPDX-License-Identifier: GPL-2.0+
> +#
> +# Copyright (C) 2020 Dario Binacchi <dariobin@libero.it>
> +#
> +
> +obj-$(CONFIG_ARCH_OMAP2PLUS) += clk.o
> +obj-$(CONFIG_CLK_TI_AM3_DPLL) += clk-am3-dpll.o clk-am3-dpll-x2.o
> +obj-$(CONFIG_CLK_TI_CTRL) += clk-ctrl.o
> +obj-$(CONFIG_CLK_TI_DIVIDER) += clk-divider.o
> +obj-$(CONFIG_CLK_TI_GATE) += clk-gate.o
> +obj-$(CONFIG_CLK_TI_MUX) += clk-mux.o
> +obj-$(CONFIG_CLK_TI_SCI) += clk-sci.o
> diff --git a/drivers/clk/clk-ti-am3-dpll-x2.c b/drivers/clk/ti/clk-am3-dpll-x2.c
> similarity index 100%
> rename from drivers/clk/clk-ti-am3-dpll-x2.c
> rename to drivers/clk/ti/clk-am3-dpll-x2.c
> diff --git a/drivers/clk/clk-ti-am3-dpll.c b/drivers/clk/ti/clk-am3-dpll.c
> similarity index 100%
> rename from drivers/clk/clk-ti-am3-dpll.c
> rename to drivers/clk/ti/clk-am3-dpll.c
> diff --git a/drivers/clk/clk-ti-ctrl.c b/drivers/clk/ti/clk-ctrl.c
> similarity index 100%
> rename from drivers/clk/clk-ti-ctrl.c
> rename to drivers/clk/ti/clk-ctrl.c
> diff --git a/drivers/clk/clk-ti-divider.c b/drivers/clk/ti/clk-divider.c
> similarity index 99%
> rename from drivers/clk/clk-ti-divider.c
> rename to drivers/clk/ti/clk-divider.c
> index cb6923ddb5..d448197b1f 100644
> --- a/drivers/clk/clk-ti-divider.c
> +++ b/drivers/clk/ti/clk-divider.c
> @@ -17,7 +17,7 @@
> #include <linux/clk-provider.h>
> #include <linux/kernel.h>
> #include <linux/log2.h>
> -#include "clk-ti.h"
> +#include "clk.h"
>
> /*
> * The reverse of DIV_ROUND_UP: The maximum number which
> diff --git a/drivers/clk/clk-ti-gate.c b/drivers/clk/ti/clk-gate.c
> similarity index 100%
> rename from drivers/clk/clk-ti-gate.c
> rename to drivers/clk/ti/clk-gate.c
> diff --git a/drivers/clk/clk-ti-mux.c b/drivers/clk/ti/clk-mux.c
> similarity index 99%
> rename from drivers/clk/clk-ti-mux.c
> rename to drivers/clk/ti/clk-mux.c
> index 2c5801a39c..35470255af 100644
> --- a/drivers/clk/clk-ti-mux.c
> +++ b/drivers/clk/ti/clk-mux.c
> @@ -13,7 +13,7 @@
> #include <clk-uclass.h>
> #include <asm/io.h>
> #include <linux/clk-provider.h>
> -#include "clk-ti.h"
> +#include "clk.h"
>
> struct clk_ti_mux_priv {
> struct clk_bulk parents;
> diff --git a/drivers/clk/clk-ti-sci.c b/drivers/clk/ti/clk-sci.c
> similarity index 100%
> rename from drivers/clk/clk-ti-sci.c
> rename to drivers/clk/ti/clk-sci.c
> diff --git a/drivers/clk/clk-ti.c b/drivers/clk/ti/clk.c
> similarity index 96%
> rename from drivers/clk/clk-ti.c
> rename to drivers/clk/ti/clk.c
> index 594ef75270..e44b90ad6a 100644
> --- a/drivers/clk/clk-ti.c
> +++ b/drivers/clk/ti/clk.c
> @@ -7,7 +7,7 @@
>
> #include <common.h>
> #include <asm/io.h>
> -#include "clk-ti.h"
> +#include "clk.h"
>
> static void clk_ti_rmw(u32 val, u32 mask, fdt_addr_t reg)
> {
> diff --git a/drivers/clk/clk-ti.h b/drivers/clk/ti/clk.h
> similarity index 100%
> rename from drivers/clk/clk-ti.h
> rename to drivers/clk/ti/clk.h
>
prev parent reply other threads:[~2020-10-20 6:35 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-19 16:46 [PATCH v4 00/27] Add DM support for omap PWM backlight Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 01/27] clk: export generic routines Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 02/27] dt-bindings: bus: ti-sysc: resync with Linux 5.9-rc7 Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 03/27] bus: ti: add minimal sysc interconnect target driver Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 04/27] arm: dts: sync am33xx with Linux 5.9-rc7 Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 05/27] clk: add clk_round_rate() Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 06/27] clk: ti: add mux clock driver Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 07/27] arm: ti: am33xx: add DPLL_EN_FAST_RELOCK_BYPASS macro Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 08/27] clk: ti: am33xx: add DPLL clock drivers Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 09/27] clk: ti: add divider clock driver Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 10/27] clk: ti: add gate " Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 11/27] ti: am33xx: fix do_enable_clocks() to accept NULL parameters Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 12/27] clk: ti: add support for clkctrl clocks Dario Binacchi
2020-10-19 16:46 ` [PATCH v4 13/27] clk: ti: move drivers to 'ti' directory Dario Binacchi
2020-10-20 6:35 ` Lokesh Vutla [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e0ec2136-84dc-2d14-e740-e70824f33058@ti.com \
--to=lokeshvutla@ti.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox