From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter De Schrijver Subject: Re: Clocks used by another OS/CPU (was: Re: [RFC PATCH] clk: renesas: cpg-mssr: Add interface for critical core clocks) Date: Wed, 5 Jul 2017 10:25:44 +0300 Message-ID: <20170705072544.GN14819@tbergstrom-lnx.Nvidia.com> References: <20170630202453.eh6vaehkap3as4np@pengutronix.de> <8d86a9aa-1928-6b52-1487-d5fb9dae17f4@gmail.com> <20170701181408.yuocymwtj5dgt74d@pengutronix.de> <8d255a76-babe-312d-15b3-3474969f2410@arm.com> <20170704073137.GG14819@tbergstrom-lnx.Nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 8BIT Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-renesas-soc-owner@vger.kernel.org To: Sudeep Holla Cc: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , Dirk Behme , Rob Herring , Mark Rutland , "devicetree@vger.kernel.org" , Ulf Hansson , Dirk Behme , Linux PM list , "Rafael J. Wysocki" , Michael Turquette , Stephen Boyd , Linux-Renesas , Geert Uytterhoeven , Kevin Hilman , linux-clk , "linux-arm-kernel@lists.infradead.org" List-Id: linux-pm@vger.kernel.org On Tue, Jul 04, 2017 at 09:49:25AM +0100, Sudeep Holla wrote: > > > On 04/07/17 08:31, Peter De Schrijver wrote: > > On Mon, Jul 03, 2017 at 10:17:22AM +0100, Sudeep Holla wrote: > >> > >> > >> On 01/07/17 19:14, Uwe Kleine-König wrote: > >>> Hello, > >>> > >>> On Sat, Jul 01, 2017 at 07:02:48AM +0200, Dirk Behme wrote: > >> > >> [...] > >> > >>>> > >>>> > >>>> The other problem is security related. If, at all, you have to do it the > >>>> other way around, then: > >>>> > >>>> Make Linux a consumer of the other CPU's (trusted/trustzone/whatever > >>>> secured) OS clock driver. > >> > >> Yes, that's better and is getting common on newer platforms. They have > >> separate M-class(or even low A-class e.g. A5/A7) processors to handle > >> all the system management. > >> > >> The new ARM SCMI specification[0][1] is designed to standardize the > >> interface. It covers the clocks in clock protocol. > >> > > > > Yes, however this doesn't exist on older SoCs which still have multiple CPU's > > > > Agreed. But if someone is fixing/adding support in Linux as well as in > the other OS running on those cores, why not consider this interface > instead of trying to generalize something which will invariably SoC > specific. Because the firmware of the other CPUs might not be able to support this or is frozen and cannot be changed. Peter.