From: Stephen Boyd <firstname.lastname@example.org> To: <email@example.com> Cc: Alexander Dahl <firstname.lastname@example.org>, Michael Turquette <email@example.com>, Rob Herring <firstname.lastname@example.org>, email@example.com, firstname.lastname@example.org, email@example.com Subject: Re: [PATCH v2] dt-bindings: clk: Make example a bit clearer Date: Mon, 16 Sep 2019 13:13:16 -0700 Message-ID: <20190916201316.ED7EB20665@mail.kernel.org> (raw) In-Reply-To: <firstname.lastname@example.org> (Sorry my MUA fails at utf8 encoding emails) > On Wed, Sep 11, 2019 at 09:06:47AM -0700, Stephen Boyd wrote: > > > diff --git a/Documentation/devicetree/bindings/clock/clock-bindings.txt b/Documentation/devicetree/bindings/clock/clock-bindings.txt > > > index b646bbcf7f92..1d4942380918 100644 > > > --- a/Documentation/devicetree/bindings/clock/clock-bindings.txt > > > +++ b/Documentation/devicetree/bindings/clock/clock-bindings.txt > > > @@ -150,16 +150,18 @@ set to 0, or can be omitted if it is not followed by any non-zero entry. > > > compatible = "fsl,imx-uart"; > > > reg = <0xa000 0x1000>; > > > ... > > > - clocks = <&osc 0>, <&pll 1>; > > > - clock-names = "baud", "register"; > > > + clocks = ... > > > + clock-names = ... > > > > I don't see the need for this change. > > <&pll 1> is mentioned below. But it is not this instance that is > relevant. So I suggested to get rid of it to not be a source of > confusion. (But I guess you understood that.) Yes. > > > > assigned-clocks = <&clkcon 0>, <&pll 2>; > > > - assigned-clock-parents = <&pll 2>; > > > + assigned-clock-parents = <&pll 1>; > > > assigned-clock-rates = <0>, <460800>; > > > }; > > > > > > -In this example the <&pll 2> clock is set as parent of clock <&clkcon 0> and > > > -the <&pll 2> clock is assigned a frequency value of 460800 Hz. > > > +In this example the <&pll 1> clock is set as parent of clock <&clkcon 0> and > > > +the <&pll 2> clock is assigned a frequency value of 460800 Hz. A parent > > > +setting for <&pll 2> is omitted (end of list) and rate setting for <&clkcon 0> > > > +is skipped because set to <0>. > > > > Maybe you can comment that the "clocks" and "clock-names" properties > > don't matter for assigned clk rates and parents. > > Sure, a long text can explain this, but a maximal simple example is > very beneficial, too. > The paragraph at the start of "Assigned clock parents and rates" already explains how these properties work. This patch isn't going to help if the reader hasn't read that paragraph, so reordering things isn't providing more clarity. If anything, I would clarify the description of the example by indicating what parts of the example we're talking about when explaining, i.e. In this example the <&pll 2> clock is set as parent of the <&clkcon 0> clock because the first clock specifier in the assigned-clocks property is <&clkcon 0> and that matches the first clock specifier in the assigned-clock-parents property. Similarly the <&pll 2> clock is assigned a frequency value of 460800 Hz because the second frequency in the assigned-clock-rates property is 460800 Hz and that matches the second clock specifier in the assigned-clocks property <&pll 2>.
next prev parent reply index Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-08-23 10:03 Alexander Dahl 2019-09-11 16:06 ` Stephen Boyd 2019-09-11 17:49 ` Uwe Kleine-König 2019-09-16 20:13 ` Stephen Boyd [this message] 2019-10-10 14:16 ` Alexander Dahl 2019-11-08 22:18 ` Stephen Boyd
Reply instructions: You may reply publically 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=20190916201316.ED7EB20665@mail.kernel.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.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
Linux-Clk Archive on lore.kernel.org Archives are clonable: git clone --mirror https://lore.kernel.org/linux-clk/0 linux-clk/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 linux-clk linux-clk/ https://lore.kernel.org/linux-clk \ email@example.com public-inbox-index linux-clk Example config snippet for mirrors Newsgroup available over NNTP: nntp://nntp.lore.kernel.org/org.kernel.vger.linux-clk AGPL code for this site: git clone https://public-inbox.org/public-inbox.git