All of lore.kernel.org
 help / color / mirror / Atom feed
From: Geert Uytterhoeven <geert@linux-m68k.org>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/8] devicetree: binding: R-car Gen3 CMT0 and CMT1 bindings
Date: Fri, 09 Sep 2016 12:07:28 +0000	[thread overview]
Message-ID: <CAMuHMdVAuyWUid6GEJXNoyw4bH3h=QWmo7eMLyiCayc5UQaoFA@mail.gmail.com> (raw)
In-Reply-To: <1473421394-9745-2-git-send-email-bd-phuc@jinso.co.jp>

Hi Phuc-san,

On Fri, Sep 9, 2016 at 1:43 PM,  <bd-phuc@jinso.co.jp> wrote:
> Add documentation for new separate CMT0 and CMT1 DT compatible strings
> for R-Car Gen3.
>
> Signed-off-by: Bui Duc Phuc <bd-phuc@jinso.co.jp>
> ---
>  Documentation/devicetree/bindings/timer/renesas,cmt.txt | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> index 1a05c1b..72fd526 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> +++ b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> @@ -52,6 +52,8 @@ Required Properties:
>                 (CMT[01] on r8a73a4, r8a7790 and r8a7791)
>                 This is a fallback for the renesas,cmt-48-r8a73a4,
>                 renesas,cmt-48-r8a7790 and renesas,cmt-48-r8a7791 entries.
> +    - "renesas,cmt-48-gen3" for third generation 48-bit CMT
> +               (CMT[01] on r8a7795 and r8a7796)

I think the plan was to get rid of the renesas,cmt-48-* bindings, as they do
not allow to differentiate between CMT0 and CMT1, which have different feature
sets.

