* [PATCH] ARM: dts: imx6q-bx50v3: Enable secure-reg-access @ 2018-01-15 16:07 Sebastian Reichel 2018-02-02 7:18 ` Shawn Guo 0 siblings, 1 reply; 8+ messages in thread From: Sebastian Reichel @ 2018-01-15 16:07 UTC (permalink / raw) To: Shawn Guo, Sascha Hauer, Fabio Estevam Cc: Ian Ray, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Sebastian Reichel From: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> Add secure-reg-access on device tree include file for Bx50 devices to enable PMU and hardware counters for perf. Signed-off-by: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> Signed-off-by: Sebastian Reichel <sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> --- arch/arm/boot/dts/imx6q-bx50v3.dtsi | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi index 86cfd4481e72..ccaaee83e2fa 100644 --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi @@ -43,6 +43,13 @@ #include "imx6q-ba16.dtsi" / { + soc { + pmu { + compatible = "arm,cortex-a9-pmu"; + secure-reg-access; + }; + }; + clocks { mclk: clock@0 { compatible = "fixed-clock"; -- 2.15.1 -- 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 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: imx6q-bx50v3: Enable secure-reg-access 2018-01-15 16:07 [PATCH] ARM: dts: imx6q-bx50v3: Enable secure-reg-access Sebastian Reichel @ 2018-02-02 7:18 ` Shawn Guo 2018-02-02 11:10 ` Sebastian Reichel 2018-02-02 15:33 ` [PATCH] " Frank Li 0 siblings, 2 replies; 8+ messages in thread From: Shawn Guo @ 2018-02-02 7:18 UTC (permalink / raw) To: Sebastian Reichel Cc: Sascha Hauer, Frank Li, Fabio Estevam, Ian Ray, Rob Herring, Mark Rutland, devicetree, linux-kernel + Frank On Mon, Jan 15, 2018 at 05:07:22PM +0100, Sebastian Reichel wrote: > From: Peter Senna Tschudin <peter.senna@collabora.com> > > Add secure-reg-access on device tree include file for Bx50 devices > to enable PMU and hardware counters for perf. > > Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com> > Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk> > --- > arch/arm/boot/dts/imx6q-bx50v3.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > index 86cfd4481e72..ccaaee83e2fa 100644 > --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi > +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > @@ -43,6 +43,13 @@ > #include "imx6q-ba16.dtsi" > > / { > + soc { > + pmu { > + compatible = "arm,cortex-a9-pmu"; > + secure-reg-access; I'm not sure this could be a board level configuration. Shouldn't this property just be added into pmu node in imx6qdl.dtsi? Shawn > + }; > + }; > + > clocks { > mclk: clock@0 { > compatible = "fixed-clock"; > -- > 2.15.1 > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: imx6q-bx50v3: Enable secure-reg-access 2018-02-02 7:18 ` Shawn Guo @ 2018-02-02 11:10 ` Sebastian Reichel 2018-02-05 5:27 ` Shawn Guo 2018-02-02 15:33 ` [PATCH] " Frank Li 1 sibling, 1 reply; 8+ messages in thread From: Sebastian Reichel @ 2018-02-02 11:10 UTC (permalink / raw) To: Shawn Guo Cc: Sascha Hauer, Frank Li, Fabio Estevam, Ian Ray, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 2439 bytes --] Hi, On Fri, Feb 02, 2018 at 03:18:20PM +0800, Shawn Guo wrote: > + Frank > > On Mon, Jan 15, 2018 at 05:07:22PM +0100, Sebastian Reichel wrote: > > From: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> > > > > Add secure-reg-access on device tree include file for Bx50 devices > > to enable PMU and hardware counters for perf. > > > > Signed-off-by: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> > > Signed-off-by: Sebastian Reichel <sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> > > --- > > arch/arm/boot/dts/imx6q-bx50v3.dtsi | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > index 86cfd4481e72..ccaaee83e2fa 100644 > > --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > @@ -43,6 +43,13 @@ > > #include "imx6q-ba16.dtsi" > > > > / { > > + soc { > > + pmu { > > + compatible = "arm,cortex-a9-pmu"; > > + secure-reg-access; > > I'm not sure this could be a board level configuration. Shouldn't this > property just be added into pmu node in imx6qdl.dtsi? This cannot become part of imx6qdl.dtsi for two reasons: 1. It's not safe to be enabled on imx6qdl in secure state, see excerpt from DT binding documentation below 2. We cannot enable it by default, since DTS format does not provide a method to unset it in board files using a imx6qdl in secure mode. Here is an extract from the DT documentation for the PMU node: - secure-reg-access : Indicates that the ARMv7 Secure Debug Enable Register (SDER) is accessible. This will cause the driver to do any setup required that is only possible in ARMv7 secure state. If not present the ARMv7 SDER will not be touched, which means the PMU may fail to operate unless external code (bootloader or security monitor) has performed the appropriate initialisation. Note that this property is not valid for non-ARMv7 CPUs or ARMv7 CPUs booting Linux in Non-secure state. For B450v3, B650v3 and B850v3 it's safe to enable the bit and it improves debugging capabilities a lot. -- Sebastian > > Shawn > > > + }; > > + }; > > + > > clocks { > > mclk: clock@0 { > > compatible = "fixed-clock"; > > -- > > 2.15.1 > > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: imx6q-bx50v3: Enable secure-reg-access 2018-02-02 11:10 ` Sebastian Reichel @ 2018-02-05 5:27 ` Shawn Guo 2018-02-05 17:08 ` [PATCHv2] " Sebastian Reichel 0 siblings, 1 reply; 8+ messages in thread From: Shawn Guo @ 2018-02-05 5:27 UTC (permalink / raw) To: Sebastian Reichel Cc: Sascha Hauer, Frank Li, Fabio Estevam, Ian Ray, Rob Herring, Mark Rutland, devicetree, linux-kernel On Fri, Feb 02, 2018 at 12:10:04PM +0100, Sebastian Reichel wrote: > Hi, > > On Fri, Feb 02, 2018 at 03:18:20PM +0800, Shawn Guo wrote: > > + Frank > > > > On Mon, Jan 15, 2018 at 05:07:22PM +0100, Sebastian Reichel wrote: > > > From: Peter Senna Tschudin <peter.senna@collabora.com> > > > > > > Add secure-reg-access on device tree include file for Bx50 devices > > > to enable PMU and hardware counters for perf. > > > > > > Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com> > > > Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk> > > > --- > > > arch/arm/boot/dts/imx6q-bx50v3.dtsi | 7 +++++++ > > > 1 file changed, 7 insertions(+) > > > > > > diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > > index 86cfd4481e72..ccaaee83e2fa 100644 > > > --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > > +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > > @@ -43,6 +43,13 @@ > > > #include "imx6q-ba16.dtsi" > > > > > > / { > > > + soc { > > > + pmu { > > > + compatible = "arm,cortex-a9-pmu"; > > > + secure-reg-access; > > > > I'm not sure this could be a board level configuration. Shouldn't this > > property just be added into pmu node in imx6qdl.dtsi? > > This cannot become part of imx6qdl.dtsi for two reasons: > > 1. It's not safe to be enabled on imx6qdl in secure state, see > excerpt from DT binding documentation below > 2. We cannot enable it by default, since DTS format does not provide > a method to unset it in board files using a imx6qdl in secure > mode. > > Here is an extract from the DT documentation for the PMU node: > > - secure-reg-access : Indicates that the ARMv7 Secure Debug Enable Register > (SDER) is accessible. This will cause the driver to do > any setup required that is only possible in ARMv7 secure > state. If not present the ARMv7 SDER will not be touched, > which means the PMU may fail to operate unless external > code (bootloader or security monitor) has performed the > appropriate initialisation. Note that this property is > not valid for non-ARMv7 CPUs or ARMv7 CPUs booting Linux > in Non-secure state. > > For B450v3, B650v3 and B850v3 it's safe to enable the bit and it > improves debugging capabilities a lot. Okay. Then please label the pmu node in imx6qdl.dtsi and referencing the node in board dts without duplicating soc node and compatible. Shawn ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCHv2] ARM: dts: imx6q-bx50v3: Enable secure-reg-access 2018-02-05 5:27 ` Shawn Guo @ 2018-02-05 17:08 ` Sebastian Reichel [not found] ` <20180205170840.14045-1-sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> 0 siblings, 1 reply; 8+ messages in thread From: Sebastian Reichel @ 2018-02-05 17:08 UTC (permalink / raw) To: Shawn Guo, Sascha Hauer, Fabio Estevam Cc: Ian Ray, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, kernel-ZGY8ohtN/8qB+jHODAdFcQ, Sebastian Reichel From: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> Enable secure debug enable register access for Bx50v3 devices to enable PMU and hardware counters for perf. Signed-off-by: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> Signed-off-by: Sebastian Reichel <sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> --- Changes since PATCHv1: * add property by adding a label to the pmu node in imx6qdl and referencing it in bx50v3.dtsi. --- arch/arm/boot/dts/imx6q-bx50v3.dtsi | 4 ++++ arch/arm/boot/dts/imx6qdl.dtsi | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi index 916ea94d75ca..2baffc8b9094 100644 --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi @@ -388,3 +388,7 @@ #interrupt-cells = <1>; }; }; + +&pmu { + secure-reg-access; +}; diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi index 59ff86695a14..2b921de44c02 100644 --- a/arch/arm/boot/dts/imx6qdl.dtsi +++ b/arch/arm/boot/dts/imx6qdl.dtsi @@ -143,7 +143,7 @@ }; }; - pmu { + pmu: pmu { compatible = "arm,cortex-a9-pmu"; interrupt-parent = <&gpc>; interrupts = <0 94 IRQ_TYPE_LEVEL_HIGH>; -- 2.15.1 -- 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 ^ permalink raw reply related [flat|nested] 8+ messages in thread
[parent not found: <20180205170840.14045-1-sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org>]
* Re: [PATCHv2] ARM: dts: imx6q-bx50v3: Enable secure-reg-access [not found] ` <20180205170840.14045-1-sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> @ 2018-02-05 17:17 ` Fabio Estevam 2018-02-08 6:44 ` Shawn Guo 1 sibling, 0 replies; 8+ messages in thread From: Fabio Estevam @ 2018-02-05 17:17 UTC (permalink / raw) To: Sebastian Reichel Cc: Shawn Guo, Sascha Hauer, Fabio Estevam, Ian Ray, Rob Herring, Mark Rutland, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, linux-kernel, kernel-ZGY8ohtN/8qB+jHODAdFcQ On Mon, Feb 5, 2018 at 3:08 PM, Sebastian Reichel <sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> wrote: > From: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> > > Enable secure debug enable register access for Bx50v3 devices to enable > PMU and hardware counters for perf. > > Signed-off-by: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> > Signed-off-by: Sebastian Reichel <sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> Reviewed-by: Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org> -- 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCHv2] ARM: dts: imx6q-bx50v3: Enable secure-reg-access [not found] ` <20180205170840.14045-1-sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> 2018-02-05 17:17 ` Fabio Estevam @ 2018-02-08 6:44 ` Shawn Guo 1 sibling, 0 replies; 8+ messages in thread From: Shawn Guo @ 2018-02-08 6:44 UTC (permalink / raw) To: Sebastian Reichel Cc: Sascha Hauer, Fabio Estevam, Ian Ray, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, kernel-ZGY8ohtN/8qB+jHODAdFcQ On Mon, Feb 05, 2018 at 06:08:40PM +0100, Sebastian Reichel wrote: > From: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> > > Enable secure debug enable register access for Bx50v3 devices to enable > PMU and hardware counters for perf. > > Signed-off-by: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> > Signed-off-by: Sebastian Reichel <sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> > --- > Changes since PATCHv1: > * add property by adding a label to the pmu node in imx6qdl and > referencing it in bx50v3.dtsi. > --- > arch/arm/boot/dts/imx6q-bx50v3.dtsi | 4 ++++ > arch/arm/boot/dts/imx6qdl.dtsi | 2 +- > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > index 916ea94d75ca..2baffc8b9094 100644 > --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi > +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > @@ -388,3 +388,7 @@ > #interrupt-cells = <1>; > }; > }; > + > +&pmu { > + secure-reg-access; > +}; The labelled node should be sorted alphabetically. That said, it should be added before &usdhc2. I fixed it up and applied the patch. Shawn > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi > index 59ff86695a14..2b921de44c02 100644 > --- a/arch/arm/boot/dts/imx6qdl.dtsi > +++ b/arch/arm/boot/dts/imx6qdl.dtsi > @@ -143,7 +143,7 @@ > }; > }; > > - pmu { > + pmu: pmu { > compatible = "arm,cortex-a9-pmu"; > interrupt-parent = <&gpc>; > interrupts = <0 94 IRQ_TYPE_LEVEL_HIGH>; > -- > 2.15.1 > -- 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH] ARM: dts: imx6q-bx50v3: Enable secure-reg-access 2018-02-02 7:18 ` Shawn Guo 2018-02-02 11:10 ` Sebastian Reichel @ 2018-02-02 15:33 ` Frank Li 1 sibling, 0 replies; 8+ messages in thread From: Frank Li @ 2018-02-02 15:33 UTC (permalink / raw) To: Shawn Guo, Sebastian Reichel Cc: Sascha Hauer, Fabio Estevam, Ian Ray, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA > -----Original Message----- > From: Shawn Guo [mailto:shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org] > Sent: Friday, February 02, 2018 1:18 AM > To: Sebastian Reichel <sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> > Cc: Sascha Hauer <kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>; Frank Li <frank.li-3arQi8VN3Tc@public.gmane.org>; > Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org>; Ian Ray <ian.ray-JJi787mZWgc@public.gmane.org>; Rob > Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>; Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>; > devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Subject: Re: [PATCH] ARM: dts: imx6q-bx50v3: Enable secure-reg-access > > + Frank > > On Mon, Jan 15, 2018 at 05:07:22PM +0100, Sebastian Reichel wrote: > > From: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> > > > > Add secure-reg-access on device tree include file for Bx50 devices to > > enable PMU and hardware counters for perf. > > > > Signed-off-by: Peter Senna Tschudin <peter.senna-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org> > > Signed-off-by: Sebastian Reichel <sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org> > > --- > > arch/arm/boot/dts/imx6q-bx50v3.dtsi | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > index 86cfd4481e72..ccaaee83e2fa 100644 > > --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi > > @@ -43,6 +43,13 @@ > > #include "imx6q-ba16.dtsi" > > > > / { > > + soc { > > + pmu { > > + compatible = "arm,cortex-a9-pmu"; > > + secure-reg-access; > > I'm not sure this could be a board level configuration. Shouldn't this property > just be added into pmu node in imx6qdl.dtsi? I agree. It should be in imx6qdl.dtsi Frank Li > > Shawn > > > + }; > > + }; > > + > > clocks { > > mclk: clock@0 { > > compatible = "fixed-clock"; > > -- > > 2.15.1 > > -- 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-02-08 6:44 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-15 16:07 [PATCH] ARM: dts: imx6q-bx50v3: Enable secure-reg-access Sebastian Reichel
2018-02-02 7:18 ` Shawn Guo
2018-02-02 11:10 ` Sebastian Reichel
2018-02-05 5:27 ` Shawn Guo
2018-02-05 17:08 ` [PATCHv2] " Sebastian Reichel
[not found] ` <20180205170840.14045-1-sebastian.reichel-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org>
2018-02-05 17:17 ` Fabio Estevam
2018-02-08 6:44 ` Shawn Guo
2018-02-02 15:33 ` [PATCH] " Frank Li
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).