Re: Clocks on Barebox

From: andreas.willig@rafi.de
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: barebox@lists.infradead.org
Subject: Re: Clocks on Barebox
Date: Tue, 2 Jun 2015 05:42:49 +0200	[thread overview]
Message-ID: <OF4A53439D.B017BB3F-ONC1257E58.0014663E-C1257E58.00146649@o0802.rafi.inhouse> (raw)
In-Reply-To: <20150601114150.GG6325@pengutronix.de>

Hi Sascha,
I'm currently on the next branch, but i did not see changes at this point since master or stable release. Like issued earlier the clock table enumeration is empty. Is there a specific compatible attribute required? 

Mit freundlichen Grüßen aus Berg/ With kind regards

i.A. Andreas Willig
Dipl.-Ing. (FH)
Entwicklung Elektronik

RAFI GmbH & Co. KG
Postfach 2060
88276 Ravensburg
Tel.: +49 (0) 751 89-6142

mailto:andreas.willig@rafi.de, http://www.rafi.de

Kommanditgesellschaft: Sitz Berg, Amtsgericht Ulm, HRA 550059
Komplementär: RAFI Beteiligungs-GmbH, Sitz Berg, Amtsgericht Ulm HRB 550074
Geschäftsführer: Albert Wasmeier, Dipl.-Kfm. Gerhard Schenk
USt-Ident.-Nr.: DE 146 392 319, Steuer-Nr.: 77079/00291 

-----Sascha Hauer <s.hauer@pengutronix.de> schrieb: -----

 =======================
 An: andreas.willig@rafi.de
 Von: Sascha Hauer <s.hauer@pengutronix.de>
 Datum: 01.06.2015 13:41 
 Kopie: barebox@lists.infradead.org
 Betreff: Re: Clocks on Barebox
 =======================
   Hi Andreas,

On Mon, Jun 01, 2015 at 10:12:55AM +0200, andreas.willig@rafi.de wrote:
> Hi all
> i just updated from 2014 to latest Barebox and hit the following problem:
> of_clk_init from clk.c:451 does not find any entries. matches is set to __clk_of_table start where compatible=0 and data=0,
> but from my thoughts the dt is ok on this point:

I have no idea currently. You are doing nothing obviously wrong.

> from arch/arm/mach-imx/clk-imx6.c are removed:
> static int imx6_ccm_probe()
> {
> - unsigned long ckil_rate = 32768;
> - unsigned long ckih_rate = 0;
> - unsigned long osc_rate = 24000000;
> - clks[dummy] = clk_fixed("dummy", 0);
> - clks[ckil] = clk_fixed("ckil", ckil_rate);
> - clks[ckih] = clk_fixed("ckih", ckih_rate);
> - clks[osc] = clk_fixed("osc", osc_rate);
> }

These are removed because they come from the OF clock providers now. For
this you need COMMON_CLK_OF_PROVIDER enabled, but this should be
selected by the imx6 support automatically though. Let me test this on
some hardware here. What barebox version do you use? master, next, or
the latest release?

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox