linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 05/12] ARM: OMAP2+: Add support for initializing dm814x clocks
Date: Thu, 16 Jul 2015 02:37:46 -0700	[thread overview]
Message-ID: <20150716093746.GL17550@atomide.com> (raw)
In-Reply-To: <20150604222823.GC30984@atomide.com>

* Tony Lindgren <tony@atomide.com> [150604 15:30]:
> * Stephen Boyd <sboyd@codeaurora.org> [150604 11:44]:
> > On 06/03, Tony Lindgren wrote:
> > > +#include <linux/list.h>
> > 
> > Is list.h used?
...
> > > +static const char *enable_init_clks[] = {
> > > +};
> > 
> > delete?
> > 
> > > +
> > > +int __init dm814x_dt_clk_init(void)
> > > +{
> > > +	ti_dt_clocks_register(dm814_clks);
> > > +	omap2_clk_disable_autoidle_all();
> > > +	omap2_clk_enable_init_clocks(enable_init_clks,
> > > +				     ARRAY_SIZE(enable_init_clks));
> > 
> > And pass NULL and 0 here?
> 
> Sure will remove, we can add those back once the PLL parts are
> working and if/when some boot time clocks are needed.

Here's this patch updated with the above removed.

Regards,

Tony

8< -------------
From: Tony Lindgren <tony@atomide.com>
Date: Thu, 16 Jul 2015 01:55:57 -0700
Subject: [PATCH] ARM: OMAP2+: Add support for initializing dm814x clocks

Let's add a minimal clocks for dm814x to get it booted. This is
mostly a placeholder and relies on the PLLs being on from the
bootloader.

Note that the divider clocks work the same way as on dm816x and
am335x.

Cc: Matthijs van Duin <matthijsvanduin@gmail.com>
Cc: Mike Turquette <mturquette@linaro.org>
Cc: Paul Walmsley <paul@pwsan.com>
Cc: Stephen Boyd <sboyd@codeaurora.org>
Cc: Tero Kristo <t-kristo@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>

--- a/arch/arm/mach-omap2/io.c
+++ b/arch/arm/mach-omap2/io.c
@@ -558,7 +558,7 @@ void __init ti814x_init_early(void)
 	ti81xx_hwmod_init();
 	omap_hwmod_init_postsetup();
 	if (of_have_populated_dt())
-		omap_clk_soc_init = ti81xx_dt_clk_init;
+		omap_clk_soc_init = dm814x_dt_clk_init;
 }
 
 void __init ti816x_init_early(void)
@@ -575,7 +575,7 @@ void __init ti816x_init_early(void)
 	ti81xx_hwmod_init();
 	omap_hwmod_init_postsetup();
 	if (of_have_populated_dt())
-		omap_clk_soc_init = ti81xx_dt_clk_init;
+		omap_clk_soc_init = dm816x_dt_clk_init;
 }
 #endif
 
--- a/drivers/clk/ti/Makefile
+++ b/drivers/clk/ti/Makefile
@@ -2,7 +2,7 @@ obj-y					+= clk.o autoidle.o clockdomain.o
 clk-common				= dpll.o composite.o divider.o gate.o \
 					  fixed-factor.o mux.o apll.o
 obj-$(CONFIG_SOC_AM33XX)		+= $(clk-common) clk-33xx.o
-obj-$(CONFIG_SOC_TI81XX)		+= $(clk-common) fapll.o clk-816x.o
+obj-$(CONFIG_SOC_TI81XX)		+= $(clk-common) fapll.o clk-814x.o clk-816x.o
 obj-$(CONFIG_ARCH_OMAP2)		+= $(clk-common) interface.o clk-2xxx.o
 obj-$(CONFIG_ARCH_OMAP3)		+= $(clk-common) interface.o \
 					   clk-3xxx.o
