[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Mon, 11 Nov 2019 09:31:15 +0000
From: "Ardelean, Alexandru" <alexandru.Ardelean@...log.com>
To: "linux-clk@...r.kernel.org" <linux-clk@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"sboyd@...nel.org" <sboyd@...nel.org>,
"linux-renesas-soc@...r.kernel.org"
<linux-renesas-soc@...r.kernel.org>
CC: "wsa+renesas@...g-engineering.com" <wsa+renesas@...g-engineering.com>,
"jsarha@...com" <jsarha@...com>,
"horms@...ge.net.au" <horms@...ge.net.au>,
"Hennerich, Michael" <Michael.Hennerich@...log.com>,
"magnus.damm@...il.com" <magnus.damm@...il.com>,
"ce3a@....de" <ce3a@....de>,
"mturquette@...libre.com" <mturquette@...libre.com>,
"geert+renesas@...der.be" <geert+renesas@...der.be>
Subject: Re: [PATCH v3] clk: clk-gpio: propagate rate change to parent
On Fri, 2019-11-08 at 13:08 -0800, Stephen Boyd wrote:
> Quoting Alexandru Ardelean (2019-11-07 23:17:18)
> > From: Michael Hennerich <michael.hennerich@...log.com>
> >
> > For an external clock source, which is gated via a GPIO, the
> > rate change should typically be propagated to the parent clock.
> >
> > The situation where we are requiring this propagation, is when an
> > external clock is connected to override an internal clock (which
> > typically
> > has a fixed rate). The external clock can have a different rate than
> > the
> > internal one, and may also be variable, thus requiring the rate
> > propagation.
> >
> > This rate change wasn't propagated until now, and it's unclear about
> > cases
> > where this shouldn't be propagated. Thus, it's unclear whether this is
> > fixing a bug, or extending the current driver behavior. Also, it's
> > unsure
> > about whether this may break any existing setups; in the case that it
> > does,
> > a device-tree property may be added to disable this flag.
> >
> > Signed-off-by: Michael Hennerich <michael.hennerich@...log.com>
> > Signed-off-by: Alexandru Ardelean <alexandru.ardelean@...log.com>
> > ---
>
> Applied to clk-next
>
> Next time please send as a new topic instead of a reply to the original
> patch. Makes it easier for me to apply the patch.
>
Ack.
Will do that.
Thanks
Alex
Powered by blists - more mailing lists