From: Jean-Francois Moine <moinejf@free.fr>
To: "Ondřej Jirman" <megous@megous.com>
Cc: "Maxime Ripard" <maxime.ripard@free-electrons.com>,
"Mark Rutland" <mark.rutland@arm.com>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>,
dev@linux-sunxi.org,
"Michael Turquette" <mturquette@baylibre.com>,
"Stephen Boyd" <sboyd@codeaurora.org>,
"open list" <linux-kernel@vger.kernel.org>,
"Emilio López" <emilio@elopez.com.ar>,
"Chen-Yu Tsai" <wens@csie.org>,
"Rob Herring" <robh+dt@kernel.org>,
"open list:COMMON CLK FRAMEWORK" <linux-clk@vger.kernel.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 06/14] ARM: sun8i: clk: Add clk-factor rate application method
Date: Fri, 1 Jul 2016 09:47:19 +0200 [thread overview]
Message-ID: <20160701094719.9f7e28d93595d9cf7820e7dd@free.fr> (raw)
In-Reply-To: <643c2648-ddd6-700a-68c0-2981134965c0@megous.com>
On Fri, 1 Jul 2016 08:34:21 +0200
Ondřej Jirman <megous@megous.com> wrote:
> > The documentation says that only the CPU and DDR PLLs can be dynamically
> > changed after boot.
>
> The question is what exactly is meant by after boot. :) Anyway, if the
> kernel has no business changing some other PLLs, if there's code for
> changing them, should it be dropped?
No, because all the other PLLs may not be initialized by the U-boot
(audio, video, gpu...), and also, their rate may be changed safely by
stopping them (gate).
--
Ken ar c'hentañ | ** Breizh ha Linux atav! **
Jef | http://moinejf.free.fr/
WARNING: multiple messages have this Message-ID (diff)
From: Jean-Francois Moine <moinejf-GANU6spQydw@public.gmane.org>
To: "Ondřej Jirman" <megous-5qf/QAjKc83QT0dZR+AlfA@public.gmane.org>
Cc: "Maxime Ripard"
<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
"Mark Rutland" <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
dev-3kdeTeqwOZ9EV1b7eY7vFQ@public.gmane.org,
"Michael Turquette"
<mturquette-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>,
"Stephen Boyd" <sboyd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
"open list"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"Emilio López" <emilio-0Z03zUJReD5OxF6Tv1QG9Q@public.gmane.org>,
"Chen-Yu Tsai" <wens-jdAy2FN1RRM@public.gmane.org>,
"Rob Herring" <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
"open list:COMMON CLK FRAMEWORK"
<linux-clk-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH v2 06/14] ARM: sun8i: clk: Add clk-factor rate application method
Date: Fri, 1 Jul 2016 09:47:19 +0200 [thread overview]
Message-ID: <20160701094719.9f7e28d93595d9cf7820e7dd@free.fr> (raw)
In-Reply-To: <643c2648-ddd6-700a-68c0-2981134965c0-5qf/QAjKc83QT0dZR+AlfA@public.gmane.org>
On Fri, 1 Jul 2016 08:34:21 +0200
Ondřej Jirman <megous-5qf/QAjKc83QT0dZR+AlfA@public.gmane.org> wrote:
> > The documentation says that only the CPU and DDR PLLs can be dynamically
> > changed after boot.
>
> The question is what exactly is meant by after boot. :) Anyway, if the
> kernel has no business changing some other PLLs, if there's code for
> changing them, should it be dropped?
No, because all the other PLLs may not be initialized by the U-boot
(audio, video, gpu...), and also, their rate may be changed safely by
stopping them (gate).
--
Ken ar c'hentañ | ** Breizh ha Linux atav! **
Jef | http://moinejf.free.fr/
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: moinejf@free.fr (Jean-Francois Moine)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 06/14] ARM: sun8i: clk: Add clk-factor rate application method
Date: Fri, 1 Jul 2016 09:47:19 +0200 [thread overview]
Message-ID: <20160701094719.9f7e28d93595d9cf7820e7dd@free.fr> (raw)
In-Reply-To: <643c2648-ddd6-700a-68c0-2981134965c0@megous.com>
On Fri, 1 Jul 2016 08:34:21 +0200
Ond?ej Jirman <megous@megous.com> wrote:
> > The documentation says that only the CPU and DDR PLLs can be dynamically
> > changed after boot.
>
> The question is what exactly is meant by after boot. :) Anyway, if the
> kernel has no business changing some other PLLs, if there's code for
> changing them, should it be dropped?
No, because all the other PLLs may not be initialized by the U-boot
(audio, video, gpu...), and also, their rate may be changed safely by
stopping them (gate).
--
Ken ar c'henta? | ** Breizh ha Linux atav! **
Jef | http://moinejf.free.fr/
next prev parent reply other threads:[~2016-07-01 7:47 UTC|newest]
Thread overview: 183+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-25 3:44 [PATCH v2] Thermal regulation for Orange Pi PC and Orange Pi One megous at megous.com
2016-06-25 3:44 ` [PATCH v2 01/14] ARM: clk: sunxi: Add driver for the H3 THS clock megous
2016-06-25 3:44 ` megous at megous.com
2016-06-25 3:44 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-25 7:13 ` Maxime Ripard
2016-06-25 7:13 ` Maxime Ripard
2016-06-25 7:13 ` Maxime Ripard
2016-06-25 15:23 ` Ondřej Jirman
2016-06-25 15:23 ` Ondřej Jirman
2016-06-25 15:23 ` Ondřej Jirman
2016-06-28 11:52 ` Maxime Ripard
2016-06-28 11:52 ` Maxime Ripard
2016-06-28 11:52 ` Maxime Ripard
2016-06-25 3:44 ` [PATCH v2 02/14] thermal: sun8i_ths: Add support for the thermal sensor on Allwinner H3 megous
2016-06-25 3:44 ` megous at megous.com
2016-06-25 3:44 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-25 7:10 ` Maxime Ripard
2016-06-25 7:10 ` Maxime Ripard
2016-06-25 15:12 ` Ondřej Jirman
2016-06-25 15:12 ` Ondřej Jirman
2016-06-28 11:39 ` Maxime Ripard
2016-06-28 11:39 ` Maxime Ripard
2016-06-28 11:39 ` Maxime Ripard
2016-06-25 3:45 ` [PATCH v2 03/14] dt-bindings: document sun8i_ths - H3 thermal sensor driver megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-28 20:56 ` Rob Herring
2016-06-28 20:56 ` Rob Herring
2016-06-28 20:56 ` Rob Herring
2016-06-25 3:45 ` [PATCH v2 04/14] regulator: SY8106A regulator driver megous
2016-06-25 3:45 ` megous at megous.com
2016-06-26 11:26 ` Mark Brown
2016-06-26 11:26 ` Mark Brown
2016-06-26 15:07 ` Ondřej Jirman
2016-06-26 15:07 ` Ondřej Jirman
2016-06-27 14:54 ` Mark Brown
2016-06-27 14:54 ` Mark Brown
2016-06-28 16:27 ` Ondřej Jirman
2016-06-28 16:27 ` Ondřej Jirman
2016-06-27 15:10 ` Mark Brown
2016-06-27 15:10 ` Mark Brown
2016-06-25 3:45 ` [PATCH v2 05/14] dt-bindings: document " megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-26 11:27 ` Mark Brown
2016-06-26 11:27 ` Mark Brown
2016-06-26 11:27 ` Mark Brown
2016-06-26 15:10 ` Ondřej Jirman
2016-06-26 15:10 ` Ondřej Jirman
2016-06-26 15:10 ` Ondřej Jirman
2016-06-26 18:52 ` Mark Brown
2016-06-26 18:52 ` Mark Brown
2016-06-26 18:52 ` Mark Brown
2016-06-25 3:45 ` [PATCH v2 06/14] ARM: sun8i: clk: Add clk-factor rate application method megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-30 20:40 ` Maxime Ripard
2016-06-30 20:40 ` Maxime Ripard
2016-06-30 20:40 ` Maxime Ripard
2016-07-01 0:50 ` Ondřej Jirman
2016-07-01 0:50 ` Ondřej Jirman
2016-07-01 0:50 ` Ondřej Jirman
2016-07-01 5:37 ` Jean-Francois Moine
2016-07-01 5:37 ` Jean-Francois Moine
2016-07-01 6:34 ` Ondřej Jirman
2016-07-01 6:34 ` Ondřej Jirman
2016-07-01 6:34 ` Ondřej Jirman
2016-07-01 7:47 ` Jean-Francois Moine [this message]
2016-07-01 7:47 ` Jean-Francois Moine
2016-07-01 7:47 ` Jean-Francois Moine
2016-07-15 8:53 ` Maxime Ripard
2016-07-15 8:53 ` Maxime Ripard
2016-07-15 8:53 ` Maxime Ripard
2016-07-15 10:38 ` Ondřej Jirman
2016-07-15 10:38 ` Ondřej Jirman
2016-07-15 10:38 ` Ondřej Jirman
2016-07-15 13:27 ` Jean-Francois Moine
2016-07-15 13:27 ` Jean-Francois Moine
2016-07-15 13:48 ` Ondřej Jirman
2016-07-15 13:48 ` Ondřej Jirman
2016-07-15 13:48 ` Ondřej Jirman
2016-07-15 14:22 ` [linux-sunxi] " Michal Suchanek
2016-07-15 14:22 ` Michal Suchanek
2016-07-15 14:22 ` Michal Suchanek
2016-07-15 16:33 ` Ondřej Jirman
2016-07-15 16:33 ` Ondřej Jirman
2016-07-21 9:51 ` Maxime Ripard
2016-07-21 9:51 ` Maxime Ripard
2016-07-21 9:51 ` Maxime Ripard
2016-07-21 9:48 ` Maxime Ripard
2016-07-21 9:48 ` Maxime Ripard
2016-07-21 9:48 ` Maxime Ripard
2016-07-21 9:52 ` Ondřej Jirman
2016-07-21 9:52 ` Ondřej Jirman
2016-07-26 6:32 ` Maxime Ripard
2016-07-26 6:32 ` Maxime Ripard
2016-07-28 11:27 ` Changed: sunxi-ng clock code - NKMP clock implementation is wrong Ondřej Jirman
2016-07-28 11:27 ` Ondřej Jirman
2016-07-28 11:27 ` Ondřej Jirman
2016-07-28 11:38 ` [linux-sunxi] " Chen-Yu Tsai
2016-07-28 11:38 ` Chen-Yu Tsai
2016-07-28 11:38 ` Chen-Yu Tsai
2016-07-28 21:00 ` Maxime Ripard
2016-07-28 21:00 ` Maxime Ripard
2016-07-28 21:00 ` Maxime Ripard
2016-07-28 22:01 ` Ondřej Jirman
2016-07-28 22:01 ` Ondřej Jirman
2016-07-28 22:01 ` Ondřej Jirman
2016-07-31 10:31 ` Maxime Ripard
2016-07-31 10:31 ` Maxime Ripard
2016-07-31 10:31 ` Maxime Ripard
2016-07-31 22:01 ` Ondřej Jirman
2016-07-31 22:01 ` Ondřej Jirman
2016-07-31 22:01 ` Ondřej Jirman
2016-08-31 20:25 ` Maxime Ripard
2016-08-31 20:25 ` Maxime Ripard
2016-08-31 20:25 ` Maxime Ripard
2016-07-01 0:53 ` [PATCH v2 06/14] ARM: sun8i: clk: Add clk-factor rate application method Ondřej Jirman
2016-07-01 0:53 ` Ondřej Jirman
2016-07-01 0:53 ` Ondřej Jirman
2016-07-15 8:19 ` Maxime Ripard
2016-07-15 8:19 ` Maxime Ripard
2016-07-15 8:19 ` Maxime Ripard
2016-06-25 3:45 ` [PATCH v2 07/14] ARM: dts: sun8i: Use sun8i-h3-pll1-clk for pll1 in H3 megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-25 3:45 ` [PATCH v2 08/14] ARM: dts: sun8i: Add thermal sensor node to the sun8i-h3.dtsi megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-25 3:45 ` [PATCH v2 09/14] ARM: dts: sun8i: Add cpu0 label to sun8i-h3.dtsi megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-25 3:45 ` [PATCH v2 10/14] ARM: dts: sun8i: Add r_twi I2C controller megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-25 7:16 ` Maxime Ripard
2016-06-25 7:16 ` Maxime Ripard
2016-06-25 7:16 ` Maxime Ripard
2016-06-25 3:45 ` [PATCH v2 11/14] ARM: dts: sun8i: Add sy8106a regulator to Orange Pi PC megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-25 3:45 ` [PATCH v2 12/14] ARM: dts: sun8i: Setup CPU operating points for Onrage PI PC megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-25 3:45 ` [PATCH v2 13/14] ARM: dts: sun8i: Add gpio-regulator used on Orange Pi One megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-25 7:18 ` Maxime Ripard
2016-06-25 7:18 ` Maxime Ripard
2016-06-25 7:18 ` Maxime Ripard
2016-06-25 3:45 ` [PATCH v2 14/14] ARM: dts: sun8i: Enable DVFS " megous
2016-06-25 3:45 ` megous at megous.com
2016-06-25 3:45 ` megous-5qf/QAjKc83QT0dZR+AlfA
2016-06-30 11:13 ` [linux-sunxi] " Michal Suchanek
2016-06-30 11:13 ` Michal Suchanek
2016-06-30 11:13 ` Michal Suchanek
2016-06-30 14:19 ` Ondřej Jirman
2016-06-30 14:19 ` Ondřej Jirman
2016-06-30 14:19 ` Ondřej Jirman
2016-06-30 15:16 ` [linux-sunxi] " Michal Suchanek
2016-06-30 15:16 ` Michal Suchanek
2016-06-30 15:16 ` Michal Suchanek
2016-06-30 15:32 ` [linux-sunxi] " Ondřej Jirman
2016-06-30 15:32 ` Ondřej Jirman
2016-06-30 15:32 ` Ondřej Jirman
2016-06-30 15:50 ` [linux-sunxi] " Michal Suchanek
2016-06-30 15:50 ` Michal Suchanek
2016-06-30 15:50 ` Michal Suchanek
2016-06-30 15:53 ` [linux-sunxi] " Ondřej Jirman
2016-06-30 15:53 ` Ondřej Jirman
2016-06-30 15:53 ` Ondřej Jirman
2016-07-01 10:54 ` [linux-sunxi] " Michal Suchanek
2016-07-01 10:54 ` Michal Suchanek
2016-07-01 10:54 ` Michal Suchanek
2016-06-30 14:23 ` [linux-sunxi] " Siarhei Siamashka
2016-06-30 14:23 ` Siarhei Siamashka
2016-06-30 14:23 ` Siarhei Siamashka
2016-07-01 1:17 ` [linux-sunxi] " Ondřej Jirman
2016-07-01 1:17 ` Ondřej Jirman
2016-07-01 1:17 ` Ondřej Jirman
2016-07-22 0:41 ` [linux-sunxi] " Ondřej Jirman
2016-07-22 0:41 ` Ondřej Jirman
2016-07-22 0:41 ` Ondřej Jirman
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=20160701094719.9f7e28d93595d9cf7820e7dd@free.fr \
--to=moinejf@free.fr \
--cc=dev@linux-sunxi.org \
--cc=devicetree@vger.kernel.org \
--cc=emilio@elopez.com.ar \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=maxime.ripard@free-electrons.com \
--cc=megous@megous.com \
--cc=mturquette@baylibre.com \
--cc=robh+dt@kernel.org \
--cc=sboyd@codeaurora.org \
--cc=wens@csie.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.