All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sudeep Holla <sudeep.holla@arm.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 3/3] arm64: dts: r8a7795: Add pmu device nodes
Date: Tue, 15 Dec 2015 09:50:32 +0000	[thread overview]
Message-ID: <566FE268.70802@arm.com> (raw)
In-Reply-To: <CAMuHMdU940q78hpDjJEHkeyqRRPTWLxqR46K69DCTMOJHBbKcA@mail.gmail.com>



On 15/12/15 08:33, Geert Uytterhoeven wrote:
> On Mon, Dec 14, 2015 at 3:36 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
>> On 04/12/15 13:38, Dirk Behme wrote:
>>>
>>> From: Yoshifumi Hosoya <yoshifumi.hosoya.wj@renesas.com>
>>>
>>> Enabling the performance monitor unit on r8a7795.
>>>
>>> Signed-off-by: Masaru Nagai <masaru.nagai.vx@renesas.com>
>>> Signed-off-by: Yoshifumi Hosoya <yoshifumi.hosoya.wj@renesas.com>
>>> Sigend-off-by: Dirk Behme <dirk.behme@gmail.com>
>>> ---
>>> Note: This patch picked from
>>>
>>>
>>> https://git.kernel.org/cgit/linux/kernel/git/horms/renesas-bsp.git/log/?h=v4.2/rcar-3.0.x
>>>
>>> and rebased against
>>>
>>>
>>> https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/gen3-latest
>>> renesas-drivers-2015-12-01-v4.4-rc3
>>>
>>>    arch/arm64/boot/dts/renesas/r8a7795.dtsi | 12 ++++++++++++
>>>    1 file changed, 12 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> index 946a5bb..8113b71 100644
>>> --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> @@ -247,6 +247,18 @@
>>>                          power-domains = <&cpg>;
>>>                  };
>>>
>>> +               pmu {
>>> +                       compatible = "arm,armv8-pmuv3";
>>
>>
>> You can add more specific compatible "arm,cortex-a57-pmu" if it's Cortex
>> A57.
>
> Except that one is not documented.
>
> Oh, it is. Someone mistyped the commas. Patch sent.
>

Ah right, nice catch. Thanks for the patch.

-- 
Regards,
Sudeep

WARNING: multiple messages have this Message-ID (diff)
From: sudeep.holla@arm.com (Sudeep Holla)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/3] arm64: dts: r8a7795: Add pmu device nodes
Date: Tue, 15 Dec 2015 09:50:32 +0000	[thread overview]
Message-ID: <566FE268.70802@arm.com> (raw)
In-Reply-To: <CAMuHMdU940q78hpDjJEHkeyqRRPTWLxqR46K69DCTMOJHBbKcA@mail.gmail.com>