Cfr. Magnus' series "clocksource: sh_cmt: DT binding rework V4"
(https://lkml.org/lkml/2016/3/14/433).

Magnus: What's the status of your series?

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

WARNING: multiple messages have this Message-ID
From: Geert Uytterhoeven <geert@linux-m68k.org>
To: PhucBui <bd-phuc@jinso.co.jp>, Magnus Damm <magnus.damm@gmail.com>
Cc: "Daniel Lezcano" <daniel.lezcano@linaro.org>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"Linux-sh list" <linux-sh@vger.kernel.org>,
	"Geert Uytterhoeven" <geert+renesas@glider.be>,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
	linux-arm-kernel@vger.kernel.org,
	"Laurent Pinchart" <laurent.pinchart+renesas@ideasonboard.com>,
	"Simon Horman" <horms@verge.net.au>,
	"Kuninori Morimoto" <kuninori.morimoto.gx@renesas.com>,
	"Yoshihiro Shimoda" <yoshihiro.shimoda.uh@renesas.com>,
	"Ryusuke Sakato" <ryusuke.sakato.bx@renesas.com>,
	稲吉 <h-inayoshi@jinso.co.jp>,
	"Cao Minh Hiep" <cm-hiep@jinso.co.jp>
Subject: Re: [PATCH 1/8] devicetree: binding: R-car Gen3 CMT0 and CMT1 bindings
Date: Fri, 9 Sep 2016 14:07:28 +0200	[thread overview]
Message-ID: <CAMuHMdVAuyWUid6GEJXNoyw4bH3h=QWmo7eMLyiCayc5UQaoFA@mail.gmail.com> (raw)
In-Reply-To: <1473421394-9745-2-git-send-email-bd-phuc@jinso.co.jp>

Hi Phuc-san,

On Fri, Sep 9, 2016 at 1:43 PM,  <bd-phuc@jinso.co.jp> wrote:
> Add documentation for new separate CMT0 and CMT1 DT compatible strings
> for R-Car Gen3.
>
> Signed-off-by: Bui Duc Phuc <bd-phuc@jinso.co.jp>
> ---
>  Documentation/devicetree/bindings/timer/renesas,cmt.txt | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> index 1a05c1b..72fd526 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> +++ b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> @@ -52,6 +52,8 @@ Required Properties:
>                 (CMT[01] on r8a73a4, r8a7790 and r8a7791)
>                 This is a fallback for the renesas,cmt-48-r8a73a4,
>                 renesas,cmt-48-r8a7790 and renesas,cmt-48-r8a7791 entries.
> +    - "renesas,cmt-48-gen3" for third generation 48-bit CMT
> +               (CMT[01] on r8a7795 and r8a7796)

I think the plan was to get rid of the renesas,cmt-48-* bindings, as they do
not allow to differentiate between CMT0 and CMT1, which have different feature
sets.

Cfr. Magnus' series "clocksource: sh_cmt: DT binding rework V4"
(https://lkml.org/lkml/2016/3/14/433).

Magnus: What's the status of your series?

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

WARNING: multiple messages have this Message-ID
From: Geert Uytterhoeven <geert@linux-m68k.org>
To: PhucBui <bd-phuc@jinso.co.jp>, Magnus Damm <magnus.damm@gmail.com>
Cc: "Daniel Lezcano" <daniel.lezcano@linaro.org>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"Linux-sh list" <linux-sh@vger.kernel.org>,
	"Geert Uytterhoeven" <geert+renesas@glider.be>,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
	linux-arm-kernel@vger.kernel.org,
	"Laurent Pinchart" <laurent.pinchart+renesas@ideasonboard.com>,
	"Simon Horman" <horms@verge.net.au>,
	"Kuninori Morimoto" <kuninori.morimoto.gx@renesas.com>,
	"Yoshihiro Shimoda" <yoshihiro.shimoda.uh@renesas.com>,
	"Ryusuke Sakato" <ryusuke.sakato.bx@renesas.com>,
	稲吉 <h-inayoshi@jinso.co.jp>,
	"Cao Minh Hiep" <cm-hiep@jinso.co.jp>,
	Dung:人ソ <nv-dung@jinso.co.jp>
Subject: Re: [PATCH 1/8] devicetree: binding: R-car Gen3 CMT0 and CMT1 bindings
Date: Fri, 9 Sep 2016 14:07:28 +0200	[thread overview]
Message-ID: <CAMuHMdVAuyWUid6GEJXNoyw4bH3h=QWmo7eMLyiCayc5UQaoFA@mail.gmail.com> (raw)
In-Reply-To: <1473421394-9745-2-git-send-email-bd-phuc@jinso.co.jp>

Hi Phuc-san,

On Fri, Sep 9, 2016 at 1:43 PM,  <bd-phuc@jinso.co.jp> wrote:
> Add documentation for new separate CMT0 and CMT1 DT compatible strings
> for R-Car Gen3.
>
> Signed-off-by: Bui Duc Phuc <bd-phuc@jinso.co.jp>
> ---
>  Documentation/devicetree/bindings/timer/renesas,cmt.txt | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> index 1a05c1b..72fd526 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> +++ b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> @@ -52,6 +52,8 @@ Required Properties:
>                 (CMT[01] on r8a73a4, r8a7790 and r8a7791)
>                 This is a fallback for the renesas,cmt-48-r8a73a4,
>                 renesas,cmt-48-r8a7790 and renesas,cmt-48-r8a7791 entries.
> +    - "renesas,cmt-48-gen3" for third generation 48-bit CMT
> +               (CMT[01] on r8a7795 and r8a7796)

I think the plan was to get rid of the renesas,cmt-48-* bindings, as they do
not allow to differentiate between CMT0 and CMT1, which have different feature
sets.

Cfr. Magnus' series "clocksource: sh_cmt: DT binding rework V4"
(https://lkml.org/lkml/2016/3/14/433).

Magnus: What's the status of your series?

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

WARNING: multiple messages have this Message-ID
From: geert@linux-m68k.org (Geert Uytterhoeven)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/8] devicetree: binding: R-car Gen3 CMT0 and CMT1 bindings
Date: Fri, 9 Sep 2016 14:07:28 +0200	[thread overview]
Message-ID: <CAMuHMdVAuyWUid6GEJXNoyw4bH3h=QWmo7eMLyiCayc5UQaoFA@mail.gmail.com> (raw)
In-Reply-To: <1473421394-9745-2-git-send-email-bd-phuc@jinso.co.jp>

Hi Phuc-san,

On Fri, Sep 9, 2016 at 1:43 PM,  <bd-phuc@jinso.co.jp> wrote:
> Add documentation for new separate CMT0 and CMT1 DT compatible strings
> for R-Car Gen3.
>
> Signed-off-by: Bui Duc Phuc <bd-phuc@jinso.co.jp>
> ---
>  Documentation/devicetree/bindings/timer/renesas,cmt.txt | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> index 1a05c1b..72fd526 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> +++ b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> @@ -52,6 +52,8 @@ Required Properties:
>                 (CMT[01] on r8a73a4, r8a7790 and r8a7791)
>                 This is a fallback for the renesas,cmt-48-r8a73a4,
>                 renesas,cmt-48-r8a7790 and renesas,cmt-48-r8a7791 entries.
> +    - "renesas,cmt-48-gen3" for third generation 48-bit CMT
> +               (CMT[01] on r8a7795 and r8a7796)

I think the plan was to get rid of the renesas,cmt-48-* bindings, as they do
not allow to differentiate between CMT0 and CMT1, which have different feature
sets.

Cfr. Magnus' series "clocksource: sh_cmt: DT binding rework V4"
(https://lkml.org/lkml/2016/3/14/433).

Magnus: What's the status of your series?

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

  reply	other threads:[~2016-09-09 12:07 UTC|newest]

Thread overview: 76+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-09 11:43 [PATCH 0/8] clocksource: sh_cmt: Add R-car Gen3 CMT0/1 support bd-phuc
2016-09-09 11:43 ` bd-phuc at jinso.co.jp
2016-09-09 11:43 ` bd-phuc
2016-09-09 11:43 ` bd-phuc-HEF513clHfp3+QwDJ9on6Q
2016-09-09 11:43 ` [PATCH 1/8] devicetree: binding: R-car Gen3 CMT0 and CMT1 bindings bd-phuc
2016-09-09 11:43   ` bd-phuc at jinso.co.jp
2016-09-09 11:43   ` bd-phuc
2016-09-09 11:43   ` bd-phuc-HEF513clHfp3+QwDJ9on6Q
2016-09-09 12:07   ` Geert Uytterhoeven [this message]
2016-09-09 12:07     ` Geert Uytterhoeven
2016-09-09 12:07     ` Geert Uytterhoeven
2016-09-09 12:07     ` Geert Uytterhoeven
2016-09-12 15:12     ` Bui Duc Phuc
2016-09-12 15:12       ` Bui Duc Phuc
2016-09-12 15:12       ` Bui Duc Phuc
2016-09-12 15:12       ` Bui Duc Phuc
2016-09-09 11:43 ` [PATCH 2/8] ARM64: dts: r8a7795: Add CMT device to DT bd-phuc
2016-09-09 11:43   ` bd-phuc at jinso.co.jp
2016-09-09 11:43   ` bd-phuc
2016-09-09 11:43   ` bd-phuc-HEF513clHfp3+QwDJ9on6Q
2016-09-09 11:43 ` [PATCH 3/8] ARM64: dts: r8a7796: " bd-phuc
2016-09-09 11:43   ` bd-phuc at jinso.co.jp
2016-09-09 11:43   ` bd-phuc
2016-09-09 11:43   ` bd-phuc-HEF513clHfp3+QwDJ9on6Q
2016-09-09 11:43 ` [PATCH 4/8] clk: renesas: r8a7795: Add CMT clocks bd-phuc
2016-09-09 11:43   ` bd-phuc at jinso.co.jp
2016-09-09 11:43   ` bd-phuc
2016-09-09 11:43   ` bd-phuc-HEF513clHfp3+QwDJ9on6Q
2016-09-12  8:31   ` Geert Uytterhoeven
2016-09-12  8:31     ` Geert Uytterhoeven
2016-09-12  8:31     ` Geert Uytterhoeven
2016-09-12  8:31     ` Geert Uytterhoeven
2016-09-12 14:57     ` Bui Duc Phuc
2016-09-12 14:57       ` Bui Duc Phuc
2016-09-12 14:57       ` Bui Duc Phuc
2016-09-12 14:57       ` Bui Duc Phuc
2016-09-09 11:43 ` [PATCH 5/8] clk: renesas: r8a7796: " bd-phuc
2016-09-09 11:43   ` bd-phuc at jinso.co.jp
2016-09-09 11:43   ` bd-phuc
2016-09-09 11:43   ` bd-phuc-HEF513clHfp3+QwDJ9on6Q
2016-09-12  8:35   ` Geert Uytterhoeven
2016-09-12  8:35     ` Geert Uytterhoeven
2016-09-12  8:35     ` Geert Uytterhoeven
2016-09-12  8:35     ` Geert Uytterhoeven
2016-09-12 15:00     ` Bui Duc Phuc
2016-09-12 15:00       ` Bui Duc Phuc
2016-09-12 15:00       ` Bui Duc Phuc
2016-09-12 15:00       ` Bui Duc Phuc
2016-09-09 11:43 ` [PATCH 6/8] clocksource: sh_cmt: Support separate R-car Gen3 CMT0/1 bd-phuc
2016-09-09 11:43   ` bd-phuc at jinso.co.jp
2016-09-09 11:43   ` bd-phuc
2016-09-09 11:43   ` bd-phuc-HEF513clHfp3+QwDJ9on6Q
2016-09-09 12:12   ` Laurent Pinchart
2016-09-09 12:12     ` Laurent Pinchart
2016-09-09 12:12     ` Laurent Pinchart
2016-09-09 11:43 ` [PATCH 7/8] clocksource: Kconfig: Modify CMT config support 64bit bd-phuc
2016-09-09 11:43   ` bd-phuc at jinso.co.jp
2016-09-09 11:43   ` bd-phuc
2016-09-09 11:43   ` bd-phuc-HEF513clHfp3+QwDJ9on6Q
2016-09-09 11:47   ` Sergei Shtylyov
2016-09-09 11:47     ` Sergei Shtylyov
2016-09-09 11:47     ` Sergei Shtylyov
2016-09-09 11:47     ` Sergei Shtylyov
2016-09-09  3:06     ` Bui Duc Phuc
2016-09-09  3:06       ` Bui Duc Phuc
2016-09-09  3:06       ` Bui Duc Phuc
2016-09-09 12:08   ` Laurent Pinchart
2016-09-09 12:08     ` Laurent Pinchart
2016-09-09 12:08     ` Laurent Pinchart
2016-09-12  0:25     ` Bui Duc Phuc
2016-09-12  0:25       ` Bui Duc Phuc
2016-09-12  0:25       ` Bui Duc Phuc
2016-09-09 11:43 ` [PATCH 8/8] ARM64: defconfig: Enable SH_TIMER_CMT config option bd-phuc
2016-09-09 11:43   ` bd-phuc at jinso.co.jp
2016-09-09 11:43   ` bd-phuc
2016-09-09 11:43   ` bd-phuc-HEF513clHfp3+QwDJ9on6Q

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='CAMuHMdVAuyWUid6GEJXNoyw4bH3h=QWmo7eMLyiCayc5UQaoFA@mail.gmail.com' \
    --to=geert@linux-m68k.org \
    --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.