From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751858AbdF3MZm (ORCPT ); Fri, 30 Jun 2017 08:25:42 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:38355 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751545AbdF3MZl (ORCPT ); Fri, 30 Jun 2017 08:25:41 -0400 Date: Fri, 30 Jun 2017 14:25:29 +0200 From: Maxime Ripard To: Andre Przywara Cc: Jassi Brar , Sudeep Holla , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com, Chen-Yu Tsai , Icenowy Zheng , Rob Herring , Mark Rutland , devicetree@vger.kernel.org Subject: Re: [PATCH 0/8] mailbox: arm/arm64: introduce smc triggered mailbox Message-ID: <20170630122529.upjigip4hztrq67h@flea> References: <20170630095608.24943-1-andre.przywara@arm.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="l7shr4ksnggdol4b" Content-Disposition: inline In-Reply-To: <20170630095608.24943-1-andre.przywara@arm.com> User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --l7shr4ksnggdol4b Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Fri, Jun 30, 2017 at 10:56:00AM +0100, Andre Przywara wrote: > The remaining patches demonstrate usage of this feature to drive SCPI ser= vices > implemented as part of the ARM Trusted Firmware implementation used for > AArch64 based Allwinner SoCs, the Allwinner A64 in this example. > It allows to provide DVFS services, sensors support, device power domains > and potentially other services like clocks or regulators. > This allows to abstract those features in firmware, without the need to > implement explicit Linux support for each variant of some SoC design. > Those DT changes are not necessarily meant to be merged at this point. > I started implementing the firmware side of those services and put a WIP > branch on my ATF Github repo [1]. With this branch and these patches here > you get DVFS and temperature sensor support for the A64, just with this > driver and the generic SCPI support. I would go even further, and say that these changes should be done by the bootloader itself once it installed the proper monitor. These patches represent not a state of the hardware itself, but the state the bootloader let the hardware in, and that state will change =66rom one bootloader to the other, and one version to the other (obviously). This is already what we do for the other things the bootloader initializes (like simplefb, or PSCI). It just feels natural to do the same thing here. Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --l7shr4ksnggdol4b Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIcBAEBAgAGBQJZVkM5AAoJEBx+YmzsjxAg4MAP/iql+8UfaqhKFQoayyLaLq3a KCCg8oRcOpuhBtuRx9AQNbiwhUSkGTDp5HfKtfPKwCcm2PSCsRPQCqOoh+Uri3ji ULWfWDKluSZZh53WPrhXkV7eO94k+mExIRgISJl2Vj9w8YS9tG0IJITwbsBG2veq /ylxy7YawrQk8ECQBXpHfzR8Fmj7EXM2k6GwNCJ6IfAgfBpS54KSk2tKcdP6Vgv/ p0I97wKcD6/rqh6Nt2rIlY/aLFYhAgWd970ICPjFDnG3ZuDBdERgPpFnu5SxG4Is RWEuu64wyr26LR9Bw+YpfGEDvUmCZyyYsP+dpdBbabg5UwEqSgLTM6lHn6qSncSi l6mh8YwgXwaFqBFQgtwaiOGasty2YmSce9C7VMjnA9/A6j/yA1EsArpaff4rZm7Z Ad1sFoF7FtAr00o8p5DlFBAZ4uOHW9OzVkzHjMv5zhqwuQSBVe/Mmp+23xq8Ipf3 3Tf8sCoJpTKs9r+2U/1eWk7k8/yYGlZDNamenRRX4tTapU5KIN3jn3P+HHrW9DBu UvJjHlW04jjw0WHSZvUfc7eLewClxbOpgd/nY+LEJZmTs0KwRCzhiqK7i/NfuhB3 Bv0UW3KpVIWqxzrgl3BcAFnFNiW7FL9uab55nEMtEtGIuJHJg3Yr+7mcAPixdp/I 9STWBF4fBHrkKw7GIYTm =6pi/ -----END PGP SIGNATURE----- --l7shr4ksnggdol4b-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Subject: Re: [PATCH 0/8] mailbox: arm/arm64: introduce smc triggered mailbox Date: Fri, 30 Jun 2017 14:25:29 +0200 Message-ID: <20170630122529.upjigip4hztrq67h@flea> References: <20170630095608.24943-1-andre.przywara@arm.com> Reply-To: maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="l7shr4ksnggdol4b" Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Content-Disposition: inline In-Reply-To: <20170630095608.24943-1-andre.przywara-5wv7dgnIgG8@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Andre Przywara Cc: Jassi Brar , Sudeep Holla , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, Chen-Yu Tsai , Icenowy Zheng , Rob Herring , Mark Rutland , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org --l7shr4ksnggdol4b Content-Type: text/plain; charset="UTF-8" Content-Disposition: inline Hi, On Fri, Jun 30, 2017 at 10:56:00AM +0100, Andre Przywara wrote: > The remaining patches demonstrate usage of this feature to drive SCPI services > implemented as part of the ARM Trusted Firmware implementation used for > AArch64 based Allwinner SoCs, the Allwinner A64 in this example. > It allows to provide DVFS services, sensors support, device power domains > and potentially other services like clocks or regulators. > This allows to abstract those features in firmware, without the need to > implement explicit Linux support for each variant of some SoC design. > Those DT changes are not necessarily meant to be merged at this point. > I started implementing the firmware side of those services and put a WIP > branch on my ATF Github repo [1]. With this branch and these patches here > you get DVFS and temperature sensor support for the A64, just with this > driver and the generic SCPI support. I would go even further, and say that these changes should be done by the bootloader itself once it installed the proper monitor. These patches represent not a state of the hardware itself, but the state the bootloader let the hardware in, and that state will change from one bootloader to the other, and one version to the other (obviously). This is already what we do for the other things the bootloader initializes (like simplefb, or PSCI). It just feels natural to do the same thing here. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --l7shr4ksnggdol4b-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: maxime.ripard@free-electrons.com (Maxime Ripard) Date: Fri, 30 Jun 2017 14:25:29 +0200 Subject: [PATCH 0/8] mailbox: arm/arm64: introduce smc triggered mailbox In-Reply-To: <20170630095608.24943-1-andre.przywara@arm.com> References: <20170630095608.24943-1-andre.przywara@arm.com> Message-ID: <20170630122529.upjigip4hztrq67h@flea> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, On Fri, Jun 30, 2017 at 10:56:00AM +0100, Andre Przywara wrote: > The remaining patches demonstrate usage of this feature to drive SCPI services > implemented as part of the ARM Trusted Firmware implementation used for > AArch64 based Allwinner SoCs, the Allwinner A64 in this example. > It allows to provide DVFS services, sensors support, device power domains > and potentially other services like clocks or regulators. > This allows to abstract those features in firmware, without the need to > implement explicit Linux support for each variant of some SoC design. > Those DT changes are not necessarily meant to be merged at this point. > I started implementing the firmware side of those services and put a WIP > branch on my ATF Github repo [1]. With this branch and these patches here > you get DVFS and temperature sensor support for the A64, just with this > driver and the generic SCPI support. I would go even further, and say that these changes should be done by the bootloader itself once it installed the proper monitor. These patches represent not a state of the hardware itself, but the state the bootloader let the hardware in, and that state will change from one bootloader to the other, and one version to the other (obviously). This is already what we do for the other things the bootloader initializes (like simplefb, or PSCI). It just feels natural to do the same thing here. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: