Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-18 Thread Marc Zyngier
On 18/05/18 10:37, Stefan Wahren wrote: > Hi Marc, > >> Marc Zyngier hat am 18. Mai 2018 um 10:07 geschrieben: >> >> I have a pretty simple series[1] which I used to profile 32bit guests on >> an arm64 KVM host. Nobody really cared about it because running a 32bit >> kernel

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-18 Thread Marc Zyngier
On 18/05/18 10:37, Stefan Wahren wrote: > Hi Marc, > >> Marc Zyngier hat am 18. Mai 2018 um 10:07 geschrieben: >> >> I have a pretty simple series[1] which I used to profile 32bit guests on >> an arm64 KVM host. Nobody really cared about it because running a 32bit >> kernel on 64bit HW is a bit

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-18 Thread Stefan Wahren
Hi Marc, > Marc Zyngier hat am 18. Mai 2018 um 10:07 geschrieben: > > I have a pretty simple series[1] which I used to profile 32bit guests on > an arm64 KVM host. Nobody really cared about it because running a 32bit > kernel on 64bit HW is a bit odd, to say the least, and

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-18 Thread Stefan Wahren
Hi Marc, > Marc Zyngier hat am 18. Mai 2018 um 10:07 geschrieben: > > I have a pretty simple series[1] which I used to profile 32bit guests on > an arm64 KVM host. Nobody really cared about it because running a 32bit > kernel on 64bit HW is a bit odd, to say the least, and I'm probably the >

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-18 Thread Marc Zyngier
On 17/05/18 20:59, Florian Fainelli wrote: > On 05/17/2018 12:31 PM, Vince Weaver wrote: >> On Thu, 17 May 2018, Vince Weaver wrote: >> >>> On Thu, 17 May 2018, Peter Zijlstra wrote: >>> with cortex-a7 now, would it be possible to later drop that if proper >>> cortex-a53 support is added to the

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-18 Thread Marc Zyngier
On 17/05/18 20:59, Florian Fainelli wrote: > On 05/17/2018 12:31 PM, Vince Weaver wrote: >> On Thu, 17 May 2018, Vince Weaver wrote: >> >>> On Thu, 17 May 2018, Peter Zijlstra wrote: >>> with cortex-a7 now, would it be possible to later drop that if proper >>> cortex-a53 support is added to the

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Florian Fainelli
On 05/17/2018 12:31 PM, Vince Weaver wrote: > On Thu, 17 May 2018, Vince Weaver wrote: > >> On Thu, 17 May 2018, Peter Zijlstra wrote: >> with cortex-a7 now, would it be possible to later drop that if proper >> cortex-a53 support is added to the armv7 pmu driver? Or would that lead >> to all

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Florian Fainelli
On 05/17/2018 12:31 PM, Vince Weaver wrote: > On Thu, 17 May 2018, Vince Weaver wrote: > >> On Thu, 17 May 2018, Peter Zijlstra wrote: >> with cortex-a7 now, would it be possible to later drop that if proper >> cortex-a53 support is added to the armv7 pmu driver? Or would that lead >> to all

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Vince Weaver wrote: > On Thu, 17 May 2018, Peter Zijlstra wrote: > with cortex-a7 now, would it be possible to later drop that if proper > cortex-a53 support is added to the armv7 pmu driver? Or would that lead > to all kinds of back-compatability mess? For what it's

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Vince Weaver wrote: > On Thu, 17 May 2018, Peter Zijlstra wrote: > with cortex-a7 now, would it be possible to later drop that if proper > cortex-a53 support is added to the armv7 pmu driver? Or would that lead > to all kinds of back-compatability mess? For what it's

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Peter Zijlstra wrote: > On Thu, May 17, 2018 at 06:55:26PM +0200, Stefan Wahren wrote: > > > Vince Weaver hat am 17. Mai 2018 um 18:34 > > > geschrieben: > > > On Thu, 17 May 2018, Stefan Wahren wrote: > > > > > Eric Anholt hat am

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Peter Zijlstra wrote: > On Thu, May 17, 2018 at 06:55:26PM +0200, Stefan Wahren wrote: > > > Vince Weaver hat am 17. Mai 2018 um 18:34 > > > geschrieben: > > > On Thu, 17 May 2018, Stefan Wahren wrote: > > > > > Eric Anholt hat am 17. Mai 2018 um 15:17 > > > > >

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Peter Zijlstra
On Thu, May 17, 2018 at 06:55:26PM +0200, Stefan Wahren wrote: > > Vince Weaver hat am 17. Mai 2018 um 18:34 > > geschrieben: > > On Thu, 17 May 2018, Stefan Wahren wrote: > > > > Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: > > > > The a53

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Peter Zijlstra
On Thu, May 17, 2018 at 06:55:26PM +0200, Stefan Wahren wrote: > > Vince Weaver hat am 17. Mai 2018 um 18:34 > > geschrieben: > > On Thu, 17 May 2018, Stefan Wahren wrote: > > > > Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: > > > > The a53 and a7 counters seem to match up, so we

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Eric Anholt
Vince Weaver writes: > On Thu, 17 May 2018, Stefan Wahren wrote: > >> >> > Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: >> > >> > >> > The a53 and a7 counters seem to match up, so we advertise a7 so that >> > arm32 can probe. > > so how

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Eric Anholt
Vince Weaver writes: > On Thu, 17 May 2018, Stefan Wahren wrote: > >> >> > Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: >> > >> > >> > The a53 and a7 counters seem to match up, so we advertise a7 so that >> > arm32 can probe. > > so how closely did you look at the a53/a7

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Stefan Wahren
Hi, [added Peter, Ingo and Arnaldo] > Vince Weaver hat am 17. Mai 2018 um 18:34 > geschrieben: > > > On Thu, 17 May 2018, Stefan Wahren wrote: > > > > > > Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: > > > > > > > > > The a53 and a7

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Stefan Wahren
Hi, [added Peter, Ingo and Arnaldo] > Vince Weaver hat am 17. Mai 2018 um 18:34 > geschrieben: > > > On Thu, 17 May 2018, Stefan Wahren wrote: > > > > > > Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: > > > > > > > > > The a53 and a7 counters seem to match up, so we advertise a7

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Peter Robinson
On Thu, May 17, 2018 at 2:17 PM, Eric Anholt wrote: > The a53 and a7 counters seem to match up, so we advertise a7 so that > arm32 can probe. > > Signed-off-by: Eric Anholt Tested-by: Peter Robinson We've carried the same/equiv patch in

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Peter Robinson
On Thu, May 17, 2018 at 2:17 PM, Eric Anholt wrote: > The a53 and a7 counters seem to match up, so we advertise a7 so that > arm32 can probe. > > Signed-off-by: Eric Anholt Tested-by: Peter Robinson We've carried the same/equiv patch in Fedora for a while with no issues. Peter > --- >

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Stefan Wahren wrote: > > > Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: > > > > > > The a53 and a7 counters seem to match up, so we advertise a7 so that > > arm32 can probe. so how closely did you look at the a53/a7 differences? I see some

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Stefan Wahren wrote: > > > Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: > > > > > > The a53 and a7 counters seem to match up, so we advertise a7 so that > > arm32 can probe. so how closely did you look at the a53/a7 differences? I see some major differences,

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Stefan Wahren
> Vince Weaver hat am 17. Mai 2018 um 16:11 > geschrieben: > > > On Thu, 17 May 2018, Eric Anholt wrote: > > > diff --git a/arch/arm/boot/dts/bcm2837.dtsi b/arch/arm/boot/dts/bcm2837.dtsi > > index 7704bb029605..1f5e5c782835 100644 > > ---

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Stefan Wahren
> Vince Weaver hat am 17. Mai 2018 um 16:11 > geschrieben: > > > On Thu, 17 May 2018, Eric Anholt wrote: > > > diff --git a/arch/arm/boot/dts/bcm2837.dtsi b/arch/arm/boot/dts/bcm2837.dtsi > > index 7704bb029605..1f5e5c782835 100644 > > --- a/arch/arm/boot/dts/bcm2837.dtsi > > +++

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Eric Anholt wrote: > > Is that better than a53? I'm happy to switch to that. The important > part to me is the a7, since basically everyone with this hw is running > arm32. no, on further investigation it looks like a53 is more proper to use than the generic armv8. Is

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Eric Anholt wrote: > > Is that better than a53? I'm happy to switch to that. The important > part to me is the a7, since basically everyone with this hw is running > arm32. no, on further investigation it looks like a53 is more proper to use than the generic armv8. Is

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Stefan Wahren
> Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: > > > The a53 and a7 counters seem to match up, so we advertise a7 so that > arm32 can probe. > > Signed-off-by: Eric Anholt Acked-by: Stefan Wahren

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Stefan Wahren
> Eric Anholt hat am 17. Mai 2018 um 15:17 geschrieben: > > > The a53 and a7 counters seem to match up, so we advertise a7 so that > arm32 can probe. > > Signed-off-by: Eric Anholt Acked-by: Stefan Wahren

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Eric Anholt
Vince Weaver writes: > On Thu, 17 May 2018, Eric Anholt wrote: > >> diff --git a/arch/arm/boot/dts/bcm2837.dtsi b/arch/arm/boot/dts/bcm2837.dtsi >> index 7704bb029605..1f5e5c782835 100644 >> --- a/arch/arm/boot/dts/bcm2837.dtsi >> +++ b/arch/arm/boot/dts/bcm2837.dtsi >>

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Eric Anholt
Vince Weaver writes: > On Thu, 17 May 2018, Eric Anholt wrote: > >> diff --git a/arch/arm/boot/dts/bcm2837.dtsi b/arch/arm/boot/dts/bcm2837.dtsi >> index 7704bb029605..1f5e5c782835 100644 >> --- a/arch/arm/boot/dts/bcm2837.dtsi >> +++ b/arch/arm/boot/dts/bcm2837.dtsi >> @@ -17,6 +17,12 @@ >>

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Eric Anholt wrote: > diff --git a/arch/arm/boot/dts/bcm2837.dtsi b/arch/arm/boot/dts/bcm2837.dtsi > index 7704bb029605..1f5e5c782835 100644 > --- a/arch/arm/boot/dts/bcm2837.dtsi > +++ b/arch/arm/boot/dts/bcm2837.dtsi > @@ -17,6 +17,12 @@ > }; > }; > > +

Re: [PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Vince Weaver
On Thu, 17 May 2018, Eric Anholt wrote: > diff --git a/arch/arm/boot/dts/bcm2837.dtsi b/arch/arm/boot/dts/bcm2837.dtsi > index 7704bb029605..1f5e5c782835 100644 > --- a/arch/arm/boot/dts/bcm2837.dtsi > +++ b/arch/arm/boot/dts/bcm2837.dtsi > @@ -17,6 +17,12 @@ > }; > }; > > +

[PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Eric Anholt
The a53 and a7 counters seem to match up, so we advertise a7 so that arm32 can probe. Signed-off-by: Eric Anholt --- arch/arm/boot/dts/bcm2837.dtsi | 6 ++ 1 file changed, 6 insertions(+) diff --git a/arch/arm/boot/dts/bcm2837.dtsi b/arch/arm/boot/dts/bcm2837.dtsi index

[PATCH] arm: bcm2835: Add the PMU to the devicetree.

2018-05-17 Thread Eric Anholt
The a53 and a7 counters seem to match up, so we advertise a7 so that arm32 can probe. Signed-off-by: Eric Anholt --- arch/arm/boot/dts/bcm2837.dtsi | 6 ++ 1 file changed, 6 insertions(+) diff --git a/arch/arm/boot/dts/bcm2837.dtsi b/arch/arm/boot/dts/bcm2837.dtsi index