On 15/12/15 08:33, Geert Uytterhoeven wrote:
> On Mon, Dec 14, 2015 at 3:36 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
>> On 04/12/15 13:38, Dirk Behme wrote:
>>>
>>> From: Yoshifumi Hosoya <yoshifumi.hosoya.wj@renesas.com>
>>>
>>> Enabling the performance monitor unit on r8a7795.
>>>
>>> Signed-off-by: Masaru Nagai <masaru.nagai.vx@renesas.com>
>>> Signed-off-by: Yoshifumi Hosoya <yoshifumi.hosoya.wj@renesas.com>
>>> Sigend-off-by: Dirk Behme <dirk.behme@gmail.com>
>>> ---
>>> Note: This patch picked from
>>>
>>>
>>> https://git.kernel.org/cgit/linux/kernel/git/horms/renesas-bsp.git/log/?h=v4.2/rcar-3.0.x
>>>
>>> and rebased against
>>>
>>>
>>> https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/gen3-latest
>>> renesas-drivers-2015-12-01-v4.4-rc3
>>>
>>>    arch/arm64/boot/dts/renesas/r8a7795.dtsi | 12 ++++++++++++
>>>    1 file changed, 12 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> index 946a5bb..8113b71 100644
>>> --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> @@ -247,6 +247,18 @@
>>>                          power-domains = <&cpg>;
>>>                  };
>>>
>>> +               pmu {
>>> +                       compatible = "arm,armv8-pmuv3";
>>
>>
>> You can add more specific compatible "arm,cortex-a57-pmu" if it's Cortex
>> A57.
>
> Except that one is not documented.
>
> Oh, it is. Someone mistyped the commas. Patch sent.
>

Ah right, nice catch. Thanks for the patch.

-- 
Regards,
Sudeep

WARNING: multiple messages have this Message-ID (diff)
From: Sudeep Holla <sudeep.holla@arm.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Sudeep Holla <sudeep.holla@arm.com>,
	Dirk Behme <dirk.behme@gmail.com>,
	Linux-sh list <linux-sh@vger.kernel.org>,
	Simon Horman <horms@verge.net.au>,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	Yoshifumi Hosoya <yoshifumi.hosoya.wj@renesas.com>,
	Masaru Nagai <masaru.nagai.vx@renesas.com>
Subject: Re: [PATCH 3/3] arm64: dts: r8a7795: Add pmu device nodes
Date: Tue, 15 Dec 2015 09:50:32 +0000	[thread overview]
Message-ID: <566FE268.70802@arm.com> (raw)
In-Reply-To: <CAMuHMdU940q78hpDjJEHkeyqRRPTWLxqR46K69DCTMOJHBbKcA@mail.gmail.com>



On 15/12/15 08:33, Geert Uytterhoeven wrote:
> On Mon, Dec 14, 2015 at 3:36 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
>> On 04/12/15 13:38, Dirk Behme wrote:
>>>
>>> From: Yoshifumi Hosoya <yoshifumi.hosoya.wj@renesas.com>
>>>
>>> Enabling the performance monitor unit on r8a7795.
>>>
>>> Signed-off-by: Masaru Nagai <masaru.nagai.vx@renesas.com>
>>> Signed-off-by: Yoshifumi Hosoya <yoshifumi.hosoya.wj@renesas.com>
>>> Sigend-off-by: Dirk Behme <dirk.behme@gmail.com>
>>> ---
>>> Note: This patch picked from
>>>
>>>
>>> https://git.kernel.org/cgit/linux/kernel/git/horms/renesas-bsp.git/log/?h=v4.2/rcar-3.0.x
>>>
>>> and rebased against
>>>
>>>
>>> https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/gen3-latest
>>> renesas-drivers-2015-12-01-v4.4-rc3
>>>
>>>    arch/arm64/boot/dts/renesas/r8a7795.dtsi | 12 ++++++++++++
>>>    1 file changed, 12 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> index 946a5bb..8113b71 100644
>>> --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
>>> @@ -247,6 +247,18 @@
>>>                          power-domains = <&cpg>;
>>>                  };
>>>
>>> +               pmu {
>>> +                       compatible = "arm,armv8-pmuv3";
>>
>>
>> You can add more specific compatible "arm,cortex-a57-pmu" if it's Cortex
>> A57.
>
> Except that one is not documented.
>
> Oh, it is. Someone mistyped the commas. Patch sent.
>

Ah right, nice catch. Thanks for the patch.

-- 
Regards,
Sudeep

  reply	other threads:[~2015-12-15  9:50 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-04 13:38 [PATCH 1/3] arm64: dts: r8a7795: Add PSCI node Dirk Behme
2015-12-04 13:38 ` Dirk Behme
2015-12-04 13:38 ` Dirk Behme
2015-12-04 13:38 ` [PATCH 2/3] arm64: dts: r8a7795: Add Cortex-A57 CPU cores Dirk Behme
2015-12-04 13:38   ` Dirk Behme
2015-12-04 13:38   ` Dirk Behme
2015-12-11 23:07   ` Simon Horman
2015-12-11 23:07     ` Simon Horman
2015-12-11 23:07     ` Simon Horman
2015-12-15  5:12     ` Simon Horman
2015-12-15  5:12       ` Simon Horman
2015-12-15  5:12       ` Simon Horman
2015-12-15  6:31       ` Dirk Behme
2015-12-15  6:31         ` Dirk Behme
2015-12-15  6:31         ` Dirk Behme
2015-12-15  8:23       ` Geert Uytterhoeven
2015-12-15  8:23         ` Geert Uytterhoeven
2015-12-15  8:23         ` Geert Uytterhoeven
2015-12-15  8:38     ` Magnus Damm
2015-12-15  8:38       ` Magnus Damm
2015-12-15  8:38       ` Magnus Damm
2015-12-15  8:48       ` Dirk Behme
2015-12-15  8:48         ` Dirk Behme
2015-12-15  8:48         ` Dirk Behme
2015-12-15  8:55         ` Magnus Damm
2015-12-15  8:55           ` Magnus Damm
2015-12-15  8:55           ` Magnus Damm
2016-03-16  1:19         ` Simon Horman
2016-03-16  1:19           ` Simon Horman
2016-03-16  1:19           ` Simon Horman
2016-03-16  3:45           ` Khiem Nguyen
2016-03-16  3:45             ` Khiem Nguyen
2016-03-16  3:45             ` Khiem Nguyen
2016-03-16  5:36             ` Simon Horman
2016-03-16  5:36               ` Simon Horman
2016-03-16  5:36               ` Simon Horman
2015-12-04 13:38 ` [PATCH 3/3] arm64: dts: r8a7795: Add pmu device nodes Dirk Behme
2015-12-04 13:38   ` Dirk Behme
2015-12-04 13:38   ` Dirk Behme
2015-12-11 23:07   ` Simon Horman
2015-12-11 23:07     ` Simon Horman
2015-12-11 23:07     ` Simon Horman
2015-12-14 14:36   ` Sudeep Holla
2015-12-14 14:36     ` Sudeep Holla
2015-12-14 14:36     ` Sudeep Holla
2015-12-15  8:33     ` Geert Uytterhoeven
2015-12-15  8:33       ` Geert Uytterhoeven
2015-12-15  8:33       ` Geert Uytterhoeven
2015-12-15  9:50       ` Sudeep Holla [this message]
2015-12-15  9:50         ` Sudeep Holla
2015-12-15  9:50         ` Sudeep Holla
2015-12-11 23:07 ` [PATCH 1/3] arm64: dts: r8a7795: Add PSCI node Simon Horman
2015-12-11 23:07   ` Simon Horman
2015-12-11 23:07   ` Simon Horman

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=566FE268.70802@arm.com \
    --to=sudeep.holla@arm.com \
    --cc=linux-arm-kernel@lists.infradead.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.