--- /dev/null
+++ b/drivers/clk/ti/clk-814x.c
@@ -0,0 +1,31 @@
+/*
+ * 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 version 2.
+ */
+
+#include <linux/kernel.h>
+#include <linux/clk-provider.h>
+#include <linux/clk/ti.h>
+
+static struct ti_dt_clk dm814_clks[] = {
+	DT_CLK(NULL, "devosc_ck", "devosc_ck"),
+	DT_CLK(NULL, "mpu_ck", "mpu_ck"),
+	DT_CLK(NULL, "sysclk4_ck", "sysclk4_ck"),
+	DT_CLK(NULL, "sysclk6_ck", "sysclk6_ck"),
+	DT_CLK(NULL, "sysclk10_ck", "sysclk10_ck"),
+	DT_CLK(NULL, "sysclk18_ck", "sysclk18_ck"),
+	DT_CLK(NULL, "timer_sys_ck", "devosc_ck"),
+	DT_CLK(NULL, "cpsw_125mhz_gclk", "cpsw_125mhz_gclk"),
+	DT_CLK(NULL, "cpsw_cpts_rft_clk", "cpsw_cpts_rft_clk"),
+	{ .node_name = NULL },
+};
+
+int __init dm814x_dt_clk_init(void)
+{
+	ti_dt_clocks_register(dm814_clks);
+	omap2_clk_disable_autoidle_all();
+	omap2_clk_enable_init_clocks(NULL, 0);
+
+	return 0;
+}
--- a/drivers/clk/ti/clk-816x.c
+++ b/drivers/clk/ti/clk-816x.c
@@ -42,7 +42,7 @@ static const char *enable_init_clks[] = {
 	"ddr_pll_clk3",
 };
 
-int __init ti81xx_dt_clk_init(void)
+int __init dm816x_dt_clk_init(void)
 {
 	ti_dt_clocks_register(dm816x_clks);
 	omap2_clk_disable_autoidle_all();
--- a/include/linux/clk/ti.h
+++ b/include/linux/clk/ti.h
@@ -329,7 +329,8 @@ int ti_clk_add_component(struct device_node *node, struct clk_hw *hw, int type);
 int omap3430_dt_clk_init(void);
 int omap3630_dt_clk_init(void);
 int am35xx_dt_clk_init(void);
-int ti81xx_dt_clk_init(void);
+int dm814x_dt_clk_init(void);
+int dm816x_dt_clk_init(void);
 int omap4xxx_dt_clk_init(void);
 int omap5xxx_dt_clk_init(void);
 int dra7xx_dt_clk_init(void);

  reply	other threads:[~2015-07-16  9:37 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-03 16:23 [PATCH 00/12] Minimal support for dm814x-evm and hp t410 thin client Tony Lindgren
2015-06-03 16:23 ` [PATCH 01/12] ARM: OMAP2+: Fix dm814x DT_MACHINE_START Tony Lindgren
2015-06-03 16:23 ` [PATCH 02/12] ARM: OMAP2+: Fix scrm compatible for dm814x Tony Lindgren
2015-06-03 19:08   ` Sergei Shtylyov
2015-06-03 19:38     ` Tony Lindgren
2015-07-16  9:11       ` Tony Lindgren
2015-06-03 16:23 ` [PATCH 03/12] ARM: OMAP2+: Add minimal clockdomains " Tony Lindgren
2015-06-03 16:23 ` [PATCH 04/12] ARM: OMAP2+: Add custom prwdm_operations for 81xx to support dm814x Tony Lindgren
2015-06-03 16:23 ` [PATCH 05/12] ARM: OMAP2+: Add support for initializing dm814x clocks Tony Lindgren
2015-06-04 18:42   ` Stephen Boyd
2015-06-04 22:28     ` Tony Lindgren
2015-07-16  9:37       ` Tony Lindgren [this message]
2015-07-16 18:25         ` Stephen Boyd
2015-07-17  5:06           ` Tony Lindgren
2015-06-03 16:23 ` [PATCH 06/12] ARM: OMAP2+: Prepare dm81xx hwmod code for adding minimal dm814x support Tony Lindgren
2015-06-03 16:23 ` [PATCH 07/12] ARM: OMAP2: Add minimal dm814x hwmod support Tony Lindgren
2015-06-03 16:23 ` [PATCH 08/12] ARM: dts: Add minimal dm814x support Tony Lindgren
2015-06-03 16:23 ` [PATCH 09/12] ARM: dts: Add minimal clocks for dm814x Tony Lindgren
2015-06-03 16:23 ` [PATCH 10/12] ARM: dts: Add minimal dts support for dm8148-evm Tony Lindgren
2015-06-03 16:23 ` [PATCH 11/12] ARM: dts: Add minimal support for HP T410 Tony Lindgren
2015-06-03 16:23 ` [PATCH 12/12] ARM: OMAP2+: Add custom abort handler for t410 Tony Lindgren

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=20150716093746.GL17550@atomide.com \
    --to=tony@atomide.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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;
as well as URLs for NNTP newsgroup(s).