linux-kernel - Re: [PATCH 3/3] ARM: dts: bcm281xx: define real clocks

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <529F2BD9.5080808@linaro.org>
Date:	Wed, 04 Dec 2013 07:19:21 -0600
From:	Alex Elder <alex.elder@...aro.org>
To:	Mark Rutland <mark.rutland@....com>
CC:	Christian Daudt <bcm@...thebug.org>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Pawel Moll <Pawel.Moll@....com>,
	"rob.herring@...xeda.com" <rob.herring@...xeda.com>,
	Rob Landley <rob@...dley.net>,
	Russell King <linux@....linux.org.uk>,
	Stephen Warren <swarren@...dotorg.org>,
	Mike Turquette <mturquette@...aro.org>,
	"bcm-kernel-feedback-list@...adcom.com" 
	<bcm-kernel-feedback-list@...adcom.com>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 3/3] ARM: dts: bcm281xx: define real clocks

On 12/04/2013 05:19 AM, Mark Rutland wrote:
> On Wed, Dec 04, 2013 at 03:57:07AM +0000, Alex Elder wrote:
>> Replace the "fake" clocks defined in the "bcm11351.dtsi" device tree
>> file with real definitions backed by the new BCM281xx clock driver..
>>
>> Signed-off-by: Alex Elder <elder@...aro.org>
>> Reviewed-by: Matt Porter <matt.porter@...aro.org>
>> Reviewed-by: Tim Kryger <tim.kryger@...aro.org>
>> ---
>>  arch/arm/boot/dts/bcm11351.dtsi |  222
>> ++++++++++++++++++++++++++++-----------
>>  1 file changed, 161 insertions(+), 61 deletions(-)
> 
> [...]
> 
>> +		/*
>> +		    * This is a place-holder clock for peripheral
>> +		    * clocks that set their parent clock to an
>> +		    * out-of-range value to explicitly select
>> +		    * "no clock" as a parent.
>> +		    */
>> +		not_selected_clk: not_selected {
>>  			#clock-cells = <0>;
>> -		};
> 
> Huh? This doesn't seem to be used at all in this series. Why is this
> here?

This has been the source of confusion before.

You're right, it's not used in the current patch.  And
because of this, I'll delete it, and add it back once
we have a clock supported that requires it.

But while we're on the subject, here's what it's for.

Each clock has a defined set of parent clocks, and
each has a small integer value that represents it in
a register that controls their selection.  In some
cases the reset value of that register field is a
(specific) value that has no defined clock associated
with it.

Here's an example.  There's a clock "hub_clk_ssp3_audio",
which has three parent clocks:  "ref_crystal" (selector
value 0),  "ref_312m" (selector value 1), and "cx40"
(selector value 2).  But on reset, the value in that
selector field is 3.

This clock is used to represent that "none selected" value.

> Thanks,
> Mark.
> 

Thank you.  I really appreciate the reviews Mark.

					-Